MCPサーバーのリファクタリング記録：コード分割とテスト実装

1.
2├── README.ja.md
3├── dist/            # コンパイル済みファイル用
4├── docs/
5│   └── adr/        # アーキテクチャ決定記録
6├── src/
7│   ├── index.ts    # メインエントリーポイント
8│   ├── handlers.ts # MCPリクエストハンドラー
9│   ├── client.ts   # Redmine APIクライアント
10│   ├── types.ts    # 型定義
11│   └── config.ts   # 設定

1{
2  "type": "module",
3  "engines": {
4    "node": ">=18"
5  },
6  "dependencies": {
7    "@modelcontextprotocol/sdk": "^1.1.0",
8    "zod": "^3.22.4"
9  },
10  "devDependencies": {
11    "typescript": "^5.5.4",
12    "tsx": "^4.7.0",
13    "jest": "^29.7.0"
14  }
15}

1{
2  "compilerOptions": {
3    "target": "es2018",
4    "module": "Node16",
5    "moduleResolution": "Node16",
6    "declaration": true,
7    "strict": true,
8    "isolatedModules": true,
9    "esModuleInterop": true,
10    "skipLibCheck": true
11  }
12}

1src/
2├── tools/                # リソースごとのツール定義
3│   ├── issues.ts        # チケット関連
4│   ├── projects.ts      # プロジェクト関連
5│   ├── time_entries.ts  # 作業時間関連
6│   └── index.ts         # エクスポート
7├── formatters/          # レスポンス整形
8│   ├── issues.ts
9│   ├── projects.ts
10│   ├── time_entries.ts
11│   └── index.ts
12├── lib/                 # 共通ライブラリ
13│   ├── client.ts        # Redmine APIクライアント
14│   ├── config.ts        # 設定管理
15│   └── types.ts         # 型定義
16├── handlers.ts          # リクエストハンドラー
17└── index.ts            # エントリーポイント

1handlers → tools → lib
2       ↘        ↗
3     formatters

1src/
2├── lib/
3│   ├── client/           # APIクライアント関連
4│   │   ├── base.ts       # 基本クライアント機能 + エラー
5│   │   ├── issues.ts     # Issuesリソース用クライアント
6│   │   ├── projects.ts   # Projectsリソース用クライアント
7│   │   ├── time_entries.ts # TimeEntriesリソース用クライアント
8│   │   └── index.ts      # クライアントのエクスポート
9│   ├── types/            # 型定義関連
10│   │   ├── common.ts     # 共通の型定義・定数
11│   │   ├── issues/       # Issues関連の型定義
12│   │   │   ├── schema.ts # Zodスキーマ
13│   │   │   └── types.ts  # TypeScript型定義
14│   │   ├── projects/     # Projects関連の型定義
15│   │   └── time_entries/ # TimeEntries関連の型定義

1src/
2└── lib/
3    └── __tests__/
4        ├── client/           # クライアントのテスト
5        │   ├── base.test.ts  # 基本機能のテスト
6        │   ├── issues.test.ts
7        │   ├── projects.test.ts
8        │   └── time_entries.test.ts
9        └── types/            # 型定義のテスト
10            ├── common.test.ts
11            ├── issues/
12            ├── projects/
13            └── time_entries/

1describe("GET /issues/:id", () => {
2  beforeEach(() => {
3    // 実際のAPIレスポンスをキャプチャしてテストに使用
4    const actualResponse = await client.get('/issues/1234');
5
6    // fetchのモックについては以下のドキュメントを参考にしました：
7    // https://jestjs.io/docs/mock-function-api
8    // https://medium.com/swlh/how-to-mock-a-fetch-api-request-with-jest-and-typescript-bb6adf673a00
9    jest.spyOn(global, 'fetch').mockResolvedValue({
10      ok: true,
11      status: 200,
12      json: async () => actualResponse
13    });
14  });
15
16  it('handles custom fields correctly', async () => {
17    const response = await issuesClient.getIssue(1234);
18    
19    // カスタムフィールドの型と構造を検証
20    expect(response.custom_fields).toEqual(
21      expect.arrayContaining([
22        expect.objectContaining({
23          id: expect.any(Number),
24          name: expect.any(String),
25          value: expect.any(String)
26        })
27      ])
28    );
29  });
30});
31
32describe("POST/PUT/DELETE operations", () => {
33  it.skip("skipped for safety - see ADR-0004", () => {
34    // データ変更操作はスキップ
35  });
36});

1describe('GET /issues with pagination', () => {
2  it('handles large result sets correctly', async () => {
3    // 実際の大規模プロジェクトのデータを使用
4    const response = await issuesClient.listIssues({
5      project_id: 5678,
6      limit: 100,
7      offset: 200
8    });
9
10    // ページネーションメタデータの検証
11    expect(response.total_count).toBeGreaterThan(0);
12    expect(response.offset).toBe(200);
13    expect(response.limit).toBe(100);
14    expect(response.issues.length).toBeLessThanOrEqual(100);
15  });
16});