@lawaty/lacode 1.0.1 → 1.0.3

package/CHANGELOG.md

@@ -7,6 +7,38 @@ All notable changes to LaCode are recorded here. The format follows
 Versions come from git release commits; the npm version is bumped in the same
 commit as the changelog entry it describes.
 
+## [1.0.3] — one notification even when the plugin is loaded twice
+
+### Fixed
+
+- **A question or permission popup is raised once when two LaCode instances share
+  a process.** opencode auto-discovers every file under
+  `~/.config/opencode/plugins/`, so a host that also lists `@lawaty/lacode` in
+  `opencode.jsonc` loaded two copies. `done` was already deduplicated by the
+  completion marker, but the notification claim treated any claim holding our
+  own pid as ours — indistinguishable between two instances in one process. The
+  claim now carries a per-instance token, so the second instance loses while the
+  same instance may still raise a new prompt.
+
+### Changed
+
+- **Publishing syncs the published version to every host.** `postpublish` runs
+  `scripts/sync-hosts.mjs`: it installs `@lawaty/lacode@<version>`, normalises
+  each host's config to a single LaCode entry (preserving its options), disables
+  stale auto-discovered shims, and warns when a host cannot run the helper.
+
+## [1.0.2] — the permission chime plays when the popup appears
+
+### Fixed
+
+- **An actionable notification no longer delays its sound until it is answered.**
+  `sendLocal()` played the chime after `notify-send` returned, but an actionable
+  send (`--action` implies `--wait`) blocks until the user clicks a button or
+  dismisses the popup. The chime therefore played at click time — and never at
+  all when the popup was left alone — which made permission sounds look
+  intermittent. The sound now starts before the notifier runs, and a test pins
+  the order with a blocking fake notifier.
+
 ## [1.0.1] — notification sounds you can hear, and a title that names the workspace
 
 ### Fixed

package/dist/index.js

@@ -3662,6 +3662,7 @@ function scrubSlotNames(text) {
 
 // src/plugins/notify.ts
 import { spawn as spawn2 } from "node:child_process";
+import { randomUUID as randomUUID2 } from "node:crypto";
 import { existsSync as existsSync3 } from "node:fs";
 import * as path3 from "node:path";
 
@@ -3857,7 +3858,7 @@ var isPidAlive2 = (pid) => {
   }
 };
 var claimFile = (sessionID, kind) => join7(notificationStateDir(), `claim.${safeCorrelationKey(`${sessionID}.${kind}`)}`);
-var claimNotification = (sessionID, kind) => {
+var claimNotification = (sessionID, kind, instanceId = "") => {
   const path4 = claimFile(sessionID, kind);
   try {
     mkdirSync5(notificationStateDir(), { recursive: true });
@@ -3868,7 +3869,7 @@ var claimNotification = (sessionID, kind) => {
     try {
       const fd = openSync2(path4, "wx");
       try {
-        writeSync2(fd, String(process.pid));
+        writeSync2(fd, `${process.pid} ${instanceId} ${kind}`);
       } finally {
         closeSync2(fd);
       }
@@ -3888,13 +3889,21 @@ var claimNotification = (sessionID, kind) => {
     } catch (error) {
       return error.code === "ENOENT" ? "gone" : "unknown";
     }
+    const fields = raw.split(/\s+/).filter((entry) => entry.length > 0);
     let holder;
     const empty = raw === "";
     if (!empty) {
-      const parsed = Number.parseInt(raw, 10);
+      const parsed = Number.parseInt(fields[0] ?? "", 10);
       holder = Number.isFinite(parsed) && parsed > 0 ? parsed : void 0;
     }
-    if (holder === process.pid) return "ours";
+    if (holder === process.pid) {
+      const hasToken = fields.length >= 3;
+      const fileToken = hasToken ? fields[1] ?? "" : "";
+      if (hasToken && instanceId !== "" && fileToken !== instanceId) {
+        return "held";
+      }
+      return "ours";
+    }
     let age = Number.POSITIVE_INFINITY;
     try {
       age = Date.now() - statSync5(path4).mtimeMs;
@@ -4050,6 +4059,7 @@ var notifyHooks = (async ({ client, directory }, options, deps = {}) => {
   const soundEnv = (process.env[SOUND_ENV] ?? "").trim().toLowerCase();
   const soundOn = !SOUND_OFF_VALUES.includes(soundEnv) && opts.sound !== false;
   const notifier = new Notifier(COOLDOWN_MS2);
+  const instanceId = randomUUID2();
   const active = /* @__PURE__ */ new Set();
   const titles = /* @__PURE__ */ new Map();
   const pending = /* @__PURE__ */ new Set();
@@ -4345,7 +4355,7 @@ var notifyHooks = (async ({ client, directory }, options, deps = {}) => {
           void log("debug", "suppressed: completion already announced", { sessionID });
           return;
         }
-      } else if (!claimNotification(sessionID, kind)) {
+      } else if (!claimNotification(sessionID, kind, instanceId)) {
         void log("debug", "suppressed: another process is raising this", { sessionID, kind });
         return;
       } else if (outcome === "error") {

package/dist/lacode.js

@@ -369,8 +369,8 @@ function sendLocal(request, platform = process.platform) {
     // Never auto-dismissed: the notification waits for the user to close it.
     [title, body, APP_NAME_FLAG, `--urgency=${normalizeUrgency(request.urgency)}`, `--icon=${normalizeIcon(request.icon)}`, NEVER_EXPIRE_FLAG, ...trackable ? ["--print-id"] : [], ...actionArgs]
   );
-  const result = run(CHANNEL_BINARY[channel], args, actionable ? ACTION_WAIT_MS : 5e3);
   playSound();
+  const result = run(CHANNEL_BINARY[channel], args, actionable ? ACTION_WAIT_MS : 5e3);
   const printed = trackable ? splitNotifyOutput(result.stdout) : { id: "", action: result.stdout.trim() };
   const action = actionable ? printed.action : "";
   if (correlationKey !== void 0 && printed.id) recordNotificationId(correlationKey, printed.id);

package/dist/tunnel.js

@@ -362,8 +362,8 @@ function sendLocal(request, platform = process.platform) {
     // Never auto-dismissed: the notification waits for the user to close it.
     [title, body, APP_NAME_FLAG, `--urgency=${normalizeUrgency(request.urgency)}`, `--icon=${normalizeIcon(request.icon)}`, NEVER_EXPIRE_FLAG, ...trackable ? ["--print-id"] : [], ...actionArgs]
   );
-  const result = run(CHANNEL_BINARY[channel], args, actionable ? ACTION_WAIT_MS : 5e3);
   playSound();
+  const result = run(CHANNEL_BINARY[channel], args, actionable ? ACTION_WAIT_MS : 5e3);
   const printed = trackable ? splitNotifyOutput(result.stdout) : { id: "", action: result.stdout.trim() };
   const action = actionable ? printed.action : "";
   if (correlationKey !== void 0 && printed.id) recordNotificationId(correlationKey, printed.id);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lawaty/lacode",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "Ultimate opencode setup for Go users — cost-controlled pooled subagents, hang recovery, and notifications for opencode.",
   "license": "MIT",
   "type": "module",
@@ -52,6 +52,7 @@
   "scripts": {
     "build": "node scripts/build.mjs",
     "prepack": "npm run build",
+    "postpublish": "node scripts/sync-hosts.mjs",
     "test": "node --test tests/",
     "benchmark": "node tests/benchmark/run.mjs"
   },