Conversation
gagip
commented
Jan 12, 2026
- Service : 해시 키를 이용해 변경사항(diff) 내용 가져오기 기능 추가
- mcp_server: 변경사항 가져오기 api 추가
Cleaned up the import statements by removing the unused List type from typing in data_loader.py.
There was a problem hiding this comment.
Pull request overview
This pull request adds functionality to retrieve detailed commit changes (diffs) using commit hashes, providing a new API endpoint for the MCP server. The version is bumped from 0.2.0 to 0.3.0.
Changes:
- Added
get_commit_changesservice method and MCP tool for retrieving commit diffs by hash - Created new
formatters.pymodule to refactor and organize output formatting logic - Restructured documentation by moving detailed content from CLAUDE.md to separate docs/ files (usage.md, testing.md, architecture.md)
Reviewed changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Version bump from 0.2.0 to 0.3.0 |
| uv.lock | Lock file updated for new version |
| git_log_analysis/mcp/data_loader.py | Added load_commit_changes method to retrieve commit diffs by hash with error handling |
| git_log_analysis/mcp/service.py | Added get_commit_changes service wrapper with exception handling |
| git_log_analysis/mcp/formatters.py | New module with formatting functions for commit changes, search results, and project lists |
| mcp_server.py | Added get_commit_changes tool and refactored to use new formatter functions |
| tests/test_commit_changes.py | Comprehensive test suite for the new commit changes functionality |
| tests/conftest.py | Updated test fixtures to generate realistic commit hashes and changes files |
| README.md | Updated MCP tools documentation with new get_commit_changes tool and usage examples |
| CLAUDE.md | Restructured to focus on project overview and code guidelines |
| docs/usage.md | New file with detailed usage instructions |
| docs/testing.md | New file with testing infrastructure documentation |
| docs/architecture.md | New file with system architecture overview |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
README.md
Outdated
|
|
||
| 1. `search_commits`: 커밋 검색 | ||
| - `keyword`: 검색 키워드 (커밋 메시지) | ||
| - `keyword`: 검색 키워드 (커밋 메시지 및 파일 경로) |
There was a problem hiding this comment.
The description claims the search_commits tool searches "커밋 메시지 및 파일 경로" (commit message and file path), but the actual implementation in query_engine.py only searches commit messages (line 110: 'if params.keyword in commit.message.lower()'). This documentation is inaccurate and should be corrected to only mention commit messages, or the implementation should be updated to actually search file paths.
| - `keyword`: 검색 키워드 (커밋 메시지 및 파일 경로) | |
| - `keyword`: 검색 키워드 (커밋 메시지) |
| @@ -1,7 +1,5 @@ | |||
| # CLAUDE.md | |||
|
|
|||
There was a problem hiding this comment.
The comment was removed explaining that this file provides guidance to Claude Code. While the comment may have been considered redundant, removing this context makes it less clear what the purpose of CLAUDE.md is for developers who are not familiar with Claude Code integration.
| This document provides guidance and context for Claude Code / Claude Desktop (and other AI coding assistants) when interacting with this repository. |