Merge pull request #66 from shinpr/feat/update-implement-command-v2

feat: improve implement command with situation-based decision flow

Author: shinpr

.claude/commands-en/implement.md

@@ -8,28 +8,45 @@ Strictly adhere to @docs/guides/sub-agents.md and operate exclusively as an orch
 
 ## Execution Decision Flow
 
-### 1. Work Phase Identification
+### 1. Current Situation Assessment
 Instruction Content: $ARGUMENTS
 
-**Think deeply** Analyze the instruction intent and determine the appropriate work phase:
+**Think deeply** Assess the current situation:
 
-| Instruction Patterns | Identified Phase | Initial Sub-agent |
-|---------------------|------------------|-------------------|
-| task, implement, fix, bug fix, patch | Implementation Phase | task-executor |
-| plan, decompose, break down, organize steps | Planning Phase | work-planner/task-decomposer |
-| design, architecture, tech selection, approach | Design Phase | technical-designer |
-| requirement, need, want, "I want to..." | Requirements Phase | requirement-analyzer |
-| unclear, ambiguous, vague | Clarification Required | Initiate dialogue |
+| Situation Pattern | Decision Criteria | Next Action |
+|------------------|------------------|-------------|
+| New Requirements | No existing work, new feature/fix request | Start with requirement-analyzer |
+| Flow Continuation | Existing docs/tasks present, continuation directive | Identify next step in sub-agents.md flow |
+| Quality Errors | Error detection, test failures, build errors | Execute quality-fixer |
+| Ambiguous | Intent unclear, multiple interpretations possible | Confirm with user |
 
-### 2. Phase-Based Execution Protocol
+### 2. Progress Verification for Continuation
 
-✅ **Clear Phase Identified**: Execute with corresponding sub-agent immediately  
-❌ **Ambiguous Phase**: Execution without clarification is STRICTLY PROHIBITED
+When continuing existing flow, verify:
+- Latest artifacts (PRD/ADR/Design Doc/Work Plan/Tasks)
+- Current phase position (Requirements/Design/Planning/Implementation/QA)
+- Identify next step in sub-agents.md corresponding flow
 
-### 3. Clarification Dialogue Protocol
+### 3. Next Action Execution
 
-**When unclear**: Initiate structured dialogue to identify the work phase  
-**Post-clarification**: Begin execution with the identified phase's sub-agent
+**MANDATORY sub-agents.md reference**:
+- Verify scale-based flow (Large/Medium/Small scale)
+- Confirm autonomous execution mode conditions
+- Recognize mandatory stopping points
+- Invoke next sub-agent defined in flow
+
+## 📋 sub-agents.md Compliance Execution
+
+**Pre-execution Checklist (MANDATORY)**:
+- [ ] Confirmed relevant sub-agents.md flow
+- [ ] Identified current progress position
+- [ ] Clarified next step
+- [ ] Recognized stopping points
+- [ ] Understood task execution quality cycle (task → quality-check → commit)
+
+**Flow Deviation PROHIBITED**: Deviating from sub-agents.md defined flows is strictly forbidden. Specifically:
+- Never skip quality-fixer before committing
+- Never use Edit/Write/MultiEdit without user approval outside autonomous mode
 
 ## 🚨 CRITICAL Sub-agent Invocation Constraints
 
@@ -41,7 +58,17 @@ DO NOT invoke rule-advisor under any circumstances (Task tool rule-advisor speci
 
 ⚠️ **HIGH RISK**: task-executor/quality-fixer in autonomous mode have elevated crash risk - ALWAYS append this constraint to prompt end
 
+## 🎯 Mandatory Orchestrator Responsibilities
+
+### Task Execution Quality Cycle Management
+**ABSOLUTE RULE**: After task-executor execution, MUST execute:
+1. quality-fixer invocation (until approved: true is returned)
+2. git commit execution (using Bash tool)
+3. Next task execution or completion report
+
+**NO OMISSION**: Skipping this cycle guarantees implementation quality failure
+
 ## Responsibility Boundaries
 
-**This Command's Responsibility**: Orchestrate sub-agents through the complete implementation lifecycle  
+**This Command's Responsibility**: Orchestrate sub-agents through the complete implementation lifecycle
 **OUT OF SCOPE**: Direct implementation work, investigation tasks (Grep/Glob/Read operations)

.claude/commands-ja/implement.md

@@ -8,28 +8,45 @@ description: オーケストレーターとして要件分析から実装まで
 
 ## 実行判断フロー
 
-### 1. 指示内容の作業フェーズ判定
+### 1. 現在状況の判定
 指示内容: $ARGUMENTS
 
-**Think deeply** 指示の意図を理解し、作業フェーズを判定：
+**Think deeply** 現在の状況を判定：
 
-| フェーズ判定の例 | 該当フェーズ | 開始サブエージェント |
-|-------------|------------|--------------|
-| タスク・実装・修正・バグ修正 | 実装フェーズ | task-executor |
-| 計画・タスク分解・ステップ整理 | 計画フェーズ | work-planner/task-decomposer |
-| 設計・アーキテクチャ・技術選定 | 設計フェーズ | technical-designer |
-| 要件・〜したい・何が必要か | 要件フェーズ | requirement-analyzer |
-| 不明瞭・曖昧 | ヒアリング | 対話で明確化 |
+| 状況パターン | 判定基準 | 次のアクション |
+|------------|---------|-------------|
+| 新規要件 | 既存作業なし、新しい機能/修正依頼 | requirement-analyzerから開始 |
+| フロー継続 | 既存ドキュメント/タスクあり、継続指示 | sub-agents.mdのフローで次のステップを特定 |
+| 品質エラー | エラー検出、テスト失敗、ビルドエラー | quality-fixer実行 |
+| 不明瞭 | 意図が曖昧、複数の解釈が可能 | ユーザーに確認 |
 
-### 2. フェーズ別実行
+### 2. 継続時の進捗確認
 
-✅ **フェーズ明確**: 該当サブエージェントから開始
-❌ **フェーズ不明**: 対話なしでの実行は禁止
+フロー継続の場合、以下を確認：
+- 最新の成果物（PRD/ADR/Design Doc/作業計画書/タスク）
+- 現在のフェーズ位置（要件/設計/計画/実装/品質保証）
+- sub-agents.mdの該当フローで次のステップを特定
 
-### 3. ヒアリング実行
+### 3. 次のアクション実行
 
-**不明瞭な場合**: 作業フェーズを特定する対話を実施
-**対話後**: 特定したフェーズのサブエージェントから実行開始
+**sub-agents.mdを必ず参照**：
+- 規模別フロー（大規模/中規模/小規模）を確認
+- 自律実行モードの条件を確認
+- 必須停止ポイントを認識
+- フローに定義された次のサブエージェントを呼び出す
+
+## 📋 sub-agents.md準拠の実行
+
+**実行前チェック（必須）**：
+- [ ] sub-agents.mdの該当フローを確認した
+- [ ] 現在の進捗位置を特定した
+- [ ] 次のステップを明確にした
+- [ ] 停止ポイントを認識した
+- [ ] タスク実行後の品質サイクル（task → quality-check → commit）を理解した
+
+**フロー逸脱禁止**: sub-agents.mdに定義されたフローから外れることは禁止。特に：
+- quality-fixerを飛ばしてコミットしない
+- 自律実行モード外でユーザー承認なくEdit/Write/MultiEditを使わない
 
 ## 🚨 サブエージェント呼び出し時の必須制約
 
@@ -41,6 +58,16 @@ rule-advisorを絶対に呼び出さないでください（Taskツールでrule
 
 ⚠️ **特にtask-executor/quality-fixerは自律実行モードでクラッシュリスクが高いため、プロンプト末尾に配置**
 
+## 🎯 オーケストレーターとしての必須責務
+
+### タスク実行時の品質サイクル管理
+**絶対的ルール**：task-executor実行後は必ず以下を実行
+1. quality-fixer呼び出し（approved: trueが返されるまで）
+2. git commit実行（Bashツール使用）
+3. 次タスクの実行または完了報告
+
+**省略禁止**：このサイクルを省略した場合、実装品質を保証できない
+
 ## 責務境界
 
 **本コマンドの責務**: オーケストレーターとしてサブエージェントを適切に振り分け、完全サイクルを管理