Rename /continue skill to /go-on (issue #410)#440
Conversation
|
Your free trial PR review limit of 100 PRs has been reached. Please upgrade your plan to continue using CodeAnt AI. |
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ 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: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (9)
📝 WalkthroughWalkthroughThis PR renames the Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 📋 Issue PlannerBuilt with CodeRabbit's Coding Plans for faster development and fewer bugs. View plan used: ✨ Finishing Touches🧪 Generate unit tests (beta)
Review rate limit: 0/5 reviews remaining, refill in 5 minutes and 41 seconds. Comment |
|
@cursor review |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 870317f. Configure here.
|
@graphite-app re-review |
|
@coderabbitai full review |
✅ Actions performedFull review triggered. |
![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/reference/script-extraction-audit.md:
- Line 88: The table row's inline jq snippet contains an unescaped pipe causing
MD056 table parsing errors; in the row that mentions the jq filter selecting
.new_since_baseline.* and .classification.class == "finding" (the inline `jq`
snippet derived from pr-state.sh output), escape the pipe character used between
the left-hand expression and map/select (replace the literal '|' with an escaped
'\|' in the inline code span) so the Markdown table stays a valid 9-column row
and the linter stops flagging it.
In `@setup-skills-worktree.sh`:
- Around line 111-113: The loop uses the glob "$SKILLS_DIR"/*/ which excludes
broken symlinks because they are not treated as directories; change the glob to
iterate entries (e.g. "$SKILLS_DIR"/*) so dangling links are included, and
update the body to test the entry with [[ -L "$link" ]] (remove or stop relying
on the trailing slash/ ${link%/} trimming) so orphaned symlinks like
~/.claude/skills/continue get detected and removed; locate the for loop that
references SKILLS_DIR and the [[ -L ... ]] check to apply this change.
🪄 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: ASSERTIVE
Plan: Pro
Run ID: 14eb0ec8-5c70-49c2-a0c3-b8a8690585ec
⛔ Files ignored due to path filters (1)
.claude/skill-usage.csvis excluded by!**/*.csv
📒 Files selected for processing (9)
.claude/data/skill-usage.json.claude/reference/script-extraction-audit.md.claude/scripts/README.md.claude/scripts/ac-checkboxes.sh.claude/scripts/pr-state.sh.claude/skills/fixpr/audit.sh.claude/skills/go-on/SKILL.mdREADME.mdsetup-skills-worktree.sh
|
User cursoragent@cursor.com does not have a PR Review subscription. Go to Team management and add this email to the PR Review subscription. |
|
@cursor review |
- Move .claude/skills/continue to go-on; update SKILL.md frontmatter and banners - Replace slash-command references across README, scripts docs, audit reference - Seed telemetry keys: skill-usage.csv + skill-usage.json use go-on - setup-skills-worktree: drop stale skill symlinks when skill removed from worktree Co-authored-by: Bretton Auerbach <auerbachb@users.noreply.github.com>
…table - Iterate ~/.claude/skills/* (not */) for orphan + migration loops so broken symlinks after skill renames are cleaned up - Escape pipe in C-15 markdown table cell for valid MD056 parsing Co-authored-by: Bretton Auerbach <auerbachb@users.noreply.github.com>
auerbachb
force-pushed
the
cursor/rename-continue-to-go-on-7862
branch
from
cc62369 to
d79f0f7
Compare
|
@cursor review |
![codeant-ai[bot]](https://avatars.githubusercontent.com/in/646884?s=60&v=4){kind=small}
2 participants
Summary
Renames the workflow skill directory
continue→go-onso the canonical slash command is/go-on, avoiding the/cprefix collision with/continueas a Claude slash command.Updates script comment/docs references that listed
/continueas a consumer of shared helpers. Seedsgo-onin skill telemetry templates (CSV + JSON).setup-skills-worktree.sh: after syncing symlinks from the skills worktree, removes orphaned skill symlinks when the target directory no longer exists (so a renamed skill does not leave a broken~/.claude/skills/continuelink).Closes #410.
Wave 12 note
This PR should merge before other Wave 12 PRs that reference skills by name — those branches should rebase onto
mainafter this lands.Test plan
git grep -nE '/continue\b'on tracked paths — zero hits after change.github/scripts/rule-lint.shpassesbash .claude/hooks/tests/stale-worktree-warn.test.sh,python3 -m unittest tests.test_env_guard -vcoderabbit review --prompt-only— CLI not available in this environment; CI will run repo checks on the PRPost-merge (humans / local machines)
Run
bash setup-skills-worktree.sh(or rely on session-start sync) so~/.claude/skills/go-onexists and any stalecontinuesymlink is removed.Summary by CodeRabbit
Updated Features
/continueworkflow command has been renamed to/go-onacross commands and user guidance.Chores
/go-on.