@narumitw/pi-chrome-devtools 0.20.0 → 0.31.0

package/README.md

@@ -189,7 +189,8 @@ filename is deprecated and will be removed in a future major release.
 ```txt
 extensions/pi-chrome-devtools/
 ├── src/
-│   ├── chrome-devtools.ts  # Pi entrypoint and command orchestration
+│   ├── index.ts            # Pi package entrypoint
+│   ├── chrome-devtools.ts  # Extension registration and command orchestration
 │   └── *.ts                # Package-local browser, CDP, tool, and storage modules
 ├── README.md
 ├── LICENSE
@@ -197,12 +198,12 @@ extensions/pi-chrome-devtools/
 └── package.json
 ```
 
-Only `chrome-devtools.ts` is a Pi entrypoint; the other source modules are internal. The package exposes its Pi extension through `package.json`:
+`index.ts` is the Pi entrypoint and forwards to `chrome-devtools.ts`; the other source modules are internal. The package exposes its Pi extension through `package.json`:
 
 ```json
 {
   "pi": {
-    "extensions": ["./src/chrome-devtools.ts"]
+    "extensions": ["./src/index.ts"]
   }
 }
 ```

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@narumitw/pi-chrome-devtools",
-	"version": "0.20.0",
+	"version": "0.31.0",
 	"description": "Pi extension that exposes Chrome DevTools Protocol tools.",
 	"type": "module",
 	"license": "MIT",
@@ -20,7 +20,7 @@
 	],
 	"pi": {
 		"extensions": [
-			"./src/chrome-devtools.ts"
+			"./src/index.ts"
 		]
 	},
 	"scripts": {
@@ -29,11 +29,11 @@
 		"typecheck": "tsc --noEmit"
 	},
 	"dependencies": {
-		"typebox": "^1.3.3"
+		"typebox": "^1.3.8"
 	},
 	"devDependencies": {
-		"@biomejs/biome": "2.5.3",
-		"@earendil-works/pi-coding-agent": "0.80.10",
+		"@biomejs/biome": "2.5.5",
+		"@earendil-works/pi-coding-agent": "0.82.1",
 		"typescript": "7.0.2"
 	},
 	"repository": {

package/src/browser-manager.ts

@@ -1,21 +1,20 @@
-import { spawn, type ChildProcess } from "node:child_process";
+import { type ChildProcess, spawn } from "node:child_process";
 import { constants } from "node:fs";
 import { access, mkdtemp, readFile, rm } from "node:fs/promises";
 import { tmpdir } from "node:os";
-import { basename, delimiter, isAbsolute, join, resolve } from "node:path";
+import { delimiter, isAbsolute, join, resolve } from "node:path";
 import {
 	BROWSER_SHUTDOWN_WAIT_MS,
+	type BrowserCandidate,
+	type BrowserCandidateDefinition,
 	DEFAULT_ENDPOINT_RETRY_MS,
 	DEFAULT_ENDPOINT_WAIT_MS,
-	DEFAULT_HOST,
 	DEFAULT_HTTP_TIMEOUT_MS,
 	DEVTOOLS_ACTIVE_PORT_FILE,
-	MANAGED_BROWSER_PROFILE_PREFIX,
-	state,
-	type BrowserCandidate,
-	type BrowserCandidateDefinition,
 	type DevToolsPage,
+	MANAGED_BROWSER_PROFILE_PREFIX,
 	type ManagedBrowser,
+	state,
 } from "./runtime.js";
 
 function formatError(error: unknown) {
@@ -128,7 +127,12 @@ async function launchBrowserCandidate(candidate: BrowserCandidate, waitMs: numbe
 		];
 		throwIfBrowserLaunchCancelled();
 		const child = spawn(candidate.resolvedExecutable, args, { shell: false, stdio: "ignore" });
-		const launchedBrowser: ManagedBrowser = { process: child, userDataDir, exited: false, ready: false };
+		const launchedBrowser: ManagedBrowser = {
+			process: child,
+			userDataDir,
+			exited: false,
+			ready: false,
+		};
 		managedBrowser = launchedBrowser;
 		state.managedBrowser = launchedBrowser;
 
@@ -259,7 +263,8 @@ export async function shutdownManagedBrowser(
 		});
 	}
 	await rm(managedBrowser.userDataDir, { recursive: true, force: true }).catch(() => undefined);
-	if (!state.portConfigured && managedBrowser.port === state.port) state.port = state.configuredPort;
+	if (!state.portConfigured && managedBrowser.port === state.port)
+		state.port = state.configuredPort;
 }
 
 function killManagedBrowserProcess(managedBrowser: ManagedBrowser, signal?: NodeJS.Signals) {
@@ -400,9 +405,13 @@ export function launchModeLabel() {
 	if (!isLocalDevToolsHost(state.host)) return "manual remote endpoint";
 	if (!state.autoLaunchEnabled) return "manual; auto-launch disabled";
 	if (state.managedBrowser && !state.managedBrowser.exited) {
-		return state.portConfigured ? "auto-launched on explicit port" : "auto-launched on dynamic port";
+		return state.portConfigured
+			? "auto-launched on explicit port"
+			: "auto-launched on dynamic port";
 	}
-	return state.portConfigured ? "attach first; auto-launch explicit port" : "attach first; auto-launch dynamic port";
+	return state.portConfigured
+		? "attach first; auto-launch explicit port"
+		: "attach first; auto-launch dynamic port";
 }
 
 export function launchAttemptLines() {
@@ -435,7 +444,9 @@ export function launchHint() {
 }
 
 export function browserCandidateHint() {
-	return `Browser candidates: ${browserCandidateDefinitions().map((candidate) => candidate.label).join(", ")}`;
+	return `Browser candidates: ${browserCandidateDefinitions()
+		.map((candidate) => candidate.label)
+		.join(", ")}`;
 }
 
 export function chromeLaunchCommand() {
@@ -475,7 +486,9 @@ function browserCandidateDefinitions(): BrowserCandidateDefinition[] {
 
 function explicitBrowserCandidateDefinition(): BrowserCandidateDefinition[] {
 	if (!state.browserExecutable) return [];
-	return [{ label: "PI_CHROME_DEVTOOLS_BROWSER", executable: state.browserExecutable, source: "env" }];
+	return [
+		{ label: "PI_CHROME_DEVTOOLS_BROWSER", executable: state.browserExecutable, source: "env" },
+	];
 }
 
 function platformBrowserCandidateDefinitions(): BrowserCandidateDefinition[] {

package/src/cdp-client.ts

@@ -1,6 +1,6 @@
 import {
-	ensureDevToolsEndpoint,
 	devToolsEndpoint,
+	ensureDevToolsEndpoint,
 	fetchDevToolsJson,
 	formatPageListItem,
 	launchHint,
@@ -9,8 +9,8 @@ import {
 import {
 	DEFAULT_ENDPOINT_WAIT_MS,
 	DEFAULT_TIMEOUT_MS,
-	state,
 	type DevToolsPage,
+	state,
 } from "./runtime.js";
 
 interface CdpResponse<T = unknown> {

package/src/chrome-devtools.ts

@@ -1,14 +1,13 @@
 import type { ExtensionAPI, ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import { shutdownManagedBrowser } from "./browser-manager.js";
-import { loadSettings } from "./settings.js";
 import { state } from "./runtime.js";
+import { loadSettings } from "./settings.js";
 import {
 	allChromeDevtoolsTools,
 	applyChromeDevtoolsTools,
 	buildCommandGuide,
 	buildQuickstartMessage,
 	buildToolStatusMessage,
-	setSelectedChromeDevtoolsTools,
 	showToolSelector,
 	updateChromeDevtoolsTools,
 } from "./tool-selector.js";
@@ -104,16 +103,22 @@ async function handleChromeDevtoolsCommand(pi: ExtensionAPI, args: string, ctx:
 			return;
 	}
 
-	ctx.ui.notify(`Unknown /chrome-devtools command: ${args.trim()}
+	ctx.ui.notify(
+		`Unknown /chrome-devtools command: ${args.trim()}
 
-${buildCommandGuide()}`, "warning");
+${buildCommandGuide()}`,
+		"warning",
+	);
 }
 
 async function showMenu(pi: ExtensionAPI, ctx: CommandContext) {
 	if (!ctx.hasUI) {
-		ctx.ui.notify(`${buildCommandGuide()}
+		ctx.ui.notify(
+			`${buildCommandGuide()}
 
-${await buildToolStatusMessage(pi)}`, "info");
+${await buildToolStatusMessage(pi)}`,
+			"info",
+		);
 		return;
 	}
 

package/src/index.ts

@@ -0,0 +1 @@
+export { default } from "./chrome-devtools.js";

package/src/render.ts

@@ -1,7 +1,4 @@
-import type {
-	AgentToolResult,
-	ToolRenderResultOptions,
-} from "@earendil-works/pi-coding-agent";
+import type { AgentToolResult, ToolRenderResultOptions } from "@earendil-works/pi-coding-agent";
 
 const STATUS_KEY = "chrome-devtools";
 interface StatusContext {
@@ -90,9 +87,7 @@ class PiTextComponent implements RenderComponent {
 			.split(/\r?\n/)
 			.map((line) => {
 				const truncatedLine = truncateLine(line, Math.max(1, width));
-				return this.theme && this.color
-					? this.theme.fg(this.color, truncatedLine)
-					: truncatedLine;
+				return this.theme && this.color ? this.theme.fg(this.color, truncatedLine) : truncatedLine;
 			});
 	}
 }
@@ -101,7 +96,11 @@ function truncateLine(line: string, maxWidth: number) {
 	return Array.from(line).slice(0, maxWidth).join("");
 }
 
-export async function withStatus<T>(ctx: StatusContext, status: string, callback: () => Promise<T>) {
+export async function withStatus<T>(
+	ctx: StatusContext,
+	status: string,
+	callback: () => Promise<T>,
+) {
 	ctx.ui.setStatus(STATUS_KEY, status);
 	try {
 		return await callback();

package/src/screenshot.ts

@@ -1,5 +1,5 @@
 import { randomUUID } from "node:crypto";
-import { link, lstat, mkdir, mkdtemp, realpath, rename, rm, writeFile } from "node:fs/promises";
+import { lstat, mkdir, realpath, rename, rm, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { basename, dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
 import { withFileMutationQueue } from "@earendil-works/pi-coding-agent";
@@ -47,7 +47,10 @@ export async function saveScreenshot(
 	};
 }
 
-export function resolveScreenshotPath(savePath: string | undefined, cwd: string): ResolvedScreenshotPath {
+export function resolveScreenshotPath(
+	savePath: string | undefined,
+	cwd: string,
+): ResolvedScreenshotPath {
 	const cwdRoot = resolve(cwd);
 	const tempRoot = resolve(tmpdir());
 
@@ -228,7 +231,9 @@ export function isPathInsideRoot(path: string, root: string) {
 	const normalizedPath = normalizePathForComparison(resolve(path));
 	const normalizedRoot = normalizePathForComparison(resolve(root));
 	if (normalizedPath === normalizedRoot) return true;
-	const rootWithSeparator = normalizedRoot.endsWith(sep) ? normalizedRoot : `${normalizedRoot}${sep}`;
+	const rootWithSeparator = normalizedRoot.endsWith(sep)
+		? normalizedRoot
+		: `${normalizedRoot}${sep}`;
 	return normalizedPath.startsWith(rootWithSeparator);
 }
 

package/src/settings.ts

@@ -3,10 +3,7 @@ import { constants } from "node:fs";
 import { access, link, mkdir, readFile, rename, rm, writeFile } from "node:fs/promises";
 import { homedir } from "node:os";
 import { dirname, join } from "node:path";
-import {
-	CHROME_DEVTOOLS_TOOL_NAMES,
-	type ChromeDevToolsToolName,
-} from "./tool-names.js";
+import { CHROME_DEVTOOLS_TOOL_NAMES, type ChromeDevToolsToolName } from "./tool-names.js";
 
 const NEW_SETTINGS_FILE_NAME = "pi-chrome-devtools.json";
 const LEGACY_SETTINGS_FILE_NAME = "pi-chrome-devtools-settings.json";
@@ -99,10 +96,7 @@ async function migrateLegacySettings(
 ): Promise<SettingsMigrationResult> {
 	const newPath = settingsFilePath();
 	try {
-		await installSettingsFileExclusively(
-			newPath,
-			`${JSON.stringify(settings, null, 2)}\n`,
-		);
+		await installSettingsFileExclusively(newPath, `${JSON.stringify(settings, null, 2)}\n`);
 	} catch (error) {
 		return {
 			kind: "failed",
@@ -134,7 +128,9 @@ async function fileExists(filePath: string) {
 	}
 }
 
-export function normalizeChromeDevtoolsSettings(value: unknown): ChromeDevToolsSettings | undefined {
+export function normalizeChromeDevtoolsSettings(
+	value: unknown,
+): ChromeDevToolsSettings | undefined {
 	if (!value || typeof value !== "object") return undefined;
 	const settings = value as { tools?: unknown; updatedAt?: unknown };
 	if (typeof settings.updatedAt !== "number") return undefined;
@@ -190,7 +186,3 @@ function isNodeError(error: unknown): error is NodeJS.ErrnoException {
 function formatError(error: unknown) {
 	return error instanceof Error ? error.message : String(error);
 }
-
-function unique<T>(values: T[]) {
-	return Array.from(new Set(values));
-}

package/src/tool-selector.ts

@@ -1,7 +1,6 @@
 import type { ExtensionAPI, ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import {
 	browserCandidateHint,
-	chromeLaunchCommand,
 	devToolsEndpoint,
 	endpointConfigHint,
 	endpointSourceLabel,
@@ -9,12 +8,9 @@ import {
 	launchHint,
 	launchModeLabel,
 } from "./browser-manager.js";
-import { loadSettings, saveSettings, settingsFilePath } from "./settings.js";
 import { state } from "./runtime.js";
-import {
-	CHROME_DEVTOOLS_TOOL_NAMES,
-	type ChromeDevToolsToolName,
-} from "./tool-names.js";
+import { loadSettings, saveSettings, settingsFilePath } from "./settings.js";
+import { CHROME_DEVTOOLS_TOOL_NAMES, type ChromeDevToolsToolName } from "./tool-names.js";
 
 type CommandContext = ExtensionCommandContext;
 const TOOL_SELECTOR_DONE = "Done";
@@ -187,9 +183,12 @@ export async function updateChromeDevtoolsTools(
 	action: string,
 ) {
 	await setSelectedChromeDevtoolsTools(pi, ctx, selectedTools);
-	ctx.ui.notify(`Chrome DevTools tools ${action}.
+	ctx.ui.notify(
+		`Chrome DevTools tools ${action}.
 
-${await buildToolStatusMessage(pi)}`, "info");
+${await buildToolStatusMessage(pi)}`,
+		"info",
+	);
 }
 
 export async function setSelectedChromeDevtoolsTools(