goalbuddy 0.3.7 → 0.3.8

package/internal/cli/goal-maker.mjs

@@ -49,6 +49,7 @@ const optionsWithValues = new Set([
   "--task",
   "--board",
 ]);
+const pathOptions = new Set(["--board", "--goal"]);
 
 const args = process.argv.slice(2);
 const command = args[0] === "--help" || args[0] === "-h"
@@ -111,6 +112,17 @@ async function main() {
         doctorClaude();
       }
       break;
+    case "reset":
+      if (wantsHelp()) {
+        usage();
+        break;
+      }
+      if (targetMode() !== "codex") {
+        console.error("Reset currently supports --target codex only.");
+        process.exit(2);
+      }
+      resetCodex();
+      break;
     case "check-update":
     case "update-check":
       checkUpdate();
@@ -193,6 +205,35 @@ function positionalArgs() {
   return values;
 }
 
+/**
+ * Resolve goal-related paths in raw args to absolute paths.
+ * Child processes spawned with cwd=packageRoot cannot resolve
+ * relative goal paths from the user's working directory.
+ */
+function resolveChildGoalArgs(rawArgs) {
+  const out = [];
+  for (let index = 0; index < rawArgs.length; index += 1) {
+    const arg = rawArgs[index];
+    const joinedMatch = [...pathOptions].find((opt) => arg.startsWith(opt + "="));
+    if (joinedMatch) {
+      const value = arg.slice(joinedMatch.length + 1);
+      out.push(`${joinedMatch}=${value ? resolve(value) : value}`);
+    } else if (pathOptions.has(arg)) {
+      out.push(arg);
+      const value = rawArgs[++index] || "";
+      out.push(value ? resolve(value) : value);
+    } else if (optionsWithValues.has(arg)) {
+      out.push(arg);
+      out.push(rawArgs[++index] || "");
+    } else if (!arg.startsWith("-")) {
+      out.push(resolve(arg));
+    } else {
+      out.push(arg);
+    }
+  }
+  return out;
+}
+
 function usage() {
   console.log(`${canonicalProductName} for Claude Code and Codex
 
@@ -203,6 +244,7 @@ Usage:
   ${canonicalCliName} update [--target claude|codex] [--claude-home <path>] [--codex-home <path>] [--json]
   ${canonicalCliName} agents [--target claude|codex] [--claude-home <path>] [--codex-home <path>] [--force]
   ${canonicalCliName} doctor [--target claude|codex] [--claude-home <path>] [--codex-home <path>] [--goal-ready]
+  ${canonicalCliName} reset --target codex [--codex-home <path>] [--json]
   ${canonicalCliName} check-update [--json]
   ${canonicalCliName} board <docs/goals/slug> [--host <host>] [--port <port>] [--once] [--json]
   ${canonicalCliName} prompt <docs/goals/slug> [--task T###] [--board <path/to/state.yaml>] [--json]
@@ -576,20 +618,36 @@ function doctor() {
   const agents = existsSync(agentsPath)
     ? readdirSync(agentsPath).filter((file) => file.startsWith("goal_") && file.endsWith(".toml"))
     : [];
-  const missingAgents = requiredAgentFiles.filter((file) => !agents.includes(file));
+  const installSurfacePresent = plugin.skill_installed || installed || legacyInstalled;
+  const residualAgents = installSurfacePresent ? [] : agents.filter((file) => requiredAgentFiles.includes(file));
+  const missingAgents = installSurfacePresent || residualAgents.length > 0
+    ? requiredAgentFiles.filter((file) => !agents.includes(file))
+    : [];
   const staleAgents = requiredAgentFiles.filter((file) => {
     const installedAgent = join(agentsPath, file);
     const bundledAgent = join(skillSource, "agents", file);
     if (!existsSync(installedAgent) || !existsSync(bundledAgent)) return false;
     return sha256(readFileSync(installedAgent)) !== sha256(readFileSync(bundledAgent));
   });
+  const runtimeState = codexInstallState({
+    plugin,
+    installed,
+    legacyInstalled,
+    residualAgents,
+    missingAgents,
+    staleAgents,
+  });
   const goalRuntime = codexGoalRuntimeStatus();
   const warnings = [];
   const errors = [];
   if (!goalRuntime.ready) {
     warnings.push("native Codex /goal runtime is not ready; run `codex login` and `codex features enable goals` before using /goal.");
   }
-  if (!plugin.skill_installed && !installed) {
+  if (runtimeState === "fully-removed") {
+    errors.push("Codex GoalBuddy is fully removed; run `npx goalbuddy --target codex` to install.");
+  } else if (runtimeState === "residual-agents-only") {
+    errors.push(`Residual GoalBuddy Codex agents remain without plugin cache/config: ${residualAgents.join(", ")}; run a GoalBuddy reset/cleanup before treating it as removed.`);
+  } else if (!plugin.skill_installed && !installed) {
     errors.push("Codex GoalBuddy plugin is not installed; run `npx goalbuddy --target codex`.");
   }
   if (plugin.skill_installed && !plugin.enabled) {
@@ -621,7 +679,9 @@ function doctor() {
     skill_path: skillPath,
     compatibility_skill_installed: legacyInstalled,
     compatibility_skill_path: legacySkillPath,
+    runtime_state: runtimeState,
     installed_agents: agents,
+    residual_agents: residualAgents,
     missing_agents: missingAgents,
     stale_agents: staleAgents,
     goal_runtime: goalRuntime,
@@ -636,6 +696,20 @@ function doctor() {
   process.exit(installOk && goalReadyOk && errors.length === 0 ? 0 : 1);
 }
 
+function codexInstallState({ plugin, installed, legacyInstalled, residualAgents, missingAgents, staleAgents }) {
+  if (residualAgents.length > 0 && !plugin.skill_installed && !installed && !legacyInstalled) {
+    return "residual-agents-only";
+  }
+  if (!plugin.skill_installed && !installed && !legacyInstalled) {
+    return "fully-removed";
+  }
+  if (staleAgents.length > 0) return "stale-agents";
+  if (missingAgents.length > 0) return "incomplete";
+  if (plugin.skill_installed && !plugin.enabled) return "disabled";
+  if ((plugin.skill_installed && plugin.enabled) || installed) return "installed";
+  return "incomplete";
+}
+
 function checkUpdate() {
   const report = updateReport();
 
@@ -782,8 +856,95 @@ function cleanupLegacyCodexSkills() {
   return removed;
 }
 
+function resetCodex() {
+  const configPath = join(codexHome(), "config.toml");
+  const removedConfigSections = [];
+  if (existsSync(configPath)) {
+    const existing = readFileSync(configPath, "utf8");
+    let updated = existing;
+    for (const header of [`[plugins."${pluginName}@${pluginName}"]`, `[marketplaces.${pluginName}]`]) {
+      const next = removeTomlTable(updated, header);
+      if (next !== updated) {
+        removedConfigSections.push(header);
+        updated = next;
+      }
+    }
+    if (updated !== existing) writeFileSync(configPath, updated);
+  }
+
+  const removedPluginCachePaths = [];
+  const cacheRoot = pluginCacheOwnerRoot();
+  if (existsSync(cacheRoot)) {
+    rmSync(cacheRoot, { recursive: true, force: true });
+    removedPluginCachePaths.push(cacheRoot);
+  }
+
+  const removedAgents = [];
+  const agentsRoot = join(codexHome(), "agents");
+  for (const file of requiredAgentFiles) {
+    const path = join(agentsRoot, file);
+    if (!existsSync(path)) continue;
+    rmSync(path, { recursive: true, force: true });
+    removedAgents.push(path);
+  }
+
+  const removedLegacySkillPaths = cleanupLegacyCodexSkills();
+  const report = {
+    reset: true,
+    target: "codex",
+    codex_home: codexHome(),
+    config_path: configPath,
+    removed_config_sections: removedConfigSections,
+    removed_plugin_cache_paths: removedPluginCachePaths,
+    removed_agents: removedAgents,
+    removed_legacy_skill_paths: removedLegacySkillPaths,
+  };
+
+  if (hasFlag("--json")) {
+    printJson(report);
+    return report;
+  }
+
+  console.log(`Reset ${canonicalProductName} Codex-owned runtime files`);
+  console.log(`Config sections: ${removedConfigSections.length ? removedConfigSections.join(", ") : "none"}`);
+  console.log(`Plugin cache: ${removedPluginCachePaths.length ? removedPluginCachePaths.join(", ") : "none"}`);
+  console.log(`Agents: ${removedAgents.length ? removedAgents.join(", ") : "none"}`);
+  console.log(`Legacy personal skills: ${removedLegacySkillPaths.length ? removedLegacySkillPaths.join(", ") : "none"}`);
+  return report;
+}
+
+function removeTomlTable(text, header) {
+  const normalized = text.endsWith("\n") || text.length === 0 ? text : `${text}\n`;
+  const lines = normalized.split("\n");
+  const output = [];
+  let skipping = false;
+  let removed = false;
+  const descendantPrefix = `${header.slice(0, -1)}.`;
+
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (trimmed === header || trimmed.startsWith(descendantPrefix)) {
+      skipping = true;
+      removed = true;
+      continue;
+    }
+    if (skipping && /^\s*\[/.test(line)) {
+      skipping = trimmed.startsWith(descendantPrefix);
+      if (skipping) continue;
+    }
+    if (!skipping) output.push(line);
+  }
+
+  if (!removed) return text;
+  return output.join("\n").replace(/\n{3,}/g, "\n\n").replace(/\n*$/, "\n");
+}
+
+function pluginCacheOwnerRoot() {
+  return join(codexHome(), "plugins", "cache", pluginName);
+}
+
 function pluginCacheRoot(version) {
-  return join(codexHome(), "plugins", "cache", pluginName, pluginName, version);
+  return join(pluginCacheOwnerRoot(), pluginName, version);
 }
 
 function enablePluginConfig() {
@@ -913,8 +1074,9 @@ async function board() {
     process.exit(2);
   }
 
+  const absoluteGoal = resolve(goal);
   const script = ensureLocalBoardSurface();
-  const scriptArgs = [script, "--goal", goal];
+  const scriptArgs = [script, "--goal", absoluteGoal];
   for (const option of ["--host", "--port"]) {
     const value = optionValue(option);
     if (value) scriptArgs.push(option, value);
@@ -945,7 +1107,7 @@ async function prompt() {
   }
 
   const script = join(skillSource, "scripts", "render-task-prompt.mjs");
-  const scriptArgs = [script, ...args.slice(1)];
+  const scriptArgs = [script, ...resolveChildGoalArgs(args.slice(1))];
   const result = spawnSync(process.execPath, scriptArgs, {
     cwd: packageRoot,
     encoding: "utf8",
@@ -959,7 +1121,7 @@ async function prompt() {
 
 async function parallelPlan() {
   const script = join(skillSource, "scripts", "parallel-plan.mjs");
-  const scriptArgs = [script, ...args.slice(1).filter((arg) => arg !== "--parallel-plan")];
+  const scriptArgs = [script, ...resolveChildGoalArgs(args.slice(1).filter((arg) => arg !== "--parallel-plan"))];
   const result = spawnSync(process.execPath, scriptArgs, {
     cwd: packageRoot,
     encoding: "utf8",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "goalbuddy",
-  "version": "0.3.7",
+  "version": "0.3.8",
   "description": "A /goal operating loop for Codex and Claude Code: goal oracles, local boards, receipts, and verification.",
   "type": "module",
   "bin": {
@@ -9,11 +9,10 @@
   },
   "files": [
     ".agents/plugins/marketplace.json",
+    "CHANGELOG.md",
     "README.md",
-    "RELEASE-0.3.5.md",
-    "RELEASE-0.3.7.md",
     "CONTRIBUTING.md",
-    "examples",
+    "docs/releases",
     "plugins/goalbuddy",
     "internal/assets",
     "internal/cli",
@@ -24,8 +23,8 @@
     "goalbuddy/templates"
   ],
   "scripts": {
-    "check": "node --check internal/cli/*.mjs goalbuddy/scripts/*.mjs && node --test internal/test/*.test.mjs",
-    "test": "node --test internal/test/*.test.mjs",
+    "check": "node --check internal/cli/*.mjs goalbuddy/scripts/*.mjs goalbuddy/surfaces/local-goal-board/scripts/*.mjs goalbuddy/surfaces/local-goal-board/scripts/lib/*.mjs && node --test internal/test/*.test.mjs goalbuddy/surfaces/local-goal-board/test/*.test.mjs",
+    "test": "node --test internal/test/*.test.mjs goalbuddy/surfaces/local-goal-board/test/*.test.mjs",
     "pack:dry-run": "npm pack --dry-run",
     "postinstall": "node internal/cli/postinstall.mjs",
     "publish:check": "node internal/cli/check-publish-version.mjs && npm run pack:dry-run",

package/plugins/goalbuddy/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "goalbuddy",
-  "version": "0.3.7",
+  "version": "0.3.8",
   "description": "Turn broad Codex and Claude Code work into pressured /goal runs with oracles, local boards, receipts, and verification.",
   "author": {
     "name": "tolibear",

package/plugins/goalbuddy/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "goalbuddy",
-  "version": "0.3.7",
+  "version": "0.3.8",
   "description": "Turn broad Codex and Claude Code work into pressured /goal runs with oracles, local boards, receipts, and verification.",
   "author": {
     "name": "tolibear",

package/plugins/goalbuddy/README.md

@@ -2,7 +2,7 @@
 
 GoalBuddy packages the canonical `goal-prep` skill as a plugin so teams can install the reusable workflow in **Codex** and **Claude Code**, while keeping the npm CLI for local setup, doctor checks, and the built-in local board surface.
 
-Version 0.3.7 is the Goalmaxxed release: goal oracles, largest safe useful slices, built-in local boards, receipt pressure, final-proof gates, and a smaller core without the extension catalog.
+Version 0.3.8 keeps the Goalmaxxed core and adds board-hub guardrails so agents reuse the shared local board hub instead of stopping it after an unregistered board URL returns 404.
 
 ## What It Contains
 

package/plugins/goalbuddy/skills/goalbuddy/SKILL.md

@@ -95,6 +95,8 @@ Recommended options:
 
 If the user chooses the local live board, create the goal directory, `notes/`, and an initial minimal `state.yaml` as soon as the slug is known, then run `npx goalbuddy board docs/goals/<slug>` and open the printed local URL in the AI coding agent's in-app browser (the Codex in-app Browser, the Claude Code preview, or the user's regular browser). The default local hub is `http://goalbuddy.localhost:41737/`, and board URLs normally look like `http://goalbuddy.localhost:41737/<slug>/`. In short: start the local board before filling the task list so the board pops up right away and cards populate live as `state.yaml` changes. Include the printed board URL in the final prep response as an actual clickable Markdown link, for example `[Open GoalBuddy board](http://goalbuddy.localhost:41737/<slug>/)`. Do not put the board URL only in a code block, quote, HTML comment, or prose that the UI cannot click.
 
+If `http://goalbuddy.localhost:41737/<slug>/` returns 404, do not assume the existing process is stale and do not stop it. First check `http://127.0.0.1:41737/api/boards`. If that endpoint returns board JSON, the port is the shared multi-board hub; rerun `npx goalbuddy board docs/goals/<slug>` with the absolute goal path if needed so the new goal registers on the same port. Only stop a specific process on 41737 when `/api/boards` is missing, returns 404, or otherwise proves the listener is not a current GoalBuddy multi-board hub.
+
 If the user wants an external board, GitHub sync, Slack digest, Linear handoff, or any other custom integration, do not install a GoalBuddy catalog item. Treat it as normal implementation work: create a concrete task that designs and verifies that integration inside the target repo or asks the operator for the required credentials and scope.
 
 Ask before board creation when the request is vague, strategic, improvement-oriented, or open-ended and the user has not explicitly said to use defaults. Ask one guided question at a time with 2-3 options and a recommended default, then wait. Continue the diagnostic intake until the user's answers are sufficient to choose the board shape. Do not create or repair `docs/goals/<slug>/` until the diagnostic intake is complete or the user explicitly accepts defaults.

package/plugins/goalbuddy/skills/goalbuddy/scripts/render-task-prompt.mjs

@@ -237,29 +237,43 @@ function receiptSchema(role) {
   if (role === "worker") {
     return {
       result: "done | blocked",
+      task_id: "<T###>",
+      board_path: "<path to state.yaml>",
       changed_files: [],
-      commands: [{ cmd: "<command>", status: "pass | fail | not_run" }],
-      summary: "<=120 words",
+      commands: [],
+      summary: "<=120 words>",
       remaining_blockers: [],
+      verification_attempts: 1,
+      stopped_because: null,
     };
   }
   if (role === "judge") {
     return {
       result: "done | blocked",
+      task_id: "<T###>",
+      board_path: "<path to state.yaml>",
       decision: "approved | rejected | approve_subgoal | reject_subgoal | not_complete | complete",
       full_outcome_complete: false,
+      rationale: "<=120 words>",
       evidence: [],
+      subgoal_contract: null,
+      parallel_safety: null,
       blocked_tasks: [],
+      missing_evidence: [],
       required_board_updates: [],
     };
   }
   return {
     result: "done | blocked",
-    summary: "<=120 words",
+    task_id: "<T###>",
+    board_path: "<path to state.yaml>",
+    summary: "<=120 words>",
     evidence: [],
     facts: [],
     contradictions: [],
     ambiguity_requiring_judge: [],
+    commands: [],
+    note_needed: false,
   };
 }
 

package/plugins/goalbuddy/skills/goalbuddy/surfaces/local-goal-board/scripts/local-goal-board.mjs

@@ -228,8 +228,7 @@ export async function startBoardServer(options = {}) {
 
       const route = routeBoardRequest(url.pathname, boards, initialBoard);
       if (!route.board) {
-        response.writeHead(404);
-        response.end("Not found");
+        sendUnregisteredBoardPath(response, url.pathname, boards, baseUrl);
         return;
       }
       if (route.pathname === "/api/board") {
@@ -400,6 +399,28 @@ function routeBoardRequest(pathname, boards, initialBoard) {
   return matches[0] || { board: null, pathname };
 }
 
+function sendUnregisteredBoardPath(response, pathname, boards, baseUrl) {
+  response.writeHead(404, {
+    "Content-Type": "text/plain; charset=utf-8",
+    "Cache-Control": "no-store",
+  });
+  const registeredBoards = [...boards.values()].map((board) => {
+    const summary = boardSummary(board, baseUrl);
+    return `- ${summary.title}: ${summary.url}`;
+  });
+  response.end([
+    `GoalBuddy board path is not registered in this local hub: ${pathname}`,
+    "",
+    "This server is the GoalBuddy multi-board hub. Do not stop it just because a /<slug>/ board URL returned 404.",
+    "Start or rerun `npx goalbuddy board <goal-dir>` to register that goal on this same port, then open the printed /<slug>/ URL.",
+    "",
+    "Registered boards:",
+    registeredBoards.length ? registeredBoards.join("\n") : "- none",
+    "",
+    `Hub API: ${baseUrl}/api/boards`,
+  ].join("\n"));
+}
+
 function stripBoardPathPrefix(pathname, boardPath) {
   const prefix = boardPath.endsWith("/") ? boardPath.slice(0, -1) : boardPath;
   if (pathname === prefix) return "/";

package/plugins/goalbuddy/skills/goalbuddy/surfaces/local-goal-board/test/local-goal-board.test.mjs

@@ -558,6 +558,33 @@ test("serves multiple local boards from one shared hub URL", async () => {
   }
 });
 
+test("unregistered board paths explain hub reuse instead of stale-port cleanup", async () => {
+  const root = mkdtempSync(join(tmpdir(), "goalbuddy-local-board-unregistered-"));
+  const goalDir = join(root, "first-goal");
+  try {
+    mkdirSync(join(goalDir, "notes"), { recursive: true });
+    writeFileSync(join(goalDir, "state.yaml"), stateYaml("active", { title: "First Goal", slug: "first-goal" }));
+
+    const server = await startBoardServer({ goalDir, host: "127.0.0.1", port: 0 });
+    try {
+      const baseUrl = new URL(server.url).origin;
+      const missingResponse = await fetch(`${baseUrl}/rinova-client-revision-redesign/`);
+      assert.equal(missingResponse.status, 404);
+      const message = await missingResponse.text();
+      assert.match(message, /board path is not registered/i);
+      assert.match(message, /multi-board hub/i);
+      assert.match(message, /Do not stop it just because a \/<slug>\/ board URL returned 404/);
+      assert.match(message, /npx goalbuddy board <goal-dir>/);
+      assert.match(message, /First Goal/);
+      assert.match(message, /\/api\/boards/);
+    } finally {
+      await server.close();
+    }
+  } finally {
+    rmSync(root, { recursive: true, force: true });
+  }
+});
+
 async function readUntil(reader, pattern) {
   const decoder = new TextDecoder();
   let text = "";

package/examples/improve-goal-maker/goal.md

@@ -1,51 +0,0 @@
-# Improve Goal Maker
-
-## Objective
-
-Improve the local Goal Maker skill by discovering the highest-leverage repo-backed changes, completing the first safe implementation tranche, and leaving a reviewable handoff for anything larger.
-
-## Goal Kind
-
-`open_ended`
-
-## Current Tranche
-
-Map the current Goal Maker implementation, identify concrete improvement candidates with verification paths, select one small and reversible tranche, implement it, verify it, and finish with an audit receipt.
-
-## Non-Negotiable Constraints
-
-- Keep Goal Maker v2 control-file semantics intact: `goal.md`, `state.yaml`, and `notes/` only at each goal root.
-- Do not implement from assumptions; Scout evidence or Judge selection must precede Worker edits.
-- Preserve compatibility with Codex agent roles and the existing checker unless a Judge task explicitly chooses otherwise.
-- Keep changes small, reviewable, and backed by local verification commands.
-- Do not use destructive git operations or require credentials.
-
-## Stop Rule
-
-Stop when the tranche audit passes, all safe local work is blocked, or continuing would require owner input, credentials, destructive operations, or strategy the board cannot decide.
-
-## Canonical Board
-
-Machine truth lives at:
-
-`examples/improve-goal-maker/state.yaml`
-
-If this charter and `state.yaml` disagree, `state.yaml` wins for task status, active task, receipts, verification freshness, and completion truth.
-
-## Run Command
-
-```text
-/goal Follow examples/improve-goal-maker/goal.md
-```
-
-## PM Loop
-
-On every `/goal` continuation:
-
-1. Read this charter.
-2. Read `state.yaml`.
-3. Work only on the active board task.
-4. Assign Scout, Judge, Worker, or PM according to the task.
-5. Write a compact task receipt.
-6. Update the board.
-7. Select the next active task or finish with a Judge/PM audit receipt.

package/examples/improve-goal-maker/notes/T001-repo-map.md

@@ -1,59 +0,0 @@
-# T001: Goal Maker Repo Map
-
-Task: `T001`
-Kind: `scout`
-Status: `current`
-
-## Summary
-
-Goal Maker is a small dependency-free npm package that installs a Codex skill, three agent role definitions, templates, and a v2 board checker. The core verification path is healthy: syntax checks and all current checker tests pass, a temporary install plus doctor run succeeds, and the package dry run includes the expected runtime files. The highest-leverage next investigation is not packaging correctness; it is workflow reliability around checker blind spots, agent-installation assumptions, and template/skill contract drift.
-
-## Repo Map
-
-- `README.md` describes the package purpose, install commands, board model, repository layout, and status caveats.
-- `CONTRIBUTING.md` defines the local verification contract: `npm run check`, temporary install/doctor, and `npm pack --dry-run`.
-- `package.json` exposes the `goal-maker` bin, keeps runtime dependency-free, packages the skill runtime plus docs/assets, and defines `check`, `test`, and `pack:dry-run`.
-- `goal-maker/SKILL.md` is the installed skill contract: direct `$goal-maker` creates or repairs a v2 board, stops for user choice, and `/goal` owns the PM loop.
-- `goal-maker/bin/goal-maker.mjs` installs the skill and bundled `goal_*.toml` agents into a Codex home and supports `install`, `update`, `agents`, `doctor`, and help.
-- `goal-maker/scripts/check-goal-state.mjs` validates v2 board shape with a lightweight line-oriented parser.
-- `goal-maker/scripts/install-agents.mjs` is a secondary agent copy helper.
-- `goal-maker/templates/goal.md`, `state.yaml`, and `note.md` seed new goal control files.
-- `goal-maker/agents/goal_scout.toml`, `goal_worker.toml`, and `goal_judge.toml` define the intended role constraints.
-- `goal-maker/test/check-goal-state.test.mjs` covers key checker invariants.
-
-## Verification Commands
-
-- `npm run check` passed: Node syntax checks and 7 node:test cases passed.
-- Temporary install plus doctor passed: `skill_installed: true`; installed agents were `goal_judge.toml`, `goal_scout.toml`, and `goal_worker.toml`.
-- `npm pack --dry-run` passed and listed the expected package files, including `README.md`, `CONTRIBUTING.md`, `internal/assets/`, `package.json`, and `goal-maker/` runtime files.
-- `node goal-maker/scripts/check-goal-state.mjs examples/improve-goal-maker/state.yaml` passed with `ok: true`.
-
-## Health Signals
-
-- Current tests cover a valid Scout board, active Worker requirements, legacy v1 rejection, one-active-task enforcement, done-task receipt requirements, unexpected root entries, and final audit completion.
-- The checker intentionally avoids dependencies, but it parses YAML with regular expressions and line indentation. This is acceptable for the template subset but creates likely blind spots for richer valid YAML or misleading strings inside quoted values.
-- The package has two agent-installation paths: the CLI in `goal-maker/bin/goal-maker.mjs` and the helper in `goal-maker/scripts/install-agents.mjs`. They are similar but not identical.
-- The direct skill contract says `$goal-maker` must verify Scout, Worker, and Judge agents are installed or explain what is missing; current templates mark them `installed`, while actual verification is behavior implemented by the PM thread, not the CLI/checker.
-
-## Ranked Improvement Candidates
-
-1. Add checker tests and fixes for root-path robustness and YAML parsing edge cases, especially misleading legacy markers inside quoted values or comments.
-2. Reconcile or document the two agent-installation paths so CLI behavior, helper behavior, and README guidance cannot drift.
-3. Make agent installation status less assumption-driven in generated boards or add checker warnings when agent status values are placeholders.
-4. Add tests for the CLI install/doctor/package path so the contributor verification contract is covered by `npm run check`, not just manual commands.
-5. Tighten docs/templates around what `/goal` PM may do in Codex environments where actual subagent dispatch is unavailable and roles are simulated in-thread.
-
-## Candidate Next Tasks
-
-- Investigate checker blind spots with focused failing test candidates before choosing a fix.
-- Compare `goal-maker/bin/goal-maker.mjs` and `goal-maker/scripts/install-agents.mjs` and decide whether to consolidate, test, or document the helper.
-- Judge the first implementation tranche after the risk map, preferring a small test-backed checker or CLI reliability fix.
-
-## Board Receipt Snippet
-
-```yaml
-receipt:
-  result: done
-  note: notes/T001-repo-map.md
-  summary: "Repo map completed; verification is green, install/package checks work, and the top improvement area is Goal Maker workflow reliability around checker blind spots, agent installation assumptions, and template/skill drift."
-```

package/examples/improve-goal-maker/notes/T002-risk-map.md

@@ -1,37 +0,0 @@
-# T002: Workflow Risk Map
-
-Task: `T002`
-Kind: `scout`
-Status: `current`
-
-## Summary
-
-The highest-confidence risks are small and testable. First, `goal-maker doctor` reports success when one of the required bundled agents is missing, which weakens the "verify Scout, Worker, and Judge agents" contract. Second, the board checker accepts mismatched task `type` and `assignee` values, even though Goal Maker semantics depend on those fields agreeing. A lower-priority drift risk is that `update` does not overwrite stale existing agent definitions unless `--force` is supplied, despite README wording that says install or update the skill and bundled agents.
-
-## Evidence
-
-- `goal-maker/bin/goal-maker.mjs` `doctor()` exits based only on whether `skills/goal-maker/SKILL.md` exists. In a temp Codex home with `goal_worker.toml` removed, `doctor` exited 0 and reported only `goal_judge.toml` and `goal_scout.toml`.
-- `goal-maker/scripts/check-goal-state.mjs` independently validates `task.type` and `task.assignee`, but does not validate pairs. A temp board with `type: scout` and `assignee: Worker` passed with `ok: true`.
-- `goal-maker/scripts/check-goal-state.mjs` does not require or validate the `agents:` section. A temp board with the section omitted passed with `ok: true`.
-- `goal-maker/bin/goal-maker.mjs` `installAgents()` skips existing `goal_*.toml` files unless `--force` is present. In a temp Codex home with a stale `goal_scout.toml`, `goal-maker update` left the stale file unchanged.
-- `goal-maker/scripts/install-agents.mjs` copies any `.toml` except names containing `config-snippet`, while the CLI copies only `goal_*.toml`. Current files make this harmless, but it is a drift point.
-
-## Small Safe Implementation Candidates
-
-1. Add CLI tests for `doctor` and make `doctor` fail when any required agent file is missing. Allowed files would likely be `goal-maker/bin/goal-maker.mjs` and a new focused CLI test file.
-2. Add checker tests for task type/assignee mismatch and fix the checker to reject mismatches. Allowed files would be `goal-maker/scripts/check-goal-state.mjs` and `goal-maker/test/check-goal-state.test.mjs`.
-3. Add checker tests for missing or invalid `agents:` statuses and reject or warn. This is slightly more policy-heavy because templates currently treat agent status as PM-populated board metadata.
-4. Decide whether `update` should overwrite existing bundled agents or whether README/CLI wording should clarify that `--force` is required for agent refresh.
-
-## Recommended First Tranche
-
-Prefer the `doctor` missing-agent fix. It directly supports the `$goal-maker` requirement to verify Scout, Worker, and Judge availability, has tight behavior, and can be verified with a new CLI test plus existing checks. The checker mismatch fix is also small, but it is less directly connected to the user-visible setup flow.
-
-## Board Receipt Snippet
-
-```yaml
-receipt:
-  result: done
-  note: notes/T002-risk-map.md
-  summary: "Risk map completed; doctor exits green with missing agents, checker accepts type/assignee mismatches, and update leaves stale agents unless forced."
-```