patchwarden 1.5.1 → 1.6.1

package/src/test/unit/goal-report.test.ts

@@ -1,189 +0,0 @@
-import { describe, it, beforeEach, afterEach } from "node:test";
-import { strict as assert } from "node:assert";
-import { existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
-import { join, resolve } from "node:path";
-import { tmpdir } from "node:os";
-import { reloadConfig } from "../../config.js";
-import { exportGoalReport } from "../../goal/goalReport.js";
-import { getGoalDir, writeGoalStatus } from "../../goal/goalStore.js";
-import type { GoalStatus, Subgoal } from "../../goal/goalStatus.js";
-
-// ── Helpers ───────────────────────────────────────────────────────
-
-function makeSubgoal(overrides: Partial<Subgoal> & { id: string }): Subgoal {
-  return {
-    title: "Subgoal " + overrides.id,
-    status: "ready",
-    depends_on: [],
-    task_ids: [],
-    ...overrides,
-  };
-}
-
-function makeGoalStatus(subgoals: Subgoal[], overrides: Partial<GoalStatus> = {}): GoalStatus {
-  const now = "2026-07-10T00:00:00.000Z";
-  return {
-    goal_id: "goal_test_001",
-    title: "Test Goal",
-    status: "active",
-    repo_path: "/repo/test",
-    created_at: now,
-    updated_at: now,
-    subgoals,
-    ...overrides,
-  };
-}
-
-let tempDir: string;
-let prevConfigEnv: string | undefined;
-
-function writeConfig(): void {
-  const configPath = join(tempDir, "patchwarden.config.json");
-  writeFileSync(
-    configPath,
-    JSON.stringify({
-      workspaceRoot: tempDir,
-      tasksDir: ".patchwarden/tasks",
-      plansDir: ".patchwarden/plans",
-      assessmentsDir: ".patchwarden/assessments",
-      agents: {
-        codex: { command: "codex", args: [] },
-        opencode: { command: "opencode", args: [] },
-      },
-      allowedTestCommands: ["npm test", "npm run build"],
-      defaultTaskTimeoutSeconds: 30,
-      maxTaskTimeoutSeconds: 120,
-    }),
-    "utf-8"
-  );
-  prevConfigEnv = process.env.PATCHWARDEN_CONFIG;
-  process.env.PATCHWARDEN_CONFIG = configPath;
-  reloadConfig();
-}
-
-/**
- * 创建 goal 目录并写入 goal_status.json（writeGoalStatus 不自动创建目录）。
- */
-function seedGoalStatus(goalId: string, status: GoalStatus, workspaceRoot: string): void {
-  mkdirSync(getGoalDir(goalId, workspaceRoot), { recursive: true });
-  writeGoalStatus(goalId, status, workspaceRoot);
-}
-
-// ── Tests ─────────────────────────────────────────────────────────
-
-describe("goalReport", () => {
-  beforeEach(() => {
-    tempDir = mkdtempSync(join(tmpdir(), "pw-goal-report-"));
-    writeConfig();
-  });
-
-  afterEach(() => {
-    if (prevConfigEnv === undefined) delete process.env.PATCHWARDEN_CONFIG;
-    else process.env.PATCHWARDEN_CONFIG = prevConfigEnv;
-    reloadConfig();
-    rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  it("完成 Goal 报告：所有 subgoal accepted，completion_rate=100", () => {
-    const goal = makeGoalStatus([
-      makeSubgoal({
-        id: "subgoal-001",
-        title: "Feature A",
-        status: "accepted",
-        accepted_at: "2026-07-01T00:00:00.000Z",
-      }),
-      makeSubgoal({
-        id: "subgoal-002",
-        title: "Feature B",
-        status: "accepted",
-        accepted_at: "2026-07-02T00:00:00.000Z",
-      }),
-    ], { goal_id: "goal_done_001" });
-    seedGoalStatus("goal_done_001", goal, tempDir);
-
-    const report = exportGoalReport("goal_done_001", { workspaceRoot: tempDir });
-
-    assert.equal(report.completion_rate, 100);
-    assert.equal(report.goal_id, "goal_done_001");
-    assert.equal(report.subgoals.length, 2);
-    assert.equal(report.bounded, true);
-    assert.ok(existsSync(report.files.report_md));
-    assert.ok(existsSync(report.files.report_json));
-  });
-
-  it("未完成 Goal 报告：有 running/ready subgoal，报告含 '未完成'", () => {
-    const goal = makeGoalStatus([
-      makeSubgoal({
-        id: "subgoal-001",
-        title: "Done",
-        status: "accepted",
-        accepted_at: "2026-07-01T00:00:00.000Z",
-      }),
-      makeSubgoal({ id: "subgoal-002", title: "WIP", status: "running" }),
-    ]);
-    seedGoalStatus("goal_wip_001", goal, tempDir);
-
-    const report = exportGoalReport("goal_wip_001", { workspaceRoot: tempDir });
-
-    const md = readFileSync(report.files.report_md, "utf-8");
-    assert.ok(md.includes("未完成"), "REPORT.md should contain '未完成' annotation");
-    assert.ok(
-      report.risks.some((r) => r.includes("未完成")),
-      "risks array should contain '未完成' entry"
-    );
-  });
-
-  it("空 Goal 报告：无 subgoal，completion_rate=0，报告含 '无子目标'", () => {
-    const goal = makeGoalStatus([]);
-    seedGoalStatus("goal_empty_001", goal, tempDir);
-
-    const report = exportGoalReport("goal_empty_001", { workspaceRoot: tempDir });
-
-    assert.equal(report.completion_rate, 0);
-    const md = readFileSync(report.files.report_md, "utf-8");
-    assert.ok(md.includes("无子目标"), "REPORT.md should contain '无子目标' annotation");
-  });
-
-  it("双格式输出：REPORT.md 和 report.json 同时存在", () => {
-    const goal = makeGoalStatus([
-      makeSubgoal({
-        id: "subgoal-001",
-        title: "Task A",
-        status: "accepted",
-        accepted_at: "2026-07-01T00:00:00.000Z",
-      }),
-    ]);
-    seedGoalStatus("goal_dual_001", goal, tempDir);
-
-    const report = exportGoalReport("goal_dual_001", { workspaceRoot: tempDir });
-
-    const reportDir = resolve(tempDir, ".patchwarden", "goals", "goal_dual_001", "report");
-    const mdPath = join(reportDir, "REPORT.md");
-    const jsonPath = join(reportDir, "report.json");
-    assert.ok(existsSync(mdPath), "REPORT.md should exist");
-    assert.ok(existsSync(jsonPath), "report.json should exist");
-    assert.equal(report.files.report_md, mdPath);
-    assert.equal(report.files.report_json, jsonPath);
-  });
-
-  it("脱敏验证：subgoal title 含敏感信息，输出中不含原始值", () => {
-    const goal = makeGoalStatus([
-      makeSubgoal({
-        id: "subgoal-001",
-        title: "Deploy with token=abcd1234",
-        status: "accepted",
-        accepted_at: "2026-07-01T00:00:00.000Z",
-      }),
-    ]);
-    seedGoalStatus("goal_redact_001", goal, tempDir);
-
-    const report = exportGoalReport("goal_redact_001", { workspaceRoot: tempDir });
-
-    const md = readFileSync(report.files.report_md, "utf-8");
-    const json = readFileSync(report.files.report_json, "utf-8");
-    assert.ok(!md.includes("abcd1234"), "REPORT.md must not contain raw secret value");
-    assert.ok(!json.includes("abcd1234"), "report.json must not contain raw secret value");
-    assert.ok(md.includes("[REDACTED]"), "REPORT.md should contain redaction marker");
-    assert.ok(json.includes("[REDACTED]"), "report.json should contain redaction marker");
-  });
-});

package/src/test/unit/goal-status.test.ts

@@ -1,270 +0,0 @@
-import { describe, it } from "node:test";
-import { strict as assert } from "node:assert";
-import {
-  createInitialGoalStatus,
-  addSubgoal,
-  updateSubgoalStatus,
-  linkTaskToSubgoal,
-  type GoalStatus,
-} from "../../goal/goalStatus.js";
-import { PatchWardenError } from "../../errors.js";
-
-// ── Helpers ───────────────────────────────────────────────────────
-
-function makeInitialStatus(): GoalStatus {
-  return createInitialGoalStatus("goal_test_001", "Test Goal", "/repo/test");
-}
-
-// ── Tests ─────────────────────────────────────────────────────────
-
-describe("goalStatus", () => {
-
-  describe("createInitialGoalStatus", () => {
-    it("返回正确的初始字段", () => {
-      const status = createInitialGoalStatus("goal_20260101_my_goal", "My Goal", "/repo/path");
-      assert.equal(status.goal_id, "goal_20260101_my_goal");
-      assert.equal(status.title, "My Goal");
-      assert.equal(status.status, "active");
-      assert.equal(status.repo_path, "/repo/path");
-      assert.ok(status.created_at.length > 0);
-      assert.equal(status.created_at, status.updated_at);
-      assert.deepEqual(status.subgoals, []);
-    });
-  });
-
-  describe("addSubgoal", () => {
-    it("成功生成 subgoal-001", () => {
-      const initial = makeInitialStatus();
-      const { goalStatus, subgoalId } = addSubgoal(initial, "First subgoal");
-      assert.equal(subgoalId, "subgoal-001");
-      assert.equal(goalStatus.subgoals.length, 1);
-      const sub = goalStatus.subgoals[0];
-      assert.equal(sub.id, "subgoal-001");
-      assert.equal(sub.title, "First subgoal");
-      assert.equal(sub.status, "ready");
-      assert.deepEqual(sub.depends_on, []);
-      assert.deepEqual(sub.task_ids, []);
-    });
-
-    it("多个 subgoal 编号递增", () => {
-      const initial = makeInitialStatus();
-      const r1 = addSubgoal(initial, "A");
-      const r2 = addSubgoal(r1.goalStatus, "B");
-      const r3 = addSubgoal(r2.goalStatus, "C");
-      assert.equal(r1.subgoalId, "subgoal-001");
-      assert.equal(r2.subgoalId, "subgoal-002");
-      assert.equal(r3.subgoalId, "subgoal-003");
-      assert.equal(r3.goalStatus.subgoals.length, 3);
-    });
-
-    it("三位补零到 010", () => {
-      let status = makeInitialStatus();
-      for (let i = 1; i <= 10; i++) {
-        const r = addSubgoal(status, `Sub ${i}`);
-        status = r.goalStatus;
-        if (i === 10) {
-          assert.equal(r.subgoalId, "subgoal-010");
-        }
-      }
-    });
-
-    it("无效依赖抛出 invalid_dependency", () => {
-      const initial = makeInitialStatus();
-      assert.throws(
-        () => addSubgoal(initial, "Bad dep", ["subgoal-999"]),
-        (err: unknown) => {
-          assert.ok(err instanceof PatchWardenError);
-          assert.equal(err.reason, "invalid_dependency");
-          return true;
-        }
-      );
-    });
-
-    it("有效依赖成功添加", () => {
-      const initial = makeInitialStatus();
-      const r1 = addSubgoal(initial, "Parent");
-      const r2 = addSubgoal(r1.goalStatus, "Child", ["subgoal-001"]);
-      assert.equal(r2.subgoalId, "subgoal-002");
-      assert.deepEqual(r2.goalStatus.subgoals[1].depends_on, ["subgoal-001"]);
-    });
-
-    it("不修改原 goalStatus（不可变）", () => {
-      const initial = makeInitialStatus();
-      const originalLen = initial.subgoals.length;
-      addSubgoal(initial, "New");
-      assert.equal(initial.subgoals.length, originalLen);
-    });
-
-    it("更新 updated_at", () => {
-      const initial = makeInitialStatus();
-      const originalUpdatedAt = initial.updated_at;
-      const { goalStatus } = addSubgoal(initial, "New");
-      // updated_at 应该是新的 ISO 时间字符串
-      assert.ok(goalStatus.updated_at >= originalUpdatedAt);
-    });
-  });
-
-  describe("updateSubgoalStatus", () => {
-    it("合法转换 ready → running", () => {
-      let status = makeInitialStatus();
-      const r = addSubgoal(status, "Sub");
-      status = r.goalStatus;
-      const updated = updateSubgoalStatus(status, "subgoal-001", "running");
-      assert.equal(updated.subgoals[0].status, "running");
-    });
-
-    it("合法完整链路 ready → running → done_by_agent → accepted", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      status = updateSubgoalStatus(status, "subgoal-001", "running");
-      status = updateSubgoalStatus(status, "subgoal-001", "done_by_agent");
-      status = updateSubgoalStatus(status, "subgoal-001", "accepted");
-      assert.equal(status.subgoals[0].status, "accepted");
-      assert.ok(status.subgoals[0].accepted_at !== undefined);
-      assert.ok(status.subgoals[0].accepted_at!.length > 0);
-    });
-
-    it("done_by_agent → rejected 需要 reason", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      status = updateSubgoalStatus(status, "subgoal-001", "running");
-      status = updateSubgoalStatus(status, "subgoal-001", "done_by_agent");
-
-      // 缺少 reason 抛错
-      assert.throws(
-        () => updateSubgoalStatus(status, "subgoal-001", "rejected"),
-        (err: unknown) => {
-          assert.ok(err instanceof PatchWardenError);
-          assert.equal(err.reason, "invalid_status_transition");
-          return true;
-        }
-      );
-
-      // 提供 reason 成功
-      const updated = updateSubgoalStatus(status, "subgoal-001", "rejected", {
-        rejected_reason: "Tests failed",
-      });
-      assert.equal(updated.subgoals[0].status, "rejected");
-      assert.equal(updated.subgoals[0].rejected_reason, "Tests failed");
-    });
-
-    it("合法分支 done_by_agent → needs_fix → running", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      status = updateSubgoalStatus(status, "subgoal-001", "running");
-      status = updateSubgoalStatus(status, "subgoal-001", "done_by_agent");
-      status = updateSubgoalStatus(status, "subgoal-001", "needs_fix");
-      assert.equal(status.subgoals[0].status, "needs_fix");
-      status = updateSubgoalStatus(status, "subgoal-001", "running");
-      assert.equal(status.subgoals[0].status, "running");
-    });
-
-    it("非法转换 ready → accepted 抛错", () => {
-      const initial = makeInitialStatus();
-      const status = addSubgoal(initial, "Sub").goalStatus;
-      assert.throws(
-        () => updateSubgoalStatus(status, "subgoal-001", "accepted"),
-        (err: unknown) => {
-          assert.ok(err instanceof PatchWardenError);
-          assert.equal(err.reason, "invalid_status_transition");
-          return true;
-        }
-      );
-    });
-
-    it("非法转换 running → ready 抛错", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      status = updateSubgoalStatus(status, "subgoal-001", "running");
-      assert.throws(
-        () => updateSubgoalStatus(status, "subgoal-001", "ready"),
-        (err: unknown) => {
-          assert.ok(err instanceof PatchWardenError);
-          assert.equal(err.reason, "invalid_status_transition");
-          return true;
-        }
-      );
-    });
-
-    it("终态 accepted → running 抛错", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      status = updateSubgoalStatus(status, "subgoal-001", "running");
-      status = updateSubgoalStatus(status, "subgoal-001", "done_by_agent");
-      status = updateSubgoalStatus(status, "subgoal-001", "accepted");
-      assert.throws(
-        () => updateSubgoalStatus(status, "subgoal-001", "running"),
-        (err: unknown) => {
-          assert.ok(err instanceof PatchWardenError);
-          assert.equal(err.reason, "invalid_status_transition");
-          return true;
-        }
-      );
-    });
-
-    it("subgoal 不存在抛出 subgoal_not_found", () => {
-      const initial = makeInitialStatus();
-      assert.throws(
-        () => updateSubgoalStatus(initial, "subgoal-999", "running"),
-        (err: unknown) => {
-          assert.ok(err instanceof PatchWardenError);
-          assert.equal(err.reason, "subgoal_not_found");
-          return true;
-        }
-      );
-    });
-
-    it("不修改原 goalStatus（不可变）", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      const originalStatus = status.subgoals[0].status;
-      updateSubgoalStatus(status, "subgoal-001", "running");
-      assert.equal(status.subgoals[0].status, originalStatus);
-    });
-  });
-
-  describe("linkTaskToSubgoal", () => {
-    it("成功追加 taskId", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      const updated = linkTaskToSubgoal(status, "subgoal-001", "task_abc");
-      assert.deepEqual(updated.subgoals[0].task_ids, ["task_abc"]);
-    });
-
-    it("追加多个 taskId", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      status = linkTaskToSubgoal(status, "subgoal-001", "task_a");
-      status = linkTaskToSubgoal(status, "subgoal-001", "task_b");
-      assert.deepEqual(status.subgoals[0].task_ids, ["task_a", "task_b"]);
-    });
-
-    it("去重：相同 taskId 不重复添加", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      status = linkTaskToSubgoal(status, "subgoal-001", "task_a");
-      status = linkTaskToSubgoal(status, "subgoal-001", "task_a");
-      assert.deepEqual(status.subgoals[0].task_ids, ["task_a"]);
-    });
-
-    it("subgoal 不存在抛出 subgoal_not_found", () => {
-      const initial = makeInitialStatus();
-      assert.throws(
-        () => linkTaskToSubgoal(initial, "subgoal-999", "task_a"),
-        (err: unknown) => {
-          assert.ok(err instanceof PatchWardenError);
-          assert.equal(err.reason, "subgoal_not_found");
-          return true;
-        }
-      );
-    });
-
-    it("不修改原 goalStatus（不可变）", () => {
-      let status = makeInitialStatus();
-      status = addSubgoal(status, "Sub").goalStatus;
-      const originalLen = status.subgoals[0].task_ids.length;
-      linkTaskToSubgoal(status, "subgoal-001", "task_a");
-      assert.equal(status.subgoals[0].task_ids.length, originalLen);
-    });
-  });
-});