@xynogen/pix-9router 0.3.10 → 0.3.14

package/README.md

@@ -35,7 +35,7 @@ export ROUTER_API_KEY="your-key-here"
 
 - **Provider**: on load, fetches `/models` from the router and registers them with Pi. The [modelgrep](https://modelgrep.com) catalog (via [`@xynogen/pix-data`](https://github.com/xynogen/pix-mono/tree/main/packages/pix-data)'s shared cache) is used internally to fill missing context window / modality fields where the router response omits them. Model list is cached at `~/.cache/pi/9router.json` (TTL 30 min).
 - **fetch / search**: POST to `/web/fetch` and `/search` on the router (which proxies to exa). If the router is unreachable, falls back to `curl` (for `fetch`, raw URL fetch; for `search`, the same `/search` endpoint via curl). Tool output is rendered dimmed so fetched web content reads like faded context rather than primary output.
-- **transcribe**: POST to `/audio/transcriptions` (Deepgram Nova 3 by default). Accepts any audio file path; falls back to curl.
+- **transcribe**: POST to `/audio/transcriptions` (Deepgram Nova 3 by default). Accepts any audio file path; falls back to curl. Pass the optional `output_file` to write the full transcription to disk (parent dirs created, path pre-flight checked) and return only a short path summary to the model — useful for long results; without it the text is returned inline (truncated to 50,000 chars).
 
 ## Compact results
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-9router",
-	"version": "0.3.10",
+	"version": "0.3.14",
 	"description": "Pi extension — 9Router provider + fetch/search tools via router API",
 	"type": "module",
 	"main": "src/index.ts",
@@ -43,9 +43,9 @@
 		"@earendil-works/pi-tui": "*"
 	},
 	"dependencies": {
-		"@xynogen/pix-data": "^0.4.0",
-		"@xynogen/pix-pretty": "^1.7.21",
+		"@xynogen/pix-data": "^0.4.3",
+		"@xynogen/pix-pretty": "^1.11.2",
 		"typebox": "^1.1.38",
-		"@xynogen/pix-runtime": "^0.4.0"
+		"@xynogen/pix-runtime": "^0.5.3"
 	}
 }

package/src/fetch.ts

@@ -241,10 +241,12 @@ export default function registerFetch(pi: ExtensionAPI): void {
 		),
 		parameters: Type.Object({
 			url: Type.String({ description: "URL to fetch" }),
-			format: StringEnum(["markdown", "text", "html"] as const, {
-				description:
-					'Required choice. Enter exactly "markdown" for readable Markdown, "text" for plain text, or "html" for raw HTML.',
-			}),
+			format: Type.Optional(
+				StringEnum(["markdown", "text", "html"] as const, {
+					description:
+						'Optional choice. Enter exactly "markdown" for readable Markdown, "text" for plain text, or "html" for raw HTML. Default is "markdown".',
+				}),
+			),
 			max_characters: Type.Optional(
 				Type.Number({
 					description: "Max characters (default 1000, 0 = unlimited)",

package/src/format.test.ts

@@ -101,6 +101,30 @@ describe("fetch execution metadata", () => {
 		expect(result.content).toEqual(originalContent);
 	});
 
+	it("defaults format to markdown when omitted (schema now optional)", async () => {
+		let sentFormat: unknown;
+		const raw = JSON.stringify({ url: "https://example.com", content: { text: "hello" } });
+		const result = await executeFetch(
+			// No `format` — valid now that the schema no longer requires it.
+			{ url: "https://example.com" },
+			undefined,
+			undefined,
+			{
+				apiPost: async (_path, body) => {
+					sentFormat = (body as { format?: unknown }).format;
+					return raw;
+				},
+				curl: async () => {
+					throw new Error("curl should not run");
+				},
+			},
+		);
+
+		// The default flows through to both the API request and the metadata.
+		expect(sentFormat).toBe("markdown");
+		expect(result.details).toMatchObject({ outcome: "success", format: "markdown" });
+	});
+
 	it("reports curl fallback metadata while preserving the fallback banner", async () => {
 		const result = await executeFetch(
 			{ url: "https://example.com", format: "html", max_characters: 1000 },

package/src/tool-schema.test.ts

@@ -4,6 +4,7 @@ import registerSearch from "./search.ts";
 
 function captureParameters(register: (pi: never) => void): {
 	properties: Record<string, { type?: string; enum?: string[]; description?: string }>;
+	required?: string[];
 } {
 	let parameters: unknown;
 	register({
@@ -14,6 +15,7 @@ function captureParameters(register: (pi: never) => void): {
 	if (!parameters) throw new Error("tool parameters not captured");
 	return parameters as {
 		properties: Record<string, { type?: string; enum?: string[]; description?: string }>;
+		required?: string[];
 	};
 }
 
@@ -38,3 +40,21 @@ describe("enum-like tool parameters", () => {
 		expect(format.description).toContain('"html"');
 	});
 });
+
+describe("fetch parameter requiredness", () => {
+	test("only url is required; format is optional and defaults to markdown", () => {
+		const params = captureParameters(registerFetch);
+
+		// url must be required, format must NOT be — this is the exact validator
+		// bug the optionality fix addresses (executeFetch already defaults format
+		// to "markdown"), so guard against a regression back to a required format.
+		expect(params.required).toEqual(["url"]);
+		expect(params.required).not.toContain("format");
+
+		// format stays a fully described enum in properties even while optional.
+		const format = params.properties.format;
+		if (!format) throw new Error("format schema not found");
+		expect(format.enum).toEqual(["markdown", "text", "html"]);
+		expect(format.description?.toLowerCase()).toContain("default");
+	});
+});