@oh-my-pi/pi-coding-agent 17.3.0 → 17.3.2

package/src/utils/external-editor.ts

@@ -1,7 +1,6 @@
 /**
  * Utilities for launching an external text editor ($VISUAL / $EDITOR).
  */
-import { spawn } from "node:child_process";
 import * as fs from "node:fs/promises";
 import * as os from "node:os";
 import * as path from "node:path";
@@ -51,17 +50,17 @@ export async function openInEditor(
 	try {
 		await Bun.write(tmpFile, content);
 
-		const [editor, ...editorArgs] = editorCmd.split(" ");
-		const stdio = options?.stdio ?? ["inherit", "inherit", "inherit"];
-		const child =
+		const [stdin, stdout, stderr] = options?.stdio ?? ["inherit", "inherit", "inherit"];
+		const cmd =
 			process.platform === "win32"
-				? spawn(editor, [...editorArgs, tmpFile], { stdio, shell: true })
-				: spawn($which("sh") ?? "sh", ["-c", `${editorCmd} "$1"`, "sh", tmpFile], { stdio });
-		const { promise, reject, resolve } = Promise.withResolvers<number>();
-		child.once("exit", (code, signal) => resolve(code ?? (signal ? -1 : 0)));
-		child.once("error", error => reject(error));
-		const exitCode = await promise;
-
+				? ["cmd", "/c", `${editorCmd} "${tmpFile}"`]
+				: [$which("sh") ?? "sh", "-c", `${editorCmd} "$1"`, "sh", tmpFile];
+		const child = Bun.spawn(cmd, {
+			stdin,
+			stdout,
+			stderr,
+		});
+		const exitCode = await child.exited;
 		if (exitCode === 0) {
 			const text = await Bun.file(tmpFile).text();
 			if (options?.trimTrailingNewline === false) {

package/src/utils/git.ts

@@ -226,6 +226,11 @@ export const GIT_COMMAND_OUTPUT_LIMIT_BYTES = 8 * 1024 * 1024;
  * degrades instead of freezing the UI indefinitely.
  */
 export const GIT_SPAWN_SYNC_TIMEOUT_MS = 5_000;
+/**
+ * Stat-poll interval for {@link head.watch}. One `stat` per interval keeps an
+ * always-on status line cheap while surfacing a branch switch within a second.
+ */
+export const HEAD_WATCH_INTERVAL_MS = 1000;
 
 const GIT_COMMAND_TIMEOUT_EXIT_CODE = 124;
 // Exit code returned when the `git` binary cannot be launched at all (spawn
@@ -2296,6 +2301,28 @@ export const head = {
 		if (result.exitCode !== 0) return null;
 		return result.stdout.trim() || null;
 	},
+
+	/**
+	 * Watch the repository's HEAD for branch moves. Returns a disposer.
+	 *
+	 * Deliberately stat-polls via `fs.watchFile` instead of `fs.watch`: git
+	 * swaps HEAD with `HEAD.lock` + atomic rename, which unlinks the HEAD inode
+	 * — and Bun's inotify-backed `fs.watch` permanently stops delivering events
+	 * after observing a rename in the watched directory (oven-sh/bun#24875), so
+	 * an event watcher fires once and then freezes on Linux (issue #8412 was
+	 * the same freeze for file-inode watches on every platform). A path-based
+	 * stat poll re-resolves the path each interval and survives inode swaps
+	 * everywhere. Reftable repos keep ref state in `<gitDir>/reftable` (their
+	 * HEAD file is a static stub), so the poll targets that directory instead.
+	 */
+	watch(repository: GitRepository, onChange: () => void): () => void {
+		const target = isReftableRepoSync(repository) ? path.join(repository.gitDir, "reftable") : repository.headPath;
+		const listener = (curr: fs.Stats, prev: fs.Stats) => {
+			if (curr.mtimeMs !== prev.mtimeMs || curr.ino !== prev.ino || curr.size !== prev.size) onChange();
+		};
+		fs.watchFile(target, { interval: HEAD_WATCH_INTERVAL_MS }, listener).unref();
+		return () => fs.unwatchFile(target, listener);
+	},
 };
 
 // ════════════════════════════════════════════════════════════════════════════

package/src/utils/shell-snapshot.ts

@@ -208,10 +208,14 @@ fi
 /**
  * Create a shell snapshot, caching the result.
  * Returns the path to the snapshot file, or null if creation failed.
+ *
+ * `timeoutMs` is configurable so callers exercising failure handling do not
+ * have to wait out the production startup budget.
  */
 export async function getOrCreateSnapshot(
 	shell: string,
 	env: Record<string, string | undefined>,
+	timeoutMs = SNAPSHOT_TIMEOUT_MS,
 ): Promise<string | null> {
 	const cacheKey = shell;
 	// Return cached snapshot if valid
@@ -284,7 +288,7 @@ export async function getOrCreateSnapshot(
 			stdin: "ignore",
 			stdout: "ignore",
 			stderr: "ignore",
-			timeout: SNAPSHOT_TIMEOUT_MS,
+			timeout: timeoutMs,
 			killSignal: "SIGKILL",
 		});
 

package/src/web/search/providers/gemini.ts

@@ -9,11 +9,7 @@
  * endpoint.
  */
 import { type AuthStorage, type FetchImpl, type OAuthAccess, withOAuthAccess } from "@oh-my-pi/pi-ai";
-import {
-	ANTIGRAVITY_SYSTEM_INSTRUCTION,
-	getAntigravityUserAgent,
-	getGeminiCliHeaders,
-} from "@oh-my-pi/pi-catalog/wire/gemini-headers";
+import { getAntigravityUserAgent, getGeminiCliHeaders } from "@oh-my-pi/pi-catalog/wire/gemini-headers";
 import { fetchWithRetry, USER_AGENT } from "@oh-my-pi/pi-utils";
 
 import type { SearchCitation, SearchResponse, SearchSource } from "../../../web/search/types";
@@ -441,10 +437,9 @@ async function callGeminiSearch(
 			};
 
 	const normalizedSystemPrompt = systemPrompt?.toWellFormed();
-	const systemInstructionParts: Array<{ text: string }> = [
-		...(auth.isAntigravity ? [{ text: ANTIGRAVITY_SYSTEM_INSTRUCTION }] : []),
-		...(normalizedSystemPrompt ? [{ text: normalizedSystemPrompt }] : []),
-	];
+	const systemInstructionParts: Array<{ text: string }> = normalizedSystemPrompt
+		? [{ text: normalizedSystemPrompt }]
+		: [];
 
 	const requestBody: Record<string, unknown> = {
 		project: auth.projectId,