Skip to content

chore: Deduplicate yarn.lock - #6706

Open
antonis wants to merge 1 commit into
mainfrom
chore/dedupe-yarn-lock
Open

chore: Deduplicate yarn.lock#6706
antonis wants to merge 1 commit into
mainfrom
chore/dedupe-yarn-lock

Conversation

@antonis

@antonis antonis commented Sep 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Chore (internal-only hygiene)

📜 Description

Runs yarn dedupe to collapse 569 redundant transitive dependency copies onto already-resolved higher versions. yarn dedupe --check now reports no further duplicates.

Only yarn.lock is modified (−5709 / +769 lines).

💡 Motivation and Context

yarn dedupe --check previously reported hundreds of transitive packages that could be collapsed onto a single higher version already present in the tree. Deduping:

  • Shrinks the dependency graph — 569 duplicate package instances removed, so a smaller node_modules and less disk churn.
  • Faster, lighter installs for contributors and CI (fewer fetch/link operations, smaller lockfile to resolve).
  • Reduces version skew — fewer copies of the same package at slightly different patch versions in the tree.

Safety (verified in #6690):

  • yarn.lock is not shipped in the published npm tarball — consumers resolve their own tree from @sentry/react-native's package.json ranges.
  • All affected packages are dev/build tooling (@babel/*, es-abstract, get-intrinsic, metro-config, etc.). No runtime @sentry/* dependency is affected.
  • Every version move stays within the same semver major — versions already permitted by existing ^/~ ranges that a clean-cache install would land on anyway.

Closes #6690

💚 How did you test it?

  • yarn dedupe --check → no packages can be deduped
  • yarn build → passes
  • yarn test → passes
  • cd packages/core && yarn api-report:check → API report up to date (no public API drift)
  • Idempotency: re-running yarn install leaves yarn.lock byte-identical and dedupe --check still clean — the deduped state is stable.
  • Shipped output is unchanged: built the full SDK (build:sdk → tsc, downlevel, build:tools) on main and on this branch and SHA-256'd all 752 compiled files (.js / .d.ts / .map) — byte-for-byte identical. No @sentry/* runtime dep moved; the SDK's compiler stays pinned at TypeScript 5.9.3 on both branches.

📝 Checklist

  • I added tests to verify the changes — N/A, lockfile-only change
  • No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
  • I updated the docs if needed — N/A
  • Review from the native team if needed — N/A
  • No breaking change or entry added to the changelog — internal-only, no user-visible change

🔮 Next steps

None.

🤖 Generated with Claude Code

@antonis @claude
chore: Deduplicate yarn.lock
28d34ef 
Run `yarn dedupe` to collapse 569 redundant transitive dependency
copies onto already-resolved higher versions. Lockfile-only hygiene;
all moves stay within the same semver major and no runtime @sentry/*
dependency is affected.

Fixes #6690

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

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).

  • chore: Deduplicate yarn.lock by antonis in #6706
  • chore(ci): Only build production builds for visionOS and tvOS by antonis in #6700
  • chore(deps): update Sentry Android Gradle Plugin to v6.22.0 by github-actions in #6699

🤖 This preview updates automatically when you update the PR.

@antonis antonis added the ready-to-merge Triggers the full CI test suite label Sep 11, 2026
cursor[bot]
cursor Bot reviewed Sep 11, 2026
View reviewed changes

@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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 28d34ef. Configure here.

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 3845.11 ms 1237.18 ms -2607.93 ms
Size 5.15 MiB 6.89 MiB 1.74 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
7a89652+dirty 3861.46 ms 1229.61 ms -2631.85 ms
eb55864+dirty 3869.04 ms 1241.75 ms -2627.29 ms
5b7e8a7+dirty 3851.22 ms 1223.79 ms -2627.43 ms
5fe1c6c+dirty 1220.79 ms 1217.63 ms -3.16 ms
fa21fca+dirty 3840.43 ms 1224.42 ms -2616.02 ms
580fb5c+dirty 3836.13 ms 1218.72 ms -2617.41 ms
94af3bd+dirty 3849.30 ms 1235.15 ms -2614.15 ms
a2585ce+dirty 3822.57 ms 1221.30 ms -2601.27 ms
f3215d3+dirty 3842.73 ms 1219.33 ms -2623.40 ms
a216cb9+dirty 3839.33 ms 1225.26 ms -2614.07 ms

App size

Revision Plain With Sentry Diff
7a89652+dirty 5.15 MiB 6.70 MiB 1.55 MiB
eb55864+dirty 5.15 MiB 6.88 MiB 1.73 MiB
5b7e8a7+dirty 5.15 MiB 6.70 MiB 1.54 MiB
5fe1c6c+dirty 3.38 MiB 4.77 MiB 1.39 MiB
fa21fca+dirty 4.98 MiB 6.55 MiB 1.57 MiB
580fb5c+dirty 4.98 MiB 6.46 MiB 1.48 MiB
94af3bd+dirty 5.15 MiB 6.69 MiB 1.53 MiB
a2585ce+dirty 4.98 MiB 6.55 MiB 1.57 MiB
f3215d3+dirty 5.15 MiB 6.67 MiB 1.52 MiB
a216cb9+dirty 4.98 MiB 6.51 MiB 1.53 MiB

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 507.18 ms 570.14 ms 62.96 ms
Size 50.56 MiB 56.49 MiB 5.93 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
a2585ce+dirty 426.36 ms 483.26 ms 56.90 ms
882f8ae+dirty 399.98 ms 427.06 ms 27.08 ms
af33f3b+dirty 434.90 ms 506.14 ms 71.24 ms
ecf47a2+dirty 420.40 ms 458.02 ms 37.62 ms
acd838e+dirty 422.63 ms 462.39 ms 39.76 ms
7fd0012+dirty 444.73 ms 470.08 ms 25.35 ms
580fb5c+dirty 436.34 ms 471.63 ms 35.28 ms
5ca03f9+dirty 423.30 ms 467.00 ms 43.70 ms
5c1e987+dirty 423.52 ms 471.64 ms 48.12 ms
822d35b+dirty 429.31 ms 498.04 ms 68.73 ms

App size

Revision Plain With Sentry Diff
a2585ce+dirty 49.74 MiB 55.36 MiB 5.61 MiB
882f8ae+dirty 48.30 MiB 53.60 MiB 5.29 MiB
af33f3b+dirty 49.74 MiB 55.09 MiB 5.35 MiB
ecf47a2+dirty 49.74 MiB 54.82 MiB 5.07 MiB
acd838e+dirty 48.30 MiB 53.60 MiB 5.30 MiB
7fd0012+dirty 50.56 MiB 56.46 MiB 5.90 MiB
580fb5c+dirty 49.74 MiB 54.79 MiB 5.05 MiB
5ca03f9+dirty 49.74 MiB 55.26 MiB 5.52 MiB
5c1e987+dirty 43.75 MiB 48.08 MiB 4.33 MiB
822d35b+dirty 49.74 MiB 54.84 MiB 5.10 MiB

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 3879.65 ms 1241.57 ms -2638.08 ms
Size 5.15 MiB 6.89 MiB 1.74 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
5fe1c6c+dirty 1201.36 ms 1209.15 ms 7.78 ms
7c6200e+dirty 3828.16 ms 1217.20 ms -2610.95 ms
41d6254+dirty 3849.78 ms 1233.91 ms -2615.86 ms
5789645+dirty 3841.36 ms 1214.81 ms -2626.55 ms
a9df5ce+dirty 3856.02 ms 1225.22 ms -2630.80 ms
9c84b9a+dirty 3833.41 ms 1221.79 ms -2611.62 ms
1a2e7e0+dirty 3852.95 ms 1233.78 ms -2619.18 ms
a636fa4+dirty 3846.16 ms 1210.10 ms -2636.05 ms
9b5615c+dirty 3837.21 ms 1215.33 ms -2621.89 ms
3d31fcf+dirty 3857.46 ms 1237.17 ms -2620.29 ms

App size

Revision Plain With Sentry Diff
5fe1c6c+dirty 3.38 MiB 4.77 MiB 1.39 MiB
7c6200e+dirty 5.08 MiB 6.68 MiB 1.60 MiB
41d6254+dirty 5.15 MiB 6.70 MiB 1.54 MiB
5789645+dirty 4.98 MiB 6.50 MiB 1.52 MiB
a9df5ce+dirty 4.98 MiB 6.55 MiB 1.57 MiB
9c84b9a+dirty 4.98 MiB 6.55 MiB 1.57 MiB
1a2e7e0+dirty 4.98 MiB 6.46 MiB 1.49 MiB
a636fa4+dirty 4.98 MiB 6.55 MiB 1.57 MiB
9b5615c+dirty 4.98 MiB 6.51 MiB 1.53 MiB
3d31fcf+dirty 4.98 MiB 6.56 MiB 1.58 MiB

@antonis
antonis marked this pull request as ready for review September 11, 2026 13:49
@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 428.64 ms 456.34 ms 27.70 ms
Size 50.56 MiB 56.49 MiB 5.93 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
ae37560+dirty 428.96 ms 456.86 ms 27.90 ms
d7d54c6+dirty 423.54 ms 498.56 ms 75.02 ms
5ca03f9+dirty 412.83 ms 459.40 ms 46.57 ms
44abcc2+dirty 554.18 ms 578.85 ms 24.66 ms
8929511+dirty 469.49 ms 502.65 ms 33.16 ms
0a147b2+dirty 442.80 ms 522.24 ms 79.44 ms
09a902f+dirty 423.02 ms 472.18 ms 49.16 ms
18a8255+dirty 410.50 ms 444.35 ms 33.85 ms
eb93136+dirty 500.37 ms 532.58 ms 32.21 ms
890d145+dirty 486.42 ms 514.85 ms 28.43 ms

App size

Revision Plain With Sentry Diff
ae37560+dirty 48.30 MiB 53.60 MiB 5.29 MiB
d7d54c6+dirty 49.74 MiB 55.09 MiB 5.34 MiB
5ca03f9+dirty 49.74 MiB 55.26 MiB 5.52 MiB
44abcc2+dirty 49.74 MiB 55.36 MiB 5.62 MiB
8929511+dirty 43.94 MiB 49.02 MiB 5.08 MiB
0a147b2+dirty 49.74 MiB 55.08 MiB 5.34 MiB
09a902f+dirty 49.74 MiB 54.81 MiB 5.07 MiB
18a8255+dirty 49.74 MiB 55.38 MiB 5.64 MiB
eb93136+dirty 48.30 MiB 53.58 MiB 5.28 MiB
890d145+dirty 43.94 MiB 49.00 MiB 5.06 MiB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursor cursor[bot] cursor[bot] left review comments
@alwx alwx Awaiting requested review from alwx alwx is a code owner
@lucas-zimerman lucas-zimerman Awaiting requested review from lucas-zimerman lucas-zimerman is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

ready-to-merge Triggers the full CI test suite

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

chore: Deduplicate yarn.lock to collapse redundant transitive dependency copies

1 participant

@antonis