Skip to content

fix(core): Improve native init log message clarity#5801

Merged
antonis merged 1 commit into
mainfrom
antonis/auto-init-log
Mar 11, 2026
Merged

fix(core): Improve native init log message clarity#5801
antonis merged 1 commit into
mainfrom
antonis/auto-init-log

Conversation

@antonis
Copy link
Copy Markdown
Contributor

@antonis antonis commented Mar 11, 2026

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Improves the console.info message shown when sentry.options.json is detected and automatic native initialization is skipped from JS.

Before:

Initializing Sentry JS with the options file. Expecting manual native initialization before JS. Native will not be initialized automatically.

After:

[Sentry] Using options file. Native SDK is expected to be initialized before JS, skipping automatic native initialization from JS.

💡 Motivation and Context

Fixes #5798

The previous message sounded like a warning that something was broken, causing users to think native init wasn't working. The new message clarifies this is the expected flow when using useNativeInit: true with the Expo plugin.

💚 How did you test it?

  • Existing tests in sdk.test.ts pass (initialization from sentry.options.json suite)
  • Reproduced locally with a fresh Expo project using useNativeInit: true

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 11, 2026

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


  • fix(core): Improve native init log message clarity by antonis in #5801
  • chore(deps): update CLI to v3.3.2 by github-actions in #5799
  • chore(deps): update Maestro to v2.3.0 by github-actions in #5790
  • chore(deps): update JavaScript SDK to v10.43.0 by github-actions in #5791
  • chore(deps): update CLI to v3.3.1 by github-actions in #5793
  • fix(core): Defer initial navigation span creation until navigation container is registered by antonis in #5789
  • Ref(Sample): Add existing bundled playgroundscreen into sample app by lucas-zimerman in #5778
  • Auto tracing for expo-image and expo-asset by alwx in #5718
  • Expo Updates: add tags with Expo Updates context vars to make them searchable/filterable by alwx in #5788
  • chore(deps): bump actions/setup-node from 6.2.0 to 6.3.0 by dependabot in #5784
  • chore(deps): bump github/codeql-action from 4.32.4 to 4.32.6 by dependabot in #5781
  • chore(deps): bump getsentry/craft/.github/workflows/changelog-preview.yml from 2.23.1 to 2.23.2 by dependabot in #5782
  • chore(deps): bump getsentry/craft from 2.21.7 to 2.23.2 by dependabot in #5783
  • chore(deps): bump tar to ^7.5.10 by antonis in #5777

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 11, 2026

Fails
🚫 Pull request is not ready for merge, please add the "ready-to-merge" label to the pull request

Generated by 🚫 dangerJS against 720b318

@antonis antonis marked this pull request as ready for review March 11, 2026 09:44
Copy link
Copy Markdown
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@antonis antonis merged commit caa79c2 into main Mar 11, 2026
61 of 70 checks passed
@antonis antonis deleted the antonis/auto-init-log branch March 11, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Native init not working?

2 participants