Re: B-030 実装計画: YOLO_AGENT環境変数チェック

概要

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