chore(deps): update dependency typescript to v6#1554
Conversation
338ae11 to
f2ab99b
Compare
…bility The baseUrl option is deprecated in TypeScript 6 and no imports depend on it. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 10 minutes and 52 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughTypeScript devDependency bumped from Changes
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 the current code and only fix it if needed.
Inline comments:
In `@package.json`:
- Line 85: The package.json currently pins TypeScript to 6.0.3 but still uses
older `@typescript-eslint` packages; update `@typescript-eslint/parser` and
`@typescript-eslint/eslint-plugin` (or typescript-eslint) to versions ^8.58.0 so
their peer dependency range includes TypeScript 6.x; locate the dependencies for
`@typescript-eslint/parser` and `@typescript-eslint/eslint-plugin` (or
typescript-eslint) in package.json and change their versions to ^8.58.0, then
run npm/yarn install and re-run linting to confirm no peer dependency errors.
🪄 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: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 82f84e67-7cf7-4601-975d-8d2ed922ae8f
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (2)
package.jsontsconfig.json
💤 Files with no reviewable changes (1)
- tsconfig.json
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
Update @typescript-eslint/parser and typescript-eslint from ^8.46.0 to ^8.58.0 to satisfy peer dependency requirements for TypeScript 6.x. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Take @types/node ~22 and engines >=22.11.0 from main, keep @typescript-eslint packages at ^8.58.0 for TypeScript 6 support. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@package.json`:
- Line 85: The package.json currently pins "typescript": "6.0.3" and your
typecheck script fails to find Jest types; update package.json to use a caret
range for TypeScript (change "typescript": "6.0.3" to "typescript": "^6.0.3")
for consistency and either upgrade `@types/jest` to a TypeScript 6-compatible
release (bump "@types/jest" to a compatible version) or adjust tsconfig.json's
"types" (or "typeRoots") so Jest typings are included; locate the "typecheck"
script in package.json and the tsconfig.json "types" array to make the
corresponding change.
🪄 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: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: ff5a532a-f6d0-4116-b8e1-39fa51f64b44
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (1)
package.json
| "tsx": "^4.20.6", | ||
| "typescript": "5.9.3", | ||
| "typescript-eslint": "^8.46.0" | ||
| "typescript": "6.0.3", |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify TypeScript 6.0 build and typecheck compatibility
echo "=== Running typecheck script ==="
npm run typecheck
echo -e "\n=== Running build script ==="
npm run build
echo -e "\n=== Checking for TypeScript compiler errors in strict mode ==="
# Look for common TS 6.0 strictness issues in the flagged files
rg -n "\\[key: string\\]: any" src/vault/interfaces/key.interface.ts
rg -n "as unknown as" src/webhooks/webhooks.ts
rg -n "as any" src/common/utils/runtime-info.spec.tsRepository: workos/workos-node
Length of output: 1122
🏁 Script executed:
cat -n package.json | grep -A 5 -B 5 "jest"Repository: workos/workos-node
Length of output: 1279
🏁 Script executed:
cat -n package.json | sed -n '80,95p'Repository: workos/workos-node
Length of output: 534
🏁 Script executed:
# Check if there's a tsconfig.json with jest types
cat tsconfig.json | grep -A 10 "types"Repository: workos/workos-node
Length of output: 138
🏁 Script executed:
git log --oneline package.json | head -5Repository: workos/workos-node
Length of output: 142
🏁 Script executed:
# Check the previous TypeScript version
git show HEAD~1:package.json | grep -A 2 -B 2 '"typescript"'Repository: workos/workos-node
Length of output: 98
🏁 Script executed:
# Check current git status and recent changes
git statusRepository: workos/workos-node
Length of output: 261
🏁 Script executed:
# Check if there's a git diff showing the changes
git diff HEAD package.json | head -30Repository: workos/workos-node
Length of output: 44
🏁 Script executed:
# Look at the commit that updated TypeScript
git show 3908462:package.json | grep -A 2 -B 2 '"typescript"'Repository: workos/workos-node
Length of output: 175
🏁 Script executed:
# Check what the jest types error actually means
# Try to see if there's a mismatch between jest version and `@types/jest`
npm ls jest `@types/jest`Repository: workos/workos-node
Length of output: 117
🏁 Script executed:
# Check if there are any known issues with TypeScript 6.0 and `@types/jest` 30.0.0
# by looking at the jest configuration in package.json
cat -n package.json | grep -A 20 '"jest"'Repository: workos/workos-node
Length of output: 1434
Address TypeScript 6.0 jest types compatibility issue and version pinning inconsistency.
Two issues to resolve:
-
TypeScript 6.0 + jest types incompatibility: The
typecheckscript fails with "Cannot find type definition file for 'jest'" despite@types/jestbeing listed in devDependencies. This appears to be a breaking change between TypeScript 6.0.3 and@types/jest30.0.0. Consider:- Upgrading
@types/jestto a TypeScript 6.0-compatible version, or - Updating tsconfig.json's types configuration
- Upgrading
-
Version pinning inconsistency: This line uses exact version
6.0.3while all other devDependencies use caret ranges (e.g.,^8.58.0,^30.0.0). Change to^6.0.3for consistency unless exact pinning is intentional.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@package.json` at line 85, The package.json currently pins "typescript":
"6.0.3" and your typecheck script fails to find Jest types; update package.json
to use a caret range for TypeScript (change "typescript": "6.0.3" to
"typescript": "^6.0.3") for consistency and either upgrade `@types/jest` to a
TypeScript 6-compatible release (bump "@types/jest" to a compatible version) or
adjust tsconfig.json's "types" (or "typeRoots") so Jest typings are included;
locate the "typecheck" script in package.json and the tsconfig.json "types"
array to make the corresponding change.
Change "typescript": "6.0.3" to "typescript": "^6.0.3" for consistency with all other devDependencies in package.json. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This PR contains the following updates:
5.9.3→6.0.3Warning
Some dependencies could not be looked up. Check the warning logs for more information.
Release Notes
microsoft/TypeScript (typescript)
v6.0.3Compare Source
v6.0.2Compare Source
Configuration
📅 Schedule: (in timezone UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.
Summary by CodeRabbit