From e5f296a3e0b682d209e1a80839bf5157935a2b5a Mon Sep 17 00:00:00 2001 From: nrslib <38722970+nrslib@users.noreply.github.com> Date: Mon, 2 Mar 2026 22:22:24 +0900 Subject: [PATCH] =?UTF-8?q?feat:=20TAKT=E9=96=8B=E7=99=BA=E5=90=91?= =?UTF-8?q?=E3=81=91=E3=83=AC=E3=83=93=E3=83=A5=E3=83=BC=EF=BC=8B=E4=BF=AE?= =?UTF-8?q?=E6=AD=A3=E3=83=AB=E3=83=BC=E3=83=97=E3=83=94=E3=83=BC=E3=82=B9?= =?UTF-8?q?=EF=BC=88takt-default-review-fix=EF=BC=89=E3=82=92=E8=BF=BD?= =?UTF-8?q?=E5=8A=A0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- builtins/en/piece-categories.yaml | 1 + .../en/pieces/takt-default-review-fix.yaml | 245 ++++++++++++++++++ builtins/ja/piece-categories.yaml | 1 + .../ja/pieces/takt-default-review-fix.yaml | 245 ++++++++++++++++++ 4 files changed, 492 insertions(+) create mode 100644 builtins/en/pieces/takt-default-review-fix.yaml create mode 100644 builtins/ja/pieces/takt-default-review-fix.yaml 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