agentbox-app 0.1.7 → 0.1.12

package/main/settings.mjs

@@ -15,7 +15,7 @@
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
-import { saveConfig } from './config.mjs';
+import { codexChoiceToOpen, saveConfig } from './config.mjs';
 import { resolveClaudeBin, forgetClaudeBin, INSTALL_URL } from './claude-bin.mjs';
 import { AGENT_MODES } from '../shared/agents.mjs';
 import { analyticsKey, diagnosticsOn } from './analytics.mjs';
@@ -23,7 +23,9 @@ import { CLAUDE_PERMISSION_MODES } from '../shared/work-items.mjs';
 import { accountSentence, engineTroubleNote } from '../shared/spawn-trouble.mjs';
 import { readPlan } from './claude-plan.mjs';
 import { effectiveProfiles } from './account-discovery.mjs';
-import { machineSlots, machineNote, MAX_SLOTS } from './machine.mjs';
+import { machineSlots, machineNote, autoAgents, MAX_SLOTS } from './machine.mjs';
+import { NAME } from '../shared/product-name.mjs';
+import { autoSlots } from './memory-gate.mjs';
 import { accountIdentity, claudeLoginCommand, duplicateAccountNote, linkAccountTooling, makeClaudeHome } from './account-tooling.mjs';
 import { isEngine } from '../shared/engines.mjs';
 import { codexAccount, codexLoginCommand, makeCodexHome } from './codex-account.mjs';
@@ -270,6 +272,10 @@ export function recheckClaude(config) {
   // has just found Claude Code updates it rather than leaving the rest of the
   // app pointing at where it used to not be.
   if (state.claudeFound && state.claudeBin) config.claudeBin = state.claudeBin;
+  // And whether it is here at all, which is what decides that a Mac with only
+  // Codex runs on Codex (`Supervisor#_enginesFound`). Only a certain answer
+  // moves it, so an unsure search never takes Claude Code away from anybody.
+  if (state.claudeFound || state.claudeCertain) config.claudeFound = state.claudeFound;
   return state;
 }
 
@@ -319,6 +325,12 @@ export function codexState(config, where = {}) {
   };
 }
 
+/** `codexState` under the names the walk's last card reads beside Claude's. */
+export function codexFoundState(config, where = {}) {
+  const state = codexState(config, where);
+  return { codexFound: state.found, codexCertain: state.certain, codexInstallUrl: state.url };
+}
+
 /**
  * LOOK AGAIN, PROPERLY, for the second engine. The twin of `recheckClaude`
  *  above, and it earns its place for a sharper reason: finding Codex here is
@@ -329,6 +341,10 @@ export function recheckCodex(config) {
   forgetCodexBin();
   const state = codexState(config);
   config.codexBin = state.found ? state.bin : null;
+  // Found while the app is open, by this button or the plan setup: the choice
+  // opens now, as it would have at the next launch (main/config.mjs).
+  const opened = codexChoiceToOpen(config, state.found);
+  if (opened && config.appDir) saveConfig(config, { engineChoice: opened });
   return state;
 }
 
@@ -366,9 +382,9 @@ export function readSettings({ config, supervisor, store }) {
     const trouble = troubles[profile] ?? null;
     const resting = (supervisor._profileCooldown?.[profile] ?? 0) >= now;
     // WHOSE SUBSCRIPTION THIS ROW IS, not merely which folder it lives in.
-    // The page named its rows after folders, so nothing on it could tell her
-    // that her work account had been replaced by her personal one;
-    // main/account-tooling.mjs has the morning that cost. Null when the
+    // The page named its rows after folders, so nothing on it could tell a
+    // person that one of their accounts had been replaced by the other;
+    // main/account-tooling.mjs has how that happens. Null when the
     // folder holds no login, and then the row reads as it always did.
     const identity = accountIdentity(profile === 'default' ? null : profile);
     return {
@@ -421,9 +437,8 @@ export function readSettings({ config, supervisor, store }) {
       // THE MARK, already a drawable url or null, resolved once by listProducts.
       logo: product.logo,
       // ONE SWITCH A PROJECT STILL HAS (w-d19d6d387c, 2026-09-22). It had five
-      // controls. Hers, on a picture of them: "I still don't actually
-      // understand what those toggles do, even after reading them... I would
-      // lean towards just having project-level instructions." The drive, the
+      // controls, and what they did was not clear even after reading their
+      // labels, so project-level instructions replace them. The drive, the
       // per-project pause and personal mode are gone from the app entirely.
       // `permission` stays readable because `projectSessionArgs` is still
       // honoured for anyone who sets one in the config by hand, and the page
@@ -434,15 +449,40 @@ export function readSettings({ config, supervisor, store }) {
       permission: override ? permissionMode(override) : 'workspace',
       permissionArgs: override ?? null,
       codexMode: config.projectCodexMode?.[product.slug] ?? 'workspace',
+      // THE SUPERVISOR IS ASKED RATHER THAN THE CONFIG FILE, so the page shows
+      // what will actually happen: a tie naming a login that has been signed out
+      // is ignored when work is handed out, and reads 'any' here too.
+      account: supervisor._projectProfile?.(product.slug) ?? 'any',
       instructions,
       running: sessions.filter((s) => s.product === product.slug).length,
     };
   });
 
+  // ARCHIVED PROJECTS, only so the Projects page can bring one back. Name and
+  // folder, the two things that tell three projects called Daydream apart.
+  let archivedProjects = [];
+  try {
+    archivedProjects = store.listProducts({ includeArchived: true })
+      .filter((p) => p.archived === true)
+      .map((p) => ({ slug: p.slug, name: p.name, dir: p.dir }));
+  } catch {}
+
   return {
+    archivedProjects,
     workspace: {
       agentsRunning: !supervisor.paused,
       sessionsAtOnce: config.maxConcurrentSessions,
+      // AUTOMATIC, AND WHAT IT WORKS OUT TO (w-e5225b62ba). `agentsAuto` is who
+      // is deciding; `agentsTotal` is what the whole Mac is running at once,
+      // which is the number the page shows, because the per-account one is
+      // arithmetic nobody should have to do. `agentsNudge` is how many steps
+      // down "Something felt slow" has taken it.
+      agentsAuto: !!config.agentsAuto,
+      agentsTotal: status.capacity,
+      agentsNudge: Math.max(0, Math.round(Number(config.agentsNudge) || 0)),
+      // What Automatic would say right now, so a manual number can be compared
+      // with it without turning Automatic back on to find out.
+      agentsAutoTotal: autoAgents({ gated: !!config.memoryGate, nudge: config.agentsNudge }),
       // THE PLAN THAT SET THE NUMBER ABOVE, when a plan set it. Null in the
       // ordinary case: she picked the number herself, or it is the three every
       // install has always had. It is only ever filled in when somebody would
@@ -457,6 +497,8 @@ export function readSettings({ config, supervisor, store }) {
       slotsSuggested: slotsHere.slots,
       slotsMax: MAX_SLOTS,
       machineNote: machineNote(slotsHere, config.maxConcurrentSessions),
+      memoryGate: memoryGateSettings({ config, supervisor }),
+      leftovers: leftoverSettings({ config, supervisor }),
       capacity: status.capacity,
       running: sessions.length,
       model: parseSessionArgs(workspaceArgs).model,
@@ -509,6 +551,10 @@ export function readSettings({ config, supervisor, store }) {
       // Someone who reads "install it, then open this screen again" and does
       // exactly that has to see it turn found, without restarting the app.
       ...claudeState(config),
+      // AND WHETHER CODEX IS, because the app needs one of the two and not
+      // both: the last card of the walk only shuts the inbox when neither is
+      // here (renderer/src/App.tsx).
+      ...codexFoundState(config),
       // AND WHAT THIS MAC'S CLAUDE CODE CALLS ITS MODELS.
       //
       // In the same trip, and for the same reason the line above is looked up
@@ -577,21 +623,26 @@ function engineSettings(supervisor, config) {
   // and that stopped being true somewhere between codex-cli versions.
   // main/codex-account.mjs holds the measurement. Null when nobody is signed
   // in, and the card then reads exactly as it always did.
-  const codex = supervisor.engineChoiceOpened()
-    ? {
-      ...codexState(config),
-      trouble: engineTroubleFor(supervisor, 'codex'),
-      account: codexAccount(supervisor._codexHome()),
-      // EVERY CODEX LOGIN ON THE MAC, NOT JUST THE FIRST. The supervisor has
-      // run one app-server per CODEX_HOME since a second login became real;
-      // what never existed was a screen that could see them. One entry per
-      // `codexProfiles` word, resolved through the supervisor's own mapping so
-      // the card and the fleet cannot disagree about which folder a name
-      // means.
-      accounts: codexAccountRows(supervisor),
-    }
-    : null;
-  if (choices.length < 2) {
+  // A Mac that runs on Codex alone has asked about it by having nothing else.
+  //
+  // AND THE PAGE IS DRAWN ON EVERY MAC NOW (w-d5d632e503). A tester who only
+  // uses Codex looked for it in Settings, found nothing, and had no way to
+  // tell "not installed" from "not offered". Not found, the page says so and
+  // links the install, which is the question that person had.
+  const codexIsHome = engine === 'codex';
+  const codex = {
+    ...codexState(config),
+    trouble: engineTroubleFor(supervisor, 'codex'),
+    account: codexAccount(supervisor._codexHome()),
+    // EVERY CODEX LOGIN ON THE MAC, NOT JUST THE FIRST. The supervisor has
+    // run one app-server per CODEX_HOME since a second login became real;
+    // what never existed was a screen that could see them. One entry per
+    // `codexProfiles` word, resolved through the supervisor's own mapping so
+    // the card and the fleet cannot disagree about which folder a name
+    // means.
+    accounts: codexAccountRows(supervisor),
+  };
+  if (choices.length < 2 && !codexIsHome) {
     return { engine, engineChoices: choices, codex, codexModels: [], codexModelDefault: null, codexModel: null };
   }
   const home = supervisor._codexHome();
@@ -680,6 +731,24 @@ export function setProjectSetting({ config, supervisor }, { product, key, value
       saveConfig(config, { projectCodexMode: Object.keys(map).length ? map : undefined });
       break;
     }
+    // WHICH CLAUDE LOGIN THIS PROJECT'S AGENTS RUN ON. 'any' is the absence of
+    // an opinion, like 'workspace' above, and it deletes the entry rather than
+    // writing a word: a config file with no `projectAccounts` is the file every
+    // Mac had before this existed.
+    //
+    // AND A NAME THIS MAC CANNOT HONOUR IS REFUSED AT THE DOOR rather than
+    // written and ignored later. The supervisor does ignore one (`_projectProfile`),
+    // because an account can be signed out after the fact, but a pick made
+    // against a list we just read has no excuse to be wrong.
+    case 'account': {
+      const known = effectiveProfiles(config.authProfiles, { home: config.home });
+      if (value !== 'any' && !known.includes(value)) throw new Error(`unknown Claude account: ${value}`);
+      const map = { ...(config.projectAccounts ?? {}) };
+      if (value === 'any') delete map[product];
+      else map[product] = value;
+      saveConfig(config, { projectAccounts: Object.keys(map).length ? map : undefined });
+      break;
+    }
     default:
       throw new Error(`unknown project setting: ${key}`);
   }
@@ -754,8 +823,8 @@ export function addCodexAccount({ config, supervisor }) {
  * accounting while this was a product we sold. She withdrew that on
  * w-3498e0cad2 ("we're no longer offering this as a commercially viable
  * product... the rules have changed so I think you should allow
- * multi-accounting"), and this is the door that decision asked for. Her reason
- * for wanting it is the ordinary one: a work login and a personal login, both
+ * multi-accounting"), and this is the door that decision asked for. The
+ * ordinary reason for wanting it: a work login and a personal login, both
  * live, on one Mac.
  *
  * Everything else is the Codex flow exactly. Main makes the folder and
@@ -781,19 +850,143 @@ export function addClaudeAccount({ config, supervisor }) {
   return { home: made.home, profile: made.profile, command: claudeLoginCommand(made.home) };
 }
 
+/**
+ * The Agents page's memory rows: the switch, the number of heavy commands at
+ * once (null is Auto), what Auto works out to here, and one sentence about
+ * right now while it is on.
+ */
+export function memoryGateSettings({ config, supervisor }) {
+  const on = !!config.memoryGate;
+  const slots = Number.isFinite(config.memoryGateSlots) ? config.memoryGateSlots : null;
+  // What Auto works out to on this Mac, whether or not somebody picked.
+  const slotsAuto = autoSlots(os.totalmem());
+  let now = null;
+  if (on) {
+    let s = null;
+    try { s = supervisor.memoryGateStatus?.() ?? null; } catch {}
+    if (s?.role === 'standby') now = `Another ${NAME} on this computer is coordinating.`;
+    else if (s) {
+      const heavy = (s.running ?? []).filter((r) => r.holdsSlot).length;
+      const waiting = (s.waiting ?? []).length;
+      const mem = s.pressure === 'critical' ? 'Memory is very short.' : s.pressure === 'tight' ? 'Memory is tight.' : 'Memory is fine.';
+      const run = heavy ? `${heavy} heavy command${heavy === 1 ? '' : 's'} running` : 'Nothing heavy running';
+      now = `${mem} ${run}${waiting ? `, ${waiting} waiting` : ''}.`;
+    }
+  }
+  return { on, slots, slotsAuto, slotsMax: MAX_SLOTS, now, reading: memoryReading({ config, supervisor }) };
+}
+
+/**
+ * THE FIGURES THE RUNNING PAGE LEADS WITH (w-e5225b62ba), rather than the same
+ * facts tacked onto the end of three grey paragraphs: "6 running now", "Holding
+ * 3 commands now", "Nothing running now".
+ *
+ * Every field is null when it is not known, and the page leaves out what is
+ * null. A figure invented to fill a slot is worse than a gap: this card's whole
+ * job is to be the one place on the screen that is measured.
+ *
+ * `usedPct` is the kernel's own `kern.memorystatus_level` turned round, which is
+ * what the gate already polls, so the bar and the word above it cannot disagree.
+ */
+export function memoryReading({ config, supervisor }) {
+  const out = { heavy: null, waiting: null, pressure: null, usedPct: null, asked: null, waited: null, refused: null };
+  if (!config.memoryGate) return out;
+  let s = null;
+  try { s = supervisor.memoryGateStatus?.() ?? null; } catch { return out; }
+  if (!s || s.role === 'standby') return out;
+  out.heavy = (s.running ?? []).filter((r) => r.holdsSlot).length;
+  out.waiting = (s.waiting ?? []).length;
+  out.pressure = s.pressure ?? null;
+  if (Number.isFinite(s.freePct)) out.usedPct = Math.max(0, Math.min(100, 100 - s.freePct));
+  const c = s.counts ?? {};
+  if (Number.isFinite(c.asked)) out.asked = c.asked;
+  if (Number.isFinite(c.waited)) out.waited = c.waited;
+  if (Number.isFinite(c.refused)) out.refused = c.refused;
+  return out;
+}
+
+/**
+ * The Agents page's leftovers row: the switch, and one sentence about what
+ * finished agents have left running while it is on.
+ */
+export function leftoverSettings({ config, supervisor }) {
+  const on = !!config.cleanupLeftovers;
+  let now = null;
+  let s = null;
+  try { s = supervisor.leftoverStatus?.() ?? null; } catch {}
+  const list = s?.leftovers ?? [];
+  const stoppable = list.reduce((n, l) => n + (l.programs ?? 0), 0);
+  const kept = list.reduce((n, l) => n + (l.kept ?? 0), 0);
+  const total = stoppable + kept;
+  // OFF, IT SAYS WHAT TURNING IT ON WOULD STOP, before anybody does: the
+  // consent is to these programs too, not only to future ones (agreed with
+  // Codex, 2026-10-05).
+  if (!on) {
+    if (total && stoppable) {
+      now = `Right now finished agents have left ${total} program${total === 1 ? '' : 's'} running; turning this on stops ${stoppable === total ? (total === 1 ? 'it' : 'them') : `${stoppable} of them`}, two hours from then.`;
+    }
+    return { on, now };
+  }
+  {
+    if (!total) now = 'Nothing left running by finished agents.';
+    else {
+      const next = list.filter((l) => l.programs && l.stopsAt).map((l) => l.stopsAt).sort((a, b) => a - b)[0];
+      const when = (ms) => {
+        const m = Math.max(1, Math.round((ms - Date.now()) / 60_000));
+        return m >= 60 ? `${Math.floor(m / 60)} h${m % 60 ? ` ${m % 60} m` : ''}` : `${m} m`;
+      };
+      now = `Finished agents have left ${total} program${total === 1 ? '' : 's'} running.`;
+      if (stoppable) now += ` ${stoppable} ${next && next > Date.now() ? `stop in ${when(next)}` : 'are stopping'}`;
+      if (kept) now += `${stoppable ? ';' : ''} ${kept} ${kept === 1 ? 'is' : 'are'} kept`;
+      now += '.';
+      const survivors = s?.survivors?.length ?? 0;
+      if (survivors) now += ` ${survivors} did not stop when asked.`;
+    }
+  }
+  return { on, now };
+}
+
 export function setWorkspaceSetting({ config, supervisor }, { key, value }) {
   switch (key) {
     case 'agentsRunning':
       supervisor.paused = !value;
       supervisor.onChange?.();
       break;
+    // WHO DECIDES THE NUMBER (w-e5225b62ba). Turning Automatic on does not
+    // throw away the number somebody set: it stays in the file, unread, so
+    // turning Automatic off again gives them their own number back rather than
+    // a fresh guess.
+    case 'agentsAuto':
+      saveConfig(config, { agentsAuto: !!value });
+      supervisor.wake?.();
+      supervisor.onChange?.();
+      break;
+    // SOMETHING FELT SLOW. The one lever a person has that no reading can
+    // replace: they are sitting in front of the Mac and the app is not. It only
+    // ever subtracts, and main/machine.mjs refuses to take it below one.
+    case 'agentsFeltSlow': {
+      const now = Math.max(0, Math.round(Number(config.agentsNudge) || 0));
+      saveConfig(config, { agentsNudge: value === 'reset' ? 0 : now + 1 });
+      supervisor.wake?.();
+      supervisor.onChange?.();
+      break;
+    }
     case 'sessionsAtOnce': {
       // THE MACHINE DOES NOT CLAMP THIS, and for one round it did. These are
       // developers using the tool as they want, so the range is the control's
       // own range, the same on every Mac. What the hardware does is SUGGEST a
       // starting number, one row up.
       const n = Math.max(1, Math.min(MAX_SLOTS, Math.round(Number(value) || 1)));
-      saveConfig(config, { maxConcurrentSessions: n });
+      // Setting a number by hand IS taking the decision back, so the page never
+      // shows a stepper that the supervisor is quietly ignoring.
+      saveConfig(config, { maxConcurrentSessions: n, agentsAuto: false });
+      // An explicit choice replaces the startup plan suggestion for BOTH
+      // engines. Leaving this marker set makes Codex ignore the new number
+      // until loadConfig runs again at restart. It is derived, not persisted.
+      config.planSlotsFrom = null;
+      // Fill newly available slots through the usual queue and its guards.
+      // Lowering the cap leaves existing work running and gates new starts.
+      supervisor.wake?.();
       break;
     }
     // Written as an opt-OUT (`diagnostics: false`) so that on is what a config
@@ -801,6 +994,32 @@ export function setWorkspaceSetting({ config, supervisor }, { key, value }) {
     // see in her own file. `saveConfig` mutates the live config too, and
     // main/analytics.mjs reads the switch at every send, so off takes effect
     // from the moment she moves it and not at the next launch.
+    // HOLD HEAVY WORK WHEN MEMORY IS SHORT (w-3958c3753d). Takes effect now:
+    // the coordinator starts or stops this moment, and workers spawned from
+    // here on carry the check or do not. A worker already running keeps the
+    // hooks it was started with; with the switch off its hook finds no socket
+    // and lets every command straight through.
+    case 'memoryGate':
+      saveConfig(config, { memoryGate: !!value });
+      supervisor.setMemoryGate?.(!!value)?.catch?.((e) => console.warn('zero: memory gate:', e.message));
+      supervisor.onChange?.();
+      break;
+    // STOP WHAT FINISHED AGENTS LEAVE RUNNING (main/leftovers.mjs). Turning it
+    // on records the moment, which is the earliest the clock may start from.
+    case 'cleanupLeftovers':
+      saveConfig(config, value ? { cleanupLeftovers: true, cleanupLeftoversSince: Date.now() } : { cleanupLeftovers: false });
+      supervisor.setLeftoverCleanup?.(!!value)?.catch?.((e) => console.warn('zero: leftovers:', e.message));
+      supervisor.onChange?.();
+      break;
+    case 'memoryGateSlots': {
+      const n = value === null || value === undefined || value === 'auto'
+        ? null
+        : Math.max(1, Math.min(MAX_SLOTS, Math.round(Number(value) || 1)));
+      saveConfig(config, { memoryGateSlots: n });
+      if (config.memoryGate) supervisor.setMemoryGate?.(true)?.catch?.(() => {});
+      supervisor.onChange?.();
+      break;
+    }
     case 'diagnostics':
       saveConfig(config, { diagnostics: !!value });
       break;