@opennous/cli 0.7.2 → 0.7.5

package/package.json

@@ -1,17 +1,28 @@
 {
   "name": "@opennous/cli",
-  "version": "0.7.2",
+  "version": "0.7.5",
   "description": "Nous CLI — set up Nous, connect it to your coding agent, and read the Context API from your terminal",
   "type": "module",
   "bin": {
     "nous": "src/index.js"
   },
-  "files": ["src", "README.md"],
+  "files": [
+    "src",
+    "README.md"
+  ],
   "scripts": {
     "dev": "node src/index.js",
     "start": "node src/index.js"
   },
-  "keywords": ["nous", "crm", "cli", "agents", "memory", "observability", "stripe"],
+  "keywords": [
+    "nous",
+    "crm",
+    "cli",
+    "agents",
+    "memory",
+    "observability",
+    "stripe"
+  ],
   "license": "Apache-2.0",
   "repository": {
     "type": "git",

package/src/hooks/icp-sync-reminder.mjs

@@ -0,0 +1,33 @@
+#!/usr/bin/env node
+// Nous ICP auto-sync reminder (installed by `npx @opennous/cli init`).
+//
+// A Claude Code PostToolUse hook. After the agent edits a Nous context file, it
+// reminds the agent that the file is INERT until synced: the app, the ICP tab, and
+// the ICP scoring model read the Nous GRAPH, not the repo file, so an edit does
+// nothing until mcp__nous__sync_icp runs. Standalone Node (no bash/jq) so it behaves
+// identically on Windows, macOS, and Linux.
+
+let raw = "";
+process.stdin.on("data", (c) => { raw += c; });
+process.stdin.on("end", () => {
+  try {
+    const j = JSON.parse(raw || "{}");
+    const f = (j && j.tool_input && j.tool_input.file_path) || (j && j.tool_response && j.tool_response.filePath) || "";
+    const norm = String(f).replace(/\\/g, "/");           // normalize Windows separators
+    if (!/\.md$/i.test(norm) || !/\/context\//i.test(norm)) return;
+
+    // Report the path relative to the project so the agent passes the right source_path.
+    const proj = (process.env.CLAUDE_PROJECT_DIR || "").replace(/\\/g, "/");
+    let rel = norm;
+    if (proj && norm.toLowerCase().startsWith(proj.toLowerCase())) {
+      rel = norm.slice(proj.length).replace(/^\/+/, "");
+    }
+    const base = norm.split("/").pop().toLowerCase();
+
+    const ctx = base === "icp.md"
+      ? `You just edited ${rel}, your Nous ICP file. It is INERT until synced: the app, the ICP tab, and the ICP scoring model read the Nous graph, not this file. Call the mcp__nous__sync_icp tool THIS TURN with section "ICP" and source_path "${rel}", passing the file body as the section content, so the graph and the scoring model rebuild from your edit. Do not end the turn with the ICP file edited but unsynced.`
+      : `You edited a Nous context file (${rel}). It stays inert until synced into the Nous graph. If it is an ICP, positioning, pricing, competitors, or market file, call mcp__nous__sync_icp this turn with the matching section and its source_path so the graph updates.`;
+
+    process.stdout.write(JSON.stringify({ hookSpecificOutput: { hookEventName: "PostToolUse", additionalContext: ctx } }));
+  } catch { /* never block a tool call */ }
+});

package/src/index.js

@@ -32,12 +32,16 @@ function openBrowser(url) {
   if (process.platform === "darwin") {
     cmd = "open"; args = [url];
   } else if (process.platform === "win32") {
-    // cmd.exe /c start "" "<url>". The empty "" is start's title argument (else a
-    // quoted URL is taken as the window title). windowsVerbatimArguments keeps the
-    // quotes we add, so an & in the auth URL's query string stays inside them and
-    // isn't parsed by cmd as a command separator.
+    // cmd.exe /c start "" "<url>". `start` reads a lone quoted arg as the window
+    // TITLE, so it needs an explicit empty title before the URL — otherwise it just
+    // opens a shell titled with the URL and never launches the browser.
+    // CRITICAL: the title must be a LITERAL `""` string, not JS's zero-length "".
+    // With windowsVerbatimArguments the args are joined by spaces as-is, so a
+    // zero-length string collapses to nothing and the title is lost (the URL then
+    // becomes the title). Verbatim also keeps our quotes, so an & in the auth URL's
+    // query string stays inside them and isn't parsed by cmd as a command separator.
     cmd = process.env.ComSpec || "cmd.exe";
-    args = ["/c", "start", "", `"${url}"`];
+    args = ["/c", "start", '""', `"${url}"`];
     opts.windowsVerbatimArguments = true;
   } else {
     cmd = "xdg-open"; args = [url];
@@ -957,6 +961,42 @@ function registerMcp(apiUrl) {
   return `Wrote ${path} — your agent will pick up Nous from it.`;
 }
 
+// Install the ICP auto-sync hook into the project's Claude Code config. Once in place,
+// editing a Nous context file (context/**/icp.md, etc.) reminds the agent to call
+// sync_icp that turn — because an edited ICP file is INERT until synced into the graph.
+// Copies our packaged hook script into .claude/hooks/ and merges the PostToolUse entry
+// into .claude/settings.json WITHOUT clobbering any existing hooks or settings.
+// Returns true if it was already installed. Uses the exec (args) form so there's no
+// shell/quoting difference between Windows PowerShell and macOS/Linux bash.
+function installIcpSyncHook() {
+  const claudeDir = join(process.cwd(), ".claude");
+  const hooksDir = join(claudeDir, "hooks");
+  const scriptPath = join(hooksDir, "icp-sync-reminder.mjs");
+  const settingsPath = join(claudeDir, "settings.json");
+
+  const src = readFileSync(new URL("./hooks/icp-sync-reminder.mjs", import.meta.url), "utf8");
+  mkdirSync(hooksDir, { recursive: true });
+  writeFileSync(scriptPath, src);
+
+  let settings = {};
+  if (existsSync(settingsPath)) {
+    try { settings = JSON.parse(readFileSync(settingsPath, "utf8")) || {}; } catch { settings = {}; }
+  }
+  settings.hooks = settings.hooks || {};
+  const post = settings.hooks.PostToolUse = settings.hooks.PostToolUse || [];
+  const already = post.some((e) => (e.hooks || []).some((h) =>
+    (Array.isArray(h.args) && h.args.some((a) => String(a).includes("icp-sync-reminder"))) ||
+    String(h.command || "").includes("icp-sync-reminder")));
+  if (!already) {
+    post.push({
+      matcher: "Write|Edit|MultiEdit",
+      hooks: [{ type: "command", command: "node", args: [scriptPath], timeout: 10 }],
+    });
+    writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + "\n");
+  }
+  return already;
+}
+
 // nous init — the one-line "get started": sign in, register the MCP, hand off to the agent.
 // This is the command the install script and the marketing site point to.
 program
@@ -976,10 +1016,18 @@ program
         console.log("\nCouldn't register the MCP automatically. Add it with:");
         console.log(`  claude mcp add nous -- ${MCP_CMD} ${MCP_ARGS.join(" ")}`);
       }
+
+      // Auto-sync the ICP: a Claude Code hook so editing your ICP file reminds the
+      // agent to sync it into Nous (the file is inert until synced). Best-effort —
+      // a convenience, never a reason to fail setup.
+      try {
+        const already = installIcpSyncHook();
+        if (!already) console.log("Installed the ICP auto-sync hook — edits to your ICP file now sync to Nous automatically.");
+      } catch { /* non-fatal */ }
     }
 
-    console.log('\nYou\'re set. Tell your agent: "set up my GTM context layer and connect it to');
-    console.log("Nous\" — it audits your project's context, drafts what's missing, and syncs it.\n");
+    console.log("\nYou're set. Tell your agent:\n");
+    console.log('  "set up my Nous workspace and walk me through onboarding"\n');
     console.log(dim("Docs: https://docs.opennous.cloud/mcp/introduction"));
   });