pi-lean-portal 0.6.0 → 0.7.0

package/README.md

@@ -331,8 +331,8 @@ the toolset's packaged default:
 - Omit a `toolsetDefaults` key to use the toolset's packaged default (`web`
   and `search` default `true`; `web-learn` defaults `false`).
 - The `search` key only applies when `pi-lean-search` is installed.
-- Pins do not apply in spawned subagent children (see pi-tool-masking
-  1.3.0's `piToolMasking.childPolicy` for the opt-out).
+- Pins do not apply in spawned subagent children (see pi-tool-masking's
+  `piToolMasking.childPolicy` for the opt-out).
 
 ### `browser.maxStorageStateSize`
 

package/backends/python-base/pi_browser_bridge/playwright_base.py

@@ -1260,7 +1260,7 @@ class PlaywrightBridge:
 
         try:
             locator.click(timeout=5_000)  # Focus first
-            locator.fill(text)
+            locator.fill(text, timeout=5_000)
 
             result = self._build_interaction_result(task_id, page)
         except Exception as exc:

package/browser-toggle.ts

@@ -3,17 +3,26 @@ import type {
 	ExtensionContext,
 } from "@earendil-works/pi-coding-agent";
 import { updateFooterStatus, getLastCtx, setLastCtx } from "./tools/utils.js";
-import {
-	defineToolset,
-	TOOLSET_EVENTS,
-	getDefaultResolutionMode,
-} from "pi-tool-masking";
+import { defineToolset, TOOLSET_EVENTS } from "pi-tool-masking";
 import type { ToolsetSpec } from "pi-tool-masking";
 
-// Focus-mode guard: refuse actuating subcommands while allowlist focus is
-// holding the line (an upstream pi-tool-masking consumer).
-function isFocusHolding(): boolean {
-	return getDefaultResolutionMode() === "allowlist";
+// Catch AllowlistModeError by name, never `instanceof` — the handle may come
+// from a different library copy via the shared registry. Returns true when
+// the toggle ran, false when it was refused (nothing changed).
+function refuseOnAllowlist(ctx: ExtensionContext, toggle: () => unknown): boolean {
+	try {
+		toggle();
+		return true;
+	} catch (err) {
+		if ((err as { name?: string } | undefined)?.name === "AllowlistModeError") {
+			ctx.ui.notify(
+				"Focus mode (allowlist) is active — this toolset can't be toggled while focus is holding the line. Exit focus there first.",
+				"warning",
+			);
+			return false;
+		}
+		throw err;
+	}
 }
 
 // ---- Toolset specs -----------------------------------------------
@@ -138,36 +147,38 @@ export default function initBrowserToggle(pi: ExtensionAPI) {
 		handler: async (args, ctx) => {
 			const cmd = args.trim().toLowerCase();
 
-			// Focus-mode guard: refuse actuating subcommands while allowlist
-			// focus holds the line (an upstream pi-tool-masking consumer) —
-			// a sibling toggle must not write a focus-indistinguishable
-			// {enabled} entry.
-			// Read-only subcommands (status/profile/cookies/bare /web) stay
-			// unguarded, matching the focus controller's treatment of its own
-			// read-only commands.
-			if (["on", "off", "learn"].includes(cmd) && isFocusHolding()) {
-				ctx.ui.notify(
-					"Focus mode (allowlist) is active — this toolset can't be toggled while focus is holding the line. Exit focus there first.",
-					"warning",
-				);
-				return;
-			}
-
 			if (cmd === "on") {
-				webToolset.enable(pi);
-				learnToolset.disable(pi);
+				if (
+					!refuseOnAllowlist(ctx, () => {
+						webToolset.enable(pi, ctx.sessionManager);
+						learnToolset.disable(pi, ctx.sessionManager);
+					})
+				)
+					return;
 				ctx.ui.notify(
 					"🌐 Browser tools enabled. /web learn to make web-learn available.",
 					"info",
 				);
 			} else if (cmd === "learn") {
-				learnToolset.enable(pi); // cascades web on via requires
+				if (
+					!refuseOnAllowlist(
+						ctx,
+						() => learnToolset.enable(pi, ctx.sessionManager), // cascades web on via requires
+					)
+				)
+					return;
 				ctx.ui.notify(
 					"📖 web-learn tool is now available. Agent will save/update guides when asked.",
 					"info",
 				);
 			} else if (cmd === "off") {
-				webToolset.disable(pi); // cascades learn off via requires
+				if (
+					!refuseOnAllowlist(
+						ctx,
+						() => webToolset.disable(pi, ctx.sessionManager), // cascades learn off via requires
+					)
+				)
+					return;
 				ctx.ui.notify("🌐 Browser tools disabled. /web on to re-enable.", "info");
 			} else if (cmd === "profile" || cmd.startsWith("profile ")) {
 				const sub = cmd.slice("profile".length).trim();

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lean-portal",
-	"version": "0.6.0",
+	"version": "0.7.0",
 	"description": "Interactive Playwright web browsing for Pi. Chromium and Firefox built in; author stealth backends on either engine. Snapshots and screenshots cache to disk, saving context. Configurable profiles/cookies. Agents write site guides, auto-matched by domain.",
 	"keywords": [
 		"pi-package",
@@ -55,7 +55,7 @@
 	},
 	"dependencies": {
 		"node-html-parser": "^6.1.0",
-		"pi-tool-masking": "^1.3.0",
+		"pi-tool-masking": "^2.0.1",
 		"playwright": "^1.60.0",
 		"turndown": "^7.2.0"
 	},