npm - agent-dag - Versions diffs - 1.33.49 → 1.33.51 - Mend

agent-dag 1.33.49 → 1.33.51

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (5) hide show

package/dist/web/assets/{index-DXPYK_Sr.js → index-O75ELHTf.js} +20 -20
package/dist/web/index.html +1 -1
package/package.json +1 -1
package/src/server/index.mjs +18 -4
package/src/server/self-update.mjs +90 -28

package/dist/web/index.html CHANGED Viewed

@@ -5,7 +5,7 @@
     <meta name="viewport" content="width=device-width, initial-scale=1" />
     <title>agents-deck</title>
     <link rel="icon" href="data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 100 100'%3E%3Ctext y='84' font-size='84'%3E%E2%97%89%3C/text%3E%3C/svg%3E" />
-    <script type="module" crossorigin src="/assets/index-DXPYK_Sr.js"></script>
+    <script type="module" crossorigin src="/assets/index-O75ELHTf.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-CF2yG9oG.css">
   </head>
   <body>

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "agent-dag",
-  "version": "1.33.49",
+  "version": "1.33.51",
   "description": "Live deck of Claude Code and Codex agents — watch parallel subagents fork, call tools, and return on one calm canvas. Also available as npx ccdeck and npx agent-dag.",
   "type": "module",
   "bin": {

package/src/server/index.mjs CHANGED Viewed

@@ -898,9 +898,18 @@ async function codexScanOnce(firstRun) {
   }
 }
-function startCodexWatcher(workspace) {
+export function startCodexWatcher(workspace) {
   codexWorkspace = workspace ?? "";
-  if (!existsSync(CODEX_SESSIONS_DIR)) return null;
+  // Deliberately not gated on CODEX_SESSIONS_DIR existing. `codex login`
+  // creates ~/.codex, but sessions/ only appears when the first session
+  // starts — and rollout tailing is the only Codex capture path there is, so
+  // bailing out here meant a fresh install had to restart the deck before any
+  // Codex agent ever showed up, while the banner claimed to be watching.
+  // listRecentCodexRollouts returns [] while the directory is missing, so the
+  // poll below is a cheap no-op that doubles as the existence re-check. A
+  // filesystem watch is no help: fs.watch on a missing path throws, and
+  // watching the parent recursively is macOS/Windows-only.
+  //
   // Initial catalog: create roots for in-progress sessions, skip their
   // history, then poll for new lines.
   codexScanOnce(true).catch(() => {});
@@ -909,6 +918,12 @@ function startCodexWatcher(workspace) {
   return codexWatchTimer;
 }
+/** Envelopes newer than `seq` from the ring buffer, oldest first. */
+export function eventsSince(seq) {
+  const after = Number(seq) || 0;
+  return events.filter(e => e.seq > after);
+}
 function pushEvent(raw, source, opts = {}) {
   // Synchronous enrichment: if we already know this session's model, stamp
   // it on the payload so the client's recursive scanner picks it up.
@@ -1513,8 +1528,7 @@ export async function startServer({ port = 4317, host = "127.0.0.1", persist = n
     if (req.method === "POST" && url.pathname === "/api/cswap-auto")  return guard(handleCswapAutoAction(req, res), res);
     if (req.method === "GET" && url.pathname === "/api/events") {
-      const since = Number(url.searchParams.get("since") ?? 0);
-      return send(res, 200, events.filter(e => e.seq > since));
+      return send(res, 200, eventsSince(url.searchParams.get("since") ?? 0));
     }
     // POST /api/clear — wipe in-memory buffer + persistence file (UI reset)

package/src/server/self-update.mjs CHANGED Viewed

@@ -45,12 +45,26 @@ const CHECK_MS = 3600_000;
 // enough that a full npm outage costs a dozen ~20-byte requests an hour.
 const RETRY_MS = 300_000;
 const FETCH_TIMEOUT_MS = 6_000;
-// A third marker path: ccusage owns ~/.agents-deck/ccusage/.last-update-check
+// A third marker family: ccusage owns ~/.agents-deck/ccusage/.last-update-check
 // and cswap owns ~/.agents-deck/.cswap-update-check. Sharing one would make the
 // three features fight over a single daily slot.
-const MARKER = join(homedir(), ".agents-deck", ".self-update-check");
-let _inflight = null; // dedupe concurrent /api/version calls into one fetch
+const MARKER_DIR = join(homedir(), ".agents-deck");
+// The name this deck was published under is part of the marker's identity.
+//
+// A single ~/.agents-deck/.self-update-check was shared by every deck on the
+// machine, so whichever one asked npm first pinned the answer for all of them
+// until the window expired — including a freshly started `npx ccdeck` that had
+// never written it, and including decks running a DIFFERENT package where the
+// cached version means nothing. Reported with four decks alive at once: three
+// served the same `checkedAt` to the millisecond and none of them showed the
+// seven releases that had shipped meanwhile. One file per package name means
+// `agents-deck`, `ccdeck` and `agent-dag` decks stop silencing each other; the
+// old shared path stays here only to be inherited from once.
+const LEGACY_MARKER = join(MARKER_DIR, ".self-update-check");
+// Dedupe concurrent /api/version calls into one fetch, per package name — two
+// names are two different questions and must not be answered with one answer.
+const _inflight = new Map();
 // ── version comparison ───────────────────────────────────────────────────────
@@ -186,18 +200,44 @@ async function fetchLatest(name) {
   }
 }
+/** Marker file name for a package: `ccdeck` → `.self-update-check-ccdeck`.
+ *
+ *  Pure, and deliberately strict about what reaches the filesystem — a package
+ *  name may be scoped (`@scope/pkg`), and `/`, `\` and `:` are either a path
+ *  separator or outright illegal in a Windows file name. Everything outside
+ *  `[a-z0-9._-]` collapses to `-`, the scope's leading `@` is dropped so the
+ *  common case reads plainly, and the tail is trimmed so a pathological name
+ *  cannot produce a path the OS refuses. An unusable name falls back to the
+ *  default rather than to the shared file this fix exists to get rid of. */
+export function markerFileName(name = "agents-deck") {
+  const raw = typeof name === "string" ? name.trim().toLowerCase() : "";
+  const safe = raw
+    .replace(/^@/, "")
+    .replace(/[^a-z0-9._-]+/g, "-")
+    .slice(0, 64)
+    // Trimmed after the slice, and at both ends: Windows silently drops a
+    // trailing dot from a file name, so a name that ends in one would write to
+    // a path that is not the path we would later read.
+    .replace(/^[-.]+|[-.]+$/g, "");
+  return `.self-update-check-${safe || "agents-deck"}`;
+}
+function markerPath(name) {
+  return join(MARKER_DIR, markerFileName(name));
+}
 // The marker carries the answer, not just the timestamp. The existing markers
-// store only an mtime, which means a restart inside the 24h window forgets what
-// npm said and shows nothing until the window expires.
+// store only an mtime, which means a restart inside the window forgets what npm
+// said and shows nothing until the window expires.
 //
 // It carries the outcome too. `at` is when npm last ANSWERED and `version` is
 // what it said; `failedAt` is when the last attempt failed, and is null the
 // moment one succeeds. Keeping them apart is what lets the deck say "checked
 // 3m ago" and "could not reach npm" as the different things they are, instead
 // of reporting a timeout as a fresh, up-to-date check.
-function readMarker() {
+function readMarkerFile(path) {
   try {
-    const m = JSON.parse(readFileSync(MARKER, "utf8"));
+    const m = JSON.parse(readFileSync(path, "utf8"));
     // Either half is enough to be worth keeping: a marker written by a first
     // attempt that failed has no `at` yet, and markers written before
     // `failedAt` existed have no `failedAt` at all.
@@ -206,10 +246,26 @@ function readMarker() {
     return null;
   }
 }
-function writeMarker(marker) {
+// The unsuffixed marker every earlier deck wrote, read at most once per name
+// per process: upgrading to this version should not throw away an answer npm
+// already gave. It is consulted only while the per-name file is still missing,
+// so the shared file never goes back to being in charge.
+const _legacy = new Map();
+function legacyMarker(name) {
+  const key = markerFileName(name);
+  if (!_legacy.has(key)) _legacy.set(key, readMarkerFile(LEGACY_MARKER));
+  return _legacy.get(key);
+}
+function readMarker(name) {
+  return readMarkerFile(markerPath(name)) ?? legacyMarker(name);
+}
+function writeMarker(name, marker) {
+  const path = markerPath(name);
   try {
-    mkdirSync(dirname(MARKER), { recursive: true });
-    writeFileSync(MARKER, JSON.stringify(marker));
+    mkdirSync(dirname(path), { recursive: true });
+    writeFileSync(path, JSON.stringify(marker));
   } catch { /* a read-only home must not break the deck */ }
 }
@@ -226,11 +282,12 @@ export function nextMarker({ prev, now, ok, version }) {
   return { at: prev?.at ?? null, version: prev?.version ?? null, failedAt: now };
 }
-// The marker is shared by every deck on the machine, so one deck's check
-// silences the others for the rest of the window. Asking once per PROCESS fixes
-// that, and it lands the check exactly where the user expects the truth: a
-// freshly started deck — which for `npx ccdeck` is every single run.
-let _askedThisProcess = false;
+// Even a per-package marker is shared by every deck running that package, so
+// one deck's check still answers for the others inside the window. Asking once
+// per PROCESS lands the check exactly where the user expects the truth: a
+// freshly started deck — which for `npx ccdeck` is every single run. Keyed by
+// name, so a process that asked about one package has not asked about another.
+const _askedThisProcess = new Set();
 /**
  * Whether to ask npm, or reuse the answer on disk. Pure, because "why did no
@@ -258,22 +315,27 @@ export function checkDue({ at, failedAt, now, first = false, force = false, ttlM
  *  `force` skips the window: the first call in this process, and an explicit
  *  "check now" from the UI. */
 async function latestOnNpm(name, now, force = false) {
-  const m = readMarker();
-  const first = !_askedThisProcess;
-  _askedThisProcess = true;
+  const m = readMarker(name);
+  const key = markerFileName(name);
+  const first = !_askedThisProcess.has(key);
+  _askedThisProcess.add(key);
   if (!checkDue({ at: m?.at, failedAt: m?.failedAt, now, first, force })) return m?.version ?? null;
-  if (_inflight) return _inflight;
-  _inflight = fetchLatest(name)
-    // Record the outcome, not just the moment. Only an answer stamps `at`; a
-    // failure takes the short retry window instead of the hour, and keeps the
-    // version we already knew rather than erasing it.
+  const pending = _inflight.get(key);
+  if (pending) return pending;
+  const run = fetchLatest(name)
+    // Record the outcome, not just the moment, and record it against THIS
+    // package: only an answer stamps `at`; a failure takes the short retry
+    // window instead of the hour, keeps the version we already knew rather
+    // than erasing it, and lands in this name's marker rather than spending
+    // another package's window on a lookup that was never about it.
     .then(({ ok, version }) => {
-      writeMarker(nextMarker({ prev: m, now, ok, version }));
+      writeMarker(name, nextMarker({ prev: m, now, ok, version }));
       return (ok ? version : null) ?? m?.version ?? null;
     })
     .catch(() => m?.version ?? null)
-    .finally(() => { _inflight = null; });
-  return _inflight;
+    .finally(() => { _inflight.delete(key); });
+  _inflight.set(key, run);
+  return run;
 }
 // ── the notice ───────────────────────────────────────────────────────────────
@@ -448,7 +510,7 @@ export async function versionReport({ running, pkgRoot, name = "agents-deck", no
     process.env.AGENTS_DECK_NO_UPDATE_CHECK === "1" ||
     process.env.AGENTS_DECK_NO_INSTALL === "1";
   const latest = skipRegistry ? null : await latestOnNpm(name, now, force);
-  const marker = skipRegistry ? null : readMarker();
+  const marker = skipRegistry ? null : readMarker(name);
   const blocked = upgradeBlock(pkgRoot);
   return {
     name,