From afefdc9f9244cfc04a81ba442a3ca677dd7a4076 Mon Sep 17 00:00:00 2001
From: "Peter.Morton" <peter.morton@verint.com>
Date: Sun, 22 Mar 2026 12:46:36 -0500
Subject: [PATCH] feat: add proper input schema to obsidian_search tool
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Added complete inputSchema definition for the obsidian_search tool
with all parameters matching the Obsidian CLI specification:

Parameters (per 'obsidian search help'):
- query (string, required): Search query text
- path (string, optional): Limit search to folder path
- limit (number, optional): Maximum number of results
- total (boolean, optional): Return match count instead of files
- case (boolean, optional): Case sensitive search
- format (enum, optional): Output format (text|json, default: text)

The inputSchema is now properly exposed via tools/list, enabling:
- Better parameter documentation in MCP clients
- Automatic parameter validation
- Type hints in Claude Desktop
- Improved developer experience

Previously the schema was empty { properties: {} }, now it fully
documents all available parameters with types and descriptions.

Files changed:
- src/tools/search.ts: Added complete inputSchema definition

Build: ✅ 0 errors
Validation: ✅ Manifest passes

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---
 src/tools/search.ts | 64 +++++++++++++++++++++++++++++++++++++++++++--
 1 file changed, 62 insertions(+), 2 deletions(-)

diff --git a/src/tools/search.ts b/src/tools/search.ts
index 5621ccf..f9be64d 100644
--- a/src/tools/search.ts
+++ b/src/tools/search.ts
@@ -21,10 +21,70 @@ export async function registerSearchTools(server: ObsidianMCPServer): Promise<vo
   server.registerTool(
     'obsidian_search',
     'Search vault for text. Returns matching files or optionally match counts using `total` flag. Supports path filtering, result limits, case sensitivity, and multiple output formats (text/json).',
-    { type: 'object', properties: {} },
+    {
+      type: 'object',
+      required: ['query'],
+      properties: {
+        query: {
+          type: 'string',
+          description: 'Search query text (required)',
+        },
+        path: {
+          type: 'string',
+          description: 'Limit search to folder path (optional)',
+        },
+        limit: {
+          type: 'number',
+          description: 'Maximum number of results to return (optional)',
+        },
+        total: {
+          type: 'boolean',
+          description: 'Return match count instead of file list (optional)',
+        },
+        case: {
+          type: 'boolean',
+          description: 'Case sensitive search (optional)',
+        },
+        format: {
+          type: 'string',
+          enum: ['text', 'json'],
+          description: 'Output format (optional, default: text)',
+        },
+      },
+    },
     createToolHandler(
       'Search vault for text',
-      { type: 'object', properties: {} },
+      {
+        type: 'object',
+        required: ['query'],
+        properties: {
+          query: {
+            type: 'string',
+            description: 'Search query text (required)',
+          },
+          path: {
+            type: 'string',
+            description: 'Limit search to folder path (optional)',
+          },
+          limit: {
+            type: 'number',
+            description: 'Maximum number of results to return (optional)',
+          },
+          total: {
+            type: 'boolean',
+            description: 'Return match count instead of file list (optional)',
+          },
+          case: {
+            type: 'boolean',
+            description: 'Case sensitive search (optional)',
+          },
+          format: {
+            type: 'string',
+            enum: ['text', 'json'],
+            description: 'Output format (optional, default: text)',
+          },
+        },
+      },
       async (args) => {
         const validated = searchSchema.parse(args) as any;
         const sanitized = sanitizeParameters(validated) as any;