From 90fac81bad43bfa94014bf7d912aad3cafefb535 Mon Sep 17 00:00:00 2001 From: Marcin Dyguda Date: Thu, 12 Feb 2026 15:25:11 +0100 Subject: [PATCH] Add --skip-frameworks flag to /workflows:review command When passed, skips the 3 framework-specific agents (kieran-rails-reviewer, dhh-rails-reviewer, rails-turbo-expert) so non-Rails codebases get faster, more relevant reviews. Default behavior is unchanged. Co-Authored-By: Claude Opus 4.6 --- .../commands/workflows/review.md | 41 +++++++++++-------- 1 file changed, 25 insertions(+), 16 deletions(-) diff --git a/plugins/compound-engineering/commands/workflows/review.md b/plugins/compound-engineering/commands/workflows/review.md index e6593034..87319143 100644 --- a/plugins/compound-engineering/commands/workflows/review.md +++ b/plugins/compound-engineering/commands/workflows/review.md @@ -1,7 +1,7 @@ --- name: workflows:review description: Perform exhaustive code reviews using multi-agent analysis, ultra-thinking, and worktrees -argument-hint: "[PR number, GitHub URL, branch name, or latest]" +argument-hint: "[PR number, GitHub URL, branch name, or latest] [--skip-frameworks]" --- # Review Command @@ -27,6 +27,12 @@ argument-hint: "[PR number, GitHub URL, branch name, or latest]" #$ARGUMENTS + +Check if `$ARGUMENTS` contains `--skip-frameworks`: +- If present: Set SKIP_FRAMEWORKS=true. Remove the flag from the review target string before proceeding. +- If absent: Set SKIP_FRAMEWORKS=false (default). Run all agents including framework-specific ones. + + First, I need to determine the review target type and set up the code for analysis. @@ -63,21 +69,24 @@ If a review agent flags any file in these directories for cleanup or removal, di -Run ALL or most of these agents at the same time: - -1. Task kieran-rails-reviewer(PR content) -2. Task dhh-rails-reviewer(PR title) -3. If turbo is used: Task rails-turbo-expert(PR content) -4. Task git-history-analyzer(PR content) -5. Task dependency-detective(PR content) -6. Task pattern-recognition-specialist(PR content) -7. Task architecture-strategist(PR content) -8. Task code-philosopher(PR content) -9. Task security-sentinel(PR content) -10. Task performance-oracle(PR content) -11. Task devops-harmony-analyst(PR content) -12. Task data-integrity-guardian(PR content) -13. Task agent-native-reviewer(PR content) - Verify new features are agent-accessible +Run ALL of these generic agents at the same time: + +1. Task git-history-analyzer(PR content) +2. Task dependency-detective(PR content) +3. Task pattern-recognition-specialist(PR content) +4. Task architecture-strategist(PR content) +5. Task code-philosopher(PR content) +6. Task security-sentinel(PR content) +7. Task performance-oracle(PR content) +8. Task devops-harmony-analyst(PR content) +9. Task data-integrity-guardian(PR content) +10. Task agent-native-reviewer(PR content) - Verify new features are agent-accessible + +**Unless `--skip-frameworks` flag was passed**, also run these framework-specific agents: + +11. Task kieran-rails-reviewer(PR content) +12. Task dhh-rails-reviewer(PR title) +13. If turbo is used: Task rails-turbo-expert(PR content)