@ian-pascoe/pi-advisor 0.6.0 → 0.7.0

package/src/index.ts

@@ -9,6 +9,7 @@ import { isDeepStrictEqual } from "node:util";
 import { Type } from "typebox";
 import { Value } from "typebox/value";
 import { discoverPiAgentSession } from "@ian-pascoe/pi-utils/pi-agent-session-discovery";
+import { expandEntryOnClick, expandMessageOnClick, noticeText } from "@ian-pascoe/pi-utils/ui";
 import {
   parseAdvisorOptions,
   readAdvisorLayers,
@@ -27,6 +28,7 @@ import {
   type AdvisorScopedOptions,
 } from "./advisor-menu.js";
 import {
+  advisorAskToolName as askToolName,
   advisorFooterText,
   advisorStatusHeadline,
   renderAdvisorAskCall,
@@ -48,7 +50,6 @@ interface WatchedChild {
   resourceInputs: AdvisorResourceInputs;
   observer: AdvisorObserver | undefined;
 }
-const askToolName = "advisor_ask";
 const askToolParameters = Type.Object(
   { message: Type.String({ minLength: 1 }) },
   { additionalProperties: false },
@@ -86,14 +87,23 @@ export default function advisor(pi: ExtensionAPI): void {
     attachChild,
   );
 
-  pi.registerEntryRenderer("pi-advisor-child", (entry, { expanded }, theme) =>
-    renderAdvisorChildEntry(entry.data, expanded, theme),
+  pi.registerEntryRenderer(
+    "pi-advisor-child",
+    expandEntryOnClick((entry, { expanded }, theme) =>
+      renderAdvisorChildEntry(entry.data, expanded, theme),
+    ),
   );
-  pi.registerEntryRenderer("pi-advisor-status", (entry, { expanded }, theme) =>
-    renderAdvisorStatus(entry.data, expanded, theme),
+  pi.registerEntryRenderer(
+    "pi-advisor-status",
+    expandEntryOnClick((entry, { expanded }, theme) =>
+      renderAdvisorStatus(entry.data, expanded, theme),
+    ),
   );
-  pi.registerMessageRenderer("pi-advisor", (message, options, theme) =>
-    renderAdvisorIntervention(message.details, options, theme),
+  pi.registerMessageRenderer(
+    "pi-advisor",
+    expandMessageOnClick((message, options, theme) =>
+      renderAdvisorIntervention(message.details, options, theme),
+    ),
   );
 
   /** Show the current Advisor state in the open settings menu and the footer. */
@@ -127,12 +137,14 @@ export default function advisor(pi: ExtensionAPI): void {
           openWorldHint: false,
         },
         executionMode: "sequential",
-        renderCall: (args, theme, context) => renderAdvisorAskCall(args, context.expanded, theme),
+        renderCall: (args, theme, context) =>
+          renderAdvisorAskCall(args, context.expanded, theme, context),
         renderResult: (result, options, theme, context) =>
           renderAdvisorAskResult(
             result.content.map((item) => (item.type === "text" ? item.text : "")).join(""),
             { expanded: options.expanded, isPartial: options.isPartial, isError: context.isError },
             theme,
+            context,
           ),
         execute: async (_id, { message }, signal) => {
           try {
@@ -254,7 +266,7 @@ export default function advisor(pi: ExtensionAPI): void {
             state: "paused",
             error: message,
           });
-          warn(`Advisor for ${child.agentId} paused: ${message}`);
+          warn(noticeText("Advisor", `${child.agentId} paused: ${message}`));
         },
         onStateChange: publishState,
         onIntervention: (finding) => {
@@ -339,7 +351,7 @@ export default function advisor(pi: ExtensionAPI): void {
           {
             onError: (message) => {
               pi.appendEntry("pi-advisor-status", { state: "paused", error: message });
-              warn(`Advisor paused: ${message}`);
+              warn(noticeText("Advisor", `paused: ${message}`));
             },
             onStateChange: publishState,
           },
@@ -399,7 +411,7 @@ export default function advisor(pi: ExtensionAPI): void {
       error = cause instanceof Error ? cause.message : String(cause);
       pi.appendEntry("pi-advisor-status", { state: "paused", error, usage: null, cost: null });
     }
-    if (error) ctx.ui.notify(`Advisor: ${error}`, "error");
+    if (error) ctx.ui.notify(noticeText("Advisor", error), "error");
   }
 
   /** Persist one validated change at its scope and reconfigure; undefined if superseded. */