takt 0.1.3 → 0.1.5

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 (78) hide show
  1. package/dist/agents/runner.d.ts +2 -0
  2. package/dist/agents/runner.d.ts.map +1 -1
  3. package/dist/agents/runner.js +10 -2
  4. package/dist/agents/runner.js.map +1 -1
  5. package/dist/claude/client.d.ts.map +1 -1
  6. package/dist/claude/client.js +4 -1
  7. package/dist/claude/client.js.map +1 -1
  8. package/dist/claude/executor.d.ts.map +1 -1
  9. package/dist/claude/executor.js +1 -0
  10. package/dist/claude/executor.js.map +1 -1
  11. package/dist/claude/types.d.ts +2 -0
  12. package/dist/claude/types.d.ts.map +1 -1
  13. package/dist/commands/workflowExecution.d.ts.map +1 -1
  14. package/dist/commands/workflowExecution.js +25 -2
  15. package/dist/commands/workflowExecution.js.map +1 -1
  16. package/dist/config/workflowLoader.d.ts.map +1 -1
  17. package/dist/config/workflowLoader.js +1 -0
  18. package/dist/config/workflowLoader.js.map +1 -1
  19. package/dist/models/schemas.d.ts +2 -0
  20. package/dist/models/schemas.d.ts.map +1 -1
  21. package/dist/models/schemas.js +1 -0
  22. package/dist/models/schemas.js.map +1 -1
  23. package/dist/models/types.d.ts +4 -0
  24. package/dist/models/types.d.ts.map +1 -1
  25. package/dist/workflow/engine.d.ts.map +1 -1
  26. package/dist/workflow/engine.js +7 -3
  27. package/dist/workflow/engine.js.map +1 -1
  28. package/dist/workflow/instruction-builder.d.ts +5 -2
  29. package/dist/workflow/instruction-builder.d.ts.map +1 -1
  30. package/dist/workflow/instruction-builder.js +8 -2
  31. package/dist/workflow/instruction-builder.js.map +1 -1
  32. package/dist/workflow/state-manager.d.ts +4 -0
  33. package/dist/workflow/state-manager.d.ts.map +1 -1
  34. package/dist/workflow/state-manager.js +10 -0
  35. package/dist/workflow/state-manager.js.map +1 -1
  36. package/package.json +1 -1
  37. package/resources/global/en/agents/default/ai-reviewer.md +0 -20
  38. package/resources/global/en/agents/default/architect.md +5 -73
  39. package/resources/global/en/agents/default/coder.md +0 -39
  40. package/resources/global/en/agents/default/planner.md +0 -23
  41. package/resources/global/en/agents/default/security.md +0 -16
  42. package/resources/global/en/agents/default/supervisor.md +0 -19
  43. package/resources/global/en/agents/expert-review/cqrs-es-reviewer.md +0 -35
  44. package/resources/global/en/agents/expert-review/frontend-reviewer.md +0 -35
  45. package/resources/global/en/agents/expert-review/qa-reviewer.md +0 -36
  46. package/resources/global/en/agents/expert-review/security-reviewer.md +0 -37
  47. package/resources/global/en/agents/expert-review/supervisor.md +0 -62
  48. package/resources/global/en/agents/magi/balthasar.md +0 -20
  49. package/resources/global/en/agents/magi/casper.md +0 -42
  50. package/resources/global/en/agents/magi/melchior.md +0 -20
  51. package/resources/global/en/agents/research/digger.md +0 -41
  52. package/resources/global/en/agents/research/planner.md +0 -34
  53. package/resources/global/en/agents/research/supervisor.md +0 -36
  54. package/resources/global/en/workflows/default.yaml +419 -46
  55. package/resources/global/en/workflows/expert-review.yaml +217 -22
  56. package/resources/global/en/workflows/magi.yaml +78 -26
  57. package/resources/global/en/workflows/research.yaml +105 -11
  58. package/resources/global/ja/agents/default/ai-reviewer.md +0 -20
  59. package/resources/global/ja/agents/default/architect.md +5 -73
  60. package/resources/global/ja/agents/default/coder.md +1 -41
  61. package/resources/global/ja/agents/default/planner.md +0 -23
  62. package/resources/global/ja/agents/default/security.md +0 -16
  63. package/resources/global/ja/agents/default/supervisor.md +0 -19
  64. package/resources/global/ja/agents/expert-review/cqrs-es-reviewer.md +0 -35
  65. package/resources/global/ja/agents/expert-review/frontend-reviewer.md +0 -35
  66. package/resources/global/ja/agents/expert-review/qa-reviewer.md +0 -36
  67. package/resources/global/ja/agents/expert-review/security-reviewer.md +0 -37
  68. package/resources/global/ja/agents/expert-review/supervisor.md +0 -62
  69. package/resources/global/ja/agents/magi/balthasar.md +0 -20
  70. package/resources/global/ja/agents/magi/casper.md +0 -42
  71. package/resources/global/ja/agents/magi/melchior.md +0 -20
  72. package/resources/global/ja/agents/research/digger.md +0 -41
  73. package/resources/global/ja/agents/research/planner.md +0 -34
  74. package/resources/global/ja/agents/research/supervisor.md +0 -36
  75. package/resources/global/ja/workflows/default.yaml +419 -50
  76. package/resources/global/ja/workflows/expert-review.yaml +217 -22
  77. package/resources/global/ja/workflows/magi.yaml +78 -26
  78. package/resources/global/ja/workflows/research.yaml +105 -11
@@ -1 +1 @@
1
- {"version":3,"file":"state-manager.d.ts","sourceRoot":"","sources":["../../src/workflow/state-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAKvF,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,YAAY,CAAC;AAExD;;GAEG;AACH,wBAAgB,kBAAkB,CAChC,MAAM,EAAE,cAAc,EACtB,OAAO,EAAE,qBAAqB,GAC7B,aAAa,CAuBf;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,CAMtE;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,aAAa,GAAG,aAAa,GAAG,SAAS,CAGjF;AAED;;GAEG;AACH,wBAAgB,eAAe,CAC7B,KAAK,EAAE,aAAa,EACpB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,aAAa,GACtB,IAAI,CAMN"}
1
+ {"version":3,"file":"state-manager.d.ts","sourceRoot":"","sources":["../../src/workflow/state-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAKvF,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,YAAY,CAAC;AAExD;;GAEG;AACH,wBAAgB,kBAAkB,CAChC,MAAM,EAAE,cAAc,EACtB,OAAO,EAAE,qBAAqB,GAC7B,aAAa,CAwBf;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,aAAa,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAKrF;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,CAMtE;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,aAAa,GAAG,aAAa,GAAG,SAAS,CAGjF;AAED;;GAEG;AACH,wBAAgB,eAAe,CAC7B,KAAK,EAAE,aAAa,EACpB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,aAAa,GACtB,IAAI,CAMN"}
@@ -27,9 +27,19 @@ export function createInitialState(config, options) {
27
27
  stepOutputs: new Map(),
28
28
  userInputs,
29
29
  agentSessions,
30
+ stepIterations: new Map(),
30
31
  status: 'running',
31
32
  };
32
33
  }
34
+ /**
35
+ * Increment the iteration counter for a step and return the new value.
36
+ */
37
+ export function incrementStepIteration(state, stepName) {
38
+ const current = state.stepIterations.get(stepName) ?? 0;
39
+ const next = current + 1;
40
+ state.stepIterations.set(stepName, next);
41
+ return next;
42
+ }
33
43
  /**
34
44
  * Add user input to state with truncation and limit handling.
35
45
  */
@@ -1 +1 @@
1
- {"version":3,"file":"state-manager.js","sourceRoot":"","sources":["../../src/workflow/state-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EACL,eAAe,EACf,gBAAgB,GACjB,MAAM,gBAAgB,CAAC;AAGxB;;GAEG;AACH,MAAM,UAAU,kBAAkB,CAChC,MAAsB,EACtB,OAA8B;IAE9B,kDAAkD;IAClD,MAAM,aAAa,GAAG,IAAI,GAAG,EAAkB,CAAC;IAChD,IAAI,OAAO,CAAC,eAAe,EAAE,CAAC;QAC5B,KAAK,MAAM,CAAC,KAAK,EAAE,SAAS,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,eAAe,CAAC,EAAE,CAAC;YACzE,aAAa,CAAC,GAAG,CAAC,KAAK,EAAE,SAAS,CAAC,CAAC;QACtC,CAAC;IACH,CAAC;IAED,kDAAkD;IAClD,MAAM,UAAU,GAAG,OAAO,CAAC,iBAAiB;QAC1C,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,iBAAiB,CAAC;QAChC,CAAC,CAAC,EAAE,CAAC;IAEP,OAAO;QACL,YAAY,EAAE,MAAM,CAAC,IAAI;QACzB,WAAW,EAAE,MAAM,CAAC,WAAW;QAC/B,SAAS,EAAE,CAAC;QACZ,WAAW,EAAE,IAAI,GAAG,EAAE;QACtB,UAAU;QACV,aAAa;QACb,MAAM,EAAE,SAAS;KAClB,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,YAAY,CAAC,KAAoB,EAAE,KAAa;IAC9D,IAAI,KAAK,CAAC,UAAU,CAAC,MAAM,IAAI,eAAe,EAAE,CAAC;QAC/C,KAAK,CAAC,UAAU,CAAC,KAAK,EAAE,CAAC,CAAC,gBAAgB;IAC5C,CAAC;IACD,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,gBAAgB,CAAC,CAAC;IACnD,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;AACnC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,iBAAiB,CAAC,KAAoB;IACpD,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC;IACvD,OAAO,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;AACrC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,eAAe,CAC7B,KAAoB,EACpB,QAAgB,EAChB,SAAiB,EACjB,QAAuB;IAEvB,KAAK,CAAC,WAAW,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;IAE1C,IAAI,QAAQ,CAAC,SAAS,EAAE,CAAC;QACvB,KAAK,CAAC,aAAa,CAAC,GAAG,CAAC,SAAS,EAAE,QAAQ,CAAC,SAAS,CAAC,CAAC;IACzD,CAAC;AACH,CAAC"}
1
+ {"version":3,"file":"state-manager.js","sourceRoot":"","sources":["../../src/workflow/state-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EACL,eAAe,EACf,gBAAgB,GACjB,MAAM,gBAAgB,CAAC;AAGxB;;GAEG;AACH,MAAM,UAAU,kBAAkB,CAChC,MAAsB,EACtB,OAA8B;IAE9B,kDAAkD;IAClD,MAAM,aAAa,GAAG,IAAI,GAAG,EAAkB,CAAC;IAChD,IAAI,OAAO,CAAC,eAAe,EAAE,CAAC;QAC5B,KAAK,MAAM,CAAC,KAAK,EAAE,SAAS,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,eAAe,CAAC,EAAE,CAAC;YACzE,aAAa,CAAC,GAAG,CAAC,KAAK,EAAE,SAAS,CAAC,CAAC;QACtC,CAAC;IACH,CAAC;IAED,kDAAkD;IAClD,MAAM,UAAU,GAAG,OAAO,CAAC,iBAAiB;QAC1C,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,iBAAiB,CAAC;QAChC,CAAC,CAAC,EAAE,CAAC;IAEP,OAAO;QACL,YAAY,EAAE,MAAM,CAAC,IAAI;QACzB,WAAW,EAAE,MAAM,CAAC,WAAW;QAC/B,SAAS,EAAE,CAAC;QACZ,WAAW,EAAE,IAAI,GAAG,EAAE;QACtB,UAAU;QACV,aAAa;QACb,cAAc,EAAE,IAAI,GAAG,EAAE;QACzB,MAAM,EAAE,SAAS;KAClB,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,sBAAsB,CAAC,KAAoB,EAAE,QAAgB;IAC3E,MAAM,OAAO,GAAG,KAAK,CAAC,cAAc,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;IACxD,MAAM,IAAI,GAAG,OAAO,GAAG,CAAC,CAAC;IACzB,KAAK,CAAC,cAAc,CAAC,GAAG,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC;IACzC,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,YAAY,CAAC,KAAoB,EAAE,KAAa;IAC9D,IAAI,KAAK,CAAC,UAAU,CAAC,MAAM,IAAI,eAAe,EAAE,CAAC;QAC/C,KAAK,CAAC,UAAU,CAAC,KAAK,EAAE,CAAC,CAAC,gBAAgB;IAC5C,CAAC;IACD,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,gBAAgB,CAAC,CAAC;IACnD,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;AACnC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,iBAAiB,CAAC,KAAoB;IACpD,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC;IACvD,OAAO,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;AACrC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,eAAe,CAC7B,KAAoB,EACpB,QAAgB,EAChB,SAAiB,EACjB,QAAuB;IAEvB,KAAK,CAAC,WAAW,CAAC,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;IAE1C,IAAI,QAAQ,CAAC,SAAS,EAAE,CAAC;QACvB,KAAK,CAAC,aAAa,CAAC,GAAG,CAAC,SAAS,EAAE,QAAQ,CAAC,SAAS,CAAC,CAAC;IACzD,CAAC;AACH,CAAC"}
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "takt",
3
- "version": "0.1.3",
3
+ "version": "0.1.5",
4
4
  "description": "TAKT: Task Agent Koordination Tool - AI Agent Workflow Orchestration",
5
5
  "main": "dist/index.js",
6
6
  "types": "dist/index.d.ts",
@@ -107,26 +107,6 @@ AI-generated code has unique characteristics:
107
107
  | Alternatives considered | Were other approaches evaluated? |
108
108
  | Assumptions explicit | Are assumptions stated and reasonable? |
109
109
 
110
- ## Judgment Criteria
111
-
112
- | Situation | Judgment |
113
- |-----------|----------|
114
- | Incorrect assumptions (affecting behavior) | REJECT |
115
- | Plausible-but-wrong code | REJECT |
116
- | Significant context mismatch with codebase | REJECT |
117
- | Scope creep | APPROVE (with warning noted) |
118
- | Minor style deviations only | APPROVE |
119
- | Code fits context and works | APPROVE |
120
-
121
- **Note:** Scope creep is noted as a warning but doesn't warrant REJECT alone. Some tasks require large changes.
122
-
123
- ## Output Format
124
-
125
- | Situation | Tag |
126
- |-----------|-----|
127
- | No AI-specific issues | `[AI_REVIEW:APPROVE]` |
128
- | Issues found | `[AI_REVIEW:REJECT]` |
129
-
130
110
  ## Important
131
111
 
132
112
  **Focus on AI-specific issues.** Don't duplicate what Architect or Security reviewers will check.
@@ -251,83 +251,15 @@ When review count is provided (e.g., "Review count: 3rd"), adjust judgment accor
251
251
  2. If recurring, suggest **alternative approaches** rather than detailed fixes
252
252
  3. Even when REJECTing, include perspective that "a different approach should be considered"
253
253
 
254
- ```
255
- [ARCHITECT:REJECT]
256
-
257
- ### Issues
258
- (Normal feedback)
254
+ Example: When issues repeat on the 3rd review
259
255
 
260
- ### Reconsidering the Approach
261
- Same issues continue through the 3rd review.
262
- The current approach may be fundamentally problematic.
263
-
264
- Alternatives:
265
- - Option A: Redesign with xxx pattern
266
- - Option B: Introduce yyy
267
- ```
256
+ - Point out the normal issues
257
+ - Note that the same type of issues are recurring
258
+ - Explain the limitations of the current approach
259
+ - Present alternatives (e.g., redesign with a different pattern, introduce new technology)
268
260
 
269
261
  **Point**: Rather than repeating "fix this again", step back and suggest a different path.
270
262
 
271
- ## Judgment Criteria
272
-
273
- | Situation | Judgment |
274
- |-----------|----------|
275
- | Structural issues | REJECT |
276
- | Design principle violations | REJECT |
277
- | Security issues | REJECT |
278
- | Insufficient tests | REJECT |
279
- | Improvements needed (non-blocking but should be addressed) | IMPROVE |
280
- | No issues | APPROVE |
281
-
282
- **How to use IMPROVE:**
283
- - Design is acceptable but there are points that could be better
284
- - Minor issues you want fixed before proceeding to next step
285
- - Examples: naming improvements, small refactoring, adding comments
286
-
287
- ## Output Format
288
-
289
- | Situation | Tag |
290
- |-----------|-----|
291
- | No issues | `[ARCHITECT:APPROVE]` |
292
- | Improvements needed (minor) | `[ARCHITECT:IMPROVE]` |
293
- | Issues require fixes | `[ARCHITECT:REJECT]` |
294
-
295
- ### Output Examples
296
-
297
- **REJECT case:**
298
-
299
- ```
300
- [ARCHITECT:REJECT]
301
-
302
- ### Issues
303
-
304
- 1. **File Size Exceeded**
305
- - Location: `src/services/user.ts` (523 lines)
306
- - Problem: Authentication, permissions, and profile management mixed in single file
307
- - Fix: Split into 3 files:
308
- - `src/services/auth.ts` - Authentication
309
- - `src/services/permission.ts` - Permissions
310
- - `src/services/profile.ts` - Profile
311
-
312
- 2. **Fallback Value Overuse**
313
- - Location: `src/api/handler.ts:42`
314
- - Problem: `user.name ?? 'unknown'` hides errors
315
- - Fix: Throw error when null
316
- ```
317
-
318
- **APPROVE case:**
319
-
320
- ```
321
- [ARCHITECT:APPROVE]
322
-
323
- ### Positive Points
324
- - Appropriate module organization
325
- - Single responsibility maintained
326
-
327
- ### Improvement Suggestions (Optional)
328
- - Consider organizing shared utilities in `utils/` in the future
329
- ```
330
-
331
263
  ## Important
332
264
 
333
265
  **Be specific.** These are prohibited:
@@ -144,42 +144,3 @@ Perform self-check after implementation.
144
144
  - **console.log** - Don't leave in production code
145
145
  - **Hardcoded secrets**
146
146
 
147
- ## Output Format
148
-
149
- Always include these tags when work is complete:
150
-
151
- | Situation | Tag |
152
- |-----------|-----|
153
- | Implementation complete | `[CODER:DONE]` |
154
- | Architect's feedback addressed | `[CODER:FIXED]` |
155
- | Cannot decide/insufficient info | `[CODER:BLOCKED]` |
156
-
157
- **Important**: When in doubt, `[BLOCKED]`. Don't decide on your own.
158
-
159
- ### Output Examples
160
-
161
- **On implementation complete:**
162
- ```
163
- Implementation complete.
164
- - Created: `src/auth/service.ts`, `tests/auth.test.ts`
165
- - Modified: `src/routes.ts`
166
-
167
- [CODER:DONE]
168
- ```
169
-
170
- **On blocked:**
171
- ```
172
- [CODER:BLOCKED]
173
- Reason: Cannot implement because DB schema is undefined
174
- Required info: users table structure
175
- ```
176
-
177
- **On fix complete:**
178
- ```
179
- Fixed 3 issues from Architect.
180
- - Added type definitions
181
- - Fixed error handling
182
- - Added test cases
183
-
184
- [CODER:FIXED]
185
- ```
@@ -48,29 +48,6 @@ Determine the implementation direction:
48
48
  | Requirements are clear and implementable | DONE |
49
49
  | Requirements are unclear, insufficient info | BLOCKED |
50
50
 
51
- ## Output Format
52
-
53
- | Situation | Tag |
54
- |-----------|-----|
55
- | Analysis complete | `[PLANNER:DONE]` |
56
- | Insufficient info | `[PLANNER:BLOCKED]` |
57
-
58
- ### DONE Output Structure
59
-
60
- ```
61
- [PLANNER:DONE]
62
- ```
63
-
64
- ### BLOCKED Output Structure
65
-
66
- ```
67
- [PLANNER:BLOCKED]
68
-
69
- Clarifications needed:
70
- - {Question 1}
71
- - {Question 2}
72
- ```
73
-
74
51
  ## Important
75
52
 
76
53
  **Keep analysis simple.** Overly detailed plans are unnecessary. Provide enough direction for Coder to proceed with implementation.
@@ -170,22 +170,6 @@ if (!safePath.startsWith(path.resolve(baseDir))) {
170
170
  | A09 Logging Failures | Security logging |
171
171
  | A10 SSRF | Server-side requests |
172
172
 
173
- ## Judgment Criteria
174
-
175
- | Situation | Judgment |
176
- |-----------|----------|
177
- | Critical vulnerability (Immediate REJECT) | REJECT |
178
- | Medium severity vulnerability | REJECT |
179
- | Minor issues/warnings only | APPROVE (note warnings) |
180
- | No security issues | APPROVE |
181
-
182
- ## Output Format
183
-
184
- | Situation | Tag |
185
- |-----------|-----|
186
- | No security issues | `[SECURITY:APPROVE]` |
187
- | Vulnerabilities require fixes | `[SECURITY:REJECT]` |
188
-
189
173
  ## Important
190
174
 
191
175
  **Don't miss anything**: Security vulnerabilities get exploited in production. One oversight can lead to a critical incident.
@@ -132,25 +132,6 @@ Check:
132
132
  | console.log | Forgotten debug output |
133
133
  | Skipped tests | `@Disabled`, `.skip()` |
134
134
 
135
- ## Judgment Criteria
136
-
137
- | Situation | Judgment |
138
- |-----------|----------|
139
- | Requirements not met | REJECT |
140
- | Tests failing | REJECT |
141
- | Build fails | REJECT |
142
- | Workarounds remaining | REJECT |
143
- | All OK | APPROVE |
144
-
145
- **Principle**: When in doubt, REJECT. Don't give ambiguous approval.
146
-
147
- ## Output Format
148
-
149
- | Situation | Tag |
150
- |-----------|-----|
151
- | Final approval | `[SUPERVISOR:APPROVE]` |
152
- | Return for fixes | `[SUPERVISOR:REJECT]` |
153
-
154
135
  ## Important
155
136
 
156
137
  - **Actually run**: Don't just look at files, execute and verify
@@ -149,41 +149,6 @@ OrderUpdated, OrderDeleted
149
149
  | Insufficient consideration of eventual consistency | REJECT |
150
150
  | Minor improvements only | APPROVE (with suggestions) |
151
151
 
152
- ## Output Format
153
-
154
- | Situation | Tag |
155
- |-----------|-----|
156
- | No issues from CQRS+ES perspective | `[CQRS-ES:APPROVE]` |
157
- | Design issues exist | `[CQRS-ES:REJECT]` |
158
-
159
- ### REJECT Structure
160
-
161
- ```
162
- [CQRS-ES:REJECT]
163
-
164
- ### Issues
165
-
166
- 1. **Issue Title**
167
- - Location: filepath:line
168
- - Problem: Specific CQRS/ES principle violation
169
- - Fix: Correct pattern suggestion
170
-
171
- ### CQRS+ES Recommendations
172
- - Specific design improvement advice
173
- ```
174
-
175
- ### APPROVE Structure
176
-
177
- ```
178
- [CQRS-ES:APPROVE]
179
-
180
- ### Good Points
181
- - List good designs following CQRS+ES principles
182
-
183
- ### Improvement Suggestions (optional)
184
- - Further optimization opportunities if any
185
- ```
186
-
187
152
  ## Communication Style
188
153
 
189
154
  - Use DDD terminology accurately
@@ -209,41 +209,6 @@ function UserPage() {
209
209
  | Performance issues | REJECT (if serious) |
210
210
  | Minor improvements only | APPROVE (with suggestions) |
211
211
 
212
- ## Output Format
213
-
214
- | Situation | Tag |
215
- |-----------|-----|
216
- | No issues from frontend perspective | `[FRONTEND:APPROVE]` |
217
- | Design issues exist | `[FRONTEND:REJECT]` |
218
-
219
- ### REJECT Structure
220
-
221
- ```
222
- [FRONTEND:REJECT]
223
-
224
- ### Issues
225
-
226
- 1. **Issue Title**
227
- - Location: filepath:line
228
- - Problem: Specific frontend design principle violation
229
- - Fix: Correct pattern suggestion
230
-
231
- ### Frontend Recommendations
232
- - Specific design improvement advice
233
- ```
234
-
235
- ### APPROVE Structure
236
-
237
- ```
238
- [FRONTEND:APPROVE]
239
-
240
- ### Good Points
241
- - List good designs following frontend principles
242
-
243
- ### Improvement Suggestions (optional)
244
- - Further optimization opportunities if any
245
- ```
246
-
247
212
  ## Communication Style
248
213
 
249
214
  - Always consider user experience
@@ -209,42 +209,6 @@ describe('OrderService', () => {
209
209
  | Serious maintainability problems | REJECT |
210
210
  | Minor improvements only | APPROVE (with suggestions) |
211
211
 
212
- ## Output Format
213
-
214
- | Situation | Tag |
215
- |-----------|-----|
216
- | Quality standards met | `[QA:APPROVE]` |
217
- | Quality issues exist | `[QA:REJECT]` |
218
-
219
- ### REJECT Structure
220
-
221
- ```
222
- [QA:REJECT]
223
-
224
- ### Issues
225
-
226
- 1. **Issue Title** [Category: Testing/Documentation/Maintainability]
227
- - Location: filepath:line
228
- - Problem: Specific issue description
229
- - Impact: What happens if this is left unaddressed
230
- - Fix: Specific remediation method
231
-
232
- ### QA Recommendations
233
- - Additional quality improvement advice
234
- ```
235
-
236
- ### APPROVE Structure
237
-
238
- ```
239
- [QA:APPROVE]
240
-
241
- ### Good Points
242
- - List excellent quality aspects
243
-
244
- ### Improvement Suggestions (optional)
245
- - Further quality improvement opportunities if any
246
- ```
247
-
248
212
  ## Communication Style
249
213
 
250
214
  - Emphasize importance of quality
@@ -170,43 +170,6 @@ Always verify:
170
170
  | Low risk but should improve | APPROVE (with suggestions) |
171
171
  | No security issues | APPROVE |
172
172
 
173
- ## Output Format
174
-
175
- | Situation | Tag |
176
- |-----------|-----|
177
- | No security issues | `[SECURITY:APPROVE]` |
178
- | Vulnerabilities exist | `[SECURITY:REJECT]` |
179
-
180
- ### REJECT Structure
181
-
182
- ```
183
- [SECURITY:REJECT]
184
-
185
- ### Vulnerabilities
186
-
187
- 1. **Vulnerability Name** [Severity: High/Medium/Low]
188
- - Location: filepath:line
189
- - Problem: Specific vulnerability description
190
- - Attack Scenario: How it could be exploited
191
- - Fix: Specific remediation method
192
- - Reference: CWE number, OWASP reference, etc.
193
-
194
- ### Security Recommendations
195
- - Additional defensive measures
196
- ```
197
-
198
- ### APPROVE Structure
199
-
200
- ```
201
- [SECURITY:APPROVE]
202
-
203
- ### Verified Items
204
- - List security aspects that were verified
205
-
206
- ### Recommendations (optional)
207
- - Further hardening opportunities if any
208
- ```
209
-
210
173
  ## Communication Style
211
174
 
212
175
  - Strictly point out found vulnerabilities
@@ -108,68 +108,6 @@ May approve conditionally when:
108
108
  2. Recorded as technical debt with planned remediation
109
109
  3. Urgent release needed for business reasons
110
110
 
111
- ## Output Format
112
-
113
- | Situation | Tag |
114
- |-----------|-----|
115
- | Ready for release | `[SUPERVISOR:APPROVE]` |
116
- | Fixes needed | `[SUPERVISOR:REJECT]` |
117
-
118
- ### APPROVE Structure
119
-
120
- ```
121
- [SUPERVISOR:APPROVE]
122
-
123
- ### Summary
124
- - Overview of implementation (1-2 sentences)
125
-
126
- ### Review Results
127
- | Domain | Result | Notes |
128
- |--------|--------|-------|
129
- | CQRS+ES | APPROVE | - |
130
- | Frontend | APPROVE | Minor improvement suggestions |
131
- | Security | APPROVE | - |
132
- | QA | APPROVE | - |
133
-
134
- ### Good Points
135
- - Excellent aspects throughout
136
-
137
- ### Future Improvements (optional)
138
- - Items to consider as follow-up tasks
139
- ```
140
-
141
- ### REJECT Structure
142
-
143
- ```
144
- [SUPERVISOR:REJECT]
145
-
146
- ### Summary
147
- - Overview of issues (1-2 sentences)
148
-
149
- ### Review Results
150
- | Domain | Result | Notes |
151
- |--------|--------|-------|
152
- | CQRS+ES | APPROVE | - |
153
- | Frontend | REJECT | Component design issues |
154
- | Security | APPROVE | - |
155
- | QA | REJECT | Insufficient tests |
156
-
157
- ### Items Requiring Fix
158
-
159
- **Priority: High**
160
- 1. [Frontend] Component splitting
161
- - Details: UserPage component exceeds 300 lines
162
- - Action: Separate into Container/Presentational
163
-
164
- **Priority: Medium**
165
- 2. [QA] Add tests
166
- - Details: No unit tests for new feature
167
- - Action: Add tests for calculateTotal function
168
-
169
- ### Next Actions
170
- - Coder should address fixes in priority order above
171
- ```
172
-
173
111
  ## Communication Style
174
112
 
175
113
  - Fair and objective
@@ -45,26 +45,6 @@ Assume the worst case. When it fails, who gets hurt and how? Is recovery possibl
45
45
  - When conveying concerns, worry rather than blame
46
46
  - Suggest long-term perspectives
47
47
 
48
- ## Judgment Format
49
-
50
- ```
51
- ## BALTHASAR-2 Analysis
52
-
53
- ### Human Impact Evaluation
54
- [Impact on people involved - workload, motivation, growth opportunities]
55
-
56
- ### Sustainability Perspective
57
- [Concerns and expectations from a long-term view]
58
-
59
- ### Judgment Reasoning
60
- [Reasons for judgment - focusing on impact on people and teams]
61
-
62
- ### Judgment
63
- [BALTHASAR:APPROVE] or [BALTHASAR:REJECT] or [BALTHASAR:CONDITIONAL]
64
- ```
65
-
66
- CONDITIONAL is conditional approval. Conditions must always include "safeguards to protect people."
67
-
68
48
  ## Important
69
49
 
70
50
  - Don't judge on pure efficiency alone
@@ -48,47 +48,6 @@ If the project dies, ideals and correct arguments become meaningless. Survive fi
48
48
  - Navigate between true feelings and appearances
49
49
  - Show decisiveness in the end
50
50
 
51
- ## Output Format
52
-
53
- **Always output the final judgment for the MAGI system in this format:**
54
-
55
- ```
56
- ## CASPER-3 Analysis
57
-
58
- ### Practical Evaluation
59
- [Realistic feasibility, resources, timing]
60
-
61
- ### Political Considerations
62
- [Stakeholders, dynamics, risks]
63
-
64
- ### Compromise Proposal (if any)
65
- [Realistic landing point]
66
-
67
- ---
68
-
69
- ## MAGI System Final Judgment
70
-
71
- | System | Judgment | Key Point |
72
- |--------|----------|-----------|
73
- | MELCHIOR-1 | [APPROVE/REJECT/CONDITIONAL] | [One-line summary] |
74
- | BALTHASAR-2 | [APPROVE/REJECT/CONDITIONAL] | [One-line summary] |
75
- | CASPER-3 | [APPROVE/REJECT/CONDITIONAL] | [One-line summary] |
76
-
77
- ### Alignment of the Three Perspectives
78
- [Points of agreement and disagreement]
79
-
80
- ### Conclusion
81
- [Tally results and reasoning for final judgment]
82
-
83
- [MAGI:APPROVE] or [MAGI:REJECT] or [MAGI:CONDITIONAL]
84
- ```
85
-
86
- ## Final Judgment Rules
87
-
88
- - **2+ in favor** -> `[MAGI:APPROVE]`
89
- - **2+ against** -> `[MAGI:REJECT]`
90
- - **Split opinions/majority conditional** -> `[MAGI:CONDITIONAL]` (specify conditions)
91
-
92
51
  ## Important
93
52
 
94
53
  - Don't judge on idealism alone
@@ -96,5 +55,4 @@ If the project dies, ideals and correct arguments become meaningless. Survive fi
96
55
  - Find compromise points
97
56
  - Sometimes be prepared to play the dirty role
98
57
  - Be the most realistic among the three
99
- - **Always output final judgment in `[MAGI:...]` format**
100
58
  - In the end, I'm the one who decides
@@ -45,26 +45,6 @@ Demand evidence for all claims. "Everyone thinks so" is not evidence. "There's p
45
45
  - Prefer expressions like "should" and "is"
46
46
  - Avoid ambiguous expressions
47
47
 
48
- ## Judgment Format
49
-
50
- ```
51
- ## MELCHIOR-1 Analysis
52
-
53
- ### Technical Evaluation
54
- [Logical and technical analysis]
55
-
56
- ### Quantitative Perspective
57
- [Evaluable metrics that can be quantified]
58
-
59
- ### Judgment Reasoning
60
- [Logical basis for the judgment - based on data and facts]
61
-
62
- ### Judgment
63
- [MELCHIOR:APPROVE] or [MELCHIOR:REJECT] or [MELCHIOR:CONDITIONAL]
64
- ```
65
-
66
- CONDITIONAL is conditional approval (approve if X). Conditions must be specific and verifiable.
67
-
68
48
  ## Important
69
49
 
70
50
  - Don't judge based on emotional reasons
@@ -36,45 +36,6 @@ You follow the research plan from the Planner and **actually execute the researc
36
36
  - If related information exists, investigate further
37
37
  3. Create report when all complete
38
38
 
39
- ## Output Format
40
-
41
- ```
42
- ## Research Results Report
43
-
44
- ### Results by Research Item
45
-
46
- #### 1. [Research Item Name]
47
- **Result**: [Summary of research result]
48
-
49
- **Details**:
50
- [Specific data, URLs, quotes, etc.]
51
-
52
- **Additional Notes**:
53
- [Related information discovered additionally]
54
-
55
- ---
56
-
57
- #### 2. [Research Item Name]
58
- ...
59
-
60
- ### Summary
61
-
62
- #### Key Findings
63
- - [Important finding 1]
64
- - [Important finding 2]
65
-
66
- #### Caveats/Risks
67
- - [Discovered risks]
68
-
69
- #### Items Unable to Research
70
- - [Item]: [Reason]
71
-
72
- ### Recommendation/Conclusion
73
- [Recommendations based on research results]
74
-
75
- [DIGGER:DONE]
76
- ```
77
-
78
39
  ## Example: Naming Research Results
79
40
 
80
41
  ```
@@ -123,8 +84,6 @@ Scoped packages (@yourname/wolf etc.) can be used
123
84
  1. Least GitHub collisions
124
85
  2. npm addressable via scoped packages
125
86
  3. "Hawk" image fits surveillance/hunting tools
126
-
127
- [DIGGER:DONE]
128
87
  ```
129
88
 
130
89
  ## Important