@prevalentware/opencode-goal-plugin 0.1.36 → 0.1.38

package/README.md

@@ -18,7 +18,7 @@ The OpenCode Goal Plugin adds:
 
 - `/goal <objective>` as an OpenCode command for TUI, desktop, and web.
 - A sidebar goal indicator with status, elapsed time, and objective.
-- Agent tools: `get_goal`, `get_goal_history`, `create_goal`, `set_goal`, `update_goal_objective`, `update_goal`, and `clear_goal`.
+- Agent tools: `get_goal`, `get_goal_history`, `list_all_goals`, `create_goal`, `set_goal`, `update_goal_objective`, `update_goal`, and `clear_goal`.
 - Goal close evidence: `complete` requires verified evidence, and `unmet` requires a concrete blocker.
 - Persistent per-session goal state with history, checkpoints, budgets, and owner-only file permissions.
 - Optional automatic continuation on `session.idle` / `session.status`, with no-progress pause and budget wrap-up safeguards.
@@ -239,7 +239,7 @@ This plugin follows Codex's native goal-mode semantics where OpenCode plugin hoo
 
 ```bash
 bun install
-bun test
+bun run test
 bun run lint
 bun run typecheck
 bun run build

package/dist/server.js

@@ -127,6 +127,7 @@ class StateWriteError extends Data.TaggedError("StateWriteError") {
 }
 var MAX_HISTORY_ENTRIES = 50;
 var MAX_CHECKPOINTS = 8;
+var MAX_LISTED_GOALS = 50;
 var CHECKPOINT_CHAR_LIMIT = 280;
 var DEFAULT_NO_PROGRESS_TOKEN_THRESHOLD = 50;
 var DEFAULT_MAX_NO_PROGRESS_TURNS = 2;
@@ -241,8 +242,7 @@ function parseStateText(raw, file) {
 function decodeState(value) {
   return Schema.decodeUnknown(StateSchema)(value).pipe(Effect.map(mutableState), Effect.map(normalizeState), Effect.mapError((cause) => new StateDecodeError({ cause })));
 }
-function readStateEffect() {
-  const file = statePath();
+function readStateEffect(file = statePath()) {
   return Effect.tryPromise({
     try: () => readFile(file, "utf8"),
     catch: (cause) => new StateReadError({ cause })
@@ -251,10 +251,9 @@ function readStateEffect() {
     catch: (cause) => new StateDecodeError({ cause })
   })), Effect.flatMap(decodeState), Effect.catchAll((error) => error._tag === "StateReadError" && isMissingStateFile(error.cause) ? Effect.succeed(emptyState()) : Effect.fail(error)));
 }
-function writeStateEffect(state) {
+function writeStateEffect(state, file = statePath()) {
   return Effect.tryPromise({
     try: async () => {
-      const file = statePath();
       await mkdir(dirname2(file), { recursive: true, mode: 448 });
       await atomicWriteFile(file, JSON.stringify(state, null, 2) + `
 `);
@@ -276,15 +275,18 @@ function enqueueMutation(operation) {
   return current;
 }
 async function mutate(fn) {
-  return enqueueMutation(() => Effect.runPromise(Effect.gen(function* () {
-    const state = yield* readStateEffect();
-    const result = yield* Effect.tryPromise({
-      try: () => Promise.resolve(fn(state)),
-      catch: (cause) => cause instanceof Error ? cause : new Error(String(cause))
-    });
-    yield* writeStateEffect(state);
-    return result;
-  })));
+  return enqueueMutation(() => {
+    const file = statePath();
+    return Effect.runPromise(Effect.gen(function* () {
+      const state = yield* readStateEffect(file);
+      const result = yield* Effect.tryPromise({
+        try: () => Promise.resolve(fn(state)),
+        catch: (cause) => cause instanceof Error ? cause : new Error(String(cause))
+      });
+      yield* writeStateEffect(state, file);
+      return result;
+    }));
+  });
 }
 function validateObjective(objective) {
   const value = objective.trim();
@@ -456,6 +458,30 @@ async function getGoal(sessionID) {
   const goal = state.goals[sessionID];
   return goal ? snapshot(goal) : null;
 }
+async function getAllGoals() {
+  const state = await readState();
+  const sorted = Object.values(state.goals).sort((left, right) => right.updatedAt - left.updatedAt || (left.sessionID < right.sessionID ? -1 : left.sessionID > right.sessionID ? 1 : 0));
+  const goals = sorted.slice(0, MAX_LISTED_GOALS).map(goalListItem);
+  return { goals, total: sorted.length, truncated: sorted.length > goals.length };
+}
+function goalListItem(goal) {
+  return {
+    sessionID: goal.sessionID,
+    objective: goal.objective,
+    status: goal.status,
+    tokenBudget: goal.tokenBudget,
+    tokensUsed: goal.tokensUsed,
+    timeUsedSeconds: goal.timeUsedSeconds,
+    createdAt: goal.createdAt,
+    updatedAt: goal.updatedAt,
+    closedAt: goal.closedAt ?? null,
+    maxAutoTurns: goal.maxAutoTurns,
+    maxDurationSeconds: goal.maxDurationSeconds,
+    autoTurns: goal.autoTurns,
+    stopReason: goal.stopReason,
+    remainingTokens: remainingTokens(goal)
+  };
+}
 async function getGoalInternal(sessionID) {
   const state = await readState();
   const goal = state.goals[sessionID];
@@ -1140,10 +1166,13 @@ var STALE_PENDING_MS = 30000;
 var RETRY_SETTLE_MS = 25;
 var TRANSPORT_ERROR_PATTERN = /\b(?:network|fetch|socket|connect|connection|timeout|timed out|ECONNRESET|ECONNREFUSED|ETIMEDOUT|EAI_AGAIN|ENOTFOUND|EPIPE|transport|stream|websocket|offline|internet|request failed|proxy)\b/i;
 var NON_TRANSPORT_TERMINAL_PATTERN = /\b(?:abort(?:ed)?|interrupt(?:ed|ion)?)\b/i;
-var NON_PROGRESS_TOOLS = new Set(["get_goal", "get_goal_history"]);
+var NON_PROGRESS_TOOLS = new Set(["get_goal", "get_goal_history", "list_all_goals"]);
 var TASK_TERMINAL_STATES = new Set(["completed", "error", "cancelled"]);
 var PLAN_MODE_CREATE_NOTICE = 'Goal recorded while the session is in Plan mode, so execution is paused. Do not start implementation work now. Ask the user to switch to Build mode and resume the goal (for example with "/goal resume") to begin execution.';
 var LIMITED_GOAL_NOTICE = "Safety limit reached. Do not start or continue substantive work for this goal. Summarize useful progress, remaining work, and blockers, then wait for the user to resume or edit the goal.";
+var DUPLICATE_GOAL_NOTICE = "This non-closed goal already exists. Do not call create_goal or set_goal again. The existing objective and limits were preserved; repeated-call arguments were not applied. Use the returned goal state and continue only when its status permits execution.";
+var CONFLICTING_GOAL_NOTICE = "A different non-closed goal already exists. Do not call create_goal or set_goal again. Report the conflict instead of replacing the goal; edit, clear, complete, or mark it unmet only when explicitly requested.";
+var RESTRICTED_GOAL_NOTICE = "Goal execution is not allowed from the current restricted agent or while the goal is paused for Plan mode. Switch to Build mode and resume the goal before doing substantive work.";
 var activeContinuations = new Set;
 function restrictedAgentSet(options) {
   if (options?.allow_goal_execution_from_plan === true)
@@ -1170,9 +1199,9 @@ Use the goal tools to handle this command:
 - If the arguments start with "edit ", update the current goal objective by calling update_goal_objective with the remaining text.
 - If the arguments start with "complete " or "done ", perform a completion audit against real artifacts and command output. Call update_goal with status "complete" only if the goal is achieved, using concise evidence from the audit.
 - If the arguments start with "unmet ", "blocked ", or "blocker ", call update_goal with status "unmet" only when the goal cannot be achieved or needs external input, using the remaining arguments as the blocker.
-- Otherwise, create a new goal with create_goal. Use the full arguments as the objective. If the user includes explicit budget instructions, pass token_budget, max_auto_turns, or max_duration_seconds to create_goal rather than leaving those words in the objective.
+- Otherwise, call get_goal first. If it returns a non-closed goal with the same objective, do not create it again; continue working from the returned state. If it returns a different non-closed goal, report that conflict instead of replacing it. Only when there is no non-closed goal, call create_goal once. Use the full arguments as the objective. If the user includes explicit budget instructions, pass token_budget, max_auto_turns, or max_duration_seconds to create_goal rather than leaving those words in the objective.
 
-Create a goal only from these explicit command arguments. Do not infer a goal from unrelated session context. After create_goal succeeds, continue working toward the new goal.`;
+Create a goal only from these explicit command arguments. Do not infer a goal from unrelated session context. After create_goal succeeds or returns an existing matching goal, never call it again for this command; continue working from the returned goal state.`;
 }
 function commandNameFromOptions(options) {
   const name = options?.command_name?.trim() || DEFAULT_COMMAND_NAME;
@@ -1773,18 +1802,44 @@ function getGoalToolResult(goal) {
 }
 async function createGoalFromTool(input, context, services) {
   const planningOnly = services.isPlanAgent(context.agent);
-  const goal = await createGoal(context.sessionID, input.objective, {
-    tokenBudget: input.token_budget ?? services.options.default_token_budget ?? null,
-    maxAutoTurns: input.max_auto_turns ?? null,
-    maxDurationSeconds: input.max_duration_seconds ?? services.options.max_goal_duration_seconds ?? null,
-    noProgressTokenThreshold: services.options.no_progress_token_threshold ?? null,
-    maxNoProgressTurns: services.options.max_no_progress_turns ?? null,
-    agent: typeof context.agent === "string" ? context.agent : null,
-    initialStatus: planningOnly ? "paused" : "active"
-  });
+  const objective = validateObjective(input.objective);
+  const existing = await getGoal(context.sessionID);
+  if (existing && !isClosedGoal(existing))
+    return existingGoalResult(existing, objective, planningOnly);
+  let goal;
+  try {
+    goal = await createGoal(context.sessionID, input.objective, {
+      tokenBudget: input.token_budget ?? services.options.default_token_budget ?? null,
+      maxAutoTurns: input.max_auto_turns ?? null,
+      maxDurationSeconds: input.max_duration_seconds ?? services.options.max_goal_duration_seconds ?? null,
+      noProgressTokenThreshold: services.options.no_progress_token_threshold ?? null,
+      maxNoProgressTurns: services.options.max_no_progress_turns ?? null,
+      agent: typeof context.agent === "string" ? context.agent : null,
+      initialStatus: planningOnly ? "paused" : "active"
+    });
+  } catch (error) {
+    if (!(error instanceof Error) || !error.message.includes("non-closed goal"))
+      throw error;
+    const raced = await getGoal(context.sessionID);
+    if (raced && !isClosedGoal(raced))
+      return existingGoalResult(raced, objective, planningOnly);
+    throw error;
+  }
   await services.initializeUsage?.(context.sessionID);
   return JSON.stringify(planningOnly ? { goal, plan_mode_notice: PLAN_MODE_CREATE_NOTICE } : { goal }, null, 2);
 }
+function isClosedGoal(goal) {
+  return goal.status === "complete" || goal.status === "unmet";
+}
+function existingGoalResult(goal, requestedObjective, planningOnly) {
+  const reused = goal.objective === requestedObjective;
+  return JSON.stringify({
+    goal,
+    ...reused ? { goal_reused: true, duplicate_goal_notice: DUPLICATE_GOAL_NOTICE } : { goal_conflict: true, goal_conflict_notice: CONFLICTING_GOAL_NOTICE },
+    ...goal.status === "budgetLimited" || goal.status === "usageLimited" ? { goal_mode_notice: LIMITED_GOAL_NOTICE } : {},
+    ...planningOnly || goal.stopReason === PLAN_MODE_STOP_REASON ? { plan_mode_notice: RESTRICTED_GOAL_NOTICE } : {}
+  }, null, 2);
+}
 async function updateGoalObjectiveFromTool(input, context, services) {
   const requested = input.status ?? "active";
   const planningOnly = requested === "active" && services.isPlanAgent(context.agent);
@@ -2142,8 +2197,15 @@ var server = async ({ client }, options) => {
           return JSON.stringify({ goal, history_report: formatGoalHistory(goal) }, null, 2);
         }
       },
+      list_all_goals: {
+        description: "List up to 50 public goal summaries across all sessions in this state file, ordered by most recently updated first. Elapsed time is the last persisted value; total and truncated report omitted older goals.",
+        args: {},
+        async execute() {
+          return JSON.stringify(await getAllGoals(), null, 2);
+        }
+      },
       create_goal: {
-        description: "Create a goal only when explicitly requested by the user or system/developer instructions; do not infer goals from ordinary tasks. Fails if a non-complete goal exists. While the session is in Plan mode, the goal is recorded as paused and execution requires the user to switch to Build mode.",
+        description: "Create a goal only when explicitly requested by the user or system/developer instructions; do not infer goals from ordinary tasks. If any non-closed goal exists, this returns the existing goal as either reused or conflicting and must not be retried. While the session is in Plan mode, the goal is recorded as paused and execution requires the user to switch to Build mode.",
         args: {
           objective: z.string().min(1).max(4000).describe("The concrete objective to start pursuing."),
           token_budget: z.number().int().positive().nullable().optional().describe("Optional positive token budget."),
@@ -2155,7 +2217,7 @@ var server = async ({ client }, options) => {
         }
       },
       set_goal: {
-        description: "Set a new goal when the user explicitly asks the agent to formulate and set its own goal. The model should write the objective itself based on the user's explicit request. Fails if a non-complete goal exists. While the session is in Plan mode, the goal is recorded as paused and execution requires the user to switch to Build mode.",
+        description: "Set a new goal when the user explicitly asks the agent to formulate and set its own goal. The model should write the objective itself based on the user's explicit request. If any non-closed goal exists, this returns the existing goal as either reused or conflicting and must not be retried. While the session is in Plan mode, the goal is recorded as paused and execution requires the user to switch to Build mode.",
         args: {
           objective: z.string().min(1).max(4000).describe("The model-formulated concrete objective to start pursuing."),
           token_budget: z.number().int().positive().nullable().optional().describe("Optional positive token budget."),
@@ -3017,9 +3079,18 @@ function goalToolsV2(services) {
         return { content: JSON.stringify({ goal, history_report: formatGoalHistory(goal) }, null, 2) };
       }
     },
+    {
+      name: "list_all_goals",
+      description: "List up to 50 public goal summaries across all sessions in this state file, ordered by most recently updated first. Elapsed time is the last persisted value; total and truncated report omitted older goals.",
+      input: v2ObjectSchema({}),
+      options: { codemode: false },
+      execute: async () => ({
+        content: JSON.stringify(await getAllGoals(), null, 2)
+      })
+    },
     {
       name: "create_goal",
-      description: "Create a goal only when explicitly requested by the user or system/developer instructions; do not infer goals from ordinary tasks. Fails if a non-complete goal exists. While the session is in Plan mode, the goal is recorded as paused and execution requires the user to switch to Build mode.",
+      description: "Create a goal only when explicitly requested by the user or system/developer instructions; do not infer goals from ordinary tasks. If any non-closed goal exists, this returns the existing goal as either reused or conflicting and must not be retried. While the session is in Plan mode, the goal is recorded as paused and execution requires the user to switch to Build mode.",
       input: v2ObjectSchema({
         objective: { type: "string", minLength: 1, maxLength: 4000, description: "The concrete objective to start pursuing." },
         token_budget: { type: ["integer", "null"], minimum: 1, description: "Optional positive token budget." },
@@ -3033,7 +3104,7 @@ function goalToolsV2(services) {
     },
     {
       name: "set_goal",
-      description: "Set a new goal when the user explicitly asks the agent to formulate and set its own goal. The model should write the objective itself based on the user's explicit request. Fails if a non-complete goal exists. While the session is in Plan mode, the goal is recorded as paused and execution requires the user to switch to Build mode.",
+      description: "Set a new goal when the user explicitly asks the agent to formulate and set its own goal. The model should write the objective itself based on the user's explicit request. If any non-closed goal exists, this returns the existing goal as either reused or conflicting and must not be retried. While the session is in Plan mode, the goal is recorded as paused and execution requires the user to switch to Build mode.",
       input: v2ObjectSchema({
         objective: {
           type: "string",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prevalentware/opencode-goal-plugin",
-  "version": "0.1.36",
+  "version": "0.1.38",
   "description": "OpenCode goal plugin that adds Codex-style long-running goal mode, /goal commands, persistence, and TUI status for AI coding agents.",
   "keywords": [
     "opencode",
@@ -49,6 +49,7 @@
     "lint": "eslint .",
     "pack:dry-run": "npm pack --dry-run",
     "test": "bun test",
+    "test:coverage": "bun test --coverage",
     "typecheck": "tsc --noEmit",
     "prepublishOnly": "bun run test && bun run build"
   },