@oh-my-pi/pi-coding-agent 16.2.2 → 16.2.4

package/dist/types/session/session-title-slot.d.ts

@@ -0,0 +1,19 @@
+import { type SessionTitleSlotEntry, type SessionTitleSource } from "./session-entries";
+/** Semantic title update persisted by session storage backends. */
+export interface SessionTitleUpdate {
+    title?: string;
+    source?: SessionTitleSource;
+    updatedAt: string;
+}
+/** Parse a physical title slot JSONL line. Returns undefined for legacy headers. */
+export declare function parseTitleSlotLine(line: string): SessionTitleSlotEntry | undefined;
+/** Parse the fixed-width title slot from a physical session body. */
+export declare function parseTitleSlotFromContent(content: string): SessionTitleSlotEntry | undefined;
+/** Convert a parsed title slot to the semantic storage update shape. */
+export declare function titleUpdateFromSlot(slot: SessionTitleSlotEntry | undefined): SessionTitleUpdate | undefined;
+/** Serialize the fixed-width first-line title slot, exactly 256 UTF-8 bytes including newline. */
+export declare function serializeTitleSlot(options: SessionTitleUpdate): string;
+/** Replace the physical fixed-width title slot in a full session body. */
+export declare function overlayTitleSlotContent(content: string, update: SessionTitleUpdate): string;
+/** Replace the physical fixed-width title slot in a prefix byte window. */
+export declare function overlayTitleSlotPrefix(prefix: string, prefixBytes: number, update: SessionTitleUpdate): string;

package/dist/types/ssh/connection-manager.d.ts

@@ -14,6 +14,15 @@ export interface SSHHostInfo {
     version: number;
     os: SSHHostOs;
     shell: SSHHostShell;
+    /**
+     * Shell name OMP verified can execute the POSIX transfer snippets
+     * (`head`/`cat`/`mv`/`test`/`ls`) `ssh://` uses. Probed by running
+     * `sh -lc` / `bash -lc` / `zsh -lc` against the remote and keeping the
+     * first one that round-trips a known marker. Independent of `shell`
+     * (the self-reported login shell), which may be noisy, exotic, or simply
+     * mis-classified — only `transferShell` gates ssh:// transfers.
+     */
+    transferShell?: "sh" | "bash" | "zsh";
     compatShell?: "bash" | "sh";
     compatEnabled: boolean;
 }
@@ -22,6 +31,44 @@ interface SSHArgsOptions {
     /** When true, omit `-n` so the remote command can read from our piped stdin. */
     allowStdin?: boolean;
 }
+/**
+ * Parse a raw cache-file value (or any unknown) into a normalized
+ * {@link SSHHostInfo}, dropping fields that don't pass the per-field guards.
+ * Exported so cache-layer round-tripping (incl. the new `transferShell`
+ * field, #3719) is testable without touching disk.
+ */
+export declare function parseHostInfo(value: unknown): SSHHostInfo | null;
+/**
+ * Frame marker emitted by the remote OS/shell probe. The probe wraps its
+ * payload in this prefix so the parser can ignore startup-file noise (banners,
+ * `motd`, login messages, `Last login: …`) instead of trusting only the first
+ * line of stdout. See #3719.
+ */
+export declare const HOST_PROBE_MARKER = "PI_HOST_PROBE=";
+/** Marker for the transfer-shell capability probe. */
+export declare const TRANSFER_PROBE_MARKER = "PI_TRANSFER_OK|";
+/**
+ * Find the first line of `stdout`/`stderr` that begins with `marker` and
+ * return everything after it. Used by the SSH host probe so noisy login
+ * dotfiles can't corrupt OS/shell classification by emitting text on the
+ * first line of `ssh` output.
+ *
+ * Returns `null` when no marker line is found in either stream.
+ */
+export declare function extractProbePayload(stdout: string, stderr: string, marker?: string): string | null;
+/**
+ * Find `marker` anywhere in `stdout` or `stderr` and return everything that
+ * follows it, scanning stdout first. Returns `null` when the marker is in
+ * neither stream.
+ *
+ * Used by the transfer-shell capability probe. Some remotes have broken
+ * login dotfiles that swap fd 1/2, so the marker can land on stderr even
+ * though the probe ran the printf successfully (matches the host-info
+ * probe's stderr fallback). See #3719.
+ */
+export declare function findProbeMarker(stdout: string, stderr: string, marker: string): string | null;
+/** Classify a POSIX-ish `uname -s` payload from the transfer-shell probe. */
+export declare function osFromUname(value: string): SSHHostOs | undefined;
 export declare function getHostInfo(hostName: string): Promise<SSHHostInfo | undefined>;
 export declare function getHostInfoForHost(host: SSHConnectionTarget): Promise<SSHHostInfo | undefined>;
 /**

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

@@ -6,3 +6,19 @@ export declare function buildSshTarget(username: string | undefined, host: strin
  * `ssh://` file-transfer helpers.
  */
 export declare function quotePosixPath(value: string): string;
+/**
+ * Wrap a POSIX command in `<shell> -c '<command>'` so it runs under the
+ * named shell rather than whatever `$SHELL` happens to be on the remote.
+ *
+ * Used by the `ssh://` transfer helpers and the Windows compat dispatch:
+ * OpenSSH passes our snippets to `<login-shell> -c`, so a remote whose
+ * login shell is fish/csh/tcsh (or cmd/powershell on Windows compat)
+ * can't parse `if [ … ]; then …`. Wrapping forces parsing under the
+ * shell OMP actually verified can run the snippet.
+ *
+ * `-c` (not `-lc`): the transfer snippets only call absolute POSIX
+ * builtins (`head`/`cat`/`mv`/`test`/`ls`/`mkdir`/`rm`/`dirname`) and
+ * don't need login-profile setup. Capability *probing* still uses
+ * `-lc` to mirror the user's real environment.
+ */
+export declare function wrapInPosixShell(shell: "sh" | "bash" | "zsh", command: string): string;

package/dist/types/task/executor.d.ts

@@ -23,7 +23,8 @@ import type { AuthStorage } from "../session/auth-storage";
 import type { ContextFileEntry } from "../tools";
 import type { EventBus } from "../utils/event-bus";
 import type { WorkspaceTree } from "../workspace-tree";
-import { type AgentDefinition, type AgentProgress, type ReviewFinding, type SingleResult } from "./types";
+import { type AgentDefinition, type AgentProgress, type ReviewFinding, type SingleResult, type YieldItem } from "./types";
+export type { YieldItem } from "./types";
 /**
  * Soft per-agent request budgets (assistant requests per run). When a subagent
  * crosses its budget it receives ONE steering notice asking it to wrap up; at
@@ -164,20 +165,6 @@ export interface ExecutorOptions {
      */
     keepAlive?: boolean;
 }
-export interface YieldItem {
-    data?: unknown;
-    status?: "success" | "aborted";
-    error?: string;
-    /**
-     * Set by the in-tool yield validator when it exhausted its retry budget
-     * (MAX_SCHEMA_RETRIES) and accepted a schema-invalid payload anyway.
-     * `finalizeSubprocessOutput` honors this by serializing the payload and
-     * surfacing a stderr warning, instead of re-emitting `schema_violation`
-     * — which would silently swap the subagent's "accepted" view for a
-     * different, opaque error blob in the parent's view of the result.
-     */
-    schemaOverridden?: boolean;
-}
 interface FinalizeSubprocessOutputArgs {
     rawOutput: string;
     exitCode: number;
@@ -187,6 +174,7 @@ interface FinalizeSubprocessOutputArgs {
     yieldItems?: YieldItem[];
     reportFindings?: ReviewFinding[];
     outputSchema: unknown;
+    lastAssistantText?: string;
 }
 interface FinalizeSubprocessOutputResult {
     rawOutput: string;
@@ -217,4 +205,3 @@ export declare function finalizeSubagentLifecycle(args: {
  * Run a single agent in-process.
  */
 export declare function runSubprocess(options: ExecutorOptions): Promise<SingleResult>;
-export {};

package/dist/types/task/render.d.ts

@@ -31,9 +31,4 @@ export declare function renderResult(result: {
     details?: TaskToolDetails;
     isError?: boolean;
 }, options: TaskRenderOptions, theme: Theme, args?: TaskParams): Component;
-export declare const taskToolRenderer: {
-    renderCall: typeof renderCall;
-    renderResult: typeof renderResult;
-    mergeCallAndResult: boolean;
-};
 export {};

package/dist/types/task/renderer.d.ts

@@ -0,0 +1,13 @@
+/**
+ * Task tool renderer export.
+ *
+ * Separated from render.ts to avoid circular dependency issues with
+ * tools/renderers.ts. This module has no side effects and can be safely
+ * imported without triggering the subprocessToolRegistry registration.
+ */
+import { renderCall, renderResult } from "./render";
+export declare const taskToolRenderer: {
+    readonly renderCall: typeof renderCall;
+    readonly renderResult: typeof renderResult;
+    readonly mergeCallAndResult: true;
+};

package/dist/types/task/types.d.ts

@@ -206,6 +206,22 @@ export interface AgentDefinition {
     source: AgentSource;
     filePath?: string;
 }
+/** Details extracted from a subagent `yield` tool call for final-result assembly and task rendering. */
+export interface YieldItem {
+    data?: unknown;
+    status?: "success" | "aborted";
+    error?: string;
+    /** A string label is terminal; a non-empty array of labels is incremental. */
+    type?: string | string[];
+    /** Resolve this yield's payload from the latest durable assistant text instead of `data`. */
+    useLastTurn?: boolean;
+    /**
+     * Set by the in-tool yield validator when it exhausted its retry budget and
+     * accepted schema-invalid data anyway. The executor preserves that override
+     * during post-mortem validation.
+     */
+    schemaOverridden?: boolean;
+}
 /** Progress tracking for a single agent */
 export interface AgentProgress {
     index: number;

package/dist/types/task/yield-assembly.d.ts

@@ -0,0 +1,28 @@
+import type { YieldItem } from "./types";
+/** Outcome of folding a run's yield calls into one payload, with provenance flags. */
+interface AssembledYieldResult {
+    data: unknown;
+    schemaOverridden: boolean;
+    rawText: boolean;
+    missingData: boolean;
+}
+/**
+ * Top-level output-schema property names declared as arrays (JTD `elements` →
+ * JSON `type: "array"`). An incremental yield section for such a label
+ * accumulates into a list even when the agent emits exactly one — otherwise a
+ * single `type: ["findings"]` yield would assemble as a bare object and fail
+ * array-typed schema validation.
+ */
+export declare function arrayValuedLabels(outputSchema: unknown): ReadonlySet<string>;
+/**
+ * Assemble typed yield calls into the final payload consumed by schema validation.
+ *
+ * A non-empty array `type` contributes an incremental section and never decides
+ * termination by itself. A string `type` with omitted `data` makes the last
+ * assistant turn the raw terminal result. Other string-typed yields contribute
+ * the terminal labelled section. Untyped terminal yields keep the historical
+ * "last yield wins" behavior unless no terminal yield exists, in which case
+ * accumulated typed sections finalize on idle.
+ */
+export declare function assembleYieldResult(yieldItems: YieldItem[], lastAssistantText?: string, arrayLabels?: ReadonlySet<string>): AssembledYieldResult | undefined;
+export {};

package/dist/types/tiny/text.d.ts

@@ -16,6 +16,14 @@ export declare function stripCodeBlocks(message: string): string;
 /** Prepare a raw user message for titling: drop code blocks, then bound length. */
 export declare function prepareTitleInput(message: string): string;
 export declare function formatTitleUserMessage(message: string): string;
+/** Single recent conversation turn supplied to title refresh after replanning. */
+export interface TitleConversationTurn {
+    role: "user" | "assistant";
+    text?: string;
+    thinking?: string;
+}
+/** Format recent user/assistant context for title generation after a todo replan. */
+export declare function formatTitleConversationContext(turns: readonly TitleConversationTurn[]): string;
 /**
  * True when a first user message is too low-signal to title (greeting, ack,
  * bare number, or empty once code/punctuation/emoji are stripped).

package/dist/types/tools/render-utils.d.ts

@@ -45,6 +45,8 @@ export declare const TRUNCATE_LENGTHS: {
     readonly LINE: 110;
     /** Very short (task previews, badges) */
     readonly SHORT: 40;
+    /** Idle recap status line (~40-word LLM reply) */
+    readonly RECAP: 280;
 };
 /** Human-readable key currently bound to tool-output expansion, e.g. `Ctrl+O`. */
 export declare function expandKeyHint(): string;

package/dist/types/tools/review.d.ts

@@ -1,9 +1,10 @@
 /**
- * Review tools - report_finding for structured code review.
+ * Legacy hidden review-finding tool for agents that have not migrated to
+ * incremental `yield` sections.
  *
- * Used by the reviewer agent to report findings in a structured way.
- * Hidden by default - only enabled when explicitly listed in agent's tools.
- * Reviewers finish via `yield` tool with SubmitReviewDetails schema.
+ * Hidden by default - only enabled when explicitly listed in an agent's tools.
+ * Reviewers now finish via incremental `yield`; this tool remains for
+ * compatibility with older or custom review agents.
  */
 import type { AgentTool } from "@oh-my-pi/pi-agent-core";
 import type { Theme, ThemeColor } from "../modes/theme/theme";
@@ -15,6 +16,7 @@ export interface FindingPriorityInfo {
     color: ThemeColor;
 }
 export declare const PRIORITY_LABELS: FindingPriority[];
+export declare function isFindingPriority(value: unknown): value is FindingPriority;
 export declare function getPriorityInfo(priority: FindingPriority): FindingPriorityInfo;
 declare const ReportFindingParams: import("arktype/internal/variants/object.ts").ObjectType<{
     title: string;

package/dist/types/tools/ssh.d.ts

@@ -67,7 +67,7 @@ export declare const sshToolRenderer: {
         renderContext?: SshRenderContext;
     }, uiTheme: Theme, args?: SshRenderArgs): Component;
     mergeCallAndResult: boolean;
-    provisionalPendingPreview: string;
+    provisionalPendingPreview: boolean;
     provisionalPartialResult: boolean;
 };
 export {};

package/dist/types/tools/todo.d.ts

@@ -6,6 +6,8 @@ import type { Theme } from "../modes/theme/theme";
 import type { ToolSession } from "../sdk";
 import type { SessionEntry } from "../session/session-entries";
 export type TodoStatus = "pending" | "in_progress" | "completed" | "abandoned";
+/** Operation names accepted by the todo tool and echoed in successful result details. */
+export type TodoOperation = "init" | "start" | "done" | "rm" | "drop" | "append" | "view";
 export interface TodoItem {
     content: string;
     status: TodoStatus;
@@ -19,6 +21,8 @@ export interface TodoCompletionTransition {
     content: string;
 }
 export interface TodoToolDetails {
+    /** Operation that produced this snapshot; absent on legacy transcript entries. */
+    op?: TodoOperation;
     phases: TodoPhase[];
     storage: "session" | "memory";
     completedTasks?: TodoCompletionTransition[];
@@ -35,6 +39,8 @@ declare const todoSchema: import("arktype/internal/variants/object.ts").ObjectTy
 }, {}>;
 type TodoParams = TodoSchema;
 type TodoSchema = typeof todoSchema.infer;
+/** Return the active todo task, preferring an in-progress item over the first pending item. */
+export declare function nextActionableTask(phases: readonly TodoPhase[]): TodoItem | undefined;
 export declare const USER_TODO_EDIT_CUSTOM_TYPE = "user_todo_edit";
 export declare function getLatestTodoPhasesFromEntries(entries: SessionEntry[]): TodoPhase[];
 /**

package/dist/types/tools/yield.d.ts

@@ -1,15 +1,20 @@
 /**
- * Submit result tool for structured subagent output.
+ * Result submission tool for subagent output.
  *
- * Subagents must call this tool to finish and return structured JSON output.
+ * Subagents can call this tool incrementally or terminally depending on `type`.
  */
 import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
 import type { TSchema } from "@oh-my-pi/pi-ai/types";
 import type { ToolSession } from ".";
 export interface YieldDetails {
-    data: unknown;
+    /** Successful result payload, or omitted when `useLastTurn` requests last-turn extraction. */
+    data?: unknown;
     status: "success" | "aborted";
     error?: string;
+    /** Optional result section/classification supplied by the yield caller. */
+    type?: string | string[];
+    /** True when the caller intentionally omitted success data so the executor uses the last assistant turn. */
+    useLastTurn?: boolean;
     /**
      * Set when the yield tool exhausted its in-tool schema-retry budget
      * (MAX_SCHEMA_RETRIES) and accepted the data anyway. Surfaced so the

package/dist/types/utils/thinking-display.d.ts

@@ -1,3 +1,7 @@
+import type { AgentMessage } from "@oh-my-pi/pi-agent-core";
 export declare function canonicalizeMessage(text: string | null | undefined): string;
 export declare function formatThinkingForDisplay(text: string, proseOnly: boolean): string;
+/** Whether a formatted thinking block has non-placeholder content worth rendering. */
 export declare function hasDisplayableThinking(text: string | null | undefined, formattedText: string | null | undefined): boolean;
+/** Whether an assistant message contains thinking content the TUI can reveal. */
+export declare function messageHasDisplayableThinking(message: AgentMessage, proseOnly: boolean): boolean;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "16.2.2",
+	"version": "16.2.4",
 	"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.2",
-		"@oh-my-pi/omp-stats": "16.2.2",
-		"@oh-my-pi/pi-agent-core": "16.2.2",
-		"@oh-my-pi/pi-ai": "16.2.2",
-		"@oh-my-pi/pi-catalog": "16.2.2",
-		"@oh-my-pi/pi-mnemopi": "16.2.2",
-		"@oh-my-pi/pi-natives": "16.2.2",
-		"@oh-my-pi/pi-tui": "16.2.2",
-		"@oh-my-pi/pi-utils": "16.2.2",
-		"@oh-my-pi/pi-wire": "16.2.2",
-		"@oh-my-pi/snapcompact": "16.2.2",
+		"@oh-my-pi/hashline": "16.2.4",
+		"@oh-my-pi/omp-stats": "16.2.4",
+		"@oh-my-pi/pi-agent-core": "16.2.4",
+		"@oh-my-pi/pi-ai": "16.2.4",
+		"@oh-my-pi/pi-catalog": "16.2.4",
+		"@oh-my-pi/pi-mnemopi": "16.2.4",
+		"@oh-my-pi/pi-natives": "16.2.4",
+		"@oh-my-pi/pi-tui": "16.2.4",
+		"@oh-my-pi/pi-utils": "16.2.4",
+		"@oh-my-pi/pi-wire": "16.2.4",
+		"@oh-my-pi/snapcompact": "16.2.4",
 		"@opentelemetry/api": "^1.9.1",
 		"@opentelemetry/context-async-hooks": "^2.7.1",
 		"@opentelemetry/exporter-trace-otlp-proto": "^0.218.0",