terminal-pilot 0.0.42 → 0.0.43

package/README.md

@@ -4,7 +4,7 @@
 
 Use it when plain stdio is not enough: menus, prompts, arrow-key navigation, confirmations, and terminal redraws such as `poe-code configure`.
 
-For design rationale and scope, see `docs/plans/terminal-pilot.md`.
+For design history, see [terminal-pilot](../../docs/plans/archive/terminal-pilot.md).
 
 For the MCP server, see [terminal-pilot-mcp](../terminal-pilot-mcp).
 
@@ -218,12 +218,16 @@ class TerminalScreen {
 }
 ```
 
-## Environment variables
+## Environment Variables
 
 There are **no terminal-pilot-specific environment variables**.
 
 Runtime environment is controlled per session via `newSession({ env })`. There are no package-level config files or config options beyond the per-session options.
 
+## Configuration Options
+
+Configure sessions with `newSession({ command, args, cwd, env, cols, rows, observe })`. CLI skill installation is controlled by `install`/`uninstall` flags: target agent plus optional `--local` or `--global`.
+
 ## Testing
 
 Interactive fixtures live under `packages/terminal-pilot/src/testing/`:
@@ -232,7 +236,7 @@ Interactive fixtures live under `packages/terminal-pilot/src/testing/`:
 - `menu-cli.ts` - arrow-key menu fixture
 - `fixtures.test.ts` - examples of driving both fixtures
 
-Run just the fixture tests:
+Run the fixture tests only:
 
 ```sh
 npx vitest run packages/terminal-pilot/src/testing/fixtures.test.ts

package/dist/cli.js

@@ -17,7 +17,7 @@ var globalWithRequire = globalThis;
 globalWithRequire.require ??= createRequire(path.join(process.cwd(), "package.json"));
 
 // ../toolcraft/src/cli.ts
-import { access, lstat, readFile, rename, unlink, writeFile as writeFile2 } from "node:fs/promises";
+import { readFile as readFile2 } from "node:fs/promises";
 import path5 from "node:path";
 import {
   Command as CommanderCommand,
@@ -4582,6 +4582,12 @@ function buildReport(context) {
   return `${lines.join("\n")}
 `;
 }
+function renderErrorReport(context) {
+  return {
+    content: buildReport(context),
+    redactedKeys: commandSecretEnvNames(context.command?.secrets)
+  };
+}
 async function writeErrorReport(context) {
   const env = context.env ?? process.env;
   if (!reportsEnabled(context.errorReports, env) || isSkippedError(context.error)) {
@@ -4595,7 +4601,7 @@ async function writeErrorReport(context) {
   if (reportDirMustStayWithinProject(context.errorReports)) {
     await assertReportDirWithinProject(projectRoot, reportDir);
   }
-  await writeFile(absolutePath, buildReport(context));
+  await writeFile(absolutePath, renderErrorReport(context).content);
   return {
     absolutePath,
     displayPath: relativeDisplayPath(projectRoot, absolutePath)
@@ -5193,8 +5199,8 @@ function throwValidationErrors(errors) {
 ${rendered.join("\n")}`);
 }
 
-// ../toolcraft/src/cli.ts
-configureTheme({ brand: "blue", label: "Toolcraft" });
+// ../toolcraft/src/runtime/io.ts
+import { access, lstat, readFile, rename, unlink, writeFile as writeFile2 } from "node:fs/promises";
 var RESERVED_SERVICE_NAMES = /* @__PURE__ */ new Set([
   "params",
   "secrets",
@@ -5207,6 +5213,47 @@ var RESERVED_SERVICE_NAMES = /* @__PURE__ */ new Set([
   "root"
 ]);
 var RESERVED_SERVICE_NAMES_MESSAGE = "Available reserved names: params, secrets, fetch, fs, env, diagnostics, progress, runtimeOptions, root.";
+function createFs(fs4) {
+  if (fs4 !== void 0) {
+    return fs4;
+  }
+  return {
+    readFile: async (path16, encoding = "utf8") => readFile(path16, { encoding }),
+    writeFile: async (path16, contents, options) => {
+      await writeFile2(path16, contents, options);
+    },
+    exists: async (path16) => {
+      try {
+        await access(path16);
+        return true;
+      } catch {
+        return false;
+      }
+    },
+    lstat: async (path16) => lstat(path16),
+    rename: async (fromPath, toPath) => rename(fromPath, toPath),
+    unlink: async (path16) => unlink(path16)
+  };
+}
+function createEnv(values = process.env) {
+  return {
+    get(key) {
+      return values[key];
+    }
+  };
+}
+function validateServices(services) {
+  for (const name of Object.keys(services)) {
+    if (RESERVED_SERVICE_NAMES.has(name)) {
+      throw new Error(
+        `Service name "${name}" is reserved. Choose a different name. ${RESERVED_SERVICE_NAMES_MESSAGE}`
+      );
+    }
+  }
+}
+
+// ../toolcraft/src/cli.ts
+configureTheme({ brand: "blue", label: "Toolcraft" });
 var NULL_OPTION_VALUE = /* @__PURE__ */ Symbol("toolcraft.cli.null");
 var optionalModulePaths = {
   approvals: "./human-in-loop/approvals-commands.js",
@@ -7015,32 +7062,6 @@ async function withOutputFormat2(output, fn) {
     resetOutputFormatCache();
   }
 }
-function createFs() {
-  return {
-    readFile: async (path16, encoding = "utf8") => readFile(path16, { encoding }),
-    writeFile: async (path16, contents, options) => {
-      await writeFile2(path16, contents, options);
-    },
-    exists: async (path16) => {
-      try {
-        await access(path16);
-        return true;
-      } catch {
-        return false;
-      }
-    },
-    lstat: async (path16) => lstat(path16),
-    rename: async (fromPath, toPath) => rename(fromPath, toPath),
-    unlink: async (path16) => unlink(path16)
-  };
-}
-function createEnv(values = process.env) {
-  return {
-    get(key) {
-      return values[key];
-    }
-  };
-}
 function isPlainObject4(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
@@ -7152,7 +7173,7 @@ function validatePresetFieldValue(value, field, presetPath) {
 async function loadPresetValues(fields, presetPath) {
   let rawPreset;
   try {
-    rawPreset = await readFile(presetPath, {
+    rawPreset = await readFile2(presetPath, {
       encoding: "utf8"
     });
   } catch (error3) {
@@ -7419,7 +7440,7 @@ async function loadFixtureScenario(command, selector) {
   const fixturePath = resolveFixturePath(commandPath);
   let rawFixture;
   try {
-    rawFixture = await readFile(fixturePath, {
+    rawFixture = await readFile2(fixturePath, {
       encoding: "utf8"
     });
   } catch {
@@ -7456,16 +7477,16 @@ function createFixtureEnvValues(command) {
   }
   return values;
 }
-async function resolveFixtureRuntime(command, services, requirementOptions, runtimeFetch) {
+async function resolveFixtureRuntime(command, services, requirementOptions, runtimeFetch, runtimeEnv, runtimeFs) {
   const selector = process.env.TOOLCRAFT_FIXTURE;
   if (selector === void 0 || selector.length === 0) {
     return {
-      env: createEnv(),
+      env: createEnv(runtimeEnv),
       fetch: runtimeFetch,
-      fs: createFs(),
+      fs: createFs(runtimeFs),
       isFixture: false,
       requirementOptions,
-      secrets: resolveCommandSecrets(command),
+      secrets: resolveCommandSecrets(command, runtimeEnv),
       services
     };
   }
@@ -7513,8 +7534,8 @@ function renderApprovalDeclined(error3) {
   logger2.error(error3.message);
   process.exitCode = 1;
 }
-function renderCliErrorPattern(pattern) {
-  const logger2 = createLogger();
+function renderCliErrorPattern(pattern, outputEmitter) {
+  const logger2 = createLogger(outputEmitter);
   if (pattern.kind === "usage") {
     logger2.error(
       appendUsagePointer(pattern.message, {
@@ -7564,15 +7585,6 @@ This is a bug in toolcraft or in the command definition; it cannot be worked aro
   }
   process.exitCode = 1;
 }
-function validateServices(services) {
-  for (const name of Object.keys(services)) {
-    if (RESERVED_SERVICE_NAMES.has(name)) {
-      throw new Error(
-        `Service name "${name}" is reserved. Choose a different name. ${RESERVED_SERVICE_NAMES_MESSAGE}`
-      );
-    }
-  }
-}
 function getNestedValue(target, path16) {
   return path16.reduce(
     (current, segment) => current !== null && typeof current === "object" ? current[segment] : void 0,
@@ -8205,8 +8217,8 @@ function getResolvedFlags(command) {
   const flags = command.optsWithGlobals();
   return flags;
 }
-async function executeCommand(state, services, requirementOptions, runtimeFetch, runtimeOptions, diagnosticsOptions, onErrorReportContext) {
-  const logger2 = createLogger();
+async function executeCommand(state, services, requirementOptions, runtimeFetch, runtimeOptions, runtimeEnv, runtimeFs, outputEmitter, diagnosticsOptions, onErrorReportContext) {
+  const logger2 = createLogger(outputEmitter);
   const primitives = {
     logger: logger2,
     renderTable,
@@ -8225,7 +8237,9 @@ async function executeCommand(state, services, requirementOptions, runtimeFetch,
     state.command,
     services,
     requirementOptions,
-    runtimeFetch
+    runtimeFetch,
+    runtimeEnv,
+    runtimeFs
   );
   const preflightContext = {
     ...runtime.services,
@@ -8293,7 +8307,13 @@ async function executeCommand(state, services, requirementOptions, runtimeFetch,
         renderHumanInLoopPending(result);
         return;
       }
-      const renderStatus = renderResult(state.command, result, output, primitives);
+      const renderStatus = renderResult(
+        state.command,
+        result,
+        output,
+        primitives,
+        outputEmitter === void 0 ? void 0 : (chunk) => outputEmitter(chunk.endsWith("\n") ? chunk.slice(0, -1) : chunk)
+      );
       if (renderStatus.mcpError) {
         process.exitCode = 1;
       }
@@ -8505,7 +8525,7 @@ function renderHttpError(error3, options) {
   }
 }
 async function handleRunError(error3, options) {
-  const logger2 = createLogger();
+  const logger2 = createLogger(options.outputEmitter);
   await withOutputFormat2(options.output, async () => {
     if (error3 instanceof UserError) {
       renderCliErrorPattern(
@@ -8521,16 +8541,20 @@ async function handleRunError(error3, options) {
         } : {
           kind: "runtime-user",
           message: error3.message
-        }
+        },
+        options.outputEmitter
       );
       return;
     }
     if (error3 instanceof Error && error3.name === "ToolcraftBugError") {
-      renderCliErrorPattern({
-        kind: "toolcraft-bug",
-        error: error3,
-        debugStackMode: options.debugStackMode
-      });
+      renderCliErrorPattern(
+        {
+          kind: "toolcraft-bug",
+          error: error3,
+          debugStackMode: options.debugStackMode
+        },
+        options.outputEmitter
+      );
       return;
     }
     if (error3 instanceof CommanderError2) {
@@ -8574,12 +8598,15 @@ async function handleRunError(error3, options) {
       return;
     }
     const message2 = error3 instanceof Error ? error3.message : String(error3);
-    renderCliErrorPattern({
-      kind: "unexpected",
-      message: message2,
-      stack: error3 instanceof Error ? error3.stack : void 0,
-      debugStackMode: options.debugStackMode
-    });
+    renderCliErrorPattern(
+      {
+        kind: "unexpected",
+        message: message2,
+        stack: error3 instanceof Error ? error3.stack : void 0,
+        debugStackMode: options.debugStackMode
+      },
+      options.outputEmitter
+    );
   });
 }
 function formatCommanderErrorMessage(error3) {
@@ -8855,7 +8882,8 @@ async function runCLI(roots, options = {}) {
       root
     };
     const requirementOptions = {
-      apiVersion: options.apiVersion
+      apiVersion: options.apiVersion,
+      env: options.env
     };
     validateServices(services);
     if (hasHelpFlag(argv)) {
@@ -8897,6 +8925,9 @@ async function runCLI(roots, options = {}) {
         requirementOptions,
         runtimeFetch,
         runtimeOptions,
+        options.env,
+        options.fs,
+        options.outputEmitter,
         {
           logLevel: options.logLevel,
           logger: options.logger,
@@ -8972,6 +9003,7 @@ async function runCLI(roots, options = {}) {
       argv,
       rootUsageName,
       commandPath: resolvedCommandPath,
+      outputEmitter: options.outputEmitter,
       userErrorPattern: errorReportContext?.params === void 0 ? userErrorPattern : "runtime-user"
     });
   }
@@ -12251,7 +12283,7 @@ function hasOwnErrorCode4(error3, code) {
 }
 
 // ../agent-skill-config/src/templates.ts
-import { readFile as readFile2, stat } from "node:fs/promises";
+import { readFile as readFile3, stat } from "node:fs/promises";
 import path9 from "node:path";
 import { fileURLToPath as fileURLToPath3 } from "node:url";
 
@@ -14375,7 +14407,7 @@ function sleep2(ms) {
 // src/cli.ts
 configureTheme({ brand: "green", label: "Terminal Pilot" });
 function getBundledPackageVersion() {
-  return true ? "0.0.42" : void 0;
+  return true ? "0.0.43" : void 0;
 }
 function normalizeArgv(argv) {
   if (argv.includes("--json") && !argv.some((argument) => argument === "--output" || argument.startsWith("--output="))) {