Skip to content

chore(deps): hold typescript on 6.x for the dashboard (Next 16 + TS7 breaks build)#724

Open
aaronjmars wants to merge 1 commit into
mainfrom
chore/dependabot-hold-typescript
Open

chore(deps): hold typescript on 6.x for the dashboard (Next 16 + TS7 breaks build)#724
aaronjmars wants to merge 1 commit into
mainfrom
chore/dependabot-hold-typescript

Conversation

@aaronjmars

Copy link
Copy Markdown
Collaborator

Why

Dependabot PR #720 grouped three dev-dep bumps for /apps/dashboard, one of which was typescript 6.0.3 → 7.0.2. TypeScript 7 is the native/Go port, which ships a different package layout that Next.js 16.2.10's build-time TypeScript integration doesn't recognize yet.

Verified with an A/B build (everything else held constant):

typescript tsc --noEmit next build
7.0.2 (PR #720) ✅ passes fails — Next can't detect TS7, tries to reinstall it, then crashes The "id" argument must be of type string. Received undefined
6.0.3 (pre-PR) ✅ passes succeeds — full route manifest

tsc --noEmit alone is misleading here because the standalone compiler works fine; only Next's build pipeline breaks.

Change

Adds a Dependabot ignore on the /apps/dashboard npm entry for typescript version-update:semver-major. This:

Revisit and drop the ignore once Next.js ships TS7 support.

Follow-up

Once this lands, #720 can be closed/recreated so the tsx 4.23.1 + @types/node 26.1.1 patches come through without the typescript major.

🤖 Generated with Claude Code

TypeScript 7 (the native/Go port) ships a package layout that Next.js 16's
build-time TS integration doesn't recognize yet — `next build` crashes with
"The 'id' argument must be of type string" even though `tsc --noEmit` passes.
Verified by A/B build: TS 6.0.3 builds clean, TS 7.0.2 fails.

Add a Dependabot ignore for typescript version-update:semver-major on the
/apps/dashboard npm entry so the grouped dev-deps PR (#720) stops re-proposing
the breaking major; minor/patch bumps within 6.x still flow through.

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant