@matthewfl/pi-contemplator 0.2.0 → 0.3.1

package/README.md

@@ -157,7 +157,7 @@ The observer and summarizer provide the durable memory substrate. The contemplat
 
 ## Installation
 
-`pi-contemplator` 0.2.0 requires **Pi 0.86.x**.
+`pi-contemplator` 0.3.0 requires **Pi 0.87.x**.
 
 Install the published package through Pi:
 
@@ -203,4 +203,4 @@ npm run typecheck
 
 ## License
 
-MIT. See [LICENSE](LICENSE).
+MIT. See [LICENSE](LICENSE).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@matthewfl/pi-contemplator",
-  "version": "0.2.0",
+  "version": "0.3.1",
   "description": "A Pi extension that keeps long-running agentic sessions on track with background memory, contemplation, and structural review.",
   "type": "module",
   "license": "MIT",
@@ -44,17 +44,17 @@
     "test:e2e": "node tests/e2e/rpc-contemplator.mjs && node tests/e2e/rpc-contemplator-history.mjs && node tests/e2e/rpc-summarizer.mjs && node tests/e2e/rpc-delivery.mjs && node tests/e2e/rpc-restore-review.mjs && node tests/e2e/rpc-compaction.mjs && node tests/e2e/rpc-compaction-resilience.mjs && node tests/e2e/rpc-memory-edges.mjs && node tests/e2e/rpc-observer-length.mjs && node tests/e2e/rpc-routing-isolation.mjs"
   },
   "peerDependencies": {
-    "@earendil-works/pi-agent-core": "^0.86.0",
-    "@earendil-works/pi-ai": "^0.86.0",
-    "@earendil-works/pi-coding-agent": "^0.86.0",
-    "@earendil-works/pi-tui": "^0.86.0"
+    "@earendil-works/pi-agent-core": "^0.87.0",
+    "@earendil-works/pi-ai": "^0.87.0",
+    "@earendil-works/pi-coding-agent": "^0.87.0",
+    "@earendil-works/pi-tui": "^0.87.0"
   },
   "devDependencies": {
-    "@earendil-works/pi-agent-core": "^0.86.0",
-    "@earendil-works/pi-ai": "^0.86.0",
-    "@earendil-works/pi-coding-agent": "^0.86.0",
-    "@earendil-works/pi-server": "^0.86.0",
-    "@earendil-works/pi-tui": "^0.86.0",
+    "@earendil-works/pi-agent-core": "^0.87.0",
+    "@earendil-works/pi-ai": "^0.87.0",
+    "@earendil-works/pi-coding-agent": "^0.87.0",
+    "@earendil-works/pi-server": "^0.87.0",
+    "@earendil-works/pi-tui": "^0.87.0",
     "@types/node": "^22.0.0",
     "typebox": "^1.1.38",
     "typescript": "^5.6.0",

package/src/agents/contemplator/agent.ts

@@ -885,7 +885,10 @@ export class Contemplator {
 				// A clean final-action call is the end of the contemplator turn. Do not
 				// spend another model request asking it to narrate after its decision.
 				// Citation warnings leave the loop open so it can correct the action.
-				shouldStopAfterTurn: () => intervention !== undefined && !finalActionWarned,
+				finishTurn: ({ message }) => {
+					if (message.stopReason === "error" || message.stopReason === "aborted") return undefined;
+					return intervention !== undefined && !finalActionWarned ? { action: "end" } : undefined;
+				},
 				...(supportsReasoning && selectedThinkingLevel !== "off" ? { reasoning: selectedThinkingLevel } : {}),
 			};
 			const runMessages: AgentMessage[] = [];

package/src/agents/observer/agent.ts

@@ -206,9 +206,12 @@ IMPORTANT: Now call record_observations to record the useful new observations fr
 		maxTokens: boundedMaxTokens(model, OBSERVER_AGENT_LOOP_MAX_TOKENS),
 		convertToLlm: replayTruncatedThinkingAsText,
 		toolExecution: "sequential",
-		shouldStopAfterTurn: () => {
+		finishTurn: ({ message }) => {
+			if (message.stopReason === "error" || message.stopReason === "aborted") return undefined;
 			turnCount++;
-			return doneCalled || (effectiveMaxTurns !== undefined && turnCount >= effectiveMaxTurns);
+			return doneCalled || (effectiveMaxTurns !== undefined && turnCount >= effectiveMaxTurns)
+				? { action: "end" }
+				: undefined;
 		},
 		...(reasoning && thinkingLevel !== "off" ? { reasoning: thinkingLevel } : {}),
 	};

package/src/agents/reviewer/agent.ts

@@ -165,7 +165,10 @@ export async function runStructuralReview(args: RunStructuralReviewArgs): Promis
 			maxTokens: boundedMaxTokens(args.model, remainingBudget()),
 			convertToLlm: (messages) => messages as Message[],
 			toolExecution: "sequential",
-			shouldStopAfterTurn: () => terminal !== undefined || remainingBudget() === 0,
+			finishTurn: ({ message }) => {
+				if (message.stopReason === "error" || message.stopReason === "aborted") return undefined;
+				return terminal !== undefined || remainingBudget() === 0 ? { action: "end" } : undefined;
+			},
 		};
 		const promptMessage = prompt as AgentMessage;
 		// agentLoop receives the new prompt separately. Its context must therefore
@@ -197,8 +200,8 @@ export async function runStructuralReview(args: RunStructuralReviewArgs): Promis
 		// assistant usage that the real stream wrapper did not already account for.
 		const reportedOutputTokens = assistantOutputTokens(assistants);
 		totalOutputTokens += Math.max(0, reportedOutputTokens - streamedOutputTokens);
-		// A warned terminal call deliberately leaves shouldStopAfterTurn false so
-		// the reviewer can search/recall and replace it. If the reviewer instead
+		// A warned terminal call deliberately leaves finishTurn undecided so the
+		// reviewer can search/recall and replace it. If the reviewer instead
 		// ends its turn, honor the staged outcome exactly as the warning promised.
 		if (!terminal && warnedTerminal) {
 			terminal = warnedTerminal;

package/src/agents/summarizer/agent.ts

@@ -546,7 +546,12 @@ export async function runSummarizer(args: RunSummarizerArgs): Promise<Summarizer
 				if (calls.length > 1) return { block: true, reason: "Call done alone in a later response after sibling tool results are visible." };
 				return undefined;
 			},
-			shouldStopAfterTurn: () => completedWithDone || (effectiveMaxTurns !== undefined && ++turnCount >= effectiveMaxTurns),
+			finishTurn: ({ message }) => {
+				if (message.stopReason === "error" || message.stopReason === "aborted") return undefined;
+				return completedWithDone || (effectiveMaxTurns !== undefined && ++turnCount >= effectiveMaxTurns)
+					? { action: "end" }
+					: undefined;
+			},
 			...(requireToolCall ? { onPayload: (payload: unknown) => forceRequiredToolPayload(payload, args.model.api) } : {}),
 			...(reasoning && thinkingLevel !== "off" ? { reasoning: thinkingLevel } : {}),
 		};

package/src/session-ledger/progress.ts

@@ -7,9 +7,16 @@ import {
 	type MemoryCoverageCustomType,
 } from "./types.js";
 
-const SOURCE_ENTRY_TYPES = new Set(["message", "custom_message", "branch_summary"]);
+const SOURCE_ENTRY_TYPES = new Set(["custom_message", "branch_summary"]);
 
 export function isSourceEntry(entry: Entry): boolean {
+	if (entry.type === "message") {
+		// Pi 0.87 persists its canonical system prompt as a message entry. It is
+		// extension/runtime configuration, not new conversation evidence; counting
+		// its large sections makes a fresh session immediately cross the observer
+		// threshold even though only a few hundred user-visible tokens exist.
+		return !isObject(entry.message) || entry.message.role !== "system";
+	}
 	return SOURCE_ENTRY_TYPES.has(entry.type);
 }