chore: rename qc-helper skill to proto-helper#179
Conversation
Rebrand the bundled helper skill from qc-helper to proto-helper, updating all references from Qwen Code to protoCLI and .qwen/ paths to .proto/ throughout SKILL.md, README, and divergence docs. Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
WalkthroughThis PR renames and rebrands the ChangesSkill Rename: qc-helper → proto-helper
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Review rate limit: 2/5 reviews remaining, refill in 32 minutes and 31 seconds. Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
packages/core/src/skills/bundled/proto-helper/SKILL.md (1)
38-44:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winFinish the rename in the config index (
.qwenignore→.protoignore).Line 43 still says
.qwenignore file, which conflicts with the new protoCLI naming and can point users to the wrong ignore file.Suggested patch
-| .qwenignore file | `docs/configuration/qwen-ignore.md` | +| .protoignore file | `docs/configuration/qwen-ignore.md` |🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@packages/core/src/skills/bundled/proto-helper/SKILL.md` around lines 38 - 44, Update the config index entry in SKILL.md that still references `.qwenignore`: change the display text `.qwenignore file` to `.protoignore file` and update the linked path `docs/configuration/qwen-ignore.md` to `docs/configuration/proto-ignore.md` (or the correct renamed file) so the table row reads ".protoignore file | `docs/configuration/proto-ignore.md`"; ensure both the visible name `.qwenignore` and the link target are replaced consistently.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@packages/core/src/skills/bundled/proto-helper/SKILL.md`:
- Around line 38-44: Update the config index entry in SKILL.md that still
references `.qwenignore`: change the display text `.qwenignore file` to
`.protoignore file` and update the linked path
`docs/configuration/qwen-ignore.md` to `docs/configuration/proto-ignore.md` (or
the correct renamed file) so the table row reads ".protoignore file |
`docs/configuration/proto-ignore.md`"; ensure both the visible name
`.qwenignore` and the link target are replaced consistently.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: fb468c0d-df7a-4f67-9af5-658fc9ebccc2
📒 Files selected for processing (3)
README.mddocs/architecture/divergence-from-upstream.mdpackages/core/src/skills/bundled/proto-helper/SKILL.md
Code Coverage Summary
CLI Package - Full Text ReportCore Package - Full Text ReportFor detailed HTML reports, please see the 'coverage-reports-22.x-ubuntu-latest' artifact from the main CI run. |
Summary
qc-helper→proto-helper~/.proto(was Qwen Code /~/.qwen)Test plan
/proto-helperinvokes the renamed skillqc-helperreferences in tracked files (grep already clean)Summary by CodeRabbit
New Features
Documentation