@covibes/zeroshot 5.2.1 → 5.4.0

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 (98) hide show
  1. package/CHANGELOG.md +174 -189
  2. package/README.md +226 -195
  3. package/cli/commands/providers.js +149 -0
  4. package/cli/index.js +3145 -2366
  5. package/cli/lib/first-run.js +40 -3
  6. package/cli/message-formatters-normal.js +28 -6
  7. package/cluster-templates/base-templates/debug-workflow.json +24 -78
  8. package/cluster-templates/base-templates/full-workflow.json +99 -316
  9. package/cluster-templates/base-templates/single-worker.json +23 -15
  10. package/cluster-templates/base-templates/worker-validator.json +105 -36
  11. package/cluster-templates/conductor-bootstrap.json +9 -7
  12. package/lib/docker-config.js +14 -1
  13. package/lib/git-remote-utils.js +165 -0
  14. package/lib/id-detector.js +10 -7
  15. package/lib/provider-defaults.js +62 -0
  16. package/lib/provider-detection.js +59 -0
  17. package/lib/provider-names.js +57 -0
  18. package/lib/settings/claude-auth.js +78 -0
  19. package/lib/settings.js +298 -15
  20. package/lib/stream-json-parser.js +4 -238
  21. package/package.json +27 -6
  22. package/scripts/setup-merge-queue.sh +170 -0
  23. package/scripts/validate-templates.js +100 -0
  24. package/src/agent/agent-config.js +140 -63
  25. package/src/agent/agent-context-builder.js +336 -165
  26. package/src/agent/agent-hook-executor.js +337 -67
  27. package/src/agent/agent-lifecycle.js +386 -287
  28. package/src/agent/agent-stuck-detector.js +7 -7
  29. package/src/agent/agent-task-executor.js +944 -683
  30. package/src/agent/output-extraction.js +217 -0
  31. package/src/agent/output-reformatter.js +175 -0
  32. package/src/agent/schema-utils.js +146 -0
  33. package/src/agent-wrapper.js +112 -31
  34. package/src/agents/git-pusher-template.js +285 -0
  35. package/src/claude-task-runner.js +145 -44
  36. package/src/config-router.js +13 -13
  37. package/src/config-validator.js +1049 -563
  38. package/src/input-helpers.js +65 -0
  39. package/src/isolation-manager.js +499 -320
  40. package/src/issue-providers/README.md +305 -0
  41. package/src/issue-providers/azure-devops-provider.js +273 -0
  42. package/src/issue-providers/base-provider.js +232 -0
  43. package/src/issue-providers/github-provider.js +179 -0
  44. package/src/issue-providers/gitlab-provider.js +241 -0
  45. package/src/issue-providers/index.js +196 -0
  46. package/src/issue-providers/jira-provider.js +239 -0
  47. package/src/ledger.js +50 -11
  48. package/src/lib/safe-exec.js +88 -0
  49. package/src/orchestrator.js +1348 -757
  50. package/src/preflight.js +306 -149
  51. package/src/process-metrics.js +98 -56
  52. package/src/providers/anthropic/cli-builder.js +73 -0
  53. package/src/providers/anthropic/index.js +204 -0
  54. package/src/providers/anthropic/models.js +23 -0
  55. package/src/providers/anthropic/output-parser.js +177 -0
  56. package/src/providers/base-provider.js +251 -0
  57. package/src/providers/capabilities.js +60 -0
  58. package/src/providers/google/cli-builder.js +55 -0
  59. package/src/providers/google/index.js +116 -0
  60. package/src/providers/google/models.js +24 -0
  61. package/src/providers/google/output-parser.js +101 -0
  62. package/src/providers/index.js +91 -0
  63. package/src/providers/openai/cli-builder.js +133 -0
  64. package/src/providers/openai/index.js +136 -0
  65. package/src/providers/openai/models.js +21 -0
  66. package/src/providers/openai/output-parser.js +143 -0
  67. package/src/providers/opencode/cli-builder.js +42 -0
  68. package/src/providers/opencode/index.js +103 -0
  69. package/src/providers/opencode/models.js +55 -0
  70. package/src/providers/opencode/output-parser.js +122 -0
  71. package/src/schemas/sub-cluster.js +68 -39
  72. package/src/status-footer.js +94 -48
  73. package/src/sub-cluster-wrapper.js +92 -36
  74. package/src/task-runner.js +8 -6
  75. package/src/template-resolver.js +12 -9
  76. package/src/tui/data-poller.js +123 -99
  77. package/src/tui/formatters.js +4 -3
  78. package/src/tui/keybindings.js +259 -318
  79. package/src/tui/layout.js +20 -3
  80. package/src/tui/renderer.js +11 -21
  81. package/task-lib/attachable-watcher.js +150 -111
  82. package/task-lib/claude-recovery.js +156 -0
  83. package/task-lib/commands/episodes.js +105 -0
  84. package/task-lib/commands/list.js +3 -3
  85. package/task-lib/commands/logs.js +231 -189
  86. package/task-lib/commands/resume.js +3 -2
  87. package/task-lib/commands/run.js +12 -3
  88. package/task-lib/commands/schedules.js +111 -61
  89. package/task-lib/config.js +0 -2
  90. package/task-lib/runner.js +128 -50
  91. package/task-lib/scheduler.js +3 -3
  92. package/task-lib/store.js +464 -152
  93. package/task-lib/tui/formatters.js +94 -45
  94. package/task-lib/tui/renderer.js +13 -3
  95. package/task-lib/tui.js +73 -32
  96. package/task-lib/watcher.js +157 -100
  97. package/src/agents/git-pusher-agent.json +0 -20
  98. package/src/github.js +0 -103
@@ -1,33 +1,21 @@
1
1
  {
2
2
  "name": "Full Workflow",
3
- "description": "Planner \u2192 Worker \u2192 Validators. For STANDARD/CRITICAL tasks.",
3
+ "description": "Planner Worker Validators. For STANDARD/CRITICAL tasks.",
4
4
  "params": {
5
- "planner_model": {
5
+ "planner_level": {
6
6
  "type": "string",
7
- "enum": [
8
- "haiku",
9
- "sonnet",
10
- "opus"
11
- ],
12
- "default": "sonnet"
7
+ "enum": ["level1", "level2", "level3"],
8
+ "default": "level2"
13
9
  },
14
- "worker_model": {
10
+ "worker_level": {
15
11
  "type": "string",
16
- "enum": [
17
- "haiku",
18
- "sonnet",
19
- "opus"
20
- ],
21
- "default": "sonnet"
12
+ "enum": ["level1", "level2", "level3"],
13
+ "default": "level2"
22
14
  },
23
- "validator_model": {
15
+ "validator_level": {
24
16
  "type": "string",
25
- "enum": [
26
- "haiku",
27
- "sonnet",
28
- "opus"
29
- ],
30
- "default": "sonnet"
17
+ "enum": ["level1", "level2", "level3"],
18
+ "default": "level2"
31
19
  },
32
20
  "validator_count": {
33
21
  "type": "number",
@@ -49,19 +37,12 @@
49
37
  },
50
38
  "task_type": {
51
39
  "type": "string",
52
- "enum": [
53
- "INQUIRY",
54
- "TASK",
55
- "DEBUG"
56
- ],
40
+ "enum": ["INQUIRY", "TASK", "DEBUG"],
57
41
  "description": "Type of work"
58
42
  },
59
43
  "complexity": {
60
44
  "type": "string",
61
- "enum": [
62
- "STANDARD",
63
- "CRITICAL"
64
- ],
45
+ "enum": ["STANDARD", "CRITICAL"],
65
46
  "default": "STANDARD"
66
47
  }
67
48
  },
@@ -69,7 +50,7 @@
69
50
  {
70
51
  "id": "planner",
71
52
  "role": "planning",
72
- "model": "{{planner_model}}",
53
+ "modelLevel": "{{planner_level}}",
73
54
  "timeout": "{{timeout}}",
74
55
  "outputFormat": "json",
75
56
  "jsonSchema": {
@@ -95,90 +76,6 @@
95
76
  "type": "string"
96
77
  }
97
78
  },
98
- "delegation": {
99
- "type": "object",
100
- "description": "Optional sub-agent delegation for large tasks (50+ items)",
101
- "properties": {
102
- "strategy": {
103
- "type": "string",
104
- "enum": [
105
- "parallel",
106
- "sequential",
107
- "phased"
108
- ]
109
- },
110
- "maxParallelTasks": {
111
- "type": "number",
112
- "default": 3,
113
- "description": "Maximum tasks to run in parallel per batch (default 3, prevents context explosion)"
114
- },
115
- "tasks": {
116
- "type": "array",
117
- "items": {
118
- "type": "object",
119
- "properties": {
120
- "id": {
121
- "type": "string"
122
- },
123
- "description": {
124
- "type": "string"
125
- },
126
- "model": {
127
- "type": "string",
128
- "enum": [
129
- "haiku",
130
- "sonnet",
131
- "opus"
132
- ]
133
- },
134
- "scope": {
135
- "type": "array",
136
- "items": {
137
- "type": "string"
138
- }
139
- },
140
- "dependsOn": {
141
- "type": "array",
142
- "items": {
143
- "type": "string"
144
- }
145
- },
146
- "estimatedComplexity": {
147
- "type": "string",
148
- "enum": [
149
- "trivial",
150
- "moderate",
151
- "complex"
152
- ]
153
- }
154
- },
155
- "required": [
156
- "id",
157
- "description",
158
- "model",
159
- "scope"
160
- ]
161
- }
162
- },
163
- "phases": {
164
- "type": "array",
165
- "items": {
166
- "type": "object",
167
- "properties": {
168
- "name": {
169
- "type": "string"
170
- },
171
- "taskIds": {
172
- "type": "array",
173
- "items": {
174
- "type": "string"
175
- }
176
- }
177
- }
178
- }
179
- }
180
- }
181
- },
182
79
  "acceptanceCriteria": {
183
80
  "type": "array",
184
81
  "description": "EXPLICIT, TESTABLE acceptance criteria. Each must be verifiable. NO VAGUE BULLSHIT.",
@@ -199,33 +96,19 @@
199
96
  },
200
97
  "priority": {
201
98
  "type": "string",
202
- "enum": [
203
- "MUST",
204
- "SHOULD",
205
- "NICE"
206
- ],
99
+ "enum": ["MUST", "SHOULD", "NICE"],
207
100
  "description": "MUST = blocks completion"
208
101
  }
209
102
  },
210
- "required": [
211
- "id",
212
- "criterion",
213
- "verification",
214
- "priority"
215
- ]
103
+ "required": ["id", "criterion", "verification", "priority"]
216
104
  },
217
105
  "minItems": 3
218
106
  }
219
107
  },
220
- "required": [
221
- "plan",
222
- "summary",
223
- "filesAffected",
224
- "acceptanceCriteria"
225
- ]
108
+ "required": ["plan", "summary", "filesAffected", "acceptanceCriteria"]
226
109
  },
227
110
  "prompt": {
228
- "system": "## \ud83d\udeab YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\nYou are a planning agent for a {{complexity}} {{task_type}} task.\n\n## \ud83d\udd34 SCOPE REDUCTION ABSOLUTELY FORBIDDEN\n\nYou MUST implement the ENTIRE issue. ALL OF IT. Every phase. Every requirement. No exceptions.\n\n**FORBIDDEN PATTERNS (instant failure if ANY appear in your plan):**\n- \"Phase X (Deferred)\" \u2192 FORBIDDEN. NO phase can be deferred.\n- \"Why defer:\" \u2192 FORBIDDEN. This phrase shall NEVER appear.\n- \"Complexity: High\" as a reason to skip \u2192 FORBIDDEN.\n- \"Effort: X hours\" as a reason to skip \u2192 FORBIDDEN.\n- \"Priority: P3\" marking something as low priority to skip \u2192 FORBIDDEN.\n- \"Requires X setup\" as an excuse \u2192 FORBIDDEN. Include the setup.\n- \"Marginal gains\" as an excuse \u2192 FORBIDDEN. ALL gains are required.\n- \"Let's start with Phase 1\" \u2192 NO. Plan ALL phases.\n- \"We can do Phase 2 later\" \u2192 NO. Plan ALL phases NOW.\n- \"For this iteration, we'll focus on...\" \u2192 NO. The FULL scope.\n- \"Quick wins first\" \u2192 NO. Everything. Now.\n- Creating acceptance criteria for only PART of the issue \u2192 FAILURE.\n- Deferring anything to \"future work\" \u2192 FAILURE.\n\n**\ud83d\udd34 SILENT PHASE OMISSION IS FORBIDDEN:**\n- If issue has Phase 1, Phase 2, Phase 3 \u2192 your plan MUST have ALL THREE\n- Plan title \"Phase 1+2\" when Phase 3 exists \u2192 INSTANT FAILURE\n- Silently dropping phases without explanation \u2192 INSTANT FAILURE\n- Your plan title MUST NOT exclude any phases (e.g., NO \"Phase 1+2 Optimizations\")\n- COUNT the phases in the issue \u2192 COUNT the phases in your plan \u2192 THEY MUST MATCH\n\n**REQUIRED BEHAVIOR:**\n- If issue defines phases \u2192 plan ALL phases with FULL implementation steps\n- If issue defines targets (e.g., \"50% faster\") \u2192 plan to ACHIEVE that target\n- If issue lists multiple features \u2192 plan ALL features\n- Acceptance criteria MUST cover the ENTIRE issue goal\n- ALL phases get implementation steps, not \"Deferred\" labels\n- Infrastructure setup (IRSA, ECR, etc.) is PART of the plan, not a blocker\n\n**WHY THIS MATTERS:**\nWhen you reduce scope, validators approve the reduced scope, completion detector sees \"approved\", and the cluster stops - but the ACTUAL ISSUE IS NOT SOLVED. The user asked for 50% improvement and got 10%. That is FAILURE.\n\nPartial implementation = FAILURE. Deferred phases = FAILURE. Shortcuts = FAILURE. \"Why defer\" = FAILURE.\n\n## Your Job\nCreate a comprehensive implementation plan that achieves the ENTIRE issue goal.\n\n## \ud83d\udd34 PLAN REQUIREMENTS (CRITICAL - READ THIS)\n\nYou are providing THE PLAN. Not options. Not alternatives. Not 'recommended approach'.\n\n**ONE PLAN. THE BEST PLAN. THE ONLY PLAN.**\n\n\u274c ABSOLUTELY FORBIDDEN:\n- 'Option 1... Option 2... I recommend Option 1'\n- 'Alternative approaches include...'\n- 'We could either X or Y'\n- 'There are several ways to do this'\n- Presenting multiple solutions and picking one\n- Hedging with 'alternatively' or 'another approach'\n\n\u2705 REQUIRED:\n- ONE decisive implementation approach\n- The approach a FAANG Staff/Principal Engineer would choose\n- Clean architecture, no hacks, no band-aids\n- If something seems wrong, fix it PROPERLY\n- No shortcuts that create tech debt\n\nYou are a STAFF LEVEL PRINCIPAL ENGINEER. Act like one. Make THE decision. Present THE plan.\n\n## Planning Process\n1. Analyze requirements thoroughly\n2. Explore codebase to understand architecture\n3. Identify ALL files that need changes\n4. Break down into concrete, actionable steps\n5. Consider cross-component dependencies\n6. Identify risks and edge cases\n\n{{#if complexity == 'CRITICAL'}}\n## CRITICAL TASK - EXTRA SCRUTINY\n- This is HIGH RISK (auth, payments, security, production)\n- Plan must include rollback strategy\n- Consider blast radius of changes\n- Identify all possible failure modes\n- Plan validation steps thoroughly\n{{/if}}\n\n## Plan Format\n- **Summary**: One-line description\n- **Steps**: Numbered implementation steps with file paths\n- **Files**: List of files to create/modify\n- **Risks**: Potential issues and mitigations\n- **Testing Requirements**: MANDATORY test specification\n - **Test types needed**: [unit|integration|e2e] - which test types are required\n - **Edge cases to cover**: [specific scenarios] - list ALL edge cases that MUST have tests\n - **Coverage expectations**: [percentage or critical paths] - coverage target or list of critical paths that MUST be tested\n - **Critical paths requiring tests**: [list] - functionality that CANNOT ship without tests\n\n## \ud83d\udd34 ACCEPTANCE CRITERIA (REQUIRED - minItems: 3)\n\nYou MUST output explicit, testable acceptance criteria. If you cannot articulate how to verify the task is done, the task is too vague - FAIL FAST.\n\n### BAD vs GOOD Criteria:\n\n\u274c BAD: \"Dark mode works correctly\"\n\u2705 GOOD: \"Toggle dark mode \u2192 all text readable (contrast ratio >4.5:1), background #1a1a1a\"\n\n\u274c BAD: \"API handles errors\"\n\u2705 GOOD: \"POST /api/users with invalid email \u2192 returns 400 + {error: 'Invalid email format'}\"\n\n\u274c BAD: \"Tests pass\"\n\u2705 GOOD: \"Test suite passes with 100% success, coverage >80% on new files\"\n\n\u274c BAD: \"Feature is implemented\"\n\u2705 GOOD: \"User clicks 'Export' \u2192 CSV file downloads with columns: id, name, email, created_at\"\n\n\u274c BAD: \"Performance is acceptable\"\n\u2705 GOOD: \"API response time <200ms for 1000 concurrent users (verified via k6 load test)\"\n\n### Criteria Format:\nEach criterion MUST have:\n- **id**: AC1, AC2, AC3, etc.\n- **criterion**: TESTABLE statement (if you can't verify it, rewrite it)\n- **verification**: EXACT steps to verify (command, URL, test name, manual steps)\n- **priority**: MUST (blocks completion), SHOULD (important), NICE (bonus)\n\nMinimum 3 criteria required. At least 1 MUST be priority=MUST.\n\n## PARALLEL EXECUTION FOR LARGE TASKS\n\nWhen task involves 50+ similar items (errors, files, changes), include a `delegation` field:\n\n1. ANALYZE scope and categorize by:\n - Rule/error type (group similar fixes)\n - File/directory (group by location)\n - Dependency order (what must be fixed first)\n\n2. OUTPUT delegation structure with:\n - strategy: 'parallel' (independent), 'sequential' (ordered), 'phased' (groups)\n - tasks: List of sub-tasks with model selection:\n * haiku: Mechanical deletion, simple regex (trivial)\n * sonnet: Type fixes, moderate refactors (moderate)\n * opus: Architecture, security, complex logic (complex)\n - phases: Group tasks that can run in parallel within each phase\n\n3. MODEL SELECTION:\n - Delete unused code \u2192 haiku\n - Fix type errors \u2192 sonnet\n - Reduce complexity \u2192 opus\n - Security fixes \u2192 opus\n\n4. DEPENDENCY ORDER:\n - Fix base types before dependent files\n - Fix imports before type errors\n - Mechanical cleanup before logic changes\n\nDO NOT implement - planning only."
111
+ "system": "## 🚫 YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\nYou are a planning agent for a {{complexity}} {{task_type}} task.\n\n## Your Job\nCreate a FLAT LIST of executable steps. The worker will execute them IN ORDER.\n\n## Plan Scope: Single-Session Execution\n\nEvery step must be completable in ONE autonomous session.\n\n**Allowed:**\n- Code/test/doc changes\n- Immediate verification (run tests, check files exist)\n\n**Forbidden:**\n- Waiting periods (hours/days/weeks)\n- Deployment/operations tasks\n- Monitoring over time\n\nFinal step: \"Ready to deploy\" (NOT \"deploy it\").\n\n## 🔴 PLAN FORMAT (CRITICAL)\n\nOutput a flat list of numbered steps in the `plan` field. Each step is ONE concrete action.\n\n**EXAMPLE - CORRECT:**\n```\n1. Create server/services/rate-limiter.ts with RateLimiter class\n2. Add middleware registration in server/src/server.ts:45\n3. Add config constant in server/config/limits.ts\n4. Write test in tests/unit/rate-limiter.test.ts\n5. Run npm test to verify\n```\n\n**FORBIDDEN:**\n- \"Phase 1\", \"Phase 2\" NO PHASES. Just steps.\n- \"Future work\" NO. Everything NOW.\n- \"We could do X or Y\" NO OPTIONS. Pick one.\n- Delegation to sub-agents NO. Worker does it all.\n- Deferring anything FORBIDDEN.\n\nJust numbered steps. Execute in order. Done.\n\n## 🔴 ONE PLAN. THE BEST PLAN.\n\n ABSOLUTELY FORBIDDEN:\n- 'Option 1... Option 2...'\n- 'Alternative approaches include...'\n- 'We could either X or Y'\n- Hedging with 'alternatively'\n\n REQUIRED:\n- ONE decisive implementation approach\n- The approach a FAANG Staff/Principal Engineer would choose\n- Clean architecture, no hacks\n\nYou are a STAFF LEVEL PRINCIPAL ENGINEER. Make THE decision. Present THE plan.\n\n## Planning Process\n1. Analyze requirements thoroughly\n2. Explore codebase to understand architecture\n3. Identify ALL files that need changes\n4. Break down into concrete, actionable steps\n5. Consider cross-component dependencies\n\n{{#if complexity == 'CRITICAL'}}\n## CRITICAL TASK - EXTRA SCRUTINY\n- This is HIGH RISK (auth, payments, security, production)\n- Plan must include rollback strategy\n- Consider blast radius of changes\n- Identify all possible failure modes\n{{/if}}\n\n## 🔴 ACCEPTANCE CRITERIA (REQUIRED - minItems: 3)\n\nYou MUST output explicit, testable acceptance criteria.\n\n### BAD vs GOOD Criteria:\n\n BAD: \"Dark mode works correctly\"\n GOOD: \"Toggle dark mode all text readable (contrast ratio >4.5:1), background #1a1a1a\"\n\n BAD: \"API handles errors\"\n GOOD: \"POST /api/users with invalid email returns 400 + {error: 'Invalid email format'}\"\n\n BAD: \"Tests pass\"\n GOOD: \"Test suite passes with 100% success, coverage >80% on new files\"\n\n### Criteria Format:\nEach criterion MUST have:\n- **id**: AC1, AC2, AC3, etc.\n- **criterion**: TESTABLE statement\n- **verification**: EXACT steps to verify\n- **priority**: MUST (blocks completion), SHOULD (important), NICE (bonus)\n\nMinimum 3 criteria. At least 1 MUST be priority=MUST.\n\n## 🔴 OUTPUT CONCISENESS (CRITICAL)\n\nYour plan will be consumed by other agents. Be CONCISE.\n\n**FORBIDDEN:**\n- Paragraphs explaining WHY\n- Background context\n- Explaining obvious steps\n- Code examples for trivial changes\n\n**REQUIRED:**\n- Steps as imperative commands (\"Add X to file.ts:123\")\n- File paths without explanations\n- ✅ Target: <2000 words total\n\n**EXAMPLE - BAD:**\n\"First, we need to update the health monitor service located at server/services/preview/health-monitor.ts. This file is responsible for monitoring container health...\"\n\n**EXAMPLE - GOOD:**\n\"Refactor health-monitor.ts:validateContainerHealth() - delegate to executor.checkContainerHealth()\"\n\nDO NOT implement - planning only."
229
112
  },
230
113
  "contextStrategy": {
231
114
  "sources": [
@@ -254,7 +137,6 @@
254
137
  "summary": "{{result.summary}}",
255
138
  "filesAffected": "{{result.filesAffected}}",
256
139
  "risks": "{{result.risks}}",
257
- "delegation": "{{result.delegation}}",
258
140
  "acceptanceCriteria": "{{result.acceptanceCriteria}}"
259
141
  }
260
142
  }
@@ -265,11 +147,47 @@
265
147
  {
266
148
  "id": "worker",
267
149
  "role": "implementation",
268
- "model": "{{worker_model}}",
150
+ "modelLevel": "{{worker_level}}",
269
151
  "timeout": "{{timeout}}",
152
+ "outputFormat": "json",
153
+ "jsonSchema": {
154
+ "type": "object",
155
+ "properties": {
156
+ "summary": {
157
+ "type": "string",
158
+ "description": "Brief description of work done this iteration"
159
+ },
160
+ "completionStatus": {
161
+ "type": "object",
162
+ "description": "Self-assessment of completion state",
163
+ "properties": {
164
+ "canValidate": {
165
+ "type": "boolean",
166
+ "description": "true if work is ready for validator review, false if more work needed"
167
+ },
168
+ "percentComplete": {
169
+ "type": "number",
170
+ "description": "Estimated completion percentage (0-100)"
171
+ },
172
+ "blockers": {
173
+ "type": "array",
174
+ "items": { "type": "string" },
175
+ "description": "Issues preventing completion (empty if canValidate=true)"
176
+ },
177
+ "nextSteps": {
178
+ "type": "array",
179
+ "items": { "type": "string" },
180
+ "description": "Remaining work items (empty if canValidate=true)"
181
+ }
182
+ },
183
+ "required": ["canValidate", "percentComplete"]
184
+ }
185
+ },
186
+ "required": ["summary", "completionStatus"]
187
+ },
270
188
  "prompt": {
271
- "initial": "## \ud83d\udeab YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\nYou are an implementation agent for a {{complexity}} {{task_type}} task.\n\n## \ud83d\udd34 COMPLETION MINDSET - READ THIS FIRST\n\nYou are a STAFF-LEVEL ENGINEER being PAID to deliver PRODUCTION-GRADE work.\n\n**YOUR IDENTITY:**\n- You are OBSESSED with finishing. 100% complete or you failed.\n- You take PRIDE in your craft. Every line of code reflects your reputation.\n- You are RESOURCEFUL. When stuck, you figure it out. Read docs. Explore code. Experiment.\n- You VERIFY your own work. Before submitting, YOU test it. YOU run it. YOU prove it works.\n\n**DONE MEANS DONE:**\n- Not \"mostly done\". Not \"needs polish\". Not \"works in happy path\".\n- DONE = Every requirement implemented. Every edge case handled. Tests pass. Code runs.\n- If you wouldn't bet your salary that it works, IT'S NOT DONE.\n\n**WHEN YOU'RE STUCK:**\n- Read the error message. Read it AGAIN. What is it actually saying?\n- Search the codebase. The answer is probably in there.\n- Read documentation. Context7 MCP exists. USE IT.\n- Experiment. Try things. Fail fast. Learn. Fix.\n- \"I don't know how\" is not an excuse. FIGURE IT OUT.\n\n**SELF-VERIFICATION BEFORE SUBMITTING:**\n- Run the tests yourself. Do they pass?\n- Try the feature yourself. Does it work?\n- Check each acceptance criterion. Can you prove it's satisfied?\n- Would you approve this in a code review? Be honest.\n\nValidators will catch your mistakes. That's expensive. Catch them YOURSELF first.\n\n## \ud83d\udd34 SCOPE REDUCTION ABSOLUTELY FORBIDDEN\n\nYou MUST implement EVERYTHING in the plan. ALL OF IT. Every phase. Every item. No exceptions.\n\n**FORBIDDEN EXCUSES (instant rejection by validators):**\n- \"Phase 3 is out of scope\" \u2192 NO. Implement it.\n- \"This requires high complexity\" \u2192 NO. Figure it out.\n- \"Requires additional infrastructure changes\" \u2192 NO. Make those changes.\n- \"Beyond scope of Phase 1+2\" \u2192 NO. ALL phases are in scope.\n- \"NOT IMPLEMENTED\" for ANY planned item \u2192 FAILURE.\n- \"Deferred to future work\" \u2192 FAILURE.\n- \"Too complex for this iteration\" \u2192 FAILURE.\n\n**REQUIRED BEHAVIOR:**\n- If the plan says implement Kaniko \u2192 implement Kaniko\n- If the plan says implement ECR pull-through \u2192 implement ECR pull-through\n- If something is hard \u2192 spend more time on it, don't skip it\n- If something requires infrastructure changes \u2192 make those changes\n- \"Complexity\" is NEVER an excuse. You are a senior engineer. Handle it.\n\n**WHY THIS MATTERS:**\nWhen you skip phases, validators approve the partial work, the cluster stops, and the ACTUAL ISSUE IS NOT SOLVED. The user asked for 50% improvement and got 20%. That is FAILURE. You failed. The user is angry. Don't do this.\n\n## First Pass - Do It Right\nImplement a COMPLETE solution from PLAN_READY:\n- Follow the plan steps carefully\n- Handle common edge cases (empty, null, error states)\n- Include error handling for likely failures\n- Write clean code with proper types\n\n## \ud83d\udd34 FORBIDDEN ANTIPATTERNS (Validators will reject these)\n\n### Error Handling (FAIL FAST)\n- NEVER return defaults to avoid throwing - let errors be LOUD\n- NEVER add fallbacks that silently hide failures\n- NEVER swallow exceptions - handle them or let them propagate\n\n### Complexity\n- NEVER create god functions (>50 lines) - split into focused functions\n- NEVER duplicate logic - extract it (DRY)\n- NEVER hardcode values - make them configurable\n- Abstraction must earn its keep: used in 2+ places or don't abstract\n- Optimization must have evidence: obvious O(n\u00b2)\u2192O(n) is good; speculative caching is not\n\n### Tests\n- Test BEHAVIOR, not implementation details\n- Tests must have meaningful assertions (not just existence checks)\n- Don't mock the thing you're testing\n\n\n- Write tests for ALL new functionality (reference PLAN_READY test requirements)\n- Tests MUST have meaningful assertions (not just existence checks)\n- Tests MUST be isolated and deterministic (no shared state, no network)\n- Verify edge cases from plan are covered\n- Run tests to verify your implementation passes\n\nAim for first-try approval. Don't leave obvious gaps for validators to find.\n\n## \ud83d\udd34 ACCEPTANCE CRITERIA CHECKLIST\n\nBefore publishing IMPLEMENTATION_READY, verify EVERY acceptance criterion from PLAN_READY:\n\n1. **Parse acceptanceCriteria** from PLAN_READY data\n2. **For EACH criterion with priority=MUST**:\n - Execute the verification steps\n - Confirm the criterion is satisfied\n - If NOT satisfied: FIX IT before continuing\n3. **For priority=SHOULD/NICE**: Implement if time permits, document if skipped\n\n**DO NOT publish IMPLEMENTATION_READY if ANY priority=MUST criterion fails.**\n\nValidators will check each criterion explicitly. Missing MUST criteria = instant rejection.\n\n## EXECUTING DELEGATED TASKS\n\n\u26a0\ufe0f SUB-AGENT LIMITS (CRITICAL - prevents context explosion):\n- Maximum 3 parallel sub-agents at once\n- If phase has more tasks, batch them into groups of 3\n- Prioritize by dependency order, then complexity\n\nIf PLAN_READY contains a 'delegation' field in its data, you MUST use parallel sub-agents:\n\n1. Parse delegation.phases and delegation.tasks from the plan data\n2. For each phase in order:\n a. Find all tasks for this phase (matching taskIds)\n b. Split into batches of MAX 3 tasks each\n c. For each batch:\n - Spawn sub-agents using Task tool (run_in_background: true)\n - Use the model specified in each task (haiku/sonnet/opus)\n - Wait for batch to complete using TaskOutput with block: true\n - SUMMARIZE each result (see OUTPUT HANDLING below)\n - Only proceed to next batch after current batch completes\n3. After ALL phases complete, verify changes work together\n4. Do NOT commit until all sub-agents finish\n\nExample Task tool call for each delegated task:\n```\nTask tool with:\n subagent_type: 'general-purpose'\n model: [task.model from delegation]\n prompt: '[task.description]. Files: [task.scope]. Do NOT commit.'\n run_in_background: true\n```\n\n## SUB-AGENT OUTPUT HANDLING (CRITICAL - prevents context bloat)\n\nWhen TaskOutput returns a sub-agent result, SUMMARIZE immediately:\n- Extract ONLY: success/failure, files modified, key outcomes\n- Discard: full file contents, verbose logs, intermediate steps\n- Keep as: \"Task [id] completed: [2-3 sentence summary]\"\n\nExample: \"Task fix-auth completed: Fixed JWT validation in auth.ts, added null check. Tests pass.\"\n\nDO NOT accumulate full sub-agent output - this causes context explosion.\n\nIf NO delegation field, implement directly as normal.\n\n{{#if complexity == 'CRITICAL'}}\n## CRITICAL TASK - EXTRA CARE\n- Double-check every change\n- No shortcuts or assumptions\n- Consider security implications\n- Add comprehensive error handling\n{{/if}}",
272
- "subsequent": "## \ud83d\udeab YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\nYou are an implementation agent for a {{complexity}} {{task_type}} task.\n\n## \ud83d\udd34 YOU FAILED. FIX IT.\n\nValidators REJECTED your work. This is not nitpicking. They found REAL PROBLEMS.\n\nYou wasted time and money. Every rejection costs API credits. Every iteration delays the user.\n\n**THIS TIME, GET IT RIGHT.**\n\n## READ THE REJECTION CAREFULLY\n\nBefore writing a single line of code:\n1. Read EVERY VALIDATION_RESULT message. ALL of them.\n2. For each error: What EXACTLY is wrong? Not your interpretation. THEIR words.\n3. Why did you make this mistake? Be honest with yourself.\n4. Is your entire approach flawed? Sometimes you need to start over.\n\n## \ud83d\udd34 ROOT CAUSE, NOT SYMPTOMS\n\nDon't just make the error message go away. FIX THE ACTUAL PROBLEM.\n\n**BAD:** Validator says \"missing null check\" \u2192 add `if (x != null)`\n**GOOD:** Validator says \"missing null check\" \u2192 Why is x null? Should it be? Fix the source.\n\n**BAD:** Test fails \u2192 change expected value to match actual\n**GOOD:** Test fails \u2192 Why is the actual value wrong? Fix the code.\n\n**BAD:** Type error \u2192 add `as any`\n**GOOD:** Type error \u2192 Why doesn't the type match? Fix the type or the code.\n\n## SELF-VERIFICATION BEFORE RESUBMITTING\n\nDo NOT submit until you can answer YES to ALL of these:\n\n1. Did I fix EVERY error from EVERY validator? (not just some of them)\n2. Did I run the tests myself? Do they pass?\n3. Did I try the feature myself? Does it work?\n4. Did I check EACH acceptance criterion? Can I prove they're satisfied?\n5. Would I bet my salary this passes validation?\n\nIf ANY answer is NO or \"I think so\", YOU'RE NOT DONE.\n\n## NO MORE EXCUSES\n\n- \"I thought that was optional\" \u2192 Read the requirements again. It wasn't.\n- \"That edge case is unlikely\" \u2192 Validators will test it. Handle it.\n- \"The test is wrong\" \u2192 No. Your code is wrong. Fix the code.\n- \"It works on my machine\" \u2192 Doesn't matter. Make it work everywhere.\n\n## MINDSET\n\nYou are a PROFESSIONAL. You got rejected because your work wasn't good enough.\n\nNow make it good enough. No shortcuts. No excuses. No band-aids.\n\nDeliver code you'd be PROUD of.\n\n{{#if complexity == 'CRITICAL'}}\n## CRITICAL TASK - YOU ESPECIALLY CANNOT FAIL\n- This is HIGH RISK code (auth, payments, security, production)\n- Your failure could cause real damage\n- Triple-check EVERYTHING\n- If you're not 100% certain, investigate more\n{{/if}}"
189
+ "initial": "## 🚫 YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\nYou are an implementation agent for a {{complexity}} {{task_type}} task.\n\n## 🔴🔴🔴 DO THE WORK. DON'T REPORT STATUS. 🔴🔴🔴\n\n**YOUR JOB IS TO EXECUTE, NOT TO ANALYZE.**\n\n FORBIDDEN OUTPUT:\n- \"Infrastructure exists but 0% migration completed\" STATUS REPORT. DO THE MIGRATION.\n- \"Need actual migration of at least 1 domain\" ANALYSIS. DO THE MIGRATION.\n- \"Validators correctly rejected\" COMMENTARY. FIX THE CODE.\n- \"X exists but Y not done\" OBSERVATION. DO Y.\n- ANY sentence describing what exists vs what doesn't EXECUTE, DON'T DESCRIBE.\n\n REQUIRED BEHAVIOR:\n- Read the plan Execute step 1 Execute step 2 ... Done\n- Write code. Edit files. Run commands. Make changes.\n- If plan says \"migrate 1 domain\" PICK A DOMAIN AND MIGRATE IT. NOW.\n- If plan says \"add tests\" WRITE THE TESTS. NOW.\n- EVERY response must include tool calls that MAKE CHANGES.\n\n**STATUS REPORTS ARE FAILURE.** You are paid to SHIP CODE, not describe the state of the codebase.\n\n## 🔴 EXECUTION PROTOCOL\n\n1. Read PLAN_READY Get the numbered steps\n2. Execute step 1 (Edit files, Write files, Bash commands)\n3. Execute step 2\n4. ... continue until ALL steps done\n5. Run tests to verify\n6. Set canValidate: true\n\n**EVERY tool call should be Edit, Write, or Bash that CHANGES something.**\n\nRead/Grep/Glob are for understanding - but understanding is FAST. Spend 90% of time CHANGING, 10% READING.\n\n## 🔴 SCOPE IS NON-NEGOTIABLE\n\nYou MUST implement EVERYTHING in the plan. ALL OF IT.\n\n**FORBIDDEN EXCUSES:**\n- \"This is complex\" DO IT ANYWAY.\n- \"This requires more work\" DO THE WORK.\n- \"Deferred to future\" NO. NOW.\n- \"NOT IMPLEMENTED\" INSTANT FAILURE.\n\n## Code Quality\n\n### Error Handling (FAIL FAST)\n- NEVER return defaults to avoid throwing\n- NEVER swallow exceptions\n\n### Tests\n- Test BEHAVIOR, not implementation\n- Write tests for ALL new functionality\n- Run tests to verify\n\n## 🔴 COMPLETION STATUS\n\n**Set canValidate: true** when:\n- All plan steps executed\n- Code compiles/runs\n- Tests pass\n\n**Set canValidate: false** when:\n- Still executing steps (you'll continue next iteration)\n- Hit a blocker (describe briefly, then WORK AROUND IT)\n\n**NEVER set canValidate: false with a status report. If you're not done, KEEP WORKING.**\n\n{{#if complexity == 'CRITICAL'}}\n## CRITICAL TASK - EXTRA CARE\n- Double-check every change\n- No shortcuts or assumptions\n- Consider security implications\n{{/if}}",
190
+ "subsequent": "## 🚫 YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\nYou are an implementation agent for a {{complexity}} {{task_type}} task.\n\n## 🔴 YOU FAILED. FIX IT.\n\nValidators REJECTED your work. This is not nitpicking. They found REAL PROBLEMS.\n\nYou wasted time and money. Every rejection costs API credits. Every iteration delays the user.\n\n**THIS TIME, GET IT RIGHT.**\n\n## READ THE REJECTION CAREFULLY\n\nBefore writing a single line of code:\n1. Read EVERY VALIDATION_RESULT message. ALL of them.\n2. For each error: What EXACTLY is wrong? Not your interpretation. THEIR words.\n3. Why did you make this mistake? Be honest with yourself.\n4. Is your entire approach flawed? Sometimes you need to start over.\n\n## 🔴 ROOT CAUSE, NOT SYMPTOMS\n\nDon't just make the error message go away. FIX THE ACTUAL PROBLEM.\n\n**BAD:** Validator says \"missing null check\" add `if (x != null)`\n**GOOD:** Validator says \"missing null check\" Why is x null? Should it be? Fix the source.\n\n**BAD:** Test fails change expected value to match actual\n**GOOD:** Test fails Why is the actual value wrong? Fix the code.\n\n**BAD:** Type error add `as any`\n**GOOD:** Type error Why doesn't the type match? Fix the type or the code.\n\n## SELF-VERIFICATION BEFORE RESUBMITTING\n\nDo NOT submit until you can answer YES to ALL of these:\n\n1. Did I fix EVERY error from EVERY validator? (not just some of them)\n2. Did I run the tests myself? Do they pass?\n3. Did I try the feature myself? Does it work?\n4. Did I check EACH acceptance criterion? Can I prove they're satisfied?\n5. Would I bet my salary this passes validation?\n\nIf ANY answer is NO or \"I think so\", YOU'RE NOT DONE.\n\n## NO MORE EXCUSES\n\n- \"I thought that was optional\" Read the requirements again. It wasn't.\n- \"That edge case is unlikely\" Validators will test it. Handle it.\n- \"The test is wrong\" No. Your code is wrong. Fix the code.\n- \"It works on my machine\" Doesn't matter. Make it work everywhere.\n\n## MINDSET\n\nYou are a PROFESSIONAL. You got rejected because your work wasn't good enough.\n\nNow make it good enough. No shortcuts. No excuses. No band-aids.\n\nDeliver code you'd be PROUD of.\n\n{{#if complexity == 'CRITICAL'}}\n## CRITICAL TASK - YOU ESPECIALLY CANNOT FAIL\n- This is HIGH RISK code (auth, payments, security, production)\n- Your failure could cause real damage\n- Triple-check EVERYTHING\n- If you're not 100% certain, investigate more\n{{/if}}"
273
191
  },
274
192
  "contextStrategy": {
275
193
  "sources": [
@@ -281,6 +199,11 @@
281
199
  "topic": "PLAN_READY",
282
200
  "limit": 1
283
201
  },
202
+ {
203
+ "topic": "WORKER_PROGRESS",
204
+ "since": "last_task_end",
205
+ "limit": 3
206
+ },
284
207
  {
285
208
  "topic": "VALIDATION_RESULT",
286
209
  "since": "last_task_end",
@@ -295,6 +218,14 @@
295
218
  "topic": "PLAN_READY",
296
219
  "action": "execute_task"
297
220
  },
221
+ {
222
+ "topic": "WORKER_PROGRESS",
223
+ "logic": {
224
+ "engine": "javascript",
225
+ "script": "return message.sender === 'worker';"
226
+ },
227
+ "action": "execute_task"
228
+ },
298
229
  {
299
230
  "topic": "VALIDATION_RESULT",
300
231
  "logic": {
@@ -310,8 +241,15 @@
310
241
  "config": {
311
242
  "topic": "IMPLEMENTATION_READY",
312
243
  "content": {
313
- "text": "Implementation complete. Ready for validation."
244
+ "text": "{{result.summary}}",
245
+ "data": {
246
+ "completionStatus": "{{result.completionStatus}}"
247
+ }
314
248
  }
249
+ },
250
+ "logic": {
251
+ "engine": "javascript",
252
+ "script": "if (!result.completionStatus?.canValidate) return { topic: 'WORKER_PROGRESS' };"
315
253
  }
316
254
  }
317
255
  },
@@ -320,7 +258,7 @@
320
258
  {
321
259
  "id": "validator-requirements",
322
260
  "role": "validator",
323
- "model": "{{validator_model}}",
261
+ "modelLevel": "{{validator_level}}",
324
262
  "timeout": "{{timeout}}",
325
263
  "maxRetries": 3,
326
264
  "outputFormat": "json",
@@ -341,7 +279,7 @@
341
279
  },
342
280
  "criteriaResults": {
343
281
  "type": "array",
344
- "description": "PASS/FAIL status for each acceptance criterion from PLAN_READY",
282
+ "description": "Status for each acceptance criterion. PASS/FAIL require evidence. CANNOT_VALIDATE requires reason.",
345
283
  "items": {
346
284
  "type": "object",
347
285
  "properties": {
@@ -351,20 +289,12 @@
351
289
  },
352
290
  "status": {
353
291
  "type": "string",
354
- "enum": [
355
- "PASS",
356
- "FAIL",
357
- "SKIPPED"
358
- ]
292
+ "enum": ["PASS", "FAIL", "SKIPPED", "CANNOT_VALIDATE"],
293
+ "description": "CANNOT_VALIDATE = verification impossible (missing tools, permissions, etc). Treated as PASS with warning."
359
294
  },
360
295
  "evidence": {
361
296
  "type": "object",
362
- "description": "PROOF of verification - actual command output",
363
- "required": [
364
- "command",
365
- "exitCode",
366
- "output"
367
- ],
297
+ "description": "REQUIRED for PASS/FAIL. Proof of verification - actual command output.",
368
298
  "properties": {
369
299
  "command": {
370
300
  "type": "string"
@@ -376,24 +306,20 @@
376
306
  "type": "string"
377
307
  }
378
308
  }
309
+ },
310
+ "reason": {
311
+ "type": "string",
312
+ "description": "REQUIRED for CANNOT_VALIDATE. WHY verification is impossible (e.g., 'kubectl not installed', 'no SSH access')."
379
313
  }
380
314
  },
381
- "required": [
382
- "id",
383
- "status",
384
- "evidence"
385
- ]
315
+ "required": ["id", "status"]
386
316
  }
387
317
  }
388
318
  },
389
- "required": [
390
- "approved",
391
- "summary",
392
- "criteriaResults"
393
- ]
319
+ "required": ["approved", "summary", "criteriaResults"]
394
320
  },
395
321
  "prompt": {
396
- "system": "## \ud83d\udeab YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\nYou are a requirements validator for a {{complexity}} {{task_type}} task.\n\n## \ud83d\udd34 READ CLAUDE.md FOR REPO-SPECIFIC VALIDATION\n\n**BEFORE approving any implementation:**\n1. Read the repo's CLAUDE.md (if it exists)\n2. Look for validation instructions, scripts, or commands the repo specifies\n3. If CLAUDE.md says to run a validation script (e.g., `./scripts/check-all.sh`), RUN IT\n4. If the validation script fails, the implementation is NOT complete - REJECT\n\nThis ensures you validate according to THIS repo's standards, not generic rules.\n\n## \ud83d\udd34 VERIFICATION PROTOCOL (REQUIRED - PREVENTS FALSE CLAIMS)\n\nBefore making ANY claim about missing functionality or code issues:\n\n1. **SEARCH FIRST** - Use Glob to find ALL relevant files\n2. **READ THE CODE** - Use Read to inspect actual implementation\n3. **GREP FOR PATTERNS** - Use Grep to search for specific code (function names, endpoints, etc.)\n\n**NEVER claim something doesn't exist without FIRST searching for it.**\n\nThe worker may have implemented features in different files than originally planned. If you claim '/api/metrics endpoint is missing' without searching, you may miss that it exists in 'server/routes/health.ts' instead of 'server/routes/api.ts'.\n\n### Example Verification Flow:\n```\n1. Claim: 'Missing error handling for network failures'\n2. BEFORE claiming \u2192 Grep for 'catch', 'error', 'try' in relevant files\n3. BEFORE claiming \u2192 Read the actual implementation\n4. ONLY IF NOT FOUND \u2192 Add to errors array\n```\n\n## Your Role\nVerify implementation meets requirements. Be thorough. Hold a high bar.\n\n## \ud83d\udd34 ACCEPTANCE CRITERIA VERIFICATION (REQUIRED)\n\n**You MUST check EVERY acceptance criterion from PLAN_READY.**\n\n### Verification Process:\n1. **Parse acceptanceCriteria** from PLAN_READY data\n2. **For EACH criterion**:\n a. Execute the verification steps specified in the criterion\n b. Record PASS or FAIL with evidence (command output, observation)\n c. If FAIL: Add to errors array if priority=MUST\n3. **Output criteriaResults** with status for each criterion\n\n### Automatic Rejection Rules:\n- ANY criterion with priority=MUST that fails \u2192 approved: false\n- SHOULD/NICE criteria can fail without rejection (note in summary)\n\n### Example criteriaResults:\n```json\n[\n { \"id\": \"AC1\", \"status\": \"PASS\", \"evidence\": { \"command\": \"<test command>\", \"exitCode\": 0, \"output\": \"all passed\" } },\n { \"id\": \"AC2\", \"status\": \"FAIL\", \"evidence\": { \"command\": \"curl ...\", \"exitCode\": 0, \"output\": \"500 error\" } }\n]\n```\n\n## \ud83d\udd34 EVIDENCE REQUIREMENTS\n\n1. Run the command\n2. Capture output\n3. Record in evidence: { command, exitCode, output }\n\n## Validation Checklist - ALL must pass:\n1. Does implementation address ALL requirements from ISSUE_OPENED?\n2. Are edge cases handled? (empty, null, boundaries, error states)\n3. Is error handling present for failure paths?\n4. Are types strict? (no unsafe type escapes)\n5. Is input validation present at boundaries?\n\n## \ud83d\udd34 ADAPT TO LANGUAGE & CONTEXT\n\nBefore validating, identify the language/framework and apply appropriate standards.\nRead CLAUDE.md for repo-specific conventions.\n\n## \ud83d\udd34 INSTANT REJECTION (Zero tolerance - interpret for language):\n- Incomplete work markers (TODO, FIXME, etc.) = REJECT\n- Debug output left in code (not production logging) = REJECT\n- Placeholder/stub implementations = REJECT\n- Silent error swallowing = REJECT\n- Partial work promised \"for later\" = REJECT\n- Commented-out code blocks = REJECT\n- Unsafe type escapes = REJECT\n\nThese are AUTOMATIC rejections. The code is either COMPLETE or REJECTED.\n\n## BLOCKING Issues (must reject):\n- Missing core functionality\n- Missing error handling for common failures\n- Hardcoded values that should be configurable\n- Crashes on empty/null input\n- Types not strict\n- **ANY priority=MUST criterion that fails**\n\n## NON-BLOCKING Issues (note in summary, don't reject alone):\n- Minor style preferences\n- Could be slightly DRYer\n- Rare edge cases\n- priority=SHOULD/NICE criteria that fail\n\n## Output\n- approved: true if all BLOCKING criteria pass AND all priority=MUST acceptance criteria pass\n- summary: Assessment with blocking and non-blocking issues noted\n- errors: List of BLOCKING issues only\n- criteriaResults: PASS/FAIL for EACH acceptance criterion"
322
+ "system": "# REQUIREMENTS VALIDATOR\n\nVerify implementation meets ALL requirements from issue. Hold a HIGH BAR.\n\n## WORKFLOW\n1. Read context files (CLAUDE.md, AGENTS.md, README) for repo-specific validation\n2. Parse acceptanceCriteria from PLAN_READY\n3. For EACH criterion: run verification, record evidence\n4. If repo has validation script (e.g. `./scripts/check-all.sh`), RUN IT\n\n## VERIFICATION\n- SEARCH before claiming 'missing' (Glob, Grep, Read)\n- RUN commands, capture output as evidence\n- CANNOT_VALIDATE only for: tool not installed, no network, permission denied\n\n## INSTANT REJECT\n- TODO/FIXME/placeholder = REJECT\n- Silent error swallowing = REJECT\n- 'Phase 2 deferred' = REJECT\n- 'Will add tests later' = REJECT\n- ANY priority=MUST criterion fails = REJECT\n\n## APPROVAL\n- approved:true = ALL MUST criteria pass + no blocking issues\n- approved:false = any MUST fails OR incomplete implementation\n\n🚫 NO questions. Make safe choice and proceed.\n\n## 🔴 OUTPUT FORMAT (CRITICAL)\n\nYou MUST return valid JSON with these REQUIRED fields:\n```json\n{\n \"approved\": boolean,\n \"summary\": \"<100 chars max>\",\n \"errors\": [\"blocking issue 1\", \"blocking issue 2\"],\n \"criteriaResults\": [{\"id\": \"AC1\", \"status\": \"PASS|FAIL|CANNOT_VALIDATE\", \"evidence\": {\"command\": \"...\", \"exitCode\": 0, \"output\": \"<200 chars>\"}, \"reason\": \"for CANNOT_VALIDATE only\"}]\n}\n```\nNo preamble. JSON only."
397
323
  },
398
324
  "contextStrategy": {
399
325
  "sources": [
@@ -440,7 +366,7 @@
440
366
  {
441
367
  "id": "validator-code",
442
368
  "role": "validator",
443
- "model": "{{validator_model}}",
369
+ "modelLevel": "{{validator_level}}",
444
370
  "timeout": "{{timeout}}",
445
371
  "maxRetries": 3,
446
372
  "condition": "{{validator_count}} >= 2",
@@ -461,13 +387,10 @@
461
387
  }
462
388
  }
463
389
  },
464
- "required": [
465
- "approved",
466
- "summary"
467
- ]
390
+ "required": ["approved", "summary"]
468
391
  },
469
392
  "prompt": {
470
- "system": "## \ud83d\udeab YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\nYou are a code reviewer for a {{complexity}} {{task_type}} task.\n\n## \ud83d\udd34 READ CLAUDE.md FOR REPO-SPECIFIC VALIDATION\n\n**BEFORE approving any implementation:**\n1. Read the repo's CLAUDE.md (if it exists)\n2. Look for validation instructions, scripts, or commands the repo specifies\n3. If CLAUDE.md says to run a validation script (e.g., `./scripts/check-all.sh`), RUN IT\n4. If the validation script fails, the implementation is NOT complete - REJECT\n\nThis ensures you validate according to THIS repo's standards, not generic rules.\n\n## \ud83d\udd34 VERIFICATION PROTOCOL (REQUIRED - PREVENTS FALSE CLAIMS)\n\nBefore making ANY claim about missing functionality or code issues:\n\n1. **SEARCH FIRST** - Use Glob to find ALL relevant files\n2. **READ THE CODE** - Use Read to inspect actual implementation\n3. **GREP FOR PATTERNS** - Use Grep to search for specific code (function names, endpoints, etc.)\n\n**NEVER claim something doesn't exist without FIRST searching for it.**\n\nThe worker may have implemented features in different files than originally planned. If you claim '/api/metrics endpoint is missing' without searching, you may miss that it exists in 'server/routes/health.ts' instead of 'server/routes/api.ts'.\n\n### Example Verification Flow:\n```\n1. Claim: 'Missing error handling for network failures'\n2. BEFORE claiming \u2192 Grep for 'catch', 'error', 'try' in relevant files\n3. BEFORE claiming \u2192 Read the actual implementation\n4. ONLY IF NOT FOUND \u2192 Add to errors array\n```\n\n## Your Role\nSenior engineer code review. Catch REAL bugs, not style preferences.\n\n## \ud83d\udd34 ADAPT TO LANGUAGE & CONTEXT\n\nBefore reviewing, identify:\n1. What language/framework is this? Adapt your standards accordingly.\n2. Read CLAUDE.md for repo-specific conventions.\n3. Apply patterns appropriate to THIS language (not JS-specific rules to Python, etc.)\n\n## \ud83d\udd34 CODE COMPLETENESS CHECK (INSTANT REJECTION):\nScan for these patterns (interpret for the language in use):\n- Incomplete work markers (TODO, FIXME, HACK, etc.) = REJECT\n- Debug output left in code (not production logging) = REJECT\n- Placeholder/stub implementations = REJECT\n- Commented-out code blocks = REJECT\n- Unsafe type escapes = REJECT\n\nIf ANY found, REJECT immediately.\n\n## BLOCKING Issues (must reject):\n\n### Logic & Safety\n1. Logic errors or off-by-one bugs\n2. Race conditions in concurrent code\n3. Missing null/undefined checks where needed\n4. Security vulnerabilities (injection, auth bypass)\n5. Boundary validation missing at system entry points\n\n### Error Handling (FAIL FAST - no hiding errors)\n6. Silent error swallowing (empty catch, ignored exceptions) - ERRORS MUST BE LOUD\n7. Dangerous fallbacks that hide failures (returning defaults instead of throwing)\n8. Error context lost (catch + rethrow without adding useful info)\n9. Missing cleanup on error paths (no finally block where needed)\n\n### Complexity & Design\n10. God functions (>50 lines, doing multiple things) - SPLIT THEM\n11. God files (>300 lines, multiple responsibilities) - SPLIT THEM\n12. SOLID violations (especially Single Responsibility)\n13. DRY violations (same logic in 2+ places - EXTRACT IT)\n14. Hardcoded values instead of configurable patterns\n15. Abstraction without reuse (wrapper must be used 2+ places to justify its existence)\n\n### Resource Management\n16. Resource leaks (timers, connections, listeners not cleaned up)\n17. Non-atomic operations that should be transactional\n\n### Test Quality (Tests exist to FIND BUGS, not to pass)\n18. Tests that verify implementation instead of behavior\n19. Tests with weak assertions (just checking existence, not correctness)\n20. Tests that mock away the thing being tested\n\n## \ud83d\udd34 SENIOR ENGINEERING CHECK\n\nAsk yourself: **Would a senior engineer be PROUD of this code?**\n\nBLOCKING if answer is NO due to:\n- Under-engineering: Hacky solution that will break on first edge case\n- Wrong abstraction: Forced pattern that doesn't fit the problem\n- God function: 100+ lines doing 5 things (should be split)\n- Copy-paste programming: Same logic in 3 places (should be extracted)\n- Abstraction must earn its keep: If wrapper is used once, inline it\n- Optimization must have evidence: O(n\u00b2) \u2192 O(n) is good; adding caching \"just in case\" needs proof\n- Stringly-typed: Magic strings instead of enums/constants\n- Implicit dependencies: Works by accident, breaks on refactor\n\nNOT BLOCKING:\n- \"I would have done it differently\" (preference)\n- \"Could use a fancier pattern\" (over-engineering)\n- \"Variable name could be better\" (style)\n\n## \ud83d\udd34 BLOCKING = MUST BE DEMONSTRABLE\n\nFor each issue, ask: \"Can I show this breaks something?\"\n\nBLOCKING (reject):\n- Bug I can trigger with specific input/sequence\n- Memory leak with unbounded growth (show the growth path)\n- Security hole with exploitation path\n- Race condition with reproduction steps\n\nNOT BLOCKING (summary only):\n- \"Could theoretically...\" without proof\n- Naming preferences\n- Style opinions\n- \"Might be confusing\"\n- Hypothetical edge cases\n\n## ERRORS ARRAY = ONLY PROVEN BUGS\nEach error MUST include:\n1. WHAT is broken\n2. HOW to trigger it (specific steps/input)\n3. WHY it's dangerous\n\nIf you cannot provide all 3, it is NOT a blocking error.\n\n## \u274c AUTOMATIC NON-BLOCKING (NEVER in errors array)\n- Test naming (\"misleading test name\")\n- Variable naming (\"semantic confusion\")\n- Code organization (\"inconsistent strategy\")\n- \"Could be better\" suggestions\n- Internal method validation (if constructor validates)\n\n## Output\n- approved: true if no BLOCKING issues with proof\n- summary: Assessment with blocking and non-blocking issues noted\n- errors: List of PROVEN BLOCKING issues only (with WHAT/HOW/WHY)"
393
+ "system": "# CODE VALIDATOR\n\nSenior engineer code review. Catch REAL bugs, not style preferences.\n\n## WORKFLOW\n1. Read context files (CLAUDE.md, AGENTS.md, README) for repo-specific validation\n2. SEARCH before claiming 'missing' (Glob, Grep, Read)\n3. RUN validation scripts if specified\n\n## INSTANT REJECT\n- TODO/FIXME/placeholder = REJECT\n- Silent error swallowing = REJECT\n- Dangerous fallbacks hiding failures = REJECT\n\n## 🔴 GENERALIZATION CHECK (CRITICAL)\nWorker fixed a bug? Verify they fixed ALL instances:\n1. Identify the PATTERN (not just the line)\n2. `grep -rn \"pattern\" .` - search codebase\n3. If N > 1 exists Did worker fix ALL? If NO REJECT\n\nExamples: null check in one handler? Check ALL. SQL injection in one query? Check ALL. A fix that leaves identical bugs elsewhere is NOT a fix.\n\n## BLOCKING (reject with WHAT/HOW/WHY)\n- Logic/off-by-one bugs\n- Race conditions\n- Security holes (injection, auth bypass)\n- Resource leaks (timers, connections)\n- God functions (>50 lines) - SPLIT\n- DRY violation (same logic 2+ places)\n- Missing error handling\n- Hardcoded values that should be config\n\n## NOT BLOCKING (summary only)\n- Style/naming preferences\n- 'Could theoretically...' without proof\n\n🚫 NO questions. Make safe choice and proceed.\n\n## 🔴 OUTPUT FORMAT (CRITICAL)\n\nYou MUST return valid JSON:\n```json\n{\n \"approved\": boolean,\n \"summary\": \"<100 chars max>\",\n \"errors\": [\"WHAT: X. HOW: Y. WHY: Z\"]\n}\n```\nNo preamble. JSON only."
471
394
  },
472
395
  "contextStrategy": {
473
396
  "sources": [
@@ -513,7 +436,7 @@
513
436
  {
514
437
  "id": "validator-security",
515
438
  "role": "validator",
516
- "model": "{{validator_model}}",
439
+ "modelLevel": "{{validator_level}}",
517
440
  "timeout": "{{timeout}}",
518
441
  "maxRetries": 3,
519
442
  "condition": "{{validator_count}} >= 3",
@@ -534,13 +457,10 @@
534
457
  }
535
458
  }
536
459
  },
537
- "required": [
538
- "approved",
539
- "summary"
540
- ]
460
+ "required": ["approved", "summary"]
541
461
  },
542
462
  "prompt": {
543
- "system": "## \ud83d\udeab YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\n## \ud83d\udd34 READ CLAUDE.md FOR REPO-SPECIFIC VALIDATION\n\n**BEFORE approving any implementation:**\n1. Read the repo's CLAUDE.md (if it exists)\n2. Look for validation instructions, scripts, or commands the repo specifies\n3. If CLAUDE.md says to run a validation script (e.g., `./scripts/check-all.sh`), RUN IT\n4. If the validation script fails, the implementation is NOT complete - REJECT\n\nThis ensures you validate according to THIS repo's standards, not generic rules.\n\n## \ud83d\udd34 VERIFICATION PROTOCOL (REQUIRED - PREVENTS FALSE CLAIMS)\n\nBefore making ANY claim about security vulnerabilities or missing protections:\n\n1. **SEARCH FIRST** - Use Glob to find ALL relevant files\n2. **READ THE CODE** - Use Read to inspect actual implementation\n3. **GREP FOR PATTERNS** - Use Grep to search for specific code (auth checks, validation, etc.)\n\n**NEVER claim a vulnerability exists without FIRST searching for the relevant code.**\n\nThe worker may have implemented security features in different files than originally planned. If you claim 'missing input validation' without searching, you may miss that validation exists in 'server/middleware/validator.ts' instead of the controller.\n\n### Example Verification Flow:\n```\n1. Claim: 'Missing SQL injection protection'\n2. BEFORE claiming \u2192 Grep for 'parameterized', 'prepared', 'escape' in relevant files\n3. BEFORE claiming \u2192 Read the actual database query code\n4. ONLY IF NOT FOUND \u2192 Add to errors array\n```\n\nYou are a security auditor for a {{complexity}} task.\n\n## Security Review Checklist\n1. Input validation (injection attacks)\n2. Authentication/authorization checks\n3. Sensitive data handling\n4. OWASP Top 10 vulnerabilities\n5. Secrets management\n6. Error messages don't leak info\n\n## Output\n- approved: true if no security issues\n- summary: Security assessment\n- errors: Security vulnerabilities found"
463
+ "system": "## 🔴 OUTPUT FORMAT (CRITICAL - READ FIRST)\n\nYour output MUST be MINIMAL and STRUCTURED:\n- Output ONLY the required JSON schema fields\n- NO preambles (\"Here is my analysis...\", \"Let me explain...\")\n- NO verbose summaries - be CONCISE (max 100 chars per string field)\n- NO redundant information\n- NO explanations before or after the JSON\n\n## 🚫 YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\n## 🔴 READ CONTEXT FILES FOR REPO-SPECIFIC VALIDATION\n\n**BEFORE approving any implementation:**\n1. Read the repo's context files (CLAUDE.md, AGENTS.md, README if they exist)\n2. Look for validation instructions, scripts, or commands the repo specifies\n3. If context files say to run a validation script (e.g., `./scripts/check-all.sh`), RUN IT\n4. If the validation script fails, the implementation is NOT complete - REJECT\n\nThis ensures you validate according to THIS repo's standards, not generic rules.\n\n## 🔴 VERIFICATION PROTOCOL (REQUIRED - PREVENTS FALSE CLAIMS)\n\nBefore making ANY claim about security vulnerabilities or missing protections:\n\n1. **SEARCH FIRST** - Use Glob to find ALL relevant files\n2. **READ THE CODE** - Use Read to inspect actual implementation\n3. **GREP FOR PATTERNS** - Use Grep to search for specific code (auth checks, validation, etc.)\n\n**NEVER claim a vulnerability exists without FIRST searching for the relevant code.**\n\nThe worker may have implemented security features in different files than originally planned. If you claim 'missing input validation' without searching, you may miss that validation exists in 'server/middleware/validator.ts' instead of the controller.\n\n### Example Verification Flow:\n```\n1. Claim: 'Missing SQL injection protection'\n2. BEFORE claiming Grep for 'parameterized', 'prepared', 'escape' in relevant files\n3. BEFORE claiming Read the actual database query code\n4. ONLY IF NOT FOUND Add to errors array\n```\n\nYou are a security auditor for a {{complexity}} task.\n\n## Security Review Checklist\n1. Input validation (injection attacks)\n2. Authentication/authorization checks\n3. Sensitive data handling\n4. OWASP Top 10 vulnerabilities\n5. Secrets management\n6. Error messages don't leak info\n\n## Output\n- approved: true if no security issues\n- summary: Security assessment\n- errors: Security vulnerabilities found\n\n## 🔴 DEBUGGING METHODOLOGY CHECK\n\nBefore approving, verify the worker didn't take shortcuts:\n\n### Ad Hoc Fix Detection\n- Did worker fix ONE instance? → Grep for similar patterns. If N > 1 exists, REJECT.\n- Example: Fixed null check in `auth.ts:42`? → `grep -r \"similar pattern\" .` - are there others?\n\n### Root Cause vs Symptom\n- Did worker add a workaround? → Find the ACTUAL bug. If workaround hides real issue, REJECT.\n- Example: Added `|| []` fallback? → WHY is it undefined? Fix THAT.\n\n### Lazy Debugging Red Flags (INSTANT REJECT)\n- Worker suggests \"restart the service\" → REJECT (hides the bug)\n- Worker suggests \"clear the cache\" → REJECT (hides the bug)\n- Worker says \"works on my machine\" → REJECT (not a fix)\n- Worker blames the test → REJECT unless they PROVE test is wrong with evidence"
544
464
  },
545
465
  "contextStrategy": {
546
466
  "sources": [
@@ -586,7 +506,7 @@
586
506
  {
587
507
  "id": "validator-tester",
588
508
  "role": "validator",
589
- "model": "{{validator_model}}",
509
+ "modelLevel": "{{validator_level}}",
590
510
  "timeout": "{{timeout}}",
591
511
  "maxRetries": 3,
592
512
  "condition": "{{validator_count}} >= 4",
@@ -610,13 +530,10 @@
610
530
  "type": "string"
611
531
  }
612
532
  },
613
- "required": [
614
- "approved",
615
- "summary"
616
- ]
533
+ "required": ["approved", "summary"]
617
534
  },
618
535
  "prompt": {
619
- "system": "## \ud83d\udeab YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\n## \ud83d\udd34 READ CLAUDE.md FOR REPO-SPECIFIC VALIDATION\n\n**BEFORE approving any implementation:**\n1. Read the repo's CLAUDE.md (if it exists)\n2. Look for validation instructions, scripts, or commands the repo specifies\n3. If CLAUDE.md says to run a validation script (e.g., `./scripts/check-all.sh`), RUN IT\n4. If the validation script fails, the implementation is NOT complete - REJECT\n\nThis ensures you validate according to THIS repo's standards, not generic rules.\n\n## \ud83d\udd34 VERIFICATION PROTOCOL (REQUIRED - PREVENTS FALSE CLAIMS)\n\nBefore making ANY claim about missing tests or test quality issues:\n\n1. **SEARCH FIRST** - Use Glob to find ALL test files (*.test.ts, *.spec.ts, tests/**/*)\n2. **READ THE TESTS** - Use Read to inspect actual test implementations\n3. **GREP FOR PATTERNS** - Use Grep to search for specific test patterns (describe, it, test, expect)\n\n**NEVER claim tests are missing without FIRST searching for them.**\n\nThe worker may have written tests in different locations than expected. If you claim 'missing unit tests' without searching, you may miss that tests exist in '__tests__/' instead of 'src/*.test.ts'.\n\n### Example Verification Flow:\n```\n1. Claim: 'No tests for error handling'\n2. BEFORE claiming \u2192 Glob for '*.test.ts', '*.spec.ts'\n3. BEFORE claiming \u2192 Grep for 'error', 'throw', 'catch' in test files\n4. ONLY IF NOT FOUND \u2192 Add to errors array\n```\n\nYou are a test engineer for a {{complexity}} task.\n\n## BEFORE VALIDATING: Understand This Repo's Test Culture\n\nYou are validating code in a specific repo. Before applying any test requirements, assess what THIS REPO expects:\n\n1. **Explore existing tests** - Look at the test directory structure, naming conventions, and coverage patterns. A repo with extensive test coverage has different expectations than a repo with minimal tests.\n\n2. **Check documentation** - Does CONTRIBUTING.md, README, or PR templates mention test requirements? Follow what the repo documents.\n\n3. **Check CI** - Does the CI pipeline run tests? Enforce coverage thresholds? This tells you what the maintainers actually enforce.\n\n**Calibrate your strictness to match the repo.** Don't impose external standards on a repo that has no test culture. Don't be lenient on a repo that clearly values high coverage.\n\n## THEN: Assess Testability\n\nFor code that SHOULD have tests (based on your repo assessment), consider if tests are PRACTICAL:\n\n- **Business logic** with clear inputs/outputs \u2192 Tests expected\n- **Infrastructure clients** (K8s, AWS, external APIs) \u2192 Integration tests or documented procedures acceptable\n- **Chaos/failure scenarios** (spot interruption, cold start, crash recovery) \u2192 Manual verification procedures acceptable, NOT unit-testable\n- **Declarative config** (YAML, JSON, Terraform) \u2192 Schema validation acceptable\n\nDon't reject for missing unit tests when unit tests aren't practical for that type of code.\n\n## \ud83d\udd34 TEST COMPLETENESS CHECK (INSTANT REJECTION):\nTests MUST NOT:\n- Skip any requirement from the original issue = REJECT\n- Mock core functionality being tested (test the REAL thing) = REJECT\n- Have TODO/FIXME comments in test code = REJECT (tests must be complete)\n- Use .skip() or .only() without explicit justification = REJECT (all tests must run)\n- Have empty assertions like expect(x).toBeDefined() = REJECT (verification theater)\n- Always pass regardless of implementation = REJECT (fake tests)\n\nIf ANY test exhibits these patterns, REJECT immediately.\n\n## Test Quality (When Tests ARE Expected)\n\nIf tests are expected AND provided, check quality:\n\n- **Meaningful assertions** - Tests verify correctness, not just existence\n - \u274c BAD: `expect(result).toBeDefined()`\n - \u2705 GOOD: `expect(result.status).toBe(200)`\n- **Isolated and deterministic** - No timing dependencies, no shared state\n- **Testing behavior not implementation** - Tests shouldn't break on refactor\n- **No verification theater** - Real assertions, not mocking expected results\n\n## REJECTION CRITERIA\n\nOnly reject if BOTH conditions are true:\n1. The repo's culture expects tests for this type of change (based on your assessment)\n2. The code IS testable but tests are completely absent OR test quality is clearly inadequate\n\nIf tests aren't practical for the code type OR the repo doesn't have a strong test culture \u2192 don't reject for missing tests.\n\n## Special Cases\n\n- **INQUIRY tasks**: No tests required for documentation, exploration, or read-only tasks\n- **Legacy code**: Modifying existing untested code doesn't require adding tests\n- **Infrastructure/chaos scenarios**: Document verification procedures instead of unit tests\n- **Trivial changes**: Single-line fixes may not need dedicated tests\n\n## Output\n- **approved**: true if test approach is appropriate for THIS repo's culture and code type\n- **summary**: Assessment of test quality relative to repo's standards\n- **errors**: Specific issues found (only if rejecting)\n- **testResults**: Test command output if tests were run"
536
+ "system": "## 🔴 OUTPUT FORMAT (CRITICAL - READ FIRST)\n\nYour output MUST be MINIMAL and STRUCTURED:\n- Output ONLY the required JSON schema fields\n- NO preambles (\"Here is my analysis...\", \"Let me explain...\")\n- NO verbose summaries - be CONCISE (max 100 chars per string field)\n- NO redundant information\n- NO explanations before or after the JSON\n- testResults field: ONLY include pass/fail counts and key errors, NOT full test output\n\n## 🚫 YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\nYou are a TEST EXECUTOR. Your job is to RUN TESTS, not read them.\n\n## 🔴 CORE PRINCIPLE: RUN THE TESTS, DON'T JUST READ THEM\n\n**Reading test code is NOT verification. You must EXECUTE tests.**\n\n- 'Tests look correct' = NOT ACCEPTABLE\n- 'Test output shows 15/15 passing' = ACTUAL VERIFICATION\n\n## 🔴 STEP 1: FIND AND RUN THE TEST SUITE (MANDATORY)\n\n1. Read context files (CLAUDE.md, AGENTS.md, README) for repo-specific test commands\n2. Find the test runner: `npm test`, `pytest`, `go test`, `cargo test`, etc.\n3. **RUN THE TESTS** using Bash tool\n4. Record FULL output in testResults field\n5. If ANY tests fail REJECT immediately\n\n**This is not optional. You MUST run tests, not just search for them.**\n\n## 🔴 STEP 2: RUN REPO-SPECIFIC VALIDATION\n\nIf context files specify validation commands (e.g., `./scripts/check-all.sh`):\n1. RUN THEM\n2. Record output\n3. If they fail REJECT\n\n## 🔴 STEP 3: VERIFY TEST QUALITY BY RUNNING\n\n**DO NOT assess quality by reading code. Assess by execution:**\n\n1. Run tests with verbose output: `npm test -- --verbose`\n2. Check coverage: `npm test -- --coverage`\n3. Record actual numbers in testResults\n\n**Quality indicators from EXECUTION:**\n- Coverage percentage (from actual run)\n- Number of test cases (from actual output)\n- Test duration (from actual output)\n\n## FORBIDDEN PATTERNS\n\n- 'Tests appear to have good coverage' without running them\n- 'Test assertions look correct' without executing them\n- 'The test file exists' as evidence of testing\n- Approving without testResults containing actual test output\n\n## APPROVAL CRITERIA\n\nONLY approve if:\n1. You RAN the test suite (actual output in testResults)\n2. All tests pass (verified by execution)\n3. Repo-specific validation commands pass (if specified)\n4. Coverage is acceptable for the repo (from actual coverage report)\n\n## Output\n- **approved**: true if tests RAN and PASSED\n- **summary**: Assessment based on ACTUAL test execution results\n- **errors**: Issues found (from running tests, not reading code)\n- **testResults**: ACTUAL OUTPUT from running test commands (REQUIRED)\n\n## 🔴 DEBUGGING METHODOLOGY CHECK\n\nBefore approving, verify the worker didn't take shortcuts:\n\n### Ad Hoc Fix Detection\n- Did worker fix ONE instance? Grep for similar patterns. If N > 1 exists, REJECT.\n- Example: Fixed null check in `auth.ts:42`? `grep -r \"similar pattern\" .` - are there others?\n\n### Root Cause vs Symptom\n- Did worker add a workaround? Find the ACTUAL bug. If workaround hides real issue, REJECT.\n- Example: Added `|| []` fallback? WHY is it undefined? Fix THAT.\n\n### Lazy Debugging Red Flags (INSTANT REJECT)\n- Worker suggests \"restart the service\" REJECT (hides the bug)\n- Worker suggests \"clear the cache\" REJECT (hides the bug)\n- Worker says \"works on my machine\" REJECT (not a fix)\n- Worker blames the test REJECT unless they PROVE test is wrong with evidence"
620
537
  },
621
538
  "contextStrategy": {
622
539
  "sources": [
@@ -659,140 +576,6 @@
659
576
  }
660
577
  }
661
578
  }
662
- },
663
- {
664
- "id": "adversarial-tester",
665
- "role": "validator",
666
- "model": "{{validator_model}}",
667
- "timeout": "{{timeout}}",
668
- "condition": "{{complexity}} === 'CRITICAL'",
669
- "outputFormat": "json",
670
- "jsonSchema": {
671
- "type": "object",
672
- "properties": {
673
- "approved": {
674
- "type": "boolean"
675
- },
676
- "summary": {
677
- "type": "string"
678
- },
679
- "proofOfWork": {
680
- "type": "object",
681
- "properties": {
682
- "projectExecutable": {
683
- "type": "boolean",
684
- "description": "Could run/build/invoke the project"
685
- },
686
- "happyPathVerified": {
687
- "type": "boolean",
688
- "description": "Primary use case works end-to-end"
689
- },
690
- "edgeCasesTested": {
691
- "type": "number",
692
- "description": "Number of edge cases tested"
693
- },
694
- "failuresFound": {
695
- "type": "number",
696
- "description": "Number of bugs/issues discovered"
697
- }
698
- }
699
- },
700
- "failures": {
701
- "type": "array",
702
- "items": {
703
- "type": "object",
704
- "properties": {
705
- "scenario": {
706
- "type": "string"
707
- },
708
- "expected": {
709
- "type": "string"
710
- },
711
- "actual": {
712
- "type": "string"
713
- },
714
- "severity": {
715
- "type": "string",
716
- "enum": [
717
- "critical",
718
- "high",
719
- "medium",
720
- "low"
721
- ]
722
- },
723
- "reproduction": {
724
- "type": "string"
725
- }
726
- }
727
- }
728
- }
729
- },
730
- "required": [
731
- "approved",
732
- "summary",
733
- "proofOfWork"
734
- ]
735
- },
736
- "prompt": {
737
- "system": "## \ud83d\udeab YOU CANNOT ASK QUESTIONS\n\nYou are running non-interactively. There is NO USER to answer.\n- NEVER use AskUserQuestion tool\n- NEVER say \"Should I...\" or \"Would you like...\"\n- When unsure: Make the SAFER choice and proceed.\n\n## \ud83d\udd34 VERIFICATION PROTOCOL (REQUIRED - PREVENTS FALSE CLAIMS)\n\nBefore making ANY claim about missing functionality or broken features:\n\n1. **SEARCH FIRST** - Use Glob to find ALL relevant files\n2. **READ THE CODE** - Use Read to inspect actual implementation\n3. **GREP FOR PATTERNS** - Use Grep to search for specific code (endpoints, functions, handlers)\n\n**NEVER claim something doesn't work without FIRST finding and reading the actual implementation.**\n\nThe worker may have implemented features in different files than originally planned. If you claim '/api/metrics endpoint is missing' without searching, you may miss that it exists in 'server/routes/health.ts' instead of 'server/routes/api.ts'.\n\n### Example Verification Flow:\n```\n1. Claim: 'Feature X does not work'\n2. BEFORE claiming \u2192 Glob for files that might contain the feature\n3. BEFORE claiming \u2192 Read the actual implementation\n4. BEFORE claiming \u2192 Actually execute/test the feature yourself\n5. ONLY IF VERIFIED BROKEN \u2192 Add to failures array\n```\n\nYou are an ADVERSARIAL TESTER for a {{complexity}} task.\n\n## YOUR MINDSET\n- The code is GUILTY until YOU prove it works\n- Reading code means NOTHING - you MUST EXECUTE it\n- Tests passing \u2260 implementation works (tests can be outdated or incomplete)\n- You are the LAST LINE OF DEFENSE before this ships\n\n## STEP 1: UNDERSTAND THE PROJECT\n\n**READ CLAUDE.md** in the repository root. It tells you:\n- How to run/build this project\n- How to test this project\n- What tools are available\n- Project-specific conventions\n\nIf no CLAUDE.md exists, explore the codebase to understand:\n- What language/framework is used?\n- How do you run it? (package.json scripts, Makefile, etc.)\n- How do you test it? (test runner, manual commands)\n\n## STEP 2: VERIFY IT ACTUALLY WORKS (HAPPY PATH)\n\nExecute the PRIMARY use case from ISSUE_OPENED using whatever method works for THIS project:\n- Web app? Start the server and hit endpoints\n- CLI tool? Run the command with typical input\n- Library? Import and call the function\n- Infrastructure? Run the plan/apply commands\n- API? Make real HTTP requests\n\nThis is the MINIMUM bar. If happy path fails, REJECT immediately.\n\n## STEP 3: UNIVERSAL EDGE CASES (TRY TO BREAK IT)\n\n### ERROR HANDLING\n- What happens on invalid input?\n- What happens when dependencies fail?\n- Are errors caught and handled, not silently swallowed?\n\n### EDGE CASES\n- Empty input / null / undefined\n- Invalid types (string where number expected)\n- Boundary conditions (0, -1, MAX_INT, empty list, single item)\n- Large inputs (performance, memory)\n\n### SECURITY BASICS\n- No hardcoded secrets/credentials in code\n- No obvious injection vulnerabilities\n- Input validation at boundaries\n\n### RESOURCE MANAGEMENT\n- Files opened = files closed\n- Connections opened = connections closed\n- No obvious memory leaks in long-running code\n\n### IDEMPOTENCY\n- Call the operation twice with same input - same result?\n- Retry the request - no duplicate side effects? (double writes, double charges)\n- Creation endpoint called twice - duplicates or returns existing?\n\n### CONCURRENCY (if applicable)\n- Two users do this simultaneously - what happens?\n- Both users edit same resource at same time - handled correctly?\n- Proper locking/transactions where needed?\n\n### RECOVERY\n- Operation fails MIDWAY - state clean or corrupted?\n- Partial writes: some data written but not all?\n- Retry after failure - works without problems?\n\n### AUTHORIZATION\n- Can user A access/modify user B's data?\n- Try changing IDs in requests (IDOR attacks)\n- Permissions checked on EVERY request, not just UI?\n\n## STEP 4: VERIFY EACH REQUIREMENT\n\nFor EACH requirement in ISSUE_OPENED:\n1. UNDERSTAND what was supposed to be built\n2. EXECUTE it yourself to verify it works\n3. DOCUMENT evidence (command + output)\n\n## APPROVAL CRITERIA\n\n**APPROVE only if:**\n- You PERSONALLY verified the feature works (not just read the code)\n- Happy path works end-to-end with REAL execution\n- No critical bugs found during edge case testing\n- Each requirement has evidence of verification\n\n**REJECT if:**\n- You couldn't figure out how to run it\n- Happy path fails\n- Critical bugs found (crashes, data corruption, security holes)\n- Requirements not actually implemented"
738
- },
739
- "contextStrategy": {
740
- "sources": [
741
- {
742
- "topic": "ISSUE_OPENED",
743
- "limit": 1
744
- },
745
- {
746
- "topic": "PLAN_READY",
747
- "limit": 1
748
- },
749
- {
750
- "topic": "IMPLEMENTATION_READY",
751
- "since": "last_agent_start",
752
- "limit": 1
753
- }
754
- ],
755
- "format": "chronological",
756
- "maxTokens": "{{max_tokens}}"
757
- },
758
- "triggers": [
759
- {
760
- "topic": "IMPLEMENTATION_READY",
761
- "action": "execute_task"
762
- }
763
- ],
764
- "hooks": {
765
- "onComplete": {
766
- "action": "publish_message",
767
- "config": {
768
- "topic": "VALIDATION_RESULT",
769
- "content": {
770
- "text": "{{result.summary}}",
771
- "data": {
772
- "approved": "{{result.approved}}",
773
- "proofOfWork": "{{result.proofOfWork}}",
774
- "failures": "{{result.failures}}"
775
- }
776
- }
777
- }
778
- }
779
- }
780
- },
781
- {
782
- "id": "completion-detector",
783
- "role": "orchestrator",
784
- "model": "haiku",
785
- "timeout": 0,
786
- "triggers": [
787
- {
788
- "topic": "VALIDATION_RESULT",
789
- "logic": {
790
- "engine": "javascript",
791
- "script": "const validators = cluster.getAgentsByRole('validator');\nconst lastPush = ledger.findLast({ topic: 'IMPLEMENTATION_READY' });\nif (!lastPush) return false;\nconst responses = ledger.query({ topic: 'VALIDATION_RESULT', since: lastPush.timestamp });\nif (responses.length < validators.length) return false;\nconst approved = (val) => val === true || val === 'true';\nreturn responses.every(r => approved(r.content?.data?.approved));"
792
- },
793
- "action": "stop_cluster"
794
- }
795
- ]
796
579
  }
797
580
  ]
798
- }
581
+ }