⬆️ typescript 7 - #5
Conversation
Bumps typescript from ~6.0.2 to ~7.0.2, the native (Go) compiler port. TS 7 ships platform-specific binaries, so the lockfile gains @typescript/typescript-* entries for every supported os/cpu pair. Typecheck, lint, fmt, test, build, and fallow audit all pass unchanged; no source or tsconfig changes were needed. Co-Authored-By: Claude <noreply@anthropic.com>
📝 WalkthroughWalkthroughThe package manifest updates the TypeScript development dependency from ChangesTypeScript version update
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@package.json`:
- Line 36: Update the TypeScript dependency entry in package.json to a version
within `@voidzero-dev/vite-plus-core`’s supported ^5.0.0 || ^6.0.0 peer range, or
coordinate the bump with a Vite Plus integration update that supports TypeScript
7; do not leave the project on TypeScript ~7.0.2 with the current Vite Plus
dependency.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 1f3799fe-3229-45aa-b786-2220de5eb7fe
⛔ Files ignored due to path filters (1)
bun.lockis excluded by!**/*.lock
📒 Files selected for processing (1)
package.json
| "@vanilla-extract/vite-plugin": "^5.2.2", | ||
| "fallow": "^2.66.1", | ||
| "typescript": "~6.0.2", | ||
| "typescript": "~7.0.2", |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
rg -n --hidden \
--glob '!node_modules/**' \
--glob '!dist/**' \
--glob '!build/**' \
-e "from ['\"]typescript['\"]" \
-e "require\\(['\"]typescript['\"]\\)" \
-e '"(`@typescript-eslint/`|typescript-eslint)' \
.Repository: Vilos92/vilos92.com
Length of output: 157
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
printf '\n## package.json\n'
cat -n package.json | sed -n '1,140p'
printf '\n## search for TypeScript imports/usages\n'
rg -n --hidden \
--glob '!node_modules/**' \
--glob '!dist/**' \
--glob '!build/**' \
-e "from ['\"]typescript['\"]" \
-e "require\\(['\"]typescript['\"]\\)" \
-e "import\\(['\"]typescript['\"]\\)" \
-e "\\btypescript\\b" \
.
printf '\n## other package manifests\n'
fd -a 'package.json|pnpm-lock.yaml|bun.lock|yarn.lock|package-lock.json' .
printf '\n## tsconfig files\n'
fd -a 'tsconfig*.json' .
printf '\n## TypeScript-related scripts in package.json\n'
python3 - <<'PY'
import json
with open('package.json') as f:
pkg = json.load(f)
for k,v in pkg.get('scripts', {}).items():
if 'tsc' in v or 'typescript' in v or 'vp' in v:
print(f"{k}: {v}")
PYRepository: Vilos92/vilos92.com
Length of output: 11497
Update the TypeScript bump to match the Vite Plus peer range. package.json:36 is now outside @voidzero-dev/vite-plus-core’s supported typescript: ^5.0.0 || ^6.0.0 range, so this should move together with a Vite/TS integration update or stay on TS 6 for now.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@package.json` at line 36, Update the TypeScript dependency entry in
package.json to a version within `@voidzero-dev/vite-plus-core`’s supported ^5.0.0
|| ^6.0.0 peer range, or coordinate the bump with a Vite Plus integration update
that supports TypeScript 7; do not leave the project on TypeScript ~7.0.2 with
the current Vite Plus dependency.
Bumps
typescriptfrom~6.0.2to~7.0.2— the native (Go) compiler port.Notes
tsconfig.jsonchanges were needed. The existing strict config (includingerasableSyntaxOnly,verbatimModuleSyntax, andmoduleResolution: bundler) compiles clean under TS 7.bun.lockgains@typescript/typescript-*entries for each supported os/cpu pair — includinglinux-x64, which is what CI resolves.~range rather than bun's default^, matching this repo's existing convention of pinning TypeScript to a single minor.Verification
All CI jobs were run locally against this branch:
bun run typecheckbun run lintbun run fmt:checkbun run testbun run buildbun run fallow:audit🤖 Generated with Claude Code
Summary by CodeRabbit