From dd80c3631aabb24e435dfdd3df8836247db37f73 Mon Sep 17 00:00:00 2001 From: Jesse Vincent Date: Sun, 5 Jul 2026 12:26:14 -0700 Subject: [PATCH] refactor(skills): drop workflow-index section from requesting-code-review Integration with Workflows restated the When to Request Review triggers grouped by caller (each-task/before-merge/when-stuck all appear at point of use), and the intro's mechanism-rationale sentence sold a rule the preceding sentence already states. --- skills/requesting-code-review/SKILL.md | 17 +---------------- 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/skills/requesting-code-review/SKILL.md b/skills/requesting-code-review/SKILL.md index 4b8aa605..53d9ce39 100644 --- a/skills/requesting-code-review/SKILL.md +++ b/skills/requesting-code-review/SKILL.md @@ -5,7 +5,7 @@ description: Use when completing tasks, implementing major features, or before m # Requesting Code Review -Dispatch a code reviewer subagent to catch issues before they cascade. The reviewer gets precisely crafted context for evaluation — never your session's history. This keeps the reviewer focused on the work product, not your thought process, and preserves your own context for continued work. +Dispatch a code reviewer subagent to catch issues before they cascade. The reviewer gets precisely crafted context for evaluation — never your session's history. **Core principle:** Review early, review often. @@ -72,21 +72,6 @@ You: [Fix progress indicators] [Continue to Task 3] ``` -## Integration with Workflows - -**Subagent-Driven Development:** -- Review after EACH task -- Catch issues before they compound -- Fix before moving to next task - -**Executing Plans:** -- Review after each task or at natural checkpoints -- Get feedback, apply, continue - -**Ad-Hoc Development:** -- Review before merge -- Review when stuck - ## Red Flags **Never:**