mnotes-cli 1.9.0 → 1.9.2

package/dist/client.d.ts

@@ -72,6 +72,26 @@ export declare function createClient(baseUrl: string, apiKey: string): {
             isDefault: boolean;
         }>;
     }>;
+    recallKnowledge(opts: {
+        query: string;
+        workspaceId?: string;
+        tags?: string[];
+        limit?: number;
+    }): Promise<{
+        data: {
+            results: Array<{
+                id: string;
+                title: string;
+                key: string | null;
+                excerpt: string;
+                importance: number | null;
+                tags: string[];
+                semanticScore: number;
+                freshnessScore: number;
+                finalScore: number;
+            }>;
+        };
+    }>;
     queryGraph(opts?: {
         workspaceId?: string;
         query?: string;

package/dist/client.js

@@ -51,6 +51,14 @@ function createClient(baseUrl, apiKey) {
         async listWorkspaces() {
             return request("GET", "/api/v1/workspaces");
         },
+        async recallKnowledge(opts) {
+            return request("POST", "/api/v1/knowledge/recall", {
+                query: opts.query,
+                workspaceId: opts.workspaceId,
+                tags: opts.tags,
+                limit: opts.limit,
+            });
+        },
         async queryGraph(opts) {
             const params = new URLSearchParams();
             if (opts?.workspaceId)

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

@@ -79,11 +79,11 @@ const DEFAULT_OPTS = {
         const skills = (0, skills_1.generateSkillTemplates)(DEFAULT_OPTS);
         (0, vitest_1.expect)(skills).toHaveLength(2);
     });
-    (0, vitest_1.it)("generates store and recall skills", () => {
+    (0, vitest_1.it)("generates store and recall skills as SKILL.md inside their own folder (AC-1)", () => {
         const skills = (0, skills_1.generateSkillTemplates)(DEFAULT_OPTS);
-        const filenames = skills.map((s) => s.filename);
-        (0, vitest_1.expect)(filenames).toContain("mnotes-store.md");
-        (0, vitest_1.expect)(filenames).toContain("mnotes-recall.md");
+        const paths = skills.map((s) => s.path);
+        (0, vitest_1.expect)(paths).toContain("mnotes-store/SKILL.md");
+        (0, vitest_1.expect)(paths).toContain("mnotes-recall/SKILL.md");
     });
     (0, vitest_1.it)("includes m-notes generated header (AC-8)", () => {
         const skills = (0, skills_1.generateSkillTemplates)(DEFAULT_OPTS);
@@ -97,12 +97,13 @@ const DEFAULT_OPTS = {
             (0, vitest_1.expect)(skill.content).toContain("ws-test-123");
         }
     });
-    (0, vitest_1.it)("includes frontmatter with name and trigger", () => {
+    (0, vitest_1.it)("includes frontmatter with name and description and omits non-standard fields", () => {
         const skills = (0, skills_1.generateSkillTemplates)(DEFAULT_OPTS);
         for (const skill of skills) {
             (0, vitest_1.expect)(skill.content).toContain("---");
             (0, vitest_1.expect)(skill.content).toContain("name:");
-            (0, vitest_1.expect)(skill.content).toContain("trigger:");
+            (0, vitest_1.expect)(skill.content).toContain("description:");
+            (0, vitest_1.expect)(skill.content).not.toMatch(/^trigger:/m);
         }
     });
 });
@@ -211,38 +212,38 @@ const DEFAULT_OPTS = {
     (0, vitest_1.afterEach)(() => {
         cleanTmpDir(tmpDir);
     });
-    (0, vitest_1.it)("creates skill files in .claude/skills/ (AC-4)", () => {
+    (0, vitest_1.it)("creates skill files as .claude/skills/<name>/SKILL.md (AC-4)", () => {
         const results = (0, wizard_1.scaffoldItems)(tmpDir, ["skills"], DEFAULT_OPTS);
         (0, vitest_1.expect)(results).toHaveLength(1);
         (0, vitest_1.expect)(results[0].item).toBe("skills");
         (0, vitest_1.expect)(results[0].filesWritten.length).toBeGreaterThan(0);
         const skillsDir = path.join(tmpDir, ".claude", "skills");
-        (0, vitest_1.expect)(fs.existsSync(path.join(skillsDir, "mnotes-store.md"))).toBe(true);
-        (0, vitest_1.expect)(fs.existsSync(path.join(skillsDir, "mnotes-recall.md"))).toBe(true);
+        (0, vitest_1.expect)(fs.existsSync(path.join(skillsDir, "mnotes-store", "SKILL.md"))).toBe(true);
+        (0, vitest_1.expect)(fs.existsSync(path.join(skillsDir, "mnotes-recall", "SKILL.md"))).toBe(true);
     });
     (0, vitest_1.it)("generated skill files include header (AC-8)", () => {
         (0, wizard_1.scaffoldItems)(tmpDir, ["skills"], DEFAULT_OPTS);
-        const storeContent = fs.readFileSync(path.join(tmpDir, ".claude", "skills", "mnotes-store.md"), "utf-8");
+        const storeContent = fs.readFileSync(path.join(tmpDir, ".claude", "skills", "mnotes-store", "SKILL.md"), "utf-8");
         (0, vitest_1.expect)(storeContent).toContain("Generated by m-notes CLI");
     });
     (0, vitest_1.it)("preserves user-created skill files (AC-5)", () => {
-        const skillsDir = path.join(tmpDir, ".claude", "skills");
-        fs.mkdirSync(skillsDir, { recursive: true });
-        // Write a user-created file with the same name
-        fs.writeFileSync(path.join(skillsDir, "mnotes-store.md"), "# My custom store skill\nCustom content here", "utf-8");
-        const results = (0, wizard_1.scaffoldItems)(tmpDir, ["skills"], DEFAULT_OPTS);
+        const storeDir = path.join(tmpDir, ".claude", "skills", "mnotes-store");
+        fs.mkdirSync(storeDir, { recursive: true });
+        // Write a user-created file at the same path
+        fs.writeFileSync(path.join(storeDir, "SKILL.md"), "# My custom store skill\nCustom content here", "utf-8");
+        (0, wizard_1.scaffoldItems)(tmpDir, ["skills"], DEFAULT_OPTS);
         // The user file should be preserved
-        const content = fs.readFileSync(path.join(skillsDir, "mnotes-store.md"), "utf-8");
+        const content = fs.readFileSync(path.join(storeDir, "SKILL.md"), "utf-8");
         (0, vitest_1.expect)(content).toContain("My custom store skill");
         (0, vitest_1.expect)(content).not.toContain("Generated by m-notes CLI");
         // The recall skill should still be written (new file)
-        (0, vitest_1.expect)(fs.existsSync(path.join(skillsDir, "mnotes-recall.md"))).toBe(true);
+        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-recall", "SKILL.md"))).toBe(true);
     });
     (0, vitest_1.it)("overwrites m-notes generated skill files on re-run (AC-5)", () => {
         (0, wizard_1.scaffoldItems)(tmpDir, ["skills"], DEFAULT_OPTS);
         // Re-run with different workspaceId
         (0, wizard_1.scaffoldItems)(tmpDir, ["skills"], { url: "http://new-url.com", workspaceId: "ws-new" });
-        const content = fs.readFileSync(path.join(tmpDir, ".claude", "skills", "mnotes-store.md"), "utf-8");
+        const content = fs.readFileSync(path.join(tmpDir, ".claude", "skills", "mnotes-store", "SKILL.md"), "utf-8");
         (0, vitest_1.expect)(content).toContain("ws-new");
     });
 });
@@ -298,8 +299,8 @@ const DEFAULT_OPTS = {
         (0, vitest_1.expect)(items).toContain("agents");
         // Verify files exist
         (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "settings.json"))).toBe(true);
-        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-store.md"))).toBe(true);
-        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-recall.md"))).toBe(true);
+        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-store", "SKILL.md"))).toBe(true);
+        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-recall", "SKILL.md"))).toBe(true);
         (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "agents", "knowledge-manager.md"))).toBe(true);
     });
     (0, vitest_1.it)("scaffolds partial selection", () => {
@@ -386,8 +387,8 @@ const DEFAULT_OPTS = {
         (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, "CLAUDE.md"))).toBe(true);
         // All extras should exist
         (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "settings.json"))).toBe(true);
-        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-store.md"))).toBe(true);
-        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-recall.md"))).toBe(true);
+        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-store", "SKILL.md"))).toBe(true);
+        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-recall", "SKILL.md"))).toBe(true);
         (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "agents", "knowledge-manager.md"))).toBe(true);
         // Output should mention extras
         (0, vitest_1.expect)(output).toContain("Extras installed:");
@@ -465,7 +466,7 @@ const DEFAULT_OPTS = {
         }
         // All extras should exist
         (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "settings.json"))).toBe(true);
-        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-store.md"))).toBe(true);
+        (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "skills", "mnotes-store", "SKILL.md"))).toBe(true);
         (0, vitest_1.expect)(fs.existsSync(path.join(tmpDir, ".claude", "agents", "knowledge-manager.md"))).toBe(true);
     });
 });

package/dist/commands/connect/wizard.js

@@ -170,8 +170,9 @@ function scaffoldSkills(dir, opts) {
     const skills = (0, index_1.generateSkillTemplates)(opts);
     const filesWritten = [];
     for (const skill of skills) {
-        const filePath = path.join(skillsDir, skill.filename);
+        const filePath = path.join(skillsDir, skill.path);
         if (shouldWriteGeneratedFile(filePath)) {
+            fs.mkdirSync(path.dirname(filePath), { recursive: true });
             fs.writeFileSync(filePath, skill.content, "utf-8");
             filesWritten.push(filePath);
         }

package/dist/commands/recall-knowledge.js

@@ -7,30 +7,55 @@ const output_1 = require("../output");
 function registerRecallKnowledgeCommand(program) {
     program
         .command("recall_knowledge")
-        .description("Query the knowledge graph (read-only)")
-        .option("--query <text>", "Filter nodes by label (case-insensitive)")
-        .option("--type <type>", "Filter by node type: note, tag, concept")
-        .option("--neighbors <nodeId>", "Show neighbors of a specific node")
-        .option("--depth <n>", "Neighbor traversal depth (1-3, default 1)", "1")
-        .option("--limit <n>", "Max nodes to return (default 50)", "50")
+        .description("Query knowledge entries via semantic search (read-only)")
+        .option("--query <text>", "Search query (required for semantic recall)")
+        .option("--tags <tags>", "Filter by tags (comma-separated)")
+        .option("--limit <n>", "Max results (default 10)", "10")
+        .option("--graph", "Query the knowledge graph (GraphNode/GraphEdge) instead of semantic recall")
+        .option("--type <type>", "Node type filter (graph mode only): note, tag, concept")
+        .option("--neighbors <nodeId>", "Show neighbors of a node (graph mode only)")
+        .option("--depth <n>", "Neighbor traversal depth 1-3 (graph mode only)", "1")
         .option("--workspace-id <id>", "Workspace ID")
         .action(async (opts) => {
         const globalOpts = program.opts();
         const config = (0, config_1.resolveConfig)(globalOpts);
         const client = (0, client_1.createClient)(config.baseUrl, config.apiKey);
-        const result = await client.queryGraph({
-            workspaceId: opts.workspaceId || config.workspaceId,
+        const workspaceId = opts.workspaceId || config.workspaceId;
+        // Graph mode
+        if (opts.graph) {
+            const result = await client.queryGraph({
+                workspaceId,
+                query: opts.query,
+                nodeType: opts.type,
+                neighbors: opts.neighbors,
+                depth: opts.depth ? parseInt(opts.depth, 10) : undefined,
+                limit: opts.limit ? parseInt(opts.limit, 10) : undefined,
+            });
+            if (globalOpts.json) {
+                (0, output_1.printJson)(result.data);
+            }
+            else {
+                (0, output_1.printGraph)(result.data.nodes, result.data.edges);
+            }
+            return;
+        }
+        // Semantic recall mode (default)
+        if (!opts.query) {
+            process.stderr.write("Error: --query is required for semantic recall. Use --graph for graph queries without a query.\n");
+            process.exit(1);
+        }
+        const tags = opts.tags ? opts.tags.split(",").map((t) => t.trim()) : undefined;
+        const result = await client.recallKnowledge({
             query: opts.query,
-            nodeType: opts.type,
-            neighbors: opts.neighbors,
-            depth: opts.depth ? parseInt(opts.depth, 10) : undefined,
+            workspaceId,
+            tags,
             limit: opts.limit ? parseInt(opts.limit, 10) : undefined,
         });
         if (globalOpts.json) {
-            (0, output_1.printJson)(result.data);
+            (0, output_1.printJson)(result.data.results);
         }
         else {
-            (0, output_1.printGraph)(result.data.nodes, result.data.edges);
+            (0, output_1.printKnowledgeResults)(result.data.results);
         }
     });
 }

package/dist/output.d.ts

@@ -14,6 +14,17 @@ export declare function printSearchResults(results: Array<{
     title: string;
     snippet?: string;
 }>): void;
+export declare function printKnowledgeResults(results: Array<{
+    id: string;
+    title: string;
+    key: string | null;
+    excerpt: string;
+    importance: number | null;
+    tags: string[];
+    semanticScore: number;
+    freshnessScore: number;
+    finalScore: number;
+}>): void;
 export declare function printGraph(nodes: Array<{
     id: string;
     noteId: string | null;

package/dist/output.js

@@ -4,6 +4,7 @@ exports.printJson = printJson;
 exports.printNoteList = printNoteList;
 exports.printNote = printNote;
 exports.printSearchResults = printSearchResults;
+exports.printKnowledgeResults = printKnowledgeResults;
 exports.printGraph = printGraph;
 exports.printSuccess = printSuccess;
 function printJson(data) {
@@ -43,6 +44,28 @@ function printSearchResults(results) {
         }
     }
 }
+function printKnowledgeResults(results) {
+    if (results.length === 0) {
+        process.stderr.write("No knowledge entries found.\n");
+        return;
+    }
+    for (let i = 0; i < results.length; i++) {
+        const r = results[i];
+        const score = (r.finalScore * 100).toFixed(0);
+        const key = r.key ? ` [${r.key}]` : "";
+        const tags = r.tags.length > 0 ? ` (${r.tags.join(", ")})` : "";
+        const importance = r.importance !== null ? ` imp:${r.importance}` : "";
+        console.log(`${i + 1}. ${r.title}${key}  ${score}%${importance}${tags}`);
+        if (r.excerpt) {
+            const lines = r.excerpt.split("\n").slice(0, 3).join("\n   ");
+            console.log(`   ${lines}`);
+        }
+        if (i < results.length - 1)
+            console.log("");
+    }
+    console.log("");
+    console.log(`${results.length} result(s)`);
+}
 function printGraph(nodes, edges) {
     if (nodes.length === 0) {
         process.stderr.write("Knowledge graph is empty.\n");

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

@@ -1,6 +1,7 @@
 /**
  * Templates for Claude Code skills.
- * Written to `.claude/skills/` as markdown files.
+ * Written to `.claude/skills/<skill-name>/SKILL.md` — Claude Code's skill loader
+ * only discovers skills that live in their own folder as `SKILL.md`.
  * Generated by m-notes CLI — do not edit manually.
  */
 export interface SkillTemplateOpts {
@@ -8,11 +9,14 @@ export interface SkillTemplateOpts {
     workspaceId: string;
 }
 export interface SkillFile {
-    filename: string;
+    /** Relative path within `.claude/skills/`, e.g. `mnotes-store/SKILL.md`. */
+    path: string;
     content: string;
 }
 /**
  * Generates skill files for m-notes integration.
- * Returns an array of { filename, content } pairs.
+ * Returns an array of { path, content } pairs. `path` is relative to
+ * `.claude/skills/` and always ends in `SKILL.md` so Claude Code's loader
+ * discovers the skill.
  */
 export declare function generateSkillTemplates(opts: SkillTemplateOpts): SkillFile[];

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

@@ -1,7 +1,8 @@
 "use strict";
 /**
  * Templates for Claude Code skills.
- * Written to `.claude/skills/` as markdown files.
+ * Written to `.claude/skills/<skill-name>/SKILL.md` — Claude Code's skill loader
+ * only discovers skills that live in their own folder as `SKILL.md`.
  * Generated by m-notes CLI — do not edit manually.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
@@ -9,17 +10,18 @@ exports.generateSkillTemplates = generateSkillTemplates;
 const GENERATED_HEADER = "<!-- Generated by m-notes CLI. Do not edit manually. -->";
 /**
  * Generates skill files for m-notes integration.
- * Returns an array of { filename, content } pairs.
+ * Returns an array of { path, content } pairs. `path` is relative to
+ * `.claude/skills/` and always ends in `SKILL.md` so Claude Code's loader
+ * discovers the skill.
  */
 function generateSkillTemplates(opts) {
     return [
         {
-            filename: "mnotes-store.md",
+            path: "mnotes-store/SKILL.md",
             content: `${GENERATED_HEADER}
 ---
 name: mnotes-store
-description: Store a knowledge entry in m-notes
-trigger: When the user says /store or asks to save/store knowledge, decisions, patterns, or context
+description: Store a knowledge entry in m-notes. Use when the user says /store or asks to save/store knowledge, decisions, patterns, or context.
 ---
 
 # Store Knowledge in m-notes
@@ -44,12 +46,11 @@ Use the \`knowledge_store\` MCP tool to save knowledge.
 `,
         },
         {
-            filename: "mnotes-recall.md",
+            path: "mnotes-recall/SKILL.md",
             content: `${GENERATED_HEADER}
 ---
 name: mnotes-recall
-description: Recall knowledge from m-notes using semantic search
-trigger: When the user says /recall or asks to find/recall/search knowledge, or needs context about a topic
+description: Recall knowledge from m-notes using semantic search. Use when the user says /recall or asks to find/recall/search knowledge, or needs context about a topic.
 ---
 
 # Recall Knowledge from m-notes

package/package.json

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