@prevalentware/opencode-goal-plugin 0.1.44 → 0.1.45

package/README.md

@@ -128,7 +128,8 @@ In OpenCode 1, server options use the package-and-options tuple in `opencode.jso
         "no_progress_token_threshold": 50,
         "max_no_progress_turns": 2,
         "restricted_agents": ["plan"],
-        "allow_goal_execution_from_plan": false
+        "allow_goal_execution_from_plan": false,
+        "max_objective_chars": 100000
       }
     ]
   ]
@@ -169,6 +170,10 @@ Defaults:
 - `command_name`: `"goal"`; renames the main goal command only. The reserved names `pause_goal` and `resume_goal` fall back to `goal` so the standalone controls remain available.
 - `restricted_agents`: `["plan"]`; agents (matched case-insensitively) treated as planning-only for goal execution.
 - `allow_goal_execution_from_plan`: `false`; when `true`, disables Plan-mode goal restrictions entirely.
+- `max_objective_chars`: `100000`; maximum Unicode code-point length of the submitted goal objective, completion evidence,
+  and blocker text. The previous 4000-character cap was a defect, not a compatibility constraint. The same limit is
+  advertised on V1 and V2 tool schemas and enforced at runtime, independently per plugin instance. Accepted values are
+  trimmed before persistence. Large objectives are echoed into continuation and compaction prompts.
 
 ## Goal Workflow
 

package/dist/server.js

@@ -370,21 +370,23 @@ async function mutate(fn) {
     }));
   });
 }
-function validateObjective(objective) {
-  const value = objective.trim();
-  if (!value)
-    throw new Error("goal objective must not be empty");
-  if ([...value].length > 4000)
-    throw new Error("goal objective must be at most 4000 characters");
-  return value;
-}
-function validateEvidence(evidence, label) {
-  const value = evidence?.trim();
-  if (!value)
+var DEFAULT_MAX_OBJECTIVE_CHARS = 1e5;
+function resolveMaxObjectiveChars(value) {
+  return typeof value === "number" && Number.isSafeInteger(value) && value > 0 ? value : DEFAULT_MAX_OBJECTIVE_CHARS;
+}
+function boundedText(value, limit, label) {
+  if ([...value].length > limit)
+    throw new Error(`${label} must be at most ${limit} characters`);
+  const trimmed = value.trim();
+  if (!trimmed)
     throw new Error(`${label} must not be empty`);
-  if ([...value].length > 4000)
-    throw new Error(`${label} must be at most 4000 characters`);
-  return value;
+  return trimmed;
+}
+function validateObjective(objective, limit = DEFAULT_MAX_OBJECTIVE_CHARS) {
+  return boundedText(objective, limit, "goal objective");
+}
+function validateEvidence(evidence, label, limit = DEFAULT_MAX_OBJECTIVE_CHARS) {
+  return boundedText(evidence ?? "", limit, label);
 }
 function normalizeState(state) {
   for (const goal of Object.values(state.goals))
@@ -460,7 +462,8 @@ function normalizeCreateOptions(input) {
       noProgressTokenThreshold: DEFAULT_NO_PROGRESS_TOKEN_THRESHOLD,
       maxNoProgressTurns: DEFAULT_MAX_NO_PROGRESS_TURNS,
       agent: null,
-      initialStatus: "active"
+      initialStatus: "active",
+      maxObjectiveChars: DEFAULT_MAX_OBJECTIVE_CHARS
     };
   }
   return {
@@ -470,7 +473,8 @@ function normalizeCreateOptions(input) {
     noProgressTokenThreshold: positiveIntegerOrNull(input?.noProgressTokenThreshold) ?? DEFAULT_NO_PROGRESS_TOKEN_THRESHOLD,
     maxNoProgressTurns: positiveIntegerOrNull(input?.maxNoProgressTurns) ?? DEFAULT_MAX_NO_PROGRESS_TURNS,
     agent: typeof input?.agent === "string" && input.agent.trim() ? input.agent.trim() : null,
-    initialStatus: input?.initialStatus === "paused" ? "paused" : "active"
+    initialStatus: input?.initialStatus === "paused" ? "paused" : "active",
+    maxObjectiveChars: resolveMaxObjectiveChars(input?.maxObjectiveChars)
   };
 }
 function positiveIntegerOrNull(value) {
@@ -570,8 +574,8 @@ async function getGoalInternal(sessionID) {
   return goal ? snapshotInternal(goal) : null;
 }
 async function createGoal(sessionID, objective, options) {
-  const value = validateObjective(objective);
   const normalizedOptions = normalizeCreateOptions(options);
+  const value = validateObjective(objective, resolveMaxObjectiveChars(normalizedOptions.maxObjectiveChars));
   return mutate((state) => {
     const existing = state.goals[sessionID];
     if (existing && !isClosed(existing.status)) {
@@ -623,7 +627,7 @@ async function createGoal(sessionID, objective, options) {
   });
 }
 async function updateGoalObjective(sessionID, objective, status = "active", options) {
-  const value = validateObjective(objective);
+  const value = validateObjective(objective, resolveMaxObjectiveChars(options?.maxObjectiveChars));
   const agent = typeof options?.agent === "string" && options.agent.trim() ? options.agent.trim() : null;
   const planModePause = options?.planModePause === true;
   return mutate((state) => {
@@ -714,7 +718,8 @@ async function setGoalStatus(sessionID, status, agent) {
     return snapshot(goal);
   });
 }
-async function closeGoal(sessionID, input) {
+async function closeGoal(sessionID, input, maxObjectiveChars = DEFAULT_MAX_OBJECTIVE_CHARS) {
+  const limit = resolveMaxObjectiveChars(maxObjectiveChars);
   return mutate((state) => {
     const goal = state.goals[sessionID];
     if (!goal)
@@ -727,12 +732,12 @@ async function closeGoal(sessionID, input) {
     goal.lastAccountedAt = null;
     goal.stopReason = input.status === "complete" ? null : "blocked";
     if (input.status === "complete") {
-      goal.completionEvidence = validateEvidence(input.evidence, "completion evidence");
+      goal.completionEvidence = validateEvidence(input.evidence, "completion evidence", limit);
       goal.blocker = null;
       goal.lastStatus = "Goal completed.";
       pushHistory(goal, "completed", goal.completionEvidence);
     } else {
-      goal.blocker = validateEvidence(input.blocker, "blocker");
+      goal.blocker = validateEvidence(input.blocker, "blocker", limit);
       goal.completionEvidence = null;
       goal.lastStatus = "Goal marked unmet.";
       pushHistory(goal, "unmet", goal.blocker);
@@ -740,11 +745,11 @@ async function closeGoal(sessionID, input) {
     return snapshot(goal);
   });
 }
-async function completeGoal(sessionID, evidence) {
-  return closeGoal(sessionID, { status: "complete", evidence });
+async function completeGoal(sessionID, evidence, maxObjectiveChars = DEFAULT_MAX_OBJECTIVE_CHARS) {
+  return closeGoal(sessionID, { status: "complete", evidence }, maxObjectiveChars);
 }
-async function markGoalUnmet(sessionID, blocker) {
-  return closeGoal(sessionID, { status: "unmet", blocker });
+async function markGoalUnmet(sessionID, blocker, maxObjectiveChars = DEFAULT_MAX_OBJECTIVE_CHARS) {
+  return closeGoal(sessionID, { status: "unmet", blocker }, maxObjectiveChars);
 }
 async function clearGoal(sessionID) {
   return mutate((state) => {
@@ -1269,6 +1274,15 @@ function restrictedAgentSet(options) {
   return new Set(names.map((name) => typeof name === "string" ? name.trim().toLowerCase() : "").filter(Boolean));
 }
 function goalCommandTemplate(commandName) {
+  const createGuidance = [
+    "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.",
+    "Build the objective as a complete, faithful representation of the arguments: keep every requirement, constraint, " + "scope boundary, and success criterion with no omissions or loss of meaning.",
+    "You may restructure and rephrase for clarity and coherence, but do NOT compress, truncate, or drop any content, " + "and do NOT substitute the content with references or pointers to external files.",
+    "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."
+  ].join(" ");
   return `OpenCode goal mode command "/${commandName}" was invoked.
 
 Arguments:
@@ -1287,7 +1301,7 @@ 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, 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.
+- ${createGuidance}
 
 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.`;
 }
@@ -1949,9 +1963,24 @@ function getGoalToolResult(goal) {
   }
   return JSON.stringify(result, null, 2);
 }
+function boundedGoalTextSchema(limit, description, validate) {
+  return z.string().superRefine((value, ctx) => {
+    try {
+      validate(value);
+    } catch (error) {
+      ctx.addIssue({
+        code: "custom",
+        message: error instanceof Error ? error.message : String(error)
+      });
+    }
+  }).meta({ minLength: 1, maxLength: limit, pattern: "\\S", description });
+}
+function v2GoalTextSchema(limit, description) {
+  return { type: "string", minLength: 1, maxLength: limit, pattern: "\\S", description };
+}
 async function createGoalFromTool(input, context, services) {
   const planningOnly = services.isPlanAgent(context.agent);
-  const objective = validateObjective(input.objective);
+  const objective = validateObjective(input.objective, services.maxObjectiveChars);
   const existing = await getGoal(context.sessionID);
   if (existing && !isClosedGoal(existing))
     return existingGoalResult(existing, objective, planningOnly);
@@ -1964,7 +1993,8 @@ async function createGoalFromTool(input, context, services) {
       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"
+      initialStatus: planningOnly ? "paused" : "active",
+      maxObjectiveChars: services.maxObjectiveChars
     });
   } catch (error) {
     if (!(error instanceof Error) || !error.message.includes("non-closed goal"))
@@ -1994,18 +2024,19 @@ async function updateGoalObjectiveFromTool(input, context, services) {
   const planningOnly = requested === "active" && services.isPlanAgent(context.agent);
   const goal = await updateGoalObjective(context.sessionID, input.objective, planningOnly ? "paused" : requested, {
     agent: typeof context.agent === "string" ? context.agent : null,
-    planModePause: planningOnly
+    planModePause: planningOnly,
+    maxObjectiveChars: services.maxObjectiveChars
   });
   return JSON.stringify(planningOnly ? { goal, plan_mode_notice: PLAN_MODE_CREATE_NOTICE } : { goal }, null, 2);
 }
-async function closeGoalFromTool(input, context) {
+async function closeGoalFromTool(input, context, services) {
   if (input.status === "complete") {
-    const goal = await completeGoal(context.sessionID, input.evidence ?? "");
+    const goal = await completeGoal(context.sessionID, input.evidence ?? "", services.maxObjectiveChars);
     const budget = goal.tokenBudget == null ? "" : ` Token usage: ${goal.tokensUsed}/${goal.tokenBudget}.`;
     const report = `Goal achieved. Time used: ${goal.timeUsedSeconds} seconds.${budget} Evidence: ${goal.completionEvidence}.`;
     return JSON.stringify({ goal, completion_report: report }, null, 2);
   }
-  const goal = await markGoalUnmet(context.sessionID, input.blocker ?? "");
+  const goal = await markGoalUnmet(context.sessionID, input.blocker ?? "", services.maxObjectiveChars);
   const report = `Goal unmet. Time used: ${goal.timeUsedSeconds} seconds. Blocker: ${goal.blocker}.`;
   return JSON.stringify({ goal, unmet_report: report }, null, 2);
 }
@@ -2069,6 +2100,7 @@ var server = async ({ client }, options) => {
   const maxPromptFailures = positiveIntegerOrNull2(options?.max_prompt_failures) ?? DEFAULT_MAX_PROMPT_FAILURES;
   const registerCommand = options?.register_command ?? true;
   const commandName = commandNameFromOptions(options);
+  const objectiveChars = resolveMaxObjectiveChars(options?.max_objective_chars);
   const taskTracker = new TaskTracker;
   const taskDeferredSessions = new Set;
   const scheduledContinuations = new Map;
@@ -2080,7 +2112,7 @@ var server = async ({ client }, options) => {
   const watchdogRescuedSessions = new Set;
   const planAgents = restrictedAgentSet(options);
   const isPlanAgent = (agent) => typeof agent === "string" && planAgents.has(agent.trim().toLowerCase());
-  const goalServices = { options: options ?? {}, isPlanAgent };
+  const goalServices = { options: options ?? {}, isPlanAgent, maxObjectiveChars: objectiveChars };
   const stopStateRecoveryReporting = onStateRecovery(statePath(), async ({ stateFile, quarantineFile, outcome, error }) => {
     await client.app?.log?.({
       body: {
@@ -2382,7 +2414,7 @@ var server = async ({ client }, options) => {
       create_goal: {
         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."),
+          objective: boundedGoalTextSchema(objectiveChars, "The concrete objective to start pursuing.", (value) => validateObjective(value, objectiveChars)),
           token_budget: z.number().int().positive().nullable().optional().describe("Optional positive token budget."),
           max_auto_turns: z.number().int().positive().nullable().optional().describe("Optional per-goal auto-continue limit."),
           max_duration_seconds: z.number().int().positive().nullable().optional().describe("Optional per-goal duration limit.")
@@ -2394,7 +2426,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. 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."),
+          objective: boundedGoalTextSchema(objectiveChars, "The model-formulated concrete objective to start pursuing.", (value) => validateObjective(value, objectiveChars)),
           token_budget: z.number().int().positive().nullable().optional().describe("Optional positive token budget."),
           max_auto_turns: z.number().int().positive().nullable().optional().describe("Optional per-goal auto-continue limit."),
           max_duration_seconds: z.number().int().positive().nullable().optional().describe("Optional per-goal duration limit.")
@@ -2406,7 +2438,7 @@ var server = async ({ client }, options) => {
       update_goal_objective: {
         description: "Edit the current OpenCode goal objective when the user explicitly asks to edit or replace it.",
         args: {
-          objective: z.string().min(1).max(4000).describe("The updated concrete objective."),
+          objective: boundedGoalTextSchema(objectiveChars, "The updated concrete objective.", (value) => validateObjective(value, objectiveChars)),
           status: z.enum(["active", "paused"]).optional().describe("Whether the edited goal should be active or paused.")
         },
         async execute(args, context) {
@@ -2417,11 +2449,11 @@ var server = async ({ client }, options) => {
         description: "Close the existing goal only after an audit against real evidence. Use status complete only when the objective is achieved and no required work remains, and include evidence. Use status unmet only when the objective cannot be achieved or is blocked, and include the blocker. Do not close a goal merely because work is stopping.",
         args: {
           status: z.enum(["complete", "unmet"]).describe("Required. complete means achieved; unmet means blocked or impossible."),
-          evidence: z.string().min(1).max(4000).optional().describe("Required when status is complete. Summarize the concrete evidence verified."),
-          blocker: z.string().min(1).max(4000).optional().describe("Required when status is unmet. Explain the concrete blocker or impossibility.")
+          evidence: boundedGoalTextSchema(objectiveChars, "Required when status is complete. Summarize the concrete evidence verified.", (value) => validateEvidence(value, "completion evidence", objectiveChars)).optional(),
+          blocker: boundedGoalTextSchema(objectiveChars, "Required when status is unmet. Explain the concrete blocker or impossibility.", (value) => validateEvidence(value, "blocker", objectiveChars)).optional()
         },
         async execute(args, context) {
-          return closeGoalFromTool(args, context);
+          return closeGoalFromTool(args, context, goalServices);
         }
       },
       update_goal_status: {
@@ -2642,6 +2674,7 @@ async function setupV2(context) {
   const maxPromptFailures = positiveIntegerOrNull2(options.max_prompt_failures) ?? DEFAULT_MAX_PROMPT_FAILURES;
   const registerCommand = options.register_command ?? true;
   const commandName = commandNameFromOptions(options);
+  const objectiveChars = resolveMaxObjectiveChars(options.max_objective_chars);
   const taskTracker = new TaskTracker;
   const taskDeferredSessions = new Set;
   const scheduledContinuations = new Map;
@@ -2659,6 +2692,7 @@ async function setupV2(context) {
   const stepTokenSums = new Map;
   const goalServices = {
     options,
+    maxObjectiveChars: objectiveChars,
     isPlanAgent,
     initializeUsage: async (sessionID) => {
       try {
@@ -3327,7 +3361,7 @@ function goalToolsV2(services) {
       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. 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." },
+        objective: v2GoalTextSchema(services.maxObjectiveChars, "The concrete objective to start pursuing."),
         token_budget: { type: ["integer", "null"], minimum: 1, description: "Optional positive token budget." },
         max_auto_turns: { type: ["integer", "null"], minimum: 1, description: "Optional per-goal auto-continue limit." },
         max_duration_seconds: { type: ["integer", "null"], minimum: 1, description: "Optional per-goal duration limit." }
@@ -3341,12 +3375,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. 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 model-formulated concrete objective to start pursuing."
-        },
+        objective: v2GoalTextSchema(services.maxObjectiveChars, "The model-formulated concrete objective to start pursuing."),
         token_budget: { type: ["integer", "null"], minimum: 1, description: "Optional positive token budget." },
         max_auto_turns: { type: ["integer", "null"], minimum: 1, description: "Optional per-goal auto-continue limit." },
         max_duration_seconds: { type: ["integer", "null"], minimum: 1, description: "Optional per-goal duration limit." }
@@ -3360,7 +3389,7 @@ function goalToolsV2(services) {
       name: "update_goal_objective",
       description: "Edit the current OpenCode goal objective when the user explicitly asks to edit or replace it.",
       input: v2ObjectSchema({
-        objective: { type: "string", minLength: 1, maxLength: 4000, description: "The updated concrete objective." },
+        objective: v2GoalTextSchema(services.maxObjectiveChars, "The updated concrete objective."),
         status: { type: "string", enum: ["active", "paused"], description: "Whether the edited goal should be active or paused." }
       }, ["objective"]),
       options: { codemode: false },
@@ -3377,22 +3406,12 @@ function goalToolsV2(services) {
           enum: ["complete", "unmet"],
           description: "Required. complete means achieved; unmet means blocked or impossible."
         },
-        evidence: {
-          type: "string",
-          minLength: 1,
-          maxLength: 4000,
-          description: "Required when status is complete. Summarize the concrete evidence verified."
-        },
-        blocker: {
-          type: "string",
-          minLength: 1,
-          maxLength: 4000,
-          description: "Required when status is unmet. Explain the concrete blocker or impossibility."
-        }
+        evidence: v2GoalTextSchema(services.maxObjectiveChars, "Required when status is complete. Summarize the concrete evidence verified."),
+        blocker: v2GoalTextSchema(services.maxObjectiveChars, "Required when status is unmet. Explain the concrete blocker or impossibility.")
       }, ["status"]),
       options: { codemode: false },
       execute: async (args, toolContext) => ({
-        content: await closeGoalFromTool(args, toolContext)
+        content: await closeGoalFromTool(args, toolContext, services)
       })
     },
     {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prevalentware/opencode-goal-plugin",
-  "version": "0.1.44",
+  "version": "0.1.45",
   "description": "OpenCode goal plugin that adds Codex-style long-running goal mode, /goal commands, persistence, and TUI status for AI coding agents.",
   "keywords": [
     "opencode",