publishport-opencli 1.8.5-pp.2 → 1.8.5-pp.21

package/dist/src/build-manifest.js

@@ -1,363 +1,16 @@
 #!/usr/bin/env node
-/**
- * Build-time CLI manifest compiler.
- *
- * Scans all JS CLI definitions in clis/ and pre-compiles them into a single
- * manifest.json for instant cold-start registration.
- *
- * Usage: npx tsx src/build-manifest.ts [--allow-removals[=N]]
- *
- * Output: cli-manifest.json next to clis/
- *
- * Safety invariants:
- *   - Adapters whose source file does not call `cli(...)` are silently
- *     skipped (they are helpers / type modules, not commands).
- *   - Adapters that look like commands but fail to import are reported as
- *     failures, the manifest is NOT written, and the process exits 1. This
- *     prevents a stale dist or a broken adapter from silently dropping
- *     other adapters' entries (root cause of the "manifest lost 478 lines"
- *     incident).
- *   - Net-deletions vs the existing committed manifest abort the build by
- *     default; pass `--allow-removals=N` (or just `--allow-removals` for any
- *     amount) to confirm an intentional removal.
- */
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-import { fileURLToPath, pathToFileURL } from 'node:url';
-import { getErrorMessage } from './errors.js';
-import { fullName, getRegistry } from './registry.js';
-import { findPackageRoot, getCliManifestPath } from './package-paths.js';
-import { isRecord } from './utils.js';
-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);
-// 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*\(|\bregisterSiteAuthCommands\s*\(|\bregisterArticleLogin\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
- * decide whether to abort or aggregate failures across the whole scan.
- */
-export class ManifestImportError extends Error {
-    filePath;
-    cause;
-    constructor(filePath, cause) {
-        super(`failed to scan ${filePath}: ${getErrorMessage(cause)}`);
-        this.filePath = filePath;
-        this.cause = cause;
-        this.name = 'ManifestImportError';
-    }
-}
-function toManifestArgs(args) {
-    return args.map(arg => ({
-        name: arg.name,
-        type: arg.type ?? 'str',
-        default: arg.default,
-        required: !!arg.required,
-        valueRequired: !!arg.valueRequired || undefined,
-        positional: arg.positional || undefined,
-        help: arg.help ?? '',
-        choices: arg.choices,
-    }));
-}
-function toModulePath(filePath, site) {
-    const baseName = path.basename(filePath, path.extname(filePath));
-    return `${site}/${baseName}.js`;
-}
-export function normalizeManifestPath(relativePath) {
-    return relativePath.replace(/\\/g, '/');
-}
-function toManifestRelativePath(filePath, clisDir) {
-    return normalizeManifestPath(path.relative(clisDir, filePath));
-}
-function isCliCommandValue(value, site) {
-    return isRecord(value)
-        && typeof value.site === 'string'
-        && value.site === site
-        && typeof value.name === 'string'
-        && (value.access === 'read' || value.access === 'write')
-        && Array.isArray(value.args);
-}
-function toManifestEntry(cmd, modulePath, sourceFile) {
-    return {
-        site: cmd.site,
-        name: cmd.name,
-        aliases: cmd.aliases,
-        description: cmd.description ?? '',
-        access: cmd.access,
-        example: cmd.example,
-        domain: cmd.domain,
-        strategy: (cmd.strategy ?? 'public').toString().toLowerCase(),
-        browser: cmd.browser ?? true,
-        args: toManifestArgs(cmd.args),
-        columns: cmd.columns,
-        defaultFormat: cmd.defaultFormat,
-        type: 'js',
-        modulePath,
-        sourceFile,
-        navigateBefore: cmd.navigateBefore,
-        siteSession: cmd.siteSession,
-        defaultWindowMode: cmd.defaultWindowMode,
-    };
-}
-/**
- * Load all manifest entries from a single adapter file.
- *
- * Returns `[]` for files that do not register a CLI command (helpers, types).
- * Throws `ManifestImportError` when a file looks like a CLI module but its
- * import or post-import processing fails — callers must decide whether to
- * surface or aggregate the failure.
- *
- * The third argument `clisDir` is used to compute the POSIX-style
- * `sourceFile` relative path; it defaults to the package's `clis/` dir so
- * existing test callers stay backward-compatible.
- */
-export async function loadManifestEntries(filePath, site, importer = moduleHref => import(moduleHref), clisDir = CLIS_DIR) {
-    let src;
-    try {
-        src = fs.readFileSync(filePath, 'utf-8');
-    }
-    catch (err) {
-        throw new ManifestImportError(filePath, err);
-    }
-    // Helper / test modules that do not call cli() are not commands.
-    if (!CLI_MODULE_PATTERN.test(src))
-        return [];
-    try {
-        const modulePath = toModulePath(filePath, site);
-        const registry = getRegistry();
-        const before = new Map(registry.entries());
-        const mod = await importer(pathToFileURL(filePath).href);
-        const exportedCommands = Object.values(isRecord(mod) ? mod : {})
-            .filter(value => isCliCommandValue(value, site));
-        const runtimeCommands = exportedCommands.length > 0
-            ? exportedCommands
-            : [...registry.entries()]
-                .filter(([key, cmd]) => {
-                if (cmd.site !== site)
-                    return false;
-                const previous = before.get(key);
-                return !previous || previous !== cmd;
-            })
-                .map(([, cmd]) => cmd);
-        // Manifest paths are cross-platform artifacts; keep them POSIX-style even
-        // when build-manifest runs on Windows.
-        const sourceRelative = toManifestRelativePath(filePath, clisDir);
-        const seen = new Set();
-        return runtimeCommands
-            .filter((cmd) => {
-            const key = fullName(cmd);
-            if (seen.has(key))
-                return false;
-            seen.add(key);
-            return true;
-        })
-            .sort((a, b) => a.name.localeCompare(b.name))
-            .map(cmd => toManifestEntry(cmd, modulePath, sourceRelative));
-    }
-    catch (err) {
-        throw new ManifestImportError(filePath, err);
-    }
-}
-/**
- * Scan a `clis/` directory and aggregate per-adapter results. Import
- * failures are collected in `failures` instead of crashing the whole scan,
- * but the caller (e.g. `main()`) is expected to fail loud if any failure
- * is present.
- */
-export async function scanClisDir(clisDir, importer = moduleHref => import(moduleHref)) {
-    const manifest = new Map();
-    const failures = [];
-    if (!fs.existsSync(clisDir)) {
-        return { entries: [], failures };
-    }
-    for (const site of fs.readdirSync(clisDir)) {
-        const siteDir = path.join(clisDir, site);
-        if (!fs.statSync(siteDir).isDirectory())
-            continue;
-        for (const file of fs.readdirSync(siteDir)) {
-            if (file.endsWith('.js') && !file.endsWith('.d.js') && !file.endsWith('.test.js') && file !== 'index.js') {
-                const filePath = path.join(siteDir, file);
-                try {
-                    const entries = await loadManifestEntries(filePath, site, importer, clisDir);
-                    for (const entry of entries) {
-                        const key = `${entry.site}/${entry.name}`;
-                        manifest.set(key, entry);
-                    }
-                }
-                catch (err) {
-                    if (err instanceof ManifestImportError) {
-                        failures.push(err);
-                        continue;
-                    }
-                    throw err;
-                }
-            }
-        }
-    }
-    const entries = [...manifest.values()].sort((a, b) => a.site.localeCompare(b.site) || a.name.localeCompare(b.name));
-    return { entries, failures };
-}
-export async function buildManifest() {
-    return scanClisDir(CLIS_DIR);
-}
-export function serializeManifest(manifest) {
-    return `${JSON.stringify(manifest, null, 2)}\n`;
-}
-export function findManifestMetadataIssues(entries) {
-    const issues = [];
-    for (const entry of entries) {
-        if (!Array.isArray(entry.args))
-            continue;
-        for (const arg of entry.args) {
-            if (!arg.positional)
-                continue;
-            const help = typeof arg.help === 'string' ? arg.help.trim() : '';
-            if (help === '') {
-                issues.push({
-                    site: entry.site,
-                    command: entry.name,
-                    arg: arg.name,
-                    sourceFile: entry.sourceFile,
-                    reason: 'positional arg missing non-empty `help` text',
-                });
-            }
-        }
-    }
-    return issues;
-}
-/**
- * Diff helper: returns site/name keys that exist in `prev` but not in
- * `next`. Used as a safety net to detect accidental mass-deletions caused
- * by silently failing adapter imports.
- */
-export function diffRemovedEntries(prev, next) {
-    const nextKeys = new Set(next.map(e => `${e.site}/${e.name}`));
-    return prev
-        .map(e => `${e.site}/${e.name}`)
-        .filter(key => !nextKeys.has(key))
-        .sort();
-}
-/**
- * Parse `--allow-removals` and `--allow-removals=N` from argv.
- * Bare `--allow-removals` disables the safety net (`Infinity`); the
- * numeric form sets an explicit upper bound.
- */
-export function parseBuildManifestArgs(argv) {
-    let allowRemovals = 0;
-    for (const arg of argv) {
-        if (arg === '--allow-removals') {
-            allowRemovals = Number.POSITIVE_INFINITY;
-            continue;
-        }
-        const m = arg.match(/^--allow-removals=(\d+)$/);
-        if (m) {
-            allowRemovals = Number.parseInt(m[1], 10);
-            continue;
-        }
-    }
-    return { allowRemovals };
-}
-function readExistingManifest(filePath) {
-    try {
-        if (!fs.existsSync(filePath))
-            return null;
-        const raw = fs.readFileSync(filePath, 'utf-8');
-        const parsed = JSON.parse(raw);
-        return Array.isArray(parsed) ? parsed : null;
-    }
-    catch {
-        return null;
-    }
-}
-async function main() {
-    // Runtime guard: refuse to run from dist/. tsc transitively emits this
-    // file (the test file imports from it) so dist/src/build-manifest.js
-    // physically exists. If a developer or agent runs that compiled copy,
-    // any stale dist will silently break adapter imports — the exact failure
-    // mode this script is meant to prevent. Direct them at the tsx entry
-    // before they can shoot themselves in the foot.
-    if (fileURLToPath(import.meta.url).includes(`${path.sep}dist${path.sep}`)) {
-        process.stderr.write(`❌ Refusing to run build-manifest from dist/.\n`
-            + `   Stale compiled output silently drops adapters that import renamed/removed exports.\n`
-            + `   Run \`npm run build-manifest\` (or \`tsx src/build-manifest.ts\`) from the source tree instead.\n`);
-        process.exit(1);
-    }
-    const args = parseBuildManifestArgs(process.argv.slice(2));
-    const { entries, failures } = await buildManifest();
-    if (failures.length > 0) {
-        process.stderr.write(`❌ ${failures.length} adapter(s) failed to load:\n`);
-        for (const failure of failures) {
-            const rel = path.relative(PACKAGE_ROOT, failure.filePath) || failure.filePath;
-            process.stderr.write(`  - ${rel}: ${getErrorMessage(failure.cause)}\n`);
-        }
-        process.stderr.write(`\nManifest NOT written. Likely cause: stale dist/ or a broken adapter import.\n`
-            + `Always run via tsx (\`npm run build-manifest\`), not against compiled dist/.\n`);
-        process.exit(1);
-    }
-    const metadataIssues = findManifestMetadataIssues(entries);
-    if (metadataIssues.length > 0) {
-        process.stderr.write(`❌ ${metadataIssues.length} positional arg(s) missing \`help\` text:\n`);
-        for (const issue of metadataIssues) {
-            const where = issue.sourceFile ? ` (${issue.sourceFile})` : '';
-            process.stderr.write(`  - ${issue.site}/${issue.command} positional "${issue.arg}"${where}\n`);
-        }
-        process.stderr.write(`\nEvery positional arg must declare a non-empty \`help\` string so\n`
-            + `\`opencli <site> <cmd> --help\` shows callers what the parameter is for.\n`
-            + `Add \`help: '...'\` to each arg above and re-run the build.\n`);
-        process.exit(1);
-    }
-    const existing = readExistingManifest(OUTPUT);
-    if (existing) {
-        const removed = diffRemovedEntries(existing, entries);
-        if (removed.length > args.allowRemovals) {
-            process.stderr.write(`❌ ${removed.length} manifest entries would be removed; refusing to overwrite.\n`);
-            const preview = removed.slice(0, 20);
-            for (const key of preview)
-                process.stderr.write(`  - ${key}\n`);
-            if (removed.length > preview.length) {
-                process.stderr.write(`  ... ${removed.length - preview.length} more\n`);
-            }
-            process.stderr.write(`\nIf this removal is intentional, rerun with `
-                + `\`--allow-removals=${removed.length}\` (or \`--allow-removals\` to disable the check).\n`);
-            process.exit(1);
-        }
-    }
-    fs.mkdirSync(path.dirname(OUTPUT), { recursive: true });
-    fs.writeFileSync(OUTPUT, serializeManifest(entries));
-    console.log(`✅ Manifest compiled: ${entries.length} entries → ${OUTPUT}`);
-    // Restore executable permissions on bin entries.
-    // tsc does not preserve the +x bit, so after a clean rebuild the CLI
-    // entry-point loses its executable permission, causing "Permission denied".
-    // See: https://github.com/jackwener/opencli/issues/446
-    if (process.platform !== 'win32') {
-        const projectRoot = PACKAGE_ROOT;
-        const pkgPath = path.resolve(projectRoot, 'package.json');
-        try {
-            const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
-            const bins = typeof pkg.bin === 'string'
-                ? { [pkg.name ?? 'cli']: pkg.bin }
-                : pkg.bin ?? {};
-            for (const binPath of Object.values(bins)) {
-                const abs = path.resolve(projectRoot, binPath);
-                if (fs.existsSync(abs)) {
-                    fs.chmodSync(abs, 0o755);
-                    console.log(`✅ Restored executable permission: ${binPath}`);
-                }
-            }
-        }
-        catch {
-            // Best-effort; never break the build for a permission fix.
-        }
-    }
-}
-const entrypoint = process.argv[1] ? pathToFileURL(path.resolve(process.argv[1])).href : null;
-if (entrypoint === import.meta.url) {
-    void main();
-}
+import*as V from"node:fs";import*as $ from"node:path";import{fileURLToPath as A,pathToFileURL as L}from"node:url";import{getErrorMessage as O}from"./errors.js";import{fullName as C,getRegistry as b}from"./registry.js";import{findPackageRoot as k,getCliManifestPath as E}from"./package-paths.js";import{isRecord as U}from"./utils.js";const j=k(A(import.meta.url)),w=$.join(j,"clis"),H=E(w),I=/\bcli\s*\(|\bregisterSiteAuthCommands\s*\(|\bregisterArticleLogin\s*\(|\bregisterTokenAuth\s*\(|\bmake[A-Z]\w*Command\s*\(/;export class ManifestImportError extends Error{filePath;cause;constructor(q,J){super(`failed to scan ${q}: ${O(J)}`);this.filePath=q;this.cause=J;this.name="ManifestImportError"}}function T(q){return q.map((J)=>({name:J.name,type:J.type??"str",default:J.default,required:!!J.required,valueRequired:!!J.valueRequired||void 0,positional:J.positional||void 0,help:J.help??"",choices:J.choices}))}function D(q,J){const W=$.basename(q,$.extname(q));return`${J}/${W}.js`}export function normalizeManifestPath(q){return q.replace(/\\/g,"/")}function M(q,J){return normalizeManifestPath($.relative(J,q))}function y(q,J){return U(q)&&typeof q.site==="string"&&q.site===J&&typeof q.name==="string"&&(q.access==="read"||q.access==="write")&&Array.isArray(q.args)}function R(q,J,W){return{site:q.site,name:q.name,aliases:q.aliases,description:q.description??"",access:q.access,example:q.example,domain:q.domain,strategy:(q.strategy??"public").toString().toLowerCase(),browser:q.browser??!0,args:T(q.args),columns:q.columns,defaultFormat:q.defaultFormat,type:"js",modulePath:J,sourceFile:W,navigateBefore:q.navigateBefore,siteSession:q.siteSession,defaultWindowMode:q.defaultWindowMode}}export async function loadManifestEntries(q,J,W=(G)=>import(G),X=w){let G;try{G=V.readFileSync(q,"utf-8")}catch(Q){throw new ManifestImportError(q,Q)}if(!I.test(G))return[];try{const Q=D(q,J),Y=b(),Z=new Map(Y.entries()),F=await W(L(q).href),z=Object.values(U(F)?F:{}).filter((B)=>y(B,J)),N=z.length>0?z:[...Y.entries()].filter(([B,S])=>{if(S.site!==J)return!1;const x=Z.get(B);return!x||x!==S}).map(([,B])=>B),K=M(q,X),_=new Set;return N.filter((B)=>{const S=C(B);if(_.has(S))return!1;_.add(S);return!0}).sort((B,S)=>B.name.localeCompare(S.name)).map((B)=>R(B,Q,K))}catch(Q){throw new ManifestImportError(q,Q)}}export async function scanClisDir(q,J=(W)=>import(W)){const W=new Map,X=[];if(!V.existsSync(q))return{entries:[],failures:X};for(const Q of V.readdirSync(q)){const Y=$.join(q,Q);if(!V.statSync(Y).isDirectory())continue;for(const Z of V.readdirSync(Y))if(Z.endsWith(".js")&&!Z.endsWith(".d.js")&&!Z.endsWith(".test.js")&&Z!=="index.js"){const F=$.join(Y,Z);try{const z=await loadManifestEntries(F,Q,J,q);for(const N of z){const K=`${N.site}/${N.name}`;W.set(K,N)}}catch(z){if(z instanceof ManifestImportError){X.push(z);continue}throw z}}}return{entries:[...W.values()].sort((Q,Y)=>Q.site.localeCompare(Y.site)||Q.name.localeCompare(Y.name)),failures:X}}export async function buildManifest(){return scanClisDir(w)}export function serializeManifest(q){return`${JSON.stringify(q,null,2)}
+`}export function findManifestMetadataIssues(q){const J=[];for(const W of q){if(!Array.isArray(W.args))continue;for(const X of W.args){if(!X.positional)continue;if((typeof X.help==="string"?X.help.trim():"")==="")J.push({site:W.site,command:W.name,arg:X.name,sourceFile:W.sourceFile,reason:"positional arg missing non-empty `help` text"})}}return J}export function diffRemovedEntries(q,J){const W=new Set(J.map((X)=>`${X.site}/${X.name}`));return q.map((X)=>`${X.site}/${X.name}`).filter((X)=>!W.has(X)).sort()}export function parseBuildManifestArgs(q){let J=0;for(const W of q){if(W==="--allow-removals"){J=Number.POSITIVE_INFINITY;continue}const X=W.match(/^--allow-removals=(\d+)$/);if(X){J=Number.parseInt(X[1],10);continue}}return{allowRemovals:J}}function v(q){try{if(!V.existsSync(q))return null;const J=V.readFileSync(q,"utf-8"),W=JSON.parse(J);return Array.isArray(W)?W:null}catch{return null}}async function g(){if(A(import.meta.url).includes(`${$.sep}dist${$.sep}`)){process.stderr.write(`❌ Refusing to run build-manifest from dist/.
+`+"   Stale compiled output silently drops adapters that import renamed/removed exports.\n   Run `npm run build-manifest` (or `tsx src/build-manifest.ts`) from the source tree instead.\n");process.exit(1)}const q=parseBuildManifestArgs(process.argv.slice(2)),{entries:J,failures:W}=await buildManifest();if(W.length>0){process.stderr.write(`❌ ${W.length} adapter(s) failed to load:
+`);for(const Q of W){const Y=$.relative(j,Q.filePath)||Q.filePath;process.stderr.write(`  - ${Y}: ${O(Q.cause)}
+`)}process.stderr.write(`
+Manifest NOT written. Likely cause: stale dist/ or a broken adapter import.
+Always run via tsx (\`npm run build-manifest\`), not against compiled dist/.
+`);process.exit(1)}const X=findManifestMetadataIssues(J);if(X.length>0){process.stderr.write(`❌ ${X.length} positional arg(s) missing \`help\` text:
+`);for(const Q of X){const Y=Q.sourceFile?` (${Q.sourceFile})`:"";process.stderr.write(`  - ${Q.site}/${Q.command} positional "${Q.arg}"${Y}
+`)}process.stderr.write("\nEvery positional arg must declare a non-empty `help` string so\n`opencli <site> <cmd> --help` shows callers what the parameter is for.\nAdd `help: '...'` to each arg above and re-run the build.\n");process.exit(1)}const G=v(H);if(G){const Q=diffRemovedEntries(G,J);if(Q.length>q.allowRemovals){process.stderr.write(`❌ ${Q.length} manifest entries would be removed; refusing to overwrite.
+`);const Y=Q.slice(0,20);for(const Z of Y)process.stderr.write(`  - ${Z}
+`);if(Q.length>Y.length)process.stderr.write(`  ... ${Q.length-Y.length} more
+`);process.stderr.write(`
+If this removal is intentional, rerun with \`--allow-removals=${Q.length}\` (or \`--allow-removals\` to disable the check).
+`);process.exit(1)}}V.mkdirSync($.dirname(H),{recursive:!0});V.writeFileSync(H,serializeManifest(J));console.log(`✅ Manifest compiled: ${J.length} entries → ${H}`);if(process.platform!=="win32"){const Q=j,Y=$.resolve(Q,"package.json");try{const Z=JSON.parse(V.readFileSync(Y,"utf-8")),F=typeof Z.bin==="string"?{[Z.name??"cli"]:Z.bin}:Z.bin??{};for(const z of Object.values(F)){const N=$.resolve(Q,z);if(V.existsSync(N)){V.chmodSync(N,493);console.log(`✅ Restored executable permission: ${z}`)}}}catch{}}}const p=process.argv[1]?L($.resolve(process.argv[1])).href:null;if(p===import.meta.url)g();

package/dist/src/capabilityRouting.js

@@ -1,56 +1 @@
-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',
-    'evaluate',
-    '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')
-            return false;
-        return Object.keys(step).some((op) => BROWSER_ONLY_STEPS.has(op));
-    });
-}
-export function shouldUseBrowserSession(cmd) {
-    if (!cmd.browser)
-        return false;
-    if (cmd.func)
-        return true;
-    if (!cmd.pipeline || cmd.pipeline.length === 0)
-        return true;
-    // normalizeCommand sets navigateBefore to a URL string (needs pre-nav) or
-    // boolean true (needs authenticated context, no specific URL). Either way
-    // the pipeline requires a browser session even if no step is browser-only.
-    if (cmd.navigateBefore)
-        return true;
-    return pipelineNeedsBrowserSession(cmd.pipeline);
-}
+import{getRegisteredStepNames as C}from"./pipeline/registry.js";export const BROWSER_ONLY_STEPS=new Set(["navigate","click","type","fill","wait","press","snapshot","evaluate","intercept","tap"]);export function _validateBrowserOnlyStepsAgainstRegistry(){const k=new Set(C()),z=[];for(const A of BROWSER_ONLY_STEPS)if(!k.has(A))z.push(A);return{extras:z}}function D(k){return k.some((z)=>{if(!z||typeof z!=="object")return!1;return Object.keys(z).some((A)=>BROWSER_ONLY_STEPS.has(A))})}export function shouldUseBrowserSession(k){if(!k.browser)return!1;if(k.func)return!0;if(!k.pipeline||k.pipeline.length===0)return!0;if(k.navigateBefore)return!0;return D(k.pipeline)}