@jackwener/opencli 1.7.12 → 1.7.14

package/dist/src/browser/bridge.js

@@ -54,62 +54,64 @@ export class BrowserBridge {
         const effectiveSeconds = (timeoutSeconds && timeoutSeconds > 0) ? timeoutSeconds : Math.ceil(DAEMON_SPAWN_TIMEOUT / 1000);
         const timeoutMs = effectiveSeconds * 1000;
         const health = await getDaemonHealth({ contextId });
+        // Detect stale daemon before any fast path. A stale daemon can still have
+        // the extension connected, so this cannot live only in the no-extension branch.
+        const daemonVersion = health.status?.daemonVersion;
+        const isStale = !!health.status && (!daemonVersion || daemonVersion !== PKG_VERSION);
+        let staleDaemonReplaced = false;
+        if (isStale) {
+            // Stale daemon — restart it so all browser commands run against the
+            // currently installed package code, not the old daemon binary.
+            const reason = daemonVersion
+                ? `v${daemonVersion} ≠ v${PKG_VERSION}`
+                : `pre-version daemon, CLI is v${PKG_VERSION}`;
+            if (process.env.OPENCLI_VERBOSE || process.stderr.isTTY) {
+                process.stderr.write(`⚠️  Stale daemon detected (${reason}). Restarting...\n`);
+            }
+            const shutdownAccepted = await requestDaemonShutdown();
+            const portReleased = shutdownAccepted && await waitForDaemonStop(3000);
+            if (!portReleased) {
+                // Stale daemon replacement failed — don't blindly spawn on an occupied port
+                throw new BrowserConnectError('Stale daemon could not be replaced', `A stale daemon (${reason}) is running but did not shut down.\n` +
+                    '  Run manually: opencli daemon stop && opencli doctor', 'daemon-not-running');
+            }
+            // Port released — fall through to spawn a fresh daemon
+            staleDaemonReplaced = true;
+        }
         // Fast path: everything ready
-        if (health.state === 'ready')
+        if (!staleDaemonReplaced && health.state === 'ready')
             return;
-        if (health.state === 'profile-required') {
+        if (!staleDaemonReplaced && health.state === 'profile-required') {
             throw new BrowserConnectError('Multiple Browser Bridge profiles are connected', 'Select one with --profile <name>, OPENCLI_PROFILE=<name>, or opencli profile use <name>.\n' +
                 'Run opencli profile list to see connected profiles.', 'profile-required');
         }
-        if (health.state === 'profile-disconnected') {
+        if (!staleDaemonReplaced && health.state === 'profile-disconnected') {
             const label = contextId ?? health.status.contextId ?? 'unknown';
             throw new BrowserConnectError(`Browser profile "${label}" is not connected`, 'Open the matching Chrome profile and make sure the OpenCLI extension is enabled, or choose another profile with opencli profile use <name>.', 'profile-disconnected');
         }
         // Daemon running but no extension
-        if (health.state === 'no-extension') {
-            // Detect stale daemon: version mismatch OR missing daemonVersion (pre-version daemon)
-            const daemonVersion = health.status?.daemonVersion;
-            const isStale = !daemonVersion || daemonVersion !== PKG_VERSION;
-            if (isStale) {
-                // Stale daemon — restart it so extension gets a fresh WebSocket endpoint
-                const reason = daemonVersion
-                    ? `v${daemonVersion} ≠ v${PKG_VERSION}`
-                    : `pre-version daemon, CLI is v${PKG_VERSION}`;
-                if (process.env.OPENCLI_VERBOSE || process.stderr.isTTY) {
-                    process.stderr.write(`⚠️  Stale daemon detected (${reason}). Restarting...\n`);
-                }
-                const shutdownAccepted = await requestDaemonShutdown();
-                const portReleased = shutdownAccepted && await waitForDaemonStop(3000);
-                if (!portReleased) {
-                    // Stale daemon replacement failed — don't blindly spawn on an occupied port
-                    throw new BrowserConnectError('Stale daemon could not be replaced', `A stale daemon (${reason}) is running but did not shut down.\n` +
-                        '  Run manually: opencli daemon stop && opencli doctor', 'daemon-not-running');
-                }
-                // Port released — fall through to spawn a fresh daemon
+        if (!staleDaemonReplaced && health.state === 'no-extension') {
+            // Same version — wait for extension to connect
+            if (process.env.OPENCLI_VERBOSE || process.stderr.isTTY) {
+                process.stderr.write('⏳ Waiting for Chrome/Chromium extension to connect...\n');
+                process.stderr.write('   Make sure Chrome or Chromium is open and the OpenCLI extension is enabled.\n');
             }
-            else {
-                // Same version — wait for extension to connect
-                if (process.env.OPENCLI_VERBOSE || process.stderr.isTTY) {
-                    process.stderr.write('⏳ Waiting for Chrome/Chromium extension to connect...\n');
-                    process.stderr.write('   Make sure Chrome or Chromium is open and the OpenCLI extension is enabled.\n');
-                }
-                if (await this._pollUntilReady(timeoutMs, contextId))
-                    return;
-                const finalHealth = await getDaemonHealth({ contextId });
-                if (finalHealth.state === 'profile-required') {
-                    throw new BrowserConnectError('Multiple Browser Bridge profiles are connected', 'Select one with --profile <name>, OPENCLI_PROFILE=<name>, or opencli profile use <name>.\n' +
-                        'Run opencli profile list to see connected profiles.', 'profile-required');
-                }
-                if (finalHealth.state === 'profile-disconnected') {
-                    const label = contextId ?? finalHealth.status.contextId ?? 'unknown';
-                    throw new BrowserConnectError(`Browser profile "${label}" is not connected`, 'Open the matching Chrome profile and make sure the OpenCLI extension is enabled, or choose another profile with opencli profile use <name>.', 'profile-disconnected');
-                }
-                throw new BrowserConnectError('Browser Bridge extension not connected', 'Make sure Chrome/Chromium is open and the extension is enabled.\n' +
-                    'If the extension is installed, try: opencli daemon stop && opencli doctor\n' +
-                    'If not installed:\n' +
-                    '  1. Download: https://github.com/jackwener/opencli/releases\n' +
-                    '  2. Open chrome://extensions → Developer Mode → Load unpacked', 'extension-not-connected');
+            if (await this._pollUntilReady(timeoutMs, contextId))
+                return;
+            const finalHealth = await getDaemonHealth({ contextId });
+            if (finalHealth.state === 'profile-required') {
+                throw new BrowserConnectError('Multiple Browser Bridge profiles are connected', 'Select one with --profile <name>, OPENCLI_PROFILE=<name>, or opencli profile use <name>.\n' +
+                    'Run opencli profile list to see connected profiles.', 'profile-required');
             }
+            if (finalHealth.state === 'profile-disconnected') {
+                const label = contextId ?? finalHealth.status.contextId ?? 'unknown';
+                throw new BrowserConnectError(`Browser profile "${label}" is not connected`, 'Open the matching Chrome profile and make sure the OpenCLI extension is enabled, or choose another profile with opencli profile use <name>.', 'profile-disconnected');
+            }
+            throw new BrowserConnectError('Browser Bridge extension not connected', 'Make sure Chrome/Chromium is open and the extension is enabled.\n' +
+                'If the extension is installed, try: opencli daemon stop && opencli doctor\n' +
+                'If not installed:\n' +
+                '  1. Download: https://github.com/jackwener/opencli/releases\n' +
+                '  2. Open chrome://extensions → Developer Mode → Load unpacked', 'extension-not-connected');
         }
         // No daemon — spawn one
         if (process.env.OPENCLI_VERBOSE || process.stderr.isTTY) {

package/dist/src/browser/cdp.d.ts

@@ -26,6 +26,7 @@ export declare class CDPBridge implements IBrowserFactory {
         workspace?: string;
         cdpEndpoint?: string;
         contextId?: string;
+        idleTimeout?: number;
     }): Promise<IPage>;
     close(): Promise<void>;
     send(method: string, params?: Record<string, unknown>, timeoutMs?: number): Promise<unknown>;

package/dist/src/browser/cdp.js

@@ -202,16 +202,58 @@ class CDPPage extends BasePage {
             : cookies;
     }
     async screenshot(options = {}) {
-        const result = await this.bridge.send('Page.captureScreenshot', {
-            format: options.format ?? 'png',
-            quality: options.format === 'jpeg' ? (options.quality ?? 80) : undefined,
-            captureBeyondViewport: options.fullPage ?? false,
-        });
-        const base64 = isRecord(result) && typeof result.data === 'string' ? result.data : '';
-        if (options.path) {
-            await saveBase64ToFile(base64, options.path);
+        const fullPage = options.fullPage === true;
+        const overrideWidth = options.width && options.width > 0 ? Math.ceil(options.width) : undefined;
+        // height is ignored under fullPage so the captureBeyondViewport path stays unchanged for users who pass --height alongside --full-page.
+        const overrideHeight = !fullPage && options.height && options.height > 0 ? Math.ceil(options.height) : undefined;
+        const needsOverride = overrideWidth !== undefined || overrideHeight !== undefined;
+        if (needsOverride) {
+            if (overrideWidth !== undefined && fullPage) {
+                await this.bridge.send('Emulation.setDeviceMetricsOverride', {
+                    mobile: false,
+                    width: overrideWidth,
+                    height: 0,
+                    deviceScaleFactor: 1,
+                });
+            }
+            let finalWidth = overrideWidth ?? 0;
+            let finalHeight = overrideHeight ?? 0;
+            if (fullPage) {
+                const metrics = await this.bridge.send('Page.getLayoutMetrics');
+                const m = isRecord(metrics) ? metrics : {};
+                const css = isRecord(m.cssContentSize) ? m.cssContentSize : undefined;
+                const fb = isRecord(m.contentSize) ? m.contentSize : undefined;
+                const size = css ?? fb;
+                if (size && typeof size.width === 'number' && typeof size.height === 'number') {
+                    if (finalWidth === 0)
+                        finalWidth = Math.ceil(size.width);
+                    finalHeight = Math.ceil(size.height);
+                }
+            }
+            await this.bridge.send('Emulation.setDeviceMetricsOverride', {
+                mobile: false,
+                width: finalWidth,
+                height: finalHeight,
+                deviceScaleFactor: 1,
+            });
+        }
+        try {
+            const result = await this.bridge.send('Page.captureScreenshot', {
+                format: options.format ?? 'png',
+                quality: options.format === 'jpeg' ? (options.quality ?? 80) : undefined,
+                captureBeyondViewport: !needsOverride && fullPage,
+            });
+            const base64 = isRecord(result) && typeof result.data === 'string' ? result.data : '';
+            if (options.path) {
+                await saveBase64ToFile(base64, options.path);
+            }
+            return base64;
+        }
+        finally {
+            if (needsOverride) {
+                await this.bridge.send('Emulation.clearDeviceMetricsOverride').catch(() => { });
+            }
         }
-        return base64;
     }
     async startNetworkCapture(pattern = '') {
         // Always update the filter pattern

package/dist/src/browser/daemon-client.d.ts

@@ -20,6 +20,10 @@ export interface DaemonCommand {
     format?: 'png' | 'jpeg';
     quality?: number;
     fullPage?: boolean;
+    /** Override viewport width in CSS pixels for screenshot (0 / undefined = use current) */
+    width?: number;
+    /** Override viewport height in CSS pixels for screenshot (0 / undefined = use current; ignored when fullPage) */
+    height?: number;
     /** Local file paths for set-file-input action */
     files?: string[];
     /** CSS selector for file input element (set-file-input action) */

package/dist/src/browser/errors.js

@@ -65,7 +65,7 @@ export function isTransientBrowserError(err) {
 export function formatBrowserConnectError(kind, detail) {
     switch (kind) {
         case 'daemon-not-running':
-            return new BrowserConnectError('Cannot connect to opencli daemon.' + (detail ? `\n\n${detail}` : ''), `The daemon should auto-start. If it keeps failing, make sure port ${DEFAULT_DAEMON_PORT} is available.`, kind);
+            return new BrowserConnectError('Cannot connect to opencli daemon.' + (detail ? `\n\n${detail}` : ''), `Run \`opencli doctor\` to diagnose, or \`opencli daemon restart\` to force a fresh daemon. Default port is ${DEFAULT_DAEMON_PORT}.`, kind);
         case 'extension-not-connected':
             return new BrowserConnectError('Browser Bridge extension is not connected.' + (detail ? `\n\n${detail}` : ''), 'Install the extension from GitHub Releases, then reload.', kind);
         case 'command-failed':

package/dist/src/browser/page.d.ts

@@ -41,7 +41,7 @@ export declare class Page extends BasePage {
         domain?: string;
         url?: string;
     }): Promise<BrowserCookie[]>;
-    /** Close the automation window in the extension */
+    /** Release the current automation tab lease in the extension */
     closeWindow(): Promise<void>;
     tabs(): Promise<unknown[]>;
     newTab(url?: string): Promise<string | undefined>;

package/dist/src/browser/page.js

@@ -143,7 +143,7 @@ export class Page extends BasePage {
         const result = await sendCommand('cookies', { ...this._wsOpt(), ...opts });
         return Array.isArray(result) ? result : [];
     }
-    /** Close the automation window in the extension */
+    /** Release the current automation tab lease in the extension */
     async closeWindow() {
         try {
             await sendCommand('close-window', { ...this._wsOpt() });
@@ -205,6 +205,8 @@ export class Page extends BasePage {
             format: options.format,
             quality: options.quality,
             fullPage: options.fullPage,
+            width: options.width,
+            height: options.height,
         });
         if (options.path) {
             await saveBase64ToFile(base64, options.path);

package/dist/src/browser/page.test.js

@@ -216,3 +216,32 @@ describe('Page active target tracking', () => {
         expect(evalCall?.[1]).not.toHaveProperty('page');
     });
 });
+describe('Page.screenshot', () => {
+    beforeEach(() => {
+        sendCommandMock.mockReset();
+        sendCommandFullMock.mockReset();
+        warnMock.mockReset();
+    });
+    it('forwards width / height / fullPage options to the bridge', async () => {
+        sendCommandMock.mockResolvedValueOnce('BASE64');
+        const page = new Page('browser:default');
+        const data = await page.screenshot({ fullPage: true, width: 1080 });
+        expect(data).toBe('BASE64');
+        expect(sendCommandMock).toHaveBeenCalledWith('screenshot', expect.objectContaining({
+            workspace: 'browser:default',
+            fullPage: true,
+            width: 1080,
+        }));
+    });
+    it('omits viewport overrides when none are set', async () => {
+        sendCommandMock.mockResolvedValueOnce('BASE64');
+        const page = new Page('browser:default');
+        await page.screenshot();
+        const call = sendCommandMock.mock.calls.at(-1);
+        expect(call?.[0]).toBe('screenshot');
+        const args = call?.[1];
+        expect(args.width).toBeUndefined();
+        expect(args.height).toBeUndefined();
+        expect(args.fullPage).toBeUndefined();
+    });
+});

package/dist/src/browser/target-errors.d.ts

@@ -15,8 +15,9 @@
  *   - selector_not_found:     0 matches
  *   - selector_ambiguous:     >1 matches for a write op without --nth
  *   - selector_nth_out_of_range: --nth beyond matches_n
+ *   - not_editable:           target exists but cannot accept text input
  */
-export type TargetErrorCode = 'not_found' | 'stale_ref' | 'invalid_selector' | 'selector_not_found' | 'selector_ambiguous' | 'selector_nth_out_of_range';
+export type TargetErrorCode = 'not_found' | 'stale_ref' | 'invalid_selector' | 'selector_not_found' | 'selector_ambiguous' | 'selector_nth_out_of_range' | 'not_editable';
 export interface TargetErrorInfo {
     code: TargetErrorCode;
     message: string;

package/dist/src/browser/target-errors.js

@@ -15,6 +15,7 @@
  *   - selector_not_found:     0 matches
  *   - selector_ambiguous:     >1 matches for a write op without --nth
  *   - selector_nth_out_of_range: --nth beyond matches_n
+ *   - not_editable:           target exists but cannot accept text input
  */
 export class TargetError extends Error {
     code;

package/dist/src/browser/target-resolver.d.ts

@@ -79,6 +79,25 @@ export declare function clickResolvedJs(opts?: {
  * Generate JS for type that uses the unified resolver.
  */
 export declare function typeResolvedJs(text: string): string;
+export type FillResolvedResult = {
+    ok: true;
+    actual: string;
+    expected: string;
+    length: number;
+    mode: 'input' | 'textarea' | 'contenteditable';
+} | {
+    ok: false;
+    actual: string;
+    expected: string;
+    length: number;
+    mode: 'input' | 'textarea' | 'contenteditable';
+} | {
+    ok: false;
+    reason: string;
+    tag?: string;
+    type?: string;
+    role?: string;
+};
 /**
  * Prepare the resolved element for native CDP Input.insertText.
  *
@@ -90,6 +109,12 @@ export declare function prepareNativeTypeResolvedJs(opts?: {
     skipScroll?: boolean;
     skipFocus?: boolean;
 }): string;
+/**
+ * Verify the exact value/text currently held by the resolved editable target.
+ * Assumes resolveTargetJs and prepareNativeTypeResolvedJs have already set
+ * `window.__resolved` to the normalized editable host.
+ */
+export declare function verifyFilledResolvedJs(expected: string): string;
 /**
  * Generate JS for scrollTo that uses the unified resolver.
  * Assumes resolveTargetJs has been called and __resolved is set.

package/dist/src/browser/target-resolver.js

@@ -411,6 +411,49 @@ export function prepareNativeTypeResolvedJs(opts = {}) {
     })()
   `;
 }
+/**
+ * Verify the exact value/text currently held by the resolved editable target.
+ * Assumes resolveTargetJs and prepareNativeTypeResolvedJs have already set
+ * `window.__resolved` to the normalized editable host.
+ */
+export function verifyFilledResolvedJs(expected) {
+    const safeText = JSON.stringify(expected);
+    return `
+    (() => {
+      const el = window.__resolved;
+      if (!el) return { ok: false, reason: 'no_resolved_element' };
+
+      const tag = el.tagName ? el.tagName.toLowerCase() : '';
+      const isInput = el instanceof HTMLInputElement;
+      const isTextarea = el instanceof HTMLTextAreaElement;
+      const mode = el.isContentEditable
+        ? 'contenteditable'
+        : isTextarea
+          ? 'textarea'
+          : isInput
+            ? 'input'
+            : '';
+
+      if (!mode) {
+        return {
+          ok: false,
+          reason: 'not_editable',
+          tag,
+          role: el.getAttribute ? (el.getAttribute('role') || '') : '',
+        };
+      }
+
+      const actual = mode === 'contenteditable' ? (el.innerText || '') : String(el.value || '');
+      return {
+        ok: actual === ${safeText},
+        actual,
+        expected: ${safeText},
+        length: actual.length,
+        mode,
+      };
+    })()
+  `;
+}
 /**
  * Generate JS for scrollTo that uses the unified resolver.
  * Assumes resolveTargetJs has been called and __resolved is set.

package/dist/src/browser.test.js

@@ -165,6 +165,24 @@ describe('BrowserBridge state', () => {
         const bridge = new BrowserBridge();
         await expect(bridge.connect({ timeout: 0.1 })).rejects.toThrow('Stale daemon could not be replaced');
     });
+    it('attempts stale daemon replacement even when extension is connected', async () => {
+        vi.spyOn(daemonClient, 'getDaemonHealth').mockResolvedValue({
+            state: 'ready',
+            status: {
+                ok: true,
+                pid: 1,
+                uptime: 0,
+                daemonVersion: '0.0.1',
+                extensionConnected: true,
+                pending: 0,
+                memoryMB: 0,
+                port: 0,
+            },
+        });
+        vi.spyOn(daemonClient, 'requestDaemonShutdown').mockResolvedValue(false);
+        const bridge = new BrowserBridge();
+        await expect(bridge.connect({ timeout: 0.1 })).rejects.toThrow('Stale daemon could not be replaced');
+    });
 });
 describe('stealth anti-detection', () => {
     it('generates non-empty JS string', () => {

package/dist/src/build-manifest.js

@@ -32,7 +32,13 @@ const PACKAGE_ROOT = findPackageRoot(fileURLToPath(import.meta.url));
 const CLIS_DIR = path.join(PACKAGE_ROOT, 'clis');
 // Write manifest next to clis/ so both dev and installed runtime can find it.
 const OUTPUT = getCliManifestPath(CLIS_DIR);
-const CLI_MODULE_PATTERN = /\bcli\s*\(/;
+// Module is treated as a CLI command source if it either:
+//   1. Calls `cli(...)` directly (the common case), or
+//   2. Calls a factory `make<Pascal>Command(...)` from clis/_shared/ that
+//      wraps `cli(...)`. Without (2), shared-factory adapters
+//      (codex/cursor/chatwise new/status/dump/screenshot) match no `cli(`
+//      token at the top level and silently drop out of the manifest.
+const CLI_MODULE_PATTERN = /\bcli\s*\(|\bmake[A-Z]\w*Command\s*\(/;
 /**
  * Thrown by `loadManifestEntries` when an adapter file looks like a CLI
  * module (matches CLI_MODULE_PATTERN) but cannot be imported. Callers
@@ -91,13 +97,12 @@ function toManifestEntry(cmd, modulePath, sourceFile) {
         browser: cmd.browser ?? true,
         args: toManifestArgs(cmd.args),
         columns: cmd.columns,
-        timeout: cmd.timeoutSeconds,
-        deprecated: cmd.deprecated,
-        replacedBy: cmd.replacedBy,
+        defaultFormat: cmd.defaultFormat,
         type: 'js',
         modulePath,
         sourceFile,
         navigateBefore: cmd.navigateBefore,
+        browserSession: cmd.browserSession,
     };
 }
 /**

package/dist/src/build-manifest.test.js

@@ -46,8 +46,7 @@ describe('manifest helper rules', () => {
                 ],
                 domain: 'localhost',
                 navigateBefore: 'https://example.com/session',
-                deprecated: 'legacy command',
-                replacedBy: 'opencli demo new',
+                defaultFormat: 'plain',
             }),
         }));
         expect(entries).toMatchObject([
@@ -74,8 +73,7 @@ describe('manifest helper rules', () => {
                 type: 'js',
                 modulePath: `${site}/${site}.js`,
                 navigateBefore: 'https://example.com/session',
-                deprecated: 'legacy command',
-                replacedBy: 'opencli demo new',
+                defaultFormat: 'plain',
             },
         ]);
         // Verify sourceFile is included and stable for manifest consumers.
@@ -96,8 +94,6 @@ describe('manifest helper rules', () => {
                 name: 'legacy',
                 access: 'read',
                 description: 'legacy command',
-                deprecated: 'legacy is deprecated',
-                replacedBy: 'opencli demo new',
             });
             return {};
         });
@@ -112,8 +108,6 @@ describe('manifest helper rules', () => {
                 args: [],
                 type: 'js',
                 modulePath: `${site}/${site}.js`,
-                deprecated: 'legacy is deprecated',
-                replacedBy: 'opencli demo new',
             },
         ]);
         // Verify sourceFile is included

package/dist/src/capabilityRouting.d.ts

@@ -1,4 +1,19 @@
 import type { CliCommand } from './registry.js';
-/** Pipeline steps that require a live browser session. */
+/**
+ * Pipeline steps that require a live browser session.
+ *
+ * Note: this is the *subset* of registered pipeline steps that need a page;
+ * non-browser steps (fetch, select, map, filter, sort, limit, download)
+ * deliberately stay out. The full registered-step list lives in
+ * `src/pipeline/registry.ts` and is re-derived elsewhere via
+ * `getRegisteredStepNames()` (e.g. in `validate.ts`). When you add a new
+ * pipeline step, decide whether it belongs here based on whether its handler
+ * touches the IPage object — and `src/capabilityRouting.test.ts` verifies the
+ * subset relationship is intact.
+ */
 export declare const BROWSER_ONLY_STEPS: Set<string>;
+/** Internal helper: ensure BROWSER_ONLY_STEPS is a subset of registered pipeline steps. */
+export declare function _validateBrowserOnlyStepsAgainstRegistry(): {
+    extras: string[];
+};
 export declare function shouldUseBrowserSession(cmd: CliCommand): boolean;

package/dist/src/capabilityRouting.js

@@ -1,8 +1,21 @@
-/** Pipeline steps that require a live browser session. */
+import { getRegisteredStepNames } from './pipeline/registry.js';
+/**
+ * Pipeline steps that require a live browser session.
+ *
+ * Note: this is the *subset* of registered pipeline steps that need a page;
+ * non-browser steps (fetch, select, map, filter, sort, limit, download)
+ * deliberately stay out. The full registered-step list lives in
+ * `src/pipeline/registry.ts` and is re-derived elsewhere via
+ * `getRegisteredStepNames()` (e.g. in `validate.ts`). When you add a new
+ * pipeline step, decide whether it belongs here based on whether its handler
+ * touches the IPage object — and `src/capabilityRouting.test.ts` verifies the
+ * subset relationship is intact.
+ */
 export const BROWSER_ONLY_STEPS = new Set([
     'navigate',
     'click',
     'type',
+    'fill',
     'wait',
     'press',
     'snapshot',
@@ -10,6 +23,16 @@ export const BROWSER_ONLY_STEPS = new Set([
     'intercept',
     'tap',
 ]);
+/** Internal helper: ensure BROWSER_ONLY_STEPS is a subset of registered pipeline steps. */
+export function _validateBrowserOnlyStepsAgainstRegistry() {
+    const registered = new Set(getRegisteredStepNames());
+    const extras = [];
+    for (const step of BROWSER_ONLY_STEPS) {
+        if (!registered.has(step))
+            extras.push(step);
+    }
+    return { extras };
+}
 function pipelineNeedsBrowserSession(pipeline) {
     return pipeline.some((step) => {
         if (!step || typeof step !== 'object')

package/dist/src/capabilityRouting.test.js

@@ -1,6 +1,7 @@
 import { describe, expect, it } from 'vitest';
 import { Strategy } from './registry.js';
-import { shouldUseBrowserSession } from './capabilityRouting.js';
+import { BROWSER_ONLY_STEPS, _validateBrowserOnlyStepsAgainstRegistry, shouldUseBrowserSession } from './capabilityRouting.js';
+import { getRegisteredStepNames } from './pipeline/registry.js';
 function makeCmd(partial) {
     return {
         site: 'test',
@@ -42,4 +43,21 @@ describe('shouldUseBrowserSession', () => {
             func: async () => [],
         }))).toBe(true);
     });
+    it('routes pipelines containing the fill step into a browser session', () => {
+        expect(shouldUseBrowserSession(makeCmd({
+            browser: true,
+            strategy: Strategy.PUBLIC,
+            pipeline: [{ navigate: 'https://example.com' }, { fill: { ref: '#q', text: 'hello' } }],
+        }))).toBe(true);
+    });
+});
+describe('BROWSER_ONLY_STEPS / pipeline registry alignment', () => {
+    it('is a subset of registered pipeline step names', () => {
+        const { extras } = _validateBrowserOnlyStepsAgainstRegistry();
+        expect(extras).toEqual([]);
+    });
+    it('includes fill (DOM-touching step added in PR #1222)', () => {
+        expect(BROWSER_ONLY_STEPS.has('fill')).toBe(true);
+        expect(getRegisteredStepNames()).toContain('fill');
+    });
 });