@zeldrisho/pi-web-search 0.14.3 → 0.14.5

package/CHANGELOG.md

@@ -7,6 +7,23 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.14.5] - 2026-11-01
+
+### Changed
+
+- Shorten search guidance, error, and output-limit notices.
+
+### Fixed
+
+- Avoid duplicate overflow notices in expanded search results.
+
+## [0.14.4] - 2026-10-30
+
+### Changed
+
+- Remove the package's Node.js engine restriction; Node.js 24 remains the tested runtime.
+- Simplify the model-facing tool description by removing the instruction to use web fetch for sources.
+
 ## [0.14.3] - 2026-10-28
 
 ### Changed
@@ -189,7 +206,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Add bounded Brave web search ([cbc9491](https://github.com/zeldrisho/pi-packages/commit/cbc9491dce36555ab91b6bef203ec8b380596a89))
 
-[Unreleased]: https://github.com/zeldrisho/pi-packages/compare/pi-web-search-v0.14.3...HEAD
+[Unreleased]: https://github.com/zeldrisho/pi-packages/compare/pi-web-search-v0.14.5...HEAD
+[0.14.5]: https://github.com/zeldrisho/pi-packages/compare/pi-web-search-v0.14.4...pi-web-search-v0.14.5
+[0.14.4]: https://github.com/zeldrisho/pi-packages/compare/pi-web-search-v0.14.3...pi-web-search-v0.14.4
 [0.14.3]: https://github.com/zeldrisho/pi-packages/compare/pi-web-search-v0.14.2...pi-web-search-v0.14.3
 [0.14.2]: https://github.com/zeldrisho/pi-packages/compare/pi-web-search-v0.14.1...pi-web-search-v0.14.2
 [0.14.1]: https://github.com/zeldrisho/pi-packages/compare/pi-web-search-v0.14.0...pi-web-search-v0.14.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zeldrisho/pi-web-search",
-  "version": "0.14.3",
+  "version": "0.14.5",
   "description": "Pi extension for bounded Brave web search with generated-answer fallback",
   "keywords": [
     "brave-search",
@@ -34,17 +34,14 @@
     "@earendil-works/pi-tui": "^1.4.0",
     "typebox": "^1.4.2",
     "typescript": "^7.0.2",
-    "vite": "npm:@voidzero-dev/vite-plus-core@1.3.0",
-    "vite-plus": "1.3.0"
+    "vite": "npm:@voidzero-dev/vite-plus-core@1.4.0",
+    "vite-plus": "1.4.0"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",
     "@earendil-works/pi-tui": "*",
     "typebox": "*"
   },
-  "engines": {
-    "node": ">=24"
-  },
   "pi": {
     "extensions": [
       "./src/index.ts"

package/src/index.ts

@@ -46,7 +46,7 @@ export default function (pi: ExtensionAPI) {
   pi.registerTool<typeof webSearchParameters, Partial<SearchDetails>>({
     name: "web_search",
     label: "Web Search",
-    description: "Search the web. Use web_fetch to read sources.",
+    description: "Search the web.",
     parameters: webSearchParameters,
     outputSchema: webSearchOutputSchema,
     annotations: { readOnlyHint: true, destructiveHint: false, openWorldHint: true },
@@ -72,14 +72,26 @@ export default function (pi: ExtensionAPI) {
 
       if (isPartial) return new Text(theme.fg("warning", content?.text || "Searching…"), 0, 0);
 
+      const path = result.details?.fullOutputPath;
+      const lastLine = content?.text.trimEnd().split("\n").at(-1);
+
+      const overflowNotice =
+        path &&
+        lastLine?.endsWith(`Full output: ${path}]`) &&
+        (lastLine.startsWith("[Output truncated:") ||
+          lastLine.startsWith("[Structured payload omitted due to limits."))
+          ? lastLine
+          : undefined;
+
       return content?.type === "text"
         ? renderCollapsibleOutput(
             content.text,
             expanded,
             theme,
             context?.isError,
-            result.details?.fullOutputPath
-              ? `[${result.details.truncation?.truncated ? "Output truncated. " : "Structured payload omitted. "}Full output: ${result.details.fullOutputPath}]`
+            path
+              ? (overflowNotice ??
+                  `[${result.details?.truncation?.truncated ? "Output truncated. " : "Structured payload omitted. "}Full output: ${path}]`)
               : undefined,
           )
         : new Text(theme.fg("dim", "No results"), 0, 0);

package/src/provider.ts

@@ -146,15 +146,14 @@ export async function requestText(
         // Preserve the useful HTTP status when an untrusted error body cannot be read.
       }
 
-      throw new Error(`Search provider returned HTTP ${response.status}${body ? `: ${body}` : ""}`);
+      throw new Error(`Provider returned HTTP ${response.status}${body ? `: ${body}` : ""}`);
     }
 
     return await readResponseText(response, SEARCH_MAX_RESPONSE_BYTES);
   } catch (error) {
-    if (timedOut)
-      throw new Error(`Web search timed out after ${REQUEST_TIMEOUT_MS / 1000} seconds.`);
+    if (timedOut) throw new Error(`Timed out after ${REQUEST_TIMEOUT_MS / 1000} seconds.`);
 
-    if (signal?.aborted) throw new Error("Web search was cancelled.");
+    if (signal?.aborted) throw new Error("Cancelled.");
     throw error;
   } finally {
     clearTimeout(timeout);

package/src/render.ts

@@ -37,7 +37,7 @@ export function renderCollapsibleOutput(
     );
   }
 
-  if (warning) {
+  if (warning && !visible.includes(warning)) {
     component.addChild(new Spacer(1));
     component.addChild(new Text(theme.fg("warning", warning), 0, 0));
   }

package/src/search.ts

@@ -159,16 +159,13 @@ function escapeMarkdownLinkText(value: string): string {
 function formatResults(query: string, results: SearchResult[], requestedCount: number): string {
   const returnedCount = results.length;
 
-  const retryAdvice =
-    "If evidence is insufficient, fetch promising sources or broaden the query (fewer quotes; relax nonessential site: restrictions).";
-
   const guidance =
     returnedCount < requestedCount
-      ? `Returned ${returnedCount} of ${requestedCount} requested results. ${retryAdvice} Preserve required source restrictions.`
+      ? `Returned ${returnedCount}/${requestedCount} results. If needed, read relevant pages or broaden the query; keep required site restrictions.`
       : "";
 
   if (returnedCount === 0) {
-    return `No web results found for ${JSON.stringify(query)}. Broaden the query (fewer quotes; relax nonessential site: restrictions). Preserve required source restrictions. No results do not prove absence.`;
+    return `No web results for ${JSON.stringify(query)}. Try broadening the query; keep required site restrictions. No results do not prove absence.`;
   }
 
   const entries = results.map((result, index) => {
@@ -466,7 +463,7 @@ export class SearchRuntime {
           return entry;
         },
         signal,
-        "Web search was cancelled.",
+        "Cancelled.",
       ));
 
     if (payload.results.length === 0 && !payload.answer) {
@@ -508,7 +505,7 @@ export class SearchRuntime {
                 return entry;
               },
               signal,
-              "Web search was cancelled.",
+              "Cancelled.",
             ));
           cached = cachedAnswer !== undefined;
           apiKeySource = answerCredentials.source;
@@ -598,8 +595,8 @@ export class SearchRuntime {
       }
 
       text += truncation.truncated
-        ? `\n\n[Output truncated to ${truncation.outputLines} of ${truncation.totalLines} lines (${formatSize(truncation.outputBytes)} of ${formatSize(truncation.totalBytes)}). Full output saved to: ${fullOutputPath}]`
-        : `\n\n[Structured payload omitted to fit output limits. Full output saved to: ${fullOutputPath}]`;
+        ? `\n\n[Output truncated: ${truncation.outputLines}/${truncation.totalLines} lines (${formatSize(truncation.outputBytes)}/${formatSize(truncation.totalBytes)}). Full output: ${fullOutputPath}]`
+        : `\n\n[Structured payload omitted due to limits. Full output: ${fullOutputPath}]`;
     }
 
     structured.fullOutputPath = fullOutputPath;