@lawaty/lacode 1.0.0 → 1.0.2

package/CHANGELOG.md

@@ -7,6 +7,45 @@ 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.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
+
+- **Notification sounds play again.** The plugin preferred the bash `oc-notify`
+  helper, which discards the sound field, so a chime was never played on that
+  path; it now prefers the lacode CLI, whose `sendLocal` plays
+  `sounds/<outcome>.wav`. The bash helper stays as a last-resort fallback.
+- **The four chimes survive a suspended sink.** Each WAV was 0.20–0.32 s at 42%
+  full-scale, which a suspended Bluetooth output swallowed whole while the link
+  woke. They are now 1.0–1.46 s at 90% peak, and a test pins both the duration
+  and the peak so the regression cannot return.
+- **A popup no longer fades on its own.** `interacted` is scoped to the popup it
+  was set on: a new popup starts un-interacted and auto-dismisses only when the
+  user interacts in that session while it is live. Previously every popup in a
+  session the user had ever typed in closed after five seconds.
+- **The published `bin` entry uses the `./dist/…` form** the packaging check
+  requires, so the release tarball passes its own adoption test.
+
+### Changed
+
+- **Notification titles name the workspace as well as the session.** The title
+  is now `{workspace} - {session}`, where the workspace is the basename of the
+  session's directory, falling back to the plugin directory. The popup header
+  stays `LaCode`. Internal cartographer sessions are still suppressed, because
+  the check runs on the session title before the prefix is added.
+
 ## [1.0.0] — images, ordered slot fill, and a limit that fails over
 
 ### Added

package/dist/index.js

@@ -3668,7 +3668,7 @@ import * as path3 from "node:path";
 // src/lib/notify.ts
 import { closeSync as closeSync2, mkdirSync as mkdirSync5, openSync as openSync2, readFileSync as readFileSync7, renameSync as renameSync4, statSync as statSync5, unlinkSync as unlinkSync4, writeFileSync as writeFileSync5, writeSync as writeSync2 } from "node:fs";
 import { homedir as homedir5 } from "node:os";
-import { join as join7 } from "node:path";
+import { basename as basename3, join as join7 } from "node:path";
 var INTERNAL_TITLE_PREFIXES = [
   // context-autoupdate spawns the cartographer through the session API, with no
   // parentID, so it looks exactly like a root session going idle. Notifying for
@@ -3748,8 +3748,8 @@ var shouldNotify = (input) => {
   const { info, outcome, detail } = input;
   if (!info?.id) return { notify: false, reason: "no session info" };
   if (info.parentID) return { notify: false, reason: "subagent" };
-  const title = safeTitle(info.title);
-  if (INTERNAL_TITLE_PREFIXES.some((prefix) => title.startsWith(prefix))) {
+  const sessionTitle = safeTitle(info.title);
+  if (INTERNAL_TITLE_PREFIXES.some((prefix) => sessionTitle.startsWith(prefix))) {
     return { notify: false, reason: "internal session" };
   }
   if (info.time?.compacting) return { notify: false, reason: "compacting" };
@@ -3759,12 +3759,10 @@ var shouldNotify = (input) => {
   if (outcome === "done" && input.awaitingResponse) {
     return { notify: false, reason: "awaiting your response" };
   }
+  const workspace = safeTitle(info.directory ? basename3(info.directory) : "", "");
+  const title = workspace ? safeTitle(`${workspace} - ${sessionTitle}`, sessionTitle) : sessionTitle;
   const notification = {
     notify: true,
-    // The session's name on its own, with nothing in front of it. The popup's
-    // header is the notification's APP name, and that is "LaCode" -- so a
-    // prefix here would only repeat the app, and it would repeat it wrongly:
-    // the app is not "opencode".
     title,
     message: notifyMessage(outcome, detail),
     urgency: outcome === "error" ? "critical" : "normal"
@@ -4037,10 +4035,10 @@ var findHelper = () => {
   const candidates = [
     process.env.OC_NOTIFY_HELPER,
     path3.join(home, "bin", "lacode"),
-    path3.join(home, "bin", "oc-notify"),
-    path3.join(PACKAGE_ROOT, "bin", "oc-notify"),
     path3.join(PACKAGE_ROOT, "dist", "lacode.js"),
-    path3.join(PACKAGE_ROOT, "bin", "lacode")
+    path3.join(PACKAGE_ROOT, "bin", "lacode"),
+    path3.join(home, "bin", "oc-notify"),
+    path3.join(PACKAGE_ROOT, "bin", "oc-notify")
   ].filter((candidate) => typeof candidate === "string" && candidate.length > 0);
   for (const candidate of candidates) if (existsSync3(candidate)) return candidate;
   return void 0;
@@ -4316,7 +4314,7 @@ var notifyHooks = (async ({ client, directory }, options, deps = {}) => {
         void log("debug", "session info unavailable; not notifying", { sessionID });
         return;
       }
-      const titled = info.title ? info : { ...info, title: titles.get(sessionID) };
+      const titled = { ...info, title: info.title ?? titles.get(sessionID), directory: info.directory ?? directory };
       const isRemote = Boolean(process.env.SSH_CLIENT || process.env.SSH_TTY);
       const decision = shouldNotify({
         info: titled,
@@ -4355,8 +4353,7 @@ var notifyHooks = (async ({ client, directory }, options, deps = {}) => {
       }
       const message = isRemote ? `${decision.message} (remote)` : decision.message;
       const sound = soundOn && NOTIFY_OUTCOMES.includes(outcome) ? outcome : "none";
-      const previous = live.get(sessionID);
-      const popup = { raisedAt: Date.now(), interacted: previous?.interacted ?? false };
+      const popup = { raisedAt: Date.now(), interacted: false };
       rememberLive(sessionID, popup);
       if (popup.interacted) scheduleDismiss(sessionID);
       let sent = { ok: false };

package/dist/lacode.js

@@ -14,7 +14,7 @@ import { join as join4 } from "node:path";
 // src/lib/notify.ts
 import { closeSync, mkdirSync as mkdirSync2, openSync, readFileSync as readFileSync3, renameSync as renameSync2, statSync as statSync2, unlinkSync as unlinkSync2, writeFileSync as writeFileSync2, writeSync } from "node:fs";
 import { homedir as homedir2 } from "node:os";
-import { join as join3 } from "node:path";
+import { basename, join as join3 } from "node:path";
 
 // src/lib/pool.ts
 import { existsSync, mkdirSync, readdirSync, readFileSync as readFileSync2, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
@@ -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

@@ -7,7 +7,7 @@ import { join as join4 } from "node:path";
 // src/lib/notify.ts
 import { closeSync, mkdirSync as mkdirSync2, openSync, readFileSync as readFileSync3, renameSync as renameSync2, statSync as statSync2, unlinkSync as unlinkSync2, writeFileSync as writeFileSync2, writeSync } from "node:fs";
 import { homedir as homedir2 } from "node:os";
-import { join as join3 } from "node:path";
+import { basename, join as join3 } from "node:path";
 
 // src/lib/pool.ts
 import { existsSync, mkdirSync, readdirSync, readFileSync as readFileSync2, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
@@ -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.0",
+  "version": "1.0.2",
   "description": "Ultimate opencode setup for Go users — cost-controlled pooled subagents, hang recovery, and notifications for opencode.",
   "license": "MIT",
   "type": "module",
@@ -38,7 +38,7 @@
     "./package.json": "./package.json"
   },
   "bin": {
-    "lacode": "dist/lacode.js"
+    "lacode": "./dist/lacode.js"
   },
   "files": [
     "dist",