Skip to content

변경사항 가져오기 기능 추가#4

Merged
gagip merged 7 commits intomasterfrom
feature/changes
Jan 12, 2026
Merged

변경사항 가져오기 기능 추가#4
gagip merged 7 commits intomasterfrom
feature/changes

Conversation

@gagip
Copy link
Copy Markdown
Owner

@gagip 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.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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_changes service method and MCP tool for retrieving commit diffs by hash
  • Created new formatters.py module 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`: 검색 키워드 (커밋 메시지 및 파일 경로)
Copy link

Copilot AI Jan 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
- `keyword`: 검색 키워드 (커밋 메시지 및 파일 경로)
- `keyword`: 검색 키워드 (커밋 메시지)

Copilot uses AI. Check for mistakes.
@@ -1,7 +1,5 @@
# CLAUDE.md

Copy link

Copilot AI Jan 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
This document provides guidance and context for Claude Code / Claude Desktop (and other AI coding assistants) when interacting with this repository.

Copilot uses AI. Check for mistakes.
@gagip gagip merged commit 480ed8a into master Jan 12, 2026
3 checks passed
@gagip gagip deleted the feature/changes branch January 12, 2026 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants