Repository navigation
chore: move TypeScript to 7.1.0-beta - #424
Conversation
|
@coderabbitai full review |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
bompus has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the
Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title "chore: move TypeScript to 7.1.0-beta" states the main change: the typescript dev dependency moves from 7.0.2 to 7.1.0-beta. It is short, specific, and matches the raw summary and objectives… |
| Description check | |
The description covers the change, its scope, and the verification steps. It follows the template's structure and gives reasons for the CHANGELOG decision. The template checklist items are present. Th… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
| Suppressions Explained | |
PASS: The pull request changes only package.json and bun.lock to update the TypeScript version. Added lines contain no lint, type-check, formatter suppression, or ignore directive. |
| Rule Changes Documented | |
The pull request changes only package.json and bun.lock. It does not change rules/core.md, a file under rules/modifiers/, or any skills/*/SKILL.md. It also does not add, remove, or rename a … |
Full details: Title check
-
Explanation
The title "chore: move TypeScript to 7.1.0-beta" states the main change: the typescript dev dependency moves from 7.0.2 to 7.1.0-beta. It is short, specific, and matches the raw summary and objectives.
Full details: Description check
-
Explanation
The description covers the change, its scope, and the verification steps. It follows the template's structure and gives reasons for the CHANGELOG decision. The template checklist items are present. The README Development checks item is checked, and the CHANGELOG item is intentionally unchecked with a stated reason. The description is mostly complete, so the check passes.
Full details: Rule Changes Documented
-
Explanation
The pull request changes only
package.jsonandbun.lock. It does not changerules/core.md, a file underrules/modifiers/, or anyskills/*/SKILL.md. It also does not add, remove, or rename a modifier or skill. The documentation conditions do not apply.
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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 @coderabbitai help to get the list of available commands.
|
|
@coderabbitai full review |
✅ Action performedFull review finished. |
|
@coderabbitai full review |
e9543ff to
7512627
Compare
|
|
@cubic review |
@bompus cubic can't start this review because your workspace has reached its free monthly review limit. cubic has reviewed 40,438 of the 40,000 allowed lines of code this month. Reviews resume on 11 November 2026 (in 19 days). Paid plans include much higher monthly review limits. Upgrade now to resume reviews. To help optimise your usage, you can tune cubic to get the most out of your usage limits:
|
|
@coderabbitai full review |
✅ Action performedFull review finished. |
Moves the
typescriptdev dependency from 7.0.2 to the exact pin 7.1.0-beta and updatesbun.lock. Nothing else changes:tsconfigtarget and lib, Bun's built-in checker andoxlint-tsgolintare untouched.Neither
npm run typecheck(Bun's built-in checker) nor oxlint's type-aware rules (the bundledtsgolintbinary) read this package, so the pin affects only thetscbinary innode_modules.tsc --noEmit -p tsconfig.check.jsonpasses on 7.1.0-beta.Checks on this branch, all passing:
npm run checknode --test test/*.test.mjs,bun test test/,bun test skills/npm run check:package -- --bunCHANGELOG.mdentry: a dev-dependency bump is not user-visible.Summary by cubic
Moves the
typescriptdev dependency from 7.0.2 to the exact pin 7.1.0-beta and updatesbun.lock. This only affects thetscbinary innode_modules— the tsconfig, Bun's built-in type checker, and oxlint's type-aware rules are untouched, andtsc --noEmit -p tsconfig.check.jsonpasses on 7.1.0-beta.Written for commit 7512627. Summary will update on new commits.
Summary by CodeRabbit