[failproofai-536] Pin TypeScript to 6.x — Dependabot ignore for the 7.0 native-port major#538
Conversation
… bump TypeScript 7.0 is the native (Go) port. Its npm package drops the classic compiler API from the main entry (exports["."] resolves to a version stub; the API moves to experimental typescript/unstable/*), which Next.js's build-time TS config validation and typescript-eslint (peer typescript >=4.8.4 <6.1.0) both require. Bumping typescript to 7.x (PR #499) therefore reddens every CI job, since the prepare -> next build step runs on every `bun install`. Tell Dependabot to ignore typescript major-version updates until the toolchain (Next.js, typescript-eslint) supports the native port. Tracked in #536. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Cb18kCZoxH6XcW3SDANVDf
|
Caution Review failedAn error occurred during the review process. Please try again later. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Automated code review started - full review. Results will be posted here. |
|
⏳ Phase 3: Build & verify in progress... No Docker compose found — running native Bun build and test suite. |
|
✅ Build & test complete. Results:
|
🔍 Automated Code Review — PR #538📋 Executive SummaryThis is a surgical, well-scoped PR that pins TypeScript to the 6.x line by adding a Dependabot 📊 Change Architecturegraph TD
A[Dependabot weekly scan<br/>bun ecosystem] -->|"reads new: ignore rule"| B[typescript dep check]
B -->|"semver-major? YES"| C[🟢 SKIP — no PR opened]
B -->|"semver-patch/minor? NO"| D[🔵 Open PR as usual]
A -->|"unchanged"| E[github-actions ecosystem]
style C fill:#90EE90
style B fill:#87CEEB
style A fill:#87CEEB
style E fill:#87CEEB
Legend: 🟢 New behavior | 🔵 Unchanged / existing behavior 🔴 Breaking Changes✅ No breaking changes detected. This is a configuration-only change that:
|
hermes-exosphere
left a comment
There was a problem hiding this comment.
Automated review: Approved. ✅ 2 files, 12 lines — Dependabot ignore rule for TypeScript 7.x. Build passes, all 2064 tests pass, YAML valid. No issues found.
hermes-exosphere
left a comment
There was a problem hiding this comment.
Automated review: Approved. ✅
What
Tell Dependabot to ignore
typescriptmajor-version updates, pinning the repo to the 6.x line.Why
Dependabot #499 bumped
typescript6.0.3 → 7.0.2 and turned every CI job red. TypeScript 7.0 is the native (Go) port — its npm package drops the classic compiler API from the main entry (exports["."]→ a version stub; the API moves to experimentaltypescript/unstable/*).Reproduced on a throwaway branch:
next buildfails even with type-checking off. Next.js 16.2.x runs a "TypeScript config validation" step that needs the classic API.typescript.ignoreBuildErrors: trueskips type-checking but not config validation; underCI=truethe build exits 1. Becausepreparerunsnext buildon everybun install, this reddens all jobs, not justbuild.typescript-eslint@8.64.0(viaeslint-config-next) declares peertypescript@">=4.8.4 <6.1.0"— it refuses 7.x outright, sobun run lintbreaks too.tscbinary works; a real adoption path is@typescript/native-preview(tsgo) as a separate type-check tool while keepingtypescript@6.xfor the ecosystem.A port isn't possible until Next.js and typescript-eslint ship native-port support. Full write-up and revisit criteria in #536.
Scope
.github/dependabot.yml— the ignore rule (+ explanatory comment).CHANGELOG.md—### Dependenciesentry.No source/runtime changes; no docs/README/examples affected.
#499should be closed as un-mergeable. Refs #536.🤖 Generated with Claude Code
https://claude.ai/code/session_01Cb18kCZoxH6XcW3SDANVDf