chore: standardize repository maintenance#81
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 26 minutes and 44 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 (12)
Walkthrough更新了文档站点基础配置、README 内容和多项 GitHub Actions 工作流，并同步调整了构建输出、发布脚本与相关部署配置。 Changes站点发布与自动化配置
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #81 +/- ##
=======================================
Coverage 98.09% 98.09%
=======================================
Files 11 11
Lines 421 421
Branches 121 120 -1
=======================================
Hits 413 413
Misses 7 7
Partials 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 updates the project's build, documentation, and deployment configurations, including transitioning the documentation output directory to docs-dist, adding a Vercel configuration, and modernizing the README. It also updates the TypeScript path mappings for the dumi temporary directory. The feedback points out that excluding the .dumi directory in tsconfig.json will prevent TypeScript from resolving the generated type definitions under .dumi/tmp/*, and suggests removing it from the exclusion list.
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)
.github/workflows/cloudflare-pages-preview.yml (1)
26-41: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value重复的密钥校验条件可合并为作业级 guard。
Install dependencies、Build site、Deploy preview三步重复了完全相同的三元条件表达式，Skip步骤则是其取反。后续若新增配置项（如再加一个变量），需要同步修改 4 处，易遗漏。建议在job级别用outputs或单独的判断步骤集中处理，例如：♻️ 建议的重构方向
preview: runs-on: ubuntu-latest env: CLOUDFLARE_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }} CLOUDFLARE_ACCOUNT_ID: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }} CLOUDFLARE_PAGES_PROJECT: ${{ vars.CLOUDFLARE_PAGES_PROJECT }} PREVIEW: true steps: - uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 with: persist-credentials: false + - name: Check configuration + id: cfg + run: | + if [ -n "$CLOUDFLARE_API_TOKEN" ] && [ -n "$CLOUDFLARE_ACCOUNT_ID" ] && [ -n "$CLOUDFLARE_PAGES_PROJECT" ]; then + echo "enabled=true" >> "$GITHUB_OUTPUT" + else + echo "Cloudflare Pages preview is not configured; skip deployment." + echo "enabled=false" >> "$GITHUB_OUTPUT" + fi - name: Install dependencies - if: ${{ env.CLOUDFLARE_API_TOKEN != '' && env.CLOUDFLARE_ACCOUNT_ID != '' && env.CLOUDFLARE_PAGES_PROJECT != '' }} + if: ${{ steps.cfg.outputs.enabled == 'true' }} run: npm install🤖 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/cloudflare-pages-preview.yml around lines 26 - 41, The Cloudflare Pages preview workflow repeats the same secret-check condition across multiple steps, while the skip step uses the inverse, making the logic hard to maintain. Consolidate the configuration check into a single job-level guard or one dedicated condition step in cloudflare-pages-preview.yml, then have Install dependencies, Build site, and Deploy preview reuse that shared result instead of duplicating the expression..github/workflows/codeql.yml (1)
3-9: 🚀 Performance & Scalability | 🔵 Trivial | 💤 Low value建议补充并发控制以取消过期运行。
本 PR 为其它诊断工作流（如
react-doctor.yml）添加了concurrency控制，但 CodeQL 工作流缺少同样配置。在频繁推送时会并行跑多次分析，浪费 runner 资源。建议添加：♻️ 建议补充
on: push: branches: ['master'] pull_request: branches: ['master'] schedule: - cron: '24 18 * * 1' + +concurrency: + group: codeql-${{ github.event.pull_request.number || github.ref }} + cancel-in-progress: true🤖 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/codeql.yml around lines 3 - 9, CodeQL workflow is missing the same concurrency cancellation behavior used elsewhere, so repeated pushes can leave stale runs executing in parallel. Update the workflow definition in the CodeQL configuration to add a concurrency group that keys off the workflow/ref context and enables cancellation of in-progress older runs, using the existing workflow structure near the trigger configuration.
🤖 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-component-ci.yml:
- Around line 7-8: The reusable workflow invocation in react-component-ci is
over-sharing secrets and should be tightened. In the workflow call that uses
test-utoo.yml, remove secrets: inherit and pass only the CODECOV_TOKEN
explicitly, since that is the only secret the downstream workflow needs. Also
update the workflow reference from `@main` to a specific commit SHA if possible to
reduce supply-chain risk; if SHA pinning is not available yet, still ensure the
secret scope is limited in the call site.
In `@README.md`:
- Around line 130-150: The Release section in README is inaccurate: `npm run
prepublishOnly` is only a pre-publish hook and does not perform the actual
release with `@rc-component/np`. Update the release instructions to point to the
real publish/release command used by the project (such as the
`release`/`publish` script or invoking `@rc-component/np` directly) and keep the
Development command list unchanged.
---
Nitpick comments:
In @.github/workflows/cloudflare-pages-preview.yml:
- Around line 26-41: The Cloudflare Pages preview workflow repeats the same
secret-check condition across multiple steps, while the skip step uses the
inverse, making the logic hard to maintain. Consolidate the configuration check
into a single job-level guard or one dedicated condition step in
cloudflare-pages-preview.yml, then have Install dependencies, Build site, and
Deploy preview reuse that shared result instead of duplicating the expression.
In @.github/workflows/codeql.yml:
- Around line 3-9: CodeQL workflow is missing the same concurrency cancellation
behavior used elsewhere, so repeated pushes can leave stale runs executing in
parallel. Update the workflow definition in the CodeQL configuration to add a
concurrency group that keys off the workflow/ref context and enables
cancellation of in-progress older runs, using the existing workflow structure
near the trigger configuration.
🪄 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: 3d81783f-fdee-4a12-b328-620c6fc25aaf
📒 Files selected for processing (14)
.dumirc.ts.github/FUNDING.yml.github/workflows/cloudflare-pages-preview.yml.github/workflows/codeql.yml.github/workflows/main.yml.github/workflows/react-component-ci.yml.github/workflows/react-doctor.yml.github/workflows/surge-preview.yml.gitignoreREADME.mdnow.jsonpackage.jsontsconfig.jsonvercel.json
💤 Files with no reviewable changes (2)
- now.json
- .github/workflows/main.yml
|
Deployment failed with the following error: Learn More: https://vercel.com/afc163s-projects?upgradeToPro=build-rate-limit |
|
Deployment failed with the following error: Learn More: https://vercel.com/react-component?upgradeToPro=build-rate-limit |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
1 participant
Summary
Standardize this rc-component repository as part of the Ant Design rc-component maintenance sweep.
Tracking issue: ant-design/ant-design#58514
Scope
types: "./es/index.d.ts", publishConfig, and release flow through@rc-component/np.react-component/rc-test/.github/workflows/test-utoo.yml@mainworkflow, React Doctor, Codecov, CodeQL, updated GitHub Actions versions, and guarded Surge preview fallback.docs-distoutput and remove legacynow-build/ Cloudflare Pages residue.Notes
secrets: inheritis kept untilreact-component/rc-test#176is merged, then it can be narrowed to explicitCODECOV_TOKENforwarding.