@dpesch/mantisbt-mcp-server 1.9.0 → 1.9.1

package/CHANGELOG.md

@@ -7,7 +7,11 @@ This project adheres to [Semantic Versioning](https://semver.org/).
 
 ---
 
-## [Unreleased]
+## [1.9.1] – 2026-03-30
+
+### Fixed
+- Boolean parameters (`dry_run`, `highlight`, `check_latest`, `obsolete`, `inherit`) now accept the strings `"true"` and `"false"` in addition to native booleans. MCP clients that serialize all parameters as JSON strings no longer receive error -32602. Note: `z.coerce.boolean()` was intentionally not used — it would silently convert the string `"false"` to `true` via JavaScript's `Boolean()`.
+- `update_issue`: the `fields` parameter now accepts a JSON-encoded string in addition to a plain object. Invalid JSON is caught and surfaced as a Zod validation error instead of an uncaught `SyntaxError`.
 
 ---
 

package/dist/search/tools.js

@@ -24,6 +24,7 @@ function errorText(msg) {
 // ---------------------------------------------------------------------------
 // registerSearchTools
 // ---------------------------------------------------------------------------
+const coerceBool = (val) => val === 'true' ? true : val === 'false' ? false : val;
 export function registerSearchTools(server, client, store, embedder) {
     // ---------------------------------------------------------------------------
     // search_issues
@@ -45,7 +46,7 @@ export function registerSearchTools(server, client, store, embedder) {
                 'When provided, each matching issue is fetched from MantisBT and enriched with the requested fields. ' +
                 'The relevance score is always included. Without this parameter only id and score are returned.'),
             highlight: z
-                .boolean()
+                .preprocess(coerceBool, z.boolean())
                 .default(false)
                 .describe('If true, adds a "highlights" field per result with query terms bolded (**term**) ' +
                 'in the issue summary and a short description snippet. ' +

package/dist/tools/issues.js

@@ -342,6 +342,7 @@ export function registerIssueTools(server, client, cache) {
     // ---------------------------------------------------------------------------
     // update_issue
     // ---------------------------------------------------------------------------
+    const coerceBool = (val) => val === 'true' ? true : val === 'false' ? false : val;
     // MantisBT reference shape: at least one of id or name must be provided
     const ref = z.object({ id: z.number().optional(), name: z.string().optional() })
         .refine(o => o.id !== undefined || o.name !== undefined, { message: "At least one of 'id' or 'name' must be provided" });
@@ -369,8 +370,17 @@ The "fields" object accepts any combination of:
 Important: when resolving an issue, always set BOTH status and resolution to avoid leaving resolution as "open".`,
         inputSchema: z.object({
             id: z.coerce.number().int().positive().describe('Numeric issue ID to update'),
-            dry_run: z.boolean().optional().describe('If true, return the patch payload that would be sent without actually updating the issue. Useful for previewing changes before committing them.'),
-            fields: z.object({
+            dry_run: z.preprocess(coerceBool, z.boolean().optional()).describe('If true, return the patch payload that would be sent without actually updating the issue. Useful for previewing changes before committing them.'),
+            fields: z.preprocess((v) => {
+                if (typeof v !== 'string')
+                    return v;
+                try {
+                    return JSON.parse(v);
+                }
+                catch {
+                    return v;
+                }
+            }, z.object({
                 summary: z.string().optional(),
                 description: z.string().optional(),
                 steps_to_reproduce: z.string().optional(),
@@ -386,7 +396,7 @@ Important: when resolving an issue, always set BOTH status and resolution to avo
                 target_version: ref.optional(),
                 fixed_in_version: ref.optional(),
                 view_state: ref.optional(),
-            }).strict().describe('Fields to update (partial update — only provided fields are changed; unknown keys are rejected)'),
+            }).strict().describe('Fields to update (partial update — only provided fields are changed; unknown keys are rejected)')),
         }),
         annotations: {
             readOnlyHint: false,

package/dist/tools/projects.js

@@ -8,6 +8,7 @@ function errorText(msg) {
     const hint = vh?.getUpdateHint();
     return hint ? `Error: ${msg}\n\n${hint}` : `Error: ${msg}`;
 }
+const coerceBool = (val) => val === 'true' ? true : val === 'false' ? false : val;
 export function registerProjectTools(server, client, cache) {
     // ---------------------------------------------------------------------------
     // list_projects
@@ -72,8 +73,8 @@ export function registerProjectTools(server, client, cache) {
         description: 'List all versions defined for a MantisBT project.',
         inputSchema: z.object({
             project_id: z.coerce.number().int().positive().describe('Numeric project ID'),
-            obsolete: z.boolean().default(false).describe('Include obsolete (deprecated) versions (default: false)'),
-            inherit: z.boolean().default(false).describe('Include versions inherited from parent projects (default: false)'),
+            obsolete: z.preprocess(coerceBool, z.boolean()).default(false).describe('Include obsolete (deprecated) versions (default: false)'),
+            inherit: z.preprocess(coerceBool, z.boolean()).default(false).describe('Include versions inherited from parent projects (default: false)'),
         }),
         annotations: {
             readOnlyHint: true,

package/dist/tools/version.js

@@ -1,5 +1,6 @@
 import { z } from 'zod';
 import { parseVersion, compareVersions } from '../version-hint.js';
+const coerceBool = (val) => val === 'true' ? true : val === 'false' ? false : val;
 export function registerVersionTools(server, client, versionHint, mcpVersion) {
     // ---------------------------------------------------------------------------
     // get_mcp_version
@@ -26,7 +27,7 @@ export function registerVersionTools(server, client, versionHint, mcpVersion) {
 The version is read from the X-Mantis-Version response header sent by every API call.
 The GitHub comparison requires an outbound HTTPS request to the GitHub API.`,
         inputSchema: z.object({
-            check_latest: z.boolean().default(true).describe('Whether to fetch the latest release from GitHub and compare (default: true)'),
+            check_latest: z.preprocess(coerceBool, z.boolean()).default(true).describe('Whether to fetch the latest release from GitHub and compare (default: true)'),
         }),
         annotations: {
             readOnlyHint: true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dpesch/mantisbt-mcp-server",
-  "version": "1.9.0",
+  "version": "1.9.1",
   "mcpName": "io.github.dpesch/mantisbt-mcp-server",
   "description": "MCP server for MantisBT REST API – read and manage bug tracker issues",
   "author": "Dominik Pesch",

package/server.json

@@ -3,12 +3,12 @@
   "name": "io.github.dpesch/mantisbt-mcp-server",
   "title": "MantisBT MCP Server",
   "description": "MantisBT MCP server – manage issues, notes, files, tags, and relationships. With semantic search.",
-  "version": "1.9.0",
+  "version": "1.9.1",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "@dpesch/mantisbt-mcp-server",
-      "version": "1.9.0",
+      "version": "1.9.1",
       "runtimeHint": "npx",
       "transport": {
         "type": "stdio"

package/tests/search/tools.test.ts

@@ -630,3 +630,31 @@ describe('search_issues – highlight: true combined with date filter', () => {
     expect(parsed[0]!.id).toBe(1);
   });
 });
+
+// ---------------------------------------------------------------------------
+// string-coercion – highlight as string
+// ---------------------------------------------------------------------------
+
+describe('string-coercion – search_issues highlight as string', () => {
+  it('accepts highlight "true" as boolean true', async () => {
+    const store = makeMockStore({
+      items: [{ id: 1, score: 0.9 }],
+    });
+    vi.mocked(store.getItem).mockResolvedValue({
+      id: 1,
+      vector: [],
+      metadata: { summary: 'Login error occurred', description: 'The login fails.' },
+    });
+    registerSearchTools(mockServer as never, client, store, embedder);
+
+    const result = await mockServer.callTool(
+      'search_issues',
+      { query: 'login error', top_n: 1, highlight: 'true' },
+      { validate: true },
+    );
+
+    expect(result.isError).toBeUndefined();
+    const parsed = JSON.parse(result.content[0]!.text) as Array<Record<string, unknown>>;
+    expect(parsed[0]).toHaveProperty('highlights');
+  });
+});

package/tests/tools/string-coercion.test.ts

@@ -19,6 +19,8 @@ import { registerMonitorTools } from '../../src/tools/monitors.js';
 import { registerRelationshipTools } from '../../src/tools/relationships.js';
 import { registerTagTools } from '../../src/tools/tags.js';
 import { registerProjectTools } from '../../src/tools/projects.js';
+import { registerVersionTools } from '../../src/tools/version.js';
+import { VersionHintService } from '../../src/version-hint.js';
 import { MockMcpServer, makeResponse } from '../helpers/mock-server.js';
 
 // ---------------------------------------------------------------------------
@@ -39,6 +41,7 @@ beforeEach(() => {
   registerRelationshipTools(mockServer as never, client);
   registerTagTools(mockServer as never, client);
   registerProjectTools(mockServer as never, client);
+  registerVersionTools(mockServer as never, client, new VersionHintService(), '0.0.0-test');
   vi.stubGlobal('fetch', vi.fn());
 });
 
@@ -251,3 +254,64 @@ describe('string-coercion – get_project_versions', () => {
     expect(result.isError).toBeUndefined();
   });
 });
+
+// ---------------------------------------------------------------------------
+// Boolean-Parameter als String
+// ---------------------------------------------------------------------------
+
+describe('string-coercion – update_issue fields as JSON string', () => {
+  it('accepts fields as JSON string', async () => {
+    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify({ issue: { id: 99, summary: 'Updated' } })));
+    const result = await mockServer.callTool(
+      'update_issue',
+      { id: 99, fields: '{"summary":"Updated"}' },
+      { validate: true },
+    );
+    expect(result.isError).toBeUndefined();
+  });
+
+  it('rejects invalid JSON in fields with a validation error (not an uncaught exception)', async () => {
+    const result = await mockServer.callTool(
+      'update_issue',
+      { id: 99, fields: '{invalid json' },
+      { validate: true },
+    );
+    expect(result.isError).toBe(true);
+  });
+});
+
+describe('string-coercion – update_issue dry_run as string', () => {
+  it('accepts dry_run "true" as boolean true', async () => {
+    const result = await mockServer.callTool(
+      'update_issue',
+      { id: 99, fields: { summary: 'x' }, dry_run: 'true' },
+      { validate: true },
+    );
+    expect(result.isError).toBeUndefined();
+    expect(result.content[0]?.text).toContain('"dry_run": true');
+  });
+});
+
+describe('string-coercion – get_project_versions boolean flags', () => {
+  it('accepts obsolete "true" and inherit "false" as booleans', async () => {
+    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify({ projects: [{ versions: [] }] })));
+    const result = await mockServer.callTool(
+      'get_project_versions',
+      { project_id: 3, obsolete: 'true', inherit: 'false' },
+      { validate: true },
+    );
+    expect(result.isError).toBeUndefined();
+  });
+});
+
+describe('string-coercion – get_mantis_version check_latest as string', () => {
+  it('accepts check_latest "false" as boolean false', async () => {
+    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify({ version: '2.26.0' })));
+    const result = await mockServer.callTool(
+      'get_mantis_version',
+      { check_latest: 'false' },
+      { validate: true },
+    );
+    expect(result.isError).toBeUndefined();
+  });
+});