getsentry / sentry-javascript

Official Sentry SDKs for JavaScript
https://sentry.io
MIT License
7.97k stars 1.57k forks source link

instrument.server.mjs not included after automatic configuration #14143

Open enure opened 1 day ago

enure commented 1 day ago

Is there an existing issue for this?

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/browser

SDK Version

latest

Framework Version

Remix 8.35.0

Link to Sentry event

No response

Reproduction Example/SDK Setup

No response

Steps to Reproduce

  1. Use "Automatic Configuration" command

Expected Result

The changes would include the following code included in the manual setup documentation:

NODE_OPTIONS='--import=./instrument.server.mjs' remix-serve build
# or
NODE_OPTIONS='--require=./instrument.server.cjs' remix-serve build

or...

The "Automatic Configuration" docs would say if you should include these changes in your project.

It's not clear to me if I need to prefix my build commands with the code above, but it looks like I should.

Actual Result

From what I can tell, the instrument.server.mjs file is not imported/used if you only use the automatic configuration.

chargome commented 22 hours ago

Hi @enure are you trying to set up our remix sdk using the wizard?