pi-browser-harness 0.14.0 → 0.14.1

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to pi-browser-harness will be documented in this file.
 
+## 0.14.1 — 2026-11-07
+
+### Fixed
+
+- **Browser tools load in every session, not just the first.** pi builds a fresh tool list for each session it starts in a running process (`/new`, resume, fork, and in-process subagents), but the harness remembered which tools it had registered for the whole process and skipped them all. Those sessions had no `browser_*` tools while the prompt still said they were available. The harness now registers its tools with each new session.
+
 ## 0.14.0 — 2026-11-01
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-browser-harness",
-  "version": "0.14.0",
+  "version": "0.14.1",
   "description": "Browser control extension for pi — navigate, click, type, screenshot, and extract data from real Chrome via CDP",
   "keywords": [
     "pi-package",

package/src/registry.ts

@@ -92,8 +92,9 @@ export type ToolRegistry = {
   // Append a batch atomically: a name clash anywhere in it (with the registry or within the batch)
   // rejects the whole call and leaves the registry untouched.
   readonly addAll: (tools: ReadonlyArray<BrowserTool>) => Result<void, RegistryError>;
-  // Register every tool that has not been registered with pi yet. Safe to call on every
-  // session_start: tools added after an earlier call land on the next one.
+  // Register every tool that has not been registered with this pi instance yet. Safe to call on
+  // every session_start: tools added after an earlier call land on the next one, and a fresh pi
+  // instance (pi builds one per session but reuses this cached module) gets every tool again.
   readonly register: (pi: ToolRegistrar, client: BrowserClient) => void;
 };
 
@@ -105,7 +106,9 @@ const duplicateName = (name: string): RegistryError => ({
 export const createToolRegistry = (initial: ReadonlyArray<BrowserTool> = ALL_TOOLS): ToolRegistry => {
   const tools: BrowserTool[] = [];
   const names = new Set<string>();
-  const registered = new Set<string>();
+  // Keyed by pi instance: a process-wide set made every session after the first start with no
+  // browser tools, because pi hands each session an empty tool list.
+  const registeredByPi = new WeakMap<ToolRegistrar, Set<string>>();
 
   const addAll = (batch: ReadonlyArray<BrowserTool>): Result<void, RegistryError> => {
     const seen = new Set<string>();
@@ -129,6 +132,8 @@ export const createToolRegistry = (initial: ReadonlyArray<BrowserTool> = ALL_TOO
     add,
     addAll,
     register: (pi, client) => {
+      const registered = registeredByPi.get(pi) ?? new Set<string>();
+      registeredByPi.set(pi, registered);
       for (const tool of tools) {
         if (registered.has(tool.name)) continue;
         tool.register(pi, client);