@volter/supercode-health 0.1.11 → 0.1.12

package/lib/drift.mjs

@@ -5,7 +5,7 @@
 // A process runs a file of a supercode install (the native, `…/@volter/supercode-cli-<os>-<cpu>/bin/supercode`, or
 // node running a file of an `@volter/supercode*` package); when that file was written after the process started (its
 // ctime: an extractor may set a file's mtime, never its ctime), the release on disk is not the one it runs.
-import { statSync } from 'node:fs';
+import { realpathSync, statSync } from 'node:fs';
 import { run } from './run.mjs';
 
 /** A supercode install's file: the native, or a script of an @volter/supercode* package. */
@@ -15,8 +15,26 @@ const INSTALL_FILE = /^\/\S*\/node_modules\/@volter\/supercode(?:-[a-z0-9-]+)?\/
  *  one, never a path in its arguments (a Claude session's `--settings` names supercode's hooks). */
 function ran(command) {
   const words = command.split(/\s+/u);
-  const program = /(^|\/)node$/u.test(words[0] ?? '') ? words.slice(1).find((word) => !word.startsWith('-')) : words[0];
-  return program && INSTALL_FILE.test(program) ? program : null;
+  const node = /(^|\/)node$/u.test(words[0] ?? '');
+  let program = node ? words.slice(1).find((word) => !word.startsWith('-')) : words[0];
+  if (!program?.startsWith('/')) return null;
+  // by its real path: a link (the machine's stable program, a bin link) names an install's file under another name
+  try { program = realpathSync(program); } catch { /* gone: matched as named */ }
+  return INSTALL_FILE.test(program) ? { file: program, native: !node } : null;
+}
+
+/** The inode each native process runs (its `txt` mapping, lsof's first), by pid; empty where lsof cannot say. An
+ *  install that replaced the file gave its path a new inode, while the process still runs the old one. */
+async function runningInodes(pids) {
+  const inodes = new Map();
+  if (!pids.length) return inodes;
+  const answer = await run('lsof', ['-n', '-P', '-a', '-d', 'txt', '-F', 'pi', '-p', pids.join(',')], { timeoutMs: 10_000 }).catch(() => null);
+  let pid = null;
+  for (const line of String(answer?.stdout ?? '').split('\n')) {
+    if (line.startsWith('p')) pid = Number(line.slice(1));
+    else if (line.startsWith('i') && pid && !inodes.has(pid)) inodes.set(pid, Number(line.slice(1)));
+  }
+  return inodes;
 }
 
 /** `ps`'s start time (`lstart`, as `Tue Oct  6 12:09:42 2026`) in ms, or NaN. */
@@ -29,18 +47,25 @@ const startedMs = (lstart) => Date.parse(String(lstart).trim().replace(/\s+/g, '
  */
 export async function staleSupercode({ rows = null } = {}) {
   const lines = rows ?? String((await run('ps', ['-A', '-ww', '-o', 'pid=,lstart=,command='], { timeoutMs: 10_000 })).stdout ?? '').split('\n');
-  const stale = [];
+  const found = [];
   for (const line of lines) {
     const match = /^\s*(\d+)\s+(\w{3}\s+\w{3}\s+\d+\s+[\d:]+\s+\d{4})\s+(.*)$/u.exec(line);
     if (!match) continue;
     const [, pid, lstart, command] = match;
-    const file = ran(command);
-    if (!file) continue;
+    const program = ran(command);
+    if (program) found.push({ pid: Number(pid), lstart, command, ...program });
+  }
+  // a native is read by the inode it runs, where lsof says it: the strongest reading (a ctime also moves on a chmod)
+  const inodes = rows ? new Map() : await runningInodes(found.filter((row) => row.native).map((row) => row.pid));
+  const stale = [];
+  for (const { pid, lstart, command, file } of found) {
     const started = startedMs(lstart);
-    let installed;
-    try { installed = statSync(file).ctimeMs; } catch { continue; } // a file gone is a removed install, said elsewhere
-    // ps says a start to the second: a file written within that second is the same start
-    if (Number.isNaN(started) || installed <= started + 1000) continue;
+    let stat;
+    try { stat = statSync(file); } catch { continue; } // a file gone is a removed install, said elsewhere
+    const installed = stat.ctimeMs;
+    if (inodes.has(pid)) {
+      if (inodes.get(pid) === stat.ino) continue;
+    } else if (Number.isNaN(started) || installed <= started + 1000) continue; // ps says a start to the second
     stale.push({ pid: Number(pid), command: command.slice(0, 200), file, startedAt: new Date(started).toISOString(), installedAt: new Date(installed).toISOString() });
   }
   return stale.sort((a, b) => b.installedAt.localeCompare(a.installedAt));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-health",
-  "version": "0.1.11",
+  "version": "0.1.12",
   "type": "module",
   "description": "Optional machine pack: a cheap, deterministic probe of a machine's vitals that mails the machine's maintainer when a threshold is crossed. Each feature sits behind its own permission barrier.",
   "license": "MIT",