@oh-my-pi/pi-coding-agent 17.2.9 → 17.2.11

package/src/web/search/providers/browser-headers.ts

@@ -1,28 +1,13 @@
-import { HeaderGenerator } from "header-generator";
+import { HeaderGenerator } from "@oh-my-pi/pi-utils/headers";
 
-// Lazily instantiate the singleton header generator. Bun single-file binaries do not
-// bundle header-generator's fs-loaded data_files, so construction may throw when the
-// original build-time node_modules path is absent.
-let generator: HeaderGenerator | undefined;
-let generatorUnavailable = false;
-
-function getHeaderGenerator(): HeaderGenerator | undefined {
-	if (generatorUnavailable) return undefined;
-	try {
-		generator ??= new HeaderGenerator({
-			browserListQuery: "last 3 versions",
-			devices: ["desktop"],
-			operatingSystems: ["windows", "macos", "linux"],
-			locales: ["en-US", "en"],
-			httpVersion: "2",
-			strict: false,
-		});
-		return generator;
-	} catch {
-		generatorUnavailable = true;
-		return undefined;
-	}
-}
+const generator = new HeaderGenerator({
+	browserListQuery: "last 3 versions",
+	devices: ["desktop"],
+	operatingSystems: ["windows", "macos", "linux"],
+	locales: ["en-US", "en"],
+	httpVersion: "2",
+	strict: false,
+});
 
 // A fallback desktop Mac Chrome navigation fingerprint matching
 // the previous static default setup for deterministic or non-randomized calls.
@@ -84,9 +69,8 @@ function canonicalizeHeaderNames(headers: Record<string, string>): Record<string
 
 /**
  * Build a fresh, internally consistent desktop navigation fingerprint for one HTTP request.
- * By default, this randomizes across valid modern versions of Chrome, Firefox, Edge, and Safari
- * using real-world traffic data. Set `randomized` to `false` when a fetch must preserve a
- * stable Mac Chrome identity.
+ * By default, this randomizes across coherent modern Chrome, Firefox, and Safari profiles.
+ * Set `randomized` to `false` when a fetch must preserve a stable Mac Chrome identity.
  */
 export function buildBrowserNavigationHeaders(options?: { randomized?: boolean }): Record<string, string> {
 	const randomized = options?.randomized !== false;
@@ -94,16 +78,5 @@ export function buildBrowserNavigationHeaders(options?: { randomized?: boolean }
 		return { ...CHROME_FALLBACK_HEADERS };
 	}
 
-	const generator = getHeaderGenerator();
-	if (!generator) {
-		return { ...CHROME_FALLBACK_HEADERS };
-	}
-
-	try {
-		// Generate realistic, consistent headers with the Bayesian generator
-		return canonicalizeHeaderNames(generator.getHeaders());
-	} catch {
-		// Gracefully recover to the robust default profile on unexpected generator errors
-		return { ...CHROME_FALLBACK_HEADERS };
-	}
+	return canonicalizeHeaderNames(generator.getHeaders());
 }

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

@@ -13,11 +13,7 @@ import {
 	withAuth,
 	withOAuthAccess,
 } from "@oh-my-pi/pi-ai";
-import { applyCodexResponsesLiteShape } from "@oh-my-pi/pi-ai/providers/openai-codex/request-transformer";
-import {
-	createOpenAICodexCompatibilityMetadata,
-	resolveCodexResponsesUrl,
-} from "@oh-my-pi/pi-ai/providers/openai-codex-responses";
+import { resolveCodexResponsesUrl } from "@oh-my-pi/pi-ai/providers/openai-codex-responses";
 import { getBundledModels } from "@oh-my-pi/pi-catalog/models";
 import {
 	CODEX_BASE_URL,
@@ -447,7 +443,6 @@ async function callCodexSearch(
 		systemPrompt?: string;
 		searchContextSize?: "low" | "medium" | "high";
 		model: CodexModelCandidate;
-		sessionId?: string;
 		fetch?: FetchImpl;
 		transport: CodexSearchTransport;
 	},
@@ -455,7 +450,6 @@ async function callCodexSearch(
 	const headers = buildCodexHeaders(auth.accessToken, auth.accountId, options.transport.headers);
 
 	const requestedModel = options.model.modelId;
-	const usesResponsesLite = options.model.catalogModel?.useResponsesLite === true;
 
 	const body: Record<string, unknown> = {
 		model: requestedModel,
@@ -477,21 +471,6 @@ async function callCodexSearch(
 		tool_choice: { type: "web_search" },
 		instructions: options.systemPrompt ?? DEFAULT_INSTRUCTIONS,
 	};
-	if (usesResponsesLite) {
-		const metadata = createOpenAICodexCompatibilityMetadata({
-			sessionId: options.sessionId,
-			requestKind: "turn",
-			startNewTurn: true,
-		});
-		for (const name in metadata.headers) {
-			const value = metadata.headers[name];
-			if (value !== undefined) headers.set(name, value);
-		}
-		headers.set(OPENAI_HEADERS.RESPONSES_LITE, "true");
-		body.client_metadata = metadata.clientMetadata;
-		body.reasoning = { context: "all_turns" };
-		applyCodexResponsesLiteShape(body);
-	}
 
 	const fetchImpl = options.fetch ?? fetch;
 	const response = await fetchImpl(options.transport.url, {
@@ -519,9 +498,8 @@ async function callCodexSearch(
 	let model = requestedModel;
 	let requestId = "";
 	let usage: { inputTokens: number; outputTokens: number; totalTokens: number } | undefined;
-	// Evidence that the hosted web_search tool actually ran. Lite models get
-	// `tool_choice: "auto"` and may answer without searching (#6988); a search
-	// command must reject that rather than return a non-search completion.
+	// A search command must reject a completion that did not invoke the hosted
+	// tool rather than returning an answer from the model's own knowledge (#6988).
 	let webSearchInvoked = false;
 
 	for await (const rawEvent of readSseJson<Record<string, unknown>>(response.body, options.signal)) {
@@ -651,7 +629,6 @@ async function runCodexSearchCandidates(options: {
 				systemPrompt: options.params.systemPrompt,
 				searchContextSize: "high",
 				model: candidate,
-				sessionId: options.params.sessionId,
 				fetch: options.params.fetch,
 				transport: options.transport,
 			});

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

@@ -1,5 +1,5 @@
 import type { AuthStorage } from "@oh-my-pi/pi-ai";
-import { parseHTML } from "linkedom";
+import { parseHTML } from "@oh-my-pi/pi-utils/dom";
 import type { SearchResponse, SearchSource } from "../../../web/search/types";
 import { SearchProviderError } from "../../../web/search/types";
 import { formatScraperQuery } from "../query";

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

@@ -464,7 +464,7 @@ export class ExaProvider extends SearchProvider {
 
 	#settingsAllowSearch(): boolean {
 		try {
-			if (settings.get("exa.enabled") === false || settings.get("exa.enableSearch") === false) {
+			if (settings.get("exa.enabled") === false) {
 				return false;
 			}
 		} catch {

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

@@ -1,5 +1,5 @@
 import type { AuthStorage } from "@oh-my-pi/pi-ai";
-import { parseHTML } from "linkedom";
+import { type Element, parseHTML } from "@oh-my-pi/pi-utils/dom";
 import type { SearchResponse, SearchSource } from "../../../web/search/types";
 import { SearchProviderError } from "../../../web/search/types";
 import { formatScraperQuery } from "../query";

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

@@ -1,6 +1,6 @@
 import type { AuthStorage } from "@oh-my-pi/pi-ai";
 import { untilAborted } from "@oh-my-pi/pi-utils";
-import { parseHTML } from "linkedom";
+import { parseHTML } from "@oh-my-pi/pi-utils/dom";
 import type { Page } from "puppeteer-core";
 import type { SearchResponse, SearchSource } from "../../../web/search/types";
 import { SearchProviderError } from "../../../web/search/types";

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

@@ -1,5 +1,5 @@
 import type { AuthStorage, FetchImpl } from "@oh-my-pi/pi-ai";
-import { parseHTML } from "linkedom";
+import { parseHTML } from "@oh-my-pi/pi-utils/dom";
 import type { SearchResponse, SearchSource } from "../../../web/search/types";
 import { SearchProviderError } from "../../../web/search/types";
 import { formatScraperQuery } from "../query";

package/src/markit/converters/mammoth.d.ts

@@ -1,24 +0,0 @@
-// Minimal ambient types for `mammoth` (ships no types). Declares only what
-// DocxConverter uses. See ../NOTICE.
-declare module "mammoth" {
-	interface MammothImage {
-		contentType?: string;
-		read(encoding: "base64"): Promise<string>;
-	}
-	interface ImgAttributes {
-		src: string;
-		alt?: string;
-	}
-	type ConvertImageHandler = (image: MammothImage) => Promise<ImgAttributes>;
-	interface ConvertOptions {
-		convertImage?: ConvertImageHandler;
-	}
-	interface ConvertResult {
-		value: string;
-		messages: unknown[];
-	}
-	export const images: { imgElement(fn: ConvertImageHandler): ConvertImageHandler };
-	export function convertToHtml(input: { buffer: Buffer }, options?: ConvertOptions): Promise<ConvertResult>;
-	const _default: { convertToHtml: typeof convertToHtml; images: typeof images };
-	export default _default;
-}