pi-advisor-flow 0.8.3 → 0.9.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-advisor-flow",
-  "version": "0.8.3",
+  "version": "0.9.1",
   "description": "Advanced Executor/Advisor flow for Pi, fully configurable and extendable.",
   "keywords": [
     "advisor",
@@ -51,26 +51,26 @@
     "format:check": "oxfmt --check"
   },
   "devDependencies": {
-    "@earendil-works/pi-ai": "0.87.1",
-    "@earendil-works/pi-coding-agent": "0.87.1",
-    "@earendil-works/pi-tui": "0.87.1",
-    "@oxlint/plugins": "1.85.0",
+    "@earendil-works/pi-ai": "0.99.1",
+    "@earendil-works/pi-coding-agent": "0.99.1",
+    "@earendil-works/pi-tui": "0.99.1",
+    "@oxlint/plugins": "1.86.0",
     "@types/node": "^26.6.3",
     "@typesafe-ai/sdk": "^0.6.0",
     "bun-types": "1.4.2",
     "husky": "^9.1.7",
-    "knip": "^6.38.0",
+    "knip": "6.39.0",
     "lint-staged": "^17.6.0",
-    "oxfmt": "0.70.0",
-    "oxlint": "1.85.0",
+    "oxfmt": "0.71.0",
+    "oxlint": "1.86.0",
     "typebox": "^1.3.34",
     "typescript": "^7.0.2",
-    "ultracite": "7.12.0"
+    "ultracite": "7.12.2"
   },
   "peerDependencies": {
-    "@earendil-works/pi-ai": "^0.87.1",
-    "@earendil-works/pi-coding-agent": "^0.87.1",
-    "@earendil-works/pi-tui": "^0.87.1",
+    "@earendil-works/pi-ai": "^0.87.1 || ^0.99.1",
+    "@earendil-works/pi-coding-agent": "^0.87.1 || ^0.99.1",
+    "@earendil-works/pi-tui": "^0.87.1 || ^0.99.1",
     "typebox": "^1.1.38"
   },
   "peerDependenciesMeta": {
@@ -92,10 +92,10 @@
     "safe-buffer": "npm:@socketregistry/safe-buffer@1.0.9"
   },
   "overrides": {
-    "brace-expansion": "5.0.9",
+    "brace-expansion": "5.0.12",
     "is-unicode-supported": "npm:@socketregistry/is-unicode-supported@1.0.5",
     "safe-buffer": "npm:@socketregistry/safe-buffer@1.0.9",
-    "undici": "8.9.0"
+    "undici": "8.11.2"
   },
   "lint-staged": {
     "*.{ts,tsx}": [

package/src/attachments.ts

@@ -14,16 +14,28 @@ export interface UntrackedAttachment {
   text: string;
 }
 
+const imageExtensions = new Map([
+  [".png", "image/png"],
+  [".jpg", "image/jpeg"],
+  [".jpeg", "image/jpeg"],
+  [".gif", "image/gif"],
+  [".webp", "image/webp"],
+]);
+export const imageMime = (path: string) =>
+  imageExtensions.get(path.slice(path.lastIndexOf(".")).toLowerCase());
+export const imageFileCandidate = (path: string) =>
+  Boolean(imageMime(path)) || /\.(?:avif|bmp|heic|svg|tiff?)$/iu.test(path);
+
 const PATH_SEGMENTS = /[\\/]/u;
 
-const within = (root: string, candidate: string) => {
+export const within = (root: string, candidate: string) => {
   const path = relative(root, candidate);
   return path !== "" && !path.startsWith("..") && !path.includes("../");
 };
 const normalizeRelativePath = (root: string, path: string) =>
   relative(root, resolve(root, path));
 
-const normalizeRequestedPath = (root: string, value: string) => {
+export const normalizeRequestedPath = (root: string, value: string) => {
   if (
     !isString(value) ||
     !value ||
@@ -46,7 +58,7 @@ const git = (cwd: string, args: string[]) =>
     windowsHide: true,
   });
 
-const repositoryRoot = (cwd: string) => {
+export const repositoryRoot = (cwd: string) => {
   try {
     return realpath(git(cwd, ["rev-parse", "--show-toplevel"]).trim());
   } catch {
@@ -86,7 +98,7 @@ const tracked = (cwd: string, path: string) => {
   });
 };
 
-const isPermitted = (
+export const isPermitted = (
   root: string,
   path: string,
   kind: "tracked" | "untracked"
@@ -158,7 +170,11 @@ const readFiles = async (
   let total = 0;
   for (const name of requested) {
     const normalizedName = normalizeRequestedPath(root, name);
-    if (!normalizedName || unique.has(normalizedName)) {
+    if (
+      !normalizedName ||
+      unique.has(normalizedName) ||
+      imageFileCandidate(normalizedName)
+    ) {
       continue;
     }
     unique.add(normalizedName);

package/src/commands/activation.ts

@@ -19,6 +19,7 @@ import {
   setExecutorRef,
 } from "../config/state.ts";
 import { saveConfig } from "../config/storage.ts";
+import { sameModelAdvisorDisabled } from "../tools/model-access.ts";
 import {
   loadCommandConfig,
   prepareActivationModels,
@@ -132,12 +133,16 @@ export const activateAdvisor = async (
       "record_advisor_outcome",
     ]);
   }
+  const activeModel = isMarkedSubagent()
+    ? ctx.model
+    : findConfiguredModel(ctx, executorRef);
+  runtime.updateSameModelNotice(ctx, activeModel);
   if (announce) {
     const activeExecutorRef = effectiveExecutorRef(ctx);
     const activeExecutorEffort = effectiveExecutorEffort(ctx);
     notify(
       ctx,
-      `${ADVISOR_ACTIVATION_EXPLANATION}\n\nAdvisor flow ready — Executor: ${activeExecutorRef} (thinking: ${activeExecutorEffort || "default"}) · Advisor: ${advisorRef} (thinking: ${advisorEffortRef || "default"})`,
+      `${ADVISOR_ACTIVATION_EXPLANATION}\n\nAdvisor flow ${sameModelAdvisorDisabled(ctx, activeModel) ? "configured" : "ready"} — Executor: ${activeExecutorRef} (thinking: ${activeExecutorEffort || "default"}) · Advisor: ${advisorRef} (thinking: ${advisorEffortRef || "default"})`,
       "info"
     );
   }

package/src/commands/lifecycle.ts

@@ -23,12 +23,15 @@ export const registerCommandLifecycle = (
 ) => {
   runtime.pi.on("session_start", async (_event, ctx) => {
     runtime.pendingExecutorModelRef = undefined;
+    runtime.resetSameModelNotice();
     // A malformed advisor.json or a provider auth failure must not reject a
     // lifecycle handler and break session startup.
     try {
       loadConfig(ctx);
       if (alwaysOnRef) {
         await activateAdvisor("", ctx, false);
+      } else {
+        runtime.updateSameModelNotice(ctx);
       }
     } catch (error) {
       const message = error instanceof Error ? error.message : String(error);
@@ -37,6 +40,9 @@ export const registerCommandLifecycle = (
   });
 
   runtime.pi.on("model_select", (event, ctx) => {
+    if (!runtime.suppressModelSelectionSync) {
+      runtime.updateSameModelNotice(ctx, event.model);
+    }
     // "restore" replays a stored session model and "cycle" changes the active
     // model without an explicit `/model` choice. Neither should redefine the
     // configured Executor.

package/src/commands/manual-command.ts

@@ -4,7 +4,11 @@ import {
   isSimpleMode,
 } from "../config/state.ts";
 import type { GitContextLevel } from "../git.ts";
-import { advisorModelAccessReason } from "../tools/model-access.ts";
+import {
+  advisorModelAccessReason,
+  sameModelAdvisorDisabled,
+  sameModelAdvisorNotice,
+} from "../tools/model-access.ts";
 import { resolveAdvisorRequest } from "../tools/render-common.ts";
 import { ManualAdvisorDialog } from "../ui/manual-dialog.ts";
 import type { ManualAdvisorRequest } from "../ui/types.ts";
@@ -21,6 +25,10 @@ export const registerManualCommand = (runtime: CommandRuntime) => {
       if (!loadCommandConfig(ctx)) {
         return;
       }
+      if (sameModelAdvisorDisabled(ctx)) {
+        notify(ctx, sameModelAdvisorNotice, "info");
+        return;
+      }
       const accessReason = advisorModelAccessReason(ctx);
       if (accessReason) {
         notify(ctx, accessReason, "warning");
@@ -82,6 +90,10 @@ export const registerManualCommand = (runtime: CommandRuntime) => {
         question = resolveAdvisorRequest(args);
       }
 
+      if (sameModelAdvisorDisabled(ctx)) {
+        notify(ctx, sameModelAdvisorNotice, "info");
+        return;
+      }
       if (!isSimpleMode()) {
         runtime.advisorSessionState.consumeCall();
       }
@@ -100,18 +112,38 @@ export const registerManualCommand = (runtime: CommandRuntime) => {
       runtime.scoutStatus.register(scoutStatusToken);
       runtime.manualConsultations.set(controller, scoutStatusToken);
       runtime.pi.appendEntry?.("advisor-manual-call", { progressId, question });
+      // Print mode tears the session down when the handler returns, so the
+      // consultation must complete inside the handler to keep its ctx valid.
+      if (ctx.mode === "print") {
+        await startManualConsultation(
+          runtime,
+          ctx,
+          question,
+          controller,
+          scoutStatusToken,
+          progress,
+          gitContext
+        );
+        return;
+      }
       // Intentional fire-and-forget: the consultation streams after the command
-      // handler returns. void satisfies noFloatingPromises; noVoid is ignored here
-      // because this Biome version offers no ignoreVoidAsExpression option.
-      void startManualConsultation(
-        runtime,
-        ctx,
-        question,
-        controller,
-        scoutStatusToken,
-        progress,
-        gitContext
-      );
+      // handler returns. A rejected consultation must not escape as an
+      // unhandled rejection; its error paths already notify and record.
+      void (async () => {
+        try {
+          await startManualConsultation(
+            runtime,
+            ctx,
+            question,
+            controller,
+            scoutStatusToken,
+            progress,
+            gitContext
+          );
+        } catch {
+          // Consultation error paths already notify and record; nothing to add.
+        }
+      })();
     },
   });
 };

package/src/commands/manual-consultation.ts

@@ -7,6 +7,7 @@ import { notifyHerdrAdvisorFailure } from "../herdr.ts";
 import { normalizeScreeningQuestion } from "../tools/jev-filter.ts";
 import { appendScoutLifecycleEntry } from "../tools/scout-status.ts";
 import type { ScoutToolDetails } from "../tools/types.ts";
+import { uiAvailable } from "../ui-guard.ts";
 import { advisorUsageCost, snapshotAdvisorUsage } from "../usage.ts";
 import type { AdvisorUsageSnapshot } from "../usage.ts";
 import { notify } from "./runtime.ts";
@@ -28,9 +29,10 @@ export const startManualConsultation = async (
   progress: ManualAdvisorProgressState,
   gitContext?: GitContextLevel
 ) => {
+  const isPrintMode = ctx.mode === "print";
   progress.phase = "preparing";
   runtime.requestManualRender(ctx);
-  if (ctx.hasUI) {
+  if (uiAvailable(ctx)) {
     const timer = setInterval(() => {
       if (
         controller.signal.aborted ||
@@ -118,8 +120,10 @@ export const startManualConsultation = async (
       {
         // Steer lets the current turn finish its active work; the Executor sees
         // the result before its next model call rather than being interrupted.
+        // Print mode has no follow-up turn; triggering one trips pi's turn_end
+        // entry resolution during teardown.
         deliverAs: "steer",
-        triggerTurn: true,
+        triggerTurn: !isPrintMode,
       }
     );
   } catch (error) {
@@ -147,7 +151,7 @@ export const startManualConsultation = async (
         },
         display: true,
       },
-      { deliverAs: "steer", triggerTurn: true }
+      { deliverAs: "steer", triggerTurn: !isPrintMode }
     );
     notify(ctx, `Advisor consultation failed: ${message}`, "error");
     notifyHerdrAdvisorFailure("Advisor consultation failed", message);

package/src/commands/model-commands.ts

@@ -57,6 +57,7 @@ export const registerModelCommands = (runtime: CommandRuntime) => {
         persistExecutor: true,
       });
       runtime.pendingExecutorModelRef = undefined;
+      runtime.updateSameModelNotice(ctx);
       ctx.ui.notify(
         `Saved Executor + Advisor configurations to ${path}`,
         "info"

package/src/commands/runtime.ts

@@ -7,8 +7,13 @@ import { getAdvisorSettings } from "../config/state.ts";
 import { herdrAdvisorActivity, notifyHerdrAdvisorFailure } from "../herdr.ts";
 import type { AdvisorSessionState } from "../session-state.ts";
 import { consultAdvisor } from "../tools/consultation.ts";
+import {
+  sameModelAdvisorDisabled,
+  sameModelAdvisorNotice,
+} from "../tools/model-access.ts";
 import { ScoutStatusManager } from "../tools/scout-status.ts";
 import { advisorSessionState as defaultAdvisorSessionState } from "../tools/session.ts";
+import { uiAction } from "../ui-guard.ts";
 import type {
   CommandDependencies,
   CommandRuntime as CommandRuntimeContract,
@@ -21,9 +26,7 @@ export const notify = (
   message: string,
   level: "error" | "info" | "warning"
 ) => {
-  if (ctx.hasUI) {
-    ctx.ui.notify(message, level);
-  }
+  uiAction(ctx, (ui) => ui.notify(message, level));
 };
 
 const reportManualBudgetExhausted = (ctx: ExtensionContext) => {
@@ -33,9 +36,7 @@ const reportManualBudgetExhausted = (ctx: ExtensionContext) => {
 };
 
 const requestManualRender = (ctx: ExtensionContext) => {
-  if (ctx.hasUI) {
-    ctx.ui.setStatus("advisor-manual", undefined);
-  }
+  uiAction(ctx, (ui) => ui.setStatus("advisor-manual", undefined));
 };
 
 class CommandRuntime implements CommandRuntimeContract {
@@ -55,6 +56,7 @@ class CommandRuntime implements CommandRuntimeContract {
   manualProgressSequence = 0;
   pendingExecutorModelRef: string | undefined;
   suppressModelSelectionSync = false;
+  private lastSameModelDisabled: boolean | undefined;
 
   constructor(pi: ExtensionAPI, dependencies: CommandDependencies = {}) {
     this.pi = pi;
@@ -84,6 +86,30 @@ class CommandRuntime implements CommandRuntimeContract {
     return this.pi.getActiveTools().includes("ask_advisor");
   }
 
+  resetSameModelNotice() {
+    this.lastSameModelDisabled = undefined;
+  }
+
+  updateSameModelNotice(
+    ctx: ExtensionContext,
+    model: { id: string; provider: string } | undefined = ctx.model
+  ) {
+    if (!this.flowEnabled()) {
+      return;
+    }
+    const disabled = sameModelAdvisorDisabled(ctx, model);
+    if (disabled && this.lastSameModelDisabled !== true) {
+      notify(ctx, sameModelAdvisorNotice, "info");
+    } else if (!disabled && this.lastSameModelDisabled === true) {
+      notify(
+        ctx,
+        "Advisor re-enabled: executor and advisor models differ.",
+        "info"
+      );
+    }
+    this.lastSameModelDisabled = disabled;
+  }
+
   nextManualProgressId() {
     this.manualProgressSequence += 1;
     return `manual-${this.manualProgressSequence}`;
@@ -101,14 +127,14 @@ class CommandRuntime implements CommandRuntimeContract {
   }
 
   updateAdvisorUsageStatus(ctx: ExtensionContext) {
-    if (ctx.hasUI) {
-      ctx.ui.setStatus(
+    uiAction(ctx, (ui) =>
+      ui.setStatus(
         "advisor-usage",
         getAdvisorSettings().showUsageFooter
           ? this.advisorSessionState.usageStatus()
           : undefined
-      );
-    }
+      )
+    );
   }
 }
 

package/src/commands/settings-commands.ts

@@ -36,6 +36,7 @@ export const registerSettingsCommands = (runtime: CommandRuntime) => {
             onChange: (settings) => {
               try {
                 saveAdvisorSettings(ctx, settings);
+                runtime.updateSameModelNotice(ctx);
                 runtime.updateAdvisorUsageStatus(ctx);
               } catch (error) {
                 const message =
@@ -65,6 +66,7 @@ export const registerSettingsCommands = (runtime: CommandRuntime) => {
               name !== "ask_advisor" && name !== "record_advisor_outcome"
           )
       );
+      runtime.resetSameModelNotice();
       // Leaving alwaysOn set would silently reactivate the flow next session.
       const wasAlwaysOn = alwaysOnRef;
       if (wasAlwaysOn) {

package/src/commands/settings-persistence.ts

@@ -7,6 +7,7 @@ import {
   setAdvisorCollapseResponsesRef,
   setAdvisorCompletionGateRef,
   setAdvisorCustomInvocationRef,
+  setAdvisorDisableSameModelRef,
   setAdvisorEffortRef,
   setAdvisorFailureGateRef,
   setAdvisorFailureModeRef,
@@ -68,6 +69,7 @@ const applySessionSettings = (settings: AdvisorSettings) => {
   setAdvisorPlanGateRef(settings.planGate);
   setAdvisorFailureGateRef(settings.failureGate);
   setAdvisorCompletionGateRef(settings.completionGate);
+  setAdvisorDisableSameModelRef(settings.disableSameModel ?? true);
   setAdvisorCollapseResponsesRef(settings.collapseResponses);
   setAdvisorCustomInvocationRef(settings.customRule);
   setAdvisorBlockOnBlockedRef(settings.blockOnBlocked ?? true);

package/src/commands/types.ts

@@ -71,5 +71,10 @@ export interface CommandRuntime {
     model: Parameters<ExtensionAPI["setModel"]>[0]
   ) => ReturnType<ExtensionAPI["setModel"]>;
   suppressModelSelectionSync: boolean;
+  resetSameModelNotice: () => void;
+  updateSameModelNotice: (
+    ctx: ExtensionContext,
+    model?: { id: string; provider: string }
+  ) => void;
   updateAdvisorUsageStatus: (ctx: ExtensionContext) => void;
 }

package/src/config/defaults.ts

@@ -4,6 +4,7 @@ import {
   setAdvisorCollapseResponsesRef,
   setAdvisorCompletionGateRef,
   setAdvisorCustomInvocationRef,
+  setAdvisorDisableSameModelRef,
   setAdvisorEffortRef,
   setAdvisorFailureGateRef,
   setAdvisorFailureModeRef,
@@ -75,6 +76,7 @@ export const resetDefaults = () => {
   setAdvisorFailureGateRef(true);
   setAdvisorCompletionGateRef(true);
   setAdvisorCustomInvocationRef(undefined);
+  setAdvisorDisableSameModelRef(true);
   setAdvisorCollapseResponsesRef(false);
   setAdvisorBlockOnBlockedRef(true);
   setAdvisorAutoLoopGateRef(true);
@@ -150,6 +152,11 @@ export const applyConfig = (config: AdvisorConfig) => {
     "advisorCustomInvocation",
     setAdvisorCustomInvocationRef
   );
+  applyOptionalConfig(
+    config,
+    "advisorDisableSameModel",
+    setAdvisorDisableSameModelRef
+  );
   applyOptionalConfig(
     config,
     "advisorCollapseResponses",

package/src/config/schema.ts

@@ -7,6 +7,7 @@ import {
   advisorCollapseResponsesRef,
   advisorCompletionGateRef,
   advisorCustomInvocationRef,
+  advisorDisableSameModelRef,
   advisorEffortRef,
   advisorFailureGateRef,
   advisorFailureModeRef,
@@ -200,6 +201,12 @@ export const CONFIG_SCHEMA = {
     persisted: true,
     type: "string",
   },
+  advisorDisableSameModel: {
+    accepted: "true or false",
+    current: () => advisorDisableSameModelRef,
+    persisted: true,
+    type: "boolean",
+  },
   advisorEffort: {
     accepted: "a string",
     current: () => advisorEffortRef,

package/src/config/state.ts

@@ -34,6 +34,7 @@ export let advisorPlanGateRef = true;
 export let advisorFailureGateRef = true;
 export let advisorCompletionGateRef = true;
 export let advisorCustomInvocationRef: string | undefined;
+export let advisorDisableSameModelRef = true;
 export let advisorCollapseResponsesRef = false;
 export let advisorBlockOnBlockedRef = true;
 export let advisorAutoLoopGateRef = true;
@@ -115,6 +116,9 @@ export const setAdvisorCompletionGateRef = (enabled: boolean) => {
 export const setAdvisorCustomInvocationRef = (rule: string | undefined) => {
   advisorCustomInvocationRef = rule?.trim() || undefined;
 };
+export const setAdvisorDisableSameModelRef = (enabled: boolean) => {
+  advisorDisableSameModelRef = enabled;
+};
 export const setAdvisorCollapseResponsesRef = (enabled: boolean) => {
   advisorCollapseResponsesRef = enabled;
 };
@@ -237,6 +241,7 @@ export const getAdvisorSettings = () => ({
   completionGate: advisorCompletionGateRef,
   contextMaxChars: contextMaxCharsRef,
   customRule: advisorCustomInvocationRef,
+  disableSameModel: advisorDisableSameModelRef,
   effort: advisorEffortRef,
   failureGate: advisorFailureGateRef,
   failureMode: advisorFailureModeRef,

package/src/config/types.ts

@@ -65,6 +65,7 @@ export interface AdvisorConfig {
   advisorCollapseResponses?: boolean;
   advisorCompletionGate?: boolean;
   advisorCustomInvocation?: string;
+  advisorDisableSameModel?: boolean;
   advisorEffort?: string;
   advisorFailureGate?: boolean;
   advisorGitContext?: GitContextLevel;

package/src/conversation.ts

@@ -9,6 +9,7 @@ import {
 import type { AdvisorToolPolicies } from "./config/types.ts";
 import { contentParts, isRecordOf, isString } from "./content-utils.ts";
 import type { RecordValue } from "./content-utils.ts";
+import { imageMarker } from "./images.ts";
 import { redactSecrets } from "./redaction.ts";
 import { capToolResult } from "./tool-result-cap.ts";
 
@@ -25,6 +26,11 @@ const textFromPart = <Part>(part: Part): string => {
 export const textFrom = <Content>(content: Content): string =>
   contentParts(content).map(textFromPart).join("\n").trim();
 
+const imageNotes = (content: RecordValue["content"], nonce: string) =>
+  contentParts(content)
+    .map((part) => (isRecordOf(part) ? imageMarker(part, nonce) : undefined))
+    .filter(isString);
+
 const assistantEntry = (
   message: RecordValue,
   policies: AdvisorToolPolicies,
@@ -64,12 +70,17 @@ const toolResultEntry = (
   toolResultMaxLines: number,
   toolResultMaxBytes: number,
   policies: AdvisorToolPolicies,
-  redact: boolean
+  redact: boolean,
+  describeImages: boolean,
+  imageNonce: string
 ): string => {
   const status = message.isError ? "error" : "success";
   const toolName = isString(message.toolName) ? message.toolName : "unknown";
   const policy = policies[toolName] ?? "full";
-  const source = textFrom(message.content);
+  const notes = describeImages ? imageNotes(message.content, imageNonce) : [];
+  const source = [textFrom(message.content), ...notes]
+    .filter(Boolean)
+    .join("\n");
   if (policy === "exclude") {
     return `[Tool Result for ${toolName}] (excluded by Advisor tool policy)`;
   }
@@ -95,7 +106,9 @@ export const conversationEntry = <Entry>(
   toolResultMaxLines: number,
   toolResultMaxBytes: number,
   policies: AdvisorToolPolicies,
-  redact: boolean
+  redact: boolean,
+  describeImages = false,
+  imageNonce = ""
 ): string | undefined => {
   if (!isRecordOf(entry)) {
     return;
@@ -108,7 +121,10 @@ export const conversationEntry = <Entry>(
   }
   const { message } = entry;
   if (message.role === "user") {
-    const text = textFrom(message.content);
+    const notes = describeImages ? imageNotes(message.content, imageNonce) : [];
+    const text = [textFrom(message.content), ...notes]
+      .filter(Boolean)
+      .join("\n");
     return text ? `User: ${redact ? redactSecrets(text) : text}` : undefined;
   }
   if (message.role === "assistant") {
@@ -120,7 +136,9 @@ export const conversationEntry = <Entry>(
       toolResultMaxLines,
       toolResultMaxBytes,
       policies,
-      redact
+      redact,
+      describeImages,
+      imageNonce
     );
   }
 };
@@ -179,7 +197,9 @@ export const recentConversation = (
   toolResultMaxLines = advisorToolResultMaxLinesRef,
   toolResultMaxBytes = advisorToolResultMaxBytesRef,
   policies = advisorToolPoliciesRef,
-  redact = advisorRedactSecretsRef
+  redact = advisorRedactSecretsRef,
+  describeImages = false,
+  imageNonce = ""
 ): string => {
   if (maxChars === 0) {
     return "";
@@ -192,7 +212,9 @@ export const recentConversation = (
         toolResultMaxLines,
         toolResultMaxBytes,
         policies,
-        redact
+        redact,
+        describeImages,
+        imageNonce
       )
     )
     .filter((entry): entry is string => entry !== undefined);