@juicesharp/rpiv-web-tools 0.1.2 → 0.7.0

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 juicesharp
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -4,7 +4,7 @@ Pi extension that registers the `web_search` and `web_fetch` tools, backed by
 the Brave Search API. Also ships `/web-search-config` for interactive API
 key configuration.
 
-![Brave Search API key prompt](https://raw.githubusercontent.com/juicesharp/rpiv-web-tools/main/docs/config.jpg)
+![Brave Search API key prompt](https://raw.githubusercontent.com/juicesharp/rpiv-mono/main/packages/rpiv-web-tools/docs/config.jpg)
 
 ## Installation
 

package/index.ts

@@ -13,20 +13,20 @@
  * Use the /web-search-config slash command to set the key interactively.
  */
 
+import { chmodSync, existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import { mkdtemp, writeFile } from "node:fs/promises";
+import { homedir, tmpdir } from "node:os";
+import { dirname, join } from "node:path";
 import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
 import {
 	DEFAULT_MAX_BYTES,
 	DEFAULT_MAX_LINES,
 	formatSize,
-	truncateHead,
 	type TruncationResult,
+	truncateHead,
 } from "@mariozechner/pi-coding-agent";
 import { Text } from "@mariozechner/pi-tui";
 import { Type } from "@sinclair/typebox";
-import { chmodSync, existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
-import { mkdtemp, writeFile } from "node:fs/promises";
-import { homedir, tmpdir } from "node:os";
-import { dirname, join } from "node:path";
 
 // ---------------------------------------------------------------------------
 // Config file persistence
@@ -49,7 +49,7 @@ function loadConfig(): WebToolsConfig {
 
 function saveConfig(config: WebToolsConfig): void {
 	mkdirSync(dirname(CONFIG_PATH), { recursive: true });
-	writeFileSync(CONFIG_PATH, JSON.stringify(config, null, 2) + "\n", "utf-8");
+	writeFileSync(CONFIG_PATH, `${JSON.stringify(config, null, 2)}\n`, "utf-8");
 	try {
 		chmodSync(CONFIG_PATH, 0o600);
 	} catch {
@@ -59,9 +59,9 @@ function saveConfig(config: WebToolsConfig): void {
 
 function resolveApiKey(): string | undefined {
 	const envKey = process.env.BRAVE_SEARCH_API_KEY;
-	if (envKey && envKey.trim()) return envKey.trim();
+	if (envKey?.trim()) return envKey.trim();
 	const config = loadConfig();
-	if (config.apiKey && config.apiKey.trim()) return config.apiKey.trim();
+	if (config.apiKey?.trim()) return config.apiKey.trim();
 	return undefined;
 }
 
@@ -80,16 +80,10 @@ interface SearchResponse {
 	query: string;
 }
 
-async function searchBrave(
-	query: string,
-	maxResults: number,
-	signal?: AbortSignal,
-): Promise<SearchResponse> {
+async function searchBrave(query: string, maxResults: number, signal?: AbortSignal): Promise<SearchResponse> {
 	const apiKey = resolveApiKey();
 	if (!apiKey) {
-		throw new Error(
-			"BRAVE_SEARCH_API_KEY is not set. Run /web-search-config to configure, or export the env var.",
-		);
+		throw new Error("BRAVE_SEARCH_API_KEY is not set. Run /web-search-config to configure, or export the env var.");
 	}
 
 	const url = new URL("https://api.search.brave.com/res/v1/web/search");
@@ -175,8 +169,8 @@ export default function (pi: ExtensionAPI) {
 		promptSnippet: "Search the web for up-to-date information via Brave",
 		promptGuidelines: [
 			"Use web_search for information beyond your training data — recent events, current library versions, live API documentation.",
-			"Use the current year from \"Current date:\" in your context when searching for recent information or documentation.",
-			"After answering using search results, include a \"Sources:\" section listing relevant URLs as markdown hyperlinks: [Title](URL). Never skip this.",
+			'Use the current year from "Current date:" in your context when searching for recent information or documentation.',
+			'After answering using search results, include a "Sources:" section listing relevant URLs as markdown hyperlinks: [Title](URL). Never skip this.',
 			"Domain filtering is supported to include or block specific websites.",
 			"If BRAVE_SEARCH_API_KEY is not set, ask the user to run /web-search-config before proceeding.",
 		],
@@ -199,51 +193,38 @@ export default function (pi: ExtensionAPI) {
 
 			onUpdate?.({
 				content: [{ type: "text", text: `Searching Brave for: "${params.query}"...` }],
+				details: { query: params.query, backend: "brave", resultCount: 0 },
 			});
 
-			try {
-				const response = await searchBrave(params.query, maxResults, signal);
-
-				if (response.results.length === 0) {
-					return {
-						content: [
-							{
-								type: "text",
-								text: `No results found for "${params.query}".`,
-							},
-						],
-						details: { query: params.query, backend: "brave", resultCount: 0 },
-					};
-				}
-
-				let text = `**Search results for "${response.query}":**\n\n`;
-				for (let i = 0; i < response.results.length; i++) {
-					const r = response.results[i];
-					text += `${i + 1}. **${r.title}**\n   ${r.url}\n   ${r.snippet}\n\n`;
-				}
+			const response = await searchBrave(params.query, maxResults, signal);
 
-				return {
-					content: [{ type: "text", text: text.trimEnd() }],
-					details: {
-						query: params.query,
-						backend: "brave",
-						resultCount: response.results.length,
-						results: response.results,
-					},
-				};
-			} catch (err: unknown) {
-				const message = err instanceof Error ? err.message : String(err);
+			if (response.results.length === 0) {
 				return {
 					content: [
 						{
 							type: "text",
-							text: `Web search failed: ${message}`,
+							text: `No results found for "${params.query}".`,
 						},
 					],
-					isError: true,
-					details: { query: params.query, backend: "brave", error: message },
+					details: { query: params.query, backend: "brave", resultCount: 0 },
 				};
 			}
+
+			let text = `**Search results for "${response.query}":**\n\n`;
+			for (let i = 0; i < response.results.length; i++) {
+				const r = response.results[i];
+				text += `${i + 1}. **${r.title}**\n   ${r.url}\n   ${r.snippet}\n\n`;
+			}
+
+			return {
+				content: [{ type: "text", text: text.trimEnd() }],
+				details: {
+					query: params.query,
+					backend: "brave",
+					resultCount: response.results.length,
+					results: response.results,
+				},
+			};
 		},
 
 		renderCall(args, theme, _context) {
@@ -257,9 +238,6 @@ export default function (pi: ExtensionAPI) {
 				return new Text(theme.fg("warning", "Searching..."), 0, 0);
 			}
 			const details = result.details as { resultCount?: number; results?: SearchResult[] };
-			if (result.isError) {
-				return new Text(theme.fg("error", "✗ Search failed"), 0, 0);
-			}
 			const count = details?.resultCount ?? 0;
 			let text = theme.fg("success", `✓ ${count} result${count !== 1 ? "s" : ""}`);
 			if (expanded && details?.results) {
@@ -296,7 +274,7 @@ export default function (pi: ExtensionAPI) {
 		promptGuidelines: [
 			"Use web_fetch to read the full content of a specific URL — documentation pages, blog posts, API references found via web_search.",
 			"web_fetch is complementary to web_search: search finds URLs, fetch reads them.",
-			"After answering using fetched content, include a \"Sources:\" section with a markdown hyperlink to the fetched URL.",
+			'After answering using fetched content, include a "Sources:" section with a markdown hyperlink to the fetched URL.',
 			"Large responses are truncated and spilled to a temp file — the temp path is reported in the result details.",
 		],
 		parameters: Type.Object({
@@ -321,13 +299,12 @@ export default function (pi: ExtensionAPI) {
 				throw new Error(`Invalid URL: ${url}`);
 			}
 			if (!["http:", "https:"].includes(parsedUrl.protocol)) {
-				throw new Error(
-					`Unsupported URL protocol: ${parsedUrl.protocol}. Only http and https are supported.`,
-				);
+				throw new Error(`Unsupported URL protocol: ${parsedUrl.protocol}. Only http and https are supported.`);
 			}
 
 			onUpdate?.({
 				content: [{ type: "text", text: `Fetching: ${url}...` }],
+				details: { url } as FetchDetails,
 			});
 
 			const res = await fetch(url, {
@@ -335,8 +312,7 @@ export default function (pi: ExtensionAPI) {
 				redirect: "follow",
 				headers: {
 					"User-Agent": "Mozilla/5.0 (compatible; rpiv-pi/1.0)",
-					Accept:
-						"text/html,application/xhtml+xml,application/xml;q=0.9,text/plain;q=0.8,*/*;q=0.5",
+					Accept: "text/html,application/xhtml+xml,application/xml;q=0.9,text/plain;q=0.8,*/*;q=0.5",
 				},
 			});
 
@@ -347,11 +323,7 @@ export default function (pi: ExtensionAPI) {
 			const contentType = res.headers.get("content-type") ?? "";
 			const contentLength = res.headers.get("content-length");
 
-			if (
-				contentType.includes("image/") ||
-				contentType.includes("video/") ||
-				contentType.includes("audio/")
-			) {
+			if (contentType.includes("image/") || contentType.includes("video/") || contentType.includes("audio/")) {
 				throw new Error(`Unsupported content type: ${contentType}. web_fetch supports text pages only.`);
 			}
 
@@ -417,9 +389,6 @@ export default function (pi: ExtensionAPI) {
 			if (isPartial) {
 				return new Text(theme.fg("warning", "Fetching..."), 0, 0);
 			}
-			if (result.isError) {
-				return new Text(theme.fg("error", "✗ Fetch failed"), 0, 0);
-			}
 			const details = result.details as FetchDetails | undefined;
 			let text = theme.fg("success", "✓ Fetched");
 			if (details?.title) {
@@ -460,9 +429,7 @@ export default function (pi: ExtensionAPI) {
 			const showMode = typeof args === "string" && args.includes("--show");
 
 			if (showMode) {
-				const masked = current.apiKey
-					? `${current.apiKey.slice(0, 4)}...${current.apiKey.slice(-4)}`
-					: "(not set)";
+				const masked = current.apiKey ? `${current.apiKey.slice(0, 4)}...${current.apiKey.slice(-4)}` : "(not set)";
 				const envMasked = process.env.BRAVE_SEARCH_API_KEY
 					? `${process.env.BRAVE_SEARCH_API_KEY.slice(0, 4)}...${process.env.BRAVE_SEARCH_API_KEY.slice(-4)}`
 					: "(not set)";

package/package.json

@@ -1,24 +1,38 @@
 {
   "name": "@juicesharp/rpiv-web-tools",
-  "version": "0.1.2",
+  "version": "0.7.0",
   "description": "Pi extension: web_search + web_fetch via the Brave Search API",
-  "keywords": ["pi-package", "pi-extension", "rpiv", "web-search", "brave"],
+  "keywords": [
+    "pi-package",
+    "pi-extension",
+    "rpiv",
+    "web-search",
+    "brave"
+  ],
   "type": "module",
   "license": "MIT",
   "author": "juicesharp",
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/juicesharp/rpiv-web-tools.git"
+    "url": "git+https://github.com/juicesharp/rpiv-mono.git",
+    "directory": "packages/rpiv-web-tools"
   },
-  "homepage": "https://github.com/juicesharp/rpiv-web-tools#readme",
+  "homepage": "https://github.com/juicesharp/rpiv-mono/tree/main/packages/rpiv-web-tools#readme",
   "bugs": {
-    "url": "https://github.com/juicesharp/rpiv-web-tools/issues"
+    "url": "https://github.com/juicesharp/rpiv-mono/issues"
   },
   "publishConfig": {
     "access": "public"
   },
+  "files": [
+    "index.ts",
+    "README.md",
+    "LICENSE"
+  ],
   "pi": {
-    "extensions": ["./index.ts"]
+    "extensions": [
+      "./index.ts"
+    ]
   },
   "peerDependencies": {
     "@mariozechner/pi-coding-agent": "*",