goalbuddy 0.3.5 → 0.3.6

package/goalbuddy/scripts/check-goal-state.mjs

@@ -369,6 +369,9 @@ for (const task of tasks) {
         errors.push(`Worker receipt for ${task.id} has non-passing command status: ${status}`);
       }
     }
+    if (task.receipt.scalar("needs_judge") === true) {
+      warnings.push(`Worker receipt for ${task.id} requests legacy needs_judge; GoalBuddy now lets the PM continue by default and reviews only at phase, risk, ambiguity, rejected-verification, or final-completion boundaries`);
+    }
   }
   if (task.type === "scout" && task.status === "done" && hasReceipt) {
     if (!task.receipt.has("summary")) errors.push(`Scout receipt for ${task.id} missing summary`);
@@ -381,6 +384,8 @@ for (const task of tasks) {
   }
 }
 
+warnings.push(...microSliceWarnings(tasks, activeTask, goalStatus));
+
 function validateSubgoal(task) {
   if (isChildCheck) {
     errors.push(`child task ${task.id} must not contain a nested subgoal`);
@@ -430,6 +435,77 @@ function validateSubgoal(task) {
   }
 }
 
+function microSliceWarnings(tasks, activeTaskId, goalStatus) {
+  const found = [];
+  const guidance = "Board may be micro-slicing. Prefer the largest safe useful slice.";
+  const doneTasks = tasks.filter((task) => task.status === "done");
+  const workerTasks = tasks.filter((task) => task.type === "worker");
+  const recentTinyWorkers = workerTasks.slice(-5).filter((task) => isTinyTask(task));
+  const firstMilestoneComplete = nestedScalar("goal", "first_milestone_complete") === true;
+
+  if (recentTinyWorkers.length >= 3) {
+    found.push(`${guidance} Three recent Worker tasks look tiny.`);
+  }
+
+  for (const task of tasks) {
+    if (task.type === "judge" && /pick small reviewable work|select one narrow next task/i.test(task.raw)) {
+      found.push(`${guidance} Judge instructions still ask for small or narrow work.`);
+      break;
+    }
+  }
+
+  if (goalStatus !== "active" || !activeTaskId) return [...new Set(found)];
+  const activeIndex = tasks.findIndex((task) => task.id === activeTaskId);
+  if (activeIndex === -1) return [...new Set(found)];
+  const active = tasks[activeIndex];
+  if (active.type === "worker") {
+    if (doneTasks.length >= 10 && active.allowedFiles.length > 0 && active.allowedFiles.length <= 2) {
+      found.push(`${guidance} Active Worker ${active.id} has only ${active.allowedFiles.length} allowed_files after ${doneTasks.length} completed tasks.`);
+    }
+    if (firstMilestoneComplete && isTinyTask(active)) {
+      found.push(`${guidance} The first milestone is complete, so the active Worker should move toward the next real milestone.`);
+    }
+    if (isMicroWorkerTask(active)) {
+      found.push(`${guidance} Active Worker ${active.id} looks like another helper-sized slice.`);
+    }
+  }
+  if (active.type !== "judge") return [...new Set(found)];
+
+  let pairs = 0;
+  for (let index = activeIndex; index > 0; index -= 2) {
+    const judge = tasks[index];
+    const worker = tasks[index - 1];
+    if (!isMicroJudgeForWorker(judge, worker)) break;
+    pairs += 1;
+  }
+  if (pairs >= 2) {
+    found.push(`${guidance} Micro Worker/Judge loop detected ending at ${active.id}.`);
+  }
+  return [...new Set(found)];
+}
+
+function isMicroJudgeForWorker(judge, worker) {
+  if (!judge || !worker) return false;
+  if (judge.type !== "judge" || worker.type !== "worker") return false;
+  if (!["active", "queued", "done"].includes(judge.status) || worker.status !== "done") return false;
+  const objective = String(judge.objective || "").toLowerCase();
+  return objective.includes(worker.id.toLowerCase()) && /audit|review|approve/.test(objective) && isMicroWorkerTask(worker);
+}
+
+function isMicroWorkerTask(task) {
+  if (!task || task.type !== "worker") return false;
+  const objective = String(task.objective || "").toLowerCase();
+  if (/collapsed|batch|package|tranche/.test(objective)) return false;
+  return /one narrow|single helper|one helper|per[- ]helper|per[- ]table|projection helper/.test(objective);
+}
+
+function isTinyTask(task) {
+  if (!task) return false;
+  const text = [task.objective, task.raw, task.receipt?.raw].join(" ").toLowerCase();
+  if (/collapsed|batch|package|tranche|vertical slice|milestone/.test(text)) return false;
+  return /\b(tiny|narrow|single helper|one helper|projection helper|projection function|contract file|read-only proof|doc note|validator|validation wrapper|pure helper|caller-input)\b/.test(text);
+}
+
 function matchesAllowedFile(file, allowedFiles) {
   return allowedFiles.some((pattern) => globMatch(pattern, file));
 }

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

@@ -7,7 +7,7 @@ import { parseGoalStateText } from "../extend/local-goal-board/scripts/lib/goal-
 const ROLE_DEFAULTS = {
   scout: { agent: "goal_scout", reasoning: "low", sandbox: "read-only" },
   judge: { agent: "goal_judge", reasoning: "high", sandbox: "read-only" },
-  worker: { agent: "goal_worker", reasoning: "low", sandbox: "workspace-write" },
+  worker: { agent: "goal_worker", reasoning: "medium", sandbox: "workspace-write" },
   pm: { agent: "PM", reasoning: "medium", sandbox: "workspace-write" },
 };
 
@@ -39,11 +39,13 @@ export function renderTaskPrompt(options) {
     payload: {
       metadata: {
         recommended_agent: defaults.agent,
+        required_spawn_agent_type: defaults.agent === "PM" ? null : defaults.agent,
         recommended_reasoning: reasoning,
         sandbox: defaults.sandbox,
         fork_context_allowed: role !== "worker",
         board_path: board.path,
         child_board_paths: childBoardPaths(board),
+        slice_policy: board.document.rules?.slice_policy || null,
         warnings,
       },
       task: {
@@ -143,15 +145,53 @@ function promptWarnings(board, task) {
     if (stringList(task.allowed_files).length === 0) warnings.push(`Worker task ${task.id} has no allowed_files.`);
     if (stringList(task.verify).length === 0) warnings.push(`Worker task ${task.id} has no verify commands.`);
     if (stringList(task.stop_if).length === 0) warnings.push(`Worker task ${task.id} has no stop_if conditions.`);
+    if (isFalse(board.goal.full_outcome_complete)) {
+      warnings.push(`full_outcome_complete is false and ${task.id} is an active Worker; do not stop after rendering or repairing the board. Execute the Worker unless a stop_if condition applies.`);
+    }
   }
   for (const candidate of board.tasks) {
     if (candidate?.subgoal && Number(candidate.subgoal.depth) !== 1) {
       warnings.push(`Task ${candidate.id} has subgoal.depth ${candidate.subgoal.depth || "<missing>"}; only depth 1 is supported.`);
     }
   }
+  warnings.push(...microSliceWarnings(board, task));
   return warnings;
 }
 
+function microSliceWarnings(board, task) {
+  const warnings = [];
+  const doneTasks = board.tasks.filter((candidate) => candidate?.status === "done");
+  const recentWorkers = board.tasks
+    .filter((candidate) => normalizeRole(candidate?.type) === "worker")
+    .slice(-5);
+  const recentTinyWorkers = recentWorkers.filter((candidate) => isTinyTask(candidate));
+  const activeRole = normalizeRole(task.type);
+  const activeAllowedFiles = stringList(task.allowed_files);
+  const firstMilestoneComplete = isTrue(board.goal.first_milestone_complete);
+  const microWarning = "Board may be micro-slicing. Prefer the largest safe useful slice.";
+
+  if (recentTinyWorkers.length >= 3) warnings.push(microWarning);
+  if (doneTasks.length >= 10 && activeRole === "worker" && activeAllowedFiles.length > 0 && activeAllowedFiles.length <= 2) {
+    warnings.push(`${microWarning} Active Worker ${task.id} has only ${activeAllowedFiles.length} allowed_files after ${doneTasks.length} completed tasks.`);
+  }
+  if (firstMilestoneComplete && activeRole === "worker" && isTinyTask(task)) {
+    warnings.push(`${microWarning} The first milestone is complete, so the active Worker should move toward the next real milestone.`);
+  }
+  if (activeRole === "judge" && /pick small reviewable work|select one narrow next task/i.test(String(task.objective || "") + "\n" + stringList(task.constraints).join("\n"))) {
+    warnings.push(`${microWarning} Judge instructions still ask for small or narrow work.`);
+  }
+  return [...new Set(warnings)];
+}
+
+function isTinyTask(task) {
+  const text = [
+    task?.objective,
+    stringList(task?.constraints).join(" "),
+    task?.receipt?.summary,
+  ].join(" ").toLowerCase();
+  return /\b(tiny|narrow|single helper|one helper|projection helper|projection function|contract file|read-only proof|doc note|validator|validation wrapper|pure helper|caller-input)\b/.test(text);
+}
+
 function normalizeRole(value) {
   const role = String(value || "pm").toLowerCase();
   return ROLE_DEFAULTS[role] ? role : "pm";
@@ -163,6 +203,14 @@ function normalizeReasoning(value, fallback) {
   return fallback;
 }
 
+function isFalse(value) {
+  return value === false || String(value).toLowerCase() === "false";
+}
+
+function isTrue(value) {
+  return value === true || String(value).toLowerCase() === "true";
+}
+
 function stringList(value) {
   return Array.isArray(value) ? value.filter((item) => item !== null && item !== undefined).map(String) : [];
 }
@@ -175,15 +223,14 @@ function receiptSchema(role) {
       commands: [{ cmd: "<command>", status: "pass | fail | not_run" }],
       summary: "<=120 words",
       remaining_blockers: [],
-      needs_judge: false,
     };
   }
   if (role === "judge") {
     return {
       result: "done | blocked",
-      decision: "approve_next | reject_next | approve_subgoal | reject_subgoal | not_complete | complete",
+      decision: "approved | rejected | approve_subgoal | reject_subgoal | not_complete | complete",
+      full_outcome_complete: false,
       evidence: [],
-      next_allowed_task: null,
       blocked_tasks: [],
       required_board_updates: [],
     };
@@ -204,6 +251,7 @@ function formatPrompt(payload) {
     "",
     "Metadata:",
     `- recommended_agent: ${payload.metadata.recommended_agent}`,
+    `- required_spawn_agent_type: ${payload.metadata.required_spawn_agent_type || "PM fallback"}`,
     `- recommended_reasoning: ${payload.metadata.recommended_reasoning}`,
     `- sandbox: ${payload.metadata.sandbox}`,
     `- fork_context_allowed: ${payload.metadata.fork_context_allowed}`,
@@ -213,12 +261,21 @@ function formatPrompt(payload) {
     lines.push("- child_board_paths:");
     for (const path of payload.metadata.child_board_paths) lines.push(`  - ${path}`);
   }
+  if (payload.metadata.slice_policy) {
+    lines.push(`- slice_policy: ${JSON.stringify(payload.metadata.slice_policy)}`);
+  }
   if (payload.metadata.warnings.length) {
     lines.push("- warnings:");
     for (const warning of payload.metadata.warnings) lines.push(`  - ${warning}`);
   }
 
   lines.push(
+    "",
+    "Spawn contract:",
+    `- Codex spawn_agent agent_type: ${payload.metadata.required_spawn_agent_type || "do not spawn; run as PM"}`,
+    "- Do not substitute generic scout, worker, or judge agents for GoalBuddy agents.",
+    "- If the required GoalBuddy agent is unavailable, stop spawning and continue as PM fallback or install agents.",
+    "- After one wait_agent timeout with no visible allowed-file changes, stop waiting and recover deterministically.",
     "",
     "Task:",
     `- id: ${payload.task.id}`,

package/goalbuddy/templates/agents.md

@@ -5,7 +5,7 @@ Use three generic agents. The main `/goal` thread remains PM and owns the board.
 | Agent | model_reasoning_effort | sandbox_mode | Purpose |
 |---|---:|---|---|
 | goal_scout | low | read-only | Targeted evidence mapping and candidate facts |
-| goal_worker | low | workspace-write | One bounded implementation/recovery task |
+| goal_worker | medium | workspace-write | One coherent bounded implementation/recovery slice |
 | goal_judge | high | read-only | Strategic review, escalation, completion skepticism |
 
 ## PM Thinking Policy
@@ -44,5 +44,6 @@ Rules:
 
 - Only the PM loop chooses active tasks, marks tasks done, or completes the goal.
 - Keep at most one write-capable Worker active unless disjoint write scopes are explicit in `state.yaml`.
+- Worker defaults to medium reasoning for implementation tasks and should complete the whole assigned slice.
 - Scout and Judge are read-only and safe to parallelize when their board inputs are clear.
-- Judge is high thinking.
+- Judge is high thinking and should choose the largest safe useful slice, not the narrowest helper.

package/goalbuddy/templates/goal.md

@@ -25,7 +25,7 @@
 
 ## Current Tranche
 
-<What is enough for the full owner outcome, and what is the current safe slice? For execution goals, the default is continuous: discover enough evidence, choose a safe implementation slice, implement it, verify it, audit it, then immediately advance to the next safe slice until the full original outcome is complete. Plan-only or one-slice-only stopping is valid only when explicitly requested.>
+<What is enough for the full owner outcome, and what is the current largest reversible local work package? For execution goals, the default is continuous: discover enough evidence, choose a coherent work package, implement it, verify it, review only at phase/risk/final boundaries, then immediately advance to the next work package until the full original outcome is complete. Plan-only or one-package-only stopping is valid only when explicitly requested.>
 
 ## Non-Negotiable Constraints
 
@@ -37,7 +37,21 @@ Stop only when a final audit proves the full original outcome is complete.
 
 Do not stop after planning, discovery, or Judge selection if the user asked for working software or automation and a safe Worker task can be activated.
 
-Do not stop after a single verified Worker slice when the broader owner outcome still has safe local follow-up slices. After each slice audit, advance the board to the next highest-leverage safe Worker task and continue.
+Do not stop after a single verified Worker package when the broader owner outcome still has safe local follow-up work. Advance the board to the next highest-leverage safe Worker package and continue unless a phase, risk, rejected-verification, ambiguity, or final-completion review is due.
+
+Do not create one Worker/Judge pair per repeated file, table, route, or helper. Put repeated same-shape work into one Worker package and review the package as a whole.
+
+## Slice Sizing
+
+Safe means bounded, explicit, verified, and reversible. It does not mean tiny.
+
+A good task is the largest safe useful slice.
+
+Small is not the goal. Useful is the goal.
+
+A Worker should finish the whole assigned slice. A Judge should judge the whole assigned slice. A PM should reorient the board when tasks are safe but not moving the outcome.
+
+Tiny tasks are allowed when the failure is isolated, the risk is high, the scope is unknown, or the tiny task unlocks a larger slice. Tiny tasks are bad when they keep happening, do not change behavior, only add wrappers/contracts/proof files, or avoid the real milestone.
 
 Do not stop because a slice needs owner input, credentials, production access, destructive operations, or policy decisions. Mark that exact slice blocked with a receipt, create the smallest safe follow-up or workaround task, and continue all local, non-destructive work that can still move the goal toward the full outcome.
 
@@ -67,9 +81,9 @@ On every `/goal` continuation:
 6. Assign Scout, Judge, Worker, or PM according to the task.
 7. Write a compact task receipt.
 8. Update the board.
-9. If Judge selected a safe Worker task with `allowed_files`, `verify`, and `stop_if`, activate it and continue unless blocked.
+9. If safe local work remains, choose the next largest reversible Worker package and continue unless blocked.
 10. If a problem, suggestion, or follow-up should become a repo artifact, create an approved issue/PR or ask the operator whether to create one.
-11. Treat a slice audit as a checkpoint, not completion, unless it explicitly proves the full original outcome is complete.
+11. Review at phase, risk, rejected-verification, ambiguity, or final-completion boundaries; do not review every small Worker by habit.
 12. Finish only with a Judge/PM audit receipt that maps receipts and verification back to the original user outcome and records `full_outcome_complete: true`.
 
 Issue and PR handoffs are supporting artifacts. `state.yaml` remains authoritative, and every external artifact decision must be recorded in a task receipt.

package/goalbuddy/templates/state.yaml

@@ -33,6 +33,11 @@ rules:
   missing_input_or_credentials_do_not_stop_goal: true
   preserve_and_validate_existing_plan: true
   intake_misfire_must_be_audited: true
+  slice_policy:
+    max_consecutive_tiny_tasks: 2
+    prefer_vertical_slices: true
+    judge_picks_largest_safe_slice: true
+    worker_completes_whole_slice: true
 
 agents:
   # installed | bundled_not_installed | missing | unknown
@@ -88,7 +93,7 @@ tasks:
       - "T001 receipt"
     constraints:
       - "Do not implement."
-      - "Pick small reviewable work."
+      - "Pick the largest safe useful slice with clear allowed_files, verify commands, and stop conditions."
     expected_output:
       - "Decision"
       - "Exact Worker objective"

package/internal/cli/goal-maker.mjs

@@ -83,15 +83,23 @@ async function main() {
       break;
     case "install":
     case "update":
+      if (wantsHelp()) {
+        usage();
+        break;
+      }
       if (installTargetMode() === "all") {
         await installEverywhere();
       } else if (installTargetMode() === "codex") {
-        await installAll();
+        installPlugin();
       } else {
         await installClaudeAll();
       }
       break;
     case "agents":
+      if (wantsHelp()) {
+        usage();
+        break;
+      }
       if (targetMode() === "codex") {
         installAgents();
       } else {
@@ -99,6 +107,10 @@ async function main() {
       }
       break;
     case "doctor":
+      if (wantsHelp()) {
+        usage();
+        break;
+      }
       if (targetMode() === "codex") {
         doctor();
       } else {
@@ -110,6 +122,10 @@ async function main() {
       checkUpdate();
       break;
     case "plugin":
+      if (wantsHelp()) {
+        pluginUsage();
+        break;
+      }
       plugin();
       break;
     case "extend":
@@ -164,6 +180,10 @@ function hasFlag(name) {
   return args.includes(name);
 }
 
+function wantsHelp() {
+  return hasFlag("--help") || hasFlag("-h");
+}
+
 function positional(index) {
   return positionalArgs()[index] || "";
 }
@@ -552,7 +572,7 @@ This alias has the same invocation boundary as \`$${canonicalSkillName}\`: prepa
 function installAgents({ quiet = false } = {}) {
   const source = join(skillSource, "agents");
   const target = join(codexHome(), "agents");
-  const force = hasFlag("--force") || command === "update" || command === "install";
+  const force = hasFlag("--force") || command === "update" || command === "install" || command === "default" || command === "plugin";
   mkdirSync(target, { recursive: true });
 
   const results = [];
@@ -601,6 +621,7 @@ async function installAll() {
 function doctor() {
   const skillPath = join(installedSkillRoot(), "SKILL.md");
   const legacySkillPath = join(legacyInstalledSkillRoot(), "SKILL.md");
+  const plugin = installedCodexPlugin();
   const agentsPath = join(codexHome(), "agents");
   const installed = existsSync(skillPath);
   const legacyInstalled = existsSync(legacySkillPath);
@@ -616,12 +637,38 @@ function doctor() {
   });
   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) {
+    errors.push("Codex GoalBuddy plugin is not installed; run `npx goalbuddy --target codex`.");
+  }
+  if (plugin.skill_installed && !plugin.enabled) {
+    errors.push("Codex GoalBuddy plugin cache exists but is not enabled in config.toml; run `npx goalbuddy --target codex`.");
+  }
+  for (const file of missingAgents) {
+    errors.push(`Missing GoalBuddy Codex agent: ${file}; run \`npx goalbuddy --target codex\`.`);
+  }
+  for (const file of staleAgents) {
+    errors.push(`Stale GoalBuddy Codex agent: ${file}; run \`npx goalbuddy update --target codex\`.`);
+  }
+  if (hasFlag("--goal-ready") && !goalRuntime.ready) {
+    errors.push("Native Codex /goal runtime is not ready. GoalBuddy $goal-prep and local boards are separate from OpenAI-gated native /goal.");
+  }
 
   console.log(JSON.stringify({
     codex_home: codexHome(),
+    codex_install_model: "plugin",
+    expected_state: {
+      plugin_cache: true,
+      bundled_skill: "$goal-prep",
+      standalone_personal_skill: false,
+      compatibility_skill: false,
+      agents: requiredAgentFiles,
+      native_goal: "separate OpenAI-gated Codex feature",
+    },
+    plugin,
     skill_installed: installed,
     skill_path: skillPath,
     compatibility_skill_installed: legacyInstalled,
@@ -631,11 +678,14 @@ function doctor() {
     stale_agents: staleAgents,
     goal_runtime: goalRuntime,
     warnings,
+    errors,
   }, null, 2));
 
-  const installOk = installed && missingAgents.length === 0 && staleAgents.length === 0;
+  const pluginOk = plugin.skill_installed && plugin.enabled;
+  const legacySkillOk = installed;
+  const installOk = (pluginOk || legacySkillOk) && missingAgents.length === 0 && staleAgents.length === 0;
   const goalReadyOk = !hasFlag("--goal-ready") || goalRuntime.ready;
-  process.exit(installOk && goalReadyOk ? 0 : 1);
+  process.exit(installOk && goalReadyOk && errors.length === 0 ? 0 : 1);
 }
 
 function checkUpdate() {
@@ -680,6 +730,10 @@ function updateReport() {
 
 function plugin() {
   const subcommand = positional(1) || "";
+  if (wantsHelp()) {
+    pluginUsage();
+    return;
+  }
   switch (subcommand) {
     case "install":
       installPlugin();
@@ -733,6 +787,7 @@ function installPlugin({ quiet = false } = {}) {
   cleanupPreservedExtensions([preservedExtensions.tempPath]);
   const removedLegacySkillPaths = cleanupLegacyCodexSkills();
   const configPath = enablePluginConfig();
+  const agents = installAgents({ quiet: true });
 
   const report = {
     installed: true,
@@ -743,6 +798,7 @@ function installPlugin({ quiet = false } = {}) {
     marketplace_source: source,
     cache_path: pluginCachePath,
     config_path: configPath,
+    agents,
     preserved_extensions: preservedExtensions.ids,
     removed_legacy_skill_paths: removedLegacySkillPaths,
   };
@@ -758,6 +814,7 @@ function installPlugin({ quiet = false } = {}) {
   console.log(`Marketplace: ${source}`);
   console.log(`Cache: ${pluginCachePath}`);
   console.log(`Config: ${configPath}`);
+  console.log(`Agents: ${summarizeStatuses(report.agents)}`);
   if (report.preserved_extensions.length) {
     console.log(`Preserved extensions: ${report.preserved_extensions.join(", ")}`);
   }
@@ -1298,8 +1355,24 @@ function installedSkillRoot() {
 }
 
 function installedPluginSkillRoot() {
+  return installedCodexPlugin().skill_path;
+}
+
+function installedCodexPlugin() {
   const root = join(codexHome(), "plugins", "cache", pluginName, pluginName);
-  if (!existsSync(root)) return "";
+  const configPath = join(codexHome(), "config.toml");
+  const base = {
+    installed: false,
+    enabled: pluginConfigEnabled(configPath),
+    name: `${pluginName}@${pluginName}`,
+    version: "",
+    cache_path: "",
+    manifest_path: "",
+    skill_installed: false,
+    skill_path: "",
+    config_path: configPath,
+  };
+  if (!existsSync(root)) return base;
   const versions = readdirSync(root, { withFileTypes: true })
     .filter((entry) => entry.isDirectory())
     .map((entry) => entry.name)
@@ -1307,10 +1380,37 @@ function installedPluginSkillRoot() {
     .sort(compareVersions)
     .reverse();
   for (const version of versions) {
-    const skillPath = join(root, version, "skills", canonicalSkillDirectory);
-    if (existsSync(join(skillPath, "SKILL.md"))) return skillPath;
+    const cachePath = join(root, version);
+    const skillPath = join(cachePath, "skills", canonicalSkillDirectory);
+    const manifestPath = join(cachePath, ".codex-plugin", "plugin.json");
+    if (existsSync(join(skillPath, "SKILL.md"))) {
+      return {
+        ...base,
+        installed: true,
+        version,
+        cache_path: cachePath,
+        manifest_path: manifestPath,
+        skill_installed: true,
+        skill_path: skillPath,
+      };
+    }
+  }
+  return base;
+}
+
+function pluginConfigEnabled(configPath) {
+  if (!existsSync(configPath)) return false;
+  const lines = readFileSync(configPath, "utf8").split(/\r?\n/);
+  const header = `[plugins."${pluginName}@${pluginName}"]`;
+  const start = lines.findIndex((line) => line.trim() === header);
+  if (start === -1) return false;
+  for (let index = start + 1; index < lines.length; index += 1) {
+    const line = lines[index].trim();
+    if (line.startsWith("[")) break;
+    if (/^enabled\s*=\s*true\b/.test(line)) return true;
+    if (/^enabled\s*=/.test(line)) return false;
   }
-  return "";
+  return false;
 }
 
 function activeSkillRoot() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "goalbuddy",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "description": "A /goal operating system for Codex and Claude Code: subgoals, parallel-agent-ready boards, dark mode, receipts, and verification.",
   "type": "module",
   "bin": {

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

@@ -1,6 +1,6 @@
 {
   "name": "goalbuddy",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "description": "Turn broad Claude Code work into verified GoalBuddy boards with subgoals, parallel-agent-ready handoffs, dark mode, and receipts.",
   "author": {
     "name": "tolibear",

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

@@ -1,6 +1,6 @@
 {
   "name": "goalbuddy",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "description": "Turn broad Codex and Claude Code work into verified GoalBuddy boards with subgoals, parallel-agent-ready handoffs, dark mode, and receipts.",
   "author": {
     "name": "tolibear",

package/plugins/goalbuddy/agents/goal-judge.md

@@ -13,11 +13,16 @@ Hard contract:
 - Read only. Do not edit, stage, install, or implement.
 - Read state receipts before raw files. Then read only the inputs named in the Judge task.
 - Be skeptical of progress. Lots of files, docs, or tests are not completion.
-- A safe Worker task must include exact objective, allowed_files, verify commands, and stop_if.
+- A safe Worker package must include objective, allowed_files, verify commands, and stop_if, and should cover the largest reversible local work package at that boundary.
+- Choose the largest safe useful slice: bounded, explicit, verified, reversible, and outcome-moving. Safety does not mean tiny.
+- Judge a whole useful slice, not one helper at a time.
+- Detect micro-slice loops. Reject another tiny helper when the board has enough scaffolding for vertical progress.
+- Select PM reorientation when recent receipts are safe-looking but outcome-light.
+- Prefer milestone reviews over helper reviews.
 - A safe child board must be depth 1, inside `subgoals/`, non-recursive, and linked from exactly one parent task.
 - Parallel Worker work is safe only with provably disjoint `allowed_files`. Separate boards alone are not proof.
 - Reject completion unless the full original outcome is mapped to receipts and current verification.
-- Do not choose the active task or mutate state.
+- Do not generate routine next tasks, choose the active task, or mutate state. The PM owns continuation after your review.
 
 Return exactly one parseable JSON receipt object:
 
@@ -27,11 +32,10 @@ Return exactly one parseable JSON receipt object:
     "result": "done | blocked",
     "task_id": "<T###>",
     "board_path": "<path to state.yaml>",
-    "decision": "approve_next | reject_next | approve_subgoal | reject_subgoal | not_complete | complete",
+    "decision": "approved | rejected | approve_subgoal | reject_subgoal | not_complete | complete",
     "full_outcome_complete": false,
     "rationale": "<=120 words>",
     "evidence": [],
-    "next_allowed_task": null,
     "subgoal_contract": null,
     "parallel_safety": null,
     "blocked_tasks": [],

package/plugins/goalbuddy/agents/goal-worker.md

@@ -1,12 +1,12 @@
 ---
 name: goal-worker
-description: GoalBuddy Worker. Bounded writer for exactly one active Worker task. Edits only allowed_files, runs verify, returns receipt.
+description: GoalBuddy Worker. Bounded writer for one coherent reversible Worker work package. Edits only allowed_files, runs verify, returns receipt.
 tools: Read, Edit, Write, Grep, Glob, Bash
 ---
 
 You are Worker for GoalBuddy.
 
-Default effort: low. Only use higher effort when the task explicitly sets `reasoning_hint` medium or high.
+Default effort: medium for implementation tasks. Use low only for tiny repair tasks or when the board explicitly sets `reasoning_hint` low.
 
 Hard contract:
 
@@ -18,7 +18,10 @@ Hard contract:
 - Do not create child sub-goals unless the task explicitly allows it.
 - Run the verify commands exactly as listed after edits. You may make at most two fix attempts.
 - Stop immediately if required evidence is missing, a file outside `allowed_files` is needed, source/product/tests conflict, or verification still fails after two attempts.
-- Keep the diff minimal and reversible.
+- Do not request a Judge just because the package is done. The PM decides whether this is a phase, risk, ambiguity, rejected-verification, or final-completion boundary.
+- Keep the diff coherent, bounded, and reversible. Do not shrink the assigned work below the largest safe useful slice.
+- Complete the whole assigned slice. Do not stop after the first helper if remaining work is inside `allowed_files` and verification is still feasible.
+- If the task asks for a vertical slice, complete the vertical slice.
 
 Parallel safety:
 
@@ -39,7 +42,6 @@ Return exactly one parseable JSON receipt object:
     "commands": [],
     "summary": "<=120 words>",
     "remaining_blockers": [],
-    "needs_judge": false,
     "verification_attempts": 1,
     "stopped_because": null
   }