chore(deps): apply npm group update, hold TypeScript at 5.x - #90
Merged
Merged
Conversation
Dependabot's grouped npm update bumps TypeScript to 7.0.2, which makes
`npm ci` fail with ERESOLVE before any job can run:
npm error peer typescript@">=4.8.4 <6.1.0" from typescript-eslint@8.70.0
npm error Conflicting peer dependency: typescript@6.0.3
typescript-eslint 8.x has not declared TypeScript 7 support, so Lint,
Format, Build and Test all fail at install time.
This applies every other dependency in the group and keeps TypeScript on
5.x, and adds a Dependabot `ignore` entry for `typescript >=7` so the
break is not reproposed each week. Drop that entry once typescript-eslint
declares TypeScript 7 support.
Verified locally: `npm ci`, `lint`, `type-check`, `test`, `build` and
`format:check` all pass.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
vpetersson
approved these changes
Oct 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dependabot PR #87 fails all four CI jobs. Its grouped npm update bumps TypeScript to
7.0.2, andnpm ciaborts before any job body runs:typescript-eslint 8.x has not declared TypeScript 7 support, so Lint, Format, Build and Test all fail at install time rather than on anything in the code.
What this does
ignoreentry fortypescript >=7so the same broken update is not reproposed every week.Remove the
ignoreentry and take TypeScript 7 once typescript-eslint supports it.Worth a look before merging
This group also carries
@sentry/browser10.59.0 → 11.0.0, a major release.src/utils/sentry.tsonly usesinit,setTag,setUser,setContext,captureExceptionand abeforeSendhook, all of which are unchanged in v11, and it setstracesSampleRate: 0so the new span-streaming defaults do not apply. The one thing worth a deliberate decision is v11's more permissive default data collection —beforeSendhere scrubs secrets from settings, but it does not control what the SDK attaches by default. Confirm that is acceptable, or pin@sentry/browserto^10in this PR.It also carries
vitest4 → 5 andglobals13 → 17, both majors. The suite passes on them locally.Verification
npm ci,npm run lint,type-check,test,buildandformat:checkall pass locally on this branch.Supersedes #87.
🤖 Generated with Claude Code