Memory Bank MCP Server

{ "schema": "memory_document_v2", "metadata": { "id": "unused-code-removal-plan", "title": "不要コード削除計画", "documentType": "plan", "path": "unused-code-removal-plan.json", "tags": [ "plan", "cleanup", "refactoring", "schema-package" ], "lastModified": "2025-03-28T18:15:00.000Z", "createdAt": "2025-03-28T18:15:00.000Z", "version": 1 }, "content": { "overview": { "description": "Memory Bank全体をモノレポ化するのに伴い、不要なコードや機能を削除し、コードベースをクリーンアップする詳細計画", "goals": [ "マークダウン関連の遺物を完全に削除する", "テンプレート関連の中途半端な実装によるビルドエラーを解消する", "重複するロガー実装を統一する", "コードベース全体の品質と保守性を向上させる" ] }, "deletionTargets": [ { "id": "markdown-related", "title": "マークダウン関連機能", "priority": "high", "status": "部分的に完了", "reason": "JSONベースのアーキテクチャに完全移行済みで不要。一部はすでにfeature/schema-packageで削除済み。", "locations": [ { "file": "packages/mcp/src/domain/entities/DocumentPath.ts", "elements": [ "isMarkdown", "toAlternateFormat" ], "status": "確認中" }, { "file": "packages/mcp/src/interface/controllers/BranchController.ts", "elements": [ "拡張子チェック", "マークダウン⇔JSON変換" ], "status": "一部削除済み" }, { "file": "その他のマークダウン関連ファイル", "elements": [ "マークダウンパーサー", "生成器" ], "status": "確認中" } ], "deletionApproach": "GrepToolを使用して'markdown'や'.md'に関する参照を検索し、関連コードを段階的に削除する。BranchControllerではすでに削除されたパターンを参考に他のファイルも削除。", "dependencies": [ "DIコンテナへの参照がある場合はそれも含めて削除" ] }, { "id": "template-related", "title": "テンプレート関連機能", "priority": "high", "status": "未着手", "reason": "参照のみで実装がなく、ビルドエラーの原因になっている", "locations": [ { "file": "packages/mcp/src/infrastructure/templates/FileTemplateRepository.ts", "elements": [ "クラス全体" ], "status": "確認中" }, { "file": "packages/mcp/src/domain/templates/ITemplateRepository.ts", "elements": [ "インターフェース全体" ], "status": "確認中" }, { "file": "packages/mcp/src/main/di/providers.ts", "elements": [ "テンプレート関連のDI設定" ], "status": "確認中" } ], "deletionApproach": "まずGrepToolでテンプレート関連のコードを特定し、実際に使用されているかを確認。使用されていない場合は完全に削除し、DIコンテナの参照も修正する。", "dependencies": [ "GlobalController内の参照があれば修正が必要" ] }, { "id": "logger-duplication", "title": "ロガー実装の重複", "priority": "medium", "status": "未着手", "reason": "2つの異なるロガー実装が混在している", "locations": [ { "file": "packages/mcp/src/domain/logger/", "elements": [ "ILogger", "関連ファイル" ], "status": "確認中" }, { "file": "packages/mcp/src/shared/utils/logger.ts", "elements": [ "logger実装" ], "status": "残す予定（標準として採用）" } ], "deletionApproach": "shared/utils/loggerを標準として採用し、domain/logger配下のファイルを削除。依存関係を適切に整理するため、必要に応じて抽象化インターフェースを用意。", "dependencies": [ "ロガーを使用している全てのファイル" ] }, { "id": "naming-inconsistencies", "title": "命名規則の不統一", "priority": "low", "status": "未着手", "reason": "コードベース内で命名規則やスタイルが混在している", "locations": [ { "file": "全体", "elements": [ "インターフェース名のIプレフィックス", "プライベートメンバーの_アンダースコア" ], "status": "分析中" } ], "deletionApproach": "実際のコード削除ではなく、命名規則の整理。まずは文書化し、大きな変更は別のリファクタリングフェーズで行う。", "dependencies": [] } ], "verificationStrategy": { "beforeDeletion": [ "対象コードの参照関係を特定（GrepToolで検索）", "テストがあれば実行し、カバレッジを確認", "実際に使用されているかどうかを確認" ], "afterDeletion": [ "ビルドを実行してエラーがないことを確認", "修正後のテストを実行して機能が壊れていないことを確認", "DIコンテナの設定に問題がないか確認" ] }, "deletionSteps": [ { "step": 1, "description": "まずはテンプレート関連のコードを特定し、削除または補完を行う", "tasks": [ "FileTemplateRepository.tsとITemplateRepository.tsを確認", "TemplateServiceとTemplateControllerの参照関係を整理", "DIコンテナの設定を修正", "参照しているコードを適切に修正" ], "estimatedTime": "2-3時間" }, { "step": 2, "description": "マークダウン関連の残りのコードを削除する", "tasks": [ "DocumentPath.tsのマークダウン関連メソッドの削除", "その他のマークダウン変換/パース関連コードの特定と削除", "拡張子チェックや条件分岐の削除" ], "estimatedTime": "1-2時間" }, { "step": 3, "description": "ロガー実装の統一", "tasks": [ "domain/logger/ILoggerの使用箇所確認", "shared/utils/loggerへの置き換え", "不要になったファイルの削除" ], "estimatedTime": "1-2時間" }, { "step": 4, "description": "全体的な確認とクリーンアップ", "tasks": [ "全てのビルドエラーが解消されたことを確認", "テストがパスすることを確認", "残った参照や不整合がないか最終確認" ], "estimatedTime": "1時間" } ], "risks": [ { "risk": "外部から参照されているコードを誤って削除する", "severity": "high", "mitigation": "削除前に参照を十分に確認し、テストカバレッジを確保する" }, { "risk": "テンプレート関連のコード削除によりGlobalControllerが動作しなくなる", "severity": "medium", "mitigation": "部分的な実装を残すか、mock化するなど適切な対応策を検討" }, { "risk": "依存関係の把握が不完全で予期せぬエラーが発生する", "severity": "medium", "mitigation": "小さな変更を段階的に行い、都度ビルドとテストを実行する" } ], "completionCriteria": [ "全てのマークダウン関連コードが削除されている", "テンプレート関連のビルドエラーが解消されている", "ロガー実装が統一されている", "ビルドが成功し、テストが通過する" ] } }