chore: standardize repository tooling#1229
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
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:
Walkthrough新增 React Doctor 和 Surge Preview 工作流，更新现有 CI 引用与仓库配置，并重写 README 的结构、API 和发布信息。 Changes项目自动化与文档更新
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
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 |
![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 README.md with modernized API documentation, usage examples, and development instructions, and updates tsconfig.json to include configuration and source directories. The review feedback suggests updating the README usage example to use the recommended 'options' prop instead of the legacy 'Option' component. Additionally, it points out that including '.fatherrc.js' in tsconfig.json without enabling 'allowJs' will cause a TypeScript compilation error.
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.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1229 +/- ##
==========================================
+ Coverage 99.44% 99.50% +0.05%
==========================================
Files 31 31
Lines 1263 1406 +143
Branches 462 499 +37
==========================================
+ Hits 1256 1399 +143
Misses 7 7 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 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 18-20: The checkout step is currently persisting the GitHub token
into local git config by default, which can expose credentials to later actions
in the same job. Update the actions/checkout@v5 usage in the workflow to
explicitly set persist-credentials to false alongside fetch-depth so the token
is not stored.
- Around line 18-21: The workflow currently uses mutable Action tags and leaves
checkout credentials persisted, so update the actions in the workflow to pinned
commit SHAs for both actions/checkout and millionco/react-doctor, and add
persist-credentials: false under the checkout step. Use the existing
actions/checkout and millionco/react-doctor steps to locate the changes.
In `@README.md`:
- Around line 86-96: The SearchConfig table is documenting incorrect defaults
for filterOption and optionFilterProp; only autoClearSearchValue has a real
runtime default. Update the README entry for SearchConfig so filterOption and
optionFilterProp no longer show default values (or mark them as none), and keep
the descriptions aligned with the actual behavior in the search config handling.
🪄 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: 1c17bf59-520f-4101-b8b8-3fe5bed8a7b8
📒 Files selected for processing (3)
.github/workflows/react-doctor.ymlREADME.mdtsconfig.json
afc163
force-pushed
the
codex/standardize-rc-config
branch
from
d9f9b8e to
f4de537
Compare
✅ Preview is ready!
↩️ Previous: ⚡️ 🤖 Powered by surge-preview |
|||||||||||||||
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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/surge-preview.yml:
- Around line 17-18: This workflow still uses floating action tags and leaves
checkout credentials persisted, which expands supply-chain and token exposure
risk. Update the `actions/checkout` step to disable credential persistence, and
pin both `actions/checkout` and `afc163/surge-preview` in the workflow to
immutable version identifiers (preferably full commit SHAs) so the exact action
code cannot drift over time.
🪄 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: 1c40c8a9-df2e-4cdb-a2c8-25fe24961083
📒 Files selected for processing (5)
.github/workflows/react-doctor.yml.github/workflows/surge-preview.ymlREADME.mdtsconfig.jsonvercel.json
✅ Files skipped from review due to trivial changes (2)
- tsconfig.json
- README.md
afc163
force-pushed
the
codex/standardize-rc-config
branch
from
f4de537 to
c329ff3
Compare
afc163
force-pushed
the
codex/standardize-rc-config
branch
from
c329ff3 to
ba6ddbf
Compare
|
React Doctor found no new issues. 🎉 Reviewed by React Doctor for commit |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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/test.yml:
- Around line 5-6: The reusable workflow reference in the test workflow should
remain on the mutable `@main` ref and not be changed to a fixed SHA. Keep the
existing uses: react-component/rc-test/.github/workflows/test-utoo.yml@main and
secrets: inherit as-is, since the target workflow file exists and this is the
intended reusable-workflow setup.
🪄 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: bbcc46c7-1d3b-4d05-98b4-eecd4a82912e
📒 Files selected for processing (9)
.github/FUNDING.yml.github/workflows/codeql.yml.github/workflows/react-doctor.yml.github/workflows/surge-preview.yml.github/workflows/test.ymlREADME.mdpackage.jsontsconfig.jsonvercel.json
✅ Files skipped from review due to trivial changes (2)
- .github/FUNDING.yml
- tsconfig.json
🚧 Files skipped from review as they are similar to previous changes (4)
- vercel.json
- .github/workflows/react-doctor.yml
- .github/workflows/surge-preview.yml
- README.md
|
Deployment failed with the following error: Learn More: https://vercel.com/react-component?upgradeToPro=build-rate-limit |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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/surge-preview.yml:
- Around line 19-24: In the Surge preview workflow, the current conditional
checks reference secrets.SURGE_TOKEN directly in if expressions, which breaks
workflow validation; move SURGE_TOKEN into a job-level env and update the
existing if guards to use env.SURGE_TOKEN instead. Apply this consistently to
the build step and the afc163/surge-preview action so the condition is evaluated
from env rather than secrets.
In `@tsconfig.json`:
- Line 16: The tsconfig include list is missing the root .fatherrc.js, so npm
run tsc will not type-check it. Update the include array in tsconfig.json to add
.fatherrc.js alongside .dumirc.ts, src, tests, and docs, keeping the change
localized to the tsconfig 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: 85288c03-1162-4ee2-acbb-7abadfca3928
📒 Files selected for processing (3)
.github/workflows/surge-preview.ymlREADME.mdtsconfig.json
✅ Files skipped from review due to trivial changes (1)
- README.md
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
1 participant
Summary
@rc-component/selectheader, package-specific emoji description, focused highlights, install, usage, examples, API, development, and release sections.npm install @rc-component/selectcommand, and add a subtle Ant Design ecosystem note.prepublishOnlyrelease flow.test-utoo.yml..docdumi build output.master..github/FUNDING.ymlto Ant Design sponsorship entries.optionsprop in README usage and correcting SearchConfig defaults.Compatibility
No runtime code changes. This should not introduce breaking changes.
Verification
npm run lintnpm testnpm run tscnpm run compilenpm run buildgit diff --checkRefs ant-design/ant-design#58514
Summary by CodeRabbit
tsc --noEmit、移除旧脚本）、优化 Vercel/预览输出配置，并精简 Funding；同步固定部分 CI/CodeQL 步骤版本与引用。