mnotes-cli 1.11.0 → 1.12.0

package/dist/commands/connect/__tests__/wizard.test.js

@@ -68,14 +68,16 @@ const DEFAULT_OPTS = {
 // T-1: Template generation — hooks
 // =============================================================
 (0, vitest_1.describe)("hooks template", () => {
-    (0, vitest_1.it)("generates SessionStart hook referencing bash script", () => {
+    (0, vitest_1.it)("generates SessionStart hook referencing global bash script path", () => {
         const hooks = (0, hooks_1.generateHooksTemplate)(DEFAULT_OPTS);
         (0, vitest_1.expect)(hooks.SessionStart).toBeDefined();
         (0, vitest_1.expect)(hooks.SessionStart).toHaveLength(1);
         (0, vitest_1.expect)(hooks.SessionStart[0].matcher).toBe("");
         (0, vitest_1.expect)(hooks.SessionStart[0].hooks).toHaveLength(1);
         (0, vitest_1.expect)(hooks.SessionStart[0].hooks[0].type).toBe("command");
-        (0, vitest_1.expect)(hooks.SessionStart[0].hooks[0].command).toBe(".claude/hooks/mnotes-session-start.sh");
+        // Global path — resolves at generation time via os.homedir()
+        (0, vitest_1.expect)(hooks.SessionStart[0].hooks[0].command).toContain(path.join(".claude", "hooks", "mnotes", "scripts", "mnotes-session-start.sh"));
+        (0, vitest_1.expect)(path.isAbsolute(hooks.SessionStart[0].hooks[0].command)).toBe(true);
     });
     (0, vitest_1.it)("generates hook scripts with correct URL, workspaceId, and Accept headers", () => {
         const scripts = (0, hooks_1.generateHookScripts)(DEFAULT_OPTS);
@@ -178,29 +180,45 @@ const DEFAULT_OPTS = {
 // =============================================================
 (0, vitest_1.describe)("scaffoldItems: hooks", () => {
     let tmpDir;
+    let fakeHome;
+    let origHome;
     (0, vitest_1.beforeEach)(() => {
         tmpDir = makeTmpDir();
+        fakeHome = makeTmpDir();
+        origHome = process.env.HOME;
+        process.env.HOME = fakeHome;
     });
     (0, vitest_1.afterEach)(() => {
         cleanTmpDir(tmpDir);
+        cleanTmpDir(fakeHome);
+        if (origHome === undefined)
+            delete process.env.HOME;
+        else
+            process.env.HOME = origHome;
     });
-    (0, vitest_1.it)("creates .claude/settings.json with hooks and bash scripts (AC-4)", () => {
+    (0, vitest_1.it)("writes scripts to global ~/.claude/hooks/mnotes/scripts/ and settings.json to project (AC-4)", () => {
         const results = (0, wizard_1.scaffoldItems)(tmpDir, ["hooks"], DEFAULT_OPTS);
         (0, vitest_1.expect)(results).toHaveLength(1);
         (0, vitest_1.expect)(results[0].item).toBe("hooks");
         // 2 bash scripts + settings.json
         (0, vitest_1.expect)(results[0].filesWritten).toHaveLength(3);
+        // settings.json is project-local
         const settingsPath = path.join(tmpDir, ".claude", "settings.json");
         (0, vitest_1.expect)(fs.existsSync(settingsPath)).toBe(true);
         const settings = JSON.parse(fs.readFileSync(settingsPath, "utf-8"));
         (0, vitest_1.expect)(settings.hooks).toBeDefined();
         (0, vitest_1.expect)(settings.hooks.SessionStart).toBeDefined();
         (0, vitest_1.expect)(settings.hooks.SessionStart).toHaveLength(1);
-        // Bash scripts exist and are executable
-        const startScript = path.join(tmpDir, ".claude", "hooks", "mnotes-session-start.sh");
-        const stopScript = path.join(tmpDir, ".claude", "hooks", "mnotes-session-stop.sh");
+        // Bash scripts live under fake HOME at ~/.claude/hooks/mnotes/scripts/
+        const globalScriptsDir = path.join(fakeHome, ".claude", "hooks", "mnotes", "scripts");
+        const startScript = path.join(globalScriptsDir, "mnotes-session-start.sh");
+        const stopScript = path.join(globalScriptsDir, "mnotes-session-stop.sh");
         (0, vitest_1.expect)(fs.existsSync(startScript)).toBe(true);
         (0, vitest_1.expect)(fs.existsSync(stopScript)).toBe(true);
+        // Project directory must NOT contain scripts
+        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "hooks"))).toBe(false);
+        // settings.json references the global absolute path
+        (0, vitest_1.expect)(settings.hooks.SessionStart[0].hooks[0].command).toBe(startScript);
         const startContent = fs.readFileSync(startScript, "utf-8");
         (0, vitest_1.expect)(startContent).toContain("Accept: text/event-stream");
         (0, vitest_1.expect)(startContent).toContain("Accept: application/json");
@@ -348,11 +366,16 @@ const DEFAULT_OPTS = {
 // =============================================================
 (0, vitest_1.describe)("handleClaudeCode with wizard flags", () => {
     let tmpDir;
+    let fakeHome;
+    let origHome;
     let origCwd;
     let origExit;
     let exitCode;
     (0, vitest_1.beforeEach)(() => {
         tmpDir = makeTmpDir();
+        fakeHome = makeTmpDir();
+        origHome = process.env.HOME;
+        process.env.HOME = fakeHome;
         origCwd = process.cwd;
         process.cwd = () => tmpDir;
         exitCode = undefined;
@@ -366,6 +389,11 @@ const DEFAULT_OPTS = {
         process.cwd = origCwd;
         process.exit = origExit;
         cleanTmpDir(tmpDir);
+        cleanTmpDir(fakeHome);
+        if (origHome === undefined)
+            delete process.env.HOME;
+        else
+            process.env.HOME = origHome;
         vitest_1.vi.restoreAllMocks();
     });
     (0, vitest_1.it)("--no-wizard does core setup only, no extras (AC-6)", async () => {
@@ -430,10 +458,15 @@ const DEFAULT_OPTS = {
 // =============================================================
 (0, vitest_1.describe)("CLI flag integration", () => {
     let tmpDir;
+    let fakeHome;
+    let origHome;
     let origCwd;
     let origExit;
     (0, vitest_1.beforeEach)(() => {
         tmpDir = makeTmpDir();
+        fakeHome = makeTmpDir();
+        origHome = process.env.HOME;
+        process.env.HOME = fakeHome;
         origCwd = process.cwd;
         process.cwd = () => tmpDir;
         origExit = process.exit;
@@ -445,6 +478,11 @@ const DEFAULT_OPTS = {
         process.cwd = origCwd;
         process.exit = origExit;
         cleanTmpDir(tmpDir);
+        cleanTmpDir(fakeHome);
+        if (origHome === undefined)
+            delete process.env.HOME;
+        else
+            process.env.HOME = origHome;
         vitest_1.vi.restoreAllMocks();
     });
     (0, vitest_1.it)("--no-wizard flag is recognized by commander", async () => {

package/dist/commands/connect/wizard.js

@@ -102,19 +102,23 @@ function scaffoldItems(dir, items, opts) {
     return results;
 }
 /**
- * Merges hooks into `.claude/settings.json` and writes bash scripts to `.claude/hooks/`.
+ * Merges hooks into `<project>/.claude/settings.json` and writes bash scripts to
+ * `~/.claude/hooks/mnotes/scripts/` (user-global, namespaced under `mnotes`).
+ * Scripts live globally so they're shared across projects; settings.json stays
+ * project-local so each project opts in independently.
  * Preserves all existing settings and hooks.
  */
 function scaffoldHooks(dir, opts) {
     const settingsPath = path.join(dir, ".claude", "settings.json");
     const claudeDir = path.join(dir, ".claude");
-    const hooksDir = path.join(claudeDir, "hooks");
-    fs.mkdirSync(hooksDir, { recursive: true });
+    const hookScriptsDir = (0, index_1.getHookScriptsDir)();
+    fs.mkdirSync(claudeDir, { recursive: true });
+    fs.mkdirSync(hookScriptsDir, { recursive: true });
     const filesWritten = [];
-    // 1. Write bash scripts to .claude/hooks/
+    // 1. Write bash scripts to ~/.claude/hooks/mnotes/scripts/
     const scripts = (0, index_1.generateHookScripts)(opts);
     for (const script of scripts) {
-        const scriptPath = path.join(hooksDir, script.filename);
+        const scriptPath = path.join(hookScriptsDir, script.filename);
         fs.writeFileSync(scriptPath, script.content, { mode: 0o755 });
         filesWritten.push(scriptPath);
     }

package/dist/templates/claude-code/hooks.d.ts

@@ -1,8 +1,16 @@
 /**
  * Template for Claude Code session hooks.
- * Generates bash scripts in `.claude/hooks/` and hook entries for `.claude/settings.json`.
- * Generated by m-notes CLI — do not edit manually.
+ * Generates bash scripts in `~/.claude/hooks/mnotes/scripts/` (user-global,
+ * namespaced under `mnotes`) and hook entries for `.claude/settings.json`
+ * (project-local). Generated by m-notes CLI — do not edit manually.
  */
+/**
+ * Absolute path where m-notes hook scripts live (user-global, namespaced).
+ * Resolved via function so tests can override `HOME` to point to a tmp dir.
+ */
+export declare function getHookScriptsDir(): string;
+/** @deprecated kept for backward compatibility — use `getHookScriptsDir()` so HOME overrides work. */
+export declare const HOOK_SCRIPTS_DIR: string;
 export interface HooksTemplateOpts {
     url: string;
     workspaceId: string;
@@ -29,8 +37,9 @@ export interface HookScript {
 export declare function generateHookScripts(opts: HooksTemplateOpts): HookScript[];
 /**
  * Generates the hooks object to merge into `.claude/settings.json`.
- * References bash scripts in `.claude/hooks/` instead of inline commands.
+ * References bash scripts in `~/.claude/hooks/mnotes/scripts/` (absolute path
+ * so hooks work across projects without per-project script copies).
  */
-export declare function generateHooksTemplate(opts: HooksTemplateOpts): ClaudeCodeHooks;
+export declare function generateHooksTemplate(_opts: HooksTemplateOpts): ClaudeCodeHooks;
 /** Header comment for the hooks section, used for identification. */
 export declare const HOOKS_HEADER = "m-notes: auto-generated hooks";

package/dist/templates/claude-code/hooks.js

@@ -1,13 +1,59 @@
 "use strict";
 /**
  * Template for Claude Code session hooks.
- * Generates bash scripts in `.claude/hooks/` and hook entries for `.claude/settings.json`.
- * Generated by m-notes CLI — do not edit manually.
+ * Generates bash scripts in `~/.claude/hooks/mnotes/scripts/` (user-global,
+ * namespaced under `mnotes`) and hook entries for `.claude/settings.json`
+ * (project-local). Generated by m-notes CLI — do not edit manually.
  */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.HOOKS_HEADER = void 0;
+exports.HOOKS_HEADER = exports.HOOK_SCRIPTS_DIR = void 0;
+exports.getHookScriptsDir = getHookScriptsDir;
 exports.generateHookScripts = generateHookScripts;
 exports.generateHooksTemplate = generateHooksTemplate;
+const os = __importStar(require("os"));
+const path = __importStar(require("path"));
+/**
+ * Absolute path where m-notes hook scripts live (user-global, namespaced).
+ * Resolved via function so tests can override `HOME` to point to a tmp dir.
+ */
+function getHookScriptsDir() {
+    return path.join(os.homedir(), ".claude", "hooks", "mnotes", "scripts");
+}
+/** @deprecated kept for backward compatibility — use `getHookScriptsDir()` so HOME overrides work. */
+exports.HOOK_SCRIPTS_DIR = getHookScriptsDir();
 /**
  * Generates bash scripts to be written to `.claude/hooks/`.
  */
@@ -61,9 +107,13 @@ curl -s \\
 }
 /**
  * Generates the hooks object to merge into `.claude/settings.json`.
- * References bash scripts in `.claude/hooks/` instead of inline commands.
+ * References bash scripts in `~/.claude/hooks/mnotes/scripts/` (absolute path
+ * so hooks work across projects without per-project script copies).
  */
-function generateHooksTemplate(opts) {
+function generateHooksTemplate(_opts) {
+    const dir = getHookScriptsDir();
+    const startScript = path.join(dir, "mnotes-session-start.sh");
+    const stopScript = path.join(dir, "mnotes-session-stop.sh");
     return {
         SessionStart: [
             {
@@ -71,7 +121,7 @@ function generateHooksTemplate(opts) {
                 hooks: [
                     {
                         type: "command",
-                        command: ".claude/hooks/mnotes-session-start.sh",
+                        command: startScript,
                     },
                 ],
             },
@@ -82,7 +132,7 @@ function generateHooksTemplate(opts) {
                 hooks: [
                     {
                         type: "command",
-                        command: ".claude/hooks/mnotes-session-stop.sh",
+                        command: stopScript,
                     },
                 ],
             },

package/dist/templates/claude-code/index.d.ts

@@ -1,4 +1,4 @@
-export { generateHooksTemplate, generateHookScripts, HOOKS_HEADER } from "./hooks";
+export { generateHooksTemplate, generateHookScripts, HOOKS_HEADER, HOOK_SCRIPTS_DIR, getHookScriptsDir } from "./hooks";
 export type { HooksTemplateOpts, ClaudeCodeHooks, ClaudeCodeHook, HookScript } from "./hooks";
 export { generateSkillTemplates } from "./skills";
 export type { SkillTemplateOpts, SkillFile } from "./skills";

package/dist/templates/claude-code/index.js

@@ -1,10 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.generateAgentTemplates = exports.generateSkillTemplates = exports.HOOKS_HEADER = exports.generateHookScripts = exports.generateHooksTemplate = void 0;
+exports.generateAgentTemplates = exports.generateSkillTemplates = exports.getHookScriptsDir = exports.HOOK_SCRIPTS_DIR = exports.HOOKS_HEADER = exports.generateHookScripts = exports.generateHooksTemplate = void 0;
 var hooks_1 = require("./hooks");
 Object.defineProperty(exports, "generateHooksTemplate", { enumerable: true, get: function () { return hooks_1.generateHooksTemplate; } });
 Object.defineProperty(exports, "generateHookScripts", { enumerable: true, get: function () { return hooks_1.generateHookScripts; } });
 Object.defineProperty(exports, "HOOKS_HEADER", { enumerable: true, get: function () { return hooks_1.HOOKS_HEADER; } });
+Object.defineProperty(exports, "HOOK_SCRIPTS_DIR", { enumerable: true, get: function () { return hooks_1.HOOK_SCRIPTS_DIR; } });
+Object.defineProperty(exports, "getHookScriptsDir", { enumerable: true, get: function () { return hooks_1.getHookScriptsDir; } });
 var skills_1 = require("./skills");
 Object.defineProperty(exports, "generateSkillTemplates", { enumerable: true, get: function () { return skills_1.generateSkillTemplates; } });
 var agents_1 = require("./agents");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mnotes-cli",
-  "version": "1.11.0",
+  "version": "1.12.0",
   "description": "CLI for m-notes AI knowledge base — manage notes, search, and CRUD from the terminal",
   "bin": {
     "mnotes": "./dist/index.js"