@forwardimpact/outpost 3.1.4 → 3.2.1

package/bin/fit-outpost.js

@@ -3,24 +3,15 @@
 // bag, threaded into src/outpost.js's dispatch via run(runtime, version).
 import "@forwardimpact/libpreflight/node22";
 
-import { readFileSync } from "node:fs";
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
-
+import { resolveVersion } from "@forwardimpact/libcli";
 import { createDefaultRuntime } from "@forwardimpact/libutil/runtime";
 
 import { run } from "../src/outpost.js";
 
-const __dirname = dirname(fileURLToPath(import.meta.url));
-
-// In compiled binaries (bun build --compile), `bun build --define` injects the
-// version string here so the readFileSync branch is eliminated as dead code.
-// Source execution falls through to package.json.
-const VERSION =
-  process.env.OUTPOST_VERSION ||
-  JSON.parse(readFileSync(join(__dirname, "..", "package.json"), "utf8"))
-    .version;
-
 const runtime = createDefaultRuntime();
-const code = await run(runtime, VERSION);
+const version = resolveVersion({
+  packageJsonUrl: new URL("../package.json", import.meta.url),
+  runtime,
+});
+const code = await run(runtime, version);
 if (code) runtime.proc.exit(code);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forwardimpact/outpost",
-  "version": "3.1.4",
+  "version": "3.2.1",
   "description": "Personal operations center — context from email, calendar, and knowledge assembled so preparation is continuous, not a morning scramble.",
   "homepage": "https://www.forwardimpact.team",
   "repository": {
@@ -50,7 +50,6 @@
   },
   "dependencies": {
     "@forwardimpact/libcli": "^0.1.0",
-    "@forwardimpact/libmacos": "^0.1.0",
     "@forwardimpact/libpreflight": "^0.1.0",
     "@forwardimpact/libtelemetry": "^0.1.33",
     "@forwardimpact/libutil": "^0.1.0"
@@ -58,10 +57,16 @@
   "devDependencies": {
     "@forwardimpact/libmock": "^0.1.0"
   },
+  "optionalDependencies": {
+    "@forwardimpact/libmacos": "^0.1.0"
+  },
   "engines": {
     "bun": ">=1.2.0",
     "node": ">=22.0.0"
   },
+  "os": [
+    "darwin"
+  ],
   "publishConfig": {
     "access": "public"
   }

package/src/agent-runner.js

@@ -16,6 +16,7 @@ export class AgentRunner {
   #fs;
   #proc;
   #clock;
+  #runtime;
 
   /**
    * @param {Object | (() => Object | Promise<{default?: Object}>)} spawn -
@@ -43,6 +44,7 @@ export class AgentRunner {
     this.#fs = runtime.fs;
     this.#proc = runtime.proc;
     this.#clock = runtime.clock;
+    this.#runtime = runtime;
     this.#activeChildren = new Set();
   }
 
@@ -181,14 +183,19 @@ export class AgentRunner {
         spawnArgs,
         env,
         kbPath,
+        this.#runtime,
       );
       this.#activeChildren.add(pid);
 
-      const exitCode = await spawnMod.waitForExit(pid);
+      const exitCode = await spawnMod.waitForExit(
+        pid,
+        undefined,
+        this.#runtime,
+      );
       this.#activeChildren.delete(pid);
 
-      const stdout = spawnMod.readOutput(stdoutFile);
-      const stderr = spawnMod.readOutput(stderrFile);
+      const stdout = spawnMod.readOutput(stdoutFile, this.#runtime);
+      const stderr = spawnMod.readOutput(stderrFile, this.#runtime);
 
       if (exitCode === 0) {
         this.#log(

package/src/kb-manager.js

@@ -6,11 +6,10 @@ import { join, dirname, resolve } from "node:path";
 import { homedir } from "node:os";
 import { createLogger } from "@forwardimpact/libtelemetry";
 
-const logger = createLogger("outpost");
-
 /** Manage knowledge base lifecycle including initialization, updates, and settings merging. */
 export class KBManager {
   #fs;
+  #logger;
 
   /**
    * @param {import("@forwardimpact/libutil/runtime").Runtime} runtime
@@ -21,6 +20,7 @@ export class KBManager {
     if (!runtime?.fs) throw new Error("runtime.fs is required");
     if (!logFn) throw new Error("logFn is required");
     this.#fs = runtime.fs;
+    this.#logger = createLogger("outpost", runtime);
   }
 
   /**
@@ -74,12 +74,12 @@ export class KBManager {
    */
   async copyBundledFiles(tpl, dest) {
     await this.#fs.copyFile(join(tpl, "CLAUDE.md"), join(dest, "CLAUDE.md"));
-    logger.info(`  Updated CLAUDE.md`);
+    this.#logger.info(`  Updated CLAUDE.md`);
 
     const apmSrc = join(tpl, "apm.yml");
     if (await this.#exists(apmSrc)) {
       await this.#fs.copyFile(apmSrc, join(dest, "apm.yml"));
-      logger.info(`  Updated apm.yml`);
+      this.#logger.info(`  Updated apm.yml`);
     }
 
     await this.mergeSettings(tpl, dest);
@@ -96,7 +96,9 @@ export class KBManager {
       const names = entries.map((d) =>
         sub === "agents" ? d.name.replace(".md", "") : d.name,
       );
-      logger.info(`  Updated ${names.length} ${sub}: ${names.join(", ")}`);
+      this.#logger.info(
+        `  Updated ${names.length} ${sub}: ${names.join(", ")}`,
+      );
     }
   }
 
@@ -141,7 +143,7 @@ export class KBManager {
     if (!(await this.#exists(destPath))) {
       await this.#ensureDir(join(dest, ".claude"));
       await this.#fs.copyFile(src, destPath);
-      logger.info(`  Created settings.json`);
+      this.#logger.info(`  Created settings.json`);
       return;
     }
 
@@ -154,9 +156,9 @@ export class KBManager {
 
     if (added > 0) {
       await this.#writeJSON(destPath, existing);
-      logger.info(`  Updated settings.json (${added} new entries)`);
+      this.#logger.info(`  Updated settings.json (${added} new entries)`);
     } else {
-      logger.info(`  Settings up to date`);
+      this.#logger.info(`  Settings up to date`);
     }
   }
 
@@ -193,7 +195,7 @@ export class KBManager {
 
     await this.copyBundledFiles(templateDir, dest);
 
-    logger.info(
+    this.#logger.info(
       `Knowledge base initialized at ${dest}\n\nNext steps:\n  1. Edit ${dest}/USER.md with your name, email, and domain\n  2. cd ${dest} && npx apm install\n  3. claude`,
     );
     return { ok: true, value: { dest } };
@@ -215,7 +217,7 @@ export class KBManager {
       };
     }
     await this.copyBundledFiles(templateDir, dest);
-    logger.info(`\nKnowledge base updated: ${dest}`);
+    this.#logger.info(`\nKnowledge base updated: ${dest}`);
     return { ok: true, value: { dest } };
   }
 

package/src/outpost.js

@@ -22,8 +22,6 @@ import { createCli } from "@forwardimpact/libcli";
 import { createLogger } from "@forwardimpact/libtelemetry";
 import { isoTimestamp } from "@forwardimpact/libutil";
 
-const logger = createLogger("outpost");
-
 import { StateManager } from "./state-manager.js";
 import { AgentRunner } from "./agent-runner.js";
 import { Scheduler, formatLocalTime } from "./scheduler.js";
@@ -131,6 +129,7 @@ function renderAgentStatus(name, agent, s, kbMissing) {
  */
 export async function run(runtime, version) {
   const { fs, proc, clock } = runtime;
+  const logger = createLogger("outpost", runtime);
 
   /**
    * Async existence check via the one fs surface this module uses.
@@ -401,7 +400,7 @@ export async function run(runtime, version) {
   }
 
   // --- CLI entry point -------------------------------------------------------
-  const cli = createCli(buildDefinition(version));
+  const cli = createCli(buildDefinition(version), { runtime });
   const parsed = cli.parse(proc.argv.slice(2));
   if (!parsed) return 0;
 

package/src/scheduler.js

@@ -7,7 +7,7 @@
  * construct further `Date` objects only for deterministic date *arithmetic*
  * over caller-supplied or parsed-from-state inputs (never an ambient no-arg
  * `new Date()`); the AST checker cannot distinguish the two, so this file is
- * allow-listed in check-ambient-deps.allow.yml with that reason.
+ * allow-listed in .coaligned/invariants/ambient-deps.allow.yml with that reason.
  */
 
 import { isoTimestamp } from "@forwardimpact/libutil";

package/src/socket-server.js

@@ -5,8 +5,6 @@
 import { createServer } from "node:net";
 import { createConnection } from "node:net";
 import { createLogger } from "@forwardimpact/libtelemetry";
-
-const logger = createLogger("outpost");
 import { join, resolve } from "node:path";
 import { homedir } from "node:os";
 import { computeNextWakeAt, nowFromClock } from "./scheduler.js";
@@ -326,6 +324,7 @@ export class SocketServer {
 export async function requestShutdown(socketPath, runtime) {
   if (!runtime?.fsSync) throw new Error("runtime.fsSync is required");
   if (!runtime?.clock) throw new Error("runtime.clock is required");
+  const logger = createLogger("outpost", runtime);
   if (!runtime.fsSync.existsSync(socketPath)) {
     logger.info("Daemon not running (no socket).");
     return false;