B-029 spawner重複起動バグ修正

Summary

spawnerに重複起動バグがあります。1つのメモに対して複数のエージェントが起動されてしまう問題です。

原因

watcher.ts の fs.watch が同一ファイルに対して複数イベント（IN_CREATE + IN_CLOSE_WRITE 等）を発火し、それらが DEBOUNCE_MS（200ms）以上の間隔で発生した場合、debounce が効かず handleWatcherEvent が複数回呼ばれます。さらに process-manager.ts の spawnAgent() には同一メモファイルの重複チェックがありません。

修正内容

1. process-manager.ts に processedFiles セットを追加

spawnAgent() で非PMロールの場合、memoFile が既に処理済みかチェックする:

const processedFiles = new Set<string>();

spawnAgent(role, memoFile) {
  // PM dedup (既存)
  if (role === 'project-manager' && isPmRunning()) return false;

  // 非PM: 同一メモファイルの重複チェック（新規追加）
  if (memoFile && processedFiles.has(memoFile)) {
    logger.agent(role, `skip duplicate: ${path.basename(memoFile)}`);
    return false;
  }
  if (memoFile) processedFiles.add(memoFile);

  // ... 既存ロジック
}

2. watcher.ts の DEBOUNCE_MS を 200 → 500 に変更（保険的措置）

types.ts:

export const DEBOUNCE_MS = 500;

3. index.ts の初期スキャンでも processedFiles に登録

start() 内で、初期スキャンで見つけたファイルも processedFiles に追加する（watcher イベントとの重複防止）。processedFiles を process-manager 内に持つため、spawnAgent が false を返す形で自然に重複排除される。

テスト追加

process-manager.test.ts に以下を追加:

• 同一 memoFile で spawnAgent を2回呼んだ場合、2回目は false が返る
• 異なる memoFile で呼んだ場合は両方 true が返る

Acceptance Criteria

• 同一メモファイルに対して spawnAgent() が2回以上呼ばれても、エージェントは1回しか起動されない
• ログに skip duplicate が記録される
• 既存テストが全てパス
• npm run typecheck, lint, format:check, test, build がすべてパス

コミット粒度

1コミットで十分です。

Summary

B-029 spawner重複起動バグ修正を実装しました。

Changes

1. process-manager.ts: processedFiles セットによる重複チェック

• createProcessManager 内に processedFiles = new Set<string>() を追加
• spawnAgent() で非PMロールかつ memoFile がある場合、processedFiles に既に存在すればスキップ
• ログに skip duplicate: <filename> を記録

2. types.ts: DEBOUNCE_MS を 200 → 500 に変更

• 保険的措置として fs.watch のデバウンス間隔を拡大

3. テスト追加・修正

• process-manager.test.ts: 同一 memoFile で2回 spawnAgent を呼ぶと2回目は false が返るテスト追加
• process-manager.test.ts: 異なる memoFile では両方 true が返るテスト追加
• types.test.ts: DEBOUNCE_MS の期待値を 500 に更新
• watcher.test.ts: デバウンス増加に合わせてテストの待機時間を 500ms → 1000ms に調整

Changed Files

• scripts/spawner/process-manager.ts
• scripts/spawner/types.ts
• scripts/spawner/__tests__/process-manager.test.ts
• scripts/spawner/__tests__/types.test.ts
• scripts/spawner/__tests__/watcher.test.ts

Pre-Completion Check Results

• ✅ npm run typecheck — pass
• ✅ npm run lint — pass
• ✅ npm run format:check — pass
• ✅ npm test — 1027/1027 tests passed
• ✅ npm run build — success

Validation

• 同一 memoFile に対して spawnAgent() を2回呼んでも1回しか起動されない（テストで確認）
• ログに skip duplicate が記録される
• 全既存テストがパス

Summary

B-029 実装完了しました。reviewer にレビュー依頼済みです。

Results

• processedFiles セットによる重複チェックを spawnAgent() に追加
• DEBOUNCE_MS を 200 → 500 に変更
• テスト2件追加、既存テスト調整
• 全チェック（typecheck, lint, format:check, test, build）パス

Next actions

• reviewer によるレビュー待ち

B-029 Review Result: Approved

The processedFiles dedup logic in process-manager.ts is correct and well-tested. All 48 spawner tests pass.

One optional suggestion: consider adding a test that verifies dedup fires before queueing when the concurrency limit is reached. This would cover the scenario where spawnAgent is called with a duplicate memoFile while running at max concurrency. The current code handles it correctly (dedup check is before the queue check), but an explicit test would document this behavior.

No blocking issues found. Good to merge.