@dpesch/mantisbt-mcp-server 1.10.3 → 1.11.0

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

@@ -1,317 +0,0 @@
-/**
- * Regression tests for string-to-number coercion in MCP tool schemas.
- *
- * The MCP protocol allows clients to pass numeric IDs as strings (e.g. "1940"
- * instead of 1940). Without z.coerce.number(), the Zod schema would reject
- * these inputs with error -32602 (Invalid params).
- *
- * These tests use { validate: true } to run args through the Zod schema —
- * exactly as the real MCP server does — before passing them to the handler.
- */
-
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { MantisClient } from '../../src/client.js';
-import { registerIssueTools } from '../../src/tools/issues.js';
-import { MetadataCache } from '../../src/cache.js';
-import { registerNoteTools } from '../../src/tools/notes.js';
-import { registerFileTools } from '../../src/tools/files.js';
-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';
-
-// ---------------------------------------------------------------------------
-// Setup
-// ---------------------------------------------------------------------------
-
-let mockServer: MockMcpServer;
-let client: MantisClient;
-
-beforeEach(() => {
-  mockServer = new MockMcpServer();
-  client = new MantisClient('https://mantis.example.com', 'test-token');
-  const stubCache = { loadIfValid: vi.fn(async () => null) } as unknown as MetadataCache;
-  registerIssueTools(mockServer as never, client, stubCache);
-  registerNoteTools(mockServer as never, client);
-  registerFileTools(mockServer as never, client);
-  registerMonitorTools(mockServer as never, client);
-  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());
-});
-
-afterEach(() => {
-  vi.unstubAllGlobals();
-  vi.clearAllMocks();
-});
-
-// ---------------------------------------------------------------------------
-// Issues
-// ---------------------------------------------------------------------------
-
-describe('string-coercion – get_issue', () => {
-  it('accepts issue_id as string "1940"', async () => {
-    vi.mocked(fetch).mockResolvedValue(
-      makeResponse(200, JSON.stringify({ issues: [{ id: 1940, summary: 'Test' }] }))
-    );
-
-    const result = await mockServer.callTool('get_issue', { id: '1940' }, { validate: true });
-
-    expect(result.isError).toBeUndefined();
-    const calledUrl = vi.mocked(fetch).mock.calls[0]![0] as string;
-    expect(calledUrl).toContain('1940');
-  });
-
-  it('rejects a non-numeric string', async () => {
-    const result = await mockServer.callTool('get_issue', { id: 'abc' }, { validate: true });
-    expect(result.isError).toBe(true);
-  });
-});
-
-describe('string-coercion – list_issues', () => {
-  it('accepts project_id as string', async () => {
-    vi.mocked(fetch).mockResolvedValue(
-      makeResponse(200, JSON.stringify({ issues: [], total_count: 0 }))
-    );
-
-    const result = await mockServer.callTool(
-      'list_issues',
-      { project_id: '5', page: '1', page_size: '10' },
-      { validate: true },
-    );
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-describe('string-coercion – update_issue', () => {
-  it('accepts id as 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();
-  });
-});
-
-describe('string-coercion – delete_issue', () => {
-  it('accepts id as string', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(204, ''));
-
-    const result = await mockServer.callTool('delete_issue', { id: '42' }, { validate: true });
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Notes
-// ---------------------------------------------------------------------------
-
-describe('string-coercion – list_notes', () => {
-  it('accepts issue_id as string', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify({ issues: [{ notes: [] }] })));
-
-    const result = await mockServer.callTool('list_notes', { issue_id: '1940' }, { validate: true });
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-describe('string-coercion – delete_note', () => {
-  it('accepts issue_id and note_id as strings', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(204, ''));
-
-    const result = await mockServer.callTool(
-      'delete_note',
-      { issue_id: '1940', note_id: '77' },
-      { validate: true },
-    );
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Files
-// ---------------------------------------------------------------------------
-
-describe('string-coercion – list_issue_files', () => {
-  it('accepts issue_id as string', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify({ issues: [{ attachments: [] }] })));
-
-    const result = await mockServer.callTool('list_issue_files', { issue_id: '1940' }, { validate: true });
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Monitors
-// ---------------------------------------------------------------------------
-
-describe('string-coercion – add_monitor', () => {
-  it('accepts issue_id as string', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(200, ''));
-
-    const result = await mockServer.callTool(
-      'add_monitor',
-      { issue_id: '1940', username: 'jdoe' },
-      { validate: true },
-    );
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-describe('string-coercion – remove_monitor', () => {
-  it('accepts issue_id as string', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(204, ''));
-
-    const result = await mockServer.callTool(
-      'remove_monitor',
-      { issue_id: '1940', username: 'jdoe' },
-      { validate: true },
-    );
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Relationships
-// ---------------------------------------------------------------------------
-
-describe('string-coercion – add_relationship', () => {
-  it('accepts issue_id, target_id and type_id as strings', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify({ id: 1 })));
-
-    const result = await mockServer.callTool(
-      'add_relationship',
-      { issue_id: '1940', target_id: '1941', type_id: '1' },
-      { validate: true },
-    );
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-describe('string-coercion – remove_relationship', () => {
-  it('accepts issue_id and relationship_id as strings', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(204, ''));
-
-    const result = await mockServer.callTool(
-      'remove_relationship',
-      { issue_id: '1940', relationship_id: '55' },
-      { validate: true },
-    );
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Tags
-// ---------------------------------------------------------------------------
-
-describe('string-coercion – detach_tag', () => {
-  it('accepts issue_id and tag_id as strings', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(204, ''));
-
-    const result = await mockServer.callTool(
-      'detach_tag',
-      { issue_id: '1940', tag_id: '7' },
-      { validate: true },
-    );
-
-    expect(result.isError).toBeUndefined();
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Projects
-// ---------------------------------------------------------------------------
-
-describe('string-coercion – get_project_versions', () => {
-  it('accepts project_id as string', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify({ projects: [{ versions: [] }] })));
-
-    const result = await mockServer.callTool(
-      'get_project_versions',
-      { project_id: '3' },
-      { validate: true },
-    );
-
-    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();
-  });
-});

package/tests/tools/users.test.ts

@@ -1,62 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { readFileSync, existsSync } from 'node:fs';
-import { join, dirname } from 'node:path';
-import { fileURLToPath } from 'node:url';
-import { MantisClient } from '../../src/client.js';
-import { registerUserTools } from '../../src/tools/users.js';
-import { MockMcpServer, makeResponse } from '../helpers/mock-server.js';
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-const fixturesDir = join(__dirname, '..', 'fixtures');
-
-// ---------------------------------------------------------------------------
-// Fixtures laden (mit Inline-Fallback)
-// ---------------------------------------------------------------------------
-
-const getCurrentUserFixturePath = join(fixturesDir, 'get_current_user.json');
-
-const getCurrentUserFixture = existsSync(getCurrentUserFixturePath)
-  ? (JSON.parse(readFileSync(getCurrentUserFixturePath, 'utf-8')) as { id: number; name: string; real_name?: string; email?: string })
-  : { id: 5, name: 'testuser', real_name: 'Test User', email: 'test@example.com' };
-
-// ---------------------------------------------------------------------------
-// Setup
-// ---------------------------------------------------------------------------
-
-let mockServer: MockMcpServer;
-let client: MantisClient;
-
-beforeEach(() => {
-  mockServer = new MockMcpServer();
-  client = new MantisClient('https://mantis.example.com', 'test-token');
-  registerUserTools(mockServer as never, client);
-  vi.stubGlobal('fetch', vi.fn());
-});
-
-afterEach(() => {
-  vi.unstubAllGlobals();
-});
-
-// ---------------------------------------------------------------------------
-// get_current_user
-// ---------------------------------------------------------------------------
-
-describe('get_current_user', () => {
-  it('ist registriert', () => {
-    expect(mockServer.hasToolRegistered('get_current_user')).toBe(true);
-  });
-
-  it('gibt User-Daten zurück mit id und name', async () => {
-    vi.mocked(fetch).mockResolvedValue(makeResponse(200, JSON.stringify(getCurrentUserFixture)));
-
-    const result = await mockServer.callTool('get_current_user', {});
-
-    expect(result.isError).toBeUndefined();
-    const parsed = JSON.parse(result.content[0]!.text) as { id: number; name: string };
-    expect(typeof parsed.id).toBe('number');
-    expect(typeof parsed.name).toBe('string');
-    expect(parsed.id).toBe(getCurrentUserFixture.id);
-    expect(parsed.name).toBe(getCurrentUserFixture.name);
-  });
-});

package/tests/utils/date-filter.test.ts

@@ -1,169 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { matchesDateFilter } from '../../src/date-filter.js';
-
-describe('matchesDateFilter', () => {
-
-  // ---------------------------------------------------------------------------
-  // No filter → always pass
-  // ---------------------------------------------------------------------------
-
-  it('returns true when no filter is set', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-20T10:00:00Z', created_at: '2026-03-01T00:00:00Z' },
-      {}
-    )).toBe(true);
-  });
-
-  it('returns true for an item with no dates when no filter is set', () => {
-    expect(matchesDateFilter({}, {})).toBe(true);
-  });
-
-  // ---------------------------------------------------------------------------
-  // updated_after
-  // ---------------------------------------------------------------------------
-
-  it('updated_after: passes when updated_at is after the threshold', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-25T12:00:00Z' },
-      { updated_after: '2026-03-24T00:00:00Z' }
-    )).toBe(true);
-  });
-
-  it('updated_after: fails when updated_at is before the threshold', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-23T12:00:00Z' },
-      { updated_after: '2026-03-24T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  it('updated_after: fails when updated_at equals the threshold (exclusive)', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-24T00:00:00Z' },
-      { updated_after: '2026-03-24T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  it('updated_after: fails when updated_at is missing', () => {
-    expect(matchesDateFilter(
-      { created_at: '2026-03-25T00:00:00Z' },
-      { updated_after: '2026-03-24T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  // ---------------------------------------------------------------------------
-  // updated_before
-  // ---------------------------------------------------------------------------
-
-  it('updated_before: passes when updated_at is before the threshold', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-20T00:00:00Z' },
-      { updated_before: '2026-03-25T00:00:00Z' }
-    )).toBe(true);
-  });
-
-  it('updated_before: fails when updated_at is after the threshold', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-26T00:00:00Z' },
-      { updated_before: '2026-03-25T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  it('updated_before: fails when updated_at equals the threshold (exclusive)', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-25T00:00:00Z' },
-      { updated_before: '2026-03-25T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  it('updated_before: fails when updated_at is missing', () => {
-    expect(matchesDateFilter(
-      {},
-      { updated_before: '2026-03-25T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  // ---------------------------------------------------------------------------
-  // created_after
-  // ---------------------------------------------------------------------------
-
-  it('created_after: passes when created_at is after the threshold', () => {
-    expect(matchesDateFilter(
-      { created_at: '2026-03-25T12:00:00Z' },
-      { created_after: '2026-03-24T00:00:00Z' }
-    )).toBe(true);
-  });
-
-  it('created_after: fails when created_at is before the threshold', () => {
-    expect(matchesDateFilter(
-      { created_at: '2026-03-23T00:00:00Z' },
-      { created_after: '2026-03-24T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  it('created_after: fails when created_at is missing', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-25T00:00:00Z' },
-      { created_after: '2026-03-24T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  // ---------------------------------------------------------------------------
-  // created_before
-  // ---------------------------------------------------------------------------
-
-  it('created_before: passes when created_at is before the threshold', () => {
-    expect(matchesDateFilter(
-      { created_at: '2026-03-20T00:00:00Z' },
-      { created_before: '2026-03-25T00:00:00Z' }
-    )).toBe(true);
-  });
-
-  it('created_before: fails when created_at is after the threshold', () => {
-    expect(matchesDateFilter(
-      { created_at: '2026-03-26T00:00:00Z' },
-      { created_before: '2026-03-25T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  // ---------------------------------------------------------------------------
-  // Combined filters (time window)
-  // ---------------------------------------------------------------------------
-
-  it('updated_after + updated_before: passes when updated_at is within window', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-22T00:00:00Z' },
-      { updated_after: '2026-03-20T00:00:00Z', updated_before: '2026-03-25T00:00:00Z' }
-    )).toBe(true);
-  });
-
-  it('updated_after + updated_before: fails when updated_at is outside window', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-26T00:00:00Z' },
-      { updated_after: '2026-03-20T00:00:00Z', updated_before: '2026-03-25T00:00:00Z' }
-    )).toBe(false);
-  });
-
-  it('all four filters: passes only when both dates are within their windows', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-22T00:00:00Z', created_at: '2026-03-10T00:00:00Z' },
-      {
-        updated_after: '2026-03-20T00:00:00Z',
-        updated_before: '2026-03-25T00:00:00Z',
-        created_after:  '2026-03-05T00:00:00Z',
-        created_before: '2026-03-15T00:00:00Z',
-      }
-    )).toBe(true);
-  });
-
-  it('all four filters: fails when one date is out of range', () => {
-    expect(matchesDateFilter(
-      { updated_at: '2026-03-22T00:00:00Z', created_at: '2026-03-16T00:00:00Z' }, // created_at too late
-      {
-        updated_after: '2026-03-20T00:00:00Z',
-        updated_before: '2026-03-25T00:00:00Z',
-        created_after:  '2026-03-05T00:00:00Z',
-        created_before: '2026-03-15T00:00:00Z',
-      }
-    )).toBe(false);
-  });
-});