Skip to content

bump(sentry-javascript): Bump Sentry JavaScript SDK to 10.60.0#1303

Merged
lucas-zimerman merged 4 commits into
mainfrom
lz/bump/js-10.60.0
Jun 26, 2026
Merged

bump(sentry-javascript): Bump Sentry JavaScript SDK to 10.60.0#1303
lucas-zimerman merged 4 commits into
mainfrom
lz/bump/js-10.60.0

Conversation

@lucas-zimerman

@lucas-zimerman lucas-zimerman commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

  • Bumps @sentry/{angular,react,vue,browser,core,types} from 10.52.0 to 10.60.0 across root and sample apps (ionic-angular-v7, ionic-angular-v8, ionic-vue3).
  • Renames @sentry-internal/{eslint-config-sdk,eslint-plugin-sdk,typescript} to @sentry/{eslint-config-sdk,eslint-plugin-sdk,typescript} at 10.60.0 (renamed upstream in #21371).
  • Bumps typescript from 5.0.4 to 5.8.3 (required by @sentry/typescript peer dep).
  • Overrides sendDefaultPii in BaseCapacitorOptions without deprecation, since native SDKs still rely on it.
  • Adds dataCollection to BaseCapacitorOptions with @experimental annotation — only userInfo is bridged to native as sendDefaultPii.
  • Fixes tsconfig.build.json / tsconfig.esm.json extends paths and adds moduleResolution: "node" override for CJS build.
  • Updates .eslintrc.js plugin/config references from @sentry-internal/sdk to @sentry/sdk.
  • Removes stale @ts-expect-error directives in src/vendor/fromByteArray.ts, replaced with non-null assertions.
  • Adds braces resolution to work around a Yarn 1 hoisting bug with micromatch.

💡 Motivation and Context

Keep the Sentry JavaScript SDK up to date. The @sentry-internal/* packages were renamed to @sentry/* in 10.54.0 and stopped being published under the old scope after 10.57.0, so this bump also handles the migration. The sendDefaultPii option was deprecated upstream in favor of dataCollection, but native SDKs don't support dataCollection yet, so we override it as non-deprecated for Capacitor users and bridge dataCollection.userInfo to the native layer.

💚 How did you test it?

  • yarn build passes (lint + ESM + CJS)
  • yarn test passes (233 tests)

📝 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

  • Verify sample apps build with updated deps on CI
  • Verify native iOS/Android builds are unaffected
  • Add full native dataCollection support once Sentry Cocoa and Android SDKs support it
  • Close Angular 22 Support #1302

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Comment thread src/integrations/sdkinfo.ts
Comment thread src/integrations/sdkinfo.ts Outdated
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Yarn 1 generated a corrupt lockfile entry for micromatch@4.0.8 without
its dependencies, causing CI to fail with "Cannot find module 'braces'".

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 7380168. Configure here.

Comment thread src/integrations/sdkinfo.ts
Comment thread src/nativeOptions.ts
@lucas-zimerman lucas-zimerman requested a review from antonis June 25, 2026 01:26

@antonis antonis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!
Thank you for also raising the dataCollection issue. Opened a PR on the RN side with an alternative approach to this getsentry/sentry-react-native#6345 but the current approach of marking it as experimental also resonates with me

@lucas-zimerman lucas-zimerman merged commit 66ef098 into main Jun 26, 2026
23 checks passed
@lucas-zimerman lucas-zimerman deleted the lz/bump/js-10.60.0 branch June 26, 2026 01:29
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.

Angular 22 Support

2 participants