@dpesch/mantisbt-mcp-server 1.3.0 → 1.3.1

package/CHANGELOG.md

@@ -7,6 +7,13 @@ This project adheres to [Semantic Versioning](https://semver.org/).
 
 ---
 
+## [1.3.1] – 2026-03-16
+
+### Fixed
+- `get_issue_enums` returned an empty object `{}` on MantisBT 2.x installations. The MantisBT REST API returns enum config values as pre-parsed `[{id, name, label}]` arrays, not as legacy `"id:name,..."` strings. The handler now covers both formats; the `label` field is stripped from the output.
+
+---
+
 ## [1.3.0] – 2026-03-16
 
 ### Added

package/dist/constants.js

@@ -15,6 +15,17 @@ export const RELATIONSHIP_TYPES = {
 // MantisBT default status ID for "resolved". Issues with status.id strictly
 // below this value are considered open (new/feedback/acknowledged/confirmed/assigned).
 export const MANTIS_RESOLVED_STATUS_ID = 80;
+// ---------------------------------------------------------------------------
+// Issue enum config option names
+// ---------------------------------------------------------------------------
+export const ISSUE_ENUM_OPTIONS = [
+    'severity_enum_string',
+    'status_enum_string',
+    'priority_enum_string',
+    'resolution_enum_string',
+    'reproducibility_enum_string',
+];
+// ---------------------------------------------------------------------------
 export const STATUS_NAMES = [
     'new',
     'feedback',

package/dist/tools/config.js

@@ -1,4 +1,5 @@
 import { z } from 'zod';
+import { ISSUE_ENUM_OPTIONS } from '../constants.js';
 import { getVersionHint } from '../version-hint.js';
 function errorText(msg) {
     const vh = getVersionHint();
@@ -88,15 +89,8 @@ The "name" field is the value to pass to create_issue or update_issue.`,
         },
     }, async () => {
         try {
-            const enumOptions = [
-                'severity_enum_string',
-                'status_enum_string',
-                'priority_enum_string',
-                'resolution_enum_string',
-                'reproducibility_enum_string',
-            ];
             const params = {};
-            enumOptions.forEach((opt, i) => {
+            ISSUE_ENUM_OPTIONS.forEach((opt, i) => {
                 params[`option[${i}]`] = opt;
             });
             const result = await client.get('config', params);
@@ -111,9 +105,14 @@ The "name" field is the value to pass to create_issue or update_issue.`,
             const enums = {};
             for (const { option, value } of configs) {
                 const key = keyMap[option];
-                if (key && typeof value === 'string') {
+                if (!key)
+                    continue;
+                if (typeof value === 'string') {
                     enums[key] = parseEnumString(value);
                 }
+                else if (Array.isArray(value)) {
+                    enums[key] = value.map(({ id, name }) => ({ id, name }));
+                }
             }
             return {
                 content: [{ type: 'text', text: JSON.stringify(enums, null, 2) }],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dpesch/mantisbt-mcp-server",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "MCP server for MantisBT REST API – read and manage bug tracker issues",
   "author": "Dominik Pesch",
   "license": "MIT",

package/scripts/record-fixtures.ts

@@ -2,6 +2,7 @@ import { writeFileSync, mkdirSync, readFileSync } from 'node:fs';
 import { join, dirname } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { MantisClient } from '../src/client.js';
+import { ISSUE_ENUM_OPTIONS } from '../src/constants.js';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
@@ -124,6 +125,16 @@ async function recordFixtures(): Promise<void> {
     }
   }
 
+  // GET config — enum values for severity, status, priority, resolution, reproducibility
+  try {
+    const enumParams: Record<string, string> = {};
+    ISSUE_ENUM_OPTIONS.forEach((opt, i) => { enumParams[`option[${i}]`] = opt; });
+    const configResult = await client.get<unknown>('config', enumParams);
+    saveFixture('get_issue_enums.json', configResult);
+  } catch (err) {
+    console.error('Failed to fetch config enums:', err instanceof Error ? err.message : String(err));
+  }
+
   // GET projects/{id}/versions + categories
   if (firstProjectId !== undefined) {
     const versionProjectId = projectIdWithVersions ?? firstProjectId;

package/tests/fixtures/get_issue_enums.json

@@ -0,0 +1,67 @@
+{
+  "configs": [
+    {
+      "option": "severity_enum_string",
+      "value": [
+        { "id": 10, "name": "Feature-Wunsch", "label": "Feature-Wunsch" },
+        { "id": 20, "name": "Trivial", "label": "Trivial" },
+        { "id": 30, "name": "Fehler im Text", "label": "Fehler im Text" },
+        { "id": 40, "name": "Unschönheit", "label": "Unschönheit" },
+        { "id": 50, "name": "kleinerer Fehler", "label": "kleinerer Fehler" },
+        { "id": 60, "name": "schwerer Fehler", "label": "schwerer Fehler" },
+        { "id": 70, "name": "Absturz", "label": "Absturz" },
+        { "id": 80, "name": "Blocker", "label": "Blocker" },
+        { "id": 200, "name": "Technische Schuld", "label": "Technische Schuld" },
+        { "id": 210, "name": "Wartung", "label": "Wartung" }
+      ]
+    },
+    {
+      "option": "status_enum_string",
+      "value": [
+        { "id": 10, "name": "new", "label": "neu" },
+        { "id": 20, "name": "feedback", "label": "Rückmeldung" },
+        { "id": 30, "name": "acknowledged", "label": "anerkannt" },
+        { "id": 40, "name": "confirmed", "label": "bestätigt" },
+        { "id": 50, "name": "assigned", "label": "zugewiesen" },
+        { "id": 80, "name": "resolved", "label": "erledigt" },
+        { "id": 90, "name": "closed", "label": "geschlossen" }
+      ]
+    },
+    {
+      "option": "priority_enum_string",
+      "value": [
+        { "id": 10, "name": "none", "label": "keine" },
+        { "id": 20, "name": "low", "label": "niedrig" },
+        { "id": 30, "name": "normal", "label": "normal" },
+        { "id": 40, "name": "high", "label": "hoch" },
+        { "id": 50, "name": "urgent", "label": "dringend" },
+        { "id": 60, "name": "immediate", "label": "sofort" }
+      ]
+    },
+    {
+      "option": "resolution_enum_string",
+      "value": [
+        { "id": 10, "name": "open", "label": "offen" },
+        { "id": 20, "name": "fixed", "label": "erledigt" },
+        { "id": 30, "name": "reopened", "label": "wiedereröffnet" },
+        { "id": 40, "name": "unable to duplicate", "label": "nicht reproduzierbar" },
+        { "id": 50, "name": "not fixable", "label": "unlösbar" },
+        { "id": 60, "name": "duplicate", "label": "doppelt" },
+        { "id": 70, "name": "not a bug", "label": "keine Änderung notwendig" },
+        { "id": 80, "name": "suspended", "label": "aufgeschoben" },
+        { "id": 90, "name": "wont fix", "label": "wird nicht behoben" }
+      ]
+    },
+    {
+      "option": "reproducibility_enum_string",
+      "value": [
+        { "id": 10, "name": "always", "label": "immer" },
+        { "id": 30, "name": "sometimes", "label": "manchmal" },
+        { "id": 50, "name": "random", "label": "zufällig" },
+        { "id": 70, "name": "have not tried", "label": "nicht getestet" },
+        { "id": 90, "name": "unable to duplicate", "label": "nicht reproduzierbar" },
+        { "id": 100, "name": "N/A", "label": "N/A" }
+      ]
+    }
+  ]
+}

package/tests/tools/config.test.ts

@@ -1,23 +1,39 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { readFileSync } from 'node:fs';
+import { join, dirname } from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { MantisClient } from '../../src/client.js';
 import { MetadataCache } from '../../src/cache.js';
 import { registerConfigTools } from '../../src/tools/config.js';
 import { MockMcpServer, makeResponse } from '../helpers/mock-server.js';
 
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+const fixturesDir = join(__dirname, '..', 'fixtures');
+
 // ---------------------------------------------------------------------------
-// Setup
+// Fixtures
 // ---------------------------------------------------------------------------
 
-const ENUM_FIXTURE = {
+const enumFixture = JSON.parse(
+  readFileSync(join(fixturesDir, 'get_issue_enums.json'), 'utf-8')
+) as { configs: Array<{ option: string; value: Array<{ id: number; name: string; label: string }> }> };
+
+// Legacy: some MantisBT versions return value as a comma-separated "id:name" string
+const ENUM_FIXTURE_STRING = {
   configs: [
-    { option: 'severity_enum_string',       value: '10:feature,20:trivial,30:text,40:tweak,50:minor,60:major,70:crash,80:block' },
-    { option: 'status_enum_string',         value: '10:new,20:feedback,30:acknowledged,40:confirmed,50:assigned,80:resolved,90:closed' },
-    { option: 'priority_enum_string',       value: '10:none,20:low,30:normal,40:high,50:urgent,60:immediate' },
-    { option: 'resolution_enum_string',     value: '10:open,20:fixed,30:reopened,40:unable to duplicate,50:not fixable,60:duplicate,70:no change required,80:suspended,90:wont fix' },
-    { option: 'reproducibility_enum_string', value: '10:always,30:sometimes,50:random,70:have not tried,90:unable to reproduce,100:N/A' },
+    { option: 'severity_enum_string',        value: '10:feature,50:minor,80:block' },
+    { option: 'status_enum_string',          value: '10:new,80:resolved,90:closed' },
+    { option: 'priority_enum_string',        value: '10:none,30:normal,60:immediate' },
+    { option: 'resolution_enum_string',      value: '10:open,20:fixed' },
+    { option: 'reproducibility_enum_string', value: '10:always,70:have not tried' },
   ],
 };
 
+// ---------------------------------------------------------------------------
+// Setup
+// ---------------------------------------------------------------------------
+
 let mockServer: MockMcpServer;
 let client: MantisClient;
 let cache: MetadataCache;
@@ -44,7 +60,7 @@ describe('get_issue_enums', () => {
   });
 
   it('gibt strukturierte Enum-Arrays für alle 5 Felder zurück', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify(ENUM_FIXTURE)));
+    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify(enumFixture)));
 
     const result = await mockServer.callTool('get_issue_enums', {});
 
@@ -58,23 +74,39 @@ describe('get_issue_enums', () => {
     expect(Array.isArray(parsed.reproducibility)).toBe(true);
   });
 
-  it('parst id und name korrekt', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify(ENUM_FIXTURE)));
+  it('parst Array-Format (MantisBT 2.x): id und name korrekt, label wird verworfen', async () => {
+    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify(enumFixture)));
 
     const result = await mockServer.callTool('get_issue_enums', {});
 
     const parsed = JSON.parse(result.content[0]!.text) as Record<string, Array<{ id: number; name: string }>>;
 
-    expect(parsed.severity).toContainEqual({ id: 50, name: 'minor' });
-    expect(parsed.severity).toContainEqual({ id: 80, name: 'block' });
+    // Werte aus der echten Fixture prüfen
+    expect(parsed.severity).toContainEqual({ id: 50, name: 'kleinerer Fehler' });
+    expect(parsed.severity).toContainEqual({ id: 80, name: 'Blocker' });
+    expect(parsed.severity).toContainEqual({ id: 200, name: 'Technische Schuld' });
     expect(parsed.status).toContainEqual({ id: 10, name: 'new' });
     expect(parsed.status).toContainEqual({ id: 80, name: 'resolved' });
     expect(parsed.priority).toContainEqual({ id: 30, name: 'normal' });
     expect(parsed.reproducibility).toContainEqual({ id: 70, name: 'have not tried' });
+    // label darf nicht im Output enthalten sein
+    expect(Object.keys(parsed.severity[0]!)).toEqual(['id', 'name']);
+  });
+
+  it('parst String-Format (Legacy): "id:name,..."-Strings korrekt', async () => {
+    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify(ENUM_FIXTURE_STRING)));
+
+    const result = await mockServer.callTool('get_issue_enums', {});
+
+    const parsed = JSON.parse(result.content[0]!.text) as Record<string, Array<{ id: number; name: string }>>;
+
+    expect(parsed.severity).toContainEqual({ id: 50, name: 'minor' });
+    expect(parsed.status).toContainEqual({ id: 80, name: 'resolved' });
+    expect(parsed.reproducibility).toContainEqual({ id: 70, name: 'have not tried' });
   });
 
   it('fragt alle 5 Enum-Optionen ab', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify(ENUM_FIXTURE)));
+    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify(enumFixture)));
 
     await mockServer.callTool('get_issue_enums', {});