Remix SDK invariant is not captured as Issue

[moishinetzer]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which package are you using?

@sentry/remix

SDK Version

7.5.0

Framework Version

7.5.0

Link to Sentry event

https://sentry.io/organizations/netzerorg/performance/trace/7a8e8c962aa242aabb565917ff0c891c/

Steps to Reproduce

import invariant from "tiny-invariant";

export const loader: LoaderFunction = async ({ params }) => {
  const user = params["*"];
  invariant(user, "user is required");
  const user= await getUser({ user: user});
  return json({
    user,
  });
};

Now, navigate to a route where the user does not exist.

Expected Result

Invariant is triggered and thrown and an issue is created in sentry dashboard

Actual Result

Invariant is triggered and thrown, however sentry only receives a transaction and doesn't flag it as an issue.

[moishinetzer]

Note: As a sidepoint I am also getting every 10 seconds an actual issue with the following information:

Sentry event: issues/3402471265 (Same org as specified in issue)

/myapp/node_modules/@remix-run/server-runtime/data.js

Object.captureException
{"size":0}

76.   try {
77.    result = await loader({
78.      request: stripDataParam(stripIndexParam(request)),
79.      context: loadContext,
80.      params: match.params
81.    });
82.  } catch (error) {
83.    if (!responses.isResponse(error)) {
84.      throw error; 

[AbhiPrasad]

Hey thanks for writing in. @onurtemizkan mind taking a look when you get some time to help triage this?

[moishinetzer]

My project is now seeing several issues. feel free to reach out to me or look around on my account to see them. None of them have been caused by actual user related issues and seem to be the SDK.

[AbhiPrasad]

There's some RangeError: Maximum call stack size exceeded which we should look at.

RangeError: Maximum call stack size exceeded
  File "/myapp/node_modules/@sentry/src/misc.ts", line 143, col 1, in Object.addExceptionMechanism
  File "/myapp/node_modules/@sentry/remix/cjs/utils/instrumentServer.js", line 45, col 17, in <anonymous>
  File "/myapp/node_modules/@sentry/src/scope.ts", line 515, col 1, in <anonymous>
  File "/myapp/node_modules/@sentry/src/syncpromise.ts", line 58, col 1, in new SyncPromise
  File "/myapp/node_modules/@sentry/src/scope.ts", line 510, col 1, in Scope._notifyEventProcessors
  File "/myapp/node_modules/@sentry/src/scope.ts", line 527, col 1, in <anonymous>
  File "/myapp/node_modules/@sentry/src/syncpromise.ts", line 58, col 1, in new SyncPromise
  File "/myapp/node_modules/@sentry/src/scope.ts", line 510, col 1, in Scope._notifyEventProcessors
  File "/myapp/node_modules/@sentry/src/scope.ts", line 527, col 1, in <anonymous>
  File "/myapp/node_modules/@sentry/src/syncpromise.ts", line 58, col 1, in new SyncPromise

There seems to be problems with adding the event processor, but not sure what's causing it 🤔