@mohak34/opencode-notifier 0.1.32-beta.0 → 0.1.33

package/README.md

@@ -66,12 +66,12 @@ Create `~/.config/opencode/opencode-notifier.json` with the defaults:
     "minDuration": 0
   },
   "events": {
-    "permission": { "sound": true, "notification": true },
-    "complete": { "sound": true, "notification": true },
-    "subagent_complete": { "sound": false, "notification": false },
-    "error": { "sound": true, "notification": true },
-    "question": { "sound": true, "notification": true },
-    "user_cancelled": { "sound": false, "notification": false }
+    "permission": { "sound": true, "notification": true, "command": true },
+    "complete": { "sound": true, "notification": true, "command": true },
+    "subagent_complete": { "sound": false, "notification": false, "command": true },
+    "error": { "sound": true, "notification": true, "command": true },
+    "question": { "sound": true, "notification": true, "command": true },
+    "user_cancelled": { "sound": false, "notification": false, "command": true }
   },
   "messages": {
     "permission": "Session needs permission: {sessionTitle}",
@@ -123,7 +123,7 @@ Create `~/.config/opencode/opencode-notifier.json` with the defaults:
 - `showSessionTitle` - Include the session title in notification messages via `{sessionTitle}` placeholder (default: true)
 - `showIcon` - Show OpenCode icon, Windows/Linux only (default: true)
 - `suppressWhenFocused` - Skip notifications and sounds when the terminal is the active window (default: true). See [Focus detection](#focus-detection) for platform details
-- `notificationSystem` - macOS only: `"osascript"`, `"node-notifier"`, or `"ghostty"` (default: "osascript"). Use `"ghostty"` if you're running Ghostty terminal for native OSC 777 notifications
+- `notificationSystem` - macOS only: `"osascript"`, `"node-notifier"`, or `"ghostty"` (default: "osascript"). Use `"ghostty"` if you're running Ghostty terminal for native OSC 9 notifications
 - `linux.grouping` - Linux only: replace notifications in-place instead of stacking (default: false). Requires `notify-send` 0.8+
 
 ### Events
@@ -133,18 +133,20 @@ Control each event separately:
 ```json
 {
   "events": {
-    "permission": { "sound": true, "notification": true },
-    "complete": { "sound": true, "notification": true },
-    "subagent_complete": { "sound": false, "notification": false },
-    "error": { "sound": true, "notification": true },
-    "question": { "sound": true, "notification": true },
-    "user_cancelled": { "sound": false, "notification": false }
+    "permission": { "sound": true, "notification": true, "command": true },
+    "complete": { "sound": true, "notification": true, "command": true },
+    "subagent_complete": { "sound": false, "notification": false, "command": true },
+    "error": { "sound": true, "notification": true, "command": true },
+    "question": { "sound": true, "notification": true, "command": true },
+    "user_cancelled": { "sound": false, "notification": false, "command": true }
   }
 }
 ```
 
 `user_cancelled` fires when you press ESC to abort a session. It's silent by default so intentional cancellations don't trigger error alerts. Set `sound` or `notification` to `true` if you want confirmation when cancelling.
 
+The `command` property controls whether the custom command (see [Custom commands](#custom-commands)) runs for that event. Defaults to `true` for all events. Set it to `false` to suppress the command for specific events without disabling it globally.
+
 Or use true/false for both:
 
 ```json

package/dist/index.js

@@ -1,6 +1,6 @@
 // src/index.ts
 import { basename } from "path";
-import { readFileSync as readFileSync2, writeFileSync as writeFileSync2 } from "fs";
+import { readFileSync as readFileSync2, writeFileSync } from "fs";
 
 // src/config.ts
 import { readFileSync, existsSync } from "fs";
@@ -9,7 +9,8 @@ import { homedir } from "os";
 import { fileURLToPath } from "url";
 var DEFAULT_EVENT_CONFIG = {
   sound: true,
-  notification: true
+  notification: true,
+  command: true
 };
 var DEFAULT_CONFIG = {
   sound: true,
@@ -31,11 +32,11 @@ var DEFAULT_CONFIG = {
   events: {
     permission: { ...DEFAULT_EVENT_CONFIG },
     complete: { ...DEFAULT_EVENT_CONFIG },
-    subagent_complete: { sound: false, notification: false },
+    subagent_complete: { ...DEFAULT_EVENT_CONFIG, sound: false, notification: false },
     error: { ...DEFAULT_EVENT_CONFIG },
     question: { ...DEFAULT_EVENT_CONFIG },
     interrupted: { ...DEFAULT_EVENT_CONFIG },
-    user_cancelled: { sound: false, notification: false }
+    user_cancelled: { ...DEFAULT_EVENT_CONFIG, sound: false, notification: false }
   },
   messages: {
     permission: "Session needs permission: {sessionTitle}",
@@ -82,12 +83,14 @@ function parseEventConfig(userEvent, defaultConfig) {
   if (typeof userEvent === "boolean") {
     return {
       sound: userEvent,
-      notification: userEvent
+      notification: userEvent,
+      command: userEvent
     };
   }
   return {
     sound: userEvent.sound ?? defaultConfig.sound,
-    notification: userEvent.notification ?? defaultConfig.notification
+    notification: userEvent.notification ?? defaultConfig.notification,
+    command: userEvent.command ?? defaultConfig.command
   };
 }
 function parseVolume(value, defaultVolume) {
@@ -114,7 +117,8 @@ function loadConfig() {
     const globalNotification = userConfig.notification ?? DEFAULT_CONFIG.notification;
     const defaultWithGlobal = {
       sound: globalSound,
-      notification: globalNotification
+      notification: globalNotification,
+      command: true
     };
     const userCommand = userConfig.command ?? {};
     const commandArgs = Array.isArray(userCommand.args) ? userCommand.args.filter((arg) => typeof arg === "string") : undefined;
@@ -140,11 +144,11 @@ function loadConfig() {
       events: {
         permission: parseEventConfig(userConfig.events?.permission ?? userConfig.permission, defaultWithGlobal),
         complete: parseEventConfig(userConfig.events?.complete ?? userConfig.complete, defaultWithGlobal),
-        subagent_complete: parseEventConfig(userConfig.events?.subagent_complete ?? userConfig.subagent_complete, { sound: false, notification: false }),
+        subagent_complete: parseEventConfig(userConfig.events?.subagent_complete ?? userConfig.subagent_complete, { sound: false, notification: false, command: true }),
         error: parseEventConfig(userConfig.events?.error ?? userConfig.error, defaultWithGlobal),
         question: parseEventConfig(userConfig.events?.question ?? userConfig.question, defaultWithGlobal),
         interrupted: parseEventConfig(userConfig.events?.interrupted ?? userConfig.interrupted, defaultWithGlobal),
-        user_cancelled: parseEventConfig(userConfig.events?.user_cancelled ?? userConfig.user_cancelled, { sound: false, notification: false })
+        user_cancelled: parseEventConfig(userConfig.events?.user_cancelled ?? userConfig.user_cancelled, { sound: false, notification: false, command: true })
       },
       messages: {
         permission: userConfig.messages?.permission ?? DEFAULT_CONFIG.messages.permission,
@@ -184,6 +188,9 @@ function isEventSoundEnabled(config, event) {
 function isEventNotificationEnabled(config, event) {
   return config.events[event].notification;
 }
+function isEventCommandEnabled(config, event) {
+  return config.events[event].command;
+}
 function getMessage(config, event) {
   return config.messages[event];
 }
@@ -262,6 +269,7 @@ function detectNotifySendCapabilities() {
 function sendLinuxNotificationDirect(title, message, timeout, iconPath, grouping = true) {
   return new Promise((resolve) => {
     const args = [];
+    args.push("--app-name", "opencode");
     if (iconPath) {
       args.push("--icon", iconPath);
     }
@@ -336,7 +344,8 @@ async function sendNotification(title, message, timeout, iconPath, notificationS
       title,
       message,
       timeout,
-      icon: iconPath
+      icon: iconPath,
+      "app-name": "opencode"
     };
     platformNotifier.notify(notificationOptions, () => {
       resolve();
@@ -420,7 +429,7 @@ async function playOnLinux(soundPath, volume) {
   const players = [
     { command: "paplay", args: [`--volume=${pulseVolume}`, soundPath] },
     { command: "aplay", args: [soundPath] },
-    { command: "mpv", args: ["--no-video", "--no-terminal", `--volume=${percentVolume}`, soundPath] },
+    { command: "mpv", args: ["--no-video", "--no-terminal", "--script-opts=autoload-disabled=yes", `--volume=${percentVolume}`, soundPath] },
     { command: "ffplay", args: ["-nodisp", "-autoexit", "-loglevel", "quiet", "-volume", `${percentVolume}`, soundPath] }
   ];
   for (const player of players) {
@@ -630,7 +639,7 @@ function loadTurnCount() {
 }
 function saveTurnCount(count) {
   try {
-    writeFileSync2(getStatePath(), JSON.stringify({ turn: count }));
+    writeFileSync(getStatePath(), JSON.stringify({ turn: count }));
   } catch {}
 }
 function incrementTurnCount() {
@@ -697,7 +706,7 @@ async function handleEvent(config, eventType, projectName, elapsedSeconds, sessi
     promises.push(playSound(eventType, customSoundPath, soundVolume));
   }
   const minDuration = config.command?.minDuration;
-  const shouldSkipCommand = typeof minDuration === "number" && Number.isFinite(minDuration) && minDuration > 0 && typeof elapsedSeconds === "number" && Number.isFinite(elapsedSeconds) && elapsedSeconds < minDuration;
+  const shouldSkipCommand = !isEventCommandEnabled(config, eventType) || typeof minDuration === "number" && Number.isFinite(minDuration) && minDuration > 0 && typeof elapsedSeconds === "number" && Number.isFinite(elapsedSeconds) && elapsedSeconds < minDuration;
   if (!shouldSkipCommand) {
     runCommand2(config, eventType, message, sessionTitle, projectName, timestamp, turn);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mohak34/opencode-notifier",
-  "version": "0.1.32-beta.0",
+  "version": "0.1.33",
   "description": "OpenCode plugin that sends system notifications and plays sounds when permission is needed, generation completes, or errors occur",
   "author": "mohak34",
   "license": "MIT",