@mocito/pi-goal 0.1.10 → 0.1.11

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## Unreleased
 
+## 0.1.11 - 2026-07-17
+
+### Fixed
+
+- Preserve accumulated wall-clock time when `/goal resume` is used on an already-active goal.
+- Derive persisted `piGoalVersion` metadata from the package version so release versions cannot drift.
+- Ignore malformed persisted mutations instead of allowing invalid status, budget, or accounting data to corrupt reconstructed goal state.
+
 ## 0.1.10 - 2026-07-01
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mocito/pi-goal",
-  "version": "0.1.10",
+  "version": "0.1.11",
   "description": "Persistent long-running goals for Pi.",
   "type": "module",
   "license": "MIT",
@@ -56,9 +56,9 @@
     "@earendil-works/pi-ai": "^0.80.0",
     "@earendil-works/pi-coding-agent": "^0.80.0",
     "@earendil-works/pi-tui": "^0.80.0",
-    "@types/node": "^25.9.3",
-    "typebox": "^1.2.10",
-    "tsx": "^4.22.4",
+    "@types/node": "^26.1.0",
+    "typebox": "^1.3.4",
+    "tsx": "^4.23.0",
     "typescript": "^6.0.3"
   },
   "publishConfig": {

package/src/commands.ts

@@ -94,9 +94,10 @@ async function editGoal(pi: ExtensionAPI, ctx: ExtensionCommandContext, runtime:
 function setGoalStatus(pi: ExtensionAPI, ctx: ExtensionCommandContext, runtime: CommandRuntime, status: "paused" | "active"): void {
   const current = accountCurrent(pi, ctx, runtime.getGoal());
   if (!current) return ctx.ui.notify("No goal set.", "error");
-  const time = status === "active" ? current.timeUsedSeconds : realizedTimeUsed(current);
-  const activeStartedAt = status === "active" ? nowIso() : undefined;
-  const mutation = statusMutation(current, status, time, activeStartedAt, transitionMeta(status === "active" ? "command:/goal resume" : "command:/goal pause", current, time, nowIso()));
+  const transitionedAt = nowIso();
+  const time = realizedTimeUsed(current, Date.parse(transitionedAt));
+  const activeStartedAt = status === "active" ? transitionedAt : undefined;
+  const mutation = statusMutation(current, status, time, activeStartedAt, transitionMeta(status === "active" ? "command:/goal resume" : "command:/goal pause", current, time, transitionedAt));
   appendGoalMutation(pi, mutation);
   runtime.setGoal(applyGoalMutation(current, mutation));
   runtime.afterGoalChanged(ctx, status === "active" ? "Goal resumed." : "Goal paused.");

package/src/metadata.ts

@@ -1,7 +1,15 @@
+import { createRequire } from "node:module";
 import type { BranchEntry, GoalMutationMeta, GoalState } from "./types.js";
 import { secondsBetween } from "./utils.js";
 
-export const PI_GOAL_VERSION = "0.1.7";
+const packageJson = createRequire(import.meta.url)("../package.json") as { version?: unknown };
+
+if (typeof packageJson.version !== "string" || packageJson.version.length === 0) {
+  throw new Error("pi-goal package version is missing or invalid.");
+}
+
+/** Package version that produced the persisted metadata; schema versioning is separate. */
+export const PI_GOAL_VERSION = packageJson.version;
 
 export function withPiGoalVersion(meta: GoalMutationMeta = {}): GoalMutationMeta {
   return { piGoalVersion: PI_GOAL_VERSION, ...meta };

package/src/state.ts

@@ -70,10 +70,50 @@ export function applyGoalMutation(current: GoalState | null, mutation: GoalMutat
 }
 
 function isKnownMutation(value: Partial<GoalMutation> | undefined): value is GoalMutation {
-  if (!value || value.schemaVersion !== GOAL_SCHEMA_VERSION || typeof value.kind !== "string" || typeof value.at !== "string") return false;
-  if (["create", "replace"].includes(value.kind)) return typeof value.goalId === "string" && typeof (value as any).objective === "string";
-  if (["edit", "status", "budget", "account"].includes(value.kind)) return typeof value.goalId === "string";
-  return value.kind === "clear";
+  if (!value || value.schemaVersion !== GOAL_SCHEMA_VERSION || typeof value.kind !== "string" || !validTimestamp(value.at)) return false;
+  const mutation = value as Record<string, unknown>;
+  if (value.kind === "clear") return optionalNonNegativeInteger(mutation.timeUsedSeconds);
+  if (typeof mutation.goalId !== "string" || mutation.goalId.length === 0) return false;
+  switch (value.kind) {
+    case "create":
+    case "replace":
+      return typeof mutation.objective === "string" && optionalPositiveInteger(mutation.tokenBudget);
+    case "edit":
+      return typeof mutation.objective === "string" && (mutation.status === undefined || isGoalStatus(mutation.status));
+    case "status":
+      return isGoalStatus(mutation.status)
+        && optionalNonNegativeInteger(mutation.timeUsedSeconds)
+        && (mutation.activeStartedAt === undefined || validTimestamp(mutation.activeStartedAt));
+    case "budget":
+      return optionalPositiveInteger(mutation.tokenBudget);
+    case "account":
+      return nonNegativeInteger(mutation.tokens)
+        && Array.isArray(mutation.entryIds)
+        && mutation.entryIds.every((id) => typeof id === "string");
+    default:
+      return false;
+  }
+}
+
+function isGoalStatus(value: unknown): value is GoalStatus {
+  return typeof value === "string" && ["active", "paused", "blocked", "usage_limited", "budget_limited", "complete"].includes(value);
+}
+
+function validTimestamp(value: unknown): value is string {
+  return typeof value === "string" && Number.isFinite(Date.parse(value));
+}
+
+function nonNegativeInteger(value: unknown): value is number {
+  return typeof value === "number" && Number.isFinite(value) && Number.isInteger(value) && value >= 0;
+}
+
+function optionalNonNegativeInteger(value: unknown): boolean {
+  return value === undefined || nonNegativeInteger(value);
+}
+
+function optionalPositiveInteger(value: unknown): boolean {
+  return value === undefined
+    || (typeof value === "number" && Number.isFinite(value) && Number.isInteger(value) && value > 0);
 }
 
 export function statusMutation(goal: GoalState, status: GoalStatus, timeUsedSeconds: number, activeStartedAt?: string, meta?: GoalMutationMeta): GoalMutation {