chore: upgrade TypeScript to v7 - #385
Conversation
- Bump typescript from ^5.7.2 to ^7.0.0 - TS 7.0 is a native Go port (Project Corsa), GA July 8 2026; type-checking semantics are unchanged from 6.0 - Verified: tsc --noEmit passes with zero errors, npm run build (Vite + Nitro) succeeds - Biome lint errors (56, pre-existing) and missing test files were confirmed unrelated to this change via git stash comparison against the prior TS version - No programmatic-compiler-API dependents in this stack (Vite uses bundler mode with noEmit:true; Biome is a separate Rust toolchain), so the 7.1 API gap does not apply here Closes AOSSIE-Org#381
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review. WalkthroughThe pull request updates the TypeScript development dependency in ChangesTypeScript Upgrade
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized TypeScript upgrade reports passing type-check and build checks with no actionable merge-blocking risk remaining beyond normal review. Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Link your account with GitcordThanks for opening this PR, @RounakKumarAgarwal! To receive Discord notifications and contributor tracking for this organization:
Once linked, Gitcord can notify you about reviews, merges, and more. — Posted by Gitcord |
|
Hi, I received the Gitcord bot message asking me to join the AOSSIE Discord and verify my GitHub account. However, I'm currently unable to join the Discord because my account isn't recognized as a verified contributor yet. Could someone please help me get verified or let me know how I should proceed with the Gitcord linking process? |
|
@RounakKumarAgarwal , could you share a screenshot of the error you get when you try to join https://discord.gg/hjUhu33uAn ? |
|
@RounakKumarAgarwal Just remove the node modules and package-lock then use npm install again to fix windows build error. ping me once done |
|
@PinJinx yeah bro let me know what to do ? |
|
Exactly what I have said in above message it will fix itself it's usually due to package-lock mismatch |
|
I'm doing wait |
Delete node_modules and package-lock.json and reinstall fresh, per maintainer request (@PinJinx), to resolve the Windows CI failure where npm ci reported missing lru-cache/typescript entries. Also picks up transitive dependency drift in the Oxc/Rolldown toolchain used by Vite (unrelated to TypeScript itself). Verified tsc --noEmit and npm run build both still pass cleanly.
|
Done @PinJinx |
|
Merged, Thanks for the contribution! |

Closes #381
What
Bumps
typescriptfrom^5.7.2to^7.0.0.Why
TS 7.0 (native Go compiler, "Project Corsa") went GA on July 8, 2026 with
typically 8-12x faster full builds. It's a faithful port, not a redesign —
type-checking semantics match 6.0.
Risk assessment for this repo
tsconfig.jsonusesmoduleResolution: bundler+noEmit: true— Vitedoes actual transpilation, tsc is type-check only.
there's no typed-lint plugin depending on the TS compiler API.
nothing in this stack calls into that API directly, so it doesn't apply here.
Verification
npx tsc --noEmit— passes, zero errorsnpm run build(Vite + Nitro) — passesnpm run check(Biome) — 56 pre-existing errors, confirmed unrelated bycomparing against TS 5.7.2 via
git stashnpm test(Vitest) — no test files exist in the repo currently(confirmed pre-existing, unrelated to this change)
Scoped narrowly to the TS version bump only — did not touch the pre-existing
Biome errors or missing tests, to keep this reviewable and separate from the
broader dependency cleanup in #362.
Summary by CodeRabbit