Memory Bank MCP Server

{
  "schema": "memory_document_v2",
  "metadata": {
    "id": "6a5b4c3d-2e1f-0d9c-8b7a-6c5d4e3f2b1a",
    "title": "エラーファクトリー改善計画",
    "documentType": "plan",
    "path": "error-factory-plan.json",
    "tags": [
      "improvement",
      "error-handling",
      "mcp"
    ],
    "lastModified": "2025-03-30T05:50:32.360Z",
    "createdAt": "2025-03-30T05:50:32.360Z",
    "version": 1
  },
  "content": {
    "overview": {
      "title": "MCP-2・MCP-3: エラーファクトリーメソッド改善計画",
      "description": "エラーファクトリーメソッドの一貫した使用と、不足しているファクトリーメソッドの追加に関する詳細実装計画。直接エラーコンストラクタを使用している箇所を適切なファクトリーメソッドに置き換え、必要に応じて新しいファクトリーメソッドを追加する。"
    },
    "currentState": {
      "description": "現在のコードベースでは、エラー生成の方法が統一されていない。一部のコードではファクトリーメソッドを使用しているが、他の箇所では直接エラーコンストラクタを使用している。また、特定のエラーパターンに対するファクトリーメソッドが実装されていないため、直接コンストラクタを使用せざるを得ない状況もある。",
      "examples": [
        "// 直接コンストラクタを使用（非推奨）\nthrow new DomainError('VALIDATION_ERROR', 'Files must be provided as an object');\n\n// ファクトリーメソッドを使用（推奨）\nthrow DomainErrors.documentNotFound(path);"
      ],
      "affectedFiles": [
        "BranchController.ts",
        "FileSystemBranchMemoryBankRepository.ts",
        "InfrastructureError.ts",
        "DomainError.ts",
        "ApplicationError.ts"
      ]
    },
    "missingFactoryMethods": [
      {
        "errorType": "InfrastructureError",
        "methods": [
          {
            "name": "permissionDenied",
            "description": "ファイルシステム権限エラー",
            "parameters": [
              "message: string",
              "details?: OperationDetails"
            ]
          },
          {
            "name": "fileSystemError",
            "description": "一般的なファイルシステムエラー",
            "parameters": [
              "message: string",
              "details?: OperationDetails"
            ]
          },
          {
            "name": "invalidFileContent",
            "description": "無効なファイル内容エラー",
            "parameters": [
              "message: string",
              "details?: OperationDetails"
            ]
          }
        ]
      },
      {
        "errorType": "DomainError",
        "methods": [
          {
            "name": "validationError",
            "description": "バリデーションエラー",
            "parameters": [
              "message: string",
              "details?: OperationDetails"
            ]
          },
          {
            "name": "invalidOperation",
            "description": "不正な操作エラー",
            "parameters": [
              "operation: string",
              "message: string",
              "details?: OperationDetails"
            ]
          }
        ]
      },
      {
        "errorType": "ApplicationError",
        "methods": [
          {
            "name": "configurationError",
            "description": "設定エラー",
            "parameters": [
              "message: string",
              "originalError?: Error",
              "details?: OperationDetails"
            ]
          },
          {
            "name": "validationFailed",
            "description": "ユースケース入力検証エラー",
            "parameters": [
              "useCaseName: string",
              "message: string",
              "details?: OperationDetails"
            ]
          }
        ]
      }
    ],
    "implementationPlan": {
      "newFactoryMethods": [
        {
          "path": "/packages/mcp/src/shared/errors/InfrastructureError.ts",
          "additions": [
            {
              "method": "permissionDenied",
              "code": "/**\n * ファイルシステム権限エラーを作成します\n * @param message エラーメッセージ\n * @param details 操作の詳細情報\n * @returns InfrastructureError\n */\npermissionDenied: (message: string, details?: OperationDetails) => {\n  return new InfrastructureError(\n    InfrastructureErrorCodes.FILE_PERMISSION_ERROR,\n    message,\n    details\n  );\n}"
            },
            {
              "method": "fileSystemError",
              "code": "/**\n * 一般的なファイルシステムエラーを作成します\n * @param message エラーメッセージ\n * @param details 操作の詳細情報\n * @returns InfrastructureError\n */\nfileSystemError: (message: string, details?: OperationDetails) => {\n  return new InfrastructureError(\n    InfrastructureErrorCodes.FILE_SYSTEM_ERROR,\n    message,\n    details\n  );\n}"
            },
            {
              "method": "invalidFileContent",
              "code": "/**\n * 無効なファイル内容エラーを作成します\n * @param message エラーメッセージ\n * @param details 操作の詳細情報\n * @returns InfrastructureError\n */\ninvalidFileContent: (message: string, details?: OperationDetails) => {\n  return new InfrastructureError(\n    InfrastructureErrorCodes.INVALID_FILE_CONTENT,\n    message,\n    details\n  );\n}"
            }
          ]
        },
        {
          "path": "/packages/mcp/src/shared/errors/DomainError.ts",
          "additions": [
            {
              "method": "validationError",
              "code": "/**\n * バリデーションエラーを作成します\n * @param message エラーメッセージ\n * @param details 操作の詳細情報\n * @returns DomainError\n */\nvalidationError: (message: string, details?: OperationDetails) => {\n  return new DomainError(\n    DomainErrorCodes.VALIDATION_ERROR,\n    message,\n    details\n  );\n}"
            },
            {
              "method": "invalidOperation",
              "code": "/**\n * 不正な操作エラーを作成します\n * @param operation 操作名\n * @param message エラーメッセージ\n * @param details 操作の詳細情報\n * @returns DomainError\n */\ninvalidOperation: (operation: string, message: string, details?: OperationDetails) => {\n  return new DomainError(\n    DomainErrorCodes.INVALID_OPERATION,\n    `Invalid operation '${operation}': ${message}`,\n    details\n  );\n}"
            }
          ]
        },
        {
          "path": "/packages/mcp/src/shared/errors/ApplicationError.ts",
          "additions": [
            {
              "method": "configurationError",
              "code": "/**\n * 設定エラーを作成します\n * @param message エラーメッセージ\n * @param originalError 元のエラー\n * @param details 操作の詳細情報\n * @returns ApplicationError\n */\nconfigurationError: (message: string, originalError?: Error, details?: OperationDetails) => {\n  return new ApplicationError(\n    ApplicationErrorCodes.CONFIGURATION_ERROR,\n    message,\n    originalError,\n    details\n  );\n}"
            },
            {
              "method": "validationFailed",
              "code": "/**\n * ユースケース入力検証エラーを作成します\n * @param useCaseName ユースケース名\n * @param message エラーメッセージ\n * @param details 操作の詳細情報\n * @returns ApplicationError\n */\nvalidationFailed: (useCaseName: string, message: string, details?: OperationDetails) => {\n  return new ApplicationError(\n    ApplicationErrorCodes.VALIDATION_FAILED,\n    `Validation failed in ${useCaseName}: ${message}`,\n    undefined,\n    details\n  );\n}"
            }
          ]
        }
      ],
      "errorCodeUpdates": [
        {
          "path": "/packages/mcp/src/shared/errors/InfrastructureError.ts",
          "additions": [
            "FILE_PERMISSION_ERROR = 'INFRASTRUCTURE_FILE_PERMISSION_ERROR'",
            "INVALID_FILE_CONTENT = 'INFRASTRUCTURE_INVALID_FILE_CONTENT'"
          ]
        },
        {
          "path": "/packages/mcp/src/shared/errors/DomainError.ts",
          "additions": [
            "VALIDATION_ERROR = 'DOMAIN_ERROR.VALIDATION_ERROR'",
            "INVALID_OPERATION = 'DOMAIN_ERROR.INVALID_OPERATION'"
          ]
        },
        {
          "path": "/packages/mcp/src/shared/errors/ApplicationError.ts",
          "additions": [
            "CONFIGURATION_ERROR = 'APPLICATION_ERROR.CONFIGURATION_ERROR'",
            "VALIDATION_FAILED = 'APPLICATION_ERROR.VALIDATION_FAILED'"
          ]
        }
      ],
      "factoryUsageUpdates": [
        {
          "path": "/packages/mcp/src/interface/controllers/BranchController.ts",
          "replacements": [
            {
              "original": "throw new DomainError('VALIDATION_ERROR', 'Files must be provided as an object');",
              "replacement": "throw DomainErrors.validationError('Files must be provided as an object');"
            },
            {
              "original": "throw new DomainError('VALIDATION_ERROR', `Content must be provided for path: ${path}`);",
              "replacement": "throw DomainErrors.validationError(`Content must be provided for path: ${path}`);"
            }
          ]
        },
        {
          "path": "/packages/mcp/src/infrastructure/repositories/file-system/FileSystemBranchMemoryBankRepository.ts",
          "replacements": [
            {
              "original": "throw new InfrastructureError('FILE_NOT_FOUND', `Branch not found: ${branchName}`);",
              "replacement": "throw InfrastructureErrors.fileNotFound(`Branch not found: ${branchName}`);"
            },
            {
              "original": "throw new InfrastructureError('FILE_SYSTEM_ERROR', `Failed to read file: ${fullPath}`, { error: error.message });",
              "replacement": "throw InfrastructureErrors.fileSystemError(`Failed to read file: ${fullPath}`, { error: error.message });"
            },
            {
              "original": "throw new InfrastructureError('INFRASTRUCTURE_INVALID_FILE_CONTENT', `Invalid JSON structure at: ${fullPath}`);",
              "replacement": "throw InfrastructureErrors.invalidFileContent(`Invalid JSON structure at: ${fullPath}`);"
            }
          ]
        }
      ]
    },
    "testingPlan": {
      "approach": "ユニットテストとエラー検証",
      "newTests": [
        {
          "path": "/packages/mcp/tests/unit/shared/errors/InfrastructureError.test.ts",
          "tests": [
            "permissionDenied()ファクトリーメソッドが正しいエラーコードとメッセージで作成されること",
            "fileSystemError()ファクトリーメソッドが正しいエラーコードとメッセージで作成されること",
            "invalidFileContent()ファクトリーメソッドが正しいエラーコードとメッセージで作成されること"
          ]
        },
        {
          "path": "/packages/mcp/tests/unit/shared/errors/DomainError.test.ts",
          "tests": [
            "validationError()ファクトリーメソッドが正しいエラーコードとメッセージで作成されること",
            "invalidOperation()ファクトリーメソッドが正しいエラーコードとメッセージで作成されること"
          ]
        },
        {
          "path": "/packages/mcp/tests/unit/shared/errors/ApplicationError.test.ts",
          "tests": [
            "configurationError()ファクトリーメソッドが正しいエラーコードとメッセージで作成されること",
            "validationFailed()ファクトリーメソッドが正しいエラーコードとメッセージで作成されること"
          ]
        }
      ],
      "verificationTests": [
        "BranchControllerが正しいエラーファクトリーメソッドを使用していることを確認",
        "FileSystemBranchMemoryBankRepositoryが正しいエラーファクトリーメソッドを使用していることを確認"
      ]
    },
    "benefits": [
      "エラー処理の一貫性向上",
      "バグの早期発見",
      "トラブルシューティングの効率化",
      "コードの読みやすさ向上"
    ],
    "dependencies": [
      "BaseError.ts - 基底エラークラス",
      "ErrorUtils.ts - エラーユーティリティ",
      "共通のエラーフォーマットとHTTPステータスコード対応"
    ]
  }
}