@oh-my-pi/pi-coding-agent 15.5.6 → 15.5.8

package/src/config/settings-schema.ts

@@ -1568,16 +1568,6 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
-	"edit.hashlineAutoDropPureInsertDuplicates": {
-		type: "boolean",
-		default: false,
-		ui: {
-			tab: "editing",
-			label: "Hashline Duplicate Insert Drop",
-			description:
-				"Drop payload lines that duplicate adjacent file context — 2+-line context echoes on `↑`/`↓` inserts, and a single boundary line at either edge of an `A-B:` replacement",
-		},
-	},
 	"edit.blockAutoGenerated": {
 		type: "boolean",
 		default: true,
@@ -1605,7 +1595,7 @@ export const SETTINGS_SCHEMA = {
 			tab: "editing",
 			label: "Hash Lines",
 			description:
-				"Include file-hash headers and line numbers in read output for hashline edit mode (¶PATH#hash plus LINE:content)",
+				"Include snapshot-tag headers and line numbers in read output for hashline edit mode (¶PATH#tag plus LINE:content)",
 		},
 	},
 
@@ -2036,6 +2026,15 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
+	"tts.enabled": {
+		type: "boolean",
+		default: false,
+		ui: {
+			tab: "tools",
+			label: "Text-to-Speech",
+			description: "Enable the tts tool for xAI Grok Voice speech synthesis",
+		},
+	},
 	"recipe.enabled": {
 		type: "boolean",
 		default: true,
@@ -2074,6 +2073,17 @@ export const SETTINGS_SCHEMA = {
 		ui: { tab: "tools", label: "Read URLs", description: "Allow the read tool to fetch and process URLs" },
 	},
 
+	"vault.enabled": {
+		type: "boolean",
+		default: false,
+		ui: {
+			tab: "tools",
+			label: "Obsidian Vault",
+			description:
+				"Enable the vault:// internal URL for reading and editing Obsidian vault content via the Obsidian CLI. When disabled, vault:// resolution is refused and the vault:// entry is omitted from the system prompt.",
+		},
+	},
+
 	"github.enabled": {
 		type: "boolean",
 		default: false,
@@ -2698,7 +2708,7 @@ export const SETTINGS_SCHEMA = {
 	},
 	"providers.image": {
 		type: "enum",
-		values: ["auto", "openai", "gemini", "openrouter"] as const,
+		values: ["auto", "openai", "antigravity", "xai", "gemini", "openrouter"] as const,
 		default: "auto",
 		ui: {
 			tab: "providers",
@@ -2708,9 +2718,19 @@ export const SETTINGS_SCHEMA = {
 				{
 					value: "auto",
 					label: "Auto",
-					description: "Priority: GPT model image tool > Antigravity > OpenRouter > Gemini",
+					description: "Priority: GPT model image tool > Antigravity > xAI > OpenRouter > Gemini",
 				},
 				{ value: "openai", label: "OpenAI", description: "Uses the active GPT Responses/Codex model" },
+				{
+					value: "antigravity",
+					label: "Antigravity",
+					description: "Requires google-antigravity OAuth",
+				},
+				{
+					value: "xai",
+					label: "xAI Grok Imagine",
+					description: "Requires xAI Grok OAuth or XAI_API_KEY",
+				},
 				{ value: "gemini", label: "Gemini", description: "Requires GEMINI_API_KEY" },
 				{ value: "openrouter", label: "OpenRouter", description: "Requires OPENROUTER_API_KEY" },
 			],
@@ -2748,6 +2768,28 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
+	"providers.openrouterVariant": {
+		type: "enum",
+		values: ["default", "nitro", "floor", "online", "exacto"] as const,
+		default: "default",
+		ui: {
+			tab: "providers",
+			label: "OpenRouter Routing",
+			description:
+				"Default routing-variant suffix appended to OpenRouter model IDs (overridden when the selector already names a variant)",
+			options: [
+				{ value: "default", label: "Default", description: "No suffix; use OpenRouter's default routing" },
+				{ value: "nitro", label: ":nitro", description: "Prioritize throughput / lowest latency" },
+				{ value: "floor", label: ":floor", description: "Prioritize cheapest available provider" },
+				{ value: "online", label: ":online", description: "Enable OpenRouter's web-search plugin" },
+				{
+					value: "exacto",
+					label: ":exacto",
+					description: "Cherry-picked high-quality providers (only defined for select models)",
+				},
+			],
+		},
+	},
 	"providers.parallelFetch": {
 		type: "boolean",
 		default: true,

package/src/edit/file-snapshot-store.ts

@@ -2,21 +2,24 @@
  * Session-bound file snapshot store.
  *
  * Used by `read` and `search` to record exactly what the model saw, and by
- * the hashline patcher to recover from stale section hashes (file changed
- * externally between read and edit, or a prior in-session edit advanced
- * the hash). The store is the {@link InMemorySnapshotStore} implementation
+ * the hashline patcher to verify or recover from stale section tags (file
+ * changed externally between read and edit, or a prior in-session edit
+ * advanced the tag). The store is the {@link InMemorySnapshotStore}
  * from `@oh-my-pi/hashline`; the only coding-agent-specific concern here
- * is wiring it onto the per-session {@link ToolSession} object.
+ * is wiring it onto the per-session owner object.
  */
 import { InMemorySnapshotStore } from "@oh-my-pi/hashline";
-import type { ToolSession } from "../tools";
+
+interface FileSnapshotStoreOwner {
+	fileSnapshotStore?: InMemorySnapshotStore;
+}
 
 /**
  * Look up (or lazily create) the file snapshot store attached to a session.
  * Storage lives on `session.fileSnapshotStore` so it ages out exactly with
  * the session itself.
  */
-export function getFileSnapshotStore(session: ToolSession): InMemorySnapshotStore {
+export function getFileSnapshotStore(session: FileSnapshotStoreOwner): InMemorySnapshotStore {
 	if (!session.fileSnapshotStore) session.fileSnapshotStore = new InMemorySnapshotStore();
 	return session.fileSnapshotStore;
 }

package/src/edit/hashline/diff.ts

@@ -5,16 +5,17 @@
  * pair to {@link generateDiffString} so the renderer can show the diff
  * while the tool call is still streaming.
  *
- * Validation is intentionally light: only the section file hash is checked
+ * Validation is intentionally light: only the section snapshot tag is checked
  * (so the preview goes red when anchors are stale), no plan-mode guards
  * and no auto-generated-file refusal — those belong on the write path.
  */
 import {
-	computeFileHash,
 	Patch as HashlinePatch,
 	normalizeToLF,
 	type Patch,
 	type PatchSection,
+	type Snapshot,
+	type SnapshotStore,
 	stripBom,
 } from "@oh-my-pi/hashline";
 import { resolveToCwd } from "../../tools/path-utils";
@@ -22,7 +23,6 @@ import { generateDiffString } from "../diff";
 import { readEditFileText } from "../read-file";
 
 export interface HashlineDiffOptions {
-	autoDropPureInsertDuplicates?: boolean;
 	/**
 	 * Use the streaming-tolerant applier ({@link PatchSection.applyPartialTo})
 	 * so trailing in-flight ops do not throw or emit phantom edits. Streaming
@@ -44,20 +44,34 @@ function hasAnchorScoped(section: PatchSection): boolean {
 	return section.hasAnchorScopedEdit;
 }
 
-function validateSectionHash(section: PatchSection, text: string): string | null {
+function snapshotMatchesCurrent(snapshot: Snapshot, currentText: string, anchorLines: readonly number[]): boolean {
+	if (snapshot.fullText !== undefined) return snapshot.fullText === currentText;
+	for (const lineNumber of anchorLines) {
+		if (snapshot.get(lineNumber) === undefined) return false;
+	}
+	return snapshot.matchesLiveFile(currentText.split("\n"));
+}
+
+function validateSectionHash(
+	section: PatchSection,
+	absolutePath: string,
+	text: string,
+	snapshots: SnapshotStore,
+): string | null {
 	if (section.fileHash === undefined) {
 		return hasAnchorScoped(section)
-			? `Missing hashline file hash for anchored edit to ${section.path}; use \`¶${section.path}#hash\` from your latest read.`
+			? `Missing hashline snapshot tag for anchored edit to ${section.path}; use \`¶${section.path}#tag\` from your latest read.`
 			: null;
 	}
-	const currentHash = computeFileHash(text);
-	if (currentHash === section.fileHash) return null;
-	return `Hashline file hash mismatch for ${section.path}: section is bound to #${section.fileHash}, but current file hashes to #${currentHash}; re-read and try again.`;
+	const snapshot = snapshots.byHash(absolutePath, section.fileHash);
+	if (snapshot && snapshotMatchesCurrent(snapshot, text, section.collectAnchorLines())) return null;
+	return `Hashline snapshot tag mismatch for ${section.path}: section is bound to #${section.fileHash}, but current file does not match that snapshot; re-read and try again.`;
 }
 
 export async function computeHashlineSectionDiff(
 	section: PatchSection,
 	cwd: string,
+	snapshots: SnapshotStore,
 	options: HashlineDiffOptions = {},
 ): Promise<{ diff: string; firstChangedLine: number | undefined } | { error: string }> {
 	try {
@@ -65,11 +79,9 @@ export async function computeHashlineSectionDiff(
 		const rawContent = await readSectionText(absolutePath, section.path);
 		const { text: content } = stripBom(rawContent);
 		const normalized = normalizeToLF(content);
-		const hashError = validateSectionHash(section, normalized);
+		const hashError = validateSectionHash(section, absolutePath, normalized, snapshots);
 		if (hashError) return { error: hashError };
-		const result = options.streaming
-			? section.applyPartialTo(normalized, options)
-			: section.applyTo(normalized, options);
+		const result = options.streaming ? section.applyPartialTo(normalized) : section.applyTo(normalized);
 		if (normalized === result.text) return { error: `No changes would be made to ${section.path}.` };
 		return generateDiffString(normalized, result.text);
 	} catch (err) {
@@ -80,6 +92,7 @@ export async function computeHashlineSectionDiff(
 export async function computeHashlineDiff(
 	input: { input: string },
 	cwd: string,
+	snapshots: SnapshotStore,
 	options: HashlineDiffOptions = {},
 ): Promise<{ diff: string; firstChangedLine: number | undefined } | { error: string }> {
 	let patch: Patch;
@@ -91,5 +104,5 @@ export async function computeHashlineDiff(
 	if (patch.sections.length !== 1) {
 		return { error: "Streaming diff preview supports exactly one hashline section." };
 	}
-	return computeHashlineSectionDiff(patch.sections[0], cwd, options);
+	return computeHashlineSectionDiff(patch.sections[0], cwd, snapshots, options);
 }

package/src/edit/hashline/execute.ts

@@ -37,14 +37,19 @@ export interface ExecuteHashlineSingleOptions {
 	beginDeferredDiagnosticsForPath: (path: string) => WritethroughDeferredHandle;
 }
 
-function getHashlineApplyOptions(session: ToolSession): { autoDropPureInsertDuplicates: boolean } {
-	return {
-		autoDropPureInsertDuplicates: session.settings.get("edit.hashlineAutoDropPureInsertDuplicates"),
-	};
-}
-
 function noChangeDiagnostic(path: string): string {
-	return `Edits to ${path} resulted in no changes being made.`;
+	// The patch parsed and applied cleanly but produced no change — the
+	// `|literal` body rows matched the file content at the targeted lines
+	// byte-for-byte. The model usually misreads this as "wrong anchor, try
+	// again with a bigger payload" and starts duplicating content; the
+	// message below names the cause directly so the next turn can re-read
+	// instead of expanding the patch.
+	return (
+		`Edits to ${path} parsed and applied cleanly, but produced no change: ` +
+		`your body row(s) are byte-identical to the file at the targeted lines. ` +
+		`The bug is somewhere else — re-read the file before issuing another edit. ` +
+		`Do NOT widen the payload or add lines; verify the anchor first.`
+	);
 }
 
 function assertUniqueCanonicalPaths(prepared: readonly PreparedSection[]): void {
@@ -128,8 +133,7 @@ export async function executeHashlineSingle(
 		batchRequest: options.batchRequest,
 	});
 	const snapshots = getFileSnapshotStore(options.session);
-	const applyOptions = getHashlineApplyOptions(options.session);
-	const patcher = new Patcher({ fs, snapshots, applyOptions });
+	const patcher = new Patcher({ fs, snapshots });
 
 	// Single-section fast path: prepare, commit, render.
 	if (patch.sections.length === 1) {

package/src/edit/renderer.ts

@@ -235,24 +235,24 @@ function renderPlainTextPreview(text: string, uiTheme: Theme, filePath?: string)
 
 function formatStreamingDiff(diff: string, rawPath: string, uiTheme: Theme, label = "streaming"): string {
 	if (!diff) return "";
-	// Hunk-aware truncation keeps the change rows themselves visible and
-	// trims surrounding context proportionally so a multi-hunk diff doesn't
-	// turn into just the tail of the last hunk while streaming.
+	// Hunk-aware truncation keeps the change rows themselves visible. Tail-mode
+	// pins the visible window to the bottom of the diff so newly streamed
+	// hunks stay on screen as more arrives, instead of leaving the user stuck
+	// staring at the head of the file while the tail scrolls offscreen.
 	const {
 		text: truncatedDiff,
 		hiddenHunks,
 		hiddenLines,
-	} = truncateDiffByHunk(diff, PREVIEW_LIMITS.DIFF_COLLAPSED_HUNKS, EDIT_STREAMING_PREVIEW_LINES);
+	} = truncateDiffByHunk(diff, PREVIEW_LIMITS.DIFF_COLLAPSED_HUNKS, EDIT_STREAMING_PREVIEW_LINES, { fromTail: true });
 	let text = "\n\n";
-	text += renderDiffColored(truncatedDiff, { filePath: rawPath });
 	if (hiddenHunks > 0 || hiddenLines > 0) {
 		const remainder: string[] = [];
 		if (hiddenHunks > 0) remainder.push(`${hiddenHunks} more hunks`);
 		if (hiddenLines > 0) remainder.push(`${hiddenLines} more lines`);
-		text += uiTheme.fg("dim", `\n… (${label} +${remainder.join(", ")})`);
-	} else {
-		text += uiTheme.fg("dim", `\n(${label})`);
+		text += `${uiTheme.fg("dim", `… (${remainder.join(", ")} above)`)}\n`;
 	}
+	text += renderDiffColored(truncatedDiff, { filePath: rawPath });
+	text += uiTheme.fg("dim", `\n(${label})`);
 	return text;
 }
 
@@ -312,7 +312,7 @@ const MISSING_APPLY_PATCH_END_ERROR = "The last line of the patch must be '*** E
 
 function normalizeHashlineInputPreviewPath(rawPath: string): string {
 	const trimmed = rawPath.trim();
-	const hashStart = /#[0-9a-f]{4}$/u.exec(trimmed)?.index;
+	const hashStart = /#[0-9a-fA-F]{3}$/u.exec(trimmed)?.index;
 	const withoutHash = hashStart === undefined ? trimmed : trimmed.slice(0, hashStart);
 	if (withoutHash.length < 2) return withoutHash;
 	const first = withoutHash[0];

package/src/edit/streaming.ts

@@ -20,6 +20,7 @@ import {
 	END_PATCH_MARKER,
 	type PatchSection as HashlineInputSection,
 	Patch as HashlinePatch,
+	type SnapshotStore,
 } from "@oh-my-pi/hashline";
 import type { Theme } from "../modes/theme/theme";
 import { type EditMode, resolveEditMode } from "../utils/edit-mode";
@@ -39,9 +40,9 @@ export interface PerFileDiffPreview {
 export interface StreamingDiffContext {
 	cwd: string;
 	signal: AbortSignal;
+	snapshots: SnapshotStore;
 	fuzzyThreshold?: number;
 	allowFuzzy?: boolean;
-	hashlineAutoDropPureInsertDuplicates?: boolean;
 	/**
 	 * True while the tool's arguments are still streaming in. Strategies that
 	 * accept free-form text input (apply_patch, hashline) trim the trailing
@@ -325,9 +326,7 @@ const hashlineStrategy: EditStreamingStrategy<HashlineArgs> = {
 			// to parse; suppress until the next chunk arrives. Once args are
 			// complete, surface the error so the model sees what went wrong.
 			if (ctx.isStreaming) return null;
-			const result = await computeHashlineDiff({ input }, ctx.cwd, {
-				autoDropPureInsertDuplicates: ctx.hashlineAutoDropPureInsertDuplicates,
-			});
+			const result = await computeHashlineDiff({ input }, ctx.cwd, ctx.snapshots);
 			ctx.signal.throwIfAborted();
 			return [toPerFilePreview("", result)];
 		}
@@ -346,8 +345,7 @@ const hashlineStrategy: EditStreamingStrategy<HashlineArgs> = {
 		for (let i = 0; i < sectionsToProcess.length; i++) {
 			ctx.signal.throwIfAborted();
 			const section = sectionsToProcess[i];
-			const result = await computeHashlineSectionDiff(section, ctx.cwd, {
-				autoDropPureInsertDuplicates: ctx.hashlineAutoDropPureInsertDuplicates,
+			const result = await computeHashlineSectionDiff(section, ctx.cwd, ctx.snapshots, {
 				streaming: ctx.isStreaming,
 			});
 			ctx.signal.throwIfAborted();

package/src/eval/py/index.ts

@@ -5,7 +5,7 @@ import { checkPythonKernelAvailability } from "./kernel";
 
 const PYTHON_SESSION_PREFIX = "python:";
 
-function namespaceSessionId(sessionId: string): string {
+export function namespaceSessionId(sessionId: string): string {
 	return sessionId.startsWith(PYTHON_SESSION_PREFIX) ? sessionId : `${PYTHON_SESSION_PREFIX}${sessionId}`;
 }
 

package/src/extensibility/custom-tools/types.ts

@@ -100,7 +100,7 @@ export type CustomToolSessionEvent =
 	  }
 	| {
 			reason: "auto_compaction_start";
-			trigger: "threshold" | "overflow" | "idle";
+			trigger: "threshold" | "overflow" | "idle" | "incomplete";
 			action: "context-full" | "handoff";
 	  }
 	| {

package/src/extensibility/shared-events.ts

@@ -203,7 +203,7 @@ export interface TurnEndEvent {
 /** Fired when auto-compaction starts */
 export interface AutoCompactionStartEvent {
 	type: "auto_compaction_start";
-	reason: "threshold" | "overflow" | "idle";
+	reason: "threshold" | "overflow" | "idle" | "incomplete";
 	action: "context-full" | "handoff";
 }