Skip to content

Repository navigation

chore(deps): update dependency typescript to v7 #26

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
renovate wants to merge 1 commit into
base: main
Choose a base branch
from
Open

chore(deps): update dependency typescript to v7 #26

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
"eslint-plugin-react-hooks": "^7.1.1",
"globals": "^17.12.0",
"jsdom": "^30.1.0",
"typescript": "^6.0.3",
"typescript": "^7.0.2",

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

@henry-nsheaps henry-nsheaps Bot Oct 13, 2026 •
edited
Loading

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚨 P0 — Blocking peer-dep conflict; npm install still fails after 2026-10-17 rebase

Renovate rebased this PR again today (new head 22b8947, 2026-10-17T10:05:13Z) and refreshed its artifact-update log (new timestamp 2026-10-17T10:05:21Z). Nothing changed on either side — main still pins typescript-eslint@^8.70.0, upstream still has not published a release whose TS peer range includes 7.x, and Renovate's refreshed ERESOLVE comment reproduces the same failure byte-for-byte:

npm warn Could not resolve dependency:
npm warn peer typescript@">=4.8.4 <6.1.0" from @typescript-eslint/eslint-plugin@8.70.0
npm warn node_modules/@typescript-eslint/eslint-plugin
npm warn   @typescript-eslint/eslint-plugin@"8.70.0" from typescript-eslint@8.70.0
npm warn   node_modules/typescript-eslint
...
npm warn Conflicting peer dependency: typescript@6.0.3

Every @typescript-eslint/*@8.70.0 subpackage (eslint-plugin, parser, type-utils, typescript-estree, project-service, tsconfig-utils, utils) still declares peer typescript: ">=4.8.4 <6.1.0" per the same Renovate log. Bumping typescript to ^7.0.2 violates that range, so npm cannot regenerate package-lock.json. Upstream shipped typescript-eslint#12529 "add warning when TS 7 is detected" in 8.65.0 (now ~3 months ago) and still has not widened the peer range through 8.70.0 — the linter package explicitly refuses TS 7.

Because no updated package-lock.json accompanies this package.json change, merging as-is will break npm ci on main for every CI job that installs deps (lint, typecheck, test, build, e2e). Build-blocker, not just a typecheck concern.

Recommended options (any one unblocks):

  1. Preferred — hold this PR until typescript-eslint publishes a release whose peer range includes TS 7, then bump both together. A Renovate packageRules grouping in the shared nsheaps/renovate-config (e.g. matchPackageNames: ["typescript", "typescript-eslint", "@typescript-eslint/**"], groupName: "typescript") would prevent this class of broken PR going forward — and avoid the churn of re-reviewing the same byte-identical diff on every rebase (this is now the 10th such review).
  2. Close this PR and add a temporary allowedVersions: "<7" entry for typescript in the shared Renovate config until the ecosystem catches up.
  3. Only if you accept the ecosystem-compat risk: extend this PR to also bump typescript-eslint to a version whose peer range covers TS 7, then re-run Renovate so the lockfile actually regenerates. Do not patch around this with --legacy-peer-deps — that hides the fact that the linter's TS API assumptions may not hold under a TS major bump.

Per CLAUDE.md (TypeScript strict — no any, no @ts-ignore, line 34), silently softening strictness to squeeze past a TS 7 breaking change is also not an option.

Edit (2026-10-17): Rebased from db10281 → 22b8947 with no substantive change. Base typescript-eslint still ^8.70.0; upstream has not shipped a release widening the TS peer range. Original ERESOLVE still reproduces, as above. Preserving this comment as the canonical live finding on package.json:54; previous near-identical inline comments remain minimized as outdated.

Findings based on the diff at package.json:54 and package.json:55 on PR head 22b8947 (typescript-eslint: ^8.70.0, matching main), Renovate's refreshed ERESOLVE report (comment 5261781302, updated 2026-10-17T10:05:21Z), the typescript-eslint@8.65.0 CHANGELOG (still the most recent TS-7-relevant upstream note; nothing new through 8.70.0), and renovate.json (extends local>nsheaps/renovate-config, no packageRules).

"typescript-eslint": "^8.70.0",
"vite": "^8.3.0",
"vitest": "^5.0.1"
Expand Down
Loading