chore: standardize repository config#328
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
❌ Deploy failed
📋 Build log (last lines)🤖 Powered by surge-preview |
|||||||||
|
Warning Review limit reached
More reviews will be available in 40 minutes and 34 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?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 credits. 🚦 How do rate 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 see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
Walkthrough文档构建输出改为 Changes文档构建与内容迁移
仓库自动化与赞助信息
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Suggested reviewers
Poem
🚥 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 |
|
React Doctor found no new issues. 🎉 Reviewed by React Doctor for commit |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #328 +/- ##
=======================================
Coverage 99.39% 99.39%
=======================================
Files 2 2
Lines 164 164
Branches 40 42 +2
=======================================
Hits 163 163
Misses 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
![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 migrates the package name from rc-segmented to @rc-component/segmented, updating aliases, demo imports, and configuration files accordingly. It also updates the documentation, adds Vercel deployment configurations, and refines build scripts. The review feedback highlights potential TypeScript resolution issues in tsconfig.json: first, missing path mappings for the newly added subpath aliases (/es and /assets), and second, the risk of breaking type-checking for temporary files by removing .dumi/**/* from the include array.
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.
![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 (2)
tsconfig.json (1)
16-23: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win补齐
paths里的子路径映射。Line 19-23 现在只映射了根包，但
.dumirc.tsLine 18-20 已经同时支持了@rc-component/segmented/es和@rc-component/segmented/assets。这样会让 dumi 能解析、而 tsc/IDE 不能解析这些子路径，配置出现分叉。建议补齐映射
"paths": { "`@/`*": ["src/*"], "@@/*": [".dumi/tmp/*"], - "`@rc-component/segmented`": ["src/index.tsx"] + "`@rc-component/segmented`": ["src/index.tsx"], + "`@rc-component/segmented/es/`*": ["src/*"], + "`@rc-component/segmented/assets/`*": ["assets/*"] }🤖 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 `@tsconfig.json` around lines 16 - 23, The tsconfig paths mapping is incomplete for the package subpaths used elsewhere, so tsc and IDE resolution diverge from dumi. Update the paths section in tsconfig.json to include the same subpath aliases already supported in .dumirc.ts for `@rc-component/segmented/es` and `@rc-component/segmented/assets`, alongside the existing `@rc-component/segmented` entry, so all tooling resolves the package consistently..github/workflows/surge-preview.yml (1)
35-35: 🚀 Performance & Scalability | 🔵 Trivial | 💤 Low value建议使用
npm ci以保证可复现的依赖安装。预览构建属于 CI 流程,
npm ci会严格依据 lockfile 安装,更稳定可复现。🤖 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 @.github/workflows/surge-preview.yml at line 35, Replace the dependency installation step in the Surge preview workflow from npm install to npm ci so the CI build uses the lockfile and remains reproducible; update the install command in the workflow job that runs the preview build.
🤖 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 @.github/workflows/react-doctor.yml:
- Around line 6-7: `react-doctor.yml` already restricts `push` to `master`, but
`react-component-ci.yml` still runs on every `push`, so align the workflow
trigger strategy by updating `on` in `react-component-ci.yml` to explicitly
filter `push` to the intended default branch, or leave it unchanged only if
full-branch execution is intentional. Use the workflow trigger block in
`react-component-ci.yml` as the primary place to make this consistency decision.
In `@README.md`:
- Around line 55-67: The README props table currently shows `options` with a
default value of `[]`, but `SegmentedProps` in `src/index.tsx` defines `options`
as required, so update the documentation entry for `options` to remove the
default and make it clear it must be provided.
---
Nitpick comments:
In @.github/workflows/surge-preview.yml:
- Line 35: Replace the dependency installation step in the Surge preview
workflow from npm install to npm ci so the CI build uses the lockfile and
remains reproducible; update the install command in the workflow job that runs
the preview build.
In `@tsconfig.json`:
- Around line 16-23: The tsconfig paths mapping is incomplete for the package
subpaths used elsewhere, so tsc and IDE resolution diverge from dumi. Update the
paths section in tsconfig.json to include the same subpath aliases already
supported in .dumirc.ts for `@rc-component/segmented/es` and
`@rc-component/segmented/assets`, alongside the existing `@rc-component/segmented`
entry, so all tooling resolves the package consistently.
🪄 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: de7695f9-ac33-4e01-9692-b3059caa2e2c
📒 Files selected for processing (22)
.dumirc.ts.github/FUNDING.yml.github/workflows/codeql.yml.github/workflows/main.yml.github/workflows/preview.yml.github/workflows/react-component-ci.yml.github/workflows/react-doctor.yml.github/workflows/surge-preview.yml.gitignoreREADME.mddocs/demo/basic.tsxdocs/demo/controlled.tsxdocs/demo/custom-render.tsxdocs/demo/dynamic.tsxdocs/demo/html-title.tsxdocs/demo/name.tsxdocs/demo/refs.tsxdocs/demo/rtl.tsxdocs/index.mdpackage.jsontsconfig.jsonvercel.json
💤 Files with no reviewable changes (2)
- .github/workflows/main.yml
- .github/workflows/preview.yml
|
Deployment failed with the following error: Learn More: https://vercel.com/afc163s-projects?upgradeToPro=build-rate-limit |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
1 participant
Summary
@rc-component/segmented, including badges, install, usage, API, development, and release notes.rc-segmenteddemo imports with the scoped package alias and remove the legacy preview workflow.Test Plan
npx prettier --check --ignore-unknown README.md .dumirc.ts tsconfig.json package.json .github/dependabot.yml .github/workflows/codeql.yml .github/workflows/react-component-ci.yml .github/workflows/react-doctor.yml .github/workflows/surge-preview.yml docs/index.md docs/demo/*.tsx vercel.jsonnpm run lintnpm run tscnpm test -- --runInBandnpm run compilenpm run buildnpx vercel build --yesSummary by CodeRabbit
新功能
docs-dist，在线预览与部署体验更稳定。文档
维护
Refs ant-design/ant-design#58514