cokit-cli 1.2.4 → 1.2.7

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (83) hide show
  1. package/README.md +5 -16
  2. package/agents/brainstormer.agent.md +9 -2
  3. package/agents/code-reviewer.agent.md +59 -84
  4. package/agents/code-simplifier.agent.md +9 -6
  5. package/agents/debugger.agent.md +17 -8
  6. package/agents/docs-manager.agent.md +104 -8
  7. package/agents/fullstack-developer.agent.md +57 -13
  8. package/agents/git-manager.agent.md +2 -382
  9. package/agents/planner.agent.md +36 -8
  10. package/agents/researcher.agent.md +18 -3
  11. package/agents/tester.agent.md +13 -14
  12. package/agents/ui-ux-designer.agent.md +209 -33
  13. package/docs/README.md +4 -3
  14. package/docs/claudekit-porting-rules.md +182 -0
  15. package/docs/codebase-summary.md +11 -10
  16. package/docs/cokit-comprehensive-mapping-guide.md +5 -9
  17. package/docs/cokit-slides.md +3 -5
  18. package/docs/cokit-sync-and-maintenance-guide.md +2 -2
  19. package/docs/cokit-team-presentation.md +5 -7
  20. package/docs/guide-next-steps-speckit-cokit-implementation.md +1 -1
  21. package/docs/migration-guide.md +1 -1
  22. package/docs/project-overview-pdr.md +2 -2
  23. package/docs/project-roadmap.md +7 -8
  24. package/docs/system-architecture.md +1 -3
  25. package/package.json +1 -1
  26. package/prompts/ck-ask.prompt.md +1 -1
  27. package/prompts/ck-bootstrap.prompt.md +1 -1
  28. package/prompts/ck-cook.prompt.md +12 -12
  29. package/prompts/ck-plan-hard.prompt.md +1 -1
  30. package/prompts/ck-plan-red-team.prompt.md +227 -0
  31. package/prompts/ck-simplify.prompt.md +1 -1
  32. package/skills/code-review/SKILL.md +78 -28
  33. package/skills/cook/SKILL.md +45 -11
  34. package/skills/debug/SKILL.md +112 -17
  35. package/skills/fix/SKILL.md +20 -8
  36. package/skills/frontend-design/SKILL.md +6 -3
  37. package/skills/planning/SKILL.md +47 -15
  38. package/skills/research/SKILL.md +1 -1
  39. package/skills/scout/SKILL.md +24 -11
  40. package/skills/web-testing/SKILL.md +60 -6
  41. package/skills/web-testing/references/report-format.md +57 -0
  42. package/skills/web-testing/references/test-execution-workflow.md +118 -0
  43. package/skills/web-testing/references/ui-testing-workflow.md +97 -0
  44. package/src/commands/add.js +0 -1
  45. package/src/commands/doctor.js +2 -2
  46. package/src/commands/init.js +19 -28
  47. package/src/commands/update.js +1 -1
  48. package/templates/repo/.github/agents/brainstormer.agent.md +9 -2
  49. package/templates/repo/.github/agents/code-reviewer.agent.md +59 -84
  50. package/templates/repo/.github/agents/code-simplifier.agent.md +9 -6
  51. package/templates/repo/.github/agents/debugger.agent.md +17 -8
  52. package/templates/repo/.github/agents/docs-manager.agent.md +104 -8
  53. package/templates/repo/.github/agents/fullstack-developer.agent.md +57 -13
  54. package/templates/repo/.github/agents/git-manager.agent.md +2 -382
  55. package/templates/repo/.github/agents/planner.agent.md +36 -8
  56. package/templates/repo/.github/agents/researcher.agent.md +18 -3
  57. package/templates/repo/.github/agents/tester.agent.md +13 -14
  58. package/templates/repo/.github/agents/ui-ux-designer.agent.md +209 -33
  59. package/templates/repo/.github/prompts/ck-ask.prompt.md +1 -1
  60. package/templates/repo/.github/prompts/ck-bootstrap.prompt.md +1 -1
  61. package/templates/repo/.github/prompts/ck-cook.prompt.md +12 -12
  62. package/templates/repo/.github/prompts/ck-plan-fast.prompt.md +1 -0
  63. package/templates/repo/.github/prompts/ck-plan-hard.prompt.md +2 -1
  64. package/templates/repo/.github/prompts/ck-plan-red-team.prompt.md +227 -0
  65. package/templates/repo/.github/prompts/ck-plan.prompt.md +1 -0
  66. package/templates/repo/.github/prompts/ck-simplify.prompt.md +1 -1
  67. package/templates/repo/.github/prompts/ck-spec-specify.prompt.md +1 -0
  68. package/templates/repo/.github/skills/code-review/SKILL.md +78 -28
  69. package/templates/repo/.github/skills/cook/SKILL.md +45 -11
  70. package/templates/repo/.github/skills/debug/SKILL.md +112 -17
  71. package/templates/repo/.github/skills/fix/SKILL.md +20 -8
  72. package/templates/repo/.github/skills/frontend-design/SKILL.md +6 -3
  73. package/templates/repo/.github/skills/planning/SKILL.md +47 -15
  74. package/templates/repo/.github/skills/research/SKILL.md +1 -1
  75. package/templates/repo/.github/skills/scout/SKILL.md +24 -11
  76. package/templates/repo/.github/skills/web-testing/SKILL.md +60 -6
  77. package/templates/repo/.github/skills/web-testing/references/report-format.md +57 -0
  78. package/templates/repo/.github/skills/web-testing/references/test-execution-workflow.md +118 -0
  79. package/templates/repo/.github/skills/web-testing/references/ui-testing-workflow.md +97 -0
  80. package/prompts/ck-journal.prompt.md +0 -19
  81. package/prompts/ck-preview.prompt.md +0 -77
  82. package/templates/repo/.github/prompts/ck-journal.prompt.md +0 -19
  83. package/templates/repo/.github/prompts/ck-preview.prompt.md +0 -77
@@ -1,9 +1,9 @@
1
1
  # CoKit: Team Presentation
2
2
 
3
- **Date:** February 10, 2026 (v1.2.0)
3
+ **Date:** February 25, 2026 (v1.2.6)
4
4
  **Presenter:** [Your Name]
5
5
  **Repository:** https://github.com/camping89/cokit
6
- **Version:** 1.2.0
6
+ **Version:** 1.2.6
7
7
 
8
8
  ---
9
9
 
@@ -18,7 +18,7 @@
18
18
  - Project-specific context
19
19
 
20
20
  **Solution:** CoKit ports proven Claude Code workflow patterns to GitHub Copilot via:
21
- - 27 prompts with unified `ck-*` namespace (ClaudeKit + SpecKit) using `mode: agent`
21
+ - 25 prompts with unified `ck-*` namespace (ClaudeKit + SpecKit) using `mode: agent`
22
22
  - 27 skills teaching Copilot best practices (debugging, code-review, planning, frontend-design, databases, devops, etc.)
23
23
  - 12 specialized agents (including code-simplifier, fullstack-developer, ui-ux-designer)
24
24
  - 5 instructions (backend, frontend, testing, development, research)
@@ -79,7 +79,7 @@
79
79
  │ ┌─────────────────────────────────────────────────────────┐ │
80
80
  │ │ ✓ .github/copilot-instructions.md │ │
81
81
  │ │ ✓ .github/AGENTS.md │ │
82
- │ │ ✓ .github/prompts/ck-*.prompt.md (27 prompts) │ │
82
+ │ │ ✓ .github/prompts/ck-*.prompt.md (25 prompts) │ │
83
83
  │ │ ✓ .github/instructions/ck-*.instructions.md (5 files) │ │
84
84
  │ │ ✓ .github/collections/ck-*.collection.yml (5 bundles) │ │
85
85
  │ │ ✓ .vscode/settings.json │ │
@@ -158,8 +158,6 @@ your-project/
158
158
  | Command | Description |
159
159
  |---------|-------------|
160
160
  | `npx cokit-cli init` | Interactive setup wizard |
161
- | `npx cokit-cli init -g` | Install personal skills only |
162
- | `npx cokit-cli init -a` | Both project + personal |
163
161
  | `npx cokit-cli add <skill>` | Add specific skill |
164
162
  | `npx cokit-cli list` | Show installed components |
165
163
  | `npx cokit-cli doctor` | Diagnose setup issues |
@@ -175,7 +173,7 @@ your-project/
175
173
  **SpecKit (ck-spec-*):** Spec-driven workflow commands
176
174
  - `/ck-spec-specify`, `/ck-spec-clarify`, `/ck-spec-plan`, `/ck-spec-tasks`, `/ck-spec-implement`, etc.
177
175
 
178
- All 27 prompts unified in single namespace with cross-navigation support.
176
+ All 25 prompts unified in single namespace with cross-navigation support.
179
177
 
180
178
  ---
181
179
 
@@ -1,7 +1,7 @@
1
1
  # Next Steps Guide: Spec-Kit + Cokit Implementation
2
2
 
3
3
  **Created:** Feb 3, 2026
4
- **Status:** COMPLETE (v1.2.0 released with SpecKit integration)
4
+ **Status:** COMPLETE (v1.2.6 released with SpecKit integration)
5
5
 
6
6
  ---
7
7
 
@@ -102,7 +102,7 @@ After fixing:
102
102
 
103
103
  1. Install CoKit:
104
104
  ```bash
105
- npx cokit-cli init -a
105
+ npx cokit-cli init
106
106
  ```
107
107
 
108
108
  2. Use prompts directly:
@@ -10,7 +10,7 @@ Make GitHub Copilot smarter in 30 seconds via curated prompts, agents, and workf
10
10
 
11
11
  ## Solution
12
12
  CoKit CLI installs pre-built resources to `~/.copilot/`:
13
- - **27 prompts**: ClaudeKit (ck-*) + SpecKit (ck-spec-*)
13
+ - **25 prompts**: ClaudeKit (ck-*) + SpecKit (ck-spec-*)
14
14
  - **12 agents**: brainstormer, planner, tester, code-simplifier, fullstack-developer, ui-ux-designer, etc.
15
15
  - **27 skills**: debugging, planning, code-review, frontend-design, databases, devops, etc.
16
16
  - **5 collections**: bundled workflows
@@ -22,7 +22,7 @@ CoKit CLI installs pre-built resources to `~/.copilot/`:
22
22
  | Two Workflows | `ck-*` (flexible) vs `ck-spec-*` (spec-driven) |
23
23
  | Sync Pipeline | Auto-merge upstream ClaudeKit + SpecKit |
24
24
  | Navigation | Cross-command suggestions in all prompts |
25
- | Global Install | Single `npx cokit-cli init -g` command |
25
+ | Install | Single `npx cokit-cli init` command |
26
26
 
27
27
  ## Target Users
28
28
  - Developers using GitHub Copilot
@@ -1,9 +1,9 @@
1
1
  # CoKit Project Roadmap
2
2
 
3
3
  **Project:** CoKit - Claude Code to GitHub Copilot Port
4
- **Version:** 1.2.4
4
+ **Version:** 1.2.6
5
5
  **Start Date:** 2026-01-06
6
- **Last Updated:** 2026-02-12 (video intro + flowchart UX)
6
+ **Last Updated:** 2026-02-25 (agents/prompts/skills Copilot alignment)
7
7
  **Current Phase:** 6 of 6 (COMPLETE)
8
8
  **Overall Progress:** 100%
9
9
 
@@ -12,7 +12,7 @@
12
12
  CoKit makes it dead-simple for non-technical users to port Claude Code workflow patterns to GitHub Copilot with a single command: `npx cokit-cli init`.
13
13
 
14
14
  ### Key Features (Current)
15
- - 27 prompt files with `ck-` and `ck-spec-` prefixes
15
+ - 25 prompt files with `ck-` and `ck-spec-` prefixes
16
16
  - 27 specialized skills (debugging, code-review, planning, frontend-design, databases, devops, etc.)
17
17
  - 12 expert agents (planner, code-reviewer, debugger, tester, code-simplifier, fullstack-developer, ui-ux-designer, etc.)
18
18
  - 5 instructions (backend, frontend, testing, development, research)
@@ -60,7 +60,7 @@ CoKit makes it dead-simple for non-technical users to port Claude Code workflow
60
60
  **Status:** COMPLETE (2026-01-06, Enhanced 2026-02-10)
61
61
  **Effort:** 4h
62
62
  **Deliverables:**
63
- - 27 prompt files from ClaudeKit + SpecKit
63
+ - 25 prompt files from ClaudeKit + SpecKit
64
64
  - ClaudeKit: `ck-fix`, `ck-plan`, `ck-cook`, `ck-test`, `ck-review`, etc.
65
65
  - SpecKit: `ck-spec-specify`, `ck-spec-clarify`, `ck-spec-plan`, `ck-spec-tasks`, etc.
66
66
  - Unified namespace: `ck-*` for all commands
@@ -93,7 +93,7 @@ CoKit makes it dead-simple for non-technical users to port Claude Code workflow
93
93
  **Deliverables:**
94
94
  - Removed non-coding resources
95
95
  - Consolidated documentation with updated counts
96
- - Updated all CLI references: `cokit` → `cokit-cli`, `--global` `-g`, `--all` → `-a`
96
+ - Updated all CLI references: `cokit` → `cokit-cli`, removed `-g`/`--global` flag
97
97
  - Updated version references to 1.0.8
98
98
  - Fixed broken links (README.FLOW.md, FAQ.md)
99
99
  - Added real CoKit examples throughout docs
@@ -312,6 +312,5 @@ None at completion. All functional requirements satisfied.
312
312
 
313
313
  ---
314
314
 
315
- **Last Updated:** 2026-02-12
316
- **Project Manager:** project-manager agent
317
- **Status:** ALL PHASES COMPLETE - v1.2.4 RELEASED
315
+ **Last Updated:** 2026-02-25
316
+ **Status:** ALL PHASES COMPLETE - v1.2.6 RELEASED
@@ -37,8 +37,6 @@
37
37
  │ CLI (npx cokit-cli) │
38
38
  ├─────────────────────────────────────────────────────────────┤
39
39
  │ bin/cokit.js → src/index.js │
40
- │ ├── init -g → Install to ~/.copilot/ │
41
- │ ├── init -a → Install global + project │
42
40
  │ └── init → Interactive setup │
43
41
  └─────────────────────────────────────────────────────────────┘
44
42
  ```
@@ -57,7 +55,7 @@
57
55
 
58
56
  ### CLI Install
59
57
  ```
60
- 1. User runs: npx cokit-cli init -g
58
+ 1. User runs: npx cokit-cli init
61
59
  2. CLI copies: prompts/, agents/, skills/, instructions/, collections/
62
60
  3. Target: ~/.copilot/
63
61
  4. User gets: All ck-* commands in Copilot
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "cokit-cli",
3
- "version": "1.2.4",
3
+ "version": "1.2.7",
4
4
  "description": "Make GitHub Copilot smarter in 30 seconds",
5
5
  "main": "src/index.js",
6
6
  "bin": {
@@ -36,7 +36,7 @@ You operate by the holy trinity of software engineering: **YAGNI** (You Aren't G
36
36
 
37
37
  ## Process
38
38
  1. **Problem Understanding**: Analyze the technical question and gather architectural context.
39
- - If the architecture context doesn't contain the necessary information, use [`SlashCommand(/scout)`](`$HOME/.copilot/commands/scout.md`) to scout the codebase again.
39
+ - If the architecture context doesn't contain the necessary information, use `/ck-scout` to scout the codebase again.
40
40
  2. **Expert Consultation**:
41
41
  - Systems Designer: Define system boundaries, data flows, and component relationships
42
42
  - Technology Strategist: Evaluate technology choices, patterns, and industry best practices
@@ -115,7 +115,7 @@ Follow strictly these following steps:
115
115
  * Create/update `./docs/project-overview.-pdr.md` (Product Development Requirements) file.
116
116
  * Create/update `./docs/code-standards.md` file.
117
117
  * Create/update `./docs/system-architecture.md` file.
118
- * Use `project-manager` agent to create a project roadmap at `./docs/project-roadmap.md` file & project progress and task status in the given plan file.
118
+ * Create a project roadmap at `./docs/project-roadmap.md` file & update project progress and task status in the given plan file.
119
119
  * **IMPORTANT:** Sacrifice grammar for the sake of concision when writing outputs.
120
120
 
121
121
  ### Onboarding
@@ -56,21 +56,21 @@ Human review required at these checkpoints (skipped with `--auto`):
56
56
  **Always enforced (all modes):**
57
57
  - **Testing:** 100% pass required (unless no-test mode)
58
58
  - **Code Review:** User approval OR auto-approve (score≥9.5, 0 critical)
59
- - **Finalize:** project-manager AND docs-manager must complete
59
+ - **Finalize:** docs-manager must complete
60
60
 
61
61
  ## Required Agents
62
62
 
63
63
  | Phase | Agent |
64
64
  |-------|-------|
65
- | Research | `@researcher` (parallel, optional in fast) |
66
- | Scout | `@scout` |
67
- | Plan | `@planner` |
68
- | UI Work | `@ui-ux-designer` |
69
- | Implement | `@fullstack-developer` (parallel mode) |
70
- | Simplify | `@code-simplifier` (post-implement) |
71
- | Testing | `@tester`, `@debugger` |
72
- | Review | `@code-reviewer` |
73
- | Finalize | `project-manager`, `@docs-manager`, `@git-manager` |
65
+ | Research | `researcher` agent (parallel, optional in fast) |
66
+ | Scout | `scout` agent |
67
+ | Plan | `planner` agent |
68
+ | UI Work | `ui-ux-designer` agent |
69
+ | Implement | `fullstack-developer` agent (parallel mode) |
70
+ | Simplify | `code-simplifier` agent (post-implement) |
71
+ | Testing | `tester`, `debugger` agents |
72
+ | Review | `code-reviewer` agent |
73
+ | Finalize | `docs-manager`, `git-manager` agents |
74
74
 
75
75
  ## Process
76
76
 
@@ -78,8 +78,8 @@ Human review required at these checkpoints (skipped with `--auto`):
78
78
  2. **Scout**: Use `/ck-scout` to discover relevant files
79
79
  3. **Research** (if applicable): Gather technical context
80
80
  4. **Plan**: Create or load implementation plan
81
- 5. **Implement**: Execute plan phases (use `@fullstack-developer` in parallel mode)
82
- 6. **Simplify**: `@code-simplifier` refines implemented code for clarity
81
+ 5. **Implement**: Execute plan phases (use `fullstack-developer` agent in parallel mode)
82
+ 6. **Simplify**: `code-simplifier` agent refines implemented code for clarity
83
83
  7. **Test**: Run tests and validate
84
84
  8. **Review**: Code review with quality gates
85
85
  9. **Finalize**: Update docs, commit changes
@@ -64,7 +64,7 @@ After plan creation, offer validation interview to confirm decisions before impl
64
64
  | `off` | Skip validation step entirely |
65
65
 
66
66
  **If mode is `prompt`:** Present the options above directly in your response and wait for user to choose.
67
- **If user chooses validation or mode is `auto`:** Execute `/ck-plan-validate {plan-path}` SlashCommand.
67
+ **If user chooses validation or mode is `auto`:** Execute `/ck-plan-validate {plan-path}`.
68
68
 
69
69
  ## Context Reminder (MANDATORY)
70
70
 
@@ -0,0 +1,227 @@
1
+ ---
2
+ agent: 'agent'
3
+ description: 'Adversarial plan review — spawn hostile reviewers to find flaws, security holes, false assumptions, failure modes'
4
+ argument-hint: 'Path to plan directory'
5
+ ---
6
+
7
+ ## Variant Notice
8
+ **IMPORTANT — Read before proceeding.**
9
+ `ck-plan-red-team` is an internal mode for adversarial plan review — it is meant to be selected automatically by AI when you run `/ck-plan`.
10
+ You don't need to call this directly. Just use `/ck-plan` and AI will pick the right mode for you.
11
+ Before executing, you MUST output the following message **exactly as written** and wait for user response:
12
+
13
+ ---
14
+ **Variant Notice**
15
+
16
+ `ck-plan-red-team` is an internal mode for adversarial plan review — it is meant to be selected automatically by AI when you run `/ck-plan`.
17
+ You don't need to call this directly. Just use `/ck-plan` and AI will pick the right mode for you.
18
+
19
+ Do you want to continue anyway, or switch to `/ck-plan`? **[Continue / Switch to /ck-plan]**
20
+
21
+ ---
22
+
23
+ Only proceed if user explicitly confirms Continue.
24
+ If user chooses "Switch to /ck-plan", run `/ck-plan` immediately — do NOT ask user to re-enter their input.
25
+
26
+ ## Your Mission
27
+
28
+ Adversarially review an implementation plan by spawning parallel reviewer agents that try to tear it apart. Each reviewer adopts a different hostile lens. You then adjudicate findings, and the user decides which to apply.
29
+
30
+ **Mindset:** Like hiring someone who hates the implementer to destroy their work.
31
+
32
+ ## Plan Resolution
33
+
34
+ 1. If `${input}` provided → Use that path
35
+ 2. Else check `## Plan Context` section → Use active plan path
36
+ 3. If no plan found → Ask user to specify path or run `/ck-plan` first
37
+
38
+ ## Workflow
39
+
40
+ ### Step 1: Read Plan Files
41
+
42
+ Read the plan directory:
43
+ - `plan.md` — Overview, phases, dependencies
44
+ - `phase-*.md` — All phase files (full content)
45
+ - Note: architecture decisions, assumptions, scope, risks, implementation steps
46
+
47
+ Collect all plan file paths for reviewers to read directly.
48
+
49
+ ### Step 2: Scale Reviewer Count
50
+
51
+ Scale reviewers based on plan complexity:
52
+
53
+ | Phase Count | Reviewers | Lenses Selected |
54
+ |-------------|-----------|-----------------|
55
+ | 1-2 phases | 2 | Security Adversary + Assumption Destroyer |
56
+ | 3-5 phases | 3 | + Failure Mode Analyst |
57
+ | 6+ phases | 4 | + Scope & Complexity Critic (all lenses) |
58
+
59
+ ### Step 3: Define Adversarial Lenses
60
+
61
+ Available lenses (select per Step 2):
62
+
63
+ | Reviewer | Lens | Focus |
64
+ |----------|------|-------|
65
+ | **Security Adversary** | Attacker mindset | Auth bypass, injection, data exposure, privilege escalation, supply chain, OWASP top 10 |
66
+ | **Failure Mode Analyst** | Murphy's Law | Race conditions, data loss, cascading failures, recovery gaps, deployment risks, rollback holes |
67
+ | **Assumption Destroyer** | Skeptic | Unstated dependencies, false "will work" claims, missing error paths, scale assumptions, integration assumptions |
68
+ | **Scope & Complexity Critic** | YAGNI enforcer | Over-engineering, premature abstraction, unnecessary complexity, missing MVP cuts, scope creep, gold plating |
69
+
70
+ ### Step 4: Spawn Reviewers
71
+
72
+ Launch reviewers simultaneously using `code-reviewer` agents in parallel.
73
+
74
+ **Each reviewer prompt MUST include:**
75
+ 1. This override: `"IGNORE your default code-review instructions. You are reviewing a PLAN DOCUMENT, not code. There is no code to lint, build, or test. Focus exclusively on plan quality."`
76
+ 2. Their specific adversarial lens and persona
77
+ 3. The plan file paths so they can read original files directly
78
+ 4. These instructions:
79
+
80
+ ```
81
+ You are a hostile reviewer. Your job is to DESTROY this plan.
82
+ Adopt the {LENS_NAME} perspective. Find every flaw you can.
83
+
84
+ Rules:
85
+ - Be specific: cite exact phase/section where the flaw lives
86
+ - Be concrete: describe the failure scenario, not just "could be a problem"
87
+ - Rate severity: Critical (blocks success) | High (significant risk) | Medium (notable concern)
88
+ - Skip trivial observations (style, naming, formatting) — not worth reporting.
89
+ - No praise. No "overall looks good". Only findings.
90
+ - 5-10 findings per reviewer. Quality over quantity.
91
+
92
+ Output format per finding:
93
+ ## Finding {N}: {title}
94
+ - **Severity:** Critical | High | Medium
95
+ - **Location:** Phase {X}, section "{name}"
96
+ - **Flaw:** {what's wrong}
97
+ - **Failure scenario:** {concrete description of how this fails}
98
+ - **Evidence:** {quote from plan or missing element}
99
+ - **Suggested fix:** {brief recommendation}
100
+ ```
101
+
102
+ ### Step 5: Collect, Deduplicate & Cap
103
+
104
+ After all reviewers complete:
105
+ 1. Collect all findings
106
+ 2. Deduplicate overlapping findings (merge if same root issue)
107
+ 3. Sort by severity: Critical → High → Medium
108
+ 4. **Cap at 15 findings:** Keep all Critical, top High by specificity, note dropped Medium count
109
+
110
+ ### Step 6: Adjudicate
111
+
112
+ For each finding, evaluate and propose a disposition:
113
+
114
+ | Disposition | Meaning |
115
+ |-------------|---------|
116
+ | **Accept** | Valid flaw — plan should be updated |
117
+ | **Reject** | False positive, acceptable risk, or already handled |
118
+
119
+ **Adjudication format:**
120
+
121
+ ```markdown
122
+ ## Red Team Findings
123
+
124
+ ### Finding 1: {title} — {SEVERITY}
125
+ **Reviewer:** {lens name}
126
+ **Location:** {phase/section}
127
+ **Flaw:** {description}
128
+ **Failure scenario:** {concrete scenario}
129
+ **Disposition:** Accept | Reject
130
+ **Rationale:** {why accept/reject — be specific}
131
+ ```
132
+
133
+ ### Step 7: User Review
134
+
135
+ Present the adjudicated findings to the user directly in your response. List all findings with their dispositions, then ask:
136
+
137
+ > **Review red-team findings.** Which dispositions do you want to change?
138
+ >
139
+ > 1. **Looks good, apply accepted findings** — proceed with current Accept/Reject
140
+ > 2. **Let me review each one** — walk through findings individually
141
+ > 3. **Reject all, plan is fine** — discard all findings
142
+
143
+ Wait for user response before proceeding.
144
+
145
+ **If "Let me review each one":**
146
+ For each finding marked Accept, present it and ask:
147
+ - "Apply this fix to the plan?" with options: **Yes, apply** | **No, reject** | **Modify suggestion**
148
+
149
+ **If "Modify suggestion":**
150
+ Ask user: "Describe your modification to this finding's suggested fix:"
151
+ Record the modified suggestion in the finding's "Suggested fix" field.
152
+ Set disposition to "Accept (modified)" in the Red Team Review table.
153
+
154
+ ### Step 8: Apply to Plan
155
+
156
+ For each accepted finding:
157
+ 1. Locate the target phase file and section
158
+ 2. Add the fix/note inline with a marker:
159
+ ```markdown
160
+ <!-- Red Team: {finding title} — {date} -->
161
+ ```
162
+ 3. If finding requires new content, add to the most relevant section
163
+ 4. If finding requires removing/changing content, edit in place
164
+
165
+ After applying, add a `## Red Team Review` section to `plan.md`.
166
+ If section already exists (repeat run), **append** a new session block — never overwrite history.
167
+
168
+ **Placement order in plan.md** (bottom of file):
169
+ 1. `## Red Team Review` (before validation)
170
+ 2. `## Validation Log` (after red-team)
171
+
172
+ This ordering matches the execution sequence: red-team → validate.
173
+
174
+ ```markdown
175
+ ## Red Team Review
176
+
177
+ ### Session — {YYYY-MM-DD}
178
+ **Findings:** {total} ({accepted} accepted, {rejected} rejected)
179
+ **Severity breakdown:** {N} Critical, {N} High, {N} Medium
180
+
181
+ | # | Finding | Severity | Disposition | Applied To |
182
+ |---|---------|----------|-------------|------------|
183
+ | 1 | {title} | Critical | Accept | Phase 2 |
184
+ | 2 | {title} | High | Reject | — |
185
+ ```
186
+
187
+ ## Output
188
+
189
+ After completion, provide summary:
190
+ - Total findings by severity
191
+ - Accepted vs rejected count
192
+ - Files modified
193
+ - Key risks addressed
194
+ - Remaining concerns (if any rejected findings were borderline)
195
+
196
+ ## Next Steps (MANDATORY)
197
+
198
+ After providing the summary, remind the user:
199
+
200
+ > **Plan updated with red-team findings.** Consider running:
201
+ > ```
202
+ > /ck-plan-validate {ABSOLUTE_PATH_TO_PLAN_DIR}/plan.md
203
+ > ```
204
+ > to re-validate decisions after changes, then:
205
+ > ```
206
+ > /ck-cook --auto {ABSOLUTE_PATH_TO_PLAN_DIR}/plan.md
207
+ > ```
208
+ > to implement.
209
+
210
+ ## Important Notes
211
+
212
+ **IMPORTANT:** Reviewers must be HOSTILE, not helpful. No softening language.
213
+ **IMPORTANT:** Deduplicate aggressively — reviewers will find overlapping issues.
214
+ **IMPORTANT:** Adjudication must be evidence-based. Don't reject valid findings to be nice.
215
+ **IMPORTANT:** If plan has a Validation Log from `/ck-plan-validate`, reviewers should check if validation answers introduced new assumptions.
216
+ **IMPORTANT:** Sacrifice grammar for concision in reports.
217
+ **IMPORTANT:** Reviewers read plan files directly — do NOT duplicate content in a summary.
218
+
219
+ ---
220
+
221
+ ## Suggested Next Steps
222
+
223
+ | Command | Description |
224
+ |---------|-------------|
225
+ | `/ck-plan-validate` | Validate plan with critical questions |
226
+ | `/ck-cook` | Implement the plan |
227
+ | `/ck-plan` | Create or modify plan |
@@ -1,5 +1,5 @@
1
1
  ---
2
- agent: 'code-simplifier'
2
+ agent: 'agent'
3
3
  description: 'Simplify and refine code for clarity and maintainability'
4
4
  argument-hint: 'Scope: file path, git diff, or "recent changes"'
5
5
  ---
@@ -1,6 +1,6 @@
1
1
  ---
2
2
  name: code-review
3
- description: Review code quality, receive feedback with technical rigor, verify completion claims. Use before PRs, after implementing features, when claiming task completion, for agent reviews.
3
+ description: Review code quality, receive feedback with technical rigor, verify completion claims. Includes edge case scouting for multi-file features. Use before PRs, after implementing features, when claiming task completion, for agent reviews.
4
4
  ---
5
5
 
6
6
  # Code Review
@@ -9,13 +9,12 @@ Guide proper code review practices emphasizing technical rigor, evidence-based c
9
9
 
10
10
  ## Overview
11
11
 
12
- Code review requires three distinct practices:
13
-
14
- 1. **Receiving feedback** - Technical evaluation over performative agreement
15
- 2. **Requesting reviews** - Systematic review via code-reviewer agent
16
- 3. **Verification gates** - Evidence before any completion claims
17
-
18
- Each practice has specific triggers and protocols detailed in reference files.
12
+ | Practice | When | Protocol |
13
+ |----------|------|----------|
14
+ | **Edge Case Scouting** | Before any review on 3+ file features | `/ck-scout` for hidden paths and untested scenarios |
15
+ | **Receiving Feedback** | Feedback from human or agent | READ → UNDERSTAND → VERIFY → EVALUATE → RESPOND → IMPLEMENT |
16
+ | **Requesting Reviews** | After each task, before merge, after major features | Delegate to `code-reviewer` agent |
17
+ | **Verification Gates** | Before any completion claim | Run command, read output, then claim |
19
18
 
20
19
  ## Core Principle
21
20
 
@@ -26,6 +25,15 @@ Always honoring **YAGNI**, **KISS**, and **DRY** principles.
26
25
 
27
26
  ## When to Use This Skill
28
27
 
28
+ ### Edge Case Scouting
29
+ Trigger when:
30
+ - Feature touches 3+ files
31
+ - Implementing complex business logic
32
+ - Before requesting formal code review
33
+ - After implementation, before testing
34
+
35
+ **Reference:** `references/requesting-code-review.md`
36
+
29
37
  ### Receiving Feedback
30
38
  Trigger when:
31
39
  - Receiving code review comments from any source
@@ -61,35 +69,74 @@ Trigger when:
61
69
  ```
62
70
  SITUATION?
63
71
 
72
+ ├─ Multi-file feature (3+ files)?
73
+ │ └─ Run edge case scouting first → /ck-scout then request review
74
+
64
75
  ├─ Received feedback
65
76
  │ ├─ Unclear items? → STOP, ask for clarification first
66
77
  │ ├─ From human partner? → Understand, then implement
67
78
  │ └─ From external reviewer? → Verify technically before implementing
68
79
 
69
80
  ├─ Completed work
70
- │ ├─ Major feature/task? → Request code-reviewer agent review
71
- │ └─ Before merge? → Request code-reviewer agent review
81
+ │ ├─ Major feature/task? → Request `code-reviewer` agent review
82
+ │ └─ Before merge? → Request `code-reviewer` agent review
72
83
 
73
84
  └─ About to claim status
74
85
  ├─ Have fresh verification? → State claim WITH evidence
75
86
  └─ No fresh verification? → RUN verification command first
76
87
  ```
77
88
 
89
+ ## Edge Case Scouting
90
+
91
+ ### When to Scout
92
+ Before formal review of any multi-file feature (3+ files changed).
93
+
94
+ ### Process
95
+ 1. Use `/ck-scout` to search for hidden code paths, edge inputs, error branches
96
+ 2. Document untested scenarios found
97
+ 3. Add tests or guards for critical edge cases
98
+ 4. Then proceed to formal `code-reviewer` review
99
+
100
+ ### What to Look For
101
+ - Null/undefined paths not covered by tests
102
+ - Error branches lacking handlers
103
+ - Boundary conditions (empty arrays, max values, concurrent calls)
104
+ - Async race conditions
105
+ - Permission/auth edge cases
106
+
107
+ ## Task-Managed Review Pipeline (Multi-File Features)
108
+
109
+ For features spanning 3+ files, use a structured pipeline:
110
+
111
+ ```
112
+ scout → review → fix → verify
113
+ ```
114
+
115
+ **Steps:**
116
+ 1. **Scout** - Use `/ck-scout` or `/ck-scout ext` to identify edge cases and gaps
117
+ 2. **Review** - Delegate to `code-reviewer` agent with full context
118
+ 3. **Fix** - Implement critical and important feedback
119
+ 4. **Verify** - Run tests, confirm fixes, then claim completion
120
+
121
+ Track progress using a checklist in your plan or task notes:
122
+ ```
123
+ - [ ] Edge case scouting complete
124
+ - [ ] `code-reviewer` review complete
125
+ - [ ] Critical issues fixed
126
+ - [ ] Verification passed
127
+ ```
128
+
78
129
  ## Receiving Feedback Protocol
79
130
 
80
131
  ### Response Pattern
81
132
  READ → UNDERSTAND → VERIFY → EVALUATE → RESPOND → IMPLEMENT
82
133
 
83
134
  ### Key Rules
84
- - No performative agreement: "You're absolutely right!", "Great point!", "Thanks for [anything]"
85
- - No implementation before verification
86
- - Restate requirement, ask questions, push back with technical reasoning, or just start working
87
- - If unclear: STOP and ask for clarification on ALL unclear items first
88
- - YAGNI check: grep for usage before implementing suggested "proper" features
89
-
90
- ### Source Handling
91
- - **Human partner:** Trusted - implement after understanding, no performative agreement
92
- - **External reviewers:** Verify technically correct, check for breakage, push back if wrong
135
+ - No performative agreement: "You're absolutely right!", "Great point!", "Thanks for [anything]"
136
+ - No implementation before verification
137
+ - Restate requirement, ask questions, push back with technical reasoning, or just start working
138
+ - If unclear: STOP and ask for clarification on ALL unclear items first
139
+ - YAGNI check: search for usage before implementing suggested "proper" features
93
140
 
94
141
  **Full protocol:** `references/code-review-reception.md`
95
142
 
@@ -101,9 +148,10 @@ READ → UNDERSTAND → VERIFY → EVALUATE → RESPOND → IMPLEMENT
101
148
  - Before merge to main
102
149
 
103
150
  ### Process
104
- 1. Get git SHAs: `BASE_SHA=$(git rev-parse HEAD~1)` and `HEAD_SHA=$(git rev-parse HEAD)`
105
- 2. Delegate to code-reviewer agent with: WHAT_WAS_IMPLEMENTED, PLAN_OR_REQUIREMENTS, BASE_SHA, HEAD_SHA, DESCRIPTION
106
- 3. Act on feedback: Fix Critical immediately, Important before proceeding, note Minor for later
151
+ 1. Scout edge cases (3+ file features): use `/ck-scout` first
152
+ 2. Get git SHAs: `BASE_SHA=$(git rev-parse HEAD~1)` and `HEAD_SHA=$(git rev-parse HEAD)`
153
+ 3. Delegate to `code-reviewer` agent with: WHAT_WAS_IMPLEMENTED, PLAN_OR_REQUIREMENTS, BASE_SHA, HEAD_SHA, DESCRIPTION
154
+ 4. Act on feedback: Fix Critical immediately, Important before proceeding, note Minor for later
107
155
 
108
156
  **Full protocol:** `references/requesting-code-review.md`
109
157
 
@@ -130,14 +178,16 @@ Using "should"/"probably"/"seems to", expressing satisfaction before verificatio
130
178
 
131
179
  ## Integration with Workflows
132
180
 
133
- - **Agent-Driven:** Review after EACH task, verify before moving to next
134
- - **Pull Requests:** Verify tests pass, request code-reviewer review before merge
181
+ - **Agent-Driven:** Scout edge cases first (3+ files), review after EACH task, verify before moving to next
182
+ - **Pull Requests:** Scout → verify tests pass request `code-reviewer` review before merge
135
183
  - **General:** Apply verification gates before any status claims, push back on invalid feedback
184
+ - **Pipeline:** For complex features use the full `scout → review → fix → verify` pipeline
136
185
 
137
186
  ## Bottom Line
138
187
 
139
- 1. Technical rigor over social performance - No performative agreement
140
- 2. Systematic review processes - Use code-reviewer agent
141
- 3. Evidence before claims - Verification gates always
188
+ 1. **Scout first** - Edge cases found before review save rework cycles
189
+ 2. Technical rigor over social performance - No performative agreement
190
+ 3. Systematic review processes - Use `code-reviewer` agent via pipeline
191
+ 4. Evidence before claims - Verification gates always
142
192
 
143
- Verify. Question. Then implement. Evidence. Then claim.
193
+ Scout. Verify. Question. Then implement. Evidence. Then claim.