chore(claude): rename and extend slash commands#555
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 16 minutes and 36 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
WalkthroughRenamed Changes
Sequence Diagram(s)(omitted — changes are documentation and checklist updates; no multi-component runtime flow requiring visualization) Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Assessment against linked issues
Out-of-scope changes
Possibly related issues
Possibly related PRs
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 #555 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 3 3
Lines 114 114
=========================================
Hits 114 114
🚀 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: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.claude/commands/pre-commit.md:
- Line 10: The coverage requirement is not enforced by the pytest command;
update the command string in the pre-commit instructions (the line containing
"uv run pytest --cov=./ --cov-report=term") to include the pytest-cov fail-under
guard so the step fails when coverage is below 80% by appending
"--cov-fail-under=80" (i.e., "uv run pytest --cov=./ --cov-report=term
--cov-fail-under=80").
In @.claude/commands/pre-release.md:
- Around line 66-68: The release instructions currently tell the user to "run
steps 2–4 of `/pre-commit`", which skips commit-time CodeRabbit review (step 5);
update the prose so it runs steps 2–5 of `/pre-commit` instead (e.g., change
"run steps 2–4" to "run steps 2–5") and keep the note that the CHANGELOG step is
already handled, ensuring the commit-time review (step 5) is enforced before
pushing and opening the PR.
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: aa41b0cb-951d-4766-879e-903c993e66c8
📒 Files selected for processing (4)
.claude/commands/pre-commit.md.claude/commands/pre-release.md.claude/commands/precommit.mdCLAUDE.md
💤 Files with no reviewable changes (1)
- .claude/commands/precommit.md
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.claude/commands/pre-release.md (1)
66-66: Markdownlint style preference (safe to ignore).Markdownlint expects list item 4 but sees 6 because the numbering continues from Phase 1. This is intentional and improves readability across the multi-phase checklist. The warning can be safely ignored or suppressed with a comment if desired.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/commands/pre-release.md at line 66, The numbered list intentionally continues from Phase 1 (the line starting "6. **After confirmation**") and triggers markdownlint; to silence this, add a markdownlint suppression comment before and after the multi-phase list (e.g., insert <!-- markdownlint-disable --> immediately above the continued list and <!-- markdownlint-enable --> after it) so the linter ignores the numbering continuity while preserving the intended numbering and readability.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.claude/commands/pre-release.md:
- Line 66: The numbered list intentionally continues from Phase 1 (the line
starting "6. **After confirmation**") and triggers markdownlint; to silence
this, add a markdownlint suppression comment before and after the multi-phase
list (e.g., insert <!-- markdownlint-disable --> immediately above the continued
list and <!-- markdownlint-enable --> after it) so the linter ignores the
numbering continuity while preserving the intended numbering and readability.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: f940186c-f61d-4ec6-9bd8-694975f472c0
📒 Files selected for processing (2)
.claude/commands/pre-commit.md.claude/commands/pre-release.md
✅ Files skipped from review due to trivial changes (1)
- .claude/commands/pre-commit.md
nanotaboada
force-pushed
the
chore/rename-and-extend-slash-commands
branch
from
148f2c2 to
3379b56
Compare
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
nanotaboada
force-pushed
the
chore/rename-and-extend-slash-commands
branch
from
3379b56 to
0db6d2b
Compare
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.claude/commands/pre-release.md (1)
71-71: Static analysis false positive: cross-phase numbering is intentional.The markdownlint warning
MD029expects step "4" but sees "6" because it counts from the current markdown list block. However, this document intentionally numbers steps sequentially across all phases (Phase 1: 1-2, Phase 2: 3-6, Phase 3: 7-9) for a unified workflow. The numbering is semantically correct; you can safely ignore or suppress this warning.Optional: Suppress the warning if preferred
If you prefer to silence this specific warning, add a markdownlint comment above line 71:
<!-- markdownlint-disable-next-line MD029 --> 6. **After confirmation**: commit. Then run steps 2–5 of `/pre-commit` (linting,Or configure
.markdownlint-cli2.jsoncto allow "ordered" style for this rule.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/commands/pre-release.md at line 71, The markdown list intentionally continues numbering across phases (the line starting "6. **After confirmation**" is correct), so silence the MD029 false positive by adding a markdownlint disable comment immediately above that list item (use <!-- markdownlint-disable-next-line MD029 -->) or alternatively adjust the markdownlint config (.markdownlint-cli2.jsonc) to allow "ordered" style for MD029; update the document to include the disable comment or config change so the linter no longer flags the intentional cross-phase numbering.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.claude/commands/pre-release.md:
- Line 71: The markdown list intentionally continues numbering across phases
(the line starting "6. **After confirmation**" is correct), so silence the MD029
false positive by adding a markdownlint disable comment immediately above that
list item (use <!-- markdownlint-disable-next-line MD029 -->) or alternatively
adjust the markdownlint config (.markdownlint-cli2.jsonc) to allow "ordered"
style for MD029; update the document to include the disable comment or config
change so the linter no longer flags the intentional cross-phase numbering.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 43e7612c-fc7a-448a-bfb9-654b36559509
📒 Files selected for processing (2)
.claude/commands/pre-commit.md.claude/commands/pre-release.md
✅ Files skipped from review due to trivial changes (1)
- .claude/commands/pre-commit.md
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
nanotaboada
force-pushed
the
chore/rename-and-extend-slash-commands
branch
from
2bc290e to
9964b7e
Compare
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
nanotaboada
force-pushed
the
chore/rename-and-extend-slash-commands
branch
2 times, most recently
from
9d906f2 to
b909d6f
Compare
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
nanotaboada
force-pushed
the
chore/rename-and-extend-slash-commands
branch
2 times, most recently
from
bd2f1ab to
a7d0e71
Compare
|
❌ The last analysis has failed. |
1 participant
Summary
.claude/commands/precommit.md→pre-commit.mdandprerelease.md→pre-release.mdto align with hyphenated slash command convention/pre-commitwith an optional CodeRabbit review step (coderabbit review --type uncommitted --prompt-only) after all checks pass — blocks on serious findings, continues on nitpicks, skips gracefully if CLI not installed/pre-releasePhase 2 to reference/pre-commitby its new name/pre-releasePhase 3 — review now happens at commit timeCLAUDE.mdreference from/precommitto/pre-commitCloses #554
🤖 Generated with Claude Code
This change is
Summary by CodeRabbit
Documentation
Chores