create-pmos 0.128.0 → 0.130.0

package/workspace/server.js

@@ -2669,10 +2669,14 @@ function adoptOrphanTmuxSessions() {
   for (const t of tmuxList().sort((a, b) => a.created - b.created)) {
     if (!t.name.startsWith('pmos-') || claimed.has(t.name)) continue;
     const dir = t.dir && isDir(t.dir) ? path.resolve(t.dir) : null;
-    // Where it belongs: the workspace containing its pane directory; else — because a pane the PM
-    // can see must be a pane the PM can reach — that directory is registered as one, exactly as
-    // `pmos <path>` would. Only the home workspace is assumed when there is no readable directory.
-    const w = workspaceForPath(dir) || (dir && registerWorkspace(dir)) || homeWorkspace();
+    const start = t.start && isDir(t.start) ? path.resolve(t.start) : null;
+    // Where it belongs: the workspace containing the directory the pane STARTED in, then the one containing where it is
+    // now. Only when neither is registered is a directory registered — the start directory first, because that is a
+    // folder a session was opened in, not wherever the PM last `cd`-ed (registering that made ~ or /tmp a permanent
+    // workspace). The current directory is the last resort, kept because a pane the PM can see must be one they can
+    // reach; the home workspace is assumed only when there is no readable directory at all.
+    const w = workspaceForPath(start) || workspaceForPath(dir) || (start && registerWorkspace(start))
+      || (dir && registerWorkspace(dir)) || homeWorkspace();
     if (!w) continue;
     const id = nextSessionId();
     // The agent is the pane's own start command, not a guess: tmux spawned it directly (see
@@ -2680,7 +2684,7 @@ function adoptOrphanTmuxSessions() {
     const agent = adoptedAgent(t.cmd);
     const s = {
       id, ws: w.id, root: w.root, name: `recovered ${agent}`, agent,
-      origin: null, cwd: dir && insideRoot(w.root, dir) ? dir : w.root,
+      origin: t.origin || null, cwd: dir && insideRoot(w.root, dir) ? dir : w.root,
       state: 'working', pty: null, tmux: t.name, note: '', buf: '',
       cols: 80, rows: 24, clients: new Set(), created: nowISO(), recovered: true,
     };
@@ -3115,6 +3119,10 @@ function tmuxList() {
     .map((name) => ({
       name,
       dir: tmuxField(name, '#{pane_current_path}'),
+      // Where the pane STARTED (the -c of new-session): the workspace it was opened in. The current path follows every
+      // `cd`, so a pane the PM moved to ~ or /tmp would otherwise file itself under that folder.
+      start: tmuxField(name, '#{pane_start_path}'),
+      origin: tmuxField(name, '#{@pmos_origin}'),
       created: Number(tmuxField(name, '#{session_created}')) || 0,
       cmd: tmuxField(name, '#{pane_start_command}'),
     }));
@@ -3461,6 +3469,9 @@ function spawnSession({ name, agent, cwd, ws, origin, initialInput, profile, ini
       s.buf = `\x1b[38;5;203m✗ ${why}\x1b[0m\r\n\x1b[38;5;244mFix the above, then ↻ respawn.\x1b[0m\r\n`;
       persist(s); pushSessions(); pushWorkspaces(); return s;
     }
+    // The origin rides on the tmux session itself (a user option), so a pane whose record is lost is adopted with it:
+    // a recovered automatic job stays automatic and the sweep can still end it. One cheap call, only when there is one.
+    if (origin) tmuxCtl(['set-option', '-t', tname, '@pmos_origin', String(origin)]);
     announceStateHealth(s);   // the unmigrated notice comes FIRST — it explains an empty skills list
     announceTree(s);   // before attach, so the advisory is the first line of the pane's record
     announceModel(s);
@@ -3990,9 +4001,19 @@ function startDayCheck(run, reason, now) {
   if (run.checks.session_id) run = dayRunLib.sessionEnded(run, run.checks.session_id, now).run;
   const sk = daySkillPrompt(run, 'day-check');
   if (sk.error) return { ok: false, error: sk.error };
-  const prompt = `${sk.prompt}\n\nThis run: /day-check ${reason} — the last check was at ${run.checks.last || 'none yet'}; read what changed since ${run.checks.last || run.started_at}.`;
-  const s = spawnSession({ name: 'Daytime check', agent: sk.task.agent || 'claude', ws: sk.w ? sk.w.id : run.ws, origin: 'day-check', initialInput: prompt + '\r',
-    modelAsk: { scheduleTier: null, skillTier: sk.task.tier || null, skill: sk.task.skill } });
+  const line = `This run: /day-check ${reason} — the last check was at ${run.checks.last || 'none yet'}; read what changed since ${run.checks.last || run.started_at}.`;
+  // The last check's session is used again while it is open and quiet: it has the skill in its context, so only this
+  // run's line is typed into it. Before, every check started a new session and the old one sat idle until the sweep
+  // ended it, about one more process per interval (workspace-session-lifecycle).
+  const prev = run.checks.prev_session && sessions.get(run.checks.prev_session);
+  let s = null;
+  if (prev && prev.pty && prev.state !== 'exited' && harness.deriveState(prev.state, prev.lastOutputAt, now).state !== 'working') {
+    try { firstInput.deliver(prev, line + '\r'); s = prev; } catch { s = null; }
+  }
+  if (!s) {
+    s = spawnSession({ name: 'Daytime check', agent: sk.task.agent || 'claude', ws: sk.w ? sk.w.id : run.ws, origin: 'day-check', initialInput: `${sk.prompt}\n\n${line}\r`,
+      modelAsk: { scheduleTier: null, skillTier: sk.task.tier || null, skill: sk.task.skill } });
+  }
   if (s.state === 'exited') return { ok: false, error: s.note || 'the check session could not start' };
   const c = dayRunLib.checkStarted(run, s.id, reason, now);
   return c.ok ? { ok: true, run: c.run, session: s.id } : c;
@@ -9013,7 +9034,7 @@ if (require.main === module) {
     });
   });
 }
-module.exports = { server, online: { ONLINE, applyOnline, tunnel, publicHost }, spawnSession, removeSession, sweepAutomaticSessions, endSupersededRuns, renameSession, sessions, libraryTree, readTextDoc,
+module.exports = { server, online: { ONLINE, applyOnline, tunnel, publicHost }, spawnSession, removeSession, sweepAutomaticSessions, endSupersededRuns, adoptOrphanTmuxSessions, 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

@@ -25,6 +25,7 @@
     "pm-items.test.js",
     "jobs-to-be-done.test.js",
     "pm-items-routes.test.js",
+    "brief-page.test.js",
     "item-steps.test.js",
     "status-facts-panel.test.js",
     "jev-claims.test.js",
@@ -122,7 +123,8 @@
     "pm-breakdown-phone-test.js",
     "first-paint-count.test.js",
     "pm-card-layout-test.js",
-    "pm-portfolio-inline-test.js"
+    "pm-portfolio-inline-test.js",
+    "pm-brief-page-test.js"
   ],
   "ui": [
     "ui-test.js"
@@ -145,6 +147,8 @@
     "pm-items.test.js": 1,
     "jobs-to-be-done.test.js": 1,
     "pm-items-routes.test.js": 1,
+    "brief-page.test.js": 1,
+    "pm-brief-page-test.js": 30,
     "item-steps.test.js": 1,
     "status-facts-panel.test.js": 1,
     "jev-claims.test.js": 3,

package/workspace/test-lists/calpop.json

@@ -0,0 +1,4 @@
+{
+  "panels": ["pm-calpop-test.js"],
+  "seconds": { "pm-calpop-test.js": 30 }
+}