pi-openai-toolkit 0.24.1 → 0.24.2

package/src/web-search/prompt.ts

@@ -1,60 +1,60 @@
-import type { WebSearchConfig } from "../types";
-import {
-	resolveWebSearchRoute,
-	WEB_RUN_TOOL_NAME,
-	WEB_SEARCH_PROMPT_MARKER,
-	type WebSearchModel,
-} from "./types";
-
-export const WEB_SEARCH_PROMPT_SECTION = `${WEB_SEARCH_PROMPT_MARKER}
-## Web Search
-
-The hosted \`web_search\` tool is part of your tool list for this model. The API executes it server-side: it is a real callable tool even though no client-side function schema or description accompanies it. Call it whenever current or online information matters instead of guessing, cite the returned sources in the answer, and never claim that web search or internet access is unavailable while this section is present.`;
-
-export const LOCAL_WEB_SEARCH_PROMPT_SECTION = `${WEB_SEARCH_PROMPT_MARKER}
-## Web Search
-
-The local \`web_search\` tool is available for this model. Use it when current or online information matters, and cite the sources returned by that tool. Do not claim that the provider API executes this local tool server-side.`;
-
-export const STANDALONE_ALPHA_WEB_SEARCH_PROMPT_SECTION = `${WEB_SEARCH_PROMPT_MARKER}
-## Web Search
-
-The \`${WEB_RUN_TOOL_NAME}\` tool is available for this model. It is a sequential local wrapper backed by CPA/Codex standalone search at the provider's \`/alpha/search\` endpoint. It supports \`search_query\`, \`image_query\`, \`open\`, \`click\`, \`find\`, \`screenshot\`, \`finance\`, \`weather\`, \`sports\`, and \`time\`. Use it for current or online information, preserve reference ids when following results, and cite the returned sources.`;
-
-function removeToolkitWebSearchPrompt(systemPrompt: string): string {
-	const markerIndex = systemPrompt.indexOf(WEB_SEARCH_PROMPT_MARKER);
-	if (markerIndex < 0) return systemPrompt;
-	return systemPrompt.slice(0, markerIndex).trimEnd();
-}
-
-function promptSectionForRoute(route: "local" | "hosted" | "standalone-alpha"): string {
-	switch (route) {
-		case "local":
-			return LOCAL_WEB_SEARCH_PROMPT_SECTION;
-		case "hosted":
-			return WEB_SEARCH_PROMPT_SECTION;
-		case "standalone-alpha":
-			return STANDALONE_ALPHA_WEB_SEARCH_PROMPT_SECTION;
-	}
-}
-
-export function appendWebSearchPrompt(args: {
-	model: WebSearchModel | undefined;
-	config: WebSearchConfig;
-	systemPrompt: string;
-	/** Lifecycle registration can override this when the selected local wrapper is unavailable. */
-	routeAvailable?: boolean;
-}): string {
-	const { model, config, systemPrompt } = args;
-	const basePrompt = removeToolkitWebSearchPrompt(systemPrompt);
-	const resolution = resolveWebSearchRoute({ model, config });
-	if (args.routeAvailable === false || resolution.route === "none" || !resolution.available) return basePrompt;
-
-	const section = promptSectionForRoute(resolution.route);
-	return `${basePrompt.trimEnd()}\n\n${section}`;
-}
-
-export const _promptTest = {
-	removeToolkitWebSearchPrompt,
-	promptSectionForRoute,
-};
+import type { WebSearchConfig } from "../types";
+import {
+	resolveWebSearchRoute,
+	WEB_RUN_TOOL_NAME,
+	WEB_SEARCH_PROMPT_MARKER,
+	type WebSearchModel,
+} from "./types";
+
+export const WEB_SEARCH_PROMPT_SECTION = `${WEB_SEARCH_PROMPT_MARKER}
+## Web Search
+
+The hosted \`web_search\` tool is part of your tool list for this model. The API executes it server-side: it is a real callable tool even though no client-side function schema or description accompanies it. Call it whenever current or online information matters instead of guessing, cite the returned sources in the answer, and never claim that web search or internet access is unavailable while this section is present.`;
+
+export const LOCAL_WEB_SEARCH_PROMPT_SECTION = `${WEB_SEARCH_PROMPT_MARKER}
+## Web Search
+
+The local \`web_search\` tool is available for this model. Use it when current or online information matters, and cite the sources returned by that tool. Do not claim that the provider API executes this local tool server-side.`;
+
+export const STANDALONE_ALPHA_WEB_SEARCH_PROMPT_SECTION = `${WEB_SEARCH_PROMPT_MARKER}
+## Web Search
+
+The \`${WEB_RUN_TOOL_NAME}\` tool is available for this model. It is a sequential local wrapper backed by CPA/Codex standalone search at the provider's \`/alpha/search\` endpoint. It supports \`search_query\`, \`image_query\`, \`open\`, \`click\`, \`find\`, \`screenshot\`, \`finance\`, \`weather\`, \`sports\`, and \`time\`. Use it for current or online information, preserve reference ids when following results, and cite the returned sources.`;
+
+function removeToolkitWebSearchPrompt(systemPrompt: string): string {
+	const markerIndex = systemPrompt.indexOf(WEB_SEARCH_PROMPT_MARKER);
+	if (markerIndex < 0) return systemPrompt;
+	return systemPrompt.slice(0, markerIndex).trimEnd();
+}
+
+function promptSectionForRoute(route: "local" | "hosted" | "standalone-alpha"): string {
+	switch (route) {
+		case "local":
+			return LOCAL_WEB_SEARCH_PROMPT_SECTION;
+		case "hosted":
+			return WEB_SEARCH_PROMPT_SECTION;
+		case "standalone-alpha":
+			return STANDALONE_ALPHA_WEB_SEARCH_PROMPT_SECTION;
+	}
+}
+
+export function appendWebSearchPrompt(args: {
+	model: WebSearchModel | undefined;
+	config: WebSearchConfig;
+	systemPrompt: string;
+	/** Lifecycle registration can override this when the selected local wrapper is unavailable. */
+	routeAvailable?: boolean;
+}): string {
+	const { model, config, systemPrompt } = args;
+	const basePrompt = removeToolkitWebSearchPrompt(systemPrompt);
+	const resolution = resolveWebSearchRoute({ model, config });
+	if (args.routeAvailable === false || resolution.route === "none" || !resolution.available) return basePrompt;
+
+	const section = promptSectionForRoute(resolution.route);
+	return `${basePrompt.trimEnd()}\n\n${section}`;
+}
+
+export const _promptTest = {
+	removeToolkitWebSearchPrompt,
+	promptSectionForRoute,
+};

package/src/web-search/types.ts

@@ -1,166 +1,166 @@
-import { VIRTUAL_MODEL_API } from "../runtime";
-import { getExactModelKey, isExactModelAllowed } from "../model-scope";
-import type { WebSearchConfig, WebSearchRoute } from "../types";
-
-export const WEB_SEARCH_CAPABLE_APIS = [
-	"openai-responses",
-	"openai-codex-responses",
-	"azure-openai-responses",
-] as const;
-/** The provider-relative alpha endpoint is only verified for OpenAI/Codex routes. */
-export const WEB_SEARCH_STANDALONE_APIS = ["openai-responses", "openai-codex-responses"] as const;
-export const WEB_SEARCH_SOURCE_INCLUDE = "web_search_call.action.sources";
-export const WEB_SEARCH_PROMPT_MARKER = "<!-- pi-openai-toolkit:web-search -->";
-export const WEB_RUN_TOOL_NAME = "web_run";
-export const LOCAL_WEB_SEARCH_TOOL_NAME = "web_search";
-
-export type WebSearchCapableApi = (typeof WEB_SEARCH_CAPABLE_APIS)[number];
-export type WebSearchStandaloneApi = (typeof WEB_SEARCH_STANDALONE_APIS)[number];
-export type WebSearchRouteSource = "exact" | "default" | "legacy" | "none";
-export type WebSearchRouteUnavailableReason = "missing-model" | "unsupported-api" | "invalid-route";
-
-export type WebSearchModel = {
-	api?: string;
-	provider?: string;
-	id?: string;
-};
-
-export type WebSearchRouteResolution =
-	| {
-			route: "none";
-			source: "none";
-			modelKey?: string;
-			reason: "disabled" | "unconfigured";
-	  }
-	| {
-			route: WebSearchRoute;
-			source: Exclude<WebSearchRouteSource, "none">;
-			modelKey?: string;
-			available: boolean;
-			reason?: WebSearchRouteUnavailableReason;
-	  };
-
-export type WebSearchPayloadOutcome =
-	| "disabled"
-	| "unsupported-model"
-	| "non-object-payload"
-	| "invalid-tools"
-	| "invalid-include"
-	| "existing-native-tool"
-	| "injected-native-tool"
-	| "local-route"
-	| "standalone-route"
-	| "removed-conflicting-tools"
-	| "unavailable-route";
-
-export type WebSearchPayloadTransform = {
-	payload: unknown;
-	outcome: WebSearchPayloadOutcome;
-	changed: boolean;
-	/** Explicit route failures must abort rather than silently select another search path. */
-	fatal?: boolean;
-	errorMessage?: string;
-};
-
-export function getWebSearchModelKey(model: WebSearchModel | undefined): string | undefined {
-	return getExactModelKey(model);
-}
-
-export function isWebSearchRoute(value: unknown): value is WebSearchRoute {
-	return value === "local" || value === "hosted" || value === "standalone-alpha";
-}
-
-function selectedRoute(args: {
-	route: WebSearchRoute;
-	source: Exclude<WebSearchRouteSource, "none">;
-	model: WebSearchModel | undefined;
-	modelKey: string | undefined;
-}): WebSearchRouteResolution {
-	if (!args.modelKey) {
-		return {
-			route: args.route,
-			source: args.source,
-			available: false,
-			reason: "missing-model",
-		};
-	}
-	if (
-		args.route !== "local" &&
-		args.model?.api !== VIRTUAL_MODEL_API &&
-		(args.route === "standalone-alpha"
-			? !isWebSearchStandaloneCapableApi(args.model?.api)
-			: !isWebSearchCapableApi(args.model?.api))
-	) {
-		return {
-			route: args.route,
-			source: args.source,
-			modelKey: args.modelKey,
-			available: false,
-			reason: "unsupported-api",
-		};
-	}
-	return {
-		route: args.route,
-		source: args.source,
-		modelKey: args.modelKey,
-		available: true,
-	};
-}
-
-export function isWebSearchCapableApi(api: unknown): api is WebSearchCapableApi {
-	return typeof api === "string" && (WEB_SEARCH_CAPABLE_APIS as readonly string[]).includes(api);
-}
-
-export function isWebSearchStandaloneCapableApi(api: unknown): api is WebSearchStandaloneApi {
-	return typeof api === "string" && (WEB_SEARCH_STANDALONE_APIS as readonly string[]).includes(api);
-}
-
-/**
- * Resolve exactly one configured Web Search route. The legacy model allowlist is
- * deliberately consulted only after explicit route fields so old configs retain
- * hosted behavior without becoming an implicit alpha-search opt-in.
- */
-export function resolveWebSearchRoute(args: {
-	model: WebSearchModel | undefined;
-	config: WebSearchConfig;
-}): WebSearchRouteResolution {
-	const { model, config } = args;
-	const modelKey = getWebSearchModelKey(model);
-	if (!config.enabled) {
-		return { route: "none", source: "none", modelKey, reason: "disabled" };
-	}
-
-	const exactRoute = modelKey ? config.routes?.[modelKey] : undefined;
-	if (exactRoute !== undefined) {
-		if (!isWebSearchRoute(exactRoute)) {
-			return { route: "none", source: "none", modelKey, reason: "unconfigured" };
-		}
-		return selectedRoute({ route: exactRoute, source: "exact", model, modelKey });
-	}
-
-	if (config.defaultRoute !== undefined) {
-		if (!isWebSearchRoute(config.defaultRoute)) {
-			return { route: "none", source: "none", modelKey, reason: "unconfigured" };
-		}
-		return selectedRoute({ route: config.defaultRoute, source: "default", model, modelKey });
-	}
-
-	if (
-		modelKey &&
-		isExactModelAllowed(model, config.models) &&
-		(model?.api === VIRTUAL_MODEL_API || isWebSearchCapableApi(model?.api))
-	) {
-		return selectedRoute({ route: "hosted", source: "legacy", model, modelKey });
-	}
-
-	return { route: "none", source: "none", modelKey, reason: "unconfigured" };
-}
-
-/** Compatibility predicate for callers that specifically need the hosted path. */
-export function isWebSearchEnabledForModel(
-	model: WebSearchModel | undefined,
-	config: WebSearchConfig,
-): boolean {
-	const resolution = resolveWebSearchRoute({ model, config });
-	return resolution.route === "hosted" && resolution.available;
-}
+import { VIRTUAL_MODEL_API } from "../runtime";
+import { getExactModelKey, isExactModelAllowed } from "../model-scope";
+import type { WebSearchConfig, WebSearchRoute } from "../types";
+
+export const WEB_SEARCH_CAPABLE_APIS = [
+	"openai-responses",
+	"openai-codex-responses",
+	"azure-openai-responses",
+] as const;
+/** The provider-relative alpha endpoint is only verified for OpenAI/Codex routes. */
+export const WEB_SEARCH_STANDALONE_APIS = ["openai-responses", "openai-codex-responses"] as const;
+export const WEB_SEARCH_SOURCE_INCLUDE = "web_search_call.action.sources";
+export const WEB_SEARCH_PROMPT_MARKER = "<!-- pi-openai-toolkit:web-search -->";
+export const WEB_RUN_TOOL_NAME = "web_run";
+export const LOCAL_WEB_SEARCH_TOOL_NAME = "web_search";
+
+export type WebSearchCapableApi = (typeof WEB_SEARCH_CAPABLE_APIS)[number];
+export type WebSearchStandaloneApi = (typeof WEB_SEARCH_STANDALONE_APIS)[number];
+export type WebSearchRouteSource = "exact" | "default" | "legacy" | "none";
+export type WebSearchRouteUnavailableReason = "missing-model" | "unsupported-api" | "invalid-route";
+
+export type WebSearchModel = {
+	api?: string;
+	provider?: string;
+	id?: string;
+};
+
+export type WebSearchRouteResolution =
+	| {
+			route: "none";
+			source: "none";
+			modelKey?: string;
+			reason: "disabled" | "unconfigured";
+	  }
+	| {
+			route: WebSearchRoute;
+			source: Exclude<WebSearchRouteSource, "none">;
+			modelKey?: string;
+			available: boolean;
+			reason?: WebSearchRouteUnavailableReason;
+	  };
+
+export type WebSearchPayloadOutcome =
+	| "disabled"
+	| "unsupported-model"
+	| "non-object-payload"
+	| "invalid-tools"
+	| "invalid-include"
+	| "existing-native-tool"
+	| "injected-native-tool"
+	| "local-route"
+	| "standalone-route"
+	| "removed-conflicting-tools"
+	| "unavailable-route";
+
+export type WebSearchPayloadTransform = {
+	payload: unknown;
+	outcome: WebSearchPayloadOutcome;
+	changed: boolean;
+	/** Explicit route failures must abort rather than silently select another search path. */
+	fatal?: boolean;
+	errorMessage?: string;
+};
+
+export function getWebSearchModelKey(model: WebSearchModel | undefined): string | undefined {
+	return getExactModelKey(model);
+}
+
+export function isWebSearchRoute(value: unknown): value is WebSearchRoute {
+	return value === "local" || value === "hosted" || value === "standalone-alpha";
+}
+
+function selectedRoute(args: {
+	route: WebSearchRoute;
+	source: Exclude<WebSearchRouteSource, "none">;
+	model: WebSearchModel | undefined;
+	modelKey: string | undefined;
+}): WebSearchRouteResolution {
+	if (!args.modelKey) {
+		return {
+			route: args.route,
+			source: args.source,
+			available: false,
+			reason: "missing-model",
+		};
+	}
+	if (
+		args.route !== "local" &&
+		args.model?.api !== VIRTUAL_MODEL_API &&
+		(args.route === "standalone-alpha"
+			? !isWebSearchStandaloneCapableApi(args.model?.api)
+			: !isWebSearchCapableApi(args.model?.api))
+	) {
+		return {
+			route: args.route,
+			source: args.source,
+			modelKey: args.modelKey,
+			available: false,
+			reason: "unsupported-api",
+		};
+	}
+	return {
+		route: args.route,
+		source: args.source,
+		modelKey: args.modelKey,
+		available: true,
+	};
+}
+
+export function isWebSearchCapableApi(api: unknown): api is WebSearchCapableApi {
+	return typeof api === "string" && (WEB_SEARCH_CAPABLE_APIS as readonly string[]).includes(api);
+}
+
+export function isWebSearchStandaloneCapableApi(api: unknown): api is WebSearchStandaloneApi {
+	return typeof api === "string" && (WEB_SEARCH_STANDALONE_APIS as readonly string[]).includes(api);
+}
+
+/**
+ * Resolve exactly one configured Web Search route. The legacy model allowlist is
+ * deliberately consulted only after explicit route fields so old configs retain
+ * hosted behavior without becoming an implicit alpha-search opt-in.
+ */
+export function resolveWebSearchRoute(args: {
+	model: WebSearchModel | undefined;
+	config: WebSearchConfig;
+}): WebSearchRouteResolution {
+	const { model, config } = args;
+	const modelKey = getWebSearchModelKey(model);
+	if (!config.enabled) {
+		return { route: "none", source: "none", modelKey, reason: "disabled" };
+	}
+
+	const exactRoute = modelKey ? config.routes?.[modelKey] : undefined;
+	if (exactRoute !== undefined) {
+		if (!isWebSearchRoute(exactRoute)) {
+			return { route: "none", source: "none", modelKey, reason: "unconfigured" };
+		}
+		return selectedRoute({ route: exactRoute, source: "exact", model, modelKey });
+	}
+
+	if (config.defaultRoute !== undefined) {
+		if (!isWebSearchRoute(config.defaultRoute)) {
+			return { route: "none", source: "none", modelKey, reason: "unconfigured" };
+		}
+		return selectedRoute({ route: config.defaultRoute, source: "default", model, modelKey });
+	}
+
+	if (
+		modelKey &&
+		isExactModelAllowed(model, config.models) &&
+		(model?.api === VIRTUAL_MODEL_API || isWebSearchCapableApi(model?.api))
+	) {
+		return selectedRoute({ route: "hosted", source: "legacy", model, modelKey });
+	}
+
+	return { route: "none", source: "none", modelKey, reason: "unconfigured" };
+}
+
+/** Compatibility predicate for callers that specifically need the hosted path. */
+export function isWebSearchEnabledForModel(
+	model: WebSearchModel | undefined,
+	config: WebSearchConfig,
+): boolean {
+	const resolution = resolveWebSearchRoute({ model, config });
+	return resolution.route === "hosted" && resolution.available;
+}

package/tsconfig.check.json

@@ -1,50 +1,50 @@
-{
-  "compilerOptions": {
-    "noEmit": true,
-    "module": "ESNext",
-    "moduleResolution": "Bundler",
-    "allowImportingTsExtensions": true,
-    "target": "ES2022",
-    "lib": ["ES2022", "DOM", "DOM.Iterable"],
-    "skipLibCheck": true,
-    "strict": true,
-    "baseUrl": ".",
-    "typeRoots": ["./node_modules/@types"]
-  },
-  "include": [
-    "src/compact-client.ts",
-    "src/compaction-projection.ts",
-    "src/pi-context-hook.ts",
-    "src/config.ts",
-    "src/debug.ts",
-    "src/details-store.ts",
-    "src/deferred-tool-carryover.ts",
-    "src/extension-runtime.ts",
-    "src/image-generation/artifacts.ts",
-    "src/image-generation/client.ts",
-    "src/image-generation/eligibility.ts",
-    "src/image-generation/extension.ts",
-    "src/image-generation/protocol.ts",
-    "src/image-generation/references.ts",
-    "src/image-generation/render.ts",
-    "src/image-generation/service.ts",
-    "src/image-generation/types.ts",
-    "src/responses-websocket/extension.ts",
-    "src/responses-websocket/transport.ts",
-    "src/model-scope.ts",
-    "src/native-fallback.ts",
-    "src/payload-rewrite.ts",
-    "src/provider-headers.ts",
-    "src/responses-headers.ts",
-    "src/remote-v2-client.ts",
-    "src/request-context-cache.ts",
-    "src/runtime.ts",
-    "src/serializer.ts",
-    "src/supported-environment.ts",
-    "src/types.ts",
-    "src/web-search/extension.ts",
-    "src/web-search/payload.ts",
-    "src/web-search/prompt.ts",
-    "src/web-search/types.ts"
-  ]
-}
+{
+  "compilerOptions": {
+    "noEmit": true,
+    "module": "ESNext",
+    "moduleResolution": "Bundler",
+    "allowImportingTsExtensions": true,
+    "target": "ES2022",
+    "lib": ["ES2022", "DOM", "DOM.Iterable"],
+    "skipLibCheck": true,
+    "strict": true,
+    "baseUrl": ".",
+    "typeRoots": ["./node_modules/@types"]
+  },
+  "include": [
+    "src/compact-client.ts",
+    "src/compaction-projection.ts",
+    "src/pi-context-hook.ts",
+    "src/config.ts",
+    "src/debug.ts",
+    "src/details-store.ts",
+    "src/deferred-tool-carryover.ts",
+    "src/extension-runtime.ts",
+    "src/image-generation/artifacts.ts",
+    "src/image-generation/client.ts",
+    "src/image-generation/eligibility.ts",
+    "src/image-generation/extension.ts",
+    "src/image-generation/protocol.ts",
+    "src/image-generation/references.ts",
+    "src/image-generation/render.ts",
+    "src/image-generation/service.ts",
+    "src/image-generation/types.ts",
+    "src/responses-websocket/extension.ts",
+    "src/responses-websocket/transport.ts",
+    "src/model-scope.ts",
+    "src/native-fallback.ts",
+    "src/payload-rewrite.ts",
+    "src/provider-headers.ts",
+    "src/responses-headers.ts",
+    "src/remote-v2-client.ts",
+    "src/request-context-cache.ts",
+    "src/runtime.ts",
+    "src/serializer.ts",
+    "src/supported-environment.ts",
+    "src/types.ts",
+    "src/web-search/extension.ts",
+    "src/web-search/payload.ts",
+    "src/web-search/prompt.ts",
+    "src/web-search/types.ts"
+  ]
+}