chore(deps): update 18 dependencies within their current major - #110
Merged
Conversation
React 19.2.4->19.2.8, Vite 8.0->8.2, Vitest 4.1.2->4.1.10, Primer, TanStack Virtual, papaparse, fflate, prettier, typescript-eslint and the React type packages. Targets are the newest release older than 7 days, matching the min-release-age policy configured on this machine rather than overriding it, so a version published hours ago can't land here. Where that holds a package one patch back the reason is recorded in the PR rather than silently pinned. eslint-plugin-react-hooks is deliberately held at 7.0.1. 7.1.x adds react-hooks/set-state-in-effect, which fails on three existing effects in App.tsx, InsightsSidebar.tsx and usePageNavigation.ts. Those are real cascading-render issues, but fixing them is a React restructure that needs its own browser verification, and usePageNavigation.ts was just changed. It does not belong inside a routine dependency bump. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3cf40db0-fafb-4793-92c6-3fe2b92d70bd
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.
Stacked on #109 (needs that
.npmrcfirst, otherwisenpm installrewrites the lockfile back to the corporate mirror).React 19.2.4 → 19.2.8, Vite 8.0 → 8.2, Vitest 4.1.2 → 4.1.10, plus Primer, TanStack Virtual, papaparse, fflate, prettier, typescript-eslint and the React type packages. All within their existing major.
On version selection
This machine has
min-release-age=7configured globally — a supply-chain control that refuses any version published in the last 7 days, which is the window that matters for a compromised-package attack. I targeted the newest release older than 7 days rather than overriding the policy. Six packages are therefore one patch behind npmjslatest, deliberately:@primer/react@tanstack/react-virtual@testing-library/user-event@vitejs/plugin-react@vitest/coverage-v8viteThey'll be eligible in a few days. Recording it here so nobody reads those as arbitrary pins.
One deliberate hold
eslint-plugin-react-hooksstays at 7.0.1. 7.1.x addsreact-hooks/set-state-in-effect, which fails on three existing effects:The rule is correct — these are genuine cascading-render patterns. But they're all "synchronize two pieces of state" effects, and fixing them properly is a React restructure that needs browser re-verification, not a
--fix.usePageNavigation.tsin particular was just changed in #106. Bundling a behavioral refactor into a routine dependency bump is how regressions ship unnoticed, so it gets its own PR.Verification
Lint clean, typecheck clean, 286 tests pass, build succeeds, coverage unchanged at 88.24 / 73.3 / 88.11 / 90.85. Lockfile still resolves entirely to npmjs.org.
Not included
Nine majors are held for separate evaluation:
@tanstack/react-table8→9,highcharts12→13,eslint9→10,jsdom29→30,typescript5.9→7.0, and others. TypeScript 7 is the Go-native compiler rewrite and is nowhere near a drive-by bump.