Remix request handler build could be a function

[topaxi]

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 SDK are you using?

@sentry/remix

SDK Version

8.x

Framework Version

Remix 2

Link to Sentry event

No response

SDK Setup

No response

Steps to Reproduce

createRequestHandler({
  build // is typed as: ServerBuild | (() => Promise<ServerBuild>)
})

See:

https://github.com/remix-run/remix/blob/2e65318b15df407b31b101362e03729e61ec5622/packages/remix-express/server.ts#L40

But Sentry is always assuming that it is an object:

sentry-javascript/packages/remix/src/utils/instrumentServer.ts

Line 553 in 257bcb0

const wrappedEntry = { ...build.entry, module: { ...build.entry.module } };

Expected Result

Sentry should honor upstream interfaces

Actual Result

Sentry does not honor upstream interfaces

[onurtemizkan]

Thanks for reporting this @topaxi.

A fix for this is going to be part of an extensive update on Remix SDK soon.

[AbhiPrasad]

We've released the changes with https://github.com/getsentry/sentry-javascript/releases/tag/8.10.0. To start using it, add autoInstrumentRemix: true to your server-side Sentry.init call!

https://docs.sentry.io/platforms/javascript/guides/remix/manual-setup/#server-side-configuration