tgrep-mcp 0.3.0 → 0.3.1

package/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2026 Shizhy
+Copyright (c) 2026 tgrep-mcp contributors
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/README.md

@@ -47,7 +47,7 @@ Two details matter:
   MCP handshake stalls.
 - Set `cwd` to the repository you want to search. See [Search root](#search-root).
 
-To pin a version, use `"args": ["-y", "tgrep-mcp@0.3.0"]`.
+To pin a version, use `"args": ["-y", "tgrep-mcp@0.3.1"]`.
 
 To run from a checkout instead:
 

package/package.json

@@ -1,9 +1,8 @@
 {
   "name": "tgrep-mcp",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "MCP server that exposes tgrep, a trigram-indexed regex search tool, over stdio.",
   "license": "MIT",
-  "author": "Shizhy <usstpeter@163.com>",
   "type": "module",
   "bin": {
     "tgrep-mcp": "server.mjs"

package/server.mjs

@@ -71,7 +71,7 @@ function resolveVersion() {
   } catch {
     // no manifest available
   }
-  return '0.3.0'
+  return '0.3.1'
 }
 
 const VERSION = resolveVersion()
@@ -1408,29 +1408,30 @@ const ERROR_SCHEMA = Object.freeze({
 })
 
 /**
- * The three output schemas below are `{ oneOf: [...] }` with NO sibling keywords, on purpose.
+ * Every output schema is one root-level `{ type: 'object' }` with `ok` as the only required field.
  *
- * The harness enforces a JSON Schema subset (dsh-tools assertSupportedJsonSchema, reached through
- * dsh-mcp-client's supportedOutputSchema) that rejects a node declaring `type`, `properties`,
- * `required`, `additionalProperties`, `items`, `enum` or `const` *beside* `oneOf` — and a rejected
- * schema is silently discarded, which would quietly relax structuredContent from required back to
- * optional without anyone noticing. So every branch is a complete object schema standing alone, and
- * the discriminating `const` carries its own `type` (the subset requires a type-correct const).
+ * Two constraints meet here and rule out the obvious `{ type: 'object', oneOf: [...] }` shape:
+ *   - The MCP spec, enforced by the SDK's ToolSchema, requires `type: 'object'` at the root of
+ *     `outputSchema`. A schema without it makes the client reject the entire tools/list result, so
+ *     the server connects but registers no tools at all.
+ *   - The harness's enforced subset (dsh-tools assertSupportedJsonSchema, reached through
+ *     dsh-mcp-client's supportedOutputSchema) rejects `type` beside `oneOf`, and silently drops the
+ *     schema, which relaxes structuredContent from required back to optional.
+ * Because no union keyword satisfies both, the union is flattened rather than nested: branch-specific
+ * fields become optional siblings of `ok`. Success and error payloads both validate; the per-branch
+ * strictness that `oneOf` would have given is lost.
  */
-const OK_BRANCH = (properties, required) => ({
+const outputSchema = (properties) => Object.freeze({
   type: 'object',
-  properties: { ok: { type: 'boolean', const: true }, ...properties },
-  required: ['ok', ...required],
+  properties: {
+    ok: { type: 'boolean', description: 'True on success; false when the result carries an error.' },
+    error: ERROR_SCHEMA,
+    ...properties,
+  },
+  required: ['ok'],
   additionalProperties: false,
 })
 
-const ERROR_BRANCH = {
-  type: 'object',
-  properties: { ok: { type: 'boolean', const: false }, error: ERROR_SCHEMA },
-  required: ['ok', 'error'],
-  additionalProperties: false,
-}
-
 const SEARCH_SUCCESS_PROPERTIES = {
   root: { type: 'string', description: 'Indexed root the search ran against.' },
   target: { type: 'string', description: 'Exact file or directory searched.' },
@@ -1452,52 +1453,37 @@ const SEARCH_SUCCESS_PROPERTIES = {
   warnings: { type: 'array', items: { type: 'string' } },
 }
 
-const SEARCH_OUTPUT_SCHEMA = Object.freeze({
-  oneOf: [
-    OK_BRANCH(SEARCH_SUCCESS_PROPERTIES, ['root', 'target', 'kind', 'backend', 'elapsedMs', 'total', 'files', 'returned', 'truncated', 'matches']),
-    ERROR_BRANCH,
-  ],
-})
+const SEARCH_OUTPUT_SCHEMA = outputSchema(SEARCH_SUCCESS_PROPERTIES)
 
-const INDEX_OUTPUT_SCHEMA = Object.freeze({
-  oneOf: [
-    OK_BRANCH({
-      root: { type: 'string' },
-      status: { type: 'string', enum: ['ready', 'failed'] },
-      builtMs: { type: 'number' },
-      indexDir: { type: 'string' },
-      indexAgeMs: { type: 'number' },
-      daemonRestarted: { type: 'boolean' },
-    }, ['root', 'status', 'indexDir']),
-    ERROR_BRANCH,
-  ],
+const INDEX_OUTPUT_SCHEMA = outputSchema({
+  root: { type: 'string' },
+  status: { type: 'string', enum: ['ready', 'failed'] },
+  builtMs: { type: 'number' },
+  indexDir: { type: 'string' },
+  indexAgeMs: { type: 'number' },
+  daemonRestarted: { type: 'boolean' },
 })
 
-const STATUS_OUTPUT_SCHEMA = Object.freeze({
-  oneOf: [
-    OK_BRANCH({
-      status: { type: 'string', enum: ['healthy', 'degraded', 'unhealthy'] },
-      version: { type: 'string' },
-      checks: {
-        type: 'array',
-        items: {
-          type: 'object',
-          properties: {
-            name: { type: 'string' },
-            status: { type: 'string', enum: ['healthy', 'degraded', 'unhealthy'] },
-            message: { type: 'string' },
-            responseTimeMs: { type: 'number' },
-            details: { type: 'object' },
-          },
-          required: ['name', 'status', 'message'],
-          additionalProperties: false,
-        },
+const STATUS_OUTPUT_SCHEMA = outputSchema({
+  status: { type: 'string', enum: ['healthy', 'degraded', 'unhealthy'] },
+  version: { type: 'string' },
+  checks: {
+    type: 'array',
+    items: {
+      type: 'object',
+      properties: {
+        name: { type: 'string' },
+        status: { type: 'string', enum: ['healthy', 'degraded', 'unhealthy'] },
+        message: { type: 'string' },
+        responseTimeMs: { type: 'number' },
+        details: { type: 'object' },
       },
-      config: { type: 'object' },
-      metrics: { type: 'object' },
-    }, ['status', 'version', 'checks', 'config', 'metrics']),
-    ERROR_BRANCH,
-  ],
+      required: ['name', 'status', 'message'],
+      additionalProperties: false,
+    },
+  },
+  config: { type: 'object' },
+  metrics: { type: 'object' },
 })
 
 /**