chore: update maintenance dependencies#445
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
Next review available in: 22 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
Walkthrough更新了开发工具链配置、类型兼容声明和 TypeScript 编译设置，并将 README 顶部徽标改为可跳转链接，同时为 Dependabot 更新项新增了分组规则。 Changes工具链升级与类型兼容
Sequence Diagram(s)预估代码审查工作量🎯 3 (Moderate) | ⏱️ ~20 minutes 可能相关的 PR
诗
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ 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 |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the project dependencies to React 19, TypeScript 6, and ESLint 9, introduces a flat ESLint configuration, groups Dependabot updates, and adds type declarations for React 19 compatibility. Feedback on these changes highlights critical type-safety concerns: disabling strict TypeScript checks in tsconfig.json reduces type safety, ambient module augmentation of react and react-dom in react-compat.d.ts can cause global type conflicts for library consumers, and manually declaring Jest globals as any in global.d.ts overrides strongly-typed definitions from @types/jest.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
✅ Preview is ready!
↩️ Previous: ⚡️ 🤖 Powered by surge-preview |
|||||||||||||||
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
global.d.ts (1)
49-56: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win不要把测试全局重新声明成
any。前面已经引入了 Jest 类型，这几行会把
describe/it/expect等全局的精确签名全部抹掉，测试侧类型检查基本失效。直接删掉这组声明更稳。建议修改
-declare const describe: any; -declare const it: any; -declare const test: any; -declare const beforeEach: any; -declare const afterEach: any; -declare const beforeAll: any; -declare const afterAll: any; -declare const expect: any;🤖 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 `@global.d.ts` around lines 49 - 56, Remove the redundant test global declarations in global.d.ts that re-declare describe, it, test, beforeEach, afterEach, beforeAll, afterAll, and expect as any, since they override the Jest typings already in use. Delete this block entirely so the existing precise Jest signatures remain available to the test code and type checking stays intact.
🤖 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 `@react-compat.d.ts`:
- Line 5: Remove the React 19 compatibility backfills from react-compat.d.ts:
delete the extra useRef() no-arg overload and the react-dom/hydrate declaration
so the type system no longer reintroduces removed legacy APIs. Keep the
remaining compat declarations only if they are still needed, and let callers
migrate to the standard React 19 patterns such as useRef(null | undefined) and
hydrateRoot.
- Around line 14-15: The react-dom compatibility declaration is reintroducing
the removed hydrate API and should be deleted from react-compat.d.ts. Remove the
hydrate declaration from the declare module 'react-dom' block so outdated calls
stop type-checking, and leave the React 19-compatible path through hydrateRoot
as the supported entry point.
---
Nitpick comments:
In `@global.d.ts`:
- Around line 49-56: Remove the redundant test global declarations in
global.d.ts that re-declare describe, it, test, beforeEach, afterEach,
beforeAll, afterAll, and expect as any, since they override the Jest typings
already in use. Delete this block entirely so the existing precise Jest
signatures remain available to the test code and type checking stays intact.
🪄 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: CHILL
Plan: Pro
Run ID: 33bc2219-7cde-4920-8110-c879fdfe8e54
📒 Files selected for processing (8)
.github/dependabot.ymlREADME.mdREADME.zh-CN.mdeslint.config.mjsglobal.d.tspackage.jsonreact-compat.d.tstsconfig.json
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #445 +/- ##
=======================================
Coverage 99.24% 99.24%
=======================================
Files 5 5
Lines 132 132
Branches 46 47 +1
=======================================
Hits 131 131
Misses 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
React Doctor skipped this pull request — it changed no React files. Reviewed by React Doctor for commit |
|
Deployment failed with the following error: Learn More: https://vercel.com/react-component?upgradeToPro=build-rate-limit |
|
Related to ant-design/ant-design#58514. |
1 participant
Summary
Test Plan
Summary by CodeRabbit