atris 3.57.3 → 3.58.5

package/lib/member-alive.js

@@ -6,7 +6,7 @@ const { spawnSync } = require('child_process');
 
 const MEMBER_OPERATE_PROGRESS_PREFIX = 'ATRIS_MEMBER_OPERATE_PROGRESS\t';
 
-// Wake decisions where the member should NOT run operate work this tick — it is waiting on a
+// Wake decisions where the member should NOT run operate work this tick: it is waiting on a
 // human (ask), on a pending review or clean scope (wait), or has no active goal yet (stop).
 const NON_OPERATE_DECISIONS = new Set(['ask', 'wait', 'stop']);
 
@@ -65,6 +65,7 @@ function resolveMemberOperateScript(cwd) {
   const candidates = [
     path.join(cwd, 'scripts', 'member-operate.mjs'),
     path.join(cwd, 'scripts', 'member-operate.js'),
+    path.join(__dirname, '..', 'scripts', 'member-operate.mjs'),
   ];
   for (const candidate of candidates) {
     if (fs.existsSync(candidate)) return candidate;
@@ -87,6 +88,7 @@ function runMemberOperateScript(scriptPath, member, opts) {
   if (opts.agent) args.push('--agent', String(opts.agent));
   if (opts.model) args.push('--model', String(opts.model));
   if (opts.noPrime) args.push('--no-prime');
+  if (opts.sharedCheckout) args.push('--shared-checkout');
   // Inherit stderr so member-operate progress lines reach the parent alive/loop process live.
   const result = spawnSync(process.execPath, args, {
     cwd: opts.cwd || process.cwd(),
@@ -188,7 +190,7 @@ function runAliveTick(name, opts = {}) {
     return tick;
   }
 
-  emitAliveProgress({ kind: 'phase', text: `Running ${name} tick — streaming live below.` });
+  emitAliveProgress({ kind: 'phase', text: `Running ${name} tick, streaming live below.` });
   tick.operate = runMemberOperateScript(operateScript, name, {
     cwd,
     execute: true,
@@ -197,6 +199,7 @@ function runAliveTick(name, opts = {}) {
     agent: opts.agent || null,
     model: opts.model || null,
     noPrime: true,
+    sharedCheckout: opts.sharedCheckout,
   });
   tick.needs_user = tick.needs_user || tick.operate.needs_user === true;
 
@@ -210,17 +213,18 @@ function runAliveTick(name, opts = {}) {
   const operateOk = tick.operate.ok !== false;
   const acceptOk = tick.auto_accept.ok !== false;
   tick.ok = operateOk && acceptOk;
+  const planned = tick.ok && tick.operate.payload?.status === 'planned';
   tick.status = tick.needs_user
     ? 'needs_user'
     : tick.ok
-      ? 'completed'
+      ? planned ? 'planned' : 'completed'
       : 'failed';
   tick.reason = tick.ok
-    ? 'alive_tick_complete'
+    ? planned ? tick.operate.reason : 'alive_tick_complete'
     : tick.operate.reason || tick.auto_accept.json?.summary
       ? 'partial_failure'
       : 'alive_tick_failed';
-  tick.receipt_path = tick.operate.receipt_path || wakeJson.receipt_path || null;
+  tick.receipt_path = tick.operate.receipt_path || null;
   tick.finished_at = stampIso();
   return tick;
 }

package/lib/memory-view.js

@@ -1,4 +1,4 @@
-// Memory view — another way to look at memory updates. Reads the workspace's
+// Memory view: another way to look at memory updates. Reads the workspace's
 // compounding memory (lessons.md, learnings.jsonl, task projection) and builds a
 // content spec rendered as a beautiful HTML page by lib/html-render.
 //
@@ -9,13 +9,13 @@
 const fs = require('fs');
 const path = require('path');
 
-// "- **[2026-06-18] some-id** — pass — text..." -> { date, id, status, text, resolved }
+// "- **[2026-06-18] some-id**, pass, text..." -> { date, id, status, text, resolved }
 // A leading "[resolved]" marker (written when a lesson's detector passes) is
 // stripped from the text and surfaced as `resolved` so the view can retire it.
 function parseLessons(text) {
   const out = [];
   for (const line of String(text || '').split('\n')) {
-    const m = line.match(/^-\s*\*\*\[([^\]]+)\]\s*([^*]+?)\*\*\s*[—:-]*\s*(pass|fail)?\s*[—:-]*\s*(.*)$/i);
+    const m = line.match(/^-\s*\*\*\[([^\]]+)\]\s*([^*]+?)\*\*\s*[\u2014:-]*\s*(pass|fail)?\s*[\u2014:-]*\s*(.*)$/i);
     if (!m) continue;
     let body = m[4].trim();
     const resolved = /\[resolved\]/i.test(body);

package/lib/mission-protected-lane.js

@@ -51,7 +51,7 @@ function changedPathsFromDiff(diff) {
 
 // Prose is not a protected surface. Docs, judge cards, wikis, journals and run
 // receipts routinely *describe* CSP or force-dynamic without changing any
-// behaviour — replaying this guard over 60 real commits, 5 of 6 hits were
+// behaviour: replaying this guard over 60 real commits, 5 of 6 hits were
 // markdown that merely mentioned the words. A guard that pauses on every judge
 // card gets switched off, which is worse than no guard. Content signals
 // therefore apply to code only; PATH signals still apply to every file.

package/lib/mission-root.js

@@ -6,7 +6,7 @@ const { spawnSync } = require('child_process');
 // The mission store lives at <workspace>/.atris/state. Historically every
 // mission entry point defaulted its root to process.cwd(), so running any
 // `atris mission ...` verb from a subdirectory (e.g. atris/features/food-ordering)
-// silently created a NESTED .atris there — a second, split-brain mission store
+// silently created a NESTED .atris there: a second, split-brain mission store
 // the fleet never reads. Resolve to the workspace root so mission state always
 // lands there, whichever subdir the caller happened to be in.
 //
@@ -14,7 +14,7 @@ const { spawnSync } = require('child_process');
 // (lib/task-db findWorkspaceRoot): a bound customer sub-workspace spine
 // (.atris/business.json | atris/atris.md) wins, else the git toplevel, else the
 // caller's cwd. Sharing one resolver keeps mission state and task state in the
-// same root — previously mission used only the git toplevel, so a mission
+// same root. previously mission used only the git toplevel, so a mission
 // started inside a bound sub-workspace split away from that workspace's task
 // state. A git worktree reports its own toplevel (its .git marker), which is
 // the correct per-worktree store, so --worktree missions are unaffected. Walk-up

package/lib/orb-context.js

@@ -1,4 +1,4 @@
-// Orb context — deterministic greeting + suggestion synthesis for the orb surface.
+// Orb context: deterministic greeting + suggestion synthesis for the orb surface.
 // Pure synthesis lives in buildOrbContext (unit-testable, no fs); collectOrbContext
 // does the read-only fs aggregation from a project root. No model calls anywhere:
 // 0ms-to-greeting is the contract (atris/features/orb-desktop/PLAN.md).
@@ -65,7 +65,7 @@ function normalizeTasks(tasks) {
 }
 
 // Known suggestion action prefixes. Stripped repeatedly (front-anchored) so that
-// "review: review: X" or "pick up pick up X" both collapse to the bare title —
+// "review: review: X" or "pick up pick up X" both collapse to the bare title.
 // used both to keep labels single-prefixed and to compare titles across sources.
 // "atrisbox" is included because Atrisbox threads are titled "Atrisbox: <label>",
 // which otherwise shields inner accumulated prefixes from stripping (seen live
@@ -85,12 +85,12 @@ function stripActionPrefixes(value) {
 }
 
 // Lowercase, prefix-stripped, whitespace-collapsed form used only for equality/
-// prefix comparison across sources (tasks vs threads) — never shown to the user.
+// prefix comparison across sources (tasks vs threads), never shown to the user.
 function normalizeForMatch(value) {
   return stripActionPrefixes(String(value || '').toLowerCase().replace(/\s+/g, ' ').trim());
 }
 
-// True when two titles are "the same work" — exact match, or one is a truncated
+// True when two titles are "the same work": exact match, or one is a truncated
 // prefix of the other (titles get cut at different lengths per source, so a
 // straight equality check would miss the common case). Guarded by a minimum
 // needle length so short titles ("fix it") don't false-positive against unrelated
@@ -106,7 +106,7 @@ function titlesOverlap(a, b) {
 }
 
 // Background Atrisbox taps write a launch-receipt thread (desk_<project>_
-// atrisbox_bg_<slug>) — it's proof a turn ran, not a resumable conversation.
+// atrisbox_bg_<slug>), proof a turn ran, not a resumable conversation.
 // They self-archive on delivery (App.tsx sendAtrisboxMessage) and main.js
 // sweeps stragglers, but excluding them from warm eligibility here too is
 // cheap (a string check, no I/O) and keeps the count honest even for threads
@@ -127,7 +127,7 @@ function normalizeThreads(threads) {
     .sort((a, b) => b.updatedAt - a.updatedAt);
 }
 
-// Dedupe by "same work" (titlesOverlap) keeping the newest of each cluster —
+// Dedupe by "same work" (titlesOverlap) keeping the newest of each cluster,
 // used before counting warm threads so the greeting number reflects distinct
 // resumable work, not the same thread title repeated under different ids.
 function dedupeByTitle(threads) {
@@ -218,7 +218,7 @@ function buildOrbContext({ nowMd = '', tasks = [], threads = [], memberLoops = [
   const suggestions = [];
   // Titles already spoken for by a suggestion, used to dedupe across sources
   // (review/resume/continue are built independently but can point at the same
-  // underlying work — e.g. a task's warm thread title vs the task's own title).
+  // underlying work, e.g. a task's warm thread title vs the task's own title).
   const usedTitles = [];
 
   function pushSuggestion(suggestion, title = '') {
@@ -268,7 +268,7 @@ function buildOrbContext({ nowMd = '', tasks = [], threads = [], memberLoops = [
     });
   }
 
-  // Explicitly blocked/stalled statuses are their own lane — distinct from the
+  // Explicitly blocked/stalled statuses are their own lane, distinct from the
   // stale-doing "finish" lane below, which infers staleness from the clock.
   const blockedTask = cleanTasks
     .filter((t) => t.status === 'blocked' || t.status === 'stalled')
@@ -310,7 +310,7 @@ function buildOrbContext({ nowMd = '', tasks = [], threads = [], memberLoops = [
   for (const thread of warmThreads) {
     if (usedTitles.some((title) => titlesOverlap(title, thread.title))) continue;
     // Thread titles inherit message text, so a picked-up thread can arrive as
-    // "pick up X: pick up X..." — strip the accumulated prefixes before reusing.
+    // "pick up X: pick up X...", strip the accumulated prefixes before reusing.
     const threadTitle = cleanTitle(stripActionPrefixes(String(thread.title || '')), 40);
     if (threadTitle.length < 3) continue;
     resumeThread = thread;
@@ -369,7 +369,7 @@ function buildOrbContext({ nowMd = '', tasks = [], threads = [], memberLoops = [
 
   // These stay at the tail even when the backlog is full, so orientation and
   // planning never disappear behind the deeper task-fed pool.
-  // 'status' is answered locally by the view from gmSnapshot — no model call.
+  // 'status' is answered locally by the view from gmSnapshot, no model call.
   const staticSuggestions = [{
     id: 'status',
     label: "what's moving",