pi-lean-dimension 0.2.1 → 0.2.2

package/node_modules/pi-lean-portal/backends/python-adapter.ts

@@ -10,7 +10,10 @@ import { join, delimiter as pathDelimiter } from "node:path";
 
 import { sessionManager } from "../core/shared/session-manager.js";
 import { persistSessionState } from "../core/shared/storage-state.js";
-import { DEFAULT_BACKENDS_ROOT } from "../core/plugin-config.js";
+import {
+	DEFAULT_BACKENDS_ROOT,
+	loadFullConfig,
+} from "../core/plugin-config.js";
 
 import {
 	DEFAULT_CAPABILITIES,
@@ -957,7 +960,8 @@ export class PythonPluginAdapter implements BrowserPlugin {
 				skip_default_viewport: !!raw.skip_default_viewport,
 				skip_networkidle: !!raw.skip_networkidle,
 				wrap_mw_eval_in_eval: !!raw.wrap_mw_eval_in_eval,
-				csp_safe_readonly_via_init_script: !!raw.csp_safe_readonly_via_init_script,
+				csp_safe_readonly_via_init_script:
+					!!raw.csp_safe_readonly_via_init_script,
 			}),
 			(error): QuirksDescriptor => ({
 				success: false,
@@ -1116,6 +1120,7 @@ export class PythonPluginAdapter implements BrowserPlugin {
 				};
 			},
 			"via Python bridge",
+			loadFullConfig().browser.maxStorageStateSize,
 		);
 	}
 

package/node_modules/pi-lean-portal/core/guides.ts

@@ -90,8 +90,7 @@ export const BUILTIN_GUIDES: Record<string, Guide> = {
 			"",
 			"### Backend Strategy",
 			"- The default `chromium` / `firefox` backends are detected by many anti-automation systems",
-			'- A stealth browser backend may be available — try `browser-navigate` with `strategy="stealth"` if the default backend is blocked',
-			"- If no stealth backend is configured, this will fail with a clear error — no harm in trying",
+			'- A stealth browser backend may be available — the `browser-navigate` `strategy` parameter lists the registered backend names; retry with a stealth backend name listed there (e.g. `strategy="camoufox"`) if the default is blocked',
 			"",
 			"### Verifying the Page After a Challenge",
 			'- Use `browser-inspect role="dialog"` to check if a challenge dialog is still present — cheaper than a full snapshot',

package/node_modules/pi-lean-portal/core/plugin-config.ts

@@ -8,6 +8,11 @@
 import { existsSync } from "node:fs";
 import { join, isAbsolute } from "node:path";
 import { USER_BACKENDS_DIR } from "./shared/paths.js";
+import {
+	sanitizeProfileName,
+	DEFAULT_MAX_STORAGE_STATE_SIZE,
+} from "./shared/storage-state.js";
+import { readMergedSettings } from "./shared/settings-reader.js";
 // ─── Plugin Config types ──────────────────────────────────────────
 
 /** A single plugin entry from the user's settings.json */
@@ -34,9 +39,6 @@ export interface PluginDetection {
 	/** Absolute or relative path to the entry point */
 	entryPoint: string;
 }
-import { sanitizeProfileName } from "./shared/storage-state.js";
-import { readMergedSettings } from "./shared/settings-reader.js";
-
 // ─── Config types & loading ────────────────────────────────────────
 
 /**
@@ -51,6 +53,12 @@ export interface BrowserConfig {
 	 * - A named profile string (e.g. "shopping", "work")
 	 */
 	defaultProfile: "none" | "session" | string;
+	/**
+	 * Byte threshold before a profile's saved storage state logs a size
+	 * warning (still saved). Defaults to 10 MB (`DEFAULT_MAX_STORAGE_STATE_SIZE`).
+	 * Set via `browser.maxStorageStateSize` in settings.json.
+	 */
+	maxStorageStateSize: number;
 }
 
 /** Raw plugin entry from settings.json (before validation) */
@@ -87,6 +95,7 @@ function parseBrowserConfig(
 	// Defaults
 	const config: BrowserConfig = {
 		defaultProfile: "session",
+		maxStorageStateSize: DEFAULT_MAX_STORAGE_STATE_SIZE,
 	};
 
 	if (!raw) return config;
@@ -114,6 +123,21 @@ function parseBrowserConfig(
 		}
 	}
 
+	// ── maxStorageStateSize ─────────────────────────────────────
+	if (raw.maxStorageStateSize !== undefined) {
+		if (
+			typeof raw.maxStorageStateSize === "number" &&
+			Number.isFinite(raw.maxStorageStateSize) &&
+			raw.maxStorageStateSize > 0
+		) {
+			config.maxStorageStateSize = Math.floor(raw.maxStorageStateSize);
+		} else {
+			errors.push(
+				`browser.maxStorageStateSize: expected a positive finite number, got ${JSON.stringify(raw.maxStorageStateSize)}`,
+			);
+		}
+	}
+
 	// Log validation errors (non-fatal, but surfaced in return)
 	for (const err of errors) {
 		console.warn(`[pi-lean-portal] Config warning: ${err}`);

package/node_modules/pi-lean-portal/core/router.ts

@@ -580,7 +580,8 @@ export async function navigate(
 				elementCount: 0,
 				error:
 					"Page appears to be blocked by anti-automation protection. " +
-					"The page may require JavaScript execution or browser interaction to render.",
+					"Retry `browser-navigate` with a stealth backend name from the `strategy` parameter's listed backends, " +
+					"or try `web-fetch` for the raw HTML (it skips JS fingerprinting but loses interactivity).",
 				backendUsed: plugin.name,
 				botDetectionWarning: true,
 			} as NavigateResult & {

package/node_modules/pi-lean-portal/core/shared/storage-state.ts

@@ -32,7 +32,7 @@ export const PROFILE_DIR = join(PORTAL_DATA_DIR, "browser-state");
 const STORAGE_STATE_VERSION = 1;
 
 /** Default size limit (10 MB) before a warning is logged on save. */
-const DEFAULT_MAX_STORAGE_STATE_SIZE = 10 * 1024 * 1024;
+export const DEFAULT_MAX_STORAGE_STATE_SIZE = 10 * 1024 * 1024;
 
 /** Profile name validation regex. */
 const PROFILE_NAME_RE = /^[a-zA-Z0-9_-]{1,64}$/;
@@ -468,6 +468,9 @@ export function deleteStorageState(profileName: string): void {
  * @param getStorageState - Callback that produces the raw storage state.
  * @param viaLabel - Optional backend label inserted into the warning
  *                   (e.g. `"via Python bridge"`) for diagnostic output.
+ * @param maxSizeBytes - Optional byte threshold for the size warning
+ *                       (defaults to `DEFAULT_MAX_STORAGE_STATE_SIZE` when
+ *                       omitted).
  * @returns The raw state object, or `undefined` if the session is
  *          non-persistent or the save failed.
  */
@@ -475,12 +478,13 @@ export async function persistSessionState(
 	session: { persistState?: boolean; profileName?: string } | null | undefined,
 	getStorageState: () => Promise<{ cookies: unknown[]; origins: unknown[] }>,
 	viaLabel: string = "",
+	maxSizeBytes?: number,
 ): Promise<{ cookies: unknown[]; origins: unknown[] } | undefined> {
 	if (!session?.persistState) return undefined;
 	const name = session.profileName ?? "default";
 	try {
 		const state = await getStorageState();
-		saveStorageState(name, state);
+		saveStorageState(name, state, maxSizeBytes);
 		return state;
 	} catch (err) {
 		console.warn(

package/node_modules/pi-lean-portal/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lean-portal",
-	"version": "0.2.1",
+	"version": "0.2.2",
 	"description": "Pi extension. Interactive web browsing for Pi — a /web toggle removes the tools from context when switched off, Playwright Chromium/Firefox deliver accessibility-tree snapshots, persistent profiles, cookies, and domain-matched guides; custom/stealth backends (Camoufox) plug in when a site blocks the shipped browsers.",
 	"keywords": [
 		"pi-package",

package/node_modules/pi-lean-search/__tests__/web-search.test.ts

@@ -5,7 +5,7 @@
  * instance and are not included here.
  */
 
-import { describe, it, expect, vi, beforeEach } from "vitest";
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
 import { readSearxngUrl } from "../search-config.js";
 import { webSearchTool } from "../web-search-tool.js";
 
@@ -150,4 +150,244 @@ describe("webSearchTool", () => {
 	it("has renderResult function", () => {
 		expect(typeof webSearchTool.renderResult).toBe("function");
 	});
+
+	describe("renderResult answer badge", () => {
+		it("shows 💡 badge when answerCount > 0 and no results", () => {
+			const mockTheme = {
+				fg: (_style: string, text: string) => text,
+			} as any;
+			const result = (webSearchTool.renderResult as any)(
+				{
+					content: [{ text: "" }],
+					details: { answerCount: 2, query: "weather" },
+				} as any,
+				{ expanded: false, isPartial: false },
+				mockTheme,
+				{} as any,
+			) as any;
+			expect(result.text).toContain("💡");
+			expect(result.text).toContain("2 answer(s)");
+			expect(result.text).toContain('"weather"');
+		});
+
+		it("shows 💡 badge alongside 🔍 when both answers and results present", () => {
+			const mockTheme = {
+				fg: (_style: string, text: string) => text,
+			} as any;
+			const result = (webSearchTool.renderResult as any)(
+				{
+					content: [{ text: "" }],
+					details: {
+						answerCount: 1,
+						resultCount: 3,
+						query: "test",
+						results: [
+							{ title: "A", url: "http://a.com", engine: "google" },
+							{ title: "B", url: "http://b.com", engine: "google" },
+							{ title: "C", url: "http://c.com", engine: "google" },
+						],
+					},
+				} as any,
+				{ expanded: false, isPartial: false },
+				mockTheme,
+				{} as any,
+			) as any;
+			expect(result.text).toContain("💡");
+			expect(result.text).toContain("1 answer(s)");
+			expect(result.text).toContain("🔍");
+			expect(result.text).toContain("3 result(s)");
+		});
+
+		it("shows answer text above results when expanded", () => {
+			const mockTheme = {
+				fg: (_style: string, text: string) => text,
+			} as any;
+			const result = (webSearchTool.renderResult as any)(
+				{
+					content: [{ text: "" }],
+					details: {
+						answerCount: 1,
+						resultCount: 1,
+						query: "weather berlin",
+						answers: [
+							{
+								template: "answer/weather.html",
+								text: "Berlin: 14°C, light rain",
+							},
+						],
+						results: [{ title: "A", url: "http://a.com" }],
+					},
+				} as any,
+				{ expanded: true, isPartial: false },
+				mockTheme,
+				{} as any,
+			) as any;
+			const answerIdx = result.text.indexOf("Berlin: 14°C, light rain");
+			const resultIdx = result.text.indexOf("A");
+			expect(answerIdx).toBeGreaterThan(-1);
+			expect(resultIdx).toBeGreaterThan(-1);
+			expect(answerIdx).toBeLessThan(resultIdx);
+		});
+
+		it("shows answer text when expanded with no results", () => {
+			const mockTheme = {
+				fg: (_style: string, text: string) => text,
+			} as any;
+			const result = (webSearchTool.renderResult as any)(
+				{
+					content: [{ text: "" }],
+					details: {
+						answerCount: 1,
+						query: "avg 1 2 3",
+						answers: [{ template: "answer/legacy.html", text: "2" }],
+					},
+				} as any,
+				{ expanded: true, isPartial: false },
+				mockTheme,
+				{} as any,
+			) as any;
+			expect(result.text).toContain("💡");
+			expect(result.text).toContain("2");
+		});
+	});
+});
+
+// ─── Execute answer rendering (mocked fetch) ─────────────────
+
+describe("execute answer rendering", () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+		// Make readSearxngUrl return a valid URL
+		vi.mocked(existsSync).mockReturnValue(true);
+		vi.mocked(readFileSync).mockReturnValue(
+			JSON.stringify({ searxng: { url: "http://localhost:8888" } }),
+		);
+	});
+
+	afterEach(() => {
+		vi.unstubAllGlobals();
+	});
+
+	const mockFetch = (json: unknown) =>
+		vi.stubGlobal(
+			"fetch",
+			vi.fn().mockResolvedValue({
+				ok: true,
+				status: 200,
+				text: async () => JSON.stringify(json),
+			}),
+		);
+
+	it("legacy answer renders + bypasses empty-results early return", async () => {
+		mockFetch({
+			results: [],
+			answers: [{ template: "answer/legacy.html", answer: "614.24" }],
+			suggestions: [],
+		});
+
+		const result = await webSearchTool.execute(
+			"call-1",
+			{ query: "614.24" },
+			undefined,
+			undefined,
+			{} as any,
+		);
+		const r = result as any;
+		expect(r.content[0].text).toContain("614.24");
+		expect(r.content[0].text).not.toContain("No web search results found");
+		expect(r.content[0].text).toContain("No web results");
+	});
+
+	it("weather answer uses server summary", async () => {
+		mockFetch({
+			results: [],
+			answers: [
+				{
+					template: "answer/weather.html",
+					current: {
+						location: { name: "Berlin", latitude: 52.52, longitude: 13.405 },
+						temperature: { val: 14, unit: "°C" },
+						condition: "light rain",
+						summary: "Berlin: 14°C, light rain",
+						feels_like: { val: 12, unit: "°C" },
+						humidity: { val: 80, unit: "%" },
+						wind_speed: { val: 18, unit: "km/h" },
+						wind_from: { name: "NW" },
+					},
+					service: "open-meteo",
+				},
+			],
+			suggestions: [],
+		});
+
+		const result = await webSearchTool.execute(
+			"call-2",
+			{ query: "weather berlin" },
+			undefined,
+			undefined,
+			{} as any,
+		);
+		// Uses server-provided summary
+		const r = result as any;
+		expect(r.content[0].text).toContain("Berlin: 14°C, light rain");
+		// Secondary detail line
+		expect(r.content[0].text).toContain("feels like 12°C");
+		expect(r.content[0].text).toContain("humidity 80%");
+		expect(r.content[0].text).toContain("wind 18km/h");
+		// forecasts not rendered
+		expect(r.content[0].text).not.toContain("forecast");
+		// source
+		expect(r.content[0].text).toContain("open-meteo");
+	});
+
+	it("unknown template falls back to legacy heuristic", async () => {
+		mockFetch({
+			results: [],
+			answers: [{ template: "answer/future.html", answer: "x" }],
+			suggestions: [],
+		});
+
+		const result = await webSearchTool.execute(
+			"call-3",
+			{ query: "future" },
+			undefined,
+			undefined,
+			{} as any,
+		);
+		const r = result as any;
+		expect(r.content[0].text).toContain("x");
+	});
+
+	it("answers + results both present (primary success path)", async () => {
+		mockFetch({
+			results: [
+				{
+					title: "Result 1",
+					url: "http://example.com/1",
+					content: "First result",
+					engine: "google",
+					score: 0.95,
+				},
+			],
+			answers: [{ template: "answer/legacy.html", answer: "42" }],
+			suggestions: [],
+		});
+
+		const result = await webSearchTool.execute(
+			"call-5",
+			{ query: "answer of life" },
+			undefined,
+			undefined,
+			{} as any,
+		);
+		// Answer above results
+		const r = result as any;
+		const answerIdx = r.content[0].text.indexOf("42");
+		const resultIdx = r.content[0].text.indexOf("Result 1");
+		expect(answerIdx).toBeGreaterThanOrEqual(0);
+		expect(resultIdx).toBeGreaterThan(answerIdx);
+		// Both present
+		expect(r.content[0].text).toContain("42");
+		expect(r.content[0].text).toContain("Result 1");
+	});
 });

package/node_modules/pi-lean-search/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lean-search",
-	"version": "0.2.1",
+	"version": "0.2.2",
 	"description": "Pi extension. SearXNG search tool for Pi — pairs with pi-lean-portal's /web toggle; search-only installs are valid, or add it to a portal install for the full web-tools suite.",
 	"keywords": [
 		"pi-package",