diff --git a/builtins/en/piece-categories.yaml b/builtins/en/piece-categories.yaml index aba1efb..1c4848b 100644 --- a/builtins/en/piece-categories.yaml +++ b/builtins/en/piece-categories.yaml @@ -53,6 +53,7 @@ piece_categories: pieces: - takt-default - takt-default-team-leader + - takt-default-review-fix Others: pieces: - research diff --git a/builtins/en/pieces/takt-default-review-fix.yaml b/builtins/en/pieces/takt-default-review-fix.yaml new file mode 100644 index 0000000..8337447 --- /dev/null +++ b/builtins/en/pieces/takt-default-review-fix.yaml @@ -0,0 +1,245 @@ +name: takt-default-review-fix +description: "TAKT development code review + fix loop (5 parallel reviewers: architecture, security, QA, testing, requirements — with iterative fixes)" +piece_config: + provider_options: + codex: + network_access: true + opencode: + network_access: true +max_movements: 30 +initial_movement: gather +loop_monitors: + - cycle: + - reviewers + - fix + threshold: 3 + judge: + persona: supervisor + instruction_template: loop-monitor-ai-fix + rules: + - condition: Healthy (progress being made) + next: reviewers + - condition: Unproductive (repeated findings, fixes not applied) + next: supervise + +movements: + - name: gather + edit: false + persona: planner + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: gather-review + output_contracts: + report: + - name: review-target.md + format: review-gather + rules: + - condition: Review target information gathered + next: reviewers + - condition: Cannot identify review target, insufficient info + next: ABORT + appendix: | + Clarifications needed: + - {Question 1} + - {Question 2} + + - name: reviewers + parallel: + - name: arch-review + edit: false + persona: architecture-reviewer + policy: review + knowledge: + - architecture + - takt + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-arch + output_contracts: + report: + - name: architecture-review.md + format: architecture-review + rules: + - condition: approved + - condition: needs_fix + + - name: security-review + edit: false + persona: security-reviewer + policy: review + knowledge: security + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-security + output_contracts: + report: + - name: security-review.md + format: security-review + rules: + - condition: approved + - condition: needs_fix + + - name: qa-review + edit: false + persona: qa-reviewer + policy: + - review + - qa + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-qa + output_contracts: + report: + - name: qa-review.md + format: qa-review + rules: + - condition: approved + - condition: needs_fix + + - name: testing-review + edit: false + persona: testing-reviewer + policy: + - review + - testing + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-test + output_contracts: + report: + - name: testing-review.md + format: testing-review + rules: + - condition: approved + - condition: needs_fix + + - name: requirements-review + edit: false + persona: requirements-reviewer + policy: review + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-requirements + output_contracts: + report: + - name: requirements-review.md + format: requirements-review + rules: + - condition: approved + - condition: needs_fix + + rules: + - condition: all("approved") + next: supervise + - condition: any("needs_fix") + next: fix + + - name: fix + edit: true + persona: coder + policy: + - coding + - testing + knowledge: + - takt + - architecture + - security + allowed_tools: + - Read + - Glob + - Grep + - Edit + - Write + - Bash + - WebSearch + - WebFetch + required_permission_mode: edit + pass_previous_response: false + instruction: fix + rules: + - condition: Fixes complete + next: reviewers + - condition: Unable to proceed with fixes + next: supervise + + - name: supervise + edit: false + persona: supervisor + policy: review + allowed_tools: + - Read + - Glob + - Grep + - WebSearch + - WebFetch + instruction: supervise + pass_previous_response: false + rules: + - condition: All validations complete, ready to merge + next: COMPLETE + - condition: Issues detected + next: fix_supervisor + output_contracts: + report: + - name: supervisor-validation.md + format: supervisor-validation + - name: summary.md + format: summary + use_judge: false + + - name: fix_supervisor + edit: true + persona: coder + policy: + - coding + - testing + knowledge: + - takt + - architecture + - security + allowed_tools: + - Read + - Glob + - Grep + - Edit + - Write + - Bash + - WebSearch + - WebFetch + instruction: fix-supervisor + pass_previous_response: false + rules: + - condition: Fixes for supervisor findings complete + next: supervise + - condition: Unable to proceed with fixes + next: supervise diff --git a/builtins/ja/piece-categories.yaml b/builtins/ja/piece-categories.yaml index 4262168..791aa5d 100644 --- a/builtins/ja/piece-categories.yaml +++ b/builtins/ja/piece-categories.yaml @@ -53,6 +53,7 @@ piece_categories: pieces: - takt-default - takt-default-team-leader + - takt-default-review-fix その他: pieces: - research diff --git a/builtins/ja/pieces/takt-default-review-fix.yaml b/builtins/ja/pieces/takt-default-review-fix.yaml new file mode 100644 index 0000000..ef47af4 --- /dev/null +++ b/builtins/ja/pieces/takt-default-review-fix.yaml @@ -0,0 +1,245 @@ +name: takt-default-review-fix +description: TAKT開発コードレビュー＋修正ループ（アーキテクチャ・セキュリティ・QA・テスト・要件の5観点から並列レビューし、指摘を修正する） +piece_config: + provider_options: + codex: + network_access: true + opencode: + network_access: true +max_movements: 30 +initial_movement: gather +loop_monitors: + - cycle: + - reviewers + - fix + threshold: 3 + judge: + persona: supervisor + instruction_template: loop-monitor-ai-fix + rules: + - condition: 健全（進捗あり） + next: reviewers + - condition: 非生産的（同じ指摘の反復・修正未反映） + next: supervise + +movements: + - name: gather + edit: false + persona: planner + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: gather-review + output_contracts: + report: + - name: review-target.md + format: review-gather + rules: + - condition: レビュー対象の情報収集完了 + next: reviewers + - condition: レビュー対象を特定できない、情報不足 + next: ABORT + appendix: | + 確認事項: + - {質問1} + - {質問2} + + - name: reviewers + parallel: + - name: arch-review + edit: false + persona: architecture-reviewer + policy: review + knowledge: + - architecture + - takt + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-arch + output_contracts: + report: + - name: architecture-review.md + format: architecture-review + rules: + - condition: approved + - condition: needs_fix + + - name: security-review + edit: false + persona: security-reviewer + policy: review + knowledge: security + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-security + output_contracts: + report: + - name: security-review.md + format: security-review + rules: + - condition: approved + - condition: needs_fix + + - name: qa-review + edit: false + persona: qa-reviewer + policy: + - review + - qa + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-qa + output_contracts: + report: + - name: qa-review.md + format: qa-review + rules: + - condition: approved + - condition: needs_fix + + - name: testing-review + edit: false + persona: testing-reviewer + policy: + - review + - testing + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-test + output_contracts: + report: + - name: testing-review.md + format: testing-review + rules: + - condition: approved + - condition: needs_fix + + - name: requirements-review + edit: false + persona: requirements-reviewer + policy: review + allowed_tools: + - Read + - Glob + - Grep + - Bash + - WebSearch + - WebFetch + instruction: review-requirements + output_contracts: + report: + - name: requirements-review.md + format: requirements-review + rules: + - condition: approved + - condition: needs_fix + + rules: + - condition: all("approved") + next: supervise + - condition: any("needs_fix") + next: fix + + - name: fix + edit: true + persona: coder + policy: + - coding + - testing + knowledge: + - takt + - architecture + - security + allowed_tools: + - Read + - Glob + - Grep + - Edit + - Write + - Bash + - WebSearch + - WebFetch + required_permission_mode: edit + pass_previous_response: false + instruction: fix + rules: + - condition: 修正が完了した + next: reviewers + - condition: 修正を進行できない + next: supervise + + - name: supervise + edit: false + persona: supervisor + policy: review + allowed_tools: + - Read + - Glob + - Grep + - WebSearch + - WebFetch + instruction: supervise + pass_previous_response: false + rules: + - condition: すべての検証が完了し、マージ可能な状態である + next: COMPLETE + - condition: 問題が検出された + next: fix_supervisor + output_contracts: + report: + - name: supervisor-validation.md + format: supervisor-validation + - name: summary.md + format: summary + use_judge: false + + - name: fix_supervisor + edit: true + persona: coder + policy: + - coding + - testing + knowledge: + - takt + - architecture + - security + allowed_tools: + - Read + - Glob + - Grep + - Edit + - Write + - Bash + - WebSearch + - WebFetch + instruction: fix-supervisor + pass_previous_response: false + rules: + - condition: 監督者の指摘に対する修正が完了した + next: supervise + - condition: 修正を進行できない + next: supervise