Skip to content

test(tanstackstart-react): Initialize Sentry from a dedicated client init file - #21397

Merged
nicohrubec merged 1 commit into
developfrom
nh/external-file-client-init
Jun 9, 2026
Merged

test(tanstackstart-react): Initialize Sentry from a dedicated client init file#21397
nicohrubec merged 1 commit into
developfrom
nh/external-file-client-init

Conversation

@nicohrubec

@nicohrubecnicohrubec commented Jun 9, 2026

Copy link
Copy Markdown
Member

Changes the setup to a dedicated init file that is imported first to ensure that Sentry init is run first. This guarantees Sentry.init() runs before any other import's side effects.

Fixes#21088

@nicohrubec
nicohrubec marked this pull request as draft June 9, 2026 11:56
@nicohrubec
nicohrubecforce-pushed the nh/external-file-client-init branch from b7f16f7 to b64c282CompareJune 9, 2026 12:12
…init file
Move Sentry.init() into a dedicated file imported first in the client entry so
it runs before any other import's side effects (ESM evaluates imports before the
module body). Adds a regression test for an error thrown in an imported module
before hydration.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec
nicohrubecforce-pushed the nh/external-file-client-init branch from b64c282 to 777773aCompareJune 9, 2026 12:17
@nicohrubec
nicohrubec marked this pull request as ready for review June 9, 2026 12:19
@nicohrubec
nicohrubec merged commit f52787f into developJun 9, 2026
46 checks passed
@nicohrubec
nicohrubec deleted the nh/external-file-client-init branch June 9, 2026 12:42
nicohrubec added a commit to getsentry/sentry-docs that referenced this pull request Jun 9, 2026
…init file (#18324)
Moves the client `Sentry.init()` into a dedicated
`src/instrument.client.ts` file that is imported first in the client
entry point. Importing it first ensures Sentry initializes before any
other import's side effects (and before hydration).
SDK PR: getsentry/sentry-javascript#21397
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sentry client initialization should be in client entry point

2 participants

@nicohrubec@andreiborza