create-pmos 0.126.0 → 0.128.0

package/workspace/server.js

@@ -639,7 +639,9 @@ function repoIdentity(root) {
   // a fault: a folder registered by path legitimately has no origin, and null is the honest answer.
   try { remote = gitInfo.originUrl(root); } catch { remote = null; }
   try {
-    const bi = gitInfo.branchInfo(root);
+    // headInfo, not branchInfo: only the branch name is used here, and branchInfo's branch list,
+    // `git status` and worktree list cost ~5 more shell-outs per workspace per TTL (session lifecycle).
+    const bi = gitInfo.headInfo(root, tree || null);
     if (bi.ok) branch = bi.detached ? null : bi.current;
     // R1/A2 — carry the CODE and the MESSAGE, not a boolean. branchInfo already separates the two
     // failures and says so in its own comment: `no-git` is an ordinary folder and "deserves no
@@ -2387,6 +2389,23 @@ function reportToMarkdown(surface) {
 }
 // Fire one schedule: run its skill → contract → deliverable → stamp → file to needs-you. Same path
 // for a scheduled tick and for run-now (R7). Pure of PTYs in v1 (deterministic report skill).
+// A schedule that fires again while its previous run is still open ends that run first, unless it is still producing
+// output or holds a call waiting for the PM. Before, every fire added a session beside the last one, and each stayed
+// up to the idle limit (workspace-session-lifecycle). Only automatic origins are ever touched.
+function endSupersededRuns(origin, now = Date.now()) {
+  const ended = [];
+  if (!sessionOrigin.isAutomatic(origin)) return ended;
+  for (const s of sessions.values()) {
+    if (s.origin !== origin || !s.pty || s.autoDelete === 'never') continue;
+    if (harness.deriveState(s.state, s.lastOutputAt, now).state === 'working') continue;
+    if (sessionShown(s, now) === 'blocked') continue;
+    s.autoEnded = true; s.note = 'ended: the schedule started a new run';
+    if (s.tmux) tmuxKillSession(s.tmux);
+    try { s.pty.kill(); } catch {}
+    ended.push(s.id);
+  }
+  return ended;
+}
 function fireSchedule(sched) {
   const w = workspaces.get(sched.ws) || homeWorkspace();
   const root = w ? w.root : SELF_ROOT;
@@ -2412,6 +2431,7 @@ function fireSchedule(sched) {
     // The schedule's recorded profile travels with the process it creates (profiles R8.3). Only an
     // AGENT-kind task has a PTY, which is why `profile_applies` is false for the skill-kind rows
     // below rather than the control simply being hidden.
+    endSupersededRuns('schedule:' + sched.id);
     const s = spawnSession({ name: sched.name, agent: task.agent || 'claude', ws: sched.ws,
       origin: 'schedule:' + sched.id, initialInput: prompt ? prompt + '\r' : null,
       profile: sched.profile == null ? null : sched.profile,
@@ -2502,6 +2522,8 @@ function recordConversation(s) {
 }
 function persist(s) {
   if (s.state === 'exited' && !s._txSaved) { s._txSaved = true; try { recordConversation(s); } catch {} }
+  // When it ended: the clock the sweep's keepExitedDays reads. A live session has none (a respawn clears it).
+  if (s.state === 'exited') { if (!s.exitedAt) s.exitedAt = nowISO(); } else if (s.exitedAt) delete s.exitedAt;
   else if (s.state !== 'exited') s._txSaved = false;
   // note IS persisted — a restored session must still say why it exited. `root` is persisted
   // so a session survives even if the workspace registry is lost (it re-registers its root).
@@ -2509,6 +2531,7 @@ function persist(s) {
     state: s.state, note: s.note || '', created: s.created, tmux: s.tmux || null, origin: s.origin || null };
   if (s.autoDelete) meta.autoDelete = s.autoDelete;
   if (s.autoEnded) meta.autoEnded = true;
+  if (s.exitedAt) meta.exitedAt = s.exitedAt;
   // The breakdown item a Hand to agent session is bound to, so a restart keeps the binding (pm-os-absorb Run 1).
   if (s.plan_item) meta.plan_item = s.plan_item;
   // pm-os-absorb Run 2 (R25): the SHA-256 of the agent's PMOS_SESSION token, never the token (D08). A restored session
@@ -2601,6 +2624,8 @@ function restorePersisted() {
       } else {
         s.state = 'exited';
         s.note = meta.note || 'restored after server restart — scrollback preserved; ↻ respawn to start a new process';
+        // A record from before exitedAt existed: its last write is when it last changed, the closest truthful clock.
+        if (!s.exitedAt) { try { s.exitedAt = new Date(fs.statSync(path.join(SESS_DIR, f)).mtimeMs).toISOString(); } catch {} }
       }
       const n = Number(String(meta.id).split('-')[1]); if (Number.isFinite(n) && n >= seq) seq = n + 1;
     } catch {}
@@ -2897,7 +2922,9 @@ function sweepAutomaticSessions(now = Date.now()) {
   const policy = readSessionPolicy();
   const done = { ended: [], removed: [] };
   for (const s of [...sessions.values()]) {
-    const act = sessionOrigin.decide({ ...s, shown: sessionShown(s, now) }, policy, now);
+    // `shown` is a function: decide() asks for it only for a live automatic session past its idle limit, so the sweep no
+    // longer resolves a transcript (a `ps` walk for a shell pane) for every session every 30 seconds.
+    const act = sessionOrigin.decide({ ...s, shown: () => sessionShown(s, now) }, policy, now);
     if (act === 'end' && s.pty) {
       s.autoEnded = true; s.note = 'ended: idle too long (automatic session)';
       if (s.tmux) tmuxKillSession(s.tmux);
@@ -8986,7 +9013,7 @@ if (require.main === module) {
     });
   });
 }
-module.exports = { server, online: { ONLINE, applyOnline, tunnel, publicHost }, spawnSession, removeSession, renameSession, sessions, libraryTree, readTextDoc,
+module.exports = { server, online: { ONLINE, applyOnline, tunnel, publicHost }, spawnSession, removeSession, sweepAutomaticSessions, endSupersededRuns, renameSession, sessions, libraryTree, readTextDoc,
   registerWorkspace, forgetWorkspace, workspaces, library, initiativeIndex, fsList,
   // R1 — exported so the suite can assert the CARRY-THROUGH: branchInfo separates an ordinary
   // non-repo folder from a real git fault, and the whole legibility defect was that distinction

package/workspace/test-files.json

@@ -53,6 +53,7 @@
     "learn.test.js",
     "session-flag.test.js",
     "session-origin.test.js",
+    "session-lifecycle.test.js",
     "tab-gone.test.js",
     "live-notes.test.js",
     "plan-items.test.js",
@@ -172,6 +173,7 @@
     "learn.test.js": 3,
     "session-flag.test.js": 1,
     "session-origin.test.js": 1,
+    "session-lifecycle.test.js": 2,
     "tab-gone.test.js": 1,
     "live-notes.test.js": 3,
     "pm-shell-test.js": 6,

package/workspace/test-lists/console3-reports.json

@@ -0,0 +1,6 @@
+{
+  "_comment": "console3 Reports dashboards: the server half (library, defaults, scope filter, layout store, page-press layout route) and the browser suite (scopes, edit, share, library, Drafts tab).",
+  "modules": ["report-dashboards.test.js"],
+  "panels": ["c3-reports-test.js"],
+  "seconds": { "report-dashboards.test.js": 3, "c3-reports-test.js": 30 }
+}

package/workspace/test-lists/local-test-kit.json

@@ -0,0 +1,5 @@
+{
+  "_comment": "local-test-kit: the demo-data builder and the launcher script of scripts/local-kit/. Starts no server and binds no port.",
+  "modules": ["local-kit.test.js"],
+  "seconds": { "local-kit.test.js": 6 }
+}