takt 0.43.0 → 0.44.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.
- package/README.md +4 -2
- package/builtins/en/config.yaml +3 -1
- package/builtins/en/workflows/backend-cqrs.yaml +21 -0
- package/builtins/en/workflows/backend.yaml +21 -0
- package/builtins/en/workflows/dual-cqrs.yaml +21 -0
- package/builtins/en/workflows/dual.yaml +21 -0
- package/builtins/en/workflows/frontend-maintenance.yaml +21 -0
- package/builtins/en/workflows/frontend.yaml +21 -0
- package/builtins/en/workflows/review-backend-cqrs.yaml +22 -0
- package/builtins/en/workflows/review-backend.yaml +22 -0
- package/builtins/en/workflows/review-default.yaml +27 -2
- package/builtins/en/workflows/review-dual-cqrs.yaml +22 -0
- package/builtins/en/workflows/review-dual.yaml +22 -0
- package/builtins/en/workflows/review-fix-backend-cqrs.yaml +22 -0
- package/builtins/en/workflows/review-fix-backend.yaml +22 -0
- package/builtins/en/workflows/review-fix-default.yaml +23 -1
- package/builtins/en/workflows/review-fix-dual-cqrs.yaml +22 -0
- package/builtins/en/workflows/review-fix-dual.yaml +22 -0
- package/builtins/en/workflows/review-fix-frontend.yaml +22 -0
- package/builtins/en/workflows/review-fix-takt-default.yaml +23 -1
- package/builtins/en/workflows/review-frontend.yaml +22 -0
- package/builtins/en/workflows/review-takt-default.yaml +27 -2
- package/builtins/en/workflows/terraform.yaml +23 -2
- package/builtins/ja/config.yaml +3 -1
- package/builtins/ja/workflows/backend-cqrs.yaml +21 -0
- package/builtins/ja/workflows/backend.yaml +21 -0
- package/builtins/ja/workflows/dual-cqrs.yaml +21 -0
- package/builtins/ja/workflows/dual.yaml +21 -0
- package/builtins/ja/workflows/frontend-maintenance.yaml +21 -0
- package/builtins/ja/workflows/frontend.yaml +21 -0
- package/builtins/ja/workflows/review-backend-cqrs.yaml +22 -0
- package/builtins/ja/workflows/review-backend.yaml +22 -0
- package/builtins/ja/workflows/review-default.yaml +26 -2
- package/builtins/ja/workflows/review-dual-cqrs.yaml +22 -0
- package/builtins/ja/workflows/review-dual.yaml +22 -0
- package/builtins/ja/workflows/review-fix-backend-cqrs.yaml +22 -0
- package/builtins/ja/workflows/review-fix-backend.yaml +22 -0
- package/builtins/ja/workflows/review-fix-default.yaml +23 -1
- package/builtins/ja/workflows/review-fix-dual-cqrs.yaml +22 -0
- package/builtins/ja/workflows/review-fix-dual.yaml +22 -0
- package/builtins/ja/workflows/review-fix-frontend.yaml +22 -0
- package/builtins/ja/workflows/review-fix-takt-default.yaml +23 -1
- package/builtins/ja/workflows/review-frontend.yaml +22 -0
- package/builtins/ja/workflows/review-takt-default.yaml +26 -2
- package/builtins/ja/workflows/terraform.yaml +23 -2
- package/dist/app/cli/program.js +1 -1
- package/dist/app/cli/program.js.map +1 -1
- package/dist/commands/repertoire/add.d.ts.map +1 -1
- package/dist/commands/repertoire/add.js +7 -11
- package/dist/commands/repertoire/add.js.map +1 -1
- package/dist/core/logging/span-to-ndjson-mapper.d.ts +10 -0
- package/dist/core/logging/span-to-ndjson-mapper.d.ts.map +1 -0
- package/dist/core/logging/span-to-ndjson-mapper.js +311 -0
- package/dist/core/logging/span-to-ndjson-mapper.js.map +1 -0
- package/dist/core/models/config-schemas.d.ts +40 -0
- package/dist/core/models/config-schemas.d.ts.map +1 -1
- package/dist/core/models/config-schemas.js +2 -0
- package/dist/core/models/config-schemas.js.map +1 -1
- package/dist/core/models/config-types.d.ts +8 -2
- package/dist/core/models/config-types.d.ts.map +1 -1
- package/dist/core/models/provider-profiles.d.ts +2 -1
- package/dist/core/models/provider-profiles.d.ts.map +1 -1
- package/dist/core/models/provider-profiles.js +1 -1
- package/dist/core/models/provider-profiles.js.map +1 -1
- package/dist/core/models/schema-base.d.ts +27 -0
- package/dist/core/models/schema-base.d.ts.map +1 -1
- package/dist/core/models/schema-base.js +2 -0
- package/dist/core/models/schema-base.js.map +1 -1
- package/dist/core/models/workflow-schemas.d.ts +59 -32
- package/dist/core/models/workflow-schemas.d.ts.map +1 -1
- package/dist/core/workflow/engine/ArpeggioRunner.d.ts +6 -1
- package/dist/core/workflow/engine/ArpeggioRunner.d.ts.map +1 -1
- package/dist/core/workflow/engine/ArpeggioRunner.js +88 -51
- package/dist/core/workflow/engine/ArpeggioRunner.js.map +1 -1
- package/dist/core/workflow/engine/OptionsBuilder.d.ts +3 -2
- package/dist/core/workflow/engine/OptionsBuilder.d.ts.map +1 -1
- package/dist/core/workflow/engine/OptionsBuilder.js +8 -1
- package/dist/core/workflow/engine/OptionsBuilder.js.map +1 -1
- package/dist/core/workflow/engine/ParallelRunner.d.ts +6 -1
- package/dist/core/workflow/engine/ParallelRunner.d.ts.map +1 -1
- package/dist/core/workflow/engine/ParallelRunner.js +31 -3
- package/dist/core/workflow/engine/ParallelRunner.js.map +1 -1
- package/dist/core/workflow/engine/StepExecutor.d.ts +5 -1
- package/dist/core/workflow/engine/StepExecutor.d.ts.map +1 -1
- package/dist/core/workflow/engine/StepExecutor.js +31 -3
- package/dist/core/workflow/engine/StepExecutor.js.map +1 -1
- package/dist/core/workflow/engine/TeamLeaderRunner.d.ts +6 -1
- package/dist/core/workflow/engine/TeamLeaderRunner.d.ts.map +1 -1
- package/dist/core/workflow/engine/TeamLeaderRunner.js +41 -7
- package/dist/core/workflow/engine/TeamLeaderRunner.js.map +1 -1
- package/dist/core/workflow/engine/WorkflowEngine.d.ts.map +1 -1
- package/dist/core/workflow/engine/WorkflowEngine.js +7 -0
- package/dist/core/workflow/engine/WorkflowEngine.js.map +1 -1
- package/dist/core/workflow/engine/WorkflowEngineSetup.d.ts.map +1 -1
- package/dist/core/workflow/engine/WorkflowEngineSetup.js +21 -1
- package/dist/core/workflow/engine/WorkflowEngineSetup.js.map +1 -1
- package/dist/core/workflow/engine/WorkflowRunLoop.d.ts +2 -0
- package/dist/core/workflow/engine/WorkflowRunLoop.d.ts.map +1 -1
- package/dist/core/workflow/engine/WorkflowRunLoop.js +10 -0
- package/dist/core/workflow/engine/WorkflowRunLoop.js.map +1 -1
- package/dist/core/workflow/engine/team-leader-part-runner.d.ts +10 -2
- package/dist/core/workflow/engine/team-leader-part-runner.d.ts.map +1 -1
- package/dist/core/workflow/engine/team-leader-part-runner.js +19 -2
- package/dist/core/workflow/engine/team-leader-part-runner.js.map +1 -1
- package/dist/core/workflow/observability/workflowSpans.d.ts +45 -2
- package/dist/core/workflow/observability/workflowSpans.d.ts.map +1 -1
- package/dist/core/workflow/observability/workflowSpans.js +270 -8
- package/dist/core/workflow/observability/workflowSpans.js.map +1 -1
- package/dist/core/workflow/permission-profile-resolution.d.ts +1 -1
- package/dist/core/workflow/permission-profile-resolution.d.ts.map +1 -1
- package/dist/core/workflow/permission-profile-resolution.js +10 -8
- package/dist/core/workflow/permission-profile-resolution.js.map +1 -1
- package/dist/core/workflow/phase-runner.d.ts +11 -1
- package/dist/core/workflow/phase-runner.d.ts.map +1 -1
- package/dist/core/workflow/phase-runner.js.map +1 -1
- package/dist/core/workflow/report-phase-runner.d.ts.map +1 -1
- package/dist/core/workflow/report-phase-runner.js +47 -11
- package/dist/core/workflow/report-phase-runner.js.map +1 -1
- package/dist/core/workflow/status-judgment-phase.d.ts.map +1 -1
- package/dist/core/workflow/status-judgment-phase.js +36 -2
- package/dist/core/workflow/status-judgment-phase.js.map +1 -1
- package/dist/core/workflow/types.d.ts +4 -0
- package/dist/core/workflow/types.d.ts.map +1 -1
- package/dist/features/repertoire/takt-repertoire-config.d.ts.map +1 -1
- package/dist/features/repertoire/takt-repertoire-config.js +2 -1
- package/dist/features/repertoire/takt-repertoire-config.js.map +1 -1
- package/dist/features/tasks/execute/workflowExecution.js +2 -0
- package/dist/features/tasks/execute/workflowExecution.js.map +1 -1
- package/dist/features/tasks/execute/workflowExecutionBootstrap.d.ts +1 -0
- package/dist/features/tasks/execute/workflowExecutionBootstrap.d.ts.map +1 -1
- package/dist/features/tasks/execute/workflowExecutionBootstrap.js +20 -1
- package/dist/features/tasks/execute/workflowExecutionBootstrap.js.map +1 -1
- package/dist/infra/codex/client.d.ts.map +1 -1
- package/dist/infra/codex/client.js +20 -7
- package/dist/infra/codex/client.js.map +1 -1
- package/dist/infra/config/configNormalizers.d.ts +2 -1
- package/dist/infra/config/configNormalizers.d.ts.map +1 -1
- package/dist/infra/config/configNormalizers.js.map +1 -1
- package/dist/infra/config/env/global-current-env-specs.d.ts.map +1 -1
- package/dist/infra/config/env/global-current-env-specs.js +2 -0
- package/dist/infra/config/env/global-current-env-specs.js.map +1 -1
- package/dist/infra/config/global/globalConfig.d.ts +1 -1
- package/dist/infra/config/global/globalConfig.d.ts.map +1 -1
- package/dist/infra/config/global/globalConfig.js +1 -1
- package/dist/infra/config/global/globalConfig.js.map +1 -1
- package/dist/infra/config/global/globalConfigCore.d.ts.map +1 -1
- package/dist/infra/config/global/globalConfigCore.js +2 -0
- package/dist/infra/config/global/globalConfigCore.js.map +1 -1
- package/dist/infra/config/global/globalConfigResolvers.d.ts +2 -0
- package/dist/infra/config/global/globalConfigResolvers.d.ts.map +1 -1
- package/dist/infra/config/global/globalConfigResolvers.js +20 -0
- package/dist/infra/config/global/globalConfigResolvers.js.map +1 -1
- package/dist/infra/config/global/globalConfigSerializer.d.ts.map +1 -1
- package/dist/infra/config/global/globalConfigSerializer.js +6 -0
- package/dist/infra/config/global/globalConfigSerializer.js.map +1 -1
- package/dist/infra/config/global/index.d.ts +1 -1
- package/dist/infra/config/global/index.d.ts.map +1 -1
- package/dist/infra/config/global/index.js +1 -1
- package/dist/infra/config/global/index.js.map +1 -1
- package/dist/infra/config/global/initialization.d.ts +4 -1
- package/dist/infra/config/global/initialization.d.ts.map +1 -1
- package/dist/infra/config/global/initialization.js +1 -0
- package/dist/infra/config/global/initialization.js.map +1 -1
- package/dist/infra/config/traced/tracedConfigSchema.d.ts.map +1 -1
- package/dist/infra/config/traced/tracedConfigSchema.js +4 -0
- package/dist/infra/config/traced/tracedConfigSchema.js.map +1 -1
- package/dist/infra/kiro/client.d.ts +8 -0
- package/dist/infra/kiro/client.d.ts.map +1 -0
- package/dist/infra/kiro/client.js +177 -0
- package/dist/infra/kiro/client.js.map +1 -0
- package/dist/infra/kiro/index.d.ts +2 -0
- package/dist/infra/kiro/index.d.ts.map +1 -0
- package/dist/infra/kiro/index.js +2 -0
- package/dist/infra/kiro/index.js.map +1 -0
- package/dist/infra/kiro/process.d.ts +13 -0
- package/dist/infra/kiro/process.d.ts.map +1 -0
- package/dist/infra/kiro/process.js +212 -0
- package/dist/infra/kiro/process.js.map +1 -0
- package/dist/infra/kiro/types.d.ts +13 -0
- package/dist/infra/kiro/types.d.ts.map +1 -0
- package/dist/infra/kiro/types.js +2 -0
- package/dist/infra/kiro/types.js.map +1 -0
- package/dist/infra/observability/monitorJsonMetricExporter.d.ts +17 -0
- package/dist/infra/observability/monitorJsonMetricExporter.d.ts.map +1 -0
- package/dist/infra/observability/monitorJsonMetricExporter.js +174 -0
- package/dist/infra/observability/monitorJsonMetricExporter.js.map +1 -0
- package/dist/infra/observability/otelFoundation.d.ts +7 -1
- package/dist/infra/observability/otelFoundation.d.ts.map +1 -1
- package/dist/infra/observability/otelFoundation.js +57 -7
- package/dist/infra/observability/otelFoundation.js.map +1 -1
- package/dist/infra/observability/sessionLogSpanProcessor.d.ts +22 -0
- package/dist/infra/observability/sessionLogSpanProcessor.d.ts.map +1 -0
- package/dist/infra/observability/sessionLogSpanProcessor.js +127 -0
- package/dist/infra/observability/sessionLogSpanProcessor.js.map +1 -0
- package/dist/infra/providers/index.d.ts.map +1 -1
- package/dist/infra/providers/index.js +2 -0
- package/dist/infra/providers/index.js.map +1 -1
- package/dist/infra/providers/kiro.d.ts +6 -0
- package/dist/infra/providers/kiro.d.ts.map +1 -0
- package/dist/infra/providers/kiro.js +43 -0
- package/dist/infra/providers/kiro.js.map +1 -0
- package/dist/infra/providers/types.d.ts +1 -0
- package/dist/infra/providers/types.d.ts.map +1 -1
- package/dist/infra/task/clone-errors.d.ts +3 -0
- package/dist/infra/task/clone-errors.d.ts.map +1 -0
- package/dist/infra/task/clone-errors.js +5 -0
- package/dist/infra/task/clone-errors.js.map +1 -0
- package/dist/infra/task/clone-exec.d.ts +3 -1
- package/dist/infra/task/clone-exec.d.ts.map +1 -1
- package/dist/infra/task/clone-exec.js +118 -49
- package/dist/infra/task/clone-exec.js.map +1 -1
- package/dist/infra/task/clone.d.ts.map +1 -1
- package/dist/infra/task/clone.js +5 -9
- package/dist/infra/task/clone.js.map +1 -1
- package/dist/shared/types/provider.d.ts +1 -1
- package/dist/shared/types/provider.d.ts.map +1 -1
- package/dist/shared/utils/spawn.d.ts +3 -4
- package/dist/shared/utils/spawn.d.ts.map +1 -1
- package/dist/shared/utils/spawn.js +6 -9
- package/dist/shared/utils/spawn.js.map +1 -1
- package/package.json +5 -3
package/README.md
CHANGED
|
@@ -4,7 +4,7 @@
|
|
|
4
4
|
|
|
5
5
|
**T**AKT **A**gent **K**oordination **T**opology — Orchestrate multiple AI agents with structured review loops, managed prompts, and guardrails.
|
|
6
6
|
|
|
7
|
-
Talk to AI to define what you want, queue it as a task, and run it with `takt run`. Planning, implementation, review, and fix loops are defined in YAML workflow files, so the process is not left to the agent's discretion. TAKT coordinates Claude Code, Codex, OpenCode, Cursor,
|
|
7
|
+
Talk to AI to define what you want, queue it as a task, and run it with `takt run`. Planning, implementation, review, and fix loops are defined in YAML workflow files, so the process is not left to the agent's discretion. TAKT coordinates Claude Code, Codex, OpenCode, Cursor, GitHub Copilot CLI, and Kiro CLI as agents with different roles, permissions, and context.
|
|
8
8
|
|
|
9
9
|
TAKT is built primarily for AI coding workflows, but the same model applies beyond coding: any task where multiple AI agents need to coordinate, or where review, judgment, and feedback loops can improve task quality.
|
|
10
10
|
|
|
@@ -42,6 +42,7 @@ These providers require an external CLI:
|
|
|
42
42
|
- `claude-terminal` — [Claude Code](https://claude.ai/code) driven in an interactive terminal session (also requires [`tmux`](https://github.com/tmux/tmux))
|
|
43
43
|
- `copilot` — [GitHub Copilot CLI](https://docs.github.com/en/copilot/github-copilot-in-the-cli)
|
|
44
44
|
- `cursor` — [Cursor Agent](https://docs.cursor.com/)
|
|
45
|
+
- `kiro` — [Kiro CLI](https://kiro.dev/docs/cli/headless/)
|
|
45
46
|
|
|
46
47
|
Optional:
|
|
47
48
|
|
|
@@ -184,7 +185,7 @@ See the [CLI Reference](./docs/cli-reference.md) for all commands and options.
|
|
|
184
185
|
Minimal `~/.takt/config.yaml`:
|
|
185
186
|
|
|
186
187
|
```yaml
|
|
187
|
-
provider: claude # claude, claude-sdk, claude-terminal, codex, opencode, cursor, or
|
|
188
|
+
provider: claude # claude, claude-sdk, claude-terminal, codex, opencode, cursor, copilot, kiro, or mock
|
|
188
189
|
model: sonnet # passed directly to provider
|
|
189
190
|
language: en # en or ja
|
|
190
191
|
```
|
|
@@ -197,6 +198,7 @@ export TAKT_OPENAI_API_KEY=sk-... # OpenAI (Codex)
|
|
|
197
198
|
export TAKT_OPENCODE_API_KEY=... # OpenCode
|
|
198
199
|
export TAKT_CURSOR_API_KEY=... # Cursor Agent (optional if logged in)
|
|
199
200
|
export TAKT_COPILOT_GITHUB_TOKEN=ghp_... # GitHub Copilot CLI
|
|
201
|
+
export TAKT_KIRO_API_KEY=... # Kiro CLI
|
|
200
202
|
```
|
|
201
203
|
|
|
202
204
|
See the [Configuration Guide](./docs/configuration.md) for all options, provider profiles, and model resolution.
|
package/builtins/en/config.yaml
CHANGED
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
language: en # UI language: en | ja
|
|
8
8
|
|
|
9
9
|
# Default provider and model
|
|
10
|
-
# provider: claude # Default provider: claude | codex | opencode | cursor | copilot | mock
|
|
10
|
+
# provider: claude # Default provider: claude | claude-sdk | claude-terminal | codex | opencode | cursor | copilot | kiro | mock
|
|
11
11
|
# model: sonnet # Default model (passed directly to provider)
|
|
12
12
|
|
|
13
13
|
# Execution control
|
|
@@ -122,12 +122,14 @@ language: en # UI language: en | ja
|
|
|
122
122
|
# openrouter_api_key: "..." # OpenRouter API key
|
|
123
123
|
# opencode_api_key: "..." # OpenCode API key
|
|
124
124
|
# cursor_api_key: "..." # Cursor API key
|
|
125
|
+
# kiro_api_key: "..." # Kiro API key
|
|
125
126
|
|
|
126
127
|
# CLI paths
|
|
127
128
|
# codex_cli_path: "/absolute/path/to/codex" # Absolute path to Codex CLI
|
|
128
129
|
# claude_cli_path: "/absolute/path/to/claude" # Absolute path to Claude Code CLI
|
|
129
130
|
# cursor_cli_path: "/absolute/path/to/cursor" # Absolute path to cursor-agent CLI
|
|
130
131
|
# copilot_cli_path: "/absolute/path/to/copilot" # Absolute path to Copilot CLI
|
|
132
|
+
# kiro_cli_path: "/absolute/path/to/kiro-cli" # Absolute path to Kiro CLI
|
|
131
133
|
# copilot_github_token: "ghp_..." # Copilot GitHub token
|
|
132
134
|
|
|
133
135
|
# Misc
|
|
@@ -335,6 +335,27 @@ steps:
|
|
|
335
335
|
report:
|
|
336
336
|
- name: qa-review.md
|
|
337
337
|
format: qa-review
|
|
338
|
+
- name: coding-review
|
|
339
|
+
edit: false
|
|
340
|
+
persona: coding-reviewer
|
|
341
|
+
policy: review
|
|
342
|
+
provider_options:
|
|
343
|
+
claude:
|
|
344
|
+
allowed_tools:
|
|
345
|
+
- Read
|
|
346
|
+
- Glob
|
|
347
|
+
- Grep
|
|
348
|
+
- Bash
|
|
349
|
+
- WebSearch
|
|
350
|
+
- WebFetch
|
|
351
|
+
instruction: review-coding
|
|
352
|
+
output_contracts:
|
|
353
|
+
report:
|
|
354
|
+
- name: coding-review.md
|
|
355
|
+
format: coding-review
|
|
356
|
+
rules:
|
|
357
|
+
- condition: approved
|
|
358
|
+
- condition: needs_fix
|
|
338
359
|
rules:
|
|
339
360
|
- condition: all("approved")
|
|
340
361
|
next: supervise
|
|
@@ -331,6 +331,27 @@ steps:
|
|
|
331
331
|
report:
|
|
332
332
|
- name: qa-review.md
|
|
333
333
|
format: qa-review
|
|
334
|
+
- name: coding-review
|
|
335
|
+
edit: false
|
|
336
|
+
persona: coding-reviewer
|
|
337
|
+
policy: review
|
|
338
|
+
provider_options:
|
|
339
|
+
claude:
|
|
340
|
+
allowed_tools:
|
|
341
|
+
- Read
|
|
342
|
+
- Glob
|
|
343
|
+
- Grep
|
|
344
|
+
- Bash
|
|
345
|
+
- WebSearch
|
|
346
|
+
- WebFetch
|
|
347
|
+
instruction: review-coding
|
|
348
|
+
output_contracts:
|
|
349
|
+
report:
|
|
350
|
+
- name: coding-review.md
|
|
351
|
+
format: coding-review
|
|
352
|
+
rules:
|
|
353
|
+
- condition: approved
|
|
354
|
+
- condition: needs_fix
|
|
334
355
|
rules:
|
|
335
356
|
- condition: all("approved")
|
|
336
357
|
next: supervise
|
|
@@ -295,6 +295,27 @@ steps:
|
|
|
295
295
|
report:
|
|
296
296
|
- name: ai-antipattern-review.md
|
|
297
297
|
format: ai-antipattern-review
|
|
298
|
+
- name: coding-review
|
|
299
|
+
edit: false
|
|
300
|
+
persona: coding-reviewer
|
|
301
|
+
policy: review
|
|
302
|
+
provider_options:
|
|
303
|
+
claude:
|
|
304
|
+
allowed_tools:
|
|
305
|
+
- Read
|
|
306
|
+
- Glob
|
|
307
|
+
- Grep
|
|
308
|
+
- Bash
|
|
309
|
+
- WebSearch
|
|
310
|
+
- WebFetch
|
|
311
|
+
instruction: review-coding
|
|
312
|
+
output_contracts:
|
|
313
|
+
report:
|
|
314
|
+
- name: coding-review.md
|
|
315
|
+
format: coding-review
|
|
316
|
+
rules:
|
|
317
|
+
- condition: approved
|
|
318
|
+
- condition: needs_fix
|
|
298
319
|
rules:
|
|
299
320
|
- condition: all("approved")
|
|
300
321
|
next: supervise
|
|
@@ -421,6 +421,27 @@ steps:
|
|
|
421
421
|
report:
|
|
422
422
|
- name: requirements-review.md
|
|
423
423
|
format: requirements-review
|
|
424
|
+
- name: coding-review
|
|
425
|
+
edit: false
|
|
426
|
+
persona: coding-reviewer
|
|
427
|
+
policy: review
|
|
428
|
+
provider_options:
|
|
429
|
+
claude:
|
|
430
|
+
allowed_tools:
|
|
431
|
+
- Read
|
|
432
|
+
- Glob
|
|
433
|
+
- Grep
|
|
434
|
+
- Bash
|
|
435
|
+
- WebSearch
|
|
436
|
+
- WebFetch
|
|
437
|
+
instruction: review-coding
|
|
438
|
+
output_contracts:
|
|
439
|
+
report:
|
|
440
|
+
- name: coding-review.md
|
|
441
|
+
format: coding-review
|
|
442
|
+
rules:
|
|
443
|
+
- condition: approved
|
|
444
|
+
- condition: needs_fix
|
|
424
445
|
rules:
|
|
425
446
|
- condition: all("approved")
|
|
426
447
|
next: supervise
|
|
@@ -397,6 +397,27 @@ steps:
|
|
|
397
397
|
format: qa-review
|
|
398
398
|
knowledge:
|
|
399
399
|
- existing-system
|
|
400
|
+
- name: coding-review
|
|
401
|
+
edit: false
|
|
402
|
+
persona: coding-reviewer
|
|
403
|
+
policy: review
|
|
404
|
+
provider_options:
|
|
405
|
+
claude:
|
|
406
|
+
allowed_tools:
|
|
407
|
+
- Read
|
|
408
|
+
- Glob
|
|
409
|
+
- Grep
|
|
410
|
+
- Bash
|
|
411
|
+
- WebSearch
|
|
412
|
+
- WebFetch
|
|
413
|
+
instruction: review-coding
|
|
414
|
+
output_contracts:
|
|
415
|
+
report:
|
|
416
|
+
- name: coding-review.md
|
|
417
|
+
format: coding-review
|
|
418
|
+
rules:
|
|
419
|
+
- condition: approved
|
|
420
|
+
- condition: needs_fix
|
|
400
421
|
rules:
|
|
401
422
|
- condition: all("approved")
|
|
402
423
|
next: supervise
|
|
@@ -365,6 +365,27 @@ steps:
|
|
|
365
365
|
report:
|
|
366
366
|
- name: qa-review.md
|
|
367
367
|
format: qa-review
|
|
368
|
+
- name: coding-review
|
|
369
|
+
edit: false
|
|
370
|
+
persona: coding-reviewer
|
|
371
|
+
policy: review
|
|
372
|
+
provider_options:
|
|
373
|
+
claude:
|
|
374
|
+
allowed_tools:
|
|
375
|
+
- Read
|
|
376
|
+
- Glob
|
|
377
|
+
- Grep
|
|
378
|
+
- Bash
|
|
379
|
+
- WebSearch
|
|
380
|
+
- WebFetch
|
|
381
|
+
instruction: review-coding
|
|
382
|
+
output_contracts:
|
|
383
|
+
report:
|
|
384
|
+
- name: coding-review.md
|
|
385
|
+
format: coding-review
|
|
386
|
+
rules:
|
|
387
|
+
- condition: approved
|
|
388
|
+
- condition: needs_fix
|
|
368
389
|
rules:
|
|
369
390
|
- condition: all("approved")
|
|
370
391
|
next: supervise
|
|
@@ -128,6 +128,28 @@ steps:
|
|
|
128
128
|
- condition: approved
|
|
129
129
|
- condition: needs_fix
|
|
130
130
|
|
|
131
|
+
- name: coding-review
|
|
132
|
+
edit: false
|
|
133
|
+
persona: coding-reviewer
|
|
134
|
+
policy: review
|
|
135
|
+
provider_options:
|
|
136
|
+
claude:
|
|
137
|
+
allowed_tools:
|
|
138
|
+
- Read
|
|
139
|
+
- Glob
|
|
140
|
+
- Grep
|
|
141
|
+
- Bash
|
|
142
|
+
- WebSearch
|
|
143
|
+
- WebFetch
|
|
144
|
+
instruction: review-coding
|
|
145
|
+
output_contracts:
|
|
146
|
+
report:
|
|
147
|
+
- name: coding-review.md
|
|
148
|
+
format: coding-review
|
|
149
|
+
rules:
|
|
150
|
+
- condition: approved
|
|
151
|
+
- condition: needs_fix
|
|
152
|
+
|
|
131
153
|
rules:
|
|
132
154
|
- condition: all("approved")
|
|
133
155
|
next: supervise
|
|
@@ -104,6 +104,28 @@ steps:
|
|
|
104
104
|
- condition: approved
|
|
105
105
|
- condition: needs_fix
|
|
106
106
|
|
|
107
|
+
- name: coding-review
|
|
108
|
+
edit: false
|
|
109
|
+
persona: coding-reviewer
|
|
110
|
+
policy: review
|
|
111
|
+
provider_options:
|
|
112
|
+
claude:
|
|
113
|
+
allowed_tools:
|
|
114
|
+
- Read
|
|
115
|
+
- Glob
|
|
116
|
+
- Grep
|
|
117
|
+
- Bash
|
|
118
|
+
- WebSearch
|
|
119
|
+
- WebFetch
|
|
120
|
+
instruction: review-coding
|
|
121
|
+
output_contracts:
|
|
122
|
+
report:
|
|
123
|
+
- name: coding-review.md
|
|
124
|
+
format: coding-review
|
|
125
|
+
rules:
|
|
126
|
+
- condition: approved
|
|
127
|
+
- condition: needs_fix
|
|
128
|
+
|
|
107
129
|
rules:
|
|
108
130
|
- condition: all("approved")
|
|
109
131
|
next: supervise
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
name: review-default
|
|
2
|
-
description: Multi-perspective Code Review - reviews changes in parallel from
|
|
2
|
+
description: Multi-perspective Code Review - reviews changes in parallel from 6 perspectives (including coding review) and outputs consolidated results
|
|
3
3
|
workflow_config:
|
|
4
4
|
provider_options:
|
|
5
5
|
codex:
|
|
@@ -158,6 +158,28 @@ steps:
|
|
|
158
158
|
- condition: approved
|
|
159
159
|
- condition: needs_fix
|
|
160
160
|
|
|
161
|
+
- name: coding-review
|
|
162
|
+
edit: false
|
|
163
|
+
persona: coding-reviewer
|
|
164
|
+
policy: review
|
|
165
|
+
provider_options:
|
|
166
|
+
claude:
|
|
167
|
+
allowed_tools:
|
|
168
|
+
- Read
|
|
169
|
+
- Glob
|
|
170
|
+
- Grep
|
|
171
|
+
- Bash
|
|
172
|
+
- WebSearch
|
|
173
|
+
- WebFetch
|
|
174
|
+
instruction: review-coding
|
|
175
|
+
output_contracts:
|
|
176
|
+
report:
|
|
177
|
+
- name: coding-review.md
|
|
178
|
+
format: coding-review
|
|
179
|
+
rules:
|
|
180
|
+
- condition: approved
|
|
181
|
+
- condition: needs_fix
|
|
182
|
+
|
|
161
183
|
rules:
|
|
162
184
|
- condition: all("approved")
|
|
163
185
|
next: supervise
|
|
@@ -195,7 +217,8 @@ steps:
|
|
|
195
217
|
- `03-qa-review.md` (QA review)
|
|
196
218
|
- `04-testing-review.md` (Testing review)
|
|
197
219
|
- `05-requirements-review.md` (Requirements review)
|
|
198
|
-
|
|
220
|
+
- `06-coding-review.md` (Coding review)
|
|
221
|
+
2. Synthesize the 6 review results
|
|
199
222
|
3. Produce a consolidated review summary with overall verdict
|
|
200
223
|
|
|
201
224
|
**Review Summary output contract:**
|
|
@@ -215,6 +238,7 @@ steps:
|
|
|
215
238
|
| QA | APPROVE/REJECT | {Brief finding} |
|
|
216
239
|
| Testing | APPROVE/REJECT | {Brief finding} |
|
|
217
240
|
| Requirements | APPROVE/REJECT | {Brief finding} |
|
|
241
|
+
| Coding | APPROVE/REJECT | {Brief finding} |
|
|
218
242
|
|
|
219
243
|
## Issues Requiring Attention
|
|
220
244
|
| # | Severity | Source | Location | Issue |
|
|
@@ -244,6 +268,7 @@ steps:
|
|
|
244
268
|
| QA | APPROVE/REJECT | {Overview} |
|
|
245
269
|
| Testing | APPROVE/REJECT | {Overview} |
|
|
246
270
|
| Requirements | APPROVE/REJECT | {Overview} |
|
|
271
|
+
| Coding | APPROVE/REJECT | {Overview} |
|
|
247
272
|
|
|
248
273
|
## Current Iteration Findings (new)
|
|
249
274
|
| # | finding_id | Severity | Source | Location | Issue | Fix Suggestion |
|
|
@@ -153,6 +153,28 @@ steps:
|
|
|
153
153
|
- condition: approved
|
|
154
154
|
- condition: needs_fix
|
|
155
155
|
|
|
156
|
+
- name: coding-review
|
|
157
|
+
edit: false
|
|
158
|
+
persona: coding-reviewer
|
|
159
|
+
policy: review
|
|
160
|
+
provider_options:
|
|
161
|
+
claude:
|
|
162
|
+
allowed_tools:
|
|
163
|
+
- Read
|
|
164
|
+
- Glob
|
|
165
|
+
- Grep
|
|
166
|
+
- Bash
|
|
167
|
+
- WebSearch
|
|
168
|
+
- WebFetch
|
|
169
|
+
instruction: review-coding
|
|
170
|
+
output_contracts:
|
|
171
|
+
report:
|
|
172
|
+
- name: coding-review.md
|
|
173
|
+
format: coding-review
|
|
174
|
+
rules:
|
|
175
|
+
- condition: approved
|
|
176
|
+
- condition: needs_fix
|
|
177
|
+
|
|
156
178
|
rules:
|
|
157
179
|
- condition: all("approved")
|
|
158
180
|
next: supervise
|
|
@@ -129,6 +129,28 @@ steps:
|
|
|
129
129
|
- condition: approved
|
|
130
130
|
- condition: needs_fix
|
|
131
131
|
|
|
132
|
+
- name: coding-review
|
|
133
|
+
edit: false
|
|
134
|
+
persona: coding-reviewer
|
|
135
|
+
policy: review
|
|
136
|
+
provider_options:
|
|
137
|
+
claude:
|
|
138
|
+
allowed_tools:
|
|
139
|
+
- Read
|
|
140
|
+
- Glob
|
|
141
|
+
- Grep
|
|
142
|
+
- Bash
|
|
143
|
+
- WebSearch
|
|
144
|
+
- WebFetch
|
|
145
|
+
instruction: review-coding
|
|
146
|
+
output_contracts:
|
|
147
|
+
report:
|
|
148
|
+
- name: coding-review.md
|
|
149
|
+
format: coding-review
|
|
150
|
+
rules:
|
|
151
|
+
- condition: approved
|
|
152
|
+
- condition: needs_fix
|
|
153
|
+
|
|
132
154
|
rules:
|
|
133
155
|
- condition: all("approved")
|
|
134
156
|
next: supervise
|
|
@@ -141,6 +141,28 @@ steps:
|
|
|
141
141
|
- condition: approved
|
|
142
142
|
- condition: needs_fix
|
|
143
143
|
|
|
144
|
+
- name: coding-review
|
|
145
|
+
edit: false
|
|
146
|
+
persona: coding-reviewer
|
|
147
|
+
policy: review
|
|
148
|
+
provider_options:
|
|
149
|
+
claude:
|
|
150
|
+
allowed_tools:
|
|
151
|
+
- Read
|
|
152
|
+
- Glob
|
|
153
|
+
- Grep
|
|
154
|
+
- Bash
|
|
155
|
+
- WebSearch
|
|
156
|
+
- WebFetch
|
|
157
|
+
instruction: review-coding
|
|
158
|
+
output_contracts:
|
|
159
|
+
report:
|
|
160
|
+
- name: coding-review.md
|
|
161
|
+
format: coding-review
|
|
162
|
+
rules:
|
|
163
|
+
- condition: approved
|
|
164
|
+
- condition: needs_fix
|
|
165
|
+
|
|
144
166
|
rules:
|
|
145
167
|
- condition: all("approved")
|
|
146
168
|
next: supervise
|
|
@@ -117,6 +117,28 @@ steps:
|
|
|
117
117
|
- condition: approved
|
|
118
118
|
- condition: needs_fix
|
|
119
119
|
|
|
120
|
+
- name: coding-review
|
|
121
|
+
edit: false
|
|
122
|
+
persona: coding-reviewer
|
|
123
|
+
policy: review
|
|
124
|
+
provider_options:
|
|
125
|
+
claude:
|
|
126
|
+
allowed_tools:
|
|
127
|
+
- Read
|
|
128
|
+
- Glob
|
|
129
|
+
- Grep
|
|
130
|
+
- Bash
|
|
131
|
+
- WebSearch
|
|
132
|
+
- WebFetch
|
|
133
|
+
instruction: review-coding
|
|
134
|
+
output_contracts:
|
|
135
|
+
report:
|
|
136
|
+
- name: coding-review.md
|
|
137
|
+
format: coding-review
|
|
138
|
+
rules:
|
|
139
|
+
- condition: approved
|
|
140
|
+
- condition: needs_fix
|
|
141
|
+
|
|
120
142
|
rules:
|
|
121
143
|
- condition: all("approved")
|
|
122
144
|
next: supervise
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
name: review-fix-default
|
|
2
|
-
description: Multi-perspective review + fix loop (architecture, security, QA, testing, requirements —
|
|
2
|
+
description: Multi-perspective review + fix loop (architecture, security, QA, testing, requirements, coding — 6 parallel reviewers with iterative fixes)
|
|
3
3
|
workflow_config:
|
|
4
4
|
provider_options:
|
|
5
5
|
codex:
|
|
@@ -171,6 +171,28 @@ steps:
|
|
|
171
171
|
- condition: approved
|
|
172
172
|
- condition: needs_fix
|
|
173
173
|
|
|
174
|
+
- name: coding-review
|
|
175
|
+
edit: false
|
|
176
|
+
persona: coding-reviewer
|
|
177
|
+
policy: review
|
|
178
|
+
provider_options:
|
|
179
|
+
claude:
|
|
180
|
+
allowed_tools:
|
|
181
|
+
- Read
|
|
182
|
+
- Glob
|
|
183
|
+
- Grep
|
|
184
|
+
- Bash
|
|
185
|
+
- WebSearch
|
|
186
|
+
- WebFetch
|
|
187
|
+
instruction: review-coding
|
|
188
|
+
output_contracts:
|
|
189
|
+
report:
|
|
190
|
+
- name: coding-review.md
|
|
191
|
+
format: coding-review
|
|
192
|
+
rules:
|
|
193
|
+
- condition: approved
|
|
194
|
+
- condition: needs_fix
|
|
195
|
+
|
|
174
196
|
rules:
|
|
175
197
|
- condition: all("approved")
|
|
176
198
|
next: supervise
|
|
@@ -166,6 +166,28 @@ steps:
|
|
|
166
166
|
- condition: approved
|
|
167
167
|
- condition: needs_fix
|
|
168
168
|
|
|
169
|
+
- name: coding-review
|
|
170
|
+
edit: false
|
|
171
|
+
persona: coding-reviewer
|
|
172
|
+
policy: review
|
|
173
|
+
provider_options:
|
|
174
|
+
claude:
|
|
175
|
+
allowed_tools:
|
|
176
|
+
- Read
|
|
177
|
+
- Glob
|
|
178
|
+
- Grep
|
|
179
|
+
- Bash
|
|
180
|
+
- WebSearch
|
|
181
|
+
- WebFetch
|
|
182
|
+
instruction: review-coding
|
|
183
|
+
output_contracts:
|
|
184
|
+
report:
|
|
185
|
+
- name: coding-review.md
|
|
186
|
+
format: coding-review
|
|
187
|
+
rules:
|
|
188
|
+
- condition: approved
|
|
189
|
+
- condition: needs_fix
|
|
190
|
+
|
|
169
191
|
rules:
|
|
170
192
|
- condition: all("approved")
|
|
171
193
|
next: supervise
|
|
@@ -142,6 +142,28 @@ steps:
|
|
|
142
142
|
- condition: approved
|
|
143
143
|
- condition: needs_fix
|
|
144
144
|
|
|
145
|
+
- name: coding-review
|
|
146
|
+
edit: false
|
|
147
|
+
persona: coding-reviewer
|
|
148
|
+
policy: review
|
|
149
|
+
provider_options:
|
|
150
|
+
claude:
|
|
151
|
+
allowed_tools:
|
|
152
|
+
- Read
|
|
153
|
+
- Glob
|
|
154
|
+
- Grep
|
|
155
|
+
- Bash
|
|
156
|
+
- WebSearch
|
|
157
|
+
- WebFetch
|
|
158
|
+
instruction: review-coding
|
|
159
|
+
output_contracts:
|
|
160
|
+
report:
|
|
161
|
+
- name: coding-review.md
|
|
162
|
+
format: coding-review
|
|
163
|
+
rules:
|
|
164
|
+
- condition: approved
|
|
165
|
+
- condition: needs_fix
|
|
166
|
+
|
|
145
167
|
rules:
|
|
146
168
|
- condition: all("approved")
|
|
147
169
|
next: supervise
|
|
@@ -141,6 +141,28 @@ steps:
|
|
|
141
141
|
- condition: approved
|
|
142
142
|
- condition: needs_fix
|
|
143
143
|
|
|
144
|
+
- name: coding-review
|
|
145
|
+
edit: false
|
|
146
|
+
persona: coding-reviewer
|
|
147
|
+
policy: review
|
|
148
|
+
provider_options:
|
|
149
|
+
claude:
|
|
150
|
+
allowed_tools:
|
|
151
|
+
- Read
|
|
152
|
+
- Glob
|
|
153
|
+
- Grep
|
|
154
|
+
- Bash
|
|
155
|
+
- WebSearch
|
|
156
|
+
- WebFetch
|
|
157
|
+
instruction: review-coding
|
|
158
|
+
output_contracts:
|
|
159
|
+
report:
|
|
160
|
+
- name: coding-review.md
|
|
161
|
+
format: coding-review
|
|
162
|
+
rules:
|
|
163
|
+
- condition: approved
|
|
164
|
+
- condition: needs_fix
|
|
165
|
+
|
|
144
166
|
rules:
|
|
145
167
|
- condition: all("approved")
|
|
146
168
|
next: supervise
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
name: review-fix-takt-default
|
|
2
|
-
description: "TAKT-focused multi-perspective review + fix loop (
|
|
2
|
+
description: "TAKT-focused multi-perspective review + fix loop (7-parallel initial review including coding review, AI antipattern pre-review on fix iterations, supervisor-finding fixes)"
|
|
3
3
|
workflow_config:
|
|
4
4
|
provider_options:
|
|
5
5
|
codex:
|
|
@@ -209,6 +209,28 @@ steps:
|
|
|
209
209
|
- condition: approved
|
|
210
210
|
- condition: needs_fix
|
|
211
211
|
|
|
212
|
+
- name: coding-review
|
|
213
|
+
edit: false
|
|
214
|
+
persona: coding-reviewer
|
|
215
|
+
policy: review
|
|
216
|
+
provider_options:
|
|
217
|
+
claude:
|
|
218
|
+
allowed_tools:
|
|
219
|
+
- Read
|
|
220
|
+
- Glob
|
|
221
|
+
- Grep
|
|
222
|
+
- Bash
|
|
223
|
+
- WebSearch
|
|
224
|
+
- WebFetch
|
|
225
|
+
instruction: review-coding
|
|
226
|
+
output_contracts:
|
|
227
|
+
report:
|
|
228
|
+
- name: coding-review.md
|
|
229
|
+
format: coding-review
|
|
230
|
+
rules:
|
|
231
|
+
- condition: approved
|
|
232
|
+
- condition: needs_fix
|
|
233
|
+
|
|
212
234
|
rules:
|
|
213
235
|
- condition: all("approved")
|
|
214
236
|
next: supervise
|
|
@@ -128,6 +128,28 @@ steps:
|
|
|
128
128
|
- condition: approved
|
|
129
129
|
- condition: needs_fix
|
|
130
130
|
|
|
131
|
+
- name: coding-review
|
|
132
|
+
edit: false
|
|
133
|
+
persona: coding-reviewer
|
|
134
|
+
policy: review
|
|
135
|
+
provider_options:
|
|
136
|
+
claude:
|
|
137
|
+
allowed_tools:
|
|
138
|
+
- Read
|
|
139
|
+
- Glob
|
|
140
|
+
- Grep
|
|
141
|
+
- Bash
|
|
142
|
+
- WebSearch
|
|
143
|
+
- WebFetch
|
|
144
|
+
instruction: review-coding
|
|
145
|
+
output_contracts:
|
|
146
|
+
report:
|
|
147
|
+
- name: coding-review.md
|
|
148
|
+
format: coding-review
|
|
149
|
+
rules:
|
|
150
|
+
- condition: approved
|
|
151
|
+
- condition: needs_fix
|
|
152
|
+
|
|
131
153
|
rules:
|
|
132
154
|
- condition: all("approved")
|
|
133
155
|
next: supervise
|