@unbrained/pm-cli 2026.5.18 → 2026.5.27

package/dist/cli/commands/test.js

@@ -1,5 +1,5 @@
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="dc5fb278-5d8c-539d-9059-22e92c20ae1f")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="ecb69cb1-7778-555a-92fd-ebbc7d42f47e")}catch(e){}}();
 import { spawn } from "node:child_process";
 import { cp, mkdir, mkdtemp, readdir, rm } from "node:fs/promises";
 import { tmpdir } from "node:os";
@@ -16,14 +16,13 @@ import { getSettingsPath, ITEM_FILE_EXTENSIONS, resolveGlobalPmRoot, resolvePmRo
 import { readSettings } from "../../core/store/settings.js";
 import { appendTrackedTestRunSummary } from "../../core/test/item-test-run-tracking.js";
 import { runInit } from "./init.js";
+import { looksLikeStructuredLinkedTestEntry, normalizeStructuredLinkedTestEntry } from "./linked-test-entry.js";
+import { LINKED_TEST_PM_CONTEXT_MODE_VALUES as PM_CONTEXT_MODE_VALUES, LINKED_TEST_PROTECTED_ENV_KEYS, parseLinkedTestAssertionEqualsMap, parseLinkedTestAssertionGteMap, parseLinkedTestBoolean as parseLinkedTestBooleanValue, parseLinkedTestContextMode as parseLinkedTestContextModeValue, parseLinkedTestEnvClear as parseLinkedTestEnvClearValue, parseLinkedTestEnvSet as parseLinkedTestEnvSetValue, parseLinkedTestMinLines, parseLinkedTestRegexList, parseLinkedTestStringList, } from "./linked-test-parsers.js";
 import { SCOPE_VALUES } from "../../types/index.js";
 const TEST_OUTPUT_MAX_BUFFER_BYTES = 20 * 1024 * 1024;
 const DEFAULT_LINKED_TEST_TIMEOUT_FORCE_KILL_DELAY_MS = 3000;
 const DEFAULT_LINKED_TEST_HEARTBEAT_INTERVAL_MS = 10000;
 const MAX_LINKED_TEST_COMMAND_LABEL_LENGTH = 120;
-const LINKED_TEST_PROTECTED_ENV_KEYS = new Set(["PM_PATH", "PM_GLOBAL_PATH", "FORCE_COLOR"]);
-const LINKED_TEST_ENV_NAME_PATTERN = /^[A-Za-z_][A-Za-z0-9_]*$/;
-const PM_CONTEXT_MODE_VALUES = ["schema", "tracker", "auto"];
 const LINKED_TEST_TRACKER_DIRS_TO_SKIP = new Set(["locks", "extensions", "runtime"]);
 const LINKED_TEST_ITEM_COUNT_DIRS_TO_SKIP = new Set(["history", "index", "search", "extensions", "locks", "runtime"]);
 const LINKED_TEST_INFRA_COLLISION_PATTERNS = [
@@ -154,165 +153,6 @@ function ensureScope(raw) {
     }
     return value;
 }
-function parseLinkedTestBooleanValue(raw, optionName, fieldLabel) {
-    if (!raw || raw.trim().length === 0) {
-        return undefined;
-    }
-    const normalized = raw.trim().toLowerCase();
-    if (normalized === "true" || normalized === "1" || normalized === "yes") {
-        return true;
-    }
-    if (normalized === "false" || normalized === "0" || normalized === "no") {
-        return false;
-    }
-    throw new PmCliError(`${optionName} ${fieldLabel} must be one of true|false|1|0|yes|no`, EXIT_CODE.USAGE);
-}
-function parseLinkedTestEnvSetValue(raw, optionName) {
-    if (!raw || raw.trim().length === 0) {
-        return undefined;
-    }
-    const assignments = raw
-        .split(/[;\n]/)
-        .map((entry) => entry.trim())
-        .filter((entry) => entry.length > 0);
-    if (assignments.length === 0) {
-        throw new PmCliError(`${optionName} env_set must include at least one KEY=VALUE assignment`, EXIT_CODE.USAGE);
-    }
-    const envSet = {};
-    for (const assignment of assignments) {
-        const separatorIndex = assignment.indexOf("=");
-        if (separatorIndex <= 0) {
-            throw new PmCliError(`${optionName} env_set entries must use KEY=VALUE and be separated by semicolons. Example: env_set=PORT=0;PLAYWRIGHT_BASE_URL=http://127.0.0.1:4173`, EXIT_CODE.USAGE);
-        }
-        const key = assignment.slice(0, separatorIndex).trim();
-        const value = assignment.slice(separatorIndex + 1);
-        if (!LINKED_TEST_ENV_NAME_PATTERN.test(key)) {
-            throw new PmCliError(`${optionName} env_set key "${key}" is invalid`, EXIT_CODE.USAGE);
-        }
-        if (LINKED_TEST_PROTECTED_ENV_KEYS.has(key.toUpperCase())) {
-            throw new PmCliError(`${optionName} env_set key "${key}" is reserved for sandbox safety`, EXIT_CODE.USAGE);
-        }
-        envSet[key] = value;
-    }
-    return Object.keys(envSet).length > 0 ? envSet : undefined;
-}
-function parseLinkedTestEnvClearValue(raw, optionName) {
-    if (!raw || raw.trim().length === 0) {
-        return undefined;
-    }
-    const values = [...new Set(raw.split(/[;,\n]/).map((entry) => entry.trim()).filter((entry) => entry.length > 0))];
-    if (values.length === 0) {
-        throw new PmCliError(`${optionName} env_clear must include at least one environment variable name`, EXIT_CODE.USAGE);
-    }
-    for (const key of values) {
-        if (!LINKED_TEST_ENV_NAME_PATTERN.test(key)) {
-            throw new PmCliError(`${optionName} env_clear key "${key}" is invalid`, EXIT_CODE.USAGE);
-        }
-        if (LINKED_TEST_PROTECTED_ENV_KEYS.has(key.toUpperCase())) {
-            throw new PmCliError(`${optionName} env_clear key "${key}" is reserved for sandbox safety`, EXIT_CODE.USAGE);
-        }
-    }
-    return values;
-}
-function parseLinkedTestStringList(raw) {
-    if (!raw || raw.trim().length === 0) {
-        return undefined;
-    }
-    const values = [...new Set(raw.split(/[;\n]/).map((entry) => entry.trim()).filter((entry) => entry.length > 0))];
-    return values.length > 0 ? values : undefined;
-}
-function parseLinkedTestRegexList(raw, optionName, fieldLabel) {
-    const values = parseLinkedTestStringList(raw);
-    if (!values || values.length === 0) {
-        return undefined;
-    }
-    for (const pattern of values) {
-        try {
-            // Validate syntax when linked-test metadata is added.
-            new RegExp(pattern, "m");
-        }
-        catch (error) {
-            throw new PmCliError(`${optionName} ${fieldLabel} includes invalid regex "${pattern}": ${error instanceof Error ? error.message : String(error)}`, EXIT_CODE.USAGE);
-        }
-    }
-    return values;
-}
-function parseLinkedTestMinLines(raw, optionName) {
-    if (!raw || raw.trim().length === 0) {
-        return undefined;
-    }
-    const parsed = parseOptionalNumber(raw, "assert_stdout_min_lines");
-    if (!Number.isInteger(parsed) || parsed < 0) {
-        throw new PmCliError(`${optionName} assert_stdout_min_lines must be an integer >= 0`, EXIT_CODE.USAGE);
-    }
-    return parsed;
-}
-function parseLinkedTestAssertionEqualsMap(raw, optionName) {
-    if (!raw || raw.trim().length === 0) {
-        return undefined;
-    }
-    const assignments = raw
-        .split(/[;\n]/)
-        .map((entry) => entry.trim())
-        .filter((entry) => entry.length > 0);
-    if (assignments.length === 0) {
-        throw new PmCliError(`${optionName} assert_json_field_equals must include at least one path=value assignment`, EXIT_CODE.USAGE);
-    }
-    const values = {};
-    for (const assignment of assignments) {
-        const separatorIndex = assignment.indexOf("=");
-        if (separatorIndex <= 0) {
-            throw new PmCliError(`${optionName} assert_json_field_equals entries must use path=value and be separated by semicolons`, EXIT_CODE.USAGE);
-        }
-        const key = assignment.slice(0, separatorIndex).trim();
-        const value = assignment.slice(separatorIndex + 1).trim();
-        if (key.length === 0 || value.length === 0) {
-            throw new PmCliError(`${optionName} assert_json_field_equals entries must include non-empty path and value`, EXIT_CODE.USAGE);
-        }
-        values[key] = value;
-    }
-    return Object.keys(values).length > 0 ? values : undefined;
-}
-function parseLinkedTestAssertionGteMap(raw, optionName) {
-    if (!raw || raw.trim().length === 0) {
-        return undefined;
-    }
-    const assignments = raw
-        .split(/[;\n]/)
-        .map((entry) => entry.trim())
-        .filter((entry) => entry.length > 0);
-    if (assignments.length === 0) {
-        throw new PmCliError(`${optionName} assert_json_field_gte must include at least one path=value assignment`, EXIT_CODE.USAGE);
-    }
-    const values = {};
-    for (const assignment of assignments) {
-        const separatorIndex = assignment.indexOf("=");
-        if (separatorIndex <= 0) {
-            throw new PmCliError(`${optionName} assert_json_field_gte entries must use path=value and be separated by semicolons`, EXIT_CODE.USAGE);
-        }
-        const key = assignment.slice(0, separatorIndex).trim();
-        const valueRaw = assignment.slice(separatorIndex + 1).trim();
-        if (key.length === 0 || valueRaw.length === 0) {
-            throw new PmCliError(`${optionName} assert_json_field_gte entries must include non-empty path and value`, EXIT_CODE.USAGE);
-        }
-        const value = Number.parseFloat(valueRaw);
-        if (!Number.isFinite(value)) {
-            throw new PmCliError(`${optionName} assert_json_field_gte value for "${key}" must be numeric`, EXIT_CODE.USAGE);
-        }
-        values[key] = value;
-    }
-    return Object.keys(values).length > 0 ? values : undefined;
-}
-function parseLinkedTestContextModeValue(raw, optionName) {
-    if (!raw || raw.trim().length === 0) {
-        return undefined;
-    }
-    const normalized = raw.trim().toLowerCase();
-    if (PM_CONTEXT_MODE_VALUES.includes(normalized)) {
-        return normalized;
-    }
-    throw new PmCliError(`${optionName} pm_context_mode must be one of: ${PM_CONTEXT_MODE_VALUES.join(", ")}`, EXIT_CODE.USAGE);
-}
 function parsePmContextMode(raw) {
     if (!raw) {
         return "schema";
@@ -390,191 +230,10 @@ function mergeEnvClearDirectives(entries, optionName) {
     }
     return [...new Set(values)];
 }
-const PM_GLOBAL_FLAGS_WITH_VALUE = new Set(["--path"]);
-const NPX_FLAGS_WITH_VALUE = new Set(["-p", "--package", "-c", "--call"]);
-const PNPM_GLOBAL_FLAGS_WITH_VALUE = new Set([
-    "-c",
-    "-C",
-    "--config",
-    "--dir",
-    "--filter",
-    "--workspace-dir",
-]);
-const NPM_GLOBAL_FLAGS_WITH_VALUE = new Set(["-C", "--prefix", "--userconfig", "--cache"]);
-const YARN_GLOBAL_FLAGS_WITH_VALUE = new Set(["--cwd"]);
-const BUN_GLOBAL_FLAGS_WITH_VALUE = new Set(["--cwd"]);
-const NPM_EXEC_SUBCOMMANDS = new Set(["exec", "x"]);
-const SCRIPT_RUN_SUBCOMMANDS = new Set(["run", "run-script"]);
-const SCRIPT_RUN_FLAGS_WITH_VALUE = new Set(["-C", "--dir", "--cwd", "-w", "--workspace", "--filter"]);
-function splitNormalizedCommandSegments(normalizedCommand) {
-    return normalizedCommand
-        .split(/&&|\|\||\||;/)
-        .map((segment) => segment.trim())
-        .filter((segment) => segment.length > 0);
-}
-function stripLeadingEnvAssignments(tokens) {
-    let start = 0;
-    if (tokens[start] === "env") {
-        start += 1;
-    }
-    while (start < tokens.length) {
-        const token = tokens[start];
-        if (/^(?:[a-z_][a-z0-9_]*|\$env:[a-z_][a-z0-9_]*)=.*/.test(token)) {
-            start += 1;
-            continue;
-        }
-        break;
-    }
-    return tokens.slice(start);
-}
-function firstPmSubcommand(args) {
-    let index = 0;
-    while (index < args.length) {
-        const token = args[index];
-        if (token === "--") {
-            index += 1;
-            continue;
-        }
-        if (token.startsWith("-")) {
-            if (PM_GLOBAL_FLAGS_WITH_VALUE.has(token)) {
-                index += 2;
-                continue;
-            }
-            index += 1;
-            continue;
-        }
-        return token;
-    }
-    return undefined;
-}
-function isPmExecutableToken(token) {
-    return token === "pm" || token.endsWith("/pm") || token.endsWith("/pm.cmd") || token.endsWith("/pm.exe");
-}
-function normalizePackageSpecifier(token) {
-    const trimmed = token.trim();
-    if (!trimmed.startsWith("@")) {
-        const versionSeparator = trimmed.indexOf("@");
-        return versionSeparator === -1 ? trimmed : trimmed.slice(0, versionSeparator);
-    }
-    const scopeSeparator = trimmed.indexOf("/");
-    if (scopeSeparator === -1) {
-        return trimmed;
-    }
-    const versionSeparator = trimmed.indexOf("@", scopeSeparator + 1);
-    return versionSeparator === -1 ? trimmed : trimmed.slice(0, versionSeparator);
-}
-function isPmCliPackageToken(token) {
-    const normalizedSpecifier = normalizePackageSpecifier(token);
-    return (normalizedSpecifier === "pm-cli" ||
-        normalizedSpecifier.endsWith("/pm-cli") ||
-        token === "pm-cli" ||
-        token.endsWith("/pm-cli"));
-}
-function isPmCliScriptToken(token) {
-    return token === "dist/cli.js" || token === "./dist/cli.js" || token.endsWith("/dist/cli.js");
-}
-function parseNpxCommand(tokens) {
-    let index = 0;
-    while (index < tokens.length) {
-        const token = tokens[index];
-        if (token === "--") {
-            index += 1;
-            break;
-        }
-        if (!token.startsWith("-")) {
-            break;
-        }
-        if (token.includes("=")) {
-            index += 1;
-            continue;
-        }
-        if (NPX_FLAGS_WITH_VALUE.has(token)) {
-            index += 2;
-            continue;
-        }
-        index += 1;
-    }
-    const command = tokens[index];
-    if (!command) {
-        return null;
-    }
-    return {
-        command,
-        args: tokens.slice(index + 1),
-    };
-}
-function parseLauncherSubcommand(tokens, flagsWithValue) {
-    let index = 0;
-    while (index < tokens.length) {
-        const token = tokens[index];
-        if (token === "--") {
-            index += 1;
-            continue;
-        }
-        if (token.startsWith("-")) {
-            if (token.includes("=")) {
-                index += 1;
-                continue;
-            }
-            if (flagsWithValue.has(token)) {
-                index += 2;
-                continue;
-            }
-            index += 1;
-            continue;
-        }
-        return {
-            subcommand: token,
-            args: tokens.slice(index + 1),
-        };
-    }
-    return null;
-}
-function parsePnpmDlxCommand(tokens) {
-    const parsed = parseLauncherSubcommand(tokens, PNPM_GLOBAL_FLAGS_WITH_VALUE);
-    if (parsed?.subcommand !== "dlx") {
-        return null;
-    }
-    return parseNpxCommand(parsed.args);
-}
-function parseNpmExecCommand(tokens) {
-    const parsed = parseLauncherSubcommand(tokens, NPM_GLOBAL_FLAGS_WITH_VALUE);
-    if (!parsed || !NPM_EXEC_SUBCOMMANDS.has(parsed.subcommand)) {
-        return null;
-    }
-    return parseNpxCommand(parsed.args);
-}
-function resolvePmSubcommandContext(args) {
-    let index = 0;
-    while (index < args.length) {
-        const token = args[index];
-        if (token === "--") {
-            index += 1;
-            continue;
-        }
-        if (token.startsWith("-")) {
-            if (PM_GLOBAL_FLAGS_WITH_VALUE.has(token)) {
-                index += 2;
-                continue;
-            }
-            index += 1;
-            continue;
-        }
-        return {
-            subcommand: token,
-            remaining: args.slice(index + 1),
-        };
-    }
-    return null;
-}
-function firstPositionalToken(tokens) {
-    for (const token of tokens) {
-        if (!token.startsWith("-")) {
-            return token;
-        }
-    }
-    return undefined;
-}
+// Linked-test command detection helpers + constants live in a sibling leaf
+// module to keep this command file under the per-file LOC budget. Re-exported
+// here so no consumer outside this file changes.
+import { BUN_GLOBAL_FLAGS_WITH_VALUE, NPM_GLOBAL_FLAGS_WITH_VALUE, PNPM_GLOBAL_FLAGS_WITH_VALUE, SCRIPT_RUN_FLAGS_WITH_VALUE, SCRIPT_RUN_SUBCOMMANDS, YARN_GLOBAL_FLAGS_WITH_VALUE, firstPmSubcommand, firstPositionalToken, isPmCliPackageToken, isPmCliScriptToken, isPmExecutableToken, parseLauncherSubcommand, parseNpmExecCommand, parseNpxCommand, parsePnpmDlxCommand, resolvePmSubcommandContext, splitNormalizedCommandSegments, stripLeadingEnvAssignments, } from "./test/linked-command-detection.js";
 function looksLikePrefixedItemId(token, idPrefix) {
     const normalizedPrefix = idPrefix.trim().toLowerCase().replace(/-+$/, "");
     if (normalizedPrefix.length === 0) {
@@ -789,18 +448,14 @@ function getRuntimeSafetySkipReason(command) {
         return undefined;
     return 'Linked test command skipped: Linked test commands must not invoke "pm test-all"; this creates recursive orchestration.';
 }
-function looksLikeStructuredLinkedTestEntry(raw) {
-    if (raw.startsWith("```") || raw.includes("\n")) {
-        return true;
-    }
-    return /^(?:[-*+]\s+)?(?:command|path|scope|timeout|timeout_seconds|pm_context_mode|env_set|env_clear|shared_host_safe|assert_stdout_contains|assert_stdout_regex|assert_stderr_contains|assert_stderr_regex|assert_stdout_min_lines|assert_json_field_equals|assert_json_field_gte|note)\s*[:=]/i.test(raw);
-}
 function parseAddEntries(raw) {
     if (!raw)
         return [];
     return raw.map((entry) => {
         const trimmed = entry.trim();
-        const kv = looksLikeStructuredLinkedTestEntry(trimmed) ? parseCsvKv(entry, "--add") : { command: trimmed };
+        const kv = looksLikeStructuredLinkedTestEntry(trimmed)
+            ? normalizeStructuredLinkedTestEntry(parseCsvKv(entry, "--add"), "--add")
+            : { command: trimmed };
         const command = kv.command?.trim() || undefined;
         const filePath = kv.path?.trim() || undefined;
         if (!command) {
@@ -1668,7 +1323,7 @@ export async function runLinkedTests(tests, defaultTimeoutSeconds, options) {
         }
     }
     finally {
-        await rm(sandboxRoot, { recursive: true, force: true });
+        await rm(sandboxRoot, { recursive: true, force: true, maxRetries: 3, retryDelay: 100 });
     }
     return results;
 }
@@ -1827,4 +1482,4 @@ export async function runTest(id, options, global) {
     };
 }
 //# sourceMappingURL=test.js.map
-//# debugId=dc5fb278-5d8c-539d-9059-22e92c20ae1f
+//# debugId=ecb69cb1-7778-555a-92fd-ebbc7d42f47e