pi-lean-dimension 0.3.2 → 0.3.3

package/node_modules/pi-tool-masking/README.md

@@ -2,7 +2,7 @@
 
 A library for pi plugin developers that groups tools into toggleable **toolsets** with persistent state and cross-extension events — eliminating the boilerplate every pi extension repeats when it wants to let users disable tools cleanly.
 
-`pi-tool-masking` is **not** a pi extension itself. It is a peer dependency that your extension imports. It owns the toggle logic, the session-restore path, and the event bus. Your extension owns the commands, the status bar, and the user-facing surfaces.
+`pi-tool-masking` is **not** a pi extension itself. It is a dependency that your extension imports. It owns the toggle logic, the session-restore path, and the event bus. Your extension owns the commands, the status bar, and the user-facing surfaces.
 
 ---
 
@@ -228,8 +228,8 @@ for (const entry of getRegisteredToolsets()) {
 ### Dependent toolsets
 
 ```ts
-// Portal tools are on by default; learn tools depend on portal
-const portalSpec: ToolsetSpec = {
+// Web tools are on by default; learn tools depend on web
+const webSpec: ToolsetSpec = {
  id: "my-plugin.web",
  names: new Set(["web-fetch", "web-snapshot"]),
  persistKey: "toolset-state:my-plugin.web",
@@ -247,6 +247,37 @@ const learnSpec: ToolsetSpec = {
 
 ---
 
+## Toolset naming
+
+`defineToolset` can't tell which extension is calling it — pi's `ExtensionAPI`
+doesn't expose the caller — so error messages can't name the responsible
+extension directly. The toolset id is the only traceability signal, which is
+why a stable, attributable id convention matters.
+
+### Convention (recommended, not enforced)
+
+Prefix toolset ids with a stable namespace: `<product-family>.<subset>`, e.g.
+`my-plugin.web`. The family may span multiple npm packages, and nothing checks
+that the prefix matches a real package — it's for human traceability in
+`/tbox list` and collision errors, not verification.
+
+### Enforcement floor
+
+`defineToolset` enforces one naming invariant: **no two toolsets may claim the
+same tool name.** Overlap is essentially always an authoring mistake and throws
+at load time:
+
+```
+[pi-tool-masking] name overlap: toolset "foo.search" claims tools already
+owned by another toolset:
+  - tool "x" already claimed by toolset "bar.web" (registered from
+    /home/u/.pi/.../bar/index.ts, source: bar)
+Each tool may belong to only one toolset. Naming convention: prefix toolset
+ids with a stable namespace (<product-family>.<subset>, e.g. "foo.web").
+```
+
+---
+
 ## How it works (for the curious)
 
 - **Registration:** `defineToolset` stores the spec and handle in a global registry (shared across module instances, so multiple extensions see the same toolsets).
@@ -267,4 +298,4 @@ This package is used by:
 
 ## License
 
-GNU Affero General Public License v3.0 (AGPL-3.0). See [LICENSE](./LICENSE).
+MIT. See [LICENSE](./LICENSE).

package/node_modules/pi-tool-masking/index.ts

@@ -8,7 +8,7 @@ import type {
 // ---------------------------------------------------------------------------
 
 export interface ToolsetSpec {
-	/** Stable id, e.g. "portal.web". Used in persist keys and event payloads. */
+	/** Stable id, e.g. "my-plugin.web". Used in persist keys and event payloads. */
 	id: string;
 	/** Human-readable name for the group. Optional — presenters fall back to id. */
 	label?: string;
@@ -16,7 +16,7 @@ export interface ToolsetSpec {
 	description?: string;
 	/** Tool names this toolset governs. */
 	names: Set<string>;
-	/** Primary persistence key the toolset writes, e.g. "toolset-state:portal.web". */
+	/** Primary persistence key the toolset writes, e.g. "toolset-state:my-plugin.web". */
 	persistKey: string;
 	/** Fresh-session fallback when no branch entry exists. */
 	defaultEnabled?: boolean;
@@ -33,7 +33,7 @@ export interface Toolset {
 }
 
 export interface ToolsetChangedEvent {
-	/** Toolset id (e.g. "portal.web"). Always set. */
+	/** Toolset id (e.g. "my-plugin.web"). Always set. */
 	id: string;
 	enabled: boolean;
 	/** Present only when emitMemberEvents is on and this is a per-member fanout event. */
@@ -165,7 +165,7 @@ function ensureRestoreHandler(pi: ExtensionAPI): void {
 		//
 		// Re-read the branch per toolset (not once before the loop): a companion
 		// mirror (§10.1) fires synchronously inside `_applyRestoreToolset` and may
-		// `appendEntry` for a toolset later in iteration order (e.g. portal.web's
+		// `appendEntry` for a toolset later in iteration order (e.g. my-plugin.web's
 		// default-false restore makes search.web disable itself). Snapshotting the
 		// branch once would hide that write from the later toolset, so it would
 		// fall back to its packaged default and desync from the companion — the
@@ -426,6 +426,39 @@ export function defineToolset(pi: ExtensionAPI, spec: ToolsetSpec): Toolset {
 		}
 	}
 
+	// Name-overlap guard: no two toolsets may claim the same tool name. Every
+	// downstream failure mode (isEnabled lying, restore order-dependence, enable
+	// no-op, skipped dependents, focus leaks, mis-attribution, double-counts)
+	// requires two toolsets claiming one name; with that unreachable, toolsets
+	// own disjoint name sets. Gather every collision in this registration into
+	// one error so the author sees the full scope in one pass. `getAllTools()` is
+	// deferred to the throw branch so a clean registration never pays for it.
+	const collisions: { name: string; owner: string }[] = [];
+	for (const [id, entry] of registry) {
+		if (id === spec.id) continue;
+		for (const name of spec.names) {
+			if (entry.spec.names.has(name)) collisions.push({ name, owner: id });
+		}
+	}
+	if (collisions.length > 0) {
+		const allTools = pi.getAllTools();
+		const lines = collisions.map(({ name, owner }) => {
+			const tool = allTools.find((t) => t.name === name);
+			const where = tool
+				? ` (registered from ${tool.sourceInfo.path}, source: ${tool.sourceInfo.source})`
+				: "";
+			return `  - tool "${name}" already claimed by toolset "${owner}"${where}`;
+		});
+		throw new Error(
+			`[pi-tool-masking] name overlap: toolset "${spec.id}" claims tools ` +
+				`already owned by another toolset:\n` +
+				lines.join("\n") +
+				"\n" +
+				`Each tool may belong to only one toolset. Naming convention: prefix ` +
+				`toolset ids with a stable namespace (<product-family>.<subset>, e.g. "my-plugin.web").`,
+		);
+	}
+
 	const toolset = new ToolsetImpl(spec);
 	registry.set(spec.id, { spec, toolset });
 

package/node_modules/pi-tool-masking/package.json

@@ -1,11 +1,14 @@
 {
 	"name": "pi-tool-masking",
-	"version": "1.0.0",
+	"version": "1.1.0",
 	"private": false,
 	"type": "module",
 	"description": "Core library for grouping pi tools into toggleable toolsets with persistent state and cross-extension events.",
-	"keywords": ["tool-masking", "pi-extension"],
-	"license": "AGPL-3.0-or-later",
+	"keywords": [
+		"tool-masking",
+		"pi-extension"
+	],
+	"license": "MIT",
 	"author": "Corey Ryan Hanson",
 	"repository": {
 		"type": "git",

package/node_modules/playwright/lib/common/index.js

@@ -793,16 +793,13 @@ function loadTsConfig(configPath) {
 ${e.message}`);
   }
 }
-function resolveConfigFile(baseConfigFile, referencedConfigFile, kind) {
-  const originalReferencedConfigFile = referencedConfigFile;
+function resolveConfigFile(baseConfigFile, referencedConfigFile) {
   if (!referencedConfigFile.endsWith(".json"))
     referencedConfigFile += ".json";
   const currentDir = import_path4.default.dirname(baseConfigFile);
   let resolvedConfigFile = import_path4.default.resolve(currentDir, referencedConfigFile);
   if (referencedConfigFile.includes("/") && referencedConfigFile.includes(".") && !import_fs4.default.existsSync(resolvedConfigFile))
     resolvedConfigFile = import_path4.default.join(currentDir, "node_modules", referencedConfigFile);
-  if (!import_fs4.default.existsSync(resolvedConfigFile))
-    throw new Error(`Failed to resolve "${kind}" path "${originalReferencedConfigFile}" referenced from ${baseConfigFile}`);
   return resolvedConfigFile;
 }
 function innerLoadTsConfig(configFilePath, references, visited = /* @__PURE__ */ new Map()) {
@@ -819,7 +816,7 @@ function innerLoadTsConfig(configFilePath, references, visited = /* @__PURE__ */
   const parsedConfig = json5.parse(cleanedJson);
   const extendsArray = Array.isArray(parsedConfig.extends) ? parsedConfig.extends : parsedConfig.extends ? [parsedConfig.extends] : [];
   for (const extendedConfig of extendsArray) {
-    const extendedConfigPath = resolveConfigFile(configFilePath, extendedConfig, "extends");
+    const extendedConfigPath = resolveConfigFile(configFilePath, extendedConfig);
     const base = innerLoadTsConfig(extendedConfigPath, references, visited);
     Object.assign(result2, base, { tsConfigPath: configFilePath });
   }
@@ -835,7 +832,7 @@ function innerLoadTsConfig(configFilePath, references, visited = /* @__PURE__ */
     result2.absoluteBaseUrl = import_path4.default.resolve(import_path4.default.dirname(configFilePath), parsedConfig.compilerOptions.baseUrl);
   }
   for (const ref of parsedConfig.references || [])
-    references.push(innerLoadTsConfig(resolveConfigFile(configFilePath, ref.path, "references"), references, visited));
+    references.push(innerLoadTsConfig(resolveConfigFile(configFilePath, ref.path), references, visited));
   if (import_path4.default.basename(configFilePath) === "jsconfig.json" && result2.allowJs === void 0)
     result2.allowJs = true;
   return result2;

package/node_modules/playwright/lib/common/index.js.txt

@@ -1,5 +1,5 @@
 # packages/playwright/lib/common/index.js
-# total: 114.4 KB
+# total: 114.2 KB
 
 ## Inlined (20)
   12.0 KB  packages/playwright/src/common/config.ts
@@ -20,7 +20,7 @@
    1.1 KB  packages/playwright/src/transform/pirates.ts
    1.1 KB  packages/playwright/src/transform/portTransport.ts
   12.3 KB  packages/playwright/src/transform/transform.ts
-   3.2 KB  packages/playwright/src/transform/tsconfig-loader.ts
+   2.9 KB  packages/playwright/src/transform/tsconfig-loader.ts
    7.0 KB  packages/playwright/src/util.ts
 
 ## External (8)

package/node_modules/playwright/lib/transform/esmLoader.js

@@ -4414,16 +4414,13 @@ function loadTsConfig(configPath) {
 ${e.message}`);
   }
 }
-function resolveConfigFile(baseConfigFile, referencedConfigFile, kind) {
-  const originalReferencedConfigFile = referencedConfigFile;
+function resolveConfigFile(baseConfigFile, referencedConfigFile) {
   if (!referencedConfigFile.endsWith(".json"))
     referencedConfigFile += ".json";
   const currentDir = import_path2.default.dirname(baseConfigFile);
   let resolvedConfigFile = import_path2.default.resolve(currentDir, referencedConfigFile);
   if (referencedConfigFile.includes("/") && referencedConfigFile.includes(".") && !import_fs2.default.existsSync(resolvedConfigFile))
     resolvedConfigFile = import_path2.default.join(currentDir, "node_modules", referencedConfigFile);
-  if (!import_fs2.default.existsSync(resolvedConfigFile))
-    throw new Error(`Failed to resolve "${kind}" path "${originalReferencedConfigFile}" referenced from ${baseConfigFile}`);
   return resolvedConfigFile;
 }
 function innerLoadTsConfig(configFilePath, references, visited = /* @__PURE__ */ new Map()) {
@@ -4440,7 +4437,7 @@ function innerLoadTsConfig(configFilePath, references, visited = /* @__PURE__ */
   const parsedConfig = import_json5.default.parse(cleanedJson);
   const extendsArray = Array.isArray(parsedConfig.extends) ? parsedConfig.extends : parsedConfig.extends ? [parsedConfig.extends] : [];
   for (const extendedConfig of extendsArray) {
-    const extendedConfigPath = resolveConfigFile(configFilePath, extendedConfig, "extends");
+    const extendedConfigPath = resolveConfigFile(configFilePath, extendedConfig);
     const base = innerLoadTsConfig(extendedConfigPath, references, visited);
     Object.assign(result2, base, { tsConfigPath: configFilePath });
   }
@@ -4456,7 +4453,7 @@ function innerLoadTsConfig(configFilePath, references, visited = /* @__PURE__ */
     result2.absoluteBaseUrl = import_path2.default.resolve(import_path2.default.dirname(configFilePath), parsedConfig.compilerOptions.baseUrl);
   }
   for (const ref of parsedConfig.references || [])
-    references.push(innerLoadTsConfig(resolveConfigFile(configFilePath, ref.path, "references"), references, visited));
+    references.push(innerLoadTsConfig(resolveConfigFile(configFilePath, ref.path), references, visited));
   if (import_path2.default.basename(configFilePath) === "jsconfig.json" && result2.allowJs === void 0)
     result2.allowJs = true;
   return result2;

package/node_modules/playwright/lib/transform/esmLoader.js.txt

@@ -1,5 +1,5 @@
 # packages/playwright/lib/transform/esmLoader.js
-# total: 283.9 KB
+# total: 283.6 KB
 
 ## Inlined (51)
    1.3 KB  node_modules/balanced-match/dist/esm/index.js
@@ -47,7 +47,7 @@
    1.1 KB  packages/playwright/src/transform/pirates.ts
    1.1 KB  packages/playwright/src/transform/portTransport.ts
   11.7 KB  packages/playwright/src/transform/transform.ts
-   3.2 KB  packages/playwright/src/transform/tsconfig-loader.ts
+   2.9 KB  packages/playwright/src/transform/tsconfig-loader.ts
    4.7 KB  packages/playwright/src/util.ts
    0.2 KB  packages/utils/crypto.ts
    0.1 KB  packages/utils/debug.ts

package/node_modules/playwright/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playwright",
-  "version": "1.62.0",
+  "version": "1.62.1",
   "description": "A high-level API to automate web browsers",
   "repository": {
     "type": "git",
@@ -50,7 +50,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "playwright-core": "1.62.0"
+    "playwright-core": "1.62.1"
   },
   "optionalDependencies": {
     "fsevents": "2.3.2"