@trygocode/notify 0.3.1 → 0.3.3

package/dist/src/claude.js

@@ -56,9 +56,14 @@ export const MCP_SERVER_ENTRY = {
  * We intentionally register ONLY `Stop` + `Notification`. `SubagentStop` is NOT
  * here on purpose — see {@link SUBAGENT_STOP_EVENT}.
  */
+// `@latest` is REQUIRED: bare `npx -y @trygocode/notify` caches the
+// first-resolved version forever and never auto-updates (npm/cli#6664). Pinning
+// `@latest` makes every hook fire the newest published version, so users get our
+// fixes without ever clearing their npx cache by hand. Matchers below key on the
+// `@trygocode/notify` substring, which `@latest` preserves.
 export const CLAUDE_HOOK_COMMANDS = {
-    Stop: 'npx -y @trygocode/notify on-stop --source claude_code --dedupe-key "$CLAUDE_SESSION_ID-stop" || true',
-    Notification: 'npx -y @trygocode/notify send --kind awaiting_input --source claude_code --title "Agent needs you" --dedupe-key "$CLAUDE_SESSION_ID-notify" || true',
+    Stop: 'npx -y @trygocode/notify@latest on-stop --source claude_code --dedupe-key "$CLAUDE_SESSION_ID-stop" || true',
+    Notification: 'npx -y @trygocode/notify@latest send --kind awaiting_input --source claude_code --title "Agent needs you" --dedupe-key "$CLAUDE_SESSION_ID-notify" || true',
 };
 /**
  * The Claude event we DELIBERATELY do not register and actively scrub on every

package/dist/src/cursor.js

@@ -48,7 +48,15 @@ export const MCP_SERVER_ENTRY = {
  * both old and new commands contain the {@link HOOK_MARKERS} tokens, so an old
  * install still upgrades idempotently and uninstalls cleanly (PRD §2.2).
  */
-export const CURSOR_STOP_COMMAND = "npx -y @trygocode/notify on-stop --source cursor --dedupe-key cursor-stop || true";
+// `@latest` is REQUIRED, not cosmetic: bare `npx -y @trygocode/notify` caches the
+// first-resolved version FOREVER and never picks up a newer publish (documented
+// npx behaviour — npm/cli#6664; the whole `clear-npx-cache` package exists for
+// this). Without `@latest`, a user who installed an old version would keep
+// running it after we ship a fix, with no way to update short of wiping
+// ~/.npm/_npx by hand. Pinning `@latest` forces npx to resolve the newest
+// version on every turn, so users self-update seamlessly. (Matchers below key on
+// the `@trygocode/notify` substring, which `@latest` preserves.)
+export const CURSOR_STOP_COMMAND = "npx -y @trygocode/notify@latest on-stop --source cursor --dedupe-key cursor-stop || true";
 /**
  * Substrings that together identify a `stop` hook entry as OURS. Used for
  * idempotent merge (replace, don't duplicate) and for surgical uninstall (remove

package/dist/src/on_stop.js

@@ -94,8 +94,16 @@ function toPushSettings(settings) {
  */
 export function projectLabel(repo, cwd) {
     const label = repo?.repo_label?.trim();
-    if (label)
+    // 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"
@@ -115,6 +123,66 @@ export function projectLabel(repo, cwd) {
     }
     return undefined;
 }
+/**
+ * The project root the stop hook should reason about. Cursor (and Claude) run
+ * stop hooks with `process.cwd()` set to the IDE's hook/config dir (e.g.
+ * `~/.cursor`), NOT the user's open project — so deriving the repo from the bare
+ * cwd yields ".cursor" instead of the real repo (the bug in the 2026-06-12
+ * screenshot). The hook stdin, however, carries the real workspace root. Prefer
+ * it (mirrors what `@trygocode/sync` already does), falling back to the cwd only
+ * when the stdin omits it.
+ *
+ * Accepts the same field spellings sync consumes:
+ * `workspace_path` / `workspaceRoots[0]` / `workspace_roots[0]` / `cwd` /
+ * `workspaceFolders[0].path|uri`. Returns undefined when nothing usable is
+ * present, so the caller keeps the process cwd.
+ */
+export function workspaceRootFromHookStdin(hookStdin) {
+    if (!hookStdin || hookStdin.trim() === "")
+        return undefined;
+    let p;
+    try {
+        p = JSON.parse(hookStdin);
+    }
+    catch {
+        return undefined;
+    }
+    const str = (v) => typeof v === "string" && v.trim() !== "" ? v.trim() : undefined;
+    const firstOf = (v) => {
+        if (Array.isArray(v) && v.length > 0) {
+            const f = v[0];
+            if (typeof f === "string")
+                return str(f);
+            if (f && typeof f === "object") {
+                const o = f;
+                // VS Code/Cursor sometimes use {path} or {uri: "file:///…"}.
+                return str(o.path) ?? str(o.fsPath) ?? fromFileUri(str(o.uri));
+            }
+        }
+        return undefined;
+    };
+    return (str(p.workspace_path) ??
+        str(p.workspacePath) ??
+        firstOf(p.workspaceRoots) ??
+        firstOf(p.workspace_roots) ??
+        firstOf(p.workspaceFolders) ??
+        str(p.cwd) ??
+        undefined);
+}
+/** Convert a `file:///abs/path` URI to a plain path; passthrough otherwise. */
+function fromFileUri(uri) {
+    if (!uri)
+        return undefined;
+    if (uri.startsWith("file://")) {
+        try {
+            return decodeURIComponent(new URL(uri).pathname) || undefined;
+        }
+        catch {
+            return undefined;
+        }
+    }
+    return uri;
+}
 /**
  * Derive the SAME stable `external_chat_id` that `@trygocode/sync` assigns to a
  * synced transcript, so a per-turn notification can deep-link straight to that
@@ -170,7 +238,17 @@ export function ideChatIdFromHookStdin(source, cwd, hookStdin) {
     const ideSessionId = str(p.conversation_id, p.conversationId, p.session_id, p.sessionId, sessionFromPath);
     if (!ideSessionId)
         return undefined;
-    const workspacePath = str(p.workspace_path, Array.isArray(p.workspaceRoots) ? p.workspaceRoots[0] : undefined, p.cwd, cwd);
+    // CRITICAL parity: the deep-link id hashes the workspace, and gocode-sync hashes
+    // the SAME workspace when it stores the chat — they MUST agree or a tapped push
+    // hits "Chat not found". gocode-sync resolves the workspace from Cursor's REAL
+    // field `workspace_roots` (snake_case), so we MUST read it here too. Reuse
+    // workspaceRootFromHookStdin (which already prefers workspace_roots and every
+    // spelling) so the two packages can never drift. Fall back to camelCase / cwd
+    // for older Cursor builds, then the process cwd.
+    const workspacePath = workspaceRootFromHookStdin(hookStdin) ??
+        str(p.workspace_path, Array.isArray(p.workspace_roots)
+            ? p.workspace_roots[0]
+            : undefined, Array.isArray(p.workspaceRoots) ? p.workspaceRoots[0] : undefined, p.cwd, cwd);
     if (!workspacePath)
         return undefined;
     // The capture side stores `source: "cursor" | "claude_code"`; the hook passes
@@ -211,6 +289,49 @@ export async function chatTitleFromHookStdin(hookStdin) {
     catch {
         return undefined;
     }
+    // Prefer the IDE's OWN assigned chat name when the transcript carries one, so
+    // the push names the chat exactly as the user sees it in the sidebar (rather
+    // than a synthesized first-line title). Verified empirically (2026-06-12):
+    //   - Claude Code writes `{"type":"ai-title","aiTitle":"<sidebar name>"}`
+    //     records, refined over the chat's life — so take the LAST one.
+    //   - Cursor agent transcripts carry NO title record (only role/message), so
+    //     this scan no-ops for Cursor and we fall through to synthesis below.
+    // Defensive: also accept a legacy Claude `{"type":"summary","summary":"…"}`
+    // record so an older/newer format degrades gracefully instead of throwing.
+    {
+        let assigned;
+        for (const line of raw.split("\n")) {
+            const t = line.trim();
+            if (!t || (!t.includes("ai-title") && !t.includes("summary")))
+                continue;
+            let rec;
+            try {
+                rec = JSON.parse(t);
+            }
+            catch {
+                continue;
+            }
+            const type = String(rec.type ?? "");
+            let cand;
+            if (type === "ai-title")
+                cand = rec.aiTitle ?? rec.ai_title;
+            else if (type === "summary")
+                cand = rec.summary;
+            if (typeof cand === "string" && cand.trim() !== "") {
+                assigned = cand.trim(); // keep scanning → last assigned title wins
+            }
+        }
+        if (assigned) {
+            if (assigned.length > 80)
+                assigned = assigned.slice(0, 79).trimEnd() + "…";
+            return assigned;
+        }
+    }
+    // Synthesis fallback (no IDE-assigned title): name the chat after the LAST
+    // user prompt — the one that JUST finished — not the first prompt of the
+    // conversation. This reminds the user where they left off (per 2026-06-12
+    // request). We keep the last good title and return it after the full scan.
+    let lastTitle;
     for (const line of raw.split("\n")) {
         const t = line.trim();
         if (!t)
@@ -247,6 +368,7 @@ export async function chatTitleFromHookStdin(hookStdin) {
         content = content
             .replace(/<timestamp(?:\s[^>]*)?>[\s\S]*?<\/timestamp>/gi, "")
             .replace(/<image_files(?:\s[^>]*)?>[\s\S]*?<\/image_files>/gi, "")
+            .replace(/<open_and_recently_viewed_files(?:\s[^>]*)?>[\s\S]*?<\/open_and_recently_viewed_files>/gi, "")
             .replace(/\[Image\](?!\()\s*/g, "")
             .replace(/<\/?user_query(?:\s[^>]*)?>/gi, "");
         const firstLine = content
@@ -260,9 +382,9 @@ export async function chatTitleFromHookStdin(hookStdin) {
             continue;
         if (title.length > 80)
             title = title.slice(0, 79).trimEnd() + "…";
-        return title;
+        lastTitle = title; // keep scanning → LAST user prompt wins
     }
-    return undefined;
+    return lastTitle;
 }
 /**
  * Env var an Autopilot (Ralph/Homer) loop exports to mark that IT owns the
@@ -303,7 +425,12 @@ export function autopilotOwnsTurn(env = process.env) {
  */
 export async function onStop(opts = {}) {
     const source = opts.source ?? "unknown";
-    const cwd = opts.cwd ?? process.cwd();
+    // Effective project root: prefer the workspace the hook stdin names (Cursor &
+    // Claude run hooks from the IDE's config dir, so process.cwd() is e.g.
+    // `~/.cursor` — using it makes the project show as ".cursor"). An explicit
+    // --cwd flag still wins (tests / manual invocations). Falls back to cwd.
+    const rawCwd = opts.cwd ?? process.cwd();
+    const cwd = opts.cwd ?? workspaceRootFromHookStdin(opts.hookStdin) ?? rawCwd;
     const deriveRepo = opts.deriveRepo ?? deriveRepoIdentity;
     const resolveSettings = opts.resolveSettings ?? resolveNotifySettings;
     const logLine = async (line) => {

package/dist/src/opencode.js

@@ -68,7 +68,10 @@ export const OPENCODE_MCP_ENTRY = {
  * Claude/Cursor. Ends in `|| true` so a failure can never block the session, and
  * carries a `--dedupe-key` so overlapping triggers coalesce server-side.
  */
-export const OPENCODE_STOP_COMMAND = "npx -y @trygocode/notify on-stop --source opencode --dedupe-key opencode-idle || true";
+// `@latest` is REQUIRED: bare `npx -y @trygocode/notify` caches the first
+// resolved version forever and never auto-updates (npm/cli#6664). Pinning
+// `@latest` makes the hook always fetch the newest publish so users self-update.
+export const OPENCODE_STOP_COMMAND = "npx -y @trygocode/notify@latest on-stop --source opencode --dedupe-key opencode-idle || true";
 /**
  * Substrings that together identify our plugin file as OURS. Used to keep
  * uninstall surgical: we only delete the plugin file when BOTH markers are

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

package/package.json

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