neuro-vault-mcp 6.0.0 → 7.0.0

package/dist/cli.js

@@ -59,10 +59,6 @@ async function parseConfig(argv) {
     type: "boolean",
     default: true,
     describe: "Enable semantic search module (Smart Connections embeddings)"
-  }).option("operations", {
-    type: "boolean",
-    default: true,
-    describe: "Enable vault operations module (Obsidian CLI required at call time)"
   }).option("obsidian-cli", {
     type: "string",
     describe: 'Path to the obsidian CLI binary (default: "obsidian" from PATH)'
@@ -71,9 +67,6 @@ async function parseConfig(argv) {
   if (rawVaults.length === 0) {
     throw new Error("--vault is required: provide at least one vault with --vault <path>");
   }
-  if (!args.semantic && !args.operations) {
-    throw new Error("At least one module must be enabled (--semantic or --operations)");
-  }
   const vaults = rawVaults.map(buildVaultConfig);
   const seen = /* @__PURE__ */ new Set();
   for (const v of vaults) {
@@ -92,10 +85,7 @@ async function parseConfig(argv) {
       modelKey: DEFAULT_MODEL_KEY,
       modelId: DEFAULT_MODEL_ID
     },
-    operations: {
-      enabled: args.operations,
-      binaryPath: args["obsidian-cli"]
-    }
+    obsidianCli: args["obsidian-cli"]
   };
 }
 
@@ -14315,19 +14305,21 @@ function registerTool(tool) {
 
 // src/lib/resolve-vault.ts
 function resolveVault(input, registry2, opts) {
-  let entry;
   if (input.vault !== void 0 && input.vault !== "") {
-    entry = registry2.require(input.vault);
-  } else if (!registry2.isMulti()) {
-    entry = registry2.list()[0];
-  } else {
-    throw new ToolHandlerError(
-      "VAULT_REQUIRED",
-      `Tool "${opts.tool}" requires a vault: parameter in multi-vault mode`,
-      { details: { tool: opts.tool, registered_vaults: registry2.names() } }
-    );
+    return registry2.require(input.vault);
+  }
+  if (!registry2.isMulti()) {
+    return registry2.list()[0];
   }
-  if (opts.requireSemantic && !entry.semanticAvailable) {
+  throw new ToolHandlerError(
+    "VAULT_REQUIRED",
+    `Tool "${opts.tool}" requires a vault: parameter in multi-vault mode`,
+    { details: { tool: opts.tool, registered_vaults: registry2.names() } }
+  );
+}
+function resolveSemanticVault(input, registry2, opts) {
+  const entry = resolveVault(input, registry2, opts);
+  if (!entry.semanticAvailable) {
     throw new ToolHandlerError(
       "SEMANTIC_INDEX_NOT_FOUND",
       `Semantic index for vault "${entry.name}" is unavailable: ${entry.semanticUnavailableReason ?? "unknown reason"}`,
@@ -14511,9 +14503,8 @@ function buildFindDuplicatesTool(deps) {
     ),
     inputSchema,
     handler: async (input) => {
-      const entry = resolveVault(input, registry2, {
-        tool: "find_duplicates",
-        requireSemantic: true
+      const entry = resolveSemanticVault(input, registry2, {
+        tool: "find_duplicates"
       });
       const corpus = entry.corpus;
       const threshold = readThreshold(input.threshold, DEFAULT_DUPLICATE_THRESHOLD, "threshold");
@@ -15612,9 +15603,8 @@ function buildGetSimilarNotesTool(deps) {
     ),
     inputSchema,
     handler: async (input) => {
-      const entry = resolveVault(input, registry2, {
-        tool: "get_similar_notes",
-        requireSemantic: true
+      const entry = resolveSemanticVault(input, registry2, {
+        tool: "get_similar_notes"
       });
       const corpus = entry.corpus;
       const notePath = normalizeNotePath(input.path);
@@ -15696,9 +15686,8 @@ function buildGetStatsTool(deps) {
     ),
     inputSchema,
     handler: async (input) => {
-      const entry = resolveVault(input, registry2, {
-        tool: "get_stats",
-        requireSemantic: true
+      const entry = resolveSemanticVault(input, registry2, {
+        tool: "get_stats"
       });
       const corpus = entry.corpus;
       try {
@@ -16179,12 +16168,15 @@ function buildSearchNotesTool(deps) {
       if (input.vault === void 0 && registry2.isMulti()) {
         return await runSemanticFanOut(
           registry2,
-          (entry2) => runSearchForEntry(entry2, input, entryDeps)
+          (entry2) => runSearchForEntry(
+            entry2,
+            input,
+            entryDeps
+          )
         );
       }
-      const entry = resolveVault(input, registry2, {
-        tool: "search_notes",
-        requireSemantic: true
+      const entry = resolveSemanticVault(input, registry2, {
+        tool: "search_notes"
       });
       return runSearchForEntry(entry, input, entryDeps);
     }
@@ -16368,9 +16360,6 @@ function buildCreateNoteTool(deps) {
     inputSchema,
     handler: async (input) => {
       const entry = resolveVault(input, registry2, { tool: "create_note" });
-      if (!entry.provider) {
-        throw invalidArgument("operations module is disabled", "vault");
-      }
       if (input.name === void 0 && input.path === void 0) {
         throw invalidArgument("Provide name or path", "name");
       }
@@ -16948,11 +16937,6 @@ function registerResource(resource) {
 // src/modules/operations/resources/vault-overview.ts
 function buildVaultOverviewResource(opts) {
   const { uri, entry } = opts;
-  if (!entry.provider) {
-    throw new Error(
-      `buildVaultOverviewResource: vault "${entry.name}" has no provider \u2014 operations module must be enabled`
-    );
-  }
   return {
     name: uri === "vault://overview" ? "vault-overview" : `vault-overview-${entry.name}`,
     uri,
@@ -17008,8 +16992,8 @@ var VaultRegistry = class _VaultRegistry {
       const reader = deps.readerFactory({ vaultRoot: v.path });
       const graph = deps.graphFactory({ reader });
       const listMatchingPaths = deps.listMatchingPathsFactory({ reader, graph });
-      const writer = config2.operationsEnabled ? deps.writerFactory({ vaultRoot: v.path }) : void 0;
-      const provider = config2.operationsEnabled ? deps.providerFactory({ vaultName: v.name, binaryPath: config2.binaryPath }) : void 0;
+      const writer = deps.writerFactory({ vaultRoot: v.path });
+      const provider = deps.providerFactory({ vaultName: v.name, binaryPath: config2.binaryPath });
       let corpus;
       let semanticAvailable = false;
       let semanticUnavailableReason;
@@ -17304,7 +17288,7 @@ var ObsidianCLIProvider = class {
     if (/vault (not found|does not exist)/i.test(stderr)) {
       return new ToolHandlerError(
         "VAULT_NOT_FOUND",
-        `Obsidian does not recognize a vault named '${this.vaultName}'. This usually means the vault was renamed in Obsidian's "Manage vaults" UI and the display name no longer matches the directory basename. Re-launch the server with --vault-name <exact-name-in-obsidian>.`,
+        `Obsidian does not recognize a vault named '${this.vaultName}'. The MCP-side alias is derived from the vault directory's basename and must match the display name Obsidian shows in "Manage vaults". Rename either the directory (and re-launch with the matching --vault path) or the vault in Obsidian so the two agree.`,
         { details: { stderr, vaultName: this.vaultName }, cause: error48 }
       );
     }
@@ -17522,16 +17506,12 @@ function buildDefaultVaultEntryDeps(overrides = {}) {
   };
 }
 async function startNeuroVaultServer(config2, deps = {}) {
-  if (!config2.semantic.enabled && !config2.operations.enabled) {
-    throw new Error("No modules enabled \u2014 pass --semantic or --operations");
-  }
   const registry2 = await VaultRegistry.create(
     {
       vaults: config2.vaults,
-      operationsEnabled: config2.operations.enabled,
       semanticEnabled: config2.semantic.enabled,
       modelKey: config2.semantic.modelKey,
-      binaryPath: config2.operations.binaryPath
+      binaryPath: config2.obsidianCli
     },
     buildDefaultVaultEntryDeps(deps.vaultEntryDeps)
   );
@@ -17552,15 +17532,9 @@ async function startNeuroVaultServer(config2, deps = {}) {
     toolRegistrations.push(...semantic.tools);
     warmup = semantic.warmup;
   }
-  if (config2.operations.enabled) {
-    const operations = createOperationsModule(
-      registry2,
-      { binaryPath: config2.operations.binaryPath },
-      deps.operations
-    );
-    toolRegistrations.push(...operations.tools);
-    resourceRegistrations.push(...operations.resources);
-  }
+  const operations = createOperationsModule(registry2, {}, deps.operations);
+  toolRegistrations.push(...operations.tools);
+  resourceRegistrations.push(...operations.resources);
   for (const tool of toolRegistrations) {
     server.registerTool(tool.name, tool.spec, tool.handler);
   }