B-030 計画依頼: YOLO_AGENT環境変数によるmemo操作権限チェック

背景

spawnerで起動されたエージェントが、他のロール宛のメモを誤ってアーカイブする問題が発生しました（commit a6f6f2f）。ownerの指示により、YOLO_AGENT環境変数を使った権限チェックを実装します。

要件（owner指示、変更不可）

1. spawnerがエージェント起動時に YOLO_AGENT=<role> 環境変数をセットする
2. npm run memo -- mark 実行時、YOLO_AGENT が設定されていれば、操作対象メモの宛先（to）と YOLO_AGENT の値が一致するか検証する
3. 不一致の場合はエラーを出して操作を中止する
4. YOLO_AGENT が未設定のときは従来通りの動作（チェックなし）

技術的な情報

Spawner側（env var設定箇所）

• ファイル: scripts/spawner/process-manager.ts
• 関数: doSpawn() (line 94-217)
• spawn呼び出し: line 125-129。現在 env プロパティ未設定
• env: { ...process.env, YOLO_AGENT: role } を追加する

Memo CLI側（チェック箇所）

• ファイル: scripts/memo/commands/mark.ts
• 関数: markMemo() (line 11-45)
• メモ発見後（line 19以降）、ファイル移動前にYOLO_AGENTチェックを追加

計画に含めてほしい内容

1. 実装ステップの詳細（変更箇所、コード例）
2. テスト計画（spawner側、memo CLI側それぞれ）
3. エッジケースの考慮（YOLO_AGENT未設定、メモ宛先がYOLO_AGENTと一致、不一致の各パターン）

B-029完了後すぐに着手したいので、計画をお願いします。

概要

spawnerがエージェント起動時に YOLO_AGENT=<role> 環境変数を設定し、memo mark コマンド実行時にその値とメモの宛先(to)を照合して権限チェックを行う。

---

実装ステップ

ステップ1: spawner側 — YOLO_AGENT 環境変数の設定

ファイル: scripts/spawner/process-manager.ts 変更箇所: doSpawn() 関数内、spawn() 呼び出し (line 125-129)

変更内容: spawn の options に env プロパティを追加する。

// 変更前 (line 125-129)
childProcess = spawn(command, args, {
  shell: false,
  stdio: ["ignore", logFd, logFd],
  cwd: process.cwd(),
});

// 変更後
childProcess = spawn(command, args, {
  shell: false,
  stdio: ["ignore", logFd, logFd],
  cwd: process.cwd(),
  env: { ...process.env, YOLO_AGENT: role },
});

注意点:

• ...process.env で既存の環境変数をすべて継承する（省略するとPATH等が失われる）
• role は MonitoredRole 型で、"project-manager" | "builder" | "reviewer" | "researcher" | "planner" | "process-engineer" のいずれか。memo の to フィールドと同じスラッグ形式

---

ステップ2: memo CLI側 — YOLO_AGENT 権限チェック

ファイル: scripts/memo/commands/mark.ts 変更箇所: markMemo() 関数内、メモ発見後・ファイル移動前 (line 19以降)

変更内容: メモの to フィールドと YOLO_AGENT 環境変数を比較するチェックを追加する。

import fs from "node:fs";
import path from "node:path";
import { scanAllMemos, type MemoState } from "../core/scanner.js";

const VALID_STATES: MemoState[] = ["inbox", "active", "archive"];

/**
 * Change a memo state by moving it to the corresponding directory.
 * Outputs: "<id>: <old_state> -> <new_state>"
 */
export function markMemo(id: string, newState: MemoState): void {
  if (!VALID_STATES.includes(newState)) {
    throw new Error(
      `Invalid state: "${newState}". Valid states: ${VALID_STATES.join(", ")}`,
    );
  }

  const memos = scanAllMemos();
  const memo = memos.find((m) => m.frontmatter.id === id);

  if (!memo) {
    throw new Error(`No memo found with ID: ${id}`);
  }

  // YOLO_AGENT permission check: if set, only allow marking memos addressed to this agent
  const yoloAgent = process.env.YOLO_AGENT;
  if (yoloAgent && memo.frontmatter.to !== yoloAgent) {
    throw new Error(
      `Permission denied: agent "${yoloAgent}" cannot mark memo addressed to "${memo.frontmatter.to}"`,
    );
  }

  const oldState = memo.state;

  if (oldState === newState) {
    console.log(`${id}: ${oldState} -> ${newState}`);
    return;
  }

  // Compute new file path by replacing the state directory
  const oldDir = path.dirname(memo.filePath);
  const fileName = path.basename(memo.filePath);
  const roleDir = path.dirname(oldDir);
  const newDir = path.join(roleDir, newState);

  // Ensure destination directory exists
  fs.mkdirSync(newDir, { recursive: true });

  const newFilePath = path.join(newDir, fileName);
  fs.renameSync(memo.filePath, newFilePath);

  console.log(`${id}: ${oldState} -> ${newState}`);
}

追加コードの要点 (line 25-29 に相当):

const yoloAgent = process.env.YOLO_AGENT;
if (yoloAgent && memo.frontmatter.to !== yoloAgent) {
  throw new Error(
    `Permission denied: agent "${yoloAgent}" cannot mark memo addressed to "${memo.frontmatter.to}"`,
  );
}

設計判断:

• process.env.YOLO_AGENT が未設定(undefined)または空文字列の場合はチェックをスキップ → 従来通りの動作
• チェックは throw new Error() でエラーにする（既存のバリデーションパターンに合わせる）
• エラーメッセージに agent名 と memo宛先名 の両方を含め、デバッグしやすくする

---

テスト計画

テスト A: spawner側 (scripts/spawner/__tests__/process-manager.test.ts)

テストケース A-1: spawn時にYOLO_AGENT環境変数が設定される

it("passes YOLO_AGENT env var matching the role", async () => {
  const logger = createMockLogger();
  // Use "env" command to print environment variables; check log output for YOLO_AGENT
  const pm = createProcessManager({
    logger,
    logsDir,
    spawnCmd: "env",
  });

  pm.spawnAgent("builder", "memo/builder/inbox/test.md");

  await new Promise((resolve) => setTimeout(resolve, 500));

  // Read the log file to verify YOLO_AGENT was set
  const logFiles = fs.readdirSync(logsDir).filter((f) => f.includes("builder"));
  expect(logFiles.length).toBeGreaterThan(0);
  const logContent = fs.readFileSync(path.join(logsDir, logFiles[0]), "utf-8");
  expect(logContent).toContain("YOLO_AGENT=builder");
});

テストケース A-2: project-managerでもYOLO_AGENTが設定される

it("passes YOLO_AGENT=project-manager for PM role", async () => {
  const logger = createMockLogger();
  const pm = createProcessManager({
    logger,
    logsDir,
    spawnCmd: "env",
  });

  pm.spawnAgent("project-manager", null);

  await new Promise((resolve) => setTimeout(resolve, 500));

  const logFiles = fs.readdirSync(logsDir).filter((f) => f.includes("project-manager"));
  expect(logFiles.length).toBeGreaterThan(0);
  const logContent = fs.readFileSync(path.join(logsDir, logFiles[0]), "utf-8");
  expect(logContent).toContain("YOLO_AGENT=project-manager");
});

テスト方法の補足: spawnCmd: "env" で起動すると、環境変数一覧がログファイルに出力される。そのログファイルを読んで YOLO_AGENT=<role> が含まれることを検証する。

---

テスト B: memo CLI側 (scripts/memo/__tests__/mark.test.ts)

既存の createMemoFile ヘルパーをそのまま活用する。process.env.YOLO_AGENT を beforeEach/afterEach でセット・クリアする。

テストケース B-1: YOLO_AGENTが一致する場合は正常に動作

test("allows mark when YOLO_AGENT matches memo to field", () => {
  process.env.YOLO_AGENT = "builder";
  createMemoFile("builder", "inbox", "perm1");

  markMemo("perm1", "active");

  const newPath = path.join(tmpDir, "builder", "active", "perm1-test-memo.md");
  expect(fs.existsSync(newPath)).toBe(true);

  delete process.env.YOLO_AGENT;
});

テストケース B-2: YOLO_AGENTが不一致の場合はエラー

test("throws error when YOLO_AGENT does not match memo to field", () => {
  process.env.YOLO_AGENT = "reviewer";
  createMemoFile("builder", "inbox", "perm2");

  expect(() => markMemo("perm2", "archive")).toThrow(
    Permission

Summary

B-030（YOLO_AGENT環境変数によるmemo操作権限チェック）の計画依頼を確認しましたが、本件は既に実装・レビュー完了済みです。

確認結果

以下の実装が既にコードベースに反映されています：

1. Spawner側 (scripts/spawner/process-manager.ts L129): env: { ...process.env, YOLO_AGENT: role } が設定済み
2. Memo CLI側 (scripts/memo/commands/mark.ts L25-29): YOLO_AGENT チェックが実装済み
3. テスト: scripts/spawner/__tests__/process-manager.test.ts および scripts/memo/__tests__/mark.test.ts にテストが存在
4. レビュー: reviewer により APPROVED（memo 19c6c22d67f）、全132テストパス

Next actions

本依頼は対応不要です。計画依頼メモをアーカイブします。