pi-lean-dimension 0.2.2 → 0.2.4

package/README.md

@@ -36,9 +36,9 @@ Pi settings (`~/.pi/agent/settings.json` or `.pi/settings.json`):
 |---|---|---|---|
 | **A — Browser** (recommended) | `pi install npm:pi-lean-portal` | 12 browser tools + `/web` command | `npx playwright install chromium firefox` |
 | **B — Full suite** | `pi install npm:pi-lean-dimension` | 13 tools (browser + search) + `/web` | Playwright browsers + SearXNG server |
-| **B-search — Search only** | `pi install npm:pi-lean-search` | `web-search` tool only | SearXNG server |
+| **C — Search only** | `pi install npm:pi-lean-search` | `web-search` tool only | SearXNG server |
 
-Notes the table doesn't carry:
+Notes the table doesn't cover:
 
 - **Browser binaries aren't downloaded during `npm install`** (configured via `.npmrc`). The first `browser-navigate` call prompts you to run `npx playwright install chromium firefox` if they're missing.
 - **SearXNG is optional for Mode B.** The browser works immediately without it; `web-search` returns a clear setup message on first call. When you do run it, point the suite at your instance in Pi settings:
@@ -100,7 +100,7 @@ When search is installed, two independent glyphs appear:
 Beyond the `/web` toggle, two surfaces are user-driven rather than hardcoded:
 
 - **Navigation guides** — `web-learn` saves site-specific playbooks that auto-match by domain and resurface in later sessions.
-- **Custom browser backends** — if a site blocks the shipped Chromium/Firefox, drop a `bridge.py` subclass into `~/.pi/agent/pi-lean-portal/user-backends/` and drive a patched engine like [Camoufox](https://github.com/daijro/camoufox) yourself. A quirks schema declares how the engine diverges from base Playwright, and `launch` options flow from `settings.json` to the subprocess at runtime. This is user-authored, user-audited code that the extension never auto-downloads — and as far as we're aware, no other Pi web plugin lets you run a browser backend you wrote yourself. Most installs never need it; the [portal README](packages/pi-lean-portal/README.md#stealth--custom-browser-backends) and [`contributed/README.md`](packages/pi-lean-portal/contributed/README.md) cover the full flow when you do.
+- **Custom browser backends** — if a site blocks the shipped Chromium/Firefox, drop a `bridge.py` subclass into `~/.pi/agent/pi-lean-portal/user-backends/` and drive a patched engine like [Camoufox](https://github.com/daijro/camoufox) yourself. A quirks schema declares how the engine diverges from base Playwright, and `launch` options flow from `settings.json` to the subprocess at runtime. This is user-authored, user-audited code that the extension never auto-downloads. Most installs never need it; the [portal README](packages/pi-lean-portal/README.md#stealth--custom-browser-backends) and [`contributed/README.md`](packages/pi-lean-portal/contributed/README.md) cover the full flow when you do.
 
 ---
 

package/node_modules/pi-lean-portal/README.md

@@ -284,7 +284,7 @@ navigation guidance for specific sites using `web-learn`.
 web-learn domain="reddit.com" content="…guidance text…"
 ```
 
-This creates a `.md` file with YAML frontmatter in the `guides/` directory.
+This creates a `.md` file with YAML frontmatter in `~/.pi/agent/pi-lean-portal/web-guides/`.
 The guide becomes available immediately via `web-guide` and appears in the guide footer on
 future navigations to that domain.
 
@@ -595,7 +595,7 @@ Size threshold for profile state warnings (default: 10 MB):
 ```jsonc
 {
   "browser": {
-    "maxStorageStateSize": 10485760  
+    "maxStorageStateSize": 10485760
   }
 }
 ```

package/node_modules/pi-lean-portal/__tests__/browser-toggle.test.ts

@@ -381,6 +381,30 @@ describe("applyBrowserState(false) — disable", () => {
 			expect(calledWith).not.toContain(tool);
 		}
 	});
+
+	it("does not re-enable tools that were already inactive", () => {
+		// /web off must only remove portal's own tools. Any other tool that
+		// was inactive (disabled by a peer toggle, a config default, etc.)
+		// must stay inactive — a toggle affects no tools outside its own set.
+		// Regression guard: the previous implementation rebuilt the set from
+		// getAllTools(), silently re-activating every registered non-browser
+		// tool. We now filter from getActiveTools() so peer toggles compose.
+		const OTHER_TOOL = { name: "peer-ext-tool", description: "x" };
+		const pi = mockPi({
+			tools: [...ALL_BROWSER_TOOLS, OTHER_TOOL, ...NON_BROWSER_TOOLS],
+			activeTools: [
+				...ALL_BROWSER_TOOLS.map((t) => t.name),
+				...NON_BROWSER_TOOLS.map((t) => t.name),
+			], // OTHER_TOOL intentionally absent (off)
+		});
+		applyBrowserState(pi, false);
+		const setActive = pi.setActiveTools as ReturnType<typeof vi.fn>;
+		const calledWith: string[] = setActive.mock.calls[0]![0] as string[];
+		expect(calledWith).not.toContain("peer-ext-tool");
+		for (const name of ALL_BROWSER_TOOLS.map((t) => t.name)) {
+			expect(calledWith).not.toContain(name);
+		}
+	});
 });
 
 // ==================================================================

package/node_modules/pi-lean-portal/__tests__/fetch-backend.test.ts

@@ -7,6 +7,7 @@
  */
 
 import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
+import { existsSync } from "node:fs";
 import { webFetch, cleanupFetchTempFiles } from "../core/fetch-backend.js";
 
 // ─── Setup / Teardown ────────────────────────────────────────
@@ -312,6 +313,20 @@ describe("webFetch — content capping", () => {
 		expect(result.content).toContain("Full content saved to");
 	});
 
+	it("preserves both temp files when two large fetches share a taskId", async () => {
+		mockFetch({ body: `<p>${longText(6000)}</p>` });
+		const a = await webFetch({ url: "http://example.com/a", taskId: "shared" });
+
+		mockFetch({ body: `<p>${"B".repeat(6000)}</p>` });
+		const b = await webFetch({ url: "http://example.com/b", taskId: "shared" });
+
+		expect(a.filePath).toBeTruthy();
+		expect(b.filePath).toBeTruthy();
+		expect(a.filePath).not.toBe(b.filePath);
+		expect(existsSync(a.filePath!)).toBe(true);
+		expect(existsSync(b.filePath!)).toBe(true);
+	});
+
 	it("reports totalChars even for small content", async () => {
 		mockFetch({ body: "<p>Hello</p>" });
 		const result = await webFetch({ url: "http://example.com/small" });

package/node_modules/pi-lean-portal/browser-toggle.ts

@@ -142,9 +142,14 @@ function applyBrowserState(pi: ExtensionAPI, enable: boolean): void {
 		const current = pi.getActiveTools();
 		pi.setActiveTools([...new Set([...current, ...registered])]);
 	} else {
-		// Keep everything that is NOT a browser+sibling tool
-		const all = pi.getAllTools().map((t) => t.name);
-		pi.setActiveTools(all.filter((name) => !registered.has(name)));
+		// Keep everything that is NOT a browser+sibling tool — but only
+		// within the *currently active* set. Using getActiveTools() (not
+		// getAllTools()) is critical for peer-toggle composition: /web off
+		// must not re-enable tools that another toggle (e.g. host's /api off)
+		// has already disabled. getAllTools() would silently re-activate
+		// every registered tool outside our own set.
+		const current = pi.getActiveTools();
+		pi.setActiveTools(current.filter((name) => !registered.has(name)));
 	}
 }
 

package/node_modules/pi-lean-portal/contributed/invisible-py/bridge.py

@@ -85,7 +85,7 @@ import time
 from typing import Any
 
 from pi_browser_bridge.playwright_base import PlaywrightBridge
-from pi_browser_bridge.bridge import SessionNotFoundError
+from pi_browser_bridge.playwright_base import SessionNotFoundError
 
 
 class InvisiblePyBridge(PlaywrightBridge):

package/node_modules/pi-lean-portal/core/fetch-backend.ts

@@ -250,14 +250,8 @@ function writeFetchTempFile(content: string, taskId: string): string {
 
 function trackFetchFile(taskId: string, filePath: string): void {
 	const existing = activeFetchFiles.get(taskId) ?? [];
-	for (const oldPath of existing) {
-		try {
-			rmSync(oldPath, { force: true });
-		} catch {
-			/* best-effort */
-		}
-	}
-	activeFetchFiles.set(taskId, [filePath]);
+	if (!existing.includes(filePath)) existing.push(filePath);
+	activeFetchFiles.set(taskId, existing);
 }
 
 interface CappedFetchContent {

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

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lean-portal",
-	"version": "0.2.2",
+	"version": "0.2.4",
 	"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/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lean-search",
-	"version": "0.2.2",
+	"version": "0.2.4",
 	"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",

package/node_modules/pi-lean-search/web-search-tool.ts

@@ -247,6 +247,7 @@ export const webSearchTool = defineTool({
 		'Use when you need recent/current information not already known. Increase `count` for broad research; keep it small for quick lookups. Filter by time_range="day" for breaking news, category="news" for journalism. Set language to match the query (e.g. "de" for German, "es" for Spanish).',
 		"Use `pageno` (1-indexed) to fetch deeper pages when `count` results are not enough.",
 		"Instant answers (calculator, unit convert, random uuid, hashes, weather, translations) appear above results \u2014 phrase queries like `avg 1 2 3` or `weather berlin` to trigger them.",
+		'Target results precisely: prefix the query with a bang to route to a specific source \u2014 `!wp <term>` (Wikipedia), `!images <term>`, `!map <term>`, or `:<lang> !<bang> <term>` (e.g. `:fr !wp paris`). Use the `engines` param to restrict upstreams (e.g. engines="google,bing"). For site-scoped queries put the operator inside `query` \u2014 `site:github.com foo`, `inurl:`, `intitle:`, `filetype:pdf` \u2014 but note these only work with engines that support them (Google, Bing, DuckDuckGo) and are silently ignored by others.',
 	],
 
 	parameters: Type.Object({

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lean-dimension",
-	"version": "0.2.2",
+	"version": "0.2.4",
 	"description": "Pi extension. The full Pi web-tools suite in one install — interactive browsing (pi-lean-portal) plus SearXNG search (pi-lean-search), unified under a /web toggle that removes the tools from context when switched off.",
 	"keywords": [
 		"pi-package",
@@ -32,8 +32,8 @@
 		]
 	},
 	"dependencies": {
-		"pi-lean-portal": "^0.2.2",
-		"pi-lean-search": "^0.2.2"
+		"pi-lean-portal": "^0.2.4",
+		"pi-lean-search": "^0.2.4"
 	},
 	"bundledDependencies": [
 		"pi-lean-portal",