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.
- package/dist/agents/runner.d.ts +2 -0
- package/dist/agents/runner.d.ts.map +1 -1
- package/dist/agents/runner.js +10 -2
- package/dist/agents/runner.js.map +1 -1
- package/dist/claude/client.d.ts.map +1 -1
- package/dist/claude/client.js +4 -1
- package/dist/claude/client.js.map +1 -1
- package/dist/claude/executor.d.ts.map +1 -1
- package/dist/claude/executor.js +1 -0
- package/dist/claude/executor.js.map +1 -1
- package/dist/claude/types.d.ts +2 -0
- package/dist/claude/types.d.ts.map +1 -1
- package/dist/commands/workflowExecution.d.ts.map +1 -1
- package/dist/commands/workflowExecution.js +25 -2
- package/dist/commands/workflowExecution.js.map +1 -1
- package/dist/config/workflowLoader.d.ts.map +1 -1
- package/dist/config/workflowLoader.js +1 -0
- package/dist/config/workflowLoader.js.map +1 -1
- package/dist/models/schemas.d.ts +2 -0
- package/dist/models/schemas.d.ts.map +1 -1
- package/dist/models/schemas.js +1 -0
- package/dist/models/schemas.js.map +1 -1
- package/dist/models/types.d.ts +4 -0
- package/dist/models/types.d.ts.map +1 -1
- package/dist/workflow/engine.d.ts.map +1 -1
- package/dist/workflow/engine.js +7 -3
- package/dist/workflow/engine.js.map +1 -1
- package/dist/workflow/instruction-builder.d.ts +5 -2
- package/dist/workflow/instruction-builder.d.ts.map +1 -1
- package/dist/workflow/instruction-builder.js +8 -2
- package/dist/workflow/instruction-builder.js.map +1 -1
- package/dist/workflow/state-manager.d.ts +4 -0
- package/dist/workflow/state-manager.d.ts.map +1 -1
- package/dist/workflow/state-manager.js +10 -0
- package/dist/workflow/state-manager.js.map +1 -1
- package/package.json +1 -1
- package/resources/global/en/agents/default/ai-reviewer.md +0 -20
- package/resources/global/en/agents/default/architect.md +5 -73
- package/resources/global/en/agents/default/coder.md +0 -39
- package/resources/global/en/agents/default/planner.md +0 -23
- package/resources/global/en/agents/default/security.md +0 -16
- package/resources/global/en/agents/default/supervisor.md +0 -19
- package/resources/global/en/agents/expert-review/cqrs-es-reviewer.md +0 -35
- package/resources/global/en/agents/expert-review/frontend-reviewer.md +0 -35
- package/resources/global/en/agents/expert-review/qa-reviewer.md +0 -36
- package/resources/global/en/agents/expert-review/security-reviewer.md +0 -37
- package/resources/global/en/agents/expert-review/supervisor.md +0 -62
- package/resources/global/en/agents/magi/balthasar.md +0 -20
- package/resources/global/en/agents/magi/casper.md +0 -42
- package/resources/global/en/agents/magi/melchior.md +0 -20
- package/resources/global/en/agents/research/digger.md +0 -41
- package/resources/global/en/agents/research/planner.md +0 -34
- package/resources/global/en/agents/research/supervisor.md +0 -36
- package/resources/global/en/workflows/default.yaml +419 -46
- package/resources/global/en/workflows/expert-review.yaml +217 -22
- package/resources/global/en/workflows/magi.yaml +78 -26
- package/resources/global/en/workflows/research.yaml +105 -11
- package/resources/global/ja/agents/default/ai-reviewer.md +0 -20
- package/resources/global/ja/agents/default/architect.md +5 -73
- package/resources/global/ja/agents/default/coder.md +1 -41
- package/resources/global/ja/agents/default/planner.md +0 -23
- package/resources/global/ja/agents/default/security.md +0 -16
- package/resources/global/ja/agents/default/supervisor.md +0 -19
- package/resources/global/ja/agents/expert-review/cqrs-es-reviewer.md +0 -35
- package/resources/global/ja/agents/expert-review/frontend-reviewer.md +0 -35
- package/resources/global/ja/agents/expert-review/qa-reviewer.md +0 -36
- package/resources/global/ja/agents/expert-review/security-reviewer.md +0 -37
- package/resources/global/ja/agents/expert-review/supervisor.md +0 -62
- package/resources/global/ja/agents/magi/balthasar.md +0 -20
- package/resources/global/ja/agents/magi/casper.md +0 -42
- package/resources/global/ja/agents/magi/melchior.md +0 -20
- package/resources/global/ja/agents/research/digger.md +0 -41
- package/resources/global/ja/agents/research/planner.md +0 -34
- package/resources/global/ja/agents/research/supervisor.md +0 -36
- package/resources/global/ja/workflows/default.yaml +419 -50
- package/resources/global/ja/workflows/expert-review.yaml +217 -22
- package/resources/global/ja/workflows/magi.yaml +78 -26
- 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,
|
|
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
|
@@ -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
|
-
|
|
261
|
-
|
|
262
|
-
|
|
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
|