Skip to content

chore: rename /review skill to /sweep#660

Merged
carlos-alm merged 3 commits intomainfrom
chore/rename-review-skill-to-sweep
Mar 28, 2026
Merged

chore: rename /review skill to /sweep#660
carlos-alm merged 3 commits intomainfrom
chore/rename-review-skill-to-sweep

Conversation

@carlos-alm
Copy link
Copy Markdown
Contributor

Summary

  • Renames the custom /review PR sweep skill to /sweep to avoid confusion with Claude Code's native /review command for PR reviews.

Test plan

  • Verify /sweep is listed in available skills
  • Verify /review no longer appears as a custom skill

Avoids confusion with Claude Code's native PR review feature.
@claude
Copy link
Copy Markdown

claude bot commented Mar 27, 2026

Claude finished @carlos-alm's task —— View job


PR Review Complete

Todo List:

  • Read and analyze the changed file
  • Phase 1: Root Cause Analysis
  • Phase 2: Configuration Change Detection
  • Phase 3: Test Integrity Assessment
  • Phase 4: Code Quality Review
  • Phase 5: Backlog Compliance Evaluation
  • Provide final assessment and recommendation

Root Cause Analysis

Problem Being Solved: The custom /review skill conflicts with Claude Code's native /review command for PR reviews, creating confusion for users.

Why This Approach: Renaming to /sweep clarifies the distinction - this is a comprehensive PR sweep tool that processes all open PRs, while Claude Code's native /review handles individual PR reviews. The name "sweep" accurately describes the behavior (sweeping through all PRs to clean them up).

Risk Assessment: Minimal risk. This is a pure rename with no logic changes. The skill functionality remains identical.

Phase Analysis

Phase 1 ✅: Clear problem identification - namespace collision between custom skill and native Claude Code feature.

Phase 2 ✅: No configuration changes detected. This is purely a file rename with a 1-line content change (skill name in metadata).

Phase 3 ✅: No test integrity concerns. The change only affects the skill name, not its logic or test assertions.

Phase 4 ✅: Code quality is maintained. The file is well-structured and the skill logic remains comprehensive and correct.

Phase 5: Backlog compliance assessment below.

Backlog Compliance

  • Zero-dep: ✓ — no new dependencies added
  • Foundation-aligned: ✓ — aligns with P5 (embeddable-first) by avoiding confusion with core CLI features
  • Problem-fit: 3 — useful for repository maintenance but doesn't directly address the core "lost AI" problem of code navigation
  • Breaking: No — purely additive rename, no API or behavior changes
  • Tier: 1 (zero-dep ✓ + foundation-aligned ✓)

Assessment

The change is straightforward and well-justified:

Correctly addresses namespace collision - Avoids confusion between custom skill and native feature
Appropriate naming choice - "sweep" accurately describes the multi-PR processing behavior
Minimal change scope - Only the skill name metadata is modified (line 2: name: reviewname: sweep)
Preserves all functionality - The comprehensive PR processing logic remains intact
Clear commit message - Explains the rationale for the change

File verification: The skill file at .claude/skills/sweep/SKILL.md contains detailed, comprehensive logic for processing all open PRs including conflict resolution, CI fixing, and reviewer feedback handling. The implementation appears robust and follows the repository's conventions.

Test plan verification: The proposed test plan is appropriate:

  • Verifying /sweep is available confirms the rename worked
  • Verifying /review is no longer a custom skill confirms no conflicts remain

Final Recommendation

  • Rating: ⭐⭐⭐⭐ (4/5)
  • Action: APPROVE
  • Reasoning: Clean, well-justified rename that resolves a legitimate namespace conflict without any functional changes. The implementation is solid and the change is minimal with clear benefits.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 27, 2026

Greptile Summary

This PR renames the custom /review PR sweep skill to /sweep to avoid naming conflicts with Claude Code's native /review command. The change consists of two parts: renaming the skill file from review/SKILL.md to sweep/SKILL.md (with the frontmatter name updated accordingly), and updating the internal scope list in the release skill to replace review with sweep.

  • Skill file renamed from .claude/skills/review/SKILL.md to .claude/skills/sweep/SKILL.md and frontmatter name updated from review to sweep
  • Release skill's internal-scope list updated: review replaced with sweep so commits scoped to sweep are treated as patch bumps
  • No logic changes — purely a rename to avoid collision with the native Claude Code /review command

Confidence Score: 5/5

Safe to merge — purely a rename with no logic changes and no stale references.

The change is a straightforward rename of a skill file and a one-line update to the release scope list. All references to the old review scope have been updated. The only remaining occurrence of /review in the codebase is a historical CHANGELOG entry, which is correct to leave as-is.

No files require special attention.

Important Files Changed

Filename Overview
.claude/skills/sweep/SKILL.md Renamed from review/SKILL.md; frontmatter name changed from review to sweep. All internal content unchanged.
.claude/skills/release/SKILL.md Internal scope list updated: review replaced with sweep so feat(sweep): commits are treated as patch bumps. No other changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User invokes /sweep] --> B[sweep/SKILL.md loaded]
    B --> C[PR Sweep Skill executes]
    C --> D[Step 0: /worktree isolation]
    D --> E[Step 1: Discover open PRs]
    E --> F[Step 2: Launch parallel subagents per PR]
    F --> G[2a-2i: Conflicts, CI, comments, re-trigger]
    G --> H[Step 3: Collect results & summarize]

    I[Old: /review command] -->|Renamed| B
    J[Native Claude Code /review] -->|No longer conflicts| K[Claude Code PR review]
Loading

Reviews (2): Last reviewed commit: "fix: update internal-scopes list to use ..." | Re-trigger Greptile

@carlos-alm
Copy link
Copy Markdown
Contributor Author

@greptile-apps Good catch on the stale review scope in the release skill's internal-scopes list.

Fixed in 1c11fba: replaced review with sweep in .claude/skills/release/SKILL.md line 54, keeping the list alphabetically sorted. feat(sweep): commits will now correctly be classified as patch bumps instead of minor bumps.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 28, 2026

Perfect! The fix looks good. Now the release skill will correctly handle feat(sweep): commits as patch bumps, keeping everything consistent with the rename.

@carlos-alm
Copy link
Copy Markdown
Contributor Author

@greptileai

@carlos-alm
Copy link
Copy Markdown
Contributor Author

@claude

@claude
Copy link
Copy Markdown

claude bot commented Mar 28, 2026

Claude finished @carlos-alm's task —— View job


PR Analysis Complete ✅

Todo List:

  • Read and analyze the current state of changed files
  • Verify the fix for stale review scope in release skill
  • Check for any remaining issues or improvements
  • Provide final assessment

Summary

This PR successfully completes the rename of the /review skill to /sweep and addresses all identified issues:

Main Changes Verified:

  1. Skill renamed: .claude/skills/review/SKILL.md.claude/skills/sweep/SKILL.md
  2. Metadata updated: Changed name: review to name: sweep in frontmatter (line 2)
  3. Release scope fixed: Updated internal scopes list in .claude/skills/release/SKILL.md (line 54) to replace review with sweep

Issue Resolution:

  • Greptile's P1 concern: ✅ Fixed in commit 1c11fba - the release skill now correctly includes sweep in the internal scopes list
  • Versioning correctness: ✅ Future feat(sweep): commits will now be classified as patch bumps (internal tooling) instead of minor bumps (user-facing features)
  • Alphabetical ordering: ✅ Maintained in the scopes list

Quality Checks:

  • Skill content unchanged: The comprehensive PR sweep logic in the skill file remains identical - only the name metadata changed
  • H1 heading appropriate: "# PR Review Sweep" correctly describes what the skill does (sweep of PRs under review), not the command name, so no change needed
  • No stale references: Searched for remaining review references - only found appropriate contextual uses in other skills (titan-forge, architect, etc.) that don't need updating
  • Historical references: CHANGELOG.md references to old /review command are expected and should remain unchanged

Final Assessment:

The PR is ready to merge. All issues identified in previous reviews have been addressed:

  • Namespace collision resolved ✅
  • Stale scope reference fixed ✅
  • Functionality preserved ✅
  • Clean, minimal change scope ✅

The implementation correctly avoids confusion between the custom /sweep skill (comprehensive PR processing across all open PRs) and Claude Code's native /review command (individual PR reviews).


@carlos-alm carlos-alm merged commit 25dd44c into main Mar 28, 2026
12 checks passed
@carlos-alm carlos-alm deleted the chore/rename-review-skill-to-sweep branch March 28, 2026 19:52
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant