nearly-cli 0.1.10 → 0.1.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nearly-cli",
-  "version": "0.1.10",
+  "version": "0.1.11",
   "description": "A pull request tells you what changed. Nearly tells you what nearly happened: the commands a human refused, the pushes policy blocked, the turns rolled back.",
   "type": "module",
   "bin": {

package/scripts/doctor.mjs

@@ -73,8 +73,20 @@ if (!health) {
   let mine = root;
   try { mine = realpathSync(root); } catch { /* compare literally */ }
   const same = health.root === mine;
-  say(same ? true : false, 'server', same ? `v${health.version}` : `a different install is answering: ${health.root || 'an older build'}`,
-    'run `nearly` here — it closes an older server that is holding the port');
+  let ours = null;
+  try { ours = JSON.parse(readFileSync(join(root, 'package.json'), 'utf8')).version; } catch { /* unknown */ }
+  if (same) {
+    say(true, 'server', `v${health.version}`);
+  } else if (health.version && health.version === ours) {
+    // Running this through npx gives a throwaway directory every time, so the
+    // paths differ even when the build is identical. Saying "a different
+    // install" there is true and useless; the version is what anyone cares
+    // about, and a matching one is holding nothing back.
+    say(null, 'server', `v${health.version} from another copy of the same version — nothing stale about it`);
+  } else {
+    say(false, 'server', `an older build is answering${health.version ? ` (v${health.version})` : ''}: ${health.root || 'it does not say where it lives'}`,
+      'run `nearly` here — it closes the older server holding the port');
+  }
 }
 
 // 4 — recordings for this branch, matched the way the record builder matches

package/scripts/post-recap.mjs

@@ -49,7 +49,12 @@ lines.push(sb.runs > 1
 lines.push('');
 lines.push(urlBase
   ? `**[Watch the record (${mmss(sb.totalS)})](${urlBase}/${slug}.html)** · ${sb.runs > 1 ? `${sb.runs} agent sessions` : `agent \`${sb.name}\``} · ${sb.model} · ${sb.date}`
-  : `Record: \`ui/records/${slug}.html\` in the Nearly checkout (${mmss(sb.totalS)}, not hosted yet) · ${sb.runs > 1 ? `${sb.runs} agent sessions` : `agent \`${sb.name}\``} · ${sb.model} · ${sb.date}`);
+  // No host configured. Everything a reviewer needs to act on is in this
+  // comment already — what was refused, and what the diff therefore cannot show
+  // them. Only the player is missing, so say where it is honestly rather than
+  // naming a path from the developer's own checkout that means nothing to
+  // anybody who installed this.
+  : `${sb.runs > 1 ? `${sb.runs} agent sessions` : `Agent \`${sb.name}\``} · ${sb.model} · ${sb.date} · ${mmss(sb.totalS)} recording, kept on the author's machine`);
 lines.push('');
 if (outcome?.notDone?.length) {
   lines.push(`> **${outcome.notDone.length} thing${outcome.notDone.length > 1 ? 's' : ''} the agent wanted to do did not happen.** The diff cannot show you this.`);
@@ -67,7 +72,7 @@ sb.scenes.forEach((s, i) => { lines.push(`${i + 1}. **${s.kind}** — ${s.narrat
 lines.push('');
 lines.push('</details>');
 lines.push('');
-lines.push(`<sub>Every number above was computed from the session recording. ${sb.polished ? 'Sentences were rewritten by a model; facts were not.' : 'No model wrote any of it.'}</sub>`);
+lines.push(`<sub>Every number above was computed from the session recording. ${sb.polished ? 'Sentences were rewritten by a model; facts were not.' : 'No model wrote any of it.'}${urlBase ? '' : ' The narrated version is not published anywhere; `nearly publish` puts it on GitHub Pages.'}</sub>`);
 // A hidden marker so we can find our own comment again on the next push and
 // edit it, instead of stacking a new one on every push until nobody reads any.
 // Deliberately carries no product name. This string is how a comment is