@ronkovic/aad 0.3.0

package/src/modules/task-execution/phases/implementer-green.ts

@@ -0,0 +1,100 @@
+import type { Task, WorkspaceInfo, EffortLevel } from "@aad/shared/types";
+import type { ClaudeProvider } from "@aad/claude-provider";
+
+export interface PhaseResult {
+  success: boolean;
+  output: string;
+  duration?: number;
+}
+
+export interface ImplementerGreenOptions {
+  effortLevel?: EffortLevel;
+  model?: string;
+  timeout?: number;
+}
+
+/**
+ * Build TDD Green phase prompt for implementer agent
+ */
+export function buildGreenPhasePrompt(task: Task, workspace: WorkspaceInfo): string {
+  const codingConventions = getCodingConventions(workspace.language);
+
+  return `implementerエージェントとして、TDD Green フェーズを実行してください。
+
+Task ID: ${task.taskId as string}
+Task Title: ${task.title}
+Task Description: ${task.description}
+
+プロジェクト情報:
+- Workspace: ${workspace.path}
+- Language: ${workspace.language}
+- Test Framework: ${workspace.testFramework}
+- Package Manager: ${workspace.packageManager}
+- Framework: ${workspace.framework}
+
+実行内容:
+1. 作成されたテストを確認する
+2. テストをパスするための最小限の実装を書く（${codingConventions}）
+3. テストを実行してパスすることを確認する
+
+注意: 過度な最適化やリファクタリングは行わず、テストをパスするための最小限のコードを書いてください。`;
+}
+
+/**
+ * Get language-specific coding conventions
+ */
+function getCodingConventions(language: string): string {
+  switch (language.toLowerCase()) {
+    case "go":
+    case "golang":
+      return "effective Go, gofmt規約に従う";
+
+    case "python":
+      return "PEP 8, Black/Ruff規約に従う";
+
+    case "rust":
+      return "Rust API guidelines, rustfmt規約に従う";
+
+    case "typescript":
+    case "javascript":
+      return "ESLint/Prettier規約に従う";
+
+    case "java":
+      return "Google Java Style Guideに従う";
+
+    case "c#":
+    case "csharp":
+      return "C# Coding Conventionsに従う";
+
+    default:
+      return "言語別のコーディング規約に従う";
+  }
+}
+
+/**
+ * Run TDD Green phase: implement minimal code to pass tests
+ */
+export async function runImplementerGreen(
+  task: Task,
+  workspace: WorkspaceInfo,
+  provider: ClaudeProvider,
+  options: ImplementerGreenOptions = {}
+): Promise<PhaseResult> {
+  const prompt = buildGreenPhasePrompt(task, workspace);
+
+  const response = await provider.call({
+    prompt,
+    cwd: workspace.path,
+    allowedTools: ["Read", "Write", "Edit", "Glob", "Grep", "Bash"],
+    permissionMode: "bypassPermissions",
+    effortLevel: options.effortLevel ?? "medium",
+    model: options.model,
+    timeout: options.timeout,
+  });
+
+  return {
+    success: response.exitCode === 0,
+    output: response.result,
+    duration: response.duration,
+  };
+}

package/src/modules/task-execution/phases/merge.ts

@@ -0,0 +1,122 @@
+import type { Task, WorkspaceInfo } from "@aad/shared/types";
+import type { ClaudeProvider } from "@aad/claude-provider";
+import type { MergeService } from "@aad/git-workspace";
+
+export interface MergePhaseResult {
+  success: boolean;
+  output: string;
+  hadConflict: boolean;
+  duration?: number;
+}
+
+export interface MergePhaseOptions {
+  mergeResolverModel?: string;
+  timeout?: number;
+}
+
+/**
+ * Build conflict resolution prompt for merge-resolver agent
+ */
+export function buildConflictResolutionPrompt(
+  task: Task,
+  conflicts: string[]
+): string {
+  const conflictList = conflicts.map((f) => `- ${f}`).join("\n");
+
+  return `merge-resolverエージェントとして、マージ競合を解決してください。
+
+Task ID: ${task.taskId as string}
+Task Title: ${task.title}
+
+競合ファイル:
+${conflictList}
+
+実行内容:
+1. 競合マーカー (<<<<<<<, =======, >>>>>>>) を確認
+2. 両方の変更を理解し、適切にマージする
+3. 競合マーカーを削除する
+4. 解決したファイルを git add する
+
+注意:
+- 両方の変更の意図を尊重し、機能を壊さないように解決してください
+- 解決後は必ず git add <ファイル> を実行してください`;
+}
+
+/**
+ * Run merge phase: merge task branch to parent branch
+ * If conflicts occur, use Claude to resolve them
+ */
+export async function runMergePhase(
+  task: Task,
+  _workspace: WorkspaceInfo,
+  taskBranch: string,
+  parentBranch: string,
+  parentWorktree: string,
+  mergeService: MergeService,
+  provider: ClaudeProvider,
+  options: MergePhaseOptions = {}
+): Promise<MergePhaseResult> {
+  const startTime = Date.now();
+
+  // Try to merge
+  const mergeResult = await mergeService.mergeToParent(
+    task.taskId,
+    taskBranch,
+    parentBranch,
+    parentWorktree
+  );
+
+  if (mergeResult.success) {
+    // Merge succeeded
+    return {
+      success: true,
+      output: mergeResult.message ?? "Merge completed successfully",
+      hadConflict: false,
+      duration: Date.now() - startTime,
+    };
+  }
+
+  // Merge failed with conflicts
+  const conflicts = mergeResult.conflicts ?? [];
+
+  if (conflicts.length === 0) {
+    // Merge failed but no conflicts detected (unexpected error)
+    return {
+      success: false,
+      output: mergeResult.message ?? "Merge failed (unknown reason)",
+      hadConflict: false,
+      duration: Date.now() - startTime,
+    };
+  }
+
+  // Resolve conflicts with Claude
+  const prompt = buildConflictResolutionPrompt(task, conflicts);
+
+  const response = await provider.call({
+    prompt,
+    cwd: parentWorktree,
+    allowedTools: ["Read", "Write", "Edit", "Glob", "Grep", "Bash"],
+    permissionMode: "bypassPermissions",
+    effortLevel: "low",
+    model: options.mergeResolverModel ?? "claude-haiku-4-5",
+    timeout: options.timeout,
+  });
+
+  const duration = Date.now() - startTime;
+
+  if (response.exitCode === 0) {
+    return {
+      success: true,
+      output: response.result,
+      hadConflict: true,
+      duration,
+    };
+  }
+
+  return {
+    success: false,
+    output: response.result,
+    hadConflict: true,
+    duration,
+  };
+}

package/src/modules/task-execution/phases/reviewer.ts

@@ -0,0 +1,160 @@
+import type { Task, WorkspaceInfo, RunId } from "@aad/shared/types";
+import type { ClaudeProvider, SubagentConfig } from "@aad/claude-provider";
+
+export interface PhaseResult {
+  success: boolean;
+  output: string;
+  duration?: number;
+}
+
+export interface ReviewerOptions {
+  model?: string;
+  timeout?: number;
+}
+
+export interface ReviewConfig {
+  teams: {
+    reviewer: boolean;
+  };
+}
+
+/**
+ * Build solo review prompt
+ */
+export function buildReviewPrompt(task: Task): string {
+  return `reviewerエージェントとして、コードレビューを実行してください。
+
+Task ID: ${task.taskId as string}
+Task Title: ${task.title}
+
+実行内容:
+1. 作成されたコードとテストを確認する
+2. 品質・セキュリティ・パフォーマンスの観点からレビューする
+3. 必要に応じて軽微な修正を行う
+4. 重大な問題があれば報告する
+
+注意: 軽微なスタイル問題は修正しても良いですが、機能を壊さないように注意してください。`;
+}
+
+/**
+ * Build Agent Teams parallel review prompt
+ */
+export function buildTeamsReviewPrompt(task: Task, runId: RunId): string {
+  return `reviewerチームリーダーとして、Agent Teamsを使った並列コードレビューを実行してください。
+
+Task ID: ${task.taskId as string}
+Task Title: ${task.title}
+
+実行内容:
+1. TeamCreateツールを使用してレビューチームを作成
+   チーム名: review-${task.taskId as string}
+
+2. 以下の3人のチームメイトをTaskツールで作成（すべてsubagent_type: general-purposeを使用）:
+   a) 品質レビュアー (名前: quality-reviewer)
+      - コードの可読性、保守性、DRY原則の遵守
+      - テストカバレッジと品質
+      - ドキュメントの充実度
+
+   b) セキュリティレビュアー (名前: security-reviewer)
+      - SQLインジェクション、XSS、CSRF対策
+      - 認証・認可の適切な実装
+      - 機密情報の扱い
+
+   c) パフォーマンスレビュアー (名前: performance-reviewer)
+      - N+1クエリ問題
+      - 不要なメモリアロケーション
+      - 適切なアルゴリズム選択
+
+3. 各チームメイトにタスクを割り当て（TaskCreateとTaskUpdate使用）
+
+4. 各チームメイトからのレビュー結果を収集
+
+5. 結果を統合してcode_review.mdを生成:
+   .aad/docs/${runId as string}/${task.taskId as string}/code_review.md
+
+6. TeamDeleteでチームをクリーンアップ
+
+重要な制約:
+- teammateMode: "in-process" を使用してtmux干渉を避ける
+- 各レビュアーの結果を必ず統合してcode_review.mdに保存
+- 重大な問題が見つかった場合は明確にマーク
+
+注意: 軽微なスタイル問題は修正しても良いですが、機能を壊さないように注意してください。`;
+}
+
+/**
+ * Build subagent configs for reviewer teams mode
+ */
+export function buildReviewerSubagents(): SubagentConfig[] {
+  return [
+    {
+      name: "quality-reviewer",
+      prompt: `品質レビューの専門家として、コードレビューを実施してください。
+以下の観点で評価してください:
+- コードの可読性と保守性
+- DRY原則の遵守
+- テストカバレッジと品質
+- ドキュメントの充実度
+- 命名規約の一貫性`,
+    },
+    {
+      name: "security-reviewer",
+      prompt: `セキュリティレビューの専門家として、コードレビューを実施してください。
+以下の観点で評価してください:
+- SQLインジェクション、XSS、CSRF対策
+- 認証・認可の適切な実装
+- 機密情報の扱い（ハードコードされた秘密鍵等）
+- 入力バリデーション
+- 依存パッケージの脆弱性`,
+    },
+    {
+      name: "performance-reviewer",
+      prompt: `パフォーマンスレビューの専門家として、コードレビューを実施してください。
+以下の観点で評価してください:
+- N+1クエリ問題
+- 不要なメモリアロケーション
+- 適切なアルゴリズム選択
+- キャッシュの活用
+- 非同期処理の適切な使用`,
+    },
+  ];
+}
+
+/**
+ * Run code review phase (solo or teams mode)
+ * Note: Review failures are non-fatal; execution continues even on error
+ */
+export async function runReviewer(
+  task: Task,
+  workspace: WorkspaceInfo,
+  runId: RunId,
+  config: ReviewConfig,
+  provider: ClaudeProvider,
+  options: ReviewerOptions = {}
+): Promise<PhaseResult> {
+  const useTeams = config.teams.reviewer;
+
+  const prompt = useTeams
+    ? buildTeamsReviewPrompt(task, runId)
+    : buildReviewPrompt(task);
+
+  const effortLevel = useTeams ? "high" : "medium";
+  const subagents = useTeams ? buildReviewerSubagents() : undefined;
+
+  const response = await provider.call({
+    prompt,
+    cwd: workspace.path,
+    allowedTools: ["Read", "Write", "Edit", "Glob", "Grep", "Bash"],
+    permissionMode: "bypassPermissions",
+    effortLevel,
+    model: options.model,
+    timeout: options.timeout,
+    ...(subagents ? { subagents } : {}),
+  });
+
+  return {
+    success: response.exitCode === 0,
+    output: response.result,
+    duration: response.duration,
+  };
+}

package/src/modules/task-execution/phases/tester-red.ts

@@ -0,0 +1,100 @@
+import type { Task, WorkspaceInfo, EffortLevel } from "@aad/shared/types";
+import type { ClaudeProvider } from "@aad/claude-provider";
+
+export interface PhaseResult {
+  success: boolean;
+  output: string;
+  duration?: number;
+}
+
+export interface TesterRedOptions {
+  effortLevel?: EffortLevel;
+  model?: string;
+  timeout?: number;
+}
+
+/**
+ * Build TDD Red phase prompt for tester agent
+ */
+export function buildRedPhasePrompt(task: Task, workspace: WorkspaceInfo): string {
+  const languagePatterns = getLanguageTestPatterns(workspace.language);
+
+  return `testerエージェントとして、TDD Red フェーズを実行してください。
+
+Task ID: ${task.taskId as string}
+Task Title: ${task.title}
+Task Description: ${task.description}
+
+プロジェクト情報:
+- Workspace: ${workspace.path}
+- Language: ${workspace.language}
+- Test Framework: ${workspace.testFramework}
+- Package Manager: ${workspace.packageManager}
+- Framework: ${workspace.framework}
+
+実行内容:
+1. タスクの要件を理解する
+2. 失敗するテストを作成する（言語に応じた適切なパターンを使用）
+   ${languagePatterns}
+3. テストを実行して失敗することを確認する
+
+注意: このフェーズでは実装コードは書かないでください。テストのみ作成してください。`;
+}
+
+/**
+ * Get language-specific test patterns
+ */
+function getLanguageTestPatterns(language: string): string {
+  switch (language.toLowerCase()) {
+    case "go":
+    case "golang":
+      return "- Go: テーブル駆動テスト";
+
+    case "python":
+      return "- Python: pytest parametrize";
+
+    case "rust":
+      return "- Rust: テストケースベクター";
+
+    case "typescript":
+    case "javascript":
+      return "- TypeScript/JavaScript: it.each (Vitest/Jest/Bun)";
+
+    case "terraform":
+      return "- Terraform: terraform validate";
+
+    case "java":
+      return "- Java: JUnit parametrized tests";
+
+    default:
+      return "- 言語に応じた適切なテストパターンを使用";
+  }
+}
+
+/**
+ * Run TDD Red phase: create failing tests
+ */
+export async function runTesterRed(
+  task: Task,
+  workspace: WorkspaceInfo,
+  provider: ClaudeProvider,
+  options: TesterRedOptions = {}
+): Promise<PhaseResult> {
+  const prompt = buildRedPhasePrompt(task, workspace);
+
+  const response = await provider.call({
+    prompt,
+    cwd: workspace.path,
+    allowedTools: ["Read", "Write", "Edit", "Glob", "Grep", "Bash"],
+    permissionMode: "bypassPermissions",
+    effortLevel: options.effortLevel ?? "medium",
+    model: options.model,
+    timeout: options.timeout,
+  });
+
+  return {
+    success: response.exitCode === 0,
+    output: response.result,
+    duration: response.duration,
+  };
+}

package/src/modules/task-execution/phases/tester-verify.ts

@@ -0,0 +1,120 @@
+import type { WorkspaceInfo } from "@aad/shared/types";
+import { TestRunnerError } from "@aad/shared/errors";
+
+export interface ProcessResult {
+  exitCode: number;
+  stdout: string;
+  stderr: string;
+}
+
+export interface ProcessSpawner {
+  spawn(
+    cmd: string,
+    args: string[],
+    opts: { cwd: string; timeout?: number }
+  ): Promise<ProcessResult>;
+}
+
+export interface TestResult {
+  success: boolean;
+  output: string;
+  error?: string;
+}
+
+/**
+ * Build test command based on detected test framework
+ */
+export function buildTestCommand(workspace: WorkspaceInfo): string[] {
+  const { testFramework, packageManager } = workspace;
+
+  switch (testFramework) {
+    case "bun-test":
+      return ["bun", "test"];
+
+    case "vitest":
+      if (packageManager === "npm") return ["npm", "run", "test"];
+      if (packageManager === "yarn") return ["yarn", "test"];
+      if (packageManager === "pnpm") return ["pnpm", "test"];
+      return ["npx", "vitest", "run"];
+
+    case "jest":
+      if (packageManager === "npm") return ["npm", "test"];
+      if (packageManager === "yarn") return ["yarn", "test"];
+      if (packageManager === "pnpm") return ["pnpm", "test"];
+      return ["npx", "jest"];
+
+    case "mocha":
+      if (packageManager === "npm") return ["npm", "test"];
+      if (packageManager === "yarn") return ["yarn", "test"];
+      return ["npx", "mocha"];
+
+    case "pytest":
+      return ["pytest", "-v"];
+
+    case "go-test":
+      return ["go", "test", "./..."];
+
+    case "cargo":
+      return ["cargo", "test"];
+
+    case "maven":
+      return ["mvn", "test"];
+
+    case "gradle":
+      return ["./gradlew", "test"];
+
+    case "unknown":
+      throw new TestRunnerError(
+        "Unsupported test framework: unknown",
+        { testFramework }
+      );
+
+    default: {
+      const exhaustive: never = testFramework;
+      throw new TestRunnerError(
+        `Unsupported test framework: ${exhaustive}`,
+        { testFramework }
+      );
+    }
+  }
+}
+
+/**
+ * Run tests in workspace and return result
+ * @param workspace - Workspace information with test framework
+ * @param spawner - Optional custom process spawner (for testing)
+ * @param timeout - Test timeout in milliseconds (default: 300000 = 5min)
+ */
+export async function runTests(
+  workspace: WorkspaceInfo,
+  spawner?: ProcessSpawner,
+  timeout = 300000
+): Promise<TestResult> {
+  const command = buildTestCommand(workspace);
+  const [cmd, ...args] = command;
+
+  if (!cmd) {
+    throw new TestRunnerError("Invalid test command: empty", { workspace });
+  }
+
+  const actualSpawner =
+    spawner ?? (await import("./default-spawner")).createDefaultSpawner();
+
+  const result = await actualSpawner.spawn(cmd, args, {
+    cwd: workspace.path,
+    timeout,
+  });
+
+  if (result.exitCode === 0) {
+    return {
+      success: true,
+      output: result.stdout,
+    };
+  }
+
+  return {
+    success: false,
+    output: result.stdout,
+    error: result.stderr,
+  };
+}