atris 3.31.0 → 3.32.0

package/lib/autoland.js

@@ -12,6 +12,8 @@ const { spawnSync } = require('child_process');
 
 const DEFAULT_DIGEST_HOUR = 9;
 const DEFAULT_ALARM_HOURS = 24;
+const OPERATOR_WHY = /\b(sav(?:e|es|ing)|burn(?:s|ing)?|wast(?:e|es|ing)|cost(?:s|ing)?|prevent\w*|stop(?:s|ping)?|break(?:s|ing)?|fail(?:s|ing)?|slow(?:s|er)?|faster|cheaper|easier|clearer|safer|simpler|trust|revenue|users?|customers?|so that|because)\b/i;
+const AGENT_JARGON = /[a-z0-9]_[a-z0-9]|\b[a-z]+[A-Z]\w*|--[a-z]|\bCLI-\d+/;
 
 function policyPath(root) {
   return path.join(root, '.atris', 'policy', 'autoland.json');
@@ -131,7 +133,7 @@ function waitingOnHuman(tasks, now = Date.now()) {
     .filter((t) => t.review?.agent_certified === true || t.metadata?.agent_certified === true)
     .map((t) => ({
       ref: t.display_id || t.legacy_ref || t.id,
-      title: String(t.title || '').slice(0, 90),
+      title: String(t.title || '').slice(0, 200),
       tag: String(t.tag || ''),
       hours: waitingHours(t, now),
     }))
@@ -168,16 +170,54 @@ function cutAtWord(text, max) {
   return `${cut.slice(0, lastSpace > max * 0.6 ? lastSpace : max)}...`;
 }
 
+// A line ends where a thought ends. Long lines close at their last complete
+// clause inside the budget and read whole: no ellipsis, nothing dangling.
+// Only when a line has no clause boundary at all does a word cut remain.
+function finishThought(text, max = 160) {
+  const clean = String(text || '').replace(/\s+/g, ' ').trim().replace(/[,;:\s]+$/, '');
+  if (clean.length <= max) return clean;
+  const cut = clean.slice(0, max);
+  const sentence = Math.max(cut.lastIndexOf('. '), cut.lastIndexOf('! '), cut.lastIndexOf('? '));
+  if (sentence > max * 0.3) return cut.slice(0, sentence + 1);
+  const clause = Math.max(cut.lastIndexOf(', '), cut.lastIndexOf('; '), cut.lastIndexOf(' ('), cut.lastIndexOf(': '));
+  if (clause > max * 0.45) return cut.slice(0, clause);
+  const space = cut.lastIndexOf(' ');
+  return `${cut.slice(0, space > max * 0.6 ? space : max)}...`;
+}
+
+function operatorReady(text) {
+  const value = String(text || '');
+  return OPERATOR_WHY.test(value) && !AGENT_JARGON.test(value);
+}
+
+function hasAgentJargon(text) {
+  return AGENT_JARGON.test(String(text || ''));
+}
+
 // One line per piece, in the work's own words: the landing sentence beats
 // the task title, because titles are written for the queue, not for a text.
+// The cut is generous on purpose: a specific sentence the reader trusts
+// (what happened + how big + how we know) beats a short one they must chase.
 function digestLine(item) {
-  const happened = String(item.happened || '').split(/(?<=[.!?])\s+/)[0] || '';
+  const happened = String(item.happened || '').replace(/^completed:\s*/i, '').split(/(?<=[.!?])\s+/)[0] || '';
   const title = String(item.title || '').replace(/^Mission XP:\s*/i, '');
   const source = happened.length >= 12 ? happened : title;
-  const line = cutAtWord(source.replace(/[.\s]+$/, ''), 72);
+  const line = finishThought(source.replace(/[.\s]+$/, ''), 160);
   return line.charAt(0).toLowerCase() + line.slice(1);
 }
 
+// A rough sentence still beats a count: when a landing sentence carries agent
+// vocabulary, strip what a reader can't use (flag dashes, ids, underscores)
+// and show it anyway. Content always ships; the gate only changes how it reads.
+function dejargon(line) {
+  return String(line || '')
+    .replace(/--([a-z][a-z-]*)/g, '$1')
+    .replace(/\bCLI-\d+\b/g, '')
+    .replace(/([a-z0-9])_([a-z0-9])/gi, '$1 $2')
+    .replace(/\s+/g, ' ')
+    .trim();
+}
+
 // The mission loop files a check-off every time it stops instead of
 // inventing work; those all say the same thing, so they collapse into one line.
 function isCleanStop(item) {
@@ -185,40 +225,105 @@ function isCleanStop(item) {
     || /^(Mission XP:\s*)?Decide and start the next useful mission/i.test(String(item.title || ''));
 }
 
-// Short enough for one iMessage, plain enough to read over coffee — and it
-// names the work, because "7 things landed" tells the score, not the story.
-function composeDigest({ accepted, waiting, landed, project }) {
+// Short enough for one iMessage, plain enough to read walking to the car.
+// Every line answers: what happened, how big, how we know. It names the work
+// because "7 things landed" tells the score, not the story — and it always
+// ends with what's waiting and what to do next, because those are the only
+// lines that ask anything of the reader.
+function composeDigest({ accepted, waiting, landed, project, nextMoves = [] }) {
   const lines = [];
-  lines.push(`atris ${project} daily`);
+  lines.push(`atris ${project}: yesterday`);
   const autoCount = accepted.auto.length;
   const humanCount = accepted.human.length;
   if (autoCount > 0) {
-    lines.push('got their final sign-off on their own (checked twice, proof on file):');
+    // Three results, air between them, the rest on ask: the whole message
+    // fits a laptop screen with no scrolling. Reading it is one glance.
     const stops = accepted.auto.filter(isCleanStop);
-    const work = accepted.auto.filter((item) => !isCleanStop(item));
-    for (const item of work.slice(0, 4)) lines.push(`- ${digestLine(item) || item.ref}`);
-    if (work.length > 4) lines.push(`- and ${work.length - 4} more`);
-    if (stops.length > 0) lines.push(`- ${plur(stops.length, 'check')} that the loop stops cleanly instead of inventing work`);
+    const work = accepted.auto
+      .filter((item) => !isCleanStop(item))
+      .map((item) => ({ item, line: digestLine(item) }));
+    // Operator-ready sentences lead; rough ones get de-jargoned and still
+    // shown. An empty report is the one thing a report must never be.
+    const ready = work.filter(({ line }) => operatorReady(line));
+    const rough = work.filter(({ line }) => !operatorReady(line))
+      .map(({ item, line }) => ({ item, line: dejargon(line) }));
+    const visibleWork = [...ready, ...rough].slice(0, 3);
+    const held = Math.max(0, work.length - visibleWork.length) + (stops.length > 0 ? 1 : 0);
+    if (visibleWork.length > 0) {
+      lines.push('landed on their own (verified twice, proof on file):');
+      for (const { item, line } of visibleWork) {
+        lines.push('');
+        lines.push(`- ${line || item.ref}${item.member ? ` (${item.member})` : ''}`);
+      }
+      lines.push('');
+      if (held > 0) lines.push(`${plur(held, 'more result')} when you want them: atris autoland digest`);
+    }
   }
-  if (humanCount > 0) lines.push(`you approved ${plur(humanCount, 'piece')} yourself`);
   if (autoCount + humanCount === 0) lines.push('nothing finished in the last day');
-  const byMember = new Map();
-  for (const item of [...accepted.auto, ...accepted.human]) {
-    if (!item.member || isCleanStop(item)) continue;
-    byMember.set(item.member, (byMember.get(item.member) || 0) + 1);
-  }
-  if (byMember.size > 0) {
-    lines.push(`workers: ${Array.from(byMember, ([who, n]) => `${who} landed ${n}`).join(', ')}`);
-  }
   if (waiting.length > 0) {
-    lines.push(`waiting on you: ${waiting.length} (oldest ${waiting[0].hours}h) — atris task reviews`);
+    // The only lines that ask the reader to act, so they name what's asking.
+    lines.push(`waiting on you (approve or bounce: atris task reviews):`);
+    for (const item of waiting.slice(0, 2)) {
+      lines.push(`- ${finishThought(dejargon(item.title), 140)}${item.hours >= 12 ? ` (${item.hours}h)` : ''}`);
+    }
+    if (waiting.length > 2) lines.push(`- and ${waiting.length - 2} more`);
   } else {
-    lines.push('nothing is waiting on you');
+    lines.push('waiting on you: nothing');
   }
   if (landed && landed.branches > 0) {
-    lines.push(`${plur(landed.branches, 'piece')} of work still in the air${landed.due > 0 ? `, ${landed.due} overdue` : ''} — atris land`);
+    lines.push(`in the air: ${plur(landed.branches, 'piece')}${landed.due > 0 ? `, ${landed.due} overdue` : ', all fresh'} (atris land)`);
+  }
+  // nextMoves: array of {title, owner}, or {moves, unexplained} where
+  // unexplained counts queue items whose sentence carries no operator why.
+  // Those are counted, not shown — a raw title the reader can't act on is
+  // noise here and a writing bug at its source.
+  const nextInfo = Array.isArray(nextMoves)
+    ? { moves: nextMoves, unexplained: 0 }
+    : (nextMoves || { moves: [], unexplained: 0 });
+  const moves = (nextInfo.moves || []).filter((move) => move && move.title).slice(0, 3);
+  const unexplained = Number(nextInfo.unexplained || 0);
+  if (moves.length > 0) {
+    lines.push('next, if you agree:');
+    for (const move of moves) {
+      lines.push(`- ${finishThought(move.title, 140)}${move.owner ? ` (best fit: ${move.owner})` : ''}`);
+    }
+    if (unexplained > 0) lines.push(`- ${plur(unexplained, 'more idea')} that can't explain themselves yet (atris now)`);
+  } else if (unexplained > 0) {
+    lines.push(`${plur(unexplained, 'idea')} in the queue can't explain themselves yet (atris now)`);
+  }
+  if (!lines.some((line) => line.includes('atris autoland digest'))) {
+    lines.push('the full story: atris autoland digest');
+  }
+  return lines.join('\n');
+}
+
+// The moment something lands, the operator hears it in one glance: the
+// landing sentence each piece wrote for a day-one PM at finish time, its
+// author, nothing else. Rough sentences get de-jargoned, never hidden.
+function composeLiveUpdate({ landedRefs, tasks, project }) {
+  const byRef = new Map((tasks || []).map((t) => [String(t.display_id || t.legacy_ref || t.id), t]));
+  const items = (landedRefs || []).map((ref) => {
+    const t = byRef.get(String(ref));
+    if (!t) return { title: String(ref) };
+    return {
+      ref: String(ref),
+      title: String(t.title || '').slice(0, 200),
+      happened: String(t.review?.landing?.happened || t.metadata?.landing_happened || '').slice(0, 300),
+      member: String(t.claimed_by || t.assigned_to || '').trim(),
+    };
+  }).filter((item) => !isCleanStop(item));
+  if (!items.length) return '';
+  const lines = [`atris ${project}: just landed`];
+  for (const item of items.slice(0, 3)) {
+    const raw = digestLine(item);
+    const line = operatorReady(raw) ? raw : dejargon(raw);
+    lines.push('');
+    lines.push(`- ${finishThought(line, 160)}${item.member ? ` (${item.member})` : ''}`);
+  }
+  if (items.length > 3) {
+    lines.push('');
+    lines.push(`and ${plur(items.length - 3, 'more piece')}: atris autoland digest`);
   }
-  lines.push('the full story: atris autoland digest');
   return lines.join('\n');
 }
 
@@ -264,12 +369,15 @@ module.exports = {
   buildCronLine,
   composeAlarm,
   composeDigest,
+  composeLiveUpdate,
   cronInstalled,
   cronMarker,
   dueForAlarm,
   installCron,
   liveAcceptAuthorization,
   markAlerted,
+  operatorReady,
+  hasAgentJargon,
   policyPath,
   readPolicy,
   readState,

package/lib/operator-next.js

@@ -0,0 +1,7 @@
+function printOperatorNext(command, { label = 'Next' } = {}) {
+  const clean = String(command || '').trim();
+  if (!clean) return;
+  console.log(`${label}: ${clean}`);
+}
+
+module.exports = { printOperatorNext };

package/lib/pulse.js

@@ -14,6 +14,7 @@
 
 const fs = require('fs');
 const path = require('path');
+const { DEFAULT_CLAUDE_RUNNER_MODEL } = require('./runner-command');
 
 const PULSE_RECEIPT_SCHEMA = 'atris.pulse_tick.v1';
 // Reuse the improve-tick scorecard schema so the brain + policy-lessons see
@@ -354,7 +355,7 @@ function buildTickScript(opts = {}) {
     stateHome,
     deadlineEpoch,
     marker = PULSE_MARKER,
-    model = 'opus',
+    model = DEFAULT_CLAUDE_RUNNER_MODEL,
     runnerProfile = '',
     runnerBin = '',
     runnerCommandTemplate = '',
@@ -419,8 +420,8 @@ log="$LOG_DIR/$stamp.log"
 
 cd "$ROOT" || { echo "$(date -Iseconds) ROOT missing" >> "$LOG_DIR/control.log"; exit 1; }
 
-# Autonomous ticks must target a live model alias, never a versioned id that can
-# retire out from under the loop (lesson: retired-model-kills-loop-silently).
+# Autonomous ticks use the pinned default unless the installer supplied a model.
+# Operators can still override per install with --model or env.
 ${runnerModelExport}
 ${runnerProfileExport}
 ${runnerBinExport}

package/lib/runner-command.js

@@ -10,24 +10,40 @@
 // ATRIS_RUNNER_PROFILE -> legacy ATRIS_CLAUDE_MODEL env -> pinned default.
 const DEFAULT_CLAUDE_RUNNER_MODEL = 'claude-opus-4-8';
 const DEFAULT_CLAUDE_RUNNER_BIN = 'claude';
-const RUNNER_PROFILES = Object.freeze({
+
+// Canonical runner profiles. Each entry is the config an operator would pick
+// on purpose. Aliases (spelling variants of the same profile) resolve to the
+// canonical config but are kept OUT of operator-facing lists so config errors
+// stay honest: "one of: atris-fast", not three duplicate spellings.
+const RUNNER_PROFILE_DEFS = Object.freeze({
   'atris-fast': Object.freeze({
     bin: 'ax',
     model: 'atris:fast',
     commandTemplate: '{bin} --fast {prompt}',
   }),
-  'atris2-fast': Object.freeze({
-    bin: 'ax',
-    model: 'atris:fast',
-    commandTemplate: '{bin} --fast {prompt}',
-  }),
-  'atris-2-fast': Object.freeze({
-    bin: 'ax',
-    model: 'atris:fast',
-    commandTemplate: '{bin} --fast {prompt}',
-  }),
 });
 
+// Alias -> canonical profile name. Every alias resolves to the same config as
+// its canonical target; adding a spelling variant here is a config change, not
+// a duplicated profile body that can drift.
+const RUNNER_PROFILE_ALIASES = Object.freeze({
+  'atris2-fast': 'atris-fast',
+  'atris-2-fast': 'atris-fast',
+});
+
+// Back-compat surface: RUNNER_PROFILES still resolves every accepted name
+// (canonical + alias) to its frozen config, so existing lookups by any spelling
+// keep working.
+const RUNNER_PROFILES = Object.freeze(
+  Object.fromEntries([
+    ...Object.entries(RUNNER_PROFILE_DEFS),
+    ...Object.entries(RUNNER_PROFILE_ALIASES).map(([alias, target]) => [alias, RUNNER_PROFILE_DEFS[target]]),
+  ])
+);
+
+// Operator-facing list: canonical profile names only (no alias noise).
+const RUNNER_PROFILE_NAMES = Object.freeze(Object.keys(RUNNER_PROFILE_DEFS));
+
 function shellWord(value) {
   const s = String(value || '');
   if (/^[A-Za-z0-9_./:-]+$/.test(s)) return s;
@@ -51,7 +67,7 @@ function resolveRunnerProfile() {
   if (!name) return null;
   const profile = RUNNER_PROFILES[name];
   if (!profile) {
-    throw new Error(`Unknown ATRIS_RUNNER_PROFILE "${name}". Known profiles: ${Object.keys(RUNNER_PROFILES).join(', ')}`);
+    throw new Error(`Unknown ATRIS_RUNNER_PROFILE "${name}". Known profiles: ${RUNNER_PROFILE_NAMES.join(', ')}`);
   }
   return profile;
 }
@@ -98,7 +114,7 @@ function buildRunnerAvailabilityCommand() {
 function runnerAvailabilityFailureMessage(error) {
   const message = error && error.message ? String(error.message).trim() : '';
   if (message.startsWith('Unknown ATRIS_RUNNER_PROFILE')) {
-    return `${message}. Set ATRIS_RUNNER_PROFILE to one of: ${Object.keys(RUNNER_PROFILES).join(', ')}.`;
+    return `${message}. Set ATRIS_RUNNER_PROFILE to one of: ${RUNNER_PROFILE_NAMES.join(', ')}.`;
   }
 
   let runnerBin = 'configured runner';
@@ -154,6 +170,9 @@ module.exports = {
   DEFAULT_CLAUDE_RUNNER_MODEL,
   DEFAULT_CLAUDE_RUNNER_BIN,
   RUNNER_PROFILES,
+  RUNNER_PROFILE_DEFS,
+  RUNNER_PROFILE_ALIASES,
+  RUNNER_PROFILE_NAMES,
   resolveRunnerProfileName,
   resolveRunnerProfile,
   resolveRunnerModel: resolveClaudeRunnerModel,

package/lib/task-db.js

@@ -1198,7 +1198,7 @@ function chatTask(db, { id, actor, content, goal, summary }) {
   };
 }
 
-function reviewTask(db, { id, actor, reward, lesson, nextTask, proof, verify, careerXpEligible = false, clearedFields = [] }) {
+function reviewTask(db, { id, actor, reward, lesson, nextTask, proof, verify, careerXpEligible = false, clearedFields = [], landing = {} }) {
   if (!id) throw new Error('id required');
   const row = getTask(db, id);
   if (!row) return { reviewed: false, reason: 'not_found' };
@@ -1213,6 +1213,15 @@ function reviewTask(db, { id, actor, reward, lesson, nextTask, proof, verify, ca
   const clearedReviewFields = Array.isArray(clearedFields)
     ? Array.from(new Set(clearedFields.filter(field => field === 'lesson' || field === 'next_task')))
     : [];
+  const landingInput = landing && typeof landing === 'object' ? landing : {};
+  let landingUpdated = false;
+  for (const key of ['happened', 'checked', 'tested', 'decision']) {
+    const cleaned = cleanLandingText(landingInput[key]);
+    if (cleaned) {
+      metadata[`landing_${key}`] = cleaned;
+      landingUpdated = true;
+    }
+  }
   const reviewingPendingProof = row.status === 'review'
     && metadata.approval_status === 'pending'
     && numericReward <= 0
@@ -1250,7 +1259,7 @@ function reviewTask(db, { id, actor, reward, lesson, nextTask, proof, verify, ca
     metadata.accepted_at = new Date().toISOString();
     metadata.accepted_by = reviewer;
   }
-  if (reviewingPendingProof || updatingPendingReviewProof || (numericReward > 0 && row.status === 'done')) {
+  if (reviewingPendingProof || updatingPendingReviewProof || (numericReward > 0 && row.status === 'done') || landingUpdated) {
     withBusyRetry(() => db.prepare(`
       UPDATE tasks
          SET metadata = ?,
@@ -1271,6 +1280,14 @@ function reviewTask(db, { id, actor, reward, lesson, nextTask, proof, verify, ca
     payload.agent_certified = metadata.agent_certified === true;
     payload.agent_certification_policy = metadata.agent_certification_policy || null;
   }
+  if (landingUpdated) {
+    payload.landing = {
+      happened: metadata.landing_happened || null,
+      checked: metadata.landing_checked || null,
+      tested: metadata.landing_tested || null,
+      decision: metadata.landing_decision || null,
+    };
+  }
   if (clearedReviewFields.length) payload.cleared_review_fields = clearedReviewFields;
   const event = appendTaskEvent(db, {
     taskId: id,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "atris",
-  "version": "3.31.0",
+  "version": "3.32.0",
   "main": "bin/atris.js",
   "bin": {
     "atris": "bin/atris.js",
@@ -55,7 +55,7 @@
     "test": "node --test",
     "lint:decks": "node scripts/lint-all-decks.js",
     "publish:release": "node scripts/publish-atris-release.js",
-    "prepare": "cp scripts/pre-commit .git/hooks/pre-commit 2>/dev/null && chmod +x .git/hooks/pre-commit || true"
+    "prepare": "cp scripts/pre-commit .git/hooks/pre-commit 2>/dev/null && cp scripts/commit-msg .git/hooks/commit-msg 2>/dev/null && cp scripts/prepare-commit-msg .git/hooks/prepare-commit-msg 2>/dev/null && chmod +x .git/hooks/pre-commit .git/hooks/commit-msg .git/hooks/prepare-commit-msg || true"
   },
   "keywords": [
     "atrisdev",