@clipboard-health/groundcrew 4.7.2 → 4.8.0

package/dist/lib/config.js

@@ -1,7 +1,7 @@
 import { __rewriteRelativeImportExtension } from "tslib";
 import { existsSync } from "node:fs";
 import { homedir } from "node:os";
-import { resolve } from "node:path";
+import path from "node:path";
 import { pathToFileURL } from "node:url";
 import { cosmiconfig } from "cosmiconfig";
 import { debug, log, readEnvironmentVariable, setLogFile } from "./util.js";
@@ -35,7 +35,7 @@ const DEFAULT_ORCHESTRATOR = {
     pollIntervalMilliseconds: 120_000,
     sessionLimitPercentage: 85,
 };
-const DEFAULT_MODEL_DEFINITIONS = {
+const BUILT_IN_MODEL_DEFINITIONS = {
     claude: {
         cmd: "claude --permission-mode auto",
         color: "#C15F3C",
@@ -47,6 +47,30 @@ const DEFAULT_MODEL_DEFINITIONS = {
         usage: { codexbar: { provider: "codex" } },
     },
 };
+const MODEL_DEFINITIONS_MIGRATION_MESSAGE = [
+    "configuration migration required: models are no longer enabled by default.",
+    "",
+    "Add the models you want to use:",
+    "",
+    "models: {",
+    '  default: "claude",',
+    "  definitions: {",
+    "    claude: {},",
+    "  },",
+    "},",
+    "",
+    "To keep the previous claude+codex behavior:",
+    "",
+    "models: {",
+    '  default: "claude",',
+    "  definitions: {",
+    "    claude: {},",
+    "    codex: {},",
+    "  },",
+    "},",
+    "",
+    "`disabled: true` is no longer supported; remove disabled model entries instead.",
+].join("\n");
 const DEFAULT_PROMPT_INITIAL = [
     "You are working on Linear ticket {{ticket}} ({{title}}) in the {{worktree}} worktree subdirectory.",
     "",
@@ -88,7 +112,7 @@ function expandHome(p) {
         return homedir();
     }
     if (p.startsWith("~/")) {
-        return resolve(homedir(), p.slice(2));
+        return path.resolve(homedir(), p.slice(2));
     }
     return p;
 }
@@ -98,45 +122,45 @@ function fail(message) {
 function isNonEmptyString(value) {
     return typeof value === "string" && value.length > 0;
 }
-function requireString(value, path) {
+function requireString(value, configKey) {
     if (!isNonEmptyString(value)) {
-        fail(`${path} must be a non-empty string (got ${JSON.stringify(value)})`);
+        fail(`${configKey} must be a non-empty string (got ${JSON.stringify(value)})`);
     }
 }
-function requirePositiveInt(value, path, min = 1) {
+function requirePositiveInt(value, configKey, min = 1) {
     if (typeof value !== "number" || !Number.isInteger(value) || value < min) {
-        fail(`${path} must be an integer ≥ ${min} (got ${JSON.stringify(value)})`);
+        fail(`${configKey} must be an integer ≥ ${min} (got ${JSON.stringify(value)})`);
     }
 }
-function requirePercent(value, path) {
+function requirePercent(value, configKey) {
     if (typeof value !== "number" ||
         !Number.isFinite(value) ||
         value <= PERCENT_MIN_EXCLUSIVE ||
         value > PERCENT_MAX) {
-        fail(`${path} must be a finite number in (0, 100] (got ${JSON.stringify(value)})`);
+        fail(`${configKey} must be a finite number in (0, 100] (got ${JSON.stringify(value)})`);
     }
 }
 function cloneModelDefinition(definition) {
     return structuredClone(definition);
 }
-function normalizeOptionalString(value, path) {
+function normalizeOptionalString(value, configKey) {
     if (value === undefined) {
         return undefined;
     }
     if (typeof value !== "string" || value.trim().length === 0) {
-        fail(`${path} must be a non-empty string`);
+        fail(`${configKey} must be a non-empty string`);
     }
     return value.trim();
 }
 const ENV_VAR_NAME_PATTERN = /^[A-Za-z_][A-Za-z0-9_]*$/;
 function validatePreLaunchEnv(modelName, value) {
-    const path = `models.definitions.${modelName}.preLaunchEnv`;
+    const configPath = `models.definitions.${modelName}.preLaunchEnv`;
     if (!Array.isArray(value)) {
-        fail(`${path} must be an array of env var names (got ${JSON.stringify(value)})`);
+        fail(`${configPath} must be an array of env var names (got ${JSON.stringify(value)})`);
     }
     for (const [index, entry] of value.entries()) {
         if (typeof entry !== "string" || !ENV_VAR_NAME_PATTERN.test(entry)) {
-            fail(`${path}[${index}] must be a POSIX env var name matching ${ENV_VAR_NAME_PATTERN.source} (got ${JSON.stringify(entry)})`);
+            fail(`${configPath}[${index}] must be a POSIX env var name matching ${ENV_VAR_NAME_PATTERN.source} (got ${JSON.stringify(entry)})`);
         }
         // Build secrets are sourced into the host launch shell, forwarded only to
         // the Safehouse *setup* wrap, and `unset` on the host before the agent
@@ -144,7 +168,7 @@ function validatePreLaunchEnv(modelName, value) {
         // fail loudly so the operator picks a different name (or removes the
         // entry) instead of debugging a missing env var at runtime.
         if (BUILD_SECRET_NAMES.includes(entry)) {
-            fail(`${path}[${index}] cannot be a BUILD_SECRET_NAMES entry (${BUILD_SECRET_NAMES.join(", ")}); ` +
+            fail(`${configPath}[${index}] cannot be a BUILD_SECRET_NAMES entry (${BUILD_SECRET_NAMES.join(", ")}); ` +
                 "those are unset on the host before the agent wrap is exec'd, so forwarding them via --env-pass would be a no-op.");
         }
     }
@@ -165,52 +189,52 @@ export function hasPreLaunchEnv(definition) {
 function isWorkspaceKindSetting(value) {
     return (typeof value === "string" && WORKSPACE_KIND_SETTINGS.includes(value));
 }
-function normalizeWorkspaceKind(value, path) {
+function normalizeWorkspaceKind(value, configKey) {
     if (value === undefined) {
         return undefined;
     }
     if (!isWorkspaceKindSetting(value)) {
-        fail(`${path} must be one of ${WORKSPACE_KIND_SETTINGS.join(", ")} (got ${JSON.stringify(value)})`);
+        fail(`${configKey} must be one of ${WORKSPACE_KIND_SETTINGS.join(", ")} (got ${JSON.stringify(value)})`);
     }
     return value;
 }
 function isLocalRunnerSetting(value) {
     return typeof value === "string" && LOCAL_RUNNER_SETTINGS.includes(value);
 }
-function normalizeLocalRunner(value, path) {
+function normalizeLocalRunner(value, configKey) {
     if (value === undefined) {
         return undefined;
     }
     if (!isLocalRunnerSetting(value)) {
-        fail(`${path} must be one of ${LOCAL_RUNNER_SETTINGS.join(", ")} (got ${JSON.stringify(value)})`);
+        fail(`${configKey} must be one of ${LOCAL_RUNNER_SETTINGS.join(", ")} (got ${JSON.stringify(value)})`);
     }
     return value;
 }
-function normalizeSandbox(value, path) {
+function normalizeSandbox(value, configKey) {
     if (!isPlainObject(value)) {
-        fail(`${path} must be an object`);
+        fail(`${configKey} must be an object`);
     }
     if (Object.hasOwn(value, "template")) {
-        failRemovedConfigKey(`${path}.template`, "Groundcrew no longer creates or re-templates sdx sandboxes.");
+        failRemovedConfigKey(`${configKey}.template`, "Groundcrew no longer creates or re-templates sdx sandboxes.");
     }
     if (Object.hasOwn(value, "kits")) {
-        failRemovedConfigKey(`${path}.kits`, "Groundcrew no longer creates sdx sandboxes or applies sandbox kits.");
+        failRemovedConfigKey(`${configKey}.kits`, "Groundcrew no longer creates sdx sandboxes or applies sandbox kits.");
     }
     const { agent, setupCommand } = value;
-    requireString(agent, `${path}.agent`);
+    requireString(agent, `${configKey}.agent`);
     const trimmedAgent = agent.trim();
     if (trimmedAgent.length === 0) {
-        fail(`${path}.agent must be a non-empty string (got ${JSON.stringify(agent)})`);
+        fail(`${configKey}.agent must be a non-empty string (got ${JSON.stringify(agent)})`);
     }
     const sandbox = { agent: trimmedAgent };
-    const normalizedSetup = normalizeOptionalString(setupCommand, `${path}.setupCommand`);
+    const normalizedSetup = normalizeOptionalString(setupCommand, `${configKey}.setupCommand`);
     if (normalizedSetup !== undefined) {
         sandbox.setupCommand = normalizedSetup;
     }
     return sandbox;
 }
-function failRemovedConfigKey(path, reason) {
-    fail(`${path} is no longer supported: ${reason} ` +
+function failRemovedConfigKey(configKey, reason) {
+    fail(`${configKey} is no longer supported: ${reason} ` +
         "Provision and manage the sandbox yourself with `sbx` (for example `sbx create --name groundcrew-<agent> <agent> <projectDir>`), then keep only `models.definitions.<model>.sandbox.agent` plus optional `setupCommand` in crew.config.ts.");
 }
 function failIfLegacyModelKeys(name, override) {
@@ -221,23 +245,16 @@ function failIfLegacyModelKeys(name, override) {
         fail(`models.definitions.${name}.isolation is no longer supported: per-model isolation is no longer supported`);
     }
     if (Object.hasOwn(override, "disabled")) {
-        if (override["disabled"] !== true) {
-            fail(`models.definitions.${name}.disabled must be exactly \`true\` when set (got ${JSON.stringify(override["disabled"])})`);
-        }
-        const conflicting = ["cmd", "color", "usage", "sandbox", "preLaunch", "preLaunchEnv"].filter((key) => Object.hasOwn(override, key));
-        if (conflicting.length > 0) {
-            fail(`models.definitions.${name}: cannot combine \`disabled: true\` with other fields (${conflicting.join(", ")}). Either disable the model or override its fields, not both.`);
-        }
+        fail(MODEL_DEFINITIONS_MIGRATION_MESSAGE);
     }
 }
 /**
- * True when `name` is a shipped default the user removed via `disabled: true`.
- * Derived from absence in `definitions` — that's the only path that removes a
- * shipped default, codified in `failIfLegacyModelKeys` + `mergeDefinitions`.
- * Consumers needing to distinguish disabled-by-user from unknown-label use this.
+ * True when `name` is a built-in preset but not present in the enabled
+ * definitions. Consumers use this to distinguish `agent-codex` when codex is
+ * not enabled from an arbitrary unknown label like `agent-typo`.
  */
-export function isShippedDefaultDisabled(config, name) {
-    return (Object.hasOwn(DEFAULT_MODEL_DEFINITIONS, name) &&
+export function isBuiltInModelNotEnabled(config, name) {
+    return (Object.hasOwn(BUILT_IN_MODEL_DEFINITIONS, name) &&
         !Object.hasOwn(config.models.definitions, name));
 }
 function isUsageDisableSentinel(usage) {
@@ -274,27 +291,17 @@ function buildOverrideCandidate(name, override, existing) {
     return candidate;
 }
 function mergeDefinitions(user) {
-    if (user !== undefined && !isPlainObject(user)) {
+    if (user === undefined) {
+        fail(MODEL_DEFINITIONS_MIGRATION_MESSAGE);
+    }
+    if (!isPlainObject(user)) {
         fail("models.definitions must be an object");
     }
-    const merged = Object.fromEntries(Object.entries(DEFAULT_MODEL_DEFINITIONS).map(([name, definition]) => [
-        name,
-        cloneModelDefinition(definition),
-    ]));
-    for (const [name, override] of Object.entries(user ?? {})) {
+    const merged = {};
+    for (const [name, override] of Object.entries(user)) {
         failIfLegacyModelKeys(name, override);
-        if (override.disabled === true) {
-            if (!Object.hasOwn(DEFAULT_MODEL_DEFINITIONS, name)) {
-                fail(`models.definitions.${name}: \`disabled: true\` is only valid for shipped defaults (${Object.keys(DEFAULT_MODEL_DEFINITIONS).join(", ")}). Remove the entry instead.`);
-            }
-            // Drop the key so downstream iterators (doctor, eligibility, usage) ignore
-            // the model automatically; `isShippedDefaultDisabled` lets the few consumers
-            // that need to distinguish disabled from unknown re-derive the set.
-            // oxlint-disable-next-line typescript/no-dynamic-delete -- `merged` is a fresh function-local clone of DEFAULT_MODEL_DEFINITIONS; no V8 dictionary-mode/pollution concerns
-            delete merged[name];
-            continue;
-        }
-        const candidate = buildOverrideCandidate(name, override, merged[name]);
+        const builtIn = BUILT_IN_MODEL_DEFINITIONS[name];
+        const candidate = buildOverrideCandidate(name, override, builtIn);
         const { cmd, color, usage, sandbox, preLaunch, preLaunchEnv } = candidate;
         if (typeof cmd !== "string" || cmd.length === 0) {
             fail(`models.definitions.${name}.cmd must be a non-empty string`);
@@ -322,9 +329,9 @@ function mergeDefinitions(user) {
 function isPlainObject(value) {
     return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-function requireObject(value, path) {
+function requireObject(value, configKey) {
     if (!isPlainObject(value)) {
-        fail(`${path} must be an object (got ${JSON.stringify(value)})`);
+        fail(`${configKey} must be an object (got ${JSON.stringify(value)})`);
     }
 }
 function failOnLegacyLinearShape(user) {
@@ -362,12 +369,12 @@ function normalizeSources(raw) {
     }
     const names = new Map();
     for (const [index, entry] of raw.entries()) {
-        const path = `sources[${index}]`;
+        const configPath = `sources[${index}]`;
         if (!isPlainObject(entry)) {
-            fail(`${path} must be an object`);
+            fail(`${configPath} must be an object`);
         }
         const { kind, name } = entry;
-        requireString(kind, `${path}.kind`);
+        requireString(kind, `${configPath}.kind`);
         // Per-adapter Zod validation runs in `buildSources`. Here we check name
         // uniqueness — the Board composer relies on it for writeback routing.
         // When `name` is omitted, the adapter's runtime default is `kind` (the
@@ -375,14 +382,14 @@ function normalizeSources(raw) {
         // dedup on the effective runtime name to catch e.g. two `{kind: "linear"}`
         // entries that would both produce a source named `"linear"`.
         if (name !== undefined) {
-            requireString(name, `${path}.name`);
+            requireString(name, `${configPath}.name`);
         }
         /* v8 ignore next @preserve -- both `name`-set and `name`-unset paths are covered by separate dedup tests; coverage for the fallback's `kind` arm only fires when both entries in the dedup set come from `name`, which the second test already covers */
         const effectiveName = name ?? kind;
         const previous = names.get(effectiveName);
         if (previous !== undefined) {
             /* v8 ignore next 3 @preserve -- the `name === undefined` ternary arm requires two unnamed entries colliding; we keep the conditional for the better error message but only one path is exercised in tests */
-            fail(`${path} would produce a source named "${effectiveName}" (from ${name === undefined ? "default `kind` since `name` is omitted" : "`name`"}), duplicating sources[${previous}]. Configure distinct \`name\` fields.`);
+            fail(`${configPath} would produce a source named "${effectiveName}" (from ${name === undefined ? "default `kind` since `name` is omitted" : "`name`"}), duplicating sources[${previous}]. Configure distinct \`name\` fields.`);
         }
         names.set(effectiveName, index);
     }
@@ -456,7 +463,6 @@ function validate(config) {
     requirePositiveInt(config.orchestrator.pollIntervalMilliseconds, "orchestrator.pollIntervalMilliseconds");
     requirePercent(config.orchestrator.sessionLimitPercentage, "orchestrator.sessionLimitPercentage");
     const { definitions } = config.models;
-    /* v8 ignore next 3 @preserve -- mergeDefinitions seeds claude+codex defaults, so an empty map is unreachable */
     if (Object.keys(definitions).length === 0) {
         fail("models.definitions must contain at least one model");
     }
@@ -468,12 +474,12 @@ function validate(config) {
         requireString(definition.color, `models.definitions.${name}.color`);
         if (definition.usage !== undefined) {
             const usagePath = `models.definitions.${name}.usage`;
-            /* v8 ignore next 3 @preserve -- mergeDefinitions only assigns usage from validated overrides or shipped defaults; reaching this guard requires hand-mutating the resolved config */
+            /* v8 ignore next 3 @preserve -- mergeDefinitions only assigns usage from validated overrides or built-in presets; reaching this guard requires hand-mutating the resolved config */
             if (typeof definition.usage !== "object" || definition.usage === null) {
                 fail(`${usagePath} must be an object`);
             }
             const { codexbar } = definition.usage;
-            /* v8 ignore next 3 @preserve -- mergeDefinitions only assigns usage from validated overrides or shipped defaults; reaching this guard requires hand-mutating the resolved config */
+            /* v8 ignore next 3 @preserve -- mergeDefinitions only assigns usage from validated overrides or built-in presets; reaching this guard requires hand-mutating the resolved config */
             if (typeof codexbar !== "object" || codexbar === null) {
                 fail(`${usagePath}.codexbar must be an object`);
             }
@@ -496,11 +502,11 @@ function validate(config) {
     if (!LOCAL_RUNNER_SETTINGS.includes(config.local.runner)) {
         fail(`local.runner must be one of ${LOCAL_RUNNER_SETTINGS.join(", ")} (got ${JSON.stringify(config.local.runner)})`);
     }
-    // Disabled-default check must run before the generic "not a key" check so
-    // the user gets the specific "is disabled" message instead of a stale-list
-    // message they can't act on without realizing they need to re-enable.
-    if (isShippedDefaultDisabled(config, config.models.default)) {
-        fail(`models.default ("${config.models.default}") is disabled. Either re-enable it or set models.default to an enabled model.`);
+    // Built-in-not-enabled check must run before the generic "not a key" check
+    // so the user gets the specific migration-oriented message for `codex`
+    // instead of a stale-list message.
+    if (isBuiltInModelNotEnabled(config, config.models.default)) {
+        fail(`models.default ("${config.models.default}") is not enabled. Add \`models.definitions.${config.models.default}: {}\` or set models.default to an enabled model.`);
     }
     if (!(config.models.default in definitions)) {
         fail(`models.default ("${config.models.default}") is not a key in models.definitions (have: ${Object.keys(definitions).join(", ")})`);
@@ -567,12 +573,12 @@ async function loadAt(filepath) {
     return result;
 }
 function findXdgConfigFile() {
-    return XDG_FALLBACK_NAMES.map((name) => xdgConfigPath("groundcrew", name)).find((path) => existsSync(path));
+    return XDG_FALLBACK_NAMES.map((name) => xdgConfigPath("groundcrew", name)).find((p) => existsSync(p));
 }
 async function discoverUserConfig() {
     const override = readEnvironmentVariable("GROUNDCREW_CONFIG");
     if (override !== undefined && override.length > 0) {
-        const overridePath = resolve(override);
+        const overridePath = path.resolve(override);
         if (!existsSync(overridePath)) {
             fail(`GROUNDCREW_CONFIG=${overridePath} not found`);
         }

package/dist/lib/launchCommand.js

@@ -1,5 +1,5 @@
 import { createRequire } from "node:module";
-import { basename, dirname, resolve } from "node:path";
+import path from "node:path";
 import { BUILD_SECRET_NAMES, hasPreLaunchEnv, } from "./config.js";
 import { shellSingleQuote } from "./shell.js";
 export { shellSingleQuote } from "./shell.js";
@@ -22,7 +22,7 @@ export function resolveSafehouseClearancePath(baseUrl = import.meta.url) {
         throw new Error("@clipboard-health/clearance is required by @clipboard-health/groundcrew but could not be resolved. " +
             "Install it alongside groundcrew (for example: `npm install -g @clipboard-health/clearance`).", { cause: error });
     }
-    return resolve(dirname(clearancePackageJson), "safehouse", "safehouse-clearance");
+    return path.resolve(path.dirname(clearancePackageJson), "safehouse", "safehouse-clearance");
 }
 const SAFEHOUSE_CLEARANCE_WRAPPER_PATH = resolveSafehouseClearancePath();
 /**
@@ -155,7 +155,7 @@ function safehouseProfileCommandName(agentCmd) {
     if (commandToken === undefined) {
         throw new Error(`Cannot infer Safehouse agent profile command from model cmd ${JSON.stringify(agentCmd)}.`);
     }
-    const commandName = basename(commandToken);
+    const commandName = path.basename(commandToken);
     if (commandName === "." ||
         commandName === ".." ||
         commandName.startsWith("-") ||
@@ -211,7 +211,7 @@ function shouldWrapWithSafehouse(arguments_) {
  * host shell because there is no groundcrew-managed sandbox to run them inside.
  */
 function buildUnwrappedHostLaunchCommand(arguments_) {
-    const promptDir = dirname(arguments_.promptFile);
+    const promptDir = path.dirname(arguments_.promptFile);
     const agentCmd = renderAgentCommand({
         agentCmd: arguments_.definition.cmd,
         worktreeDir: arguments_.worktreeDir,
@@ -243,7 +243,7 @@ function buildUnwrappedHostLaunchCommand(arguments_) {
  *   2. **Agent wrap**: `safehouse-clearance "$shim" -c '<exec agent>' sh "$_p"`
  *      where `$shim` is a `mktemp`-d symlink to `/bin/sh` named after the
  *      agent (e.g. `claude`). Safehouse selects the matching agent profile
- *      from the wrapped command's basename (`claude-code.sb` etc.) without
+ *      from the wrapped command's path.basename (`claude-code.sb` etc.) without
  *      needing every agent profile enabled globally.
  *
  * Host ordering matters: when a `preLaunch` hook is present, inherited
@@ -263,7 +263,7 @@ function buildUnwrappedHostLaunchCommand(arguments_) {
  *   reach the profile-neutral setup phase.
  */
 function buildSafehouseLaunchCommand(arguments_) {
-    const promptDir = dirname(arguments_.promptFile);
+    const promptDir = path.dirname(arguments_.promptFile);
     const safehouseCommandName = safehouseProfileCommandName(arguments_.definition.cmd);
     const agentCmd = renderAgentCommand({
         agentCmd: arguments_.definition.cmd,
@@ -315,7 +315,7 @@ function buildSdxLaunchCommand(arguments_) {
     if (arguments_.sandboxName === undefined || arguments_.definition.sandbox === undefined) {
         throw new Error("buildLaunchCommand: runner='sdx' requires sandboxName and a model `sandbox` config block (set sandbox.agent on the model in config.ts).");
     }
-    const promptDir = dirname(arguments_.promptFile);
+    const promptDir = path.dirname(arguments_.promptFile);
     const agentCmd = renderAgentCommand({
         agentCmd: arguments_.definition.cmd,
         worktreeDir: arguments_.worktreeDir,

package/dist/lib/npmGlobal.d.ts

@@ -25,6 +25,6 @@ export declare function runNpmInstallGlobal(options: RunNpmInstallOptions): Prom
 export declare function detectInstallPath(cliMetaUrl: string): string;
 export type NpmRootRunner = (command: string, args: readonly string[]) => string;
 export declare function detectNpmRootGlobal(npmBin: string, runner: NpmRootRunner): string | undefined;
-export declare function detectIsSymlink(path: string): boolean;
+export declare function detectIsSymlink(filePath: string): boolean;
 export declare function createDefaultNpmSpawner(): NpmSpawner;
 //# sourceMappingURL=npmGlobal.d.ts.map

package/dist/lib/npmGlobal.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"npmGlobal.d.ts","sourceRoot":"","sources":["../../src/lib/npmGlobal.ts"],"names":[],"mappings":"AAKA,MAAM,MAAM,WAAW,GAAG,QAAQ,GAAG,QAAQ,GAAG,KAAK,GAAG,SAAS,GAAG,SAAS,CAAC;AAE9E,MAAM,WAAW,sBAAsB;IACrC,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,GAAG,SAAS,CAAC;IAClC,SAAS,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC;CACtC;AAED,wBAAgB,eAAe,CAAC,OAAO,EAAE,sBAAsB,GAAG,WAAW,CAY5E;AAED,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,MAAM,UAAU,GAAG,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,MAAM,EAAE,KAAK,OAAO,CAAC,gBAAgB,CAAC,CAAC;AAEjG,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,OAAO,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,oBAAoB;IACnC,WAAW,EAAE,MAAM,CAAC;IACpB,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,UAAU,CAAC;CACrB;AAED,wBAAsB,mBAAmB,CAAC,OAAO,EAAE,oBAAoB,GAAG,OAAO,CAAC,YAAY,CAAC,CAQ9F;AAED,wBAAgB,iBAAiB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAE5D;AAED,MAAM,MAAM,aAAa,GAAG,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,MAAM,EAAE,KAAK,MAAM,CAAC;AAEjF,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,aAAa,GAAG,MAAM,GAAG,SAAS,CAM7F;AAED,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAMrD;AAED,wBAAgB,uBAAuB,IAAI,UAAU,CAmBpD"}
+{"version":3,"file":"npmGlobal.d.ts","sourceRoot":"","sources":["../../src/lib/npmGlobal.ts"],"names":[],"mappings":"AAKA,MAAM,MAAM,WAAW,GAAG,QAAQ,GAAG,QAAQ,GAAG,KAAK,GAAG,SAAS,GAAG,SAAS,CAAC;AAE9E,MAAM,WAAW,sBAAsB;IACrC,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,GAAG,SAAS,CAAC;IAClC,SAAS,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC;CACtC;AAED,wBAAgB,eAAe,CAAC,OAAO,EAAE,sBAAsB,GAAG,WAAW,CAY5E;AAED,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,MAAM,UAAU,GAAG,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,MAAM,EAAE,KAAK,OAAO,CAAC,gBAAgB,CAAC,CAAC;AAEjG,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,OAAO,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,oBAAoB;IACnC,WAAW,EAAE,MAAM,CAAC;IACpB,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,UAAU,CAAC;CACrB;AAED,wBAAsB,mBAAmB,CAAC,OAAO,EAAE,oBAAoB,GAAG,OAAO,CAAC,YAAY,CAAC,CAQ9F;AAED,wBAAgB,iBAAiB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAE5D;AAED,MAAM,MAAM,aAAa,GAAG,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,MAAM,EAAE,KAAK,MAAM,CAAC;AAEjF,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,aAAa,GAAG,MAAM,GAAG,SAAS,CAM7F;AAED,wBAAgB,eAAe,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAMzD;AAED,wBAAgB,uBAAuB,IAAI,UAAU,CAmBpD"}

package/dist/lib/npmGlobal.js

@@ -1,16 +1,16 @@
 import { spawn } from "node:child_process";
 import { lstatSync } from "node:fs";
-import { dirname, sep } from "node:path";
+import path from "node:path";
 import { fileURLToPath } from "node:url";
 export function classifyInstall(options) {
     const { installPath, npmRootGlobal, isSymlink } = options;
-    if (npmRootGlobal !== undefined && installPath.startsWith(`${npmRootGlobal}${sep}`)) {
+    if (npmRootGlobal !== undefined && installPath.startsWith(`${npmRootGlobal}${path.sep}`)) {
         return isSymlink(installPath) ? "linked" : "global";
     }
-    if (installPath.includes(`${sep}_npx${sep}`)) {
+    if (installPath.includes(`${path.sep}_npx${path.sep}`)) {
         return "npx";
     }
-    if (installPath.includes(`${sep}node_modules${sep}`)) {
+    if (installPath.includes(`${path.sep}node_modules${path.sep}`)) {
         return "project";
     }
     return "unknown";
@@ -25,7 +25,7 @@ export async function runNpmInstallGlobal(options) {
     };
 }
 export function detectInstallPath(cliMetaUrl) {
-    return dirname(dirname(fileURLToPath(cliMetaUrl)));
+    return path.dirname(path.dirname(fileURLToPath(cliMetaUrl)));
 }
 export function detectNpmRootGlobal(npmBin, runner) {
     try {
@@ -35,9 +35,9 @@ export function detectNpmRootGlobal(npmBin, runner) {
         return undefined;
     }
 }
-export function detectIsSymlink(path) {
+export function detectIsSymlink(filePath) {
     try {
-        return lstatSync(path).isSymbolicLink();
+        return lstatSync(filePath).isSymbolicLink();
     }
     catch {
         return false;

package/dist/lib/runState.js

@@ -1,5 +1,5 @@
 import { mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
+import path from "node:path";
 const TICKET_RE = /^[a-z][\da-z]*-\d+$/;
 const RUN_STATE_DIRECTORY_NAME = "runs";
 function ticketKey(ticket) {
@@ -10,10 +10,10 @@ function ticketKey(ticket) {
     return normalized;
 }
 export function runStateDirectory(config) {
-    return resolve(dirname(config.logging.file), RUN_STATE_DIRECTORY_NAME);
+    return path.resolve(path.dirname(config.logging.file), RUN_STATE_DIRECTORY_NAME);
 }
 export function runStatePath(config, ticket) {
-    return resolve(runStateDirectory(config), `${ticketKey(ticket)}.json`);
+    return path.resolve(runStateDirectory(config), `${ticketKey(ticket)}.json`);
 }
 function nowIso() {
     return new Date().toISOString();
@@ -80,11 +80,11 @@ function parseRunState(value) {
     };
 }
 function writeState(config, state) {
-    const path = runStatePath(config, state.ticket);
-    mkdirSync(dirname(path), { recursive: true });
-    const tmpPath = `${path}.${process.pid}.tmp`;
+    const statePath = runStatePath(config, state.ticket);
+    mkdirSync(path.dirname(statePath), { recursive: true });
+    const tmpPath = `${statePath}.${process.pid}.tmp`;
     writeFileSync(tmpPath, `${JSON.stringify(state, undefined, 2)}\n`, { mode: 0o600 });
-    renameSync(tmpPath, path);
+    renameSync(tmpPath, statePath);
 }
 export function readRunState(config, ticket) {
     let raw;

package/dist/lib/stagedLaunch.js

@@ -1,6 +1,6 @@
 import { mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
-import { join } from "node:path";
+import path from "node:path";
 import { BUILD_SECRET_NAMES } from "./config.js";
 import { shellSingleQuote } from "./launchCommand.js";
 import { readEnvironmentVariable } from "./util.js";
@@ -13,8 +13,8 @@ function renderPromptTemplate(template, variables) {
         .replaceAll("{{workspaceContinuationInstruction}}", variables.workspaceContinuationInstruction);
 }
 export function stagePromptText(input) {
-    const promptDir = mkdtempSync(join(tmpdir(), `${input.prefix}-${input.ticket}-`));
-    const promptFile = join(promptDir, "prompt.txt");
+    const promptDir = mkdtempSync(path.join(tmpdir(), `${input.prefix}-${input.ticket}-`));
+    const promptFile = path.join(promptDir, "prompt.txt");
     writeFileSync(promptFile, input.text);
     return { directory: promptDir, file: promptFile };
 }
@@ -42,12 +42,12 @@ export function stageBuildSecrets(promptDir) {
     if (lines.length === 0) {
         return undefined;
     }
-    const secretsFile = join(promptDir, "secrets.env");
+    const secretsFile = path.join(promptDir, "secrets.env");
     writeFileSync(secretsFile, `${lines.join("\n")}\n`, { mode: 0o600 });
     return secretsFile;
 }
 function stageLaunchScript(promptDir, command) {
-    const launcherFile = join(promptDir, "launch.sh");
+    const launcherFile = path.join(promptDir, "launch.sh");
     writeFileSync(launcherFile, `#!/usr/bin/env bash\n${command}\n`, { mode: 0o700 });
     return launcherFile;
 }

package/dist/lib/util.d.ts

@@ -7,7 +7,7 @@ export declare function okMark(): string;
 export declare function failMark(): string;
 export declare function styleWarning(text: string): string;
 export declare function styleDim(text: string): string;
-export declare function setLogFile(path: string | undefined): void;
+export declare function setLogFile(filePath: string | undefined): void;
 export declare function withLogOutputSuppressed<T>(operation: () => Promise<T>): Promise<T>;
 /** Important tier: always on the console (dimmed timestamp) and the log file. */
 export declare function log(message: string): void;

package/dist/lib/util.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"util.d.ts","sourceRoot":"","sources":["../../src/lib/util.ts"],"names":[],"mappings":"AAIA,wBAAsB,KAAK,CAAC,EAAE,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC,CAoB3E;AAED,wBAAgB,WAAW,CAAC,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAIlD;AAED,wBAAgB,UAAU,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAGhD;AAQD,wBAAgB,UAAU,CAAC,KAAK,EAAE,OAAO,GAAG,IAAI,CAE/C;AAED,wBAAgB,SAAS,IAAI,OAAO,CAEnC;AAWD,wBAAgB,MAAM,IAAI,MAAM,CAE/B;AAED,wBAAgB,QAAQ,IAAI,MAAM,CAEjC;AAED,wBAAgB,YAAY,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAEjD;AAED,wBAAgB,QAAQ,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAE7C;AAQD,wBAAgB,UAAU,CAAC,IAAI,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI,CAEzD;AAED,wBAAsB,uBAAuB,CAAC,CAAC,EAAE,SAAS,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC,CAOxF;AAuBD,iFAAiF;AACjF,wBAAgB,GAAG,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAOzC;AAED;;;;GAIG;AACH,wBAAgB,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAS3C;AAED,KAAK,kBAAkB,GAAG,OAAO,GAAG,MAAM,GAAG,MAAM,GAAG,SAAS,MAAM,EAAE,GAAG,SAAS,CAAC;AAUpF,wBAAgB,QAAQ,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,kBAAkB,CAAC,GAAG,IAAI,CAiBxF;AAED,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAGxE;AAED;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAMzF;AAED,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,OAAO,CAAC;IAChB,WAAW,EAAE,MAAM,EAAE,CAAC;CACvB;AAED;;;;;GAKG;AACH,wBAAgB,sBAAsB,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,KAAK,EAAE,MAAM,GAAG,iBAAiB,CAcvF;AAED,wBAAgB,YAAY,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,CAcnD"}
+{"version":3,"file":"util.d.ts","sourceRoot":"","sources":["../../src/lib/util.ts"],"names":[],"mappings":"AAIA,wBAAsB,KAAK,CAAC,EAAE,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC,CAoB3E;AAED,wBAAgB,WAAW,CAAC,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAIlD;AAED,wBAAgB,UAAU,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAGhD;AAQD,wBAAgB,UAAU,CAAC,KAAK,EAAE,OAAO,GAAG,IAAI,CAE/C;AAED,wBAAgB,SAAS,IAAI,OAAO,CAEnC;AAWD,wBAAgB,MAAM,IAAI,MAAM,CAE/B;AAED,wBAAgB,QAAQ,IAAI,MAAM,CAEjC;AAED,wBAAgB,YAAY,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAEjD;AAED,wBAAgB,QAAQ,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAE7C;AAQD,wBAAgB,UAAU,CAAC,QAAQ,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI,CAE7D;AAED,wBAAsB,uBAAuB,CAAC,CAAC,EAAE,SAAS,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC,CAOxF;AAuBD,iFAAiF;AACjF,wBAAgB,GAAG,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAOzC;AAED;;;;GAIG;AACH,wBAAgB,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAS3C;AAED,KAAK,kBAAkB,GAAG,OAAO,GAAG,MAAM,GAAG,MAAM,GAAG,SAAS,MAAM,EAAE,GAAG,SAAS,CAAC;AAUpF,wBAAgB,QAAQ,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,kBAAkB,CAAC,GAAG,IAAI,CAiBxF;AAED,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAGxE;AAED;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAMzF;AAED,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,OAAO,CAAC;IAChB,WAAW,EAAE,MAAM,EAAE,CAAC;CACvB;AAED;;;;;GAKG;AACH,wBAAgB,sBAAsB,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,KAAK,EAAE,MAAM,GAAG,iBAAiB,CAcvF;AAED,wBAAgB,YAAY,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,CAcnD"}

package/dist/lib/util.js

@@ -1,5 +1,5 @@
 import { appendFileSync, mkdirSync } from "node:fs";
-import { dirname } from "node:path";
+import path from "node:path";
 import { styleText } from "node:util";
 export async function sleep(ms, signal) {
     if (signal?.aborted === true) {
@@ -62,8 +62,8 @@ export function styleDim(text) {
 // loadConfig() resolves `logging.file`.
 let logFilePath;
 let suppressedLogDepth = 0;
-export function setLogFile(path) {
-    logFilePath = path;
+export function setLogFile(filePath) {
+    logFilePath = filePath;
 }
 export async function withLogOutputSuppressed(operation) {
     suppressedLogDepth += 1;
@@ -79,7 +79,7 @@ function appendLogLine(line) {
         return;
     }
     try {
-        mkdirSync(dirname(logFilePath), { recursive: true });
+        mkdirSync(path.dirname(logFilePath), { recursive: true });
         appendFileSync(logFilePath, `${line}\n`);
     }
     catch {