@ian-pascoe/pi-advisor 0.3.0 → 0.3.1

package/README.md

@@ -2,7 +2,7 @@
 
 `@ian-pascoe/pi-advisor` reviews a Pi agent's completed work and surfaces concise, attributed findings for material problems and worthwhile low-risk cleanup.
 
-Requires Pi `0.85.1` and Node `>=22.19.0`. Other Pi versions pause review with an explicit compatibility error.
+Requires Pi `0.87.1` and Node `>=22.19.0`. Other Pi versions pause review with an explicit compatibility error.
 
 ## Install
 
@@ -60,7 +60,7 @@ The Advisor inherits each observed agent's model and thinking level independentl
 
 ## Sessions and Context Management
 
-Each watched agent has a private, persisted native Advisor Session with fresh, session-bound extension resources. Reviews reuse that session until context or configuration changes require rebuilding. Pi 0.85.1's hidden inline `llama.cpp` built-in is recreated from the installed Pi package's shipped extension file, retaining its ordered metadata and fresh provider state. This is a fixed-version compatibility recipe, not general inline-factory support; it requires the shipped file and the built-in's expected registration shape. Other opaque inline or custom extension resources that cannot be safely recreated are reported as unsupported and pause the Advisor; they are not silently reused or omitted. The Advisor's session state remains separate from the observed agent's state.
+Each watched agent has a private, persisted native Advisor Session with fresh, session-bound extension resources. Reviews reuse that session until context or configuration changes require rebuilding. Pi 0.87.1's hidden inline `llama.cpp` built-in is recreated from the installed Pi package's shipped extension file, retaining its ordered metadata and fresh provider state. This is a fixed-version compatibility recipe, not general inline-factory support; it requires the shipped file and the built-in's expected registration shape. Other opaque inline or custom extension resources that cannot be safely recreated are reported as unsupported and pause the Advisor; they are not silently reused or omitted. The Advisor's session state remains separate from the observed agent's state.
 
 Context Management is optional. If loaded and its tools are granted, `context_notes`, `context_history`, and `context_rollover` operate on the Advisor's private context. If any of those three tools is excluded, the Advisor pauses before reviewing. There is no autogrant or hook bypass.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ian-pascoe/pi-advisor",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "private": false,
   "description": "Background review and attributed corrective advice for Pi agent sessions.",
   "keywords": [
@@ -34,11 +34,6 @@
   "dependencies": {
     "@ian-pascoe/pi-utils": "^0.2.0"
   },
-  "devDependencies": {
-    "@earendil-works/pi-agent-core": "0.85.1",
-    "@earendil-works/pi-ai": "0.85.1",
-    "@earendil-works/pi-coding-agent": "0.85.1"
-  },
   "peerDependencies": {
     "@earendil-works/pi-agent-core": "*",
     "@earendil-works/pi-ai": "*",

package/src/advisor-observer.ts

@@ -1,5 +1,12 @@
 import { isDeepStrictEqual } from "node:util";
-import { contentText, type Context, type ImageContent } from "@earendil-works/pi-ai";
+import {
+  contentText,
+  getCurrentSystemPrompt,
+  getCurrentTools,
+  toToolDeclaration,
+  type Context,
+  type ImageContent,
+} from "@earendil-works/pi-ai";
 import {
   defineTool,
   convertToLlm,
@@ -58,7 +65,7 @@ const pendingQueuesSchema = Type.Object({
   _pendingCustomMessages: Type.Array(Type.Unknown()),
 });
 
-// oxlint-disable-next-line anti-slop/no-unknown-parameters -- SAFETY: Pi 0.85.1 has no selective queue-removal API. Validate its native queue data and remove only exact owned finding identities, never unrelated messages or journal entries.
+// oxlint-disable-next-line anti-slop/no-unknown-parameters -- SAFETY: Pi 0.87.1 has no selective queue-removal API. Validate its native queue data and remove only exact owned finding identities, never unrelated messages or journal entries.
 function retractFindings(session: unknown, findings: ReadonlySet<Finding>): void {
   if (!findings.size) return;
   if (!Value.Check(pendingQueuesSchema, session))
@@ -208,14 +215,15 @@ export class AdvisorObserver {
       if (!this.closed && this.config.enabled) {
         try {
           if (this.active && !this.sameObservation(this.active)) this.reset();
-          // Native Context tools also carry execute callbacks. Copy definitions, never callbacks.
+          // Pi carries the prompt and tool deltas as system messages. Replay them into the
+          // current state and copy tool declarations, never execute callbacks.
           this.snapshot = structuredClone({
-            ...context,
-            tools: (context.tools ?? []).map(({ name, description, parameters }) => ({
-              name,
-              description,
-              parameters,
-            })),
+            systemPrompt: getCurrentSystemPrompt(context.messages),
+            tools: getCurrentTools(context.messages).map((tool) => {
+              const { name, description, parameters } = toToolDeclaration(tool);
+              return { name, description, parameters };
+            }),
+            messages: context.messages.filter((message) => message.role !== "system"),
           });
         } catch {
           this.fail("Advisor cannot capture this model context safely");

package/src/advisor-session.ts

@@ -72,7 +72,7 @@ export async function disposeAdvisorSession(runtime: AgentSessionRuntime): Promi
   }
 }
 
-// Pi 0.85.1 keeps these plain constructor inputs private; public scoped getters omit applyOverrides.
+// Pi 0.87.1 keeps these plain constructor inputs private; public scoped getters omit applyOverrides.
 /* oxlint-disable anti-slop/no-unknown-parameters -- SAFETY: Capability-check native SDK data before copying; offline SDK creation tests cover these inputs. */
 function recreationInputs(
   loader: unknown,
@@ -111,7 +111,7 @@ function recreationInputs(
   if (
     Value.Check(fileAuthSchema, store) &&
     store.authPath === store.storage.authPath &&
-    // Pi 0.85.1's bundled CLI names the same native class _AuthStorage.
+    // Pi 0.87.1's bundled CLI names the same native class _AuthStorage.
     ["AuthStorage", "_AuthStorage"].includes(Object.getPrototypeOf(store)?.constructor.name) &&
     Object.getPrototypeOf(store.storage)?.constructor.name === "FileAuthStorageBackend"
   )
@@ -136,9 +136,9 @@ export async function createAdvisorSession(
   observed: AgentSession,
   options: AdvisorSessionOptions,
 ): Promise<AgentSessionRuntime> {
-  if (VERSION !== "0.85.1")
+  if (VERSION !== "0.87.1")
     throw new Error(
-      `Unsupported Pi ${VERSION}: Advisor's native tool ceiling and session lifecycle are verified on Pi 0.85.1`,
+      `Unsupported Pi ${VERSION}: Advisor's native tool ceiling and session lifecycle are verified on Pi 0.87.1`,
     );
   options.signal?.throwIfAborted();
   const cancelled = Promise.withResolvers<never>();
@@ -203,7 +203,7 @@ async function buildAdvisorSession(
   const extensionPaths = source.getExtensions().extensions.map((extension) => {
     if (extension.resolvedPath && !extension.resolvedPath.startsWith("<"))
       return extension.resolvedPath;
-    // Pi 0.85.1 injects this built-in inline, but ships a fresh file-backed factory.
+    // Pi 0.87.1 injects this built-in inline, but ships a fresh file-backed factory.
     // Match its registration shape, not arbitrary inline closures or hidden extensions.
     if (
       extension.path === "<inline:llama.cpp>" &&

package/src/advisor-settings.ts

@@ -171,7 +171,7 @@ const storageSchema = Type.Object({
   ),
 });
 
-/** Use Pi's actual backend: its generic storage API is not exported in 0.85.1. */
+/** Use Pi's actual backend: its generic storage API is not exported in 0.87.1. */
 export async function writeAdvisorSettings(
   manager: SettingsManager,
   scope: "global" | "project",