@trygocode/notify 0.6.7 → 0.6.8

package/README.md

@@ -530,6 +530,21 @@ npm publish, real-device E2E), see
 
 ## Changelog
 
+### 0.6.8
+
+- **Fixed: Claude questions inside Cursor now notify.** When Claude Code runs
+  inside Cursor and asks a question (permission prompt, idle/elicitation, agent
+  needs input), the notification was being **suppressed** because the imported
+  Claude `Notification` payload carries `cursor_version`. That suppression is
+  correct for *completions* (Cursor's own `stop` hook already fires, so the
+  imported Claude `Stop` would double-notify) — but **wrong for questions**:
+  Cursor's native question hook (`AskQuestion` via `postToolUse`) is broken
+  upstream and Cursor has no `Notification` event, so the imported Claude
+  `Notification` is the *only* question signal. `on-notification` now always
+  forwards a genuine input-requiring notification, even when Cursor imported it.
+  The non-input `notification_type` filter and per-question `--dedupe-key` still
+  apply, so no extra noise or duplicates.
+
 ### 0.6.7
 
 - **Notification project name now matches your IDE.** The per-turn completion

package/dist/src/cli.js

@@ -20,7 +20,7 @@ import { serveStdio } from "./mcp.js";
 import { setup } from "./setup.js";
 import { uninstall } from "./uninstall.js";
 import { cmdConfig } from "./config.js";
-import { onStop, isClaudeHookImportedByCursor, parseCursorStopStatus, cursorStopStatusToKind, clickTarget, projectLabel, } from "./on_stop.js";
+import { onStop, parseCursorStopStatus, cursorStopStatusToKind, clickTarget, projectLabel, } from "./on_stop.js";
 import { notifyDesktop, requestDesktopPermission, desktopDisabledByEnv, } from "./desktop_notify.js";
 import { resolveNotifySettings } from "./config.js";
 import { deriveRepoIdentity } from "./repo_key.js";
@@ -795,8 +795,23 @@ export async function cmdOnNotification(args, deps = {}) {
     const hookStdin = deps.hookStdin !== undefined
         ? deps.hookStdin
         : await (deps.readStdin ?? readStdinIfPipe)();
-    if (isClaudeHookImportedByCursor(source, hookStdin))
-        return 0;
+    // NOTE (2026-07-28): we DELIBERATELY do NOT suppress Cursor-imported Claude
+    // hooks here (unlike the `on-stop`/completion path). The `cursor_version`
+    // suppression exists to avoid DOUBLE notifications when BOTH a native Cursor
+    // hook AND the imported Claude hook fire for the same turn. That double-fire
+    // only happens for COMPLETION: Cursor's native `stop` hook works, so its
+    // imported Claude `Stop` would duplicate it → suppress.
+    //
+    // For QUESTIONS there is no competing native hook to dedupe against: Cursor's
+    // own question signal (the `AskQuestion` `postToolUse` hook) is BROKEN upstream
+    // (Cursor docs: "postToolUse ... excluding the AskQuestion tool, which is a
+    // known bug") and Cursor has NO `Notification` event at all. So when Claude
+    // runs inside Cursor and asks a question, the imported Claude `Notification`
+    // hook is the ONLY signal we get. Suppressing it (the old behaviour) silently
+    // dropped every in-Cursor Claude question — the exact "questions don't notify"
+    // regression. We now always forward a genuine input-requiring notification,
+    // regardless of whether Cursor imported it. The per-turn `--dedupe-key` still
+    // coalesces any accidental repeat of the SAME question.
     const notificationType = parseClaudeNotificationType(hookStdin);
     if (notificationType !== undefined &&
         !CLAUDE_INPUT_NOTIFICATION_TYPES.has(notificationType)) {

package/dist/src/version.js

@@ -1,2 +1,2 @@
 // Single source of truth for the CLI version. Keep in sync with package.json.
-export const VERSION = "0.6.7";
+export const VERSION = "0.6.8";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trygocode/notify",
-  "version": "0.6.7",
+  "version": "0.6.8",
   "description": "Free phone + branded desktop notifications for any coding agent (Cursor, Claude Code, OpenCode, Ralph/Homer) via the GoCode app.",
   "license": "MIT",
   "type": "module",