@amenophis1er/foreman 0.1.3 → 0.1.5

package/README.md

@@ -1,3 +1,7 @@
+<p align="center">
+  <img src="assets/brand/logo.svg" alt="" width="72" height="72">
+</p>
+
 # Foreman
 
 Foreman runs software missions without you in the loop, and shows you
@@ -60,6 +64,19 @@ which account pays, the Claude Code install, Ollama and Codex if present, the
 browser missions will use, the port, Tailscale, the data directory — and
 exits. Nothing blocks unless it says so.
 
+**Environment.** Everything the CLI reads; `foreman --help` prints the same list.
+
+| Variable | Meaning | Default |
+|---|---|---|
+| `PORT` | listen port | `4177` |
+| `FOREMAN_HOME` | state directory: runs, settings, logs | `~/.foreman` |
+| `FOREMAN_BIND` | `auto` (loopback + Tailscale when present), `local`, or `all` | `auto` |
+| `FOREMAN_BROWSER` | browser for missions: `chrome`, `chromium`, `msedge`, `firefox` | `chrome` |
+| `FOREMAN_CLAUDE_CONFIG_DIR` | the Claude Code install missions run under | inherited |
+| `FOREMAN_CLAUDE_EXECUTABLE` | the Claude Code executable | bundled |
+| `FOREMAN_AUTH_MODE` | assert `api-key` or `subscription`; fail at start on mismatch | unset |
+| `FOREMAN_NO_TELEGRAM` | `1` to start without the Telegram bot: for a second server beside the main one, since Telegram allows one poller per bot | unset |
+
 ## A mission, start to finish
 
 1. **Fleet.** The board: what needs you (answerable right there), what is
@@ -213,6 +230,16 @@ npm run dev                  # API + Vite together
 scripts/dev-restart.sh       # restarts the server only when nothing would be lost
 ```
 
+To run a checkout beside an installed Foreman on the same machine, give it its
+own port and leave the bot to the installed one:
+
+```sh
+PORT=4178 FOREMAN_BIND=local FOREMAN_NO_TELEGRAM=1 npm start
+```
+
+Both read `~/.foreman`; look from the second, drive from the first — two
+servers running missions against one store is the one case nothing guards.
+
 Releases: `npm version <patch|minor|major> && git push --follow-tags` — CI
 tests, publishes to npm with provenance, and creates the GitHub release.
 

package/bin/foreman.mjs

@@ -43,6 +43,7 @@ Environment:
   FOREMAN_CLAUDE_CONFIG_DIR   Claude Code install missions run under
   FOREMAN_CLAUDE_EXECUTABLE   Claude Code executable (default: bundled)
   FOREMAN_AUTH_MODE           Assert 'api-key' or 'subscription'; fail on mismatch
+  FOREMAN_NO_TELEGRAM=1       Do not attach the Telegram bot (a second server beside the main one)
 `;
 
 const [command = 'start', ...rest] = process.argv.slice(2);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@amenophis1er/foreman",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Autonomous mission runner on the Claude Agent SDK: a director plans, delegates to workers, verifies, and reports — from one dashboard, your phone, or the CLI.",
   "keywords": [
     "claude",

package/src/cli.ts

@@ -13,7 +13,7 @@
  *           be up when the laptop lid is closed; this is that.
  */
 import { execFile, spawn } from 'node:child_process';
-import { chmod, mkdir, readFile, rm, writeFile } from 'node:fs/promises';
+import { access, chmod, mkdir, readFile, rm, writeFile } from 'node:fs/promises';
 import { openSync } from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
@@ -170,8 +170,12 @@ async function serviceUninstall(): Promise<number> {
 /** Is the service registered, and is it running? Null pid when registered but idle. */
 async function serviceState(): Promise<{ installed: boolean; running: boolean; pid?: number }> {
   if (process.platform === 'darwin') {
+    // Installed means the plist is on disk; a stopped service (booted out
+    // by `service stop`) is still installed, and `service start` brings it
+    // back. launchctl only knows about loaded jobs, so it answers "running".
+    const installed = await access(path.join(os.homedir(), 'Library', 'LaunchAgents', `${LABEL}.plist`)).then(() => true, () => false);
     const r = await sh('launchctl', ['print', `gui/${os.userInfo().uid}/${LABEL}`]);
-    if (r.code !== 0) return { installed: false, running: false };
+    if (r.code !== 0) return { installed, running: false };
     const pid = Number(/pid = (\d+)/.exec(r.out)?.[1]);
     return { installed: true, running: Number.isFinite(pid) && pid > 0, ...(pid ? { pid } : {}) };
   }
@@ -230,7 +234,11 @@ async function serviceRestart(): Promise<number> {
 async function serviceStatus(): Promise<number> {
   if (process.platform === 'darwin') {
     const r = await sh('launchctl', ['print', `gui/${os.userInfo().uid}/${LABEL}`]);
-    if (r.code !== 0) { console.log('Not installed. `foreman service install` keeps Foreman running.'); return 1; }
+    if (r.code !== 0) {
+      const st = await serviceState();
+      console.log(st.installed ? 'Installed · stopped — `foreman service start` starts it (it also comes back at next login).' : 'Not installed. `foreman service install` keeps Foreman running.');
+      return 1;
+    }
     const state = /state = (\w+)/.exec(r.out)?.[1] ?? 'unknown';
     const pid = /pid = (\d+)/.exec(r.out)?.[1];
     console.log(`Installed · ${state}${pid ? ` · pid ${pid}` : ''} · http://localhost:${PORT}`);

package/src/server.ts

@@ -608,6 +608,11 @@ let telegramBot: TelegramBot | null = null;
 /** (Re)build the Telegram side from the stored token and linked chat. */
 async function reattachTelegram(): Promise<boolean> {
   notifyHub.detach('telegram');
+  // A second server beside the installed one (a dev checkout on another
+  // port) must not start a second poller: Telegram allows one per bot, and
+  // two fight over getUpdates. FOREMAN_NO_TELEGRAM=1 leaves the channel to
+  // whichever server does not set it.
+  if (process.env.FOREMAN_NO_TELEGRAM === '1') return false;
   const s = await notifySettings();
   const token = await getSecret(store.root, 'telegram');
   if (!token) { void telegramBot?.stop(); telegramBot = null; return false; }
@@ -1150,30 +1155,26 @@ async function resumeRun(projectId: string, meta: RunMeta, pick: {
   directorModel?: string; directorProviderId?: string; workerModel?: string; workerProviderId?: string;
 } = {}): Promise<void> {
   const sessionId = meta.directorSessionId;
-  // Resume re-reads Settings, so changing models or tool policy after a
-  // failure takes effect on the retry. A director session cannot switch
-  // model mid-session, so a changed director model restarts the session
-  // fresh (the mission doc carries the state forward).
-  const base = await effectiveSettings(projectId);
-  const settings = {
-    ...base,
-    ...(pick.directorModel ? { directorModel: modelChoice(pick.directorModel), directorProviderId: pick.directorProviderId } : {}),
-    ...(pick.workerModel ? { workerModel: modelChoice(pick.workerModel), workerProviderId: pick.workerProviderId } : {}),
-  };
-  // A model is picked together with the provider that serves it, so a
-  // change of either moves the role. Without the provider following the
-  // model, "resume on Sonnet" after a Codex usage limit went back through
-  // the Codex gateway — which remapped the unknown id to its own default and
-  // hit the same 429. The provider id is left undefined when Settings does
-  // not pin one, which means the project's own provider, as at start.
-  const directorChanged =
-    (settings.directorModel !== undefined && settings.directorModel !== meta.directorModel)
-    || (settings.directorModel !== undefined && settings.directorProviderId !== meta.directorProviderId);
-  const workerChanged =
-    (settings.workerModel !== undefined && settings.workerModel !== meta.workerModel)
-    || (settings.workerModel !== undefined && settings.workerProviderId !== meta.workerProviderId);
-  if (directorChanged) { meta.directorModel = settings.directorModel; meta.directorProviderId = settings.directorProviderId; }
-  if (workerChanged) { meta.workerModel = settings.workerModel; meta.workerProviderId = settings.workerProviderId; }
+  // Resume re-reads Settings for tool policy and auto-allow, so a policy
+  // change after a failure takes effect on the retry. Models are the run's
+  // own unless "Resume on…" says otherwise; a changed director model then
+  // restarts the session fresh (the mission doc carries the state forward).
+  const settings = await effectiveSettings(projectId);
+  // A plain Resume keeps the run's own models. It used to re-read the
+  // project's Settings and treat any difference as "the human changed the
+  // model" — but a run whose models were chosen at start (a local model
+  // picked on the card) differs from Settings by construction, and one
+  // Resume silently handed a 9B local-model test to Fable and Opus, at $4.82,
+  // and called the result the 9B's. Changing models on resume is now only
+  // ever explicit: "Resume on…" passes `pick`. A model is picked together
+  // with the provider that serves it; a pick without a provider id means the
+  // project's own provider, as at start.
+  const directorChanged = Boolean(pick.directorModel) && (
+    modelChoice(pick.directorModel) !== meta.directorModel || pick.directorProviderId !== meta.directorProviderId);
+  const workerChanged = Boolean(pick.workerModel) && (
+    modelChoice(pick.workerModel) !== meta.workerModel || pick.workerProviderId !== meta.workerProviderId);
+  if (directorChanged) { meta.directorModel = modelChoice(pick.directorModel); meta.directorProviderId = pick.directorProviderId; }
+  if (workerChanged) { meta.workerModel = modelChoice(pick.workerModel); meta.workerProviderId = pick.workerProviderId; }
   meta.toolPolicy = settings.toolPolicy;
   meta.autoAllowReadOnly = settings.autoAllowReadOnly;
   meta.status = 'running';
@@ -1348,6 +1349,7 @@ const server = http.createServer(async (req, res) => {
           providerHasKey: await providerHasKeyOf(p),
           activeRun: run ? { ...run.meta } : null,
           lastRun: lastRun && {
+            id: lastRun.id,
             mission: lastRun.mission, title: lastRun.title, status: lastRun.status,
             createdAt: lastRun.createdAt, costUsd: lastRun.costUsd,
             // The card may print a dollar only where the dollar was real.