@astrosheep/pi-goal-next 0.1.0 → 0.1.2

package/docs/architecture.md

@@ -61,7 +61,7 @@ User messages win: `hasPendingMessages()` check + CAS conflict as backstop.
 1. `session_start`: any restored `active` goal → commit a system transition to `paused` (the journal does not contain a separate `loaded` entry). Never silently resume; the continuation generation is not explicitly invalidated by this hook.
 2. `session_before_tree`: tell continuation to void generation; next event triggers rebuild from `getBranch()` via goal-commit.
 3. `session_before_compact`: append `goal.summarize()` text to the compaction if the hook allows (verify at implementation; continuation messages are self-contained regardless).
-4. `agent_settled`: commit the accounting verdict, then — if the current goal is `budget_limited` and this goal instance has not been steered yet — send `budgetLimitPrompt(goal)` with `triggerTurn: true`. Steering is sent once per goal instance (`steeredGoalId`/`steered` closure state, reset on a null snapshot or a new goal id). Then run `continuation.onSettled()`.
+4. `agent_settled`: commit the accounting verdict, then — if the current goal is `budget_limited` and this goal instance has not been steered yet — send `budgetLimitPrompt(goal)` with `triggerTurn: true`. Steering is sent once per goal instance (`steeredGoalId`/`steered` closure state, reset on a null snapshot or a new goal id). Then run `continuation.onSettled()`, except that it skips continuation when the previous turn ended error/aborted (structured `stopReason`; such turns produced no completed work).
 
 No other business. Retry needs no lifecycle handling — accounting dedupes by message id; retry messages have new ids and are honestly counted (they cost real tokens).
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@astrosheep/pi-goal-next",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "Persistent autonomous goals for pi, with Codex-verbatim goal semantics: continuation prompts, self-audited completion, budgets, and CAS-journaled state.",
   "type": "module",
   "files": [

package/src/accounting.ts

@@ -7,7 +7,7 @@ export type Usage = {
   cacheWrite?: number | null;
   [key: string]: unknown;
 };
-export type Message = { entryId: string; role: "assistant" | "toolResult"; usage?: Usage | null; toolName?: string };
+export type Message = { entryId: string; role: "assistant" | "toolResult"; usage?: Usage | null; toolName?: string; stopReason?: string };
 export type Delta = { input: number; output: number; cacheRead: number; cacheWrite: number; unknownMessages: number; stale: boolean };
 export type Verdict = { kind: "ok" } | { kind: "budget_limited"; reason: string };
 

package/src/continuation.ts

@@ -43,6 +43,8 @@ export function createContinuation(deps: ContinuationDeps): Continuation {
     const seq = goal.continuationSeq + 1;
     const result = await deps.commit({ type: "continuation_sent", generation: leaseGeneration }, revision);
     if (result.kind !== "ok" || generation !== leaseGeneration) return;
+    const latest = deps.getSnapshot();
+    if (!latest || latest.goal.id !== goal.id || latest.goal.status !== "active") return; // user may have cleared/paused during the commit await
 
     deps.send({
       customType: "pi-goal-next/continuation",

package/src/lifecycle.ts

@@ -31,7 +31,9 @@ function messageFromEvent(event: any, ctx: any): Message | null {
   const m = candidate?.message ?? candidate ?? message;
   const entryId = m?.id ?? m?.entryId ?? message.id ?? message.entryId;
   // Entry identity is heuristic-by-position because branch entries may omit message ids.
-  return typeof entryId === "string" ? { entryId, role: message.role, usage: m?.usage ?? message.usage, toolName: m?.toolName ?? message.toolName } : null;
+  return typeof entryId === "string"
+    ? { entryId, role: message.role, usage: m?.usage ?? message.usage, toolName: m?.toolName ?? message.toolName, stopReason: m?.stopReason ?? message.stopReason }
+    : null;
 }
 
 export type PiEvents = { on(name: string, handler: (event: any, ctx: any) => unknown): void };
@@ -49,6 +51,8 @@ export function registerLifecycle(pi: PiEvents, deps: LifecycleDeps): void {
   // Budget-limit steering is sent once per goal instance; reset on null branch or a new goal id.
   let steeredGoalId: string | null = null;
   let steered = false;
+  // Stop reason of the most recent assistant turn; gates continuation on normal completion.
+  let lastAssistantStop: string | undefined;
 
   pi.on("session_start", async (_event: SessionStartEvent, ctx: any) => {
     await bootstrap();
@@ -80,6 +84,7 @@ export function registerLifecycle(pi: PiEvents, deps: LifecycleDeps): void {
     await before(ctx);
     const message = messageFromEvent(event, ctx);
     if (!message) return;
+    if (message.role === "assistant") lastAssistantStop = message.stopReason;
     const recorded = deps.accounting.recordMessage(message);
     if (recorded && !recorded.duplicate) {
       const delta = recorded.delta;
@@ -103,6 +108,8 @@ export function registerLifecycle(pi: PiEvents, deps: LifecycleDeps): void {
       steered = true;
       deps.send({ customType: "pi-goal-next/budget_limit", content: budgetLimitPrompt(goal), display: false, details: { goalId: goal.id } }, { triggerTurn: true });
     }
+    // Error/aborted turns produced no completed work; only continue after a normally-finished turn (Codex parity).
+    if (lastAssistantStop === "error" || lastAssistantStop === "aborted") return;
     await deps.continuation.onSettled();
   });
   pi.on("message_start", async (event: MessageStartEvent, ctx: any) => { await before(ctx); await deps.continuation.onMessageStart(event?.message ?? event); });