agent-dag 1.33.64 → 1.33.66

package/dist/web/index.html

@@ -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-B5DB7xOD.js"></script>
+    <script type="module" crossorigin src="/assets/index-D6KFW6DO.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-CF2yG9oG.css">
   </head>
   <body>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-dag",
-  "version": "1.33.64",
+  "version": "1.33.66",
   "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/installer.mjs

@@ -4,7 +4,7 @@
 // Both providers share the discovery dir at <claude config dir>/agent-dag/ so a
 // single running server can receive events from either CLI. Re-runs are safe;
 // entries are tagged with __agent-dag and de-duped.
-import { readFile, writeFile, mkdir, copyFile, unlink, rename, open, stat, chmod } from "node:fs/promises";
+import { readFile, writeFile, mkdir, unlink, rename, open, stat, chmod } from "node:fs/promises";
 import { existsSync } from "node:fs";
 import { homedir } from "node:os";
 import { join, resolve, dirname } from "node:path";
@@ -203,11 +203,33 @@ async function writeFileAtomic(target, text) {
   }
 }
 
+/**
+ * Install one of the packaged hook scripts without ever exposing a partial one.
+ *
+ * copyFile truncates the destination and then fills it, and the destination here
+ * is a script every live Claude Code session runs on each tool call. Starting the
+ * deck while sessions are open — the normal way this is used — puts a hook
+ * invocation inside that window sooner or later, and what it executes is an
+ * empty or half-written program: a dropped event at best, a SyntaxError in the
+ * user's session at worst. Renaming a finished copy over the name closes it, so
+ * a session opens either the old script or the new one and both are whole.
+ *
+ * Re-installs are the common case and almost always produce the same bytes, so
+ * identical content skips the write and the file is not replaced at all.
+ */
+async function installScript(src, dst) {
+  const text = await readFile(src, "utf8");
+  const current = await readFile(dst, "utf8").catch(() => null);
+  if (current === text) return false;
+  await writeFileAtomic(dst, text);
+  return true;
+}
+
 async function installHookScript(installDir) {
   await ensureDir(installDir);
   const src = join(PKG_ROOT, "hook", "hook.js");
   const dst = join(installDir, "hook.js");
-  await copyFile(src, dst);
+  await installScript(src, dst);
   return dst;
 }
 
@@ -300,5 +322,7 @@ export { AGENT_DAG_DIR, CLAUDE_DIR, CODEX_DIR, CLAUDE_EVENTS, CODEX_EVENTS };
 // Exported for the other modules that rewrite settings.json — the sound toggle
 // today. Every one of them needs the same two guarantees: a file we cannot
 // parse is never treated as an empty one, and the replacement is a single
-// rename rather than a truncate a reader can land inside.
-export { readSettingsForWrite, writeFileAtomic };
+// rename rather than a truncate a reader can land inside. installScript carries
+// the same guarantee to the hook scripts themselves, which are the files live
+// sessions are actually executing.
+export { readSettingsForWrite, writeFileAtomic, installScript };

package/src/server/sound-hook.mjs

@@ -10,13 +10,13 @@
 // Only ever touches its own entry, tagged `__agent-dag-sound`. Hooks the user
 // wrote themselves are left exactly as found — including the platform-specific
 // ones this replaces, which are reported rather than deleted.
-import { readFile, writeFile, copyFile, mkdir } from "node:fs/promises";
+import { readFile, writeFile, mkdir } from "node:fs/promises";
 import { existsSync } from "node:fs";
 import { join, dirname } from "node:path";
 import { homedir } from "node:os";
 import { fileURLToPath } from "node:url";
 import { claudeConfigDir } from "./claude-dir.mjs";
-import { readSettingsForWrite, writeFileAtomic } from "./installer.mjs";
+import { readSettingsForWrite, writeFileAtomic, installScript } from "./installer.mjs";
 
 const PKG_ROOT      = join(dirname(fileURLToPath(import.meta.url)), "..", "..");
 const CLAUDE_DIR    = claudeConfigDir();
@@ -229,7 +229,10 @@ export async function setSoundHook(enabled) {
 
   if (enabled) {
     if (!existsSync(INSTALL_DIR)) await mkdir(INSTALL_DIR, { recursive: true });
-    await copyFile(join(PKG_ROOT, "hook", "notify.js"), NOTIFY_PATH);
+    // Same reason the event forwarder is installed this way: the Stop hook fires
+    // notify.js from sessions that are already running, and toggling the sound
+    // on must not leave one of them executing a half-copied file.
+    await installScript(join(PKG_ROOT, "hook", "notify.js"), NOTIFY_PATH);
     others.push({
       [MARK]: true,
       hooks: [{