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
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
name: review-takt-default
|
|
2
|
-
description: "TAKT-focused multi-perspective review (
|
|
2
|
+
description: "TAKT-focused multi-perspective review (7 reviewers including AI antipattern and coding review)"
|
|
3
3
|
workflow_config:
|
|
4
4
|
provider_options:
|
|
5
5
|
codex:
|
|
@@ -184,6 +184,28 @@ steps:
|
|
|
184
184
|
- condition: approved
|
|
185
185
|
- condition: needs_fix
|
|
186
186
|
|
|
187
|
+
- name: coding-review
|
|
188
|
+
edit: false
|
|
189
|
+
persona: coding-reviewer
|
|
190
|
+
policy: review
|
|
191
|
+
provider_options:
|
|
192
|
+
claude:
|
|
193
|
+
allowed_tools:
|
|
194
|
+
- Read
|
|
195
|
+
- Glob
|
|
196
|
+
- Grep
|
|
197
|
+
- Bash
|
|
198
|
+
- WebSearch
|
|
199
|
+
- WebFetch
|
|
200
|
+
instruction: review-coding
|
|
201
|
+
output_contracts:
|
|
202
|
+
report:
|
|
203
|
+
- name: coding-review.md
|
|
204
|
+
format: coding-review
|
|
205
|
+
rules:
|
|
206
|
+
- condition: approved
|
|
207
|
+
- condition: needs_fix
|
|
208
|
+
|
|
187
209
|
rules:
|
|
188
210
|
- condition: all("approved")
|
|
189
211
|
next: supervise
|
|
@@ -222,7 +244,8 @@ steps:
|
|
|
222
244
|
- `04-testing-review.md` (Testing review)
|
|
223
245
|
- `05-ai-antipattern-review.md` (AI antipattern review)
|
|
224
246
|
- `06-requirements-review.md` (Requirements review)
|
|
225
|
-
|
|
247
|
+
- `07-coding-review.md` (Coding review)
|
|
248
|
+
2. Synthesize the 7 review results
|
|
226
249
|
3. Produce a consolidated review summary with overall verdict
|
|
227
250
|
|
|
228
251
|
**Review Summary output contract:**
|
|
@@ -243,6 +266,7 @@ steps:
|
|
|
243
266
|
| Testing | APPROVE/REJECT | {Brief finding} |
|
|
244
267
|
| AI Antipattern | APPROVE/REJECT | {Brief finding} |
|
|
245
268
|
| Requirements | APPROVE/REJECT | {Brief finding} |
|
|
269
|
+
| Coding | APPROVE/REJECT | {Brief finding} |
|
|
246
270
|
|
|
247
271
|
## Issues Requiring Attention
|
|
248
272
|
| # | Severity | Source | Location | Issue |
|
|
@@ -273,6 +297,7 @@ steps:
|
|
|
273
297
|
| Testing | APPROVE/REJECT | {Overview} |
|
|
274
298
|
| AI Antipattern | APPROVE/REJECT | {Overview} |
|
|
275
299
|
| Requirements | APPROVE/REJECT | {Overview} |
|
|
300
|
+
| Coding | APPROVE/REJECT | {Overview} |
|
|
276
301
|
|
|
277
302
|
## Current Iteration Findings (new)
|
|
278
303
|
| # | finding_id | Severity | Source | Location | Issue | Fix Suggestion |
|
|
@@ -119,10 +119,31 @@ steps:
|
|
|
119
119
|
report:
|
|
120
120
|
- name: ai-antipattern-review.md
|
|
121
121
|
format: ai-antipattern-review
|
|
122
|
+
- name: coding-review
|
|
123
|
+
edit: false
|
|
124
|
+
persona: coding-reviewer
|
|
125
|
+
policy: review
|
|
126
|
+
provider_options:
|
|
127
|
+
claude:
|
|
128
|
+
allowed_tools:
|
|
129
|
+
- Read
|
|
130
|
+
- Glob
|
|
131
|
+
- Grep
|
|
132
|
+
- Bash
|
|
133
|
+
- WebSearch
|
|
134
|
+
- WebFetch
|
|
135
|
+
instruction: review-coding
|
|
136
|
+
output_contracts:
|
|
137
|
+
report:
|
|
138
|
+
- name: coding-review.md
|
|
139
|
+
format: coding-review
|
|
140
|
+
rules:
|
|
141
|
+
- condition: approved
|
|
142
|
+
- condition: needs_fix
|
|
122
143
|
rules:
|
|
123
|
-
- condition: all("Complies with Terraform conventions", "No AI-specific issues")
|
|
144
|
+
- condition: all("Complies with Terraform conventions", "No AI-specific issues", "approved")
|
|
124
145
|
next: supervise
|
|
125
|
-
- condition: any("Convention violations found", "AI-specific issues found")
|
|
146
|
+
- condition: any("Convention violations found", "AI-specific issues found", "needs_fix")
|
|
126
147
|
next: supervise
|
|
127
148
|
- name: supervise
|
|
128
149
|
edit: false
|
package/builtins/ja/config.yaml
CHANGED
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
language: ja # 表示言語: ja | en
|
|
8
8
|
|
|
9
9
|
# デフォルトプロバイダー・モデル
|
|
10
|
-
# provider: claude # デフォルトプロバイダー: claude | codex | opencode | cursor | copilot | mock
|
|
10
|
+
# provider: claude # デフォルトプロバイダー: claude | claude-sdk | claude-terminal | codex | opencode | cursor | copilot | kiro | mock
|
|
11
11
|
# model: sonnet # デフォルトモデル(プロバイダーにそのまま渡される)
|
|
12
12
|
|
|
13
13
|
# 実行制御
|
|
@@ -122,12 +122,14 @@ language: ja # 表示言語: ja | en
|
|
|
122
122
|
# openrouter_api_key: "..." # OpenRouter APIキー
|
|
123
123
|
# opencode_api_key: "..." # OpenCode APIキー
|
|
124
124
|
# cursor_api_key: "..." # Cursor APIキー
|
|
125
|
+
# kiro_api_key: "..." # Kiro APIキー
|
|
125
126
|
|
|
126
127
|
# CLIパス
|
|
127
128
|
# codex_cli_path: "/absolute/path/to/codex" # Codex CLI絶対パス
|
|
128
129
|
# claude_cli_path: "/absolute/path/to/claude" # Claude Code CLI絶対パス
|
|
129
130
|
# cursor_cli_path: "/absolute/path/to/cursor" # cursor-agent CLI絶対パス
|
|
130
131
|
# copilot_cli_path: "/absolute/path/to/copilot" # Copilot CLI絶対パス
|
|
132
|
+
# kiro_cli_path: "/absolute/path/to/kiro-cli" # Kiro CLI絶対パス
|
|
131
133
|
# copilot_github_token: "ghp_..." # Copilot GitHubトークン
|
|
132
134
|
|
|
133
135
|
# その他
|
|
@@ -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: 多角レビュー -
|
|
2
|
+
description: 多角レビュー - 6つの観点(コーディングレビュー含む)から並列にコード変更をレビューし、統合結果を出力する
|
|
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レビュー)
|
|
196
218
|
- `04-testing-review.md`(テストレビュー)
|
|
197
219
|
- `05-requirements-review.md`(要件充足レビュー)
|
|
198
|
-
|
|
220
|
+
- `06-coding-review.md`(コーディングレビュー)
|
|
221
|
+
2. 6つのレビュー結果を統合
|
|
199
222
|
3. 統合レビューサマリーと総合判定を作成
|
|
200
223
|
output_contracts:
|
|
201
224
|
report:
|
|
@@ -217,6 +240,7 @@ steps:
|
|
|
217
240
|
| QA | APPROVE/REJECT | {概要} |
|
|
218
241
|
| テスト | APPROVE/REJECT | {概要} |
|
|
219
242
|
| 要件充足 | APPROVE/REJECT | {概要} |
|
|
243
|
+
| コーディング | APPROVE/REJECT | {概要} |
|
|
220
244
|
|
|
221
245
|
## 今回の指摘(new)
|
|
222
246
|
| # | finding_id | 重大度 | ソース | 場所 | 問題 | 修正案 |
|
|
@@ -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: 多角レビュー+修正ループ(アーキテクチャ・セキュリティ・QA
|
|
2
|
+
description: 多角レビュー+修正ループ(アーキテクチャ・セキュリティ・QA・テスト・要件・コーディングの6観点から並列レビューし、指摘を修正する)
|
|
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開発向け多角レビュー+修正ループ(初回
|
|
2
|
+
description: TAKT開発向け多角レビュー+修正ループ(初回7並列レビュー(コーディングレビュー含む)、修正後はAIアンチパターン事前レビュー+5並列レビュー、監督者指摘の再修正)
|
|
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
|