@oh-my-pi/pi-coding-agent 16.2.3 → 16.2.5

package/dist/types/cli/update-cli.d.ts

@@ -27,6 +27,21 @@ interface UpdateMethodResolutionOptions {
     miseDataDir?: string;
 }
 export declare function resolveUpdateMethodForTest(ompPath: string, bunBinDir: string | undefined, options?: UpdateMethodResolutionOptions): UpdateMethod;
+interface BunInstallCachePruneResult {
+    scannedPackages: number;
+    removedEntries: number;
+}
+/**
+ * Prune Bun's package cache so each package keeps only its newest cached version.
+ *
+ * Bun stores package cache entries as both a package marker directory
+ * (`react/19.2.6@@@1`) and a materialized package directory
+ * (`react@19.2.6@@@1`). Global `omp` updates can leave one full copy per
+ * release. The marker and materialized entries are removed together so the
+ * cache stays internally consistent.
+ */
+export declare function pruneBunInstallCache(cacheDir: string, packageNames?: Set<string>): Promise<BunInstallCachePruneResult>;
+export declare function resolveBunGlobalNodeModulesDirFromLocations(globalBinDir: string | undefined, cacheDir: string | undefined): string | undefined;
 /**
  * Best-effort removal of binary-update backups left by earlier runs.
  *

package/dist/types/collab/replication-shrink.d.ts

@@ -0,0 +1,39 @@
+/**
+ * Hard-cap helper for host→guest collab frames.
+ *
+ * The host wraps every {@link CollabFrame} in an AES-GCM envelope and ships it
+ * through the relay's WebSocket. WebSocket servers enforce a per-frame
+ * `maxPayloadLength` (Bun's default is 16 MB; many proxies cap lower). A
+ * single oversized payload — typically a `read`/`bash`/`search` tool result
+ * captured as one multi-megabyte string, or a tool result whose `content`
+ * array holds thousands of small blocks — would otherwise ship as its own
+ * oversized frame and trip that limit, killing the host's WebSocket with
+ * `1006 Received too big message`. `CollabSocket` treats 1006 as transient
+ * and reconnects, the next guest hello triggers the same oversized send, and
+ * the loop never breaks (issue #3739).
+ *
+ * This helper bounds any JSON-serializable payload below
+ * {@link MAX_REPLICATED_PAYLOAD_BYTES}. Already-small payloads pass through
+ * untouched; oversized ones are returned as a deep-cloned shadow where long
+ * strings are head-truncated AND long arrays are head-clipped, with
+ * `[…N chars elided for collab session]` / `[…N items elided for collab
+ * session]` markers. Both axes are needed: string truncation alone leaves
+ * the cap unenforced for a payload built of many short strings, where no
+ * field exceeds the per-string floor.
+ */
+/**
+ * Per-payload ceiling for host→guest frames. Bun's default WebSocket
+ * `maxPayloadLength` is 16 MB; we leave a generous margin so the AES-GCM
+ * envelope (+ IV + tag), the 4-byte peer header, and the outer wire wrapper
+ * fit comfortably under that on every reasonable relay.
+ */
+export declare const MAX_REPLICATED_PAYLOAD_BYTES: number;
+/**
+ * Return `value` unchanged when its JSON serialization already fits
+ * {@link MAX_REPLICATED_PAYLOAD_BYTES}; otherwise return a deep-cloned
+ * shadow shrunk along both string and array axes until the payload fits.
+ * The function is generic over `T` because the wire shape is preserved:
+ * only string leaves and array tails change; discriminator fields, ids, and
+ * other small metadata pass through untouched.
+ */
+export declare function shrinkForReplication<T>(value: T): T;

package/dist/types/config/provider-globals.d.ts

@@ -0,0 +1,7 @@
+interface ProviderGlobalSettings {
+    get(path: "providers.webSearchExclude"): unknown;
+    get(path: "providers.webSearch"): unknown;
+    get(path: "providers.image"): unknown;
+}
+export declare function applyProviderGlobalsFromSettings(settings: ProviderGlobalSettings): void;
+export {};

package/dist/types/memories/index.d.ts

@@ -14,10 +14,28 @@ export declare function startMemoryStartupTask(options: {
     agentDir: string;
     taskDepth: number;
 }): void;
+interface MemoryInstructionSession {
+    sessionManager: Pick<AgentSession["sessionManager"], "getSessionFile">;
+}
+/**
+ * Drop the per-session memory instruction snapshot after explicit memory state
+ * changes that must affect the active conversation immediately, such as
+ * `/memory clear`.
+ */
+export declare function clearMemoryToolDeveloperInstructionsCache(session: MemoryInstructionSession | undefined): void;
+/**
+ * Refresh the active session's consolidated-memory snapshot after startup maintenance.
+ *
+ * Startup may finish after the first prompt build and write `memory_summary.md`;
+ * the active session should see that summary. It must not reread `learned.md`,
+ * because a `learn` call racing with startup belongs to the next session's
+ * memory prompt, not the active prompt-cache prefix.
+ */
+export declare function refreshMemoryToolDeveloperInstructionsCacheAfterStartup(session: MemoryInstructionSession, agentDir: string, settings: Settings): Promise<void>;
 /**
  * Build memory usage instructions for prompt injection.
  */
-export declare function buildMemoryToolDeveloperInstructions(agentDir: string, settings: Settings): Promise<string | undefined>;
+export declare function buildMemoryToolDeveloperInstructions(agentDir: string, settings: Settings, session?: MemoryInstructionSession): Promise<string | undefined>;
 /**
  * Clear all persisted memory state and generated artifacts.
  */
@@ -33,3 +51,4 @@ export declare function getMemoryRoot(agentDir: string, cwd: string): string;
  * tool when `memory.backend` is `local`.
  */
 export declare function saveLearnedLesson(agentDir: string, cwd: string, input: MemoryBackendSaveInput): Promise<MemoryBackendSaveResult>;
+export {};

package/dist/types/modes/components/status-line/component.d.ts

@@ -14,6 +14,11 @@ export declare class StatusLineComponent implements Component {
     getEffectiveSettingsForTest(): EffectiveStatusLineSettings;
     setAutoCompactEnabled(enabled: boolean): void;
     setSubagentCount(count: number): void;
+    /**
+     * Compatibility shim for callers predating the simplified subagent badge.
+     * The status line now intentionally shows only the active count.
+     */
+    setSubagentHubHint(_hint: string | undefined): void;
     /** Active subagent count as currently displayed (collab state mirroring). */
     get subagentCount(): number;
     /**

package/dist/types/modes/components/status-line/types.d.ts

@@ -103,6 +103,16 @@ export interface SegmentContext {
             url: string;
         } | null;
     };
+    /**
+     * Set when the path cwd is a *linked* git worktree, naming the shared
+     * primary checkout (the project). Lets the path segment collapse the
+     * base-prefixed `<base>/<project>/<worktree>` path to the project name —
+     * the worktree/branch is already shown by the git segment.
+     */
+    worktree: {
+        projectName: string;
+        worktreeName: string;
+    } | null;
     usage: {
         tier?: string;
         fiveHour?: {

package/dist/types/modes/controllers/command-controller.d.ts

@@ -30,13 +30,13 @@ export declare class CommandController {
     handleDropCommand(): Promise<void>;
     handleForkCommand(): Promise<void>;
     /**
-     * `/move` — switch to a fresh empty session in a different directory.
+     * `/move` — relocate the current session to a different directory.
      *
      * With no `targetPath` (TUI only), opens an autocomplete overlay so the user
      * can pick or type a directory. With a `targetPath`, resolves it directly.
      * If the target directory does not exist, the user is asked whether to create
-     * it. A brand-new empty session is then started in the target directory and
-     * the current session is left behind (resumable via `/resume`).
+     * it. The active session file and artifacts are moved into the target
+     * directory's session bucket so `/resume` from that directory can find it.
      */
     handleMoveCommand(targetPath?: string): Promise<void>;
     handleRenameCommand(title: string): Promise<void>;

package/dist/types/modes/interactive-mode.d.ts

@@ -83,6 +83,7 @@ export declare class InteractiveMode implements InteractiveModeContext {
     chatContainer: TranscriptContainer;
     pendingMessagesContainer: Container;
     statusContainer: Container;
+    todoReminderContainer: Container;
     todoContainer: Container;
     subagentContainer: Container;
     btwContainer: Container;

package/dist/types/modes/theme/theme.d.ts

@@ -6,7 +6,7 @@ export type SymbolPreset = "unicode" | "nerd" | "ascii";
 /**
  * All available symbol keys organized by category.
  */
-export type SymbolKey = "status.success" | "status.error" | "status.warning" | "status.info" | "status.pending" | "status.disabled" | "status.enabled" | "status.running" | "status.shadowed" | "status.aborted" | "status.done" | "nav.cursor" | "nav.selected" | "nav.expand" | "nav.collapse" | "nav.back" | "tree.branch" | "tree.last" | "tree.vertical" | "tree.horizontal" | "tree.hook" | "boxRound.topLeft" | "boxRound.topRight" | "boxRound.bottomLeft" | "boxRound.bottomRight" | "boxRound.horizontal" | "boxRound.vertical" | "boxSharp.topLeft" | "boxSharp.topRight" | "boxSharp.bottomLeft" | "boxSharp.bottomRight" | "boxSharp.horizontal" | "boxSharp.vertical" | "boxSharp.cross" | "boxSharp.teeDown" | "boxSharp.teeUp" | "boxSharp.teeRight" | "boxSharp.teeLeft" | "sep.powerline" | "sep.powerlineThin" | "sep.powerlineLeft" | "sep.powerlineRight" | "sep.powerlineThinLeft" | "sep.powerlineThinRight" | "sep.block" | "sep.space" | "sep.asciiLeft" | "sep.asciiRight" | "sep.dot" | "sep.slash" | "sep.pipe" | "icon.model" | "icon.plan" | "icon.goal" | "icon.pause" | "icon.loop" | "icon.folder" | "icon.search" | "icon.scratchFolder" | "icon.file" | "icon.git" | "icon.branch" | "icon.pr" | "icon.tokens" | "icon.context" | "icon.cost" | "icon.time" | "icon.pi" | "icon.ghost" | "icon.agents" | "icon.job" | "icon.cache" | "icon.cacheMiss" | "icon.input" | "icon.output" | "icon.host" | "icon.session" | "icon.package" | "icon.warning" | "icon.rewind" | "icon.auto" | "icon.fast" | "icon.extensionSkill" | "icon.extensionTool" | "icon.extensionSlashCommand" | "icon.extensionMcp" | "icon.extensionRule" | "icon.extensionHook" | "icon.extensionPrompt" | "icon.extensionContextFile" | "icon.extensionInstruction" | "icon.mic" | "icon.camera" | "thinking.minimal" | "thinking.low" | "thinking.medium" | "thinking.high" | "thinking.xhigh" | "thinking.autoPending" | "checkbox.checked" | "checkbox.unchecked" | "radio.selected" | "radio.unselected" | "format.bullet" | "format.dash" | "format.bracketLeft" | "format.bracketRight" | "md.quoteBorder" | "md.hrChar" | "md.bullet" | "md.colorSwatch" | "lang.default" | "lang.typescript" | "lang.javascript" | "lang.python" | "lang.rust" | "lang.go" | "lang.java" | "lang.c" | "lang.cpp" | "lang.csharp" | "lang.ruby" | "lang.julia" | "lang.php" | "lang.swift" | "lang.kotlin" | "lang.shell" | "lang.html" | "lang.css" | "lang.json" | "lang.yaml" | "lang.markdown" | "lang.sql" | "lang.docker" | "lang.lua" | "lang.text" | "lang.env" | "lang.toml" | "lang.xml" | "lang.ini" | "lang.conf" | "lang.log" | "lang.csv" | "lang.tsv" | "lang.image" | "lang.pdf" | "lang.archive" | "lang.binary" | "tab.appearance" | "tab.model" | "tab.interaction" | "tab.context" | "tab.files" | "tab.shell" | "tab.tools" | "tab.memory" | "tab.tasks" | "tab.providers" | "tool.write" | "tool.edit" | "tool.bash" | "tool.ssh" | "tool.lsp" | "tool.gh" | "tool.webSearch" | "tool.exa" | "tool.browser" | "tool.eval" | "tool.debug" | "tool.mcp" | "tool.job" | "tool.task" | "tool.todo" | "tool.memory" | "tool.ask" | "tool.resolve" | "tool.review" | "tool.inspectImage" | "tool.goal" | "tool.irc" | "tool.delete" | "tool.move";
+export type SymbolKey = "status.success" | "status.error" | "status.warning" | "status.info" | "status.pending" | "status.disabled" | "status.enabled" | "status.running" | "status.shadowed" | "status.aborted" | "status.done" | "nav.cursor" | "nav.selected" | "nav.expand" | "nav.collapse" | "nav.back" | "tree.branch" | "tree.last" | "tree.vertical" | "tree.horizontal" | "tree.hook" | "boxRound.topLeft" | "boxRound.topRight" | "boxRound.bottomLeft" | "boxRound.bottomRight" | "boxRound.horizontal" | "boxRound.vertical" | "boxSharp.topLeft" | "boxSharp.topRight" | "boxSharp.bottomLeft" | "boxSharp.bottomRight" | "boxSharp.horizontal" | "boxSharp.vertical" | "boxSharp.cross" | "boxSharp.teeDown" | "boxSharp.teeUp" | "boxSharp.teeRight" | "boxSharp.teeLeft" | "sep.powerline" | "sep.powerlineThin" | "sep.powerlineLeft" | "sep.powerlineRight" | "sep.powerlineThinLeft" | "sep.powerlineThinRight" | "sep.block" | "sep.space" | "sep.asciiLeft" | "sep.asciiRight" | "sep.dot" | "sep.slash" | "sep.pipe" | "icon.model" | "icon.plan" | "icon.goal" | "icon.pause" | "icon.loop" | "icon.folder" | "icon.worktree" | "icon.search" | "icon.scratchFolder" | "icon.file" | "icon.git" | "icon.branch" | "icon.pr" | "icon.tokens" | "icon.context" | "icon.cost" | "icon.time" | "icon.pi" | "icon.ghost" | "icon.agents" | "icon.job" | "icon.cache" | "icon.cacheMiss" | "icon.input" | "icon.output" | "icon.host" | "icon.session" | "icon.package" | "icon.warning" | "icon.rewind" | "icon.auto" | "icon.fast" | "icon.extensionSkill" | "icon.extensionTool" | "icon.extensionSlashCommand" | "icon.extensionMcp" | "icon.extensionRule" | "icon.extensionHook" | "icon.extensionPrompt" | "icon.extensionContextFile" | "icon.extensionInstruction" | "icon.mic" | "icon.camera" | "thinking.minimal" | "thinking.low" | "thinking.medium" | "thinking.high" | "thinking.xhigh" | "thinking.autoPending" | "checkbox.checked" | "checkbox.unchecked" | "radio.selected" | "radio.unselected" | "format.bullet" | "format.dash" | "format.bracketLeft" | "format.bracketRight" | "md.quoteBorder" | "md.hrChar" | "md.bullet" | "md.colorSwatch" | "lang.default" | "lang.typescript" | "lang.javascript" | "lang.python" | "lang.rust" | "lang.go" | "lang.java" | "lang.c" | "lang.cpp" | "lang.csharp" | "lang.ruby" | "lang.julia" | "lang.php" | "lang.swift" | "lang.kotlin" | "lang.shell" | "lang.html" | "lang.css" | "lang.json" | "lang.yaml" | "lang.markdown" | "lang.sql" | "lang.docker" | "lang.lua" | "lang.text" | "lang.env" | "lang.toml" | "lang.xml" | "lang.ini" | "lang.conf" | "lang.log" | "lang.csv" | "lang.tsv" | "lang.image" | "lang.pdf" | "lang.archive" | "lang.binary" | "tab.appearance" | "tab.model" | "tab.interaction" | "tab.context" | "tab.files" | "tab.shell" | "tab.tools" | "tab.memory" | "tab.tasks" | "tab.providers" | "tool.write" | "tool.edit" | "tool.bash" | "tool.ssh" | "tool.lsp" | "tool.gh" | "tool.webSearch" | "tool.exa" | "tool.browser" | "tool.eval" | "tool.debug" | "tool.mcp" | "tool.job" | "tool.task" | "tool.todo" | "tool.memory" | "tool.ask" | "tool.resolve" | "tool.review" | "tool.inspectImage" | "tool.goal" | "tool.irc" | "tool.delete" | "tool.move";
 export type SpinnerType = "status" | "activity";
 export type ThemeColor = "accent" | "border" | "borderAccent" | "borderMuted" | "success" | "error" | "warning" | "muted" | "dim" | "text" | "thinkingText" | "userMessageText" | "customMessageText" | "customMessageLabel" | "toolTitle" | "toolOutput" | "mdHeading" | "mdLink" | "mdLinkUrl" | "mdCode" | "mdCodeBlock" | "mdCodeBlockBorder" | "mdQuote" | "mdQuoteBorder" | "mdHr" | "mdListBullet" | "toolDiffAdded" | "toolDiffRemoved" | "toolDiffContext" | "syntaxComment" | "syntaxKeyword" | "syntaxFunction" | "syntaxVariable" | "syntaxString" | "syntaxNumber" | "syntaxType" | "syntaxOperator" | "syntaxPunctuation" | "thinkingOff" | "thinkingMinimal" | "thinkingLow" | "thinkingMedium" | "thinkingHigh" | "thinkingXhigh" | "bashMode" | "pythonMode" | "statusLineSep" | "statusLineModel" | "statusLinePath" | "statusLineGitClean" | "statusLineGitDirty" | "statusLineContext" | "statusLineSpend" | "statusLineStaged" | "statusLineDirty" | "statusLineUntracked" | "statusLineOutput" | "statusLineCost" | "statusLineSubagents";
 /** Check if a string is a valid ThemeColor value */
@@ -164,6 +164,7 @@ export declare class Theme {
         pause: string;
         loop: string;
         folder: string;
+        worktree: string;
         scratchFolder: string;
         file: string;
         git: string;

package/dist/types/modes/types.d.ts

@@ -81,6 +81,7 @@ export interface InteractiveModeContext {
     chatContainer: TranscriptContainer;
     pendingMessagesContainer: Container;
     statusContainer: Container;
+    todoReminderContainer: Container;
     todoContainer: Container;
     subagentContainer: Container;
     btwContainer: Container;
@@ -104,7 +105,7 @@ export interface InteractiveModeContext {
     focusParentSession(): Promise<void>;
     /** Return the view to the main session (delegates to SessionFocusController.unfocus). */
     unfocusSession(): Promise<void>;
-    /** Clear loader, status/pending containers, streaming state, and pending tools. */
+    /** Clear loader, transient HUD/pending containers, streaming state, and pending tools. */
     clearTransientSessionUi(): void;
     settings: Settings;
     keybindings: KeybindingsManager;

package/dist/types/session/agent-session.d.ts

@@ -169,6 +169,13 @@ export interface AgentSessionConfig {
      * inherits this so its requests undergo the same shaping as the main turn.
      */
     transformProviderContext?: (context: Context, model: Model) => Context | Promise<Context>;
+    /**
+     * Stream wrapper passed to side-channel requests (`/btw`, `/omfg`, IRC
+     * auto-replies, and handoff generation) so they apply the same provider
+     * shaping and host-level request wrappers as normal agent turns. Defaults
+     * to plain `streamSimple` when omitted.
+     */
+    sideStreamFn?: StreamFn;
     /**
      * Stream wrapper passed to the advisor agent so its requests apply the
      * session's `providers.openrouterVariant`, `providers.antigravityEndpoint`,

package/dist/types/session/session-manager.d.ts

@@ -242,9 +242,8 @@ export declare class SessionManager {
     /**
      * Create a fresh empty session file in the default session directory for
      * `cwd`, writing only the session header. The returned path can be passed to
-     * `setSessionFile` / `AgentSession.switchSession` to start a new empty
-     * session in that directory. Used by `/move` to switch projects without
-     * dragging the current conversation along.
+     * `setSessionFile` / `AgentSession.switchSession` when a caller explicitly
+     * needs a brand-new persisted session at a cwd-derived path.
      */
     static createEmptySessionFile(cwd: string, storage?: SessionStorage): string;
     /**

package/dist/types/task/provider-concurrency.d.ts

@@ -0,0 +1,40 @@
+/**
+ * Per-provider LLM concurrency cap, applied around each provider HTTP request.
+ *
+ * The semaphore brackets only the streaming request itself, not the whole
+ * agent lifetime: a parent subagent releases its slot the moment its LLM
+ * stream finishes producing, so children spawned during tool execution can
+ * acquire slots for their own turns. Holding the slot across the parent's
+ * full conversation deadlocks any spawn tree whose width exceeds
+ * `maxConcurrency` because the parents wait for children that wait for
+ * slots the parents are holding (issue
+ * [#3749](https://github.com/can1357/oh-my-pi/issues/3749)).
+ */
+import type { StreamFn } from "@oh-my-pi/pi-agent-core";
+import type { Settings } from "../config/settings";
+import { Semaphore } from "./parallel";
+/**
+ * Resolve the configured concurrency ceiling for a provider, or `undefined`
+ * when the provider has no cap concept at all. A configured value `<= 0` means
+ * "unlimited" and maps to `Infinity` — still a tracked ceiling, so every run
+ * holds a slot and a later finite resize counts work started while unlimited.
+ */
+export declare function getProviderConcurrencyLimit(settings: Settings, provider: string): number | undefined;
+/**
+ * Hand out the single shared limiter for `provider` (creating one lazily) and
+ * resize it in place when the configured limit changes. Replacing the
+ * semaphore would orphan in-flight slots on the old instance and let a
+ * runtime or mixed limit value exceed the cap (issue #3464 review feedback).
+ */
+export declare function getProviderSemaphore(settings: Settings, provider: string): Semaphore | undefined;
+/**
+ * Wrap a {@link StreamFn} so every LLM HTTP request acquires the provider's
+ * concurrency slot before the request goes out and releases it when the
+ * stream finishes producing (success, error, or abort). Providers without a
+ * configured cap pass straight through.
+ *
+ * The acquire bracket is intentionally narrow (one slot per LLM call), so
+ * spawn trees deeper than `maxConcurrency` no longer deadlock on themselves —
+ * see the module-level comment for the failure mode this fixes.
+ */
+export declare function wrapStreamFnWithProviderConcurrency(settings: Settings, base: StreamFn): StreamFn;

package/dist/types/utils/git.d.ts

@@ -349,6 +349,17 @@ export declare const repo: {
      * the shared common dir (`foo.git`) because they have no primary checkout.
      */
     primaryRootSync(cwd: string): string | null;
+    /**
+     * Linked-worktree metadata for `cwd`, or `null` when `cwd` is the primary
+     * checkout (or outside a repository). `root` is the worktree's own checkout
+     * root; `primaryRoot` is the shared main checkout that names the project.
+     * Resolves purely via on-disk `.git`/`commondir` walking — no subprocess —
+     * so the status line may call it on every render.
+     */
+    linkedWorktreeSync(cwd: string): {
+        root: string;
+        primaryRoot: string;
+    } | null;
     /** Full GitRepository metadata (sync). */
     resolveSync(cwd: string): GitRepository | null;
     /** Full GitRepository metadata. */

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "16.2.3",
+	"version": "16.2.5",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -55,17 +55,17 @@
 		"@agentclientprotocol/sdk": "0.25.0",
 		"@babel/parser": "^7.29.7",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/hashline": "16.2.3",
-		"@oh-my-pi/omp-stats": "16.2.3",
-		"@oh-my-pi/pi-agent-core": "16.2.3",
-		"@oh-my-pi/pi-ai": "16.2.3",
-		"@oh-my-pi/pi-catalog": "16.2.3",
-		"@oh-my-pi/pi-mnemopi": "16.2.3",
-		"@oh-my-pi/pi-natives": "16.2.3",
-		"@oh-my-pi/pi-tui": "16.2.3",
-		"@oh-my-pi/pi-utils": "16.2.3",
-		"@oh-my-pi/pi-wire": "16.2.3",
-		"@oh-my-pi/snapcompact": "16.2.3",
+		"@oh-my-pi/hashline": "16.2.5",
+		"@oh-my-pi/omp-stats": "16.2.5",
+		"@oh-my-pi/pi-agent-core": "16.2.5",
+		"@oh-my-pi/pi-ai": "16.2.5",
+		"@oh-my-pi/pi-catalog": "16.2.5",
+		"@oh-my-pi/pi-mnemopi": "16.2.5",
+		"@oh-my-pi/pi-natives": "16.2.5",
+		"@oh-my-pi/pi-tui": "16.2.5",
+		"@oh-my-pi/pi-utils": "16.2.5",
+		"@oh-my-pi/pi-wire": "16.2.5",
+		"@oh-my-pi/snapcompact": "16.2.5",
 		"@opentelemetry/api": "^1.9.1",
 		"@opentelemetry/context-async-hooks": "^2.7.1",
 		"@opentelemetry/exporter-trace-otlp-proto": "^0.218.0",

package/src/autolearn/controller.ts

@@ -2,9 +2,9 @@
  * Auto-learn session controller (experimental).
  *
  * Subscribes to the session event stream and, after a substantive turn,
- * nudges the agent to capture reusable lessons. Default posture is passive
- * (a hidden reminder rides the next real turn); with `autolearn.autoContinue`
- * it auto-runs exactly one synthetic capture turn at stop.
+ * optionally auto-runs a synthetic capture turn. Passive mode is intentionally
+ * prompt-cache neutral: the standing system guidance remains available, but no
+ * hidden mid-session reminder is inserted into the conversation.
  *
  * Installed once per top-level session (taskDepth 0). The subscription lives
  * for the session's lifetime — `newSession` resets the session in place
@@ -14,11 +14,9 @@ import { logger } from "@oh-my-pi/pi-utils";
 import type { Settings } from "../config/settings";
 import autolearnGuidance from "../prompts/system/autolearn-guidance.md" with { type: "text" };
 import autolearnGuidanceLearn from "../prompts/system/autolearn-guidance-learn.md" with { type: "text" };
-import autolearnNudge from "../prompts/system/autolearn-nudge.md" with { type: "text" };
 import autolearnNudgeAutoContinue from "../prompts/system/autolearn-nudge-autocontinue.md" with { type: "text" };
 import type { AgentSession, AgentSessionEvent } from "../session/agent-session";
 
-const AUTOLEARN_NUDGE_PASSIVE = autolearnNudge.trim();
 const AUTOLEARN_NUDGE_AUTOCONTINUE = autolearnNudgeAutoContinue.trim();
 const DEFAULT_MIN_TOOL_CALLS = 5;
 
@@ -110,28 +108,21 @@ export class AutoLearnController {
 		// auto-continue would compete with it.
 		if (startedInGoalMode || this.#session.getGoalModeState()?.enabled) return;
 
-		// Auto-run a capture turn only when explicitly enabled; otherwise the
-		// hidden reminder rides the next real turn passively.
-		//
-		// The two paths get DIFFERENT nudge text:
-		// - Auto-continue spawns a synthetic turn with the nudge as its only
-		//   user-role payload, so the prompt must be terminal — it tells the
-		//   agent to capture and STOP, and must not be read as the user's
-		//   reply to any pending question. Without that contract, the agent
-		//   conflates the synthetic prompt with user approval and resumes
-		//   prior work (e.g. commits/pushes an unanswered "want me to push?"
-		//   question — #3504).
-		// - Passive mode appends the nudge to the user's real next message,
-		//   so the agent must answer the user normally and treat the capture
-		//   as additive — not as the whole turn's job.
+		// Auto-run a capture turn only when explicitly enabled. Passive mode used to
+		// queue a hidden custom message for the next real turn, but that mutates the
+		// persisted conversation prefix after providers have cached it. The standing
+		// auto-learn system guidance is stable; keep passive mode to that guidance
+		// so Anthropic prompt-cache prefixes survive long sessions.
 		const autoContinue = this.#settings.get("autolearn.autoContinue") === true;
-		const content = autoContinue ? AUTOLEARN_NUDGE_AUTOCONTINUE : AUTOLEARN_NUDGE_PASSIVE;
+		if (!autoContinue) return;
+
+		const content = AUTOLEARN_NUDGE_AUTOCONTINUE;
 		// Arm suppression synchronously: the synthetic capture turn's agent_end
 		// fires inside sendCustomMessage (before it resolves), so the flag must be
 		// set before then. Disarm when no turn actually started — a deferred/queued
 		// dispatch or a failed send produces no agent_end, and a latched flag would
 		// otherwise swallow the next real stop.
-		if (autoContinue) this.#suppressNext = true;
+		this.#suppressNext = true;
 
 		this.#session
 			.sendCustomMessage(
@@ -141,7 +132,7 @@ export class AutoLearnController {
 					display: false,
 					attribution: "user",
 				},
-				{ deliverAs: "nextTurn", triggerTurn: autoContinue },
+				{ deliverAs: "nextTurn", triggerTurn: true },
 			)
 			.then(started => {
 				if (!started) this.#suppressNext = false;