@trygocode/notify 0.6.6 → 0.6.8

package/README.md

@@ -530,6 +530,35 @@ 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
+  ping labels the notification with your **workspace folder name** (e.g.
+  `Cursor · OpenHandApp — Agent finished`) instead of the git `owner/repo`.
+- **Fixed the `.cursor` project label.** When a stop hook ran without a piped
+  workspace path, the label derived from the hook's cwd (`~/.cursor`) and
+  rendered as “Cursor · .cursor” (the IDE name twice). `projectLabel` now walks
+  up past any leading-dot tooling dir (`.cursor`, `.git`, `.vscode`, …) and
+  stops at the home dir, so it never surfaces a tooling/home folder as the
+  project. When no real workspace folder is derivable it falls back to the git
+  repo name, and if even that is absent it shows the runtime alone
+  (`Cursor — Agent finished`) with no confusing suffix.
+
 ### 0.6.6
 
 - **Claude questions are now classified by notification type.** Setup installs

package/dist/src/claude.js

@@ -67,10 +67,9 @@ export const CLAUDE_HOOK_COMMANDS = {
     Notification: 'npx -y @trygocode/notify@latest on-notification --source claude_code --dedupe-key "$CLAUDE_SESSION_ID-notify" --quiet || true',
 };
 /**
- * Claude `Notification` types that genuinely require the user's attention.
- * Keep this matcher in lockstep with `CLAUDE_INPUT_NOTIFICATION_TYPES` in
- * `cli.ts`, which provides a defensive payload check for existing 0.6.5
- * installs whose matcher is still empty until setup is rerun.
+ * Claude `Notification` types that genuinely require the user's attention. The
+ * dispatcher independently applies the same values as a defensive payload check
+ * for existing 0.6.5 installs whose matcher stays empty until setup reruns.
  *
  * Deliberately excluded: `auth_success`, `elicitation_complete`,
  * `elicitation_response`, and `agent_completed`.

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/on_stop.js

@@ -26,6 +26,7 @@
 //
 // Zero runtime deps — Node built-ins only, matching the package's zero-dep rule.
 import path from "node:path";
+import os from "node:os";
 import { promises as fs } from "node:fs";
 import { createHash } from "node:crypto";
 import { resolveNotifySettings } from "./config.js";
@@ -223,42 +224,57 @@ function toPushSettings(settings) {
     };
 }
 /**
- * Resolve the `project` label for a per-turn ping (T-N4 / PRD §3.4). Prefers the
- * derived repo identity's `repo_label`; when that is blank — the repo-derive
- * threw (so `repo` is undefined) or a non-git cwd produced an empty label — falls
- * back to the cwd basename so the phone ALWAYS shows SOMETHING ("better-than-
- * nothing"). Returns `undefined` only when even the basename is empty (e.g. cwd
- * is the filesystem root), so the caller still omits the field gracefully.
+ * Resolve the `project` label for a per-turn ping (T-N4 / PRD §3.4).
+ *
+ * Preference order (user decision 2026-07-27):
+ *   1. The WORKSPACE FOLDER NAME — the cwd basename (e.g. "OpenHandApp"), which
+ *      matches what the user sees in the IDE title bar / file explorer. We walk
+ *      up past any leading-dot tooling dir (`.cursor`, `.git`, `.vscode`, …) so a
+ *      hook that runs inside a config dir still names the real project folder,
+ *      never "Cursor · .cursor". We also reject the user's HOME dir as a label
+ *      (it is never a meaningful project name).
+ *   2. The git REPO NAME — the last segment of `repo_label` (`owner/repo` →
+ *      `repo`), used only when the folder walk yields nothing usable. Also
+ *      dot-guarded (a `.cursor` repo label must never surface).
+ *   3. `undefined` — no project word at all, so the caller renders the runtime
+ *      alone ("Cursor — Agent finished"). This is the intentional fallback when
+ *      there is genuinely no workspace/repo signal (a rare IDE edge case); it is
+ *      strictly better than showing a confusing dir name.
  */
 export function projectLabel(repo, cwd) {
-    const label = repo?.repo_label?.trim();
-    // The user wants the plain project/folder name (e.g. "alerc8"), not the
-    // `owner/repo` form. `repo_label` from a git origin is `owner/repo`, and the
-    // `local:` fallback already strips the prefix in deriveRepoIdentity — so take
-    // the LAST path segment as the display name. Guard against a trailing slash.
-    if (label) {
-        const lastSeg = label.replace(/\/+$/, "").split("/").pop()?.trim();
-        if (lastSeg)
-            return lastSeg;
-        return label;
-    }
-    // Fall back to the cwd basename — but SKIP tool/config dot-dirs. The stop hook
-    // can run with a cwd inside `.cursor` (or `.git`, `.vscode`, …), whose basename
-    // would otherwise become the project name and render as "Cursor · .cursor"
-    // (the IDE name shown twice). Walk up past any leading-dot segment to the first
-    // real project folder so the label is the actual repo dir, not its tooling dir.
+    const home = path.resolve(os.homedir());
+    // Directories that are never a meaningful project name, even if we land on
+    // them while walking up: the home dir itself and its parent chain (`/Users`,
+    // `/home`, filesystem root). Reaching any of these means "no project folder".
+    const isNonProjectDir = (d) => d === home || home === d + path.sep || home.startsWith(d + path.sep) || path.dirname(d) === d;
+    // (1) Prefer the workspace FOLDER name — walk up past leading-dot tooling dirs
+    // (`.cursor`, `.git`, `.vscode`, …) to the first REAL project folder. We STOP
+    // if we reach the home dir or any ancestor of it (so a hook run with cwd =
+    // `~/.cursor` and no workspace path bails to runtime-only rather than climbing
+    // into "Users"/"home"/root).
     let dir = path.resolve(cwd);
-    for (let i = 0; i < 6; i++) {
+    for (let i = 0; i < 8; i++) {
+        if (isNonProjectDir(dir))
+            break; // home / its ancestors / root → no folder label
         const base = path.basename(dir).trim();
-        if (!base)
-            break; // reached filesystem root
-        if (!base.startsWith("."))
-            return base; // first non-dot folder wins
+        if (base && !base.startsWith(".")) {
+            return base; // first real (non-dot) project folder wins
+        }
         const parent = path.dirname(dir);
         if (parent === dir)
             break; // no more parents
         dir = parent;
     }
+    // (2) Fall back to the git REPO name (last segment of `owner/repo`), dot-guarded.
+    const label = repo?.repo_label?.trim();
+    if (label) {
+        const lastSeg = label.replace(/\/+$/, "").split("/").pop()?.trim();
+        if (lastSeg && !lastSeg.startsWith("."))
+            return lastSeg;
+        if (!lastSeg && !label.startsWith("."))
+            return label;
+    }
+    // (3) No usable signal → runtime-only (undefined omits the project word).
     return undefined;
 }
 /**

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.6";
+export const VERSION = "0.6.8";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trygocode/notify",
-  "version": "0.6.6",
+  "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",