@daghis/teamcity-mcp 0.1.2 → 0.1.3

package/tests/unit/tools/compatibility-lookups.test.ts

@@ -1,150 +0,0 @@
-// Force dev mode (read-only) for lookup tools
-jest.mock('@/config', () => ({
-  getTeamCityUrl: () => 'https://example.test',
-  getTeamCityToken: () => 'token',
-  getMCPMode: () => 'dev',
-}));
-
-describe('tools: compatibility lookups', () => {
-  afterEach(() => {
-    jest.resetModules();
-    jest.clearAllMocks();
-  });
-
-  it('get_compatible_build_types_for_agent returns compatibilities', async () => {
-    await new Promise<void>((resolve, reject) => {
-      jest.isolateModules(() => {
-        (async () => {
-          const getCompatibleBuildTypes = jest.fn(async () => ({
-            data: { buildType: [{ id: 'bt1' }] },
-          }));
-          jest.doMock('@/api-client', () => ({
-            TeamCityAPI: { getInstance: () => ({ agents: { getCompatibleBuildTypes } }) },
-          }));
-          // eslint-disable-next-line @typescript-eslint/no-var-requires
-          const { getRequiredTool } = require('@/tools');
-          const res = await getRequiredTool('get_compatible_build_types_for_agent').handler({
-            agentId: 'A1',
-          });
-          const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-          expect(payload).toMatchObject({ buildType: [{ id: 'bt1' }] });
-          resolve();
-        })().catch(reject);
-      });
-    });
-  });
-
-  it('get_incompatible_build_types_for_agent returns incompatibilities', async () => {
-    await new Promise<void>((resolve, reject) => {
-      jest.isolateModules(() => {
-        (async () => {
-          const getIncompatibleBuildTypes = jest.fn(async () => ({
-            data: { buildType: [{ id: 'btX' }] },
-          }));
-          jest.doMock('@/api-client', () => ({
-            TeamCityAPI: { getInstance: () => ({ agents: { getIncompatibleBuildTypes } }) },
-          }));
-          // eslint-disable-next-line @typescript-eslint/no-var-requires
-          const { getRequiredTool } = require('@/tools');
-          const res = await getRequiredTool('get_incompatible_build_types_for_agent').handler({
-            agentId: 'A1',
-          });
-          const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-          expect(payload).toMatchObject({ buildType: [{ id: 'btX' }] });
-          resolve();
-        })().catch(reject);
-      });
-    });
-  });
-
-  it('get_compatible_agents_for_build_type filters enabled-by-default', async () => {
-    await new Promise<void>((resolve, reject) => {
-      jest.isolateModules(() => {
-        (async () => {
-          const getAllAgents = jest.fn(async () => ({ data: { agent: [{ id: '1' }], count: 1 } }));
-          jest.doMock('@/api-client', () => ({
-            TeamCityAPI: { getInstance: () => ({ agents: { getAllAgents } }) },
-          }));
-          // eslint-disable-next-line @typescript-eslint/no-var-requires
-          const { getRequiredTool } = require('@/tools');
-          const res = await getRequiredTool('get_compatible_agents_for_build_type').handler({
-            buildTypeId: 'bt1',
-          });
-          const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-          expect(Array.isArray(payload.agent)).toBe(true);
-          expect(payload.agent).toHaveLength(1);
-          resolve();
-        })().catch(reject);
-      });
-    });
-  });
-
-  it('get_compatible_agents_for_build_type can include disabled when flag set', async () => {
-    await new Promise<void>((resolve, reject) => {
-      jest.isolateModules(() => {
-        (async () => {
-          const getAllAgents = jest.fn(async () => ({ data: { agent: [], count: 0 } }));
-          jest.doMock('@/api-client', () => ({
-            TeamCityAPI: { getInstance: () => ({ agents: { getAllAgents } }) },
-          }));
-          // eslint-disable-next-line @typescript-eslint/no-var-requires
-          const { getRequiredTool } = require('@/tools');
-          const res = await getRequiredTool('get_compatible_agents_for_build_type').handler({
-            buildTypeId: 'bt1',
-            includeDisabled: true,
-          });
-          const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-          expect(Array.isArray(payload.agent)).toBe(true);
-          expect(payload.agent).toHaveLength(0);
-          resolve();
-        })().catch(reject);
-      });
-    });
-  });
-
-  it('count_compatible_agents_for_build_type returns only count', async () => {
-    await new Promise<void>((resolve, reject) => {
-      jest.isolateModules(() => {
-        (async () => {
-          const getAllAgents = jest.fn(async (_loc?: string, fields?: string) => ({
-            data: { count: fields === 'count' ? 7 : 0 },
-          }));
-          jest.doMock('@/api-client', () => ({
-            TeamCityAPI: { getInstance: () => ({ agents: { getAllAgents } }) },
-          }));
-          // eslint-disable-next-line @typescript-eslint/no-var-requires
-          const { getRequiredTool } = require('@/tools');
-          const res = await getRequiredTool('count_compatible_agents_for_build_type').handler({
-            buildTypeId: 'bt1',
-          });
-          const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-          expect(payload).toMatchObject({ count: 7 });
-          resolve();
-        })().catch(reject);
-      });
-    });
-  });
-
-  it('get_compatible_agents_for_queued_build derives buildType and looks up agents', async () => {
-    await new Promise<void>((resolve, reject) => {
-      jest.isolateModules(() => {
-        (async () => {
-          const getBuild = jest.fn(async () => ({ buildTypeId: 'btX' }));
-          const getAllAgents = jest.fn(async () => ({ data: { agent: [{ id: '2' }], count: 1 } }));
-          jest.doMock('@/api-client', () => ({
-            TeamCityAPI: { getInstance: () => ({ getBuild, agents: { getAllAgents } }) },
-          }));
-          // eslint-disable-next-line @typescript-eslint/no-var-requires
-          const { getRequiredTool } = require('@/tools');
-          const res = await getRequiredTool('get_compatible_agents_for_queued_build').handler({
-            buildId: 'b123',
-          });
-          const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-          expect(Array.isArray(payload.agent)).toBe(true);
-          expect(payload.agent).toHaveLength(1);
-          resolve();
-        })().catch(reject);
-      });
-    });
-  });
-});

package/tests/unit/tools/fetch-build-log-ambiguity.test.ts

@@ -1,70 +0,0 @@
-import { getRequiredTool } from '@/tools';
-
-// Mock TeamCity API client to simulate ambiguous and fallback scenarios
-jest.mock('@/api-client', () => {
-  const listBuilds = jest.fn(async (locator?: string) => {
-    if (locator?.includes('number:77')) {
-      // Ambiguous: two builds, no buildTypeId filter provided
-      return { build: [{ id: 1001 }, { id: 1002 }] };
-    }
-    if (locator?.includes('buildType:(id:BT_FALLBACK)') && locator.includes('number:56')) {
-      // Force fallback by returning empty for the direct locator
-      return { build: [] };
-    }
-    if (locator === 'buildType:(id:BT_FALLBACK),branch:default:any,count:100') {
-      // Fallback recent list contains the matching number
-      return { build: [{ id: 2002, number: '56' }] };
-    }
-    return { build: [] };
-  });
-
-  const getBuildLogChunk = jest.fn(
-    (buildId?: string, opts?: { startLine?: number; lineCount?: number }) => {
-      return Promise.resolve({
-        lines: [`resolved ${buildId}`],
-        startLine: opts?.startLine ?? 0,
-        nextStartLine: undefined,
-        totalLines: 1,
-      });
-    }
-  );
-
-  return {
-    TeamCityAPI: {
-      getInstance: () => ({
-        listBuilds,
-        getBuildLogChunk,
-      }),
-    },
-  };
-});
-
-describe('fetch_build_log buildNumber resolution edge cases', () => {
-  it('errors when multiple builds match and buildTypeId is missing', async () => {
-    const res = await getRequiredTool('fetch_build_log').handler({
-      buildNumber: '77',
-      page: 1,
-      pageSize: 10,
-    });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    // runTool wraps errors in a { success: false, error: { message } } shape
-    expect(payload.success).toBe(false);
-    const errObj = (payload as { error?: { message?: unknown } }).error;
-    const message =
-      typeof errObj?.message === 'string' ? errObj.message : String(errObj?.message ?? '');
-    expect(message).toContain('Multiple builds match number 77');
-  });
-
-  it('falls back to recent list when direct locator returns none', async () => {
-    const res = await getRequiredTool('fetch_build_log').handler({
-      buildNumber: 56,
-      buildTypeId: 'BT_FALLBACK',
-      page: 1,
-      pageSize: 10,
-    });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.meta.buildId).toBe('2002');
-    expect(payload.meta.buildNumber).toBe('56');
-    expect(payload.lines[0]).toBe('resolved 2002');
-  });
-});

package/tests/unit/tools/fetch-build-log-by-number.test.ts

@@ -1,66 +0,0 @@
-import { getRequiredTool } from '@/tools';
-
-// Mock TeamCity API client to support buildNumber resolution and chunk fetching
-jest.mock('@/api-client', () => {
-  const listBuilds = jest.fn(async (locator?: string) => {
-    // Provide deterministic responses based on locator
-    if (locator?.includes('buildType:(id:BT_ID)') && locator.includes('number:54')) {
-      return { build: [{ id: 1189, buildTypeId: 'BT_ID' }] };
-    }
-    if (locator?.includes('number:55')) {
-      // Single match without buildTypeId
-      return { build: [{ id: 2001, buildTypeId: 'ANY' }] };
-    }
-    return { build: [] };
-  });
-
-  const getBuildLogChunk = jest.fn(
-    (buildId?: string, opts?: { startLine?: number; lineCount?: number }) => {
-      const lines: string[] = [
-        `log for ${buildId} :: ${opts?.startLine ?? 0}-${(opts?.startLine ?? 0) + (opts?.lineCount ?? 500) - 1}`,
-      ];
-      return Promise.resolve({
-        lines,
-        startLine: opts?.startLine ?? 0,
-        nextStartLine: undefined,
-        totalLines: 1,
-      });
-    }
-  );
-
-  return {
-    TeamCityAPI: {
-      getInstance: () => ({
-        listBuilds,
-        getBuildLogChunk,
-      }),
-    },
-  };
-});
-
-describe('fetch_build_log with buildNumber', () => {
-  it('resolves buildId using buildNumber + buildTypeId and fetches chunk', async () => {
-    const res = await getRequiredTool('fetch_build_log').handler({
-      buildNumber: '54',
-      buildTypeId: 'BT_ID',
-      page: 1,
-      pageSize: 2,
-    });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.meta.buildId).toBe('1189');
-    expect(payload.meta.buildNumber).toBe('54');
-    expect(Array.isArray(payload.lines)).toBe(true);
-    expect(typeof payload.lines[0]).toBe('string');
-  });
-
-  it('resolves buildId using buildNumber alone when a single match exists', async () => {
-    const res = await getRequiredTool('fetch_build_log').handler({
-      buildNumber: 55,
-      page: 1,
-      pageSize: 5,
-    });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.meta.buildId).toBe('2001');
-    expect(payload.meta.buildNumber).toBe('55');
-  });
-});

package/tests/unit/tools/fetch-build-log-pagination.test.ts

@@ -1,73 +0,0 @@
-import { getRequiredTool } from '@/tools';
-
-// Mock TeamCity API client to provide a deterministic log and slicing behavior
-jest.mock('@/api-client', () => {
-  const fullLogLines = Array.from({ length: 10 }, (_, i) => `line ${i + 1}`);
-  const getBuildLogChunk = jest.fn(
-    (buildId?: string, opts?: { startLine?: number; lineCount?: number }) => {
-      const start = Math.max(0, opts?.startLine ?? 0);
-      const count = Math.max(1, opts?.lineCount ?? 500);
-      const end = Math.min(fullLogLines.length, start + count);
-      const lines = fullLogLines.slice(start, end);
-      return Promise.resolve({
-        lines,
-        startLine: start,
-        nextStartLine: end < fullLogLines.length ? end : undefined,
-        totalLines: fullLogLines.length,
-      });
-    }
-  );
-
-  return {
-    TeamCityAPI: {
-      getInstance: () => ({
-        getBuildLogChunk,
-      }),
-    },
-  };
-});
-
-describe('fetch_build_log pagination', () => {
-  it('returns the first page with default pageSize when only page provided', async () => {
-    const res = await getRequiredTool('fetch_build_log').handler({
-      buildId: '123',
-      page: 1,
-      pageSize: 3,
-    });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.lines).toEqual(['line 1', 'line 2', 'line 3']);
-    expect(payload.meta.page).toBe(1);
-    expect(payload.meta.pageSize).toBe(3);
-    expect(payload.meta.hasMore).toBe(true);
-    expect(payload.meta.nextPage).toBe(2);
-  });
-
-  it('returns the second page correctly', async () => {
-    const res = await getRequiredTool('fetch_build_log').handler({
-      buildId: '123',
-      page: 2,
-      pageSize: 4,
-    });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.lines).toEqual(['line 5', 'line 6', 'line 7', 'line 8']);
-    expect(payload.meta.page).toBe(2);
-    expect(payload.meta.pageSize).toBe(4);
-    expect(payload.meta.startLine).toBe(4);
-    expect(payload.meta.hasMore).toBe(true);
-  });
-
-  it('supports explicit startLine and lineCount', async () => {
-    const res = await getRequiredTool('fetch_build_log').handler({
-      buildId: 'abc',
-      startLine: 8,
-      lineCount: 5,
-    });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    // Only two lines remain starting at 9th line (0-based index 8)
-    expect(payload.lines).toEqual(['line 9', 'line 10']);
-    expect(payload.meta.pageSize).toBe(5);
-    expect(payload.meta.startLine).toBe(8);
-    expect(payload.meta.hasMore).toBe(false);
-    expect(payload.meta.totalLines).toBe(10);
-  });
-});

package/tests/unit/tools/fetch-build-log-tail.test.ts

@@ -1,33 +0,0 @@
-import { getRequiredTool } from '@/tools';
-
-// Mock TeamCity API client to provide a deterministic full log
-jest.mock('@/api-client', () => {
-  const fullLogLines = Array.from({ length: 10 }, (_, i) => `line ${i + 1}`);
-  const getBuildLog = jest.fn(() => Promise.resolve(fullLogLines.join('\n')));
-
-  return {
-    TeamCityAPI: {
-      getInstance: () => ({
-        getBuildLog,
-      }),
-    },
-  };
-});
-
-describe('fetch_build_log tail mode', () => {
-  it('returns the last N lines with correct metadata', async () => {
-    const res = await getRequiredTool('fetch_build_log').handler({
-      buildId: 'b1',
-      tail: true,
-      lineCount: 3,
-    });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-
-    expect(payload.lines).toEqual(['line 8', 'line 9', 'line 10']);
-    expect(payload.meta.mode).toBe('tail');
-    expect(payload.meta.pageSize).toBe(3);
-    expect(payload.meta.startLine).toBe(7); // 0-based index of 'line 8'
-    expect(payload.meta.totalLines).toBe(10);
-    expect(payload.meta.hasMore).toBe(true); // There are earlier lines available
-  });
-});

package/tests/unit/tools/get-status-and-results.test.ts

@@ -1,67 +0,0 @@
-// Mock config to avoid reading env
-jest.mock('@/config', () => ({
-  getTeamCityUrl: () => 'https://example.test',
-  getTeamCityToken: () => 'token',
-  getMCPMode: () => 'dev',
-}));
-
-describe('tools: enhanced status and results', () => {
-  afterEach(() => {
-    jest.resetModules();
-    jest.clearAllMocks();
-  });
-
-  it('get_build_status with useEnhanced returns JSON payload', async () => {
-    const fakeStatus = {
-      buildId: '123',
-      state: 'running',
-      status: 'SUCCESS',
-      percentageComplete: 42,
-    };
-
-    // Mock BuildStatusManager to return fakeStatus
-    const getBuildStatus = jest.fn().mockResolvedValue(fakeStatus);
-    const BuildStatusManager = jest.fn().mockImplementation(() => ({ getBuildStatus }));
-    jest.doMock('@/teamcity/build-status-manager', () => ({ BuildStatusManager }));
-
-    // Mock TeamCityAPI.getInstance to avoid env dependency
-    jest.doMock('@/api-client', () => ({ TeamCityAPI: { getInstance: () => ({ builds: {} }) } }));
-    // Re-require tools after mocking the module
-    // Mock TeamCityAPI.getInstance to avoid env dependency
-    jest.doMock('@/api-client', () => ({ TeamCityAPI: { getInstance: () => ({ builds: {} }) } }));
-    // eslint-disable-next-line @typescript-eslint/no-var-requires
-    const { getRequiredTool } = require('@/tools');
-    const res = await getRequiredTool('get_build_status').handler({
-      buildId: '123',
-      useEnhanced: true,
-      includeTests: true,
-    });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(typeof payload).toBe('object');
-    expect(payload.buildId).toBe('123');
-  });
-
-  it('get_build_results returns enriched payload via manager', async () => {
-    const fakeResults = {
-      build: { id: 99, status: 'SUCCESS', number: '1' },
-      artifacts: [{ name: 'a.txt', size: 10 }],
-      statistics: { testCount: 5 },
-    };
-
-    const getBuildResults = jest.fn().mockResolvedValue(fakeResults);
-    const BuildResultsManager = jest.fn().mockImplementation(() => ({ getBuildResults }));
-    jest.doMock('@/teamcity/build-results-manager', () => ({ BuildResultsManager }));
-
-    // eslint-disable-next-line @typescript-eslint/no-var-requires
-    const { getRequiredTool } = require('@/tools');
-    const res = await getRequiredTool('get_build_results').handler({
-      buildId: '99',
-      includeArtifacts: true,
-      includeStatistics: true,
-    });
-
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(typeof payload).toBe('object');
-    expect(payload.build?.id).toBe(99);
-  });
-});

package/tests/unit/tools/list-branches-and-parameters.test.ts

@@ -1,36 +0,0 @@
-import { getRequiredTool } from '@/tools';
-
-describe('list_branches validation and listing', () => {
-  it('rejects when neither projectId nor buildTypeId provided', async () => {
-    const res = await getRequiredTool('list_branches').handler({});
-    // Expect schema validation to surface an error
-    const msg = res.error ?? (res.content?.[0]?.text as string) ?? '';
-    expect(msg).toMatch(/Either projectId or buildTypeId is required/);
-  });
-});
-
-// Mock for list_parameters
-jest.mock('@/api-client', () => ({
-  TeamCityAPI: {
-    getInstance: () => ({
-      getBuildType: async (id: string) => ({
-        id,
-        parameters: {
-          property: [
-            { name: 'env.FOO', value: 'bar' },
-            { name: 'opt', value: '1' },
-          ],
-        },
-      }),
-    }),
-  },
-}));
-
-describe('list_parameters', () => {
-  it('returns parameters and count', async () => {
-    const res = await getRequiredTool('list_parameters').handler({ buildTypeId: 'bt1' });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.parameters).toHaveLength(2);
-    expect(payload.count).toBe(2);
-  });
-});

package/tests/unit/tools/list-build-configs-pagination.test.ts

@@ -1,39 +0,0 @@
-import { getRequiredTool } from '@/tools';
-
-// Mock TeamCityAPI to simulate paginated build type listings
-jest.mock('@/api-client', () => {
-  const getAllBuildTypes = jest.fn((locator?: string) => {
-    const startMatch = locator?.match(/start:(\d+)/);
-    const countMatch = locator?.match(/count:(\d+)/);
-    const start = startMatch?.[1] ? parseInt(startMatch[1], 10) : 0;
-    const count = countMatch?.[1] ? parseInt(countMatch[1], 10) : 100;
-
-    const all = Array.from({ length: 4 }, (_, i) => ({ id: `bt${i + 1}` }));
-    const slice = all.slice(start, Math.min(start + count, all.length));
-    return Promise.resolve({ data: { buildType: slice, count: all.length } });
-  });
-
-  return {
-    TeamCityAPI: {
-      getInstance: () => ({
-        buildTypes: { getAllBuildTypes },
-      }),
-    },
-  };
-});
-
-describe('list_build_configs pagination', () => {
-  it('returns first page with pageSize', async () => {
-    const res = await getRequiredTool('list_build_configs').handler({ pageSize: 2 });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.items).toHaveLength(2);
-    expect(payload.pagination.page).toBe(1);
-  });
-
-  it('fetches all pages when all=true', async () => {
-    const res = await getRequiredTool('list_build_configs').handler({ pageSize: 2, all: true });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.items).toHaveLength(4);
-    expect(payload.pagination.mode).toBe('all');
-  });
-});

package/tests/unit/tools/list-builds-pagination.test.ts

@@ -1,45 +0,0 @@
-import { getRequiredTool } from '@/tools';
-
-jest.mock('@/api-client', () => {
-  const getAllBuilds = jest.fn((locator?: string) => {
-    // Parse start/count from locator
-    const startMatch = locator?.match(/start:(\d+)/);
-    const countMatch = locator?.match(/count:(\d+)/);
-    const start = startMatch?.[1] ? parseInt(startMatch[1], 10) : 0;
-    const count = countMatch?.[1] ? parseInt(countMatch[1], 10) : 100;
-
-    const items = [] as Array<{ id: number }>;
-    for (let i = start; i < start + count && i < 3; i++) {
-      items.push({ id: i + 1 });
-    }
-
-    const nextHref = start + count < 3 ? '/next' : undefined;
-    return Promise.resolve({ data: { build: items, count: 3, nextHref } });
-  });
-
-  return {
-    TeamCityAPI: {
-      getInstance: () => ({
-        builds: {
-          getAllBuilds,
-        },
-      }),
-    },
-  };
-});
-
-describe('list_builds pagination', () => {
-  it('returns first page when all is not set', async () => {
-    const res = await getRequiredTool('list_builds').handler({ pageSize: 2 });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.items).toHaveLength(2);
-    expect(payload.pagination.page).toBe(1);
-  });
-
-  it('fetches all pages when all=true', async () => {
-    const res = await getRequiredTool('list_builds').handler({ pageSize: 2, all: true });
-    const payload = JSON.parse((res.content?.[0]?.text as string) ?? '{}');
-    expect(payload.items).toHaveLength(3);
-    expect(payload.pagination.mode).toBe('all');
-  });
-});