@trygocode/notify 0.3.1 → 0.3.2

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
@@ -211,6 +279,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 +358,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 +372,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 +415,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/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.2";

package/package.json

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