pi-lean-dimension 0.5.0 → 0.6.0

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

@@ -239,13 +239,13 @@ Playwright Firefox (Juggler) and Playwright Chromium (CDP) serialize ARIA trees
 - `core/shared/` — utilities used by both framework and plugins
 - Plugins import from `../../core/plugin-api.js` and `../../core/shared/*.js`
 - The router imports from `../../core/plugin-api.js` and `../../core/shared/*.js`
-- `browser-cookies.ts`, `browser-profile.ts`, `browser-status.ts` live at the portal package root and import from `core/` — they're command handlers, not plugins.
+- `browser-cookies.ts`, `browser-profile.ts`, `browser-status.ts` live at the portal package root and import from `core/` — they're command handlers, not plugins. `browser-install.ts` is also a package-root command handler, but imports only node builtins and peer packages (playwright resolution happens at call time via `createRequire`).
 
 ## Testing (portal detail)
 
 The monorepo root owns the test split principle and the summary counts table; this section is the per-file detail for tests that live in this package.
 
-**Portal structural (22 files):** router-dispatch, browser-toggle, browser-toggle-profile, browser-navigate, browser-status, session-manager, browser-data, plugin-registry, plugin-contract, plugin-config-browser, python-adapter, fetch-backend, accessibility-tree, plugin-loading, snapshot-cache, browser-inspect, web-guides, router-session, storage-state, nav-settle, probe-user-backend, ship-manifest
+**Portal structural (25 files; `ship-manifest` lives at the package root, the rest in `__tests__/`):** router-dispatch, browser-toggle, browser-toggle-profile, browser-install, browser-navigate, browser-status, session-manager, browser-data, plugin-registry, plugin-contract, plugin-config-browser, python-adapter, fetch-backend, accessibility-tree, plugin-loading, snapshot-cache, browser-inspect, web-guides, guide-providers, router-session, settings-reader, storage-state, nav-settle, probe-user-backend, ship-manifest
 
 **Python bridge unit tests (6 files, pytest):** test_accessibility, test_bot_detection, test_transport, test_browser_data, test_py_bridges, test_playwright_base_quirks (the stealth-quirk flags: `_fingerprint_managed_context`, `_skip_default_viewport`, `_scroll_via_wheel`, `_eval_prefix`)
 
@@ -255,13 +255,14 @@ The monorepo root owns the test split principle and the summary counts table; th
 
 - `plugin-contract.ts` — `runContractTests(name, factory, opts?)` validates any BrowserPlugin
 - `mock-plugin.ts` — MockPlugin for structural contract validation
+- `mock-pi.ts` — `mockCtx()` ExtensionContext stub + `captureWebHandler()` for command-handler tests
 - `test-server.ts` — `startTestServer()` returns a local HTTP server for integration tests
 - `mock-python-bridge.py` — Python bridge stub used by python-adapter tests
 
 ## Known Constraints & Debt
 
 - **Console capture in Python backends** — Both `chromium-py` and `firefox-py` inherit console capture (500-entry ring buffer) and dialog auto-dismissal from `PlaywrightBridge._setup_page_session()` in `python-base`. All Python browser backends must subclass `PlaywrightBridge` to get session setup and JSON-RPC dispatch.
-- **AbortSignal not supported on Python bridge** — the router passes `signal` through unconditionally (no capability check). The Python adapter accepts and silently ignores the signal. `supportsAbortSignal` is advertised but unenforced.
+- **AbortSignal not supported on Python bridge** — the router passes `signal` through unconditionally (no capability check). The Python adapter accepts and silently ignores the signal.
 - **Sessions are per taskId** — mapped to `browser-NNN` keys via `_sessionKeys`/`_sessionCounter` in `core/shared/task-id.ts`. Created on first navigate, cleaned up on `session_shutdown`.
 - **Python shared-context machinery removed** — Named profiles now use disk persistence (load-on-navigate via `storageState`) matching the TS Chromium plugin. Both backends use `ensure_session(task_id, config)` for all sessions.
 - **Python bridge reuses BrowserContexts across navigations** — `ensure_session()` returns the existing session on re-navigate (unlike the TS Chromium plugin which creates a fresh context per navigate). This means in-process cookies survive re-navigation without explicit save, but also means `storageState` from the router is ignored on re-navigate (the context already exists). The Python adapter's `_persistState()` saves current cookies to disk before the navigate RPC for cross-process persistence.

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

@@ -16,9 +16,10 @@
 
 ```bash
 pi install npm:pi-lean-portal
-npx playwright install chromium firefox
 ```
 
+Then run **`/web install`** inside pi to download browser binaries (these are not installed via npm).
+
 Once loaded, you'll see a notification like:
 
 > 🌐 Browser extension loaded (plugins: chromium, firefox). Try: web-fetch for static pages or browser-navigate for interactive browsing.
@@ -27,10 +28,6 @@ The browser tools are **enabled by default**: `web-fetch` for static pages,
 `browser-navigate` (plus click/type/scroll/screenshots via `@e` refs) for
 interactive browsing.
 
-> **Playwright browser binaries are not downloaded during `npm install`.**
-> Run `npx playwright install chromium firefox` separately. On first
-> `browser-navigate` without them, you'll be prompted with the exact command.
-
 ---
 
 ## `/web` Command — Browser Toggle & Profiles
@@ -51,13 +48,16 @@ toggles guide-saving mode, and manages browser profiles.
 | Command | Effect |
 | ------- | ------ |
 | `/web` | Show current toggle status and available sub-commands. |
-| `/web status` | **Detailed runtime status** including toggle state, plugin health, active sessions, and profiles on disk. |
+| `/web status` | **Detailed runtime status** including toggle state, plugin health, browser-binary presence, active sessions, and profiles on disk. |
+| `/web install` | **Install browser binaries** via the bundled playwright CLI (checkbox dialog in the TUI; prints the manual command elsewhere). |
+| `/web install chromium\|firefox` | Directly download one engine, no dialog. |
 
 ```text
 🌐 Browser tools: ✅ on  |  📖 Learn mode: ❌ off
 ────────────────────────────────────────
 Status: idle
 Plugins: chromium, firefox, chromium-py (disabled), firefox-py (disabled)
+Browsers: chromium ✓ firefox ✓
 Use web-fetch for stateless HTTP fetches.
 Active sessions: 1
   PW [chromium] https://example.com — Example Domain [profile: session]
@@ -225,9 +225,9 @@ and the router dispatches tool calls to the right plugin based on a
 | `chromium-py` | Chromium | Python/Playwright | Disabled |
 | `firefox-py` | Firefox | Python/Playwright | Disabled |
 
-> **Install Firefox:** `npx playwright install firefox` to use the Node
-> `firefox` backend. For the Python parity backends, install Playwright
-> inside `backends/python-base/.venv`.
+> **Install Firefox:** run `/web install firefox` inside pi to download the
+> Firefox binary for the Node `firefox` backend. For the Python parity
+> backends, install Playwright inside `backends/python-base/.venv`.
 
 All four shipped backends support screenshots (viewport-sized, not
 full-page), console capture, JS evaluation, bot detection, and dialog

package/node_modules/pi-lean-portal/__tests__/accessibility-tree.test.ts

@@ -148,21 +148,6 @@ describe("parseSnapshot — duplicate occurrences", () => {
 	});
 });
 
-// ─── buildLocator — structural verification ───────────────────────
-
-describe("buildLocator occurrenceIndex behavior", () => {
-	it("always uses .nth(occurrenceIndex) even when index is 0", () => {
-		// buildLocator now ALWAYS calls .nth(occurrenceIndex) to avoid strict-
-		// mode violations from duplicate role+name elements.  For unique
-		// elements with occurrenceIndex=0, .nth(0) is equivalent to the bare
-		// locator in Playwright — it targets element at index 0 which is the
-		// same as the only match.
-		const result = parseSnapshot('- button "Unique"');
-		const node = result.elements.get("e1")!;
-		expect(node.occurrenceIndex).toBe(0);
-	});
-});
-
 // ─── Snapshot text format — @e refs ──────────────────────────────
 
 describe("parseSnapshot text format", () => {