From 0b819edceb307ce2f8ba6d58b37a86329b7d6ec0 Mon Sep 17 00:00:00 2001 From: Dawid Rycerz Date: Wed, 28 May 2025 15:27:57 +0200 Subject: feat(gitlab_glab): add merge request diff functionality with large diff mitigation - Add get_mr_diff method to GitLabServer class with support for all glab mr diff options - Implement smart large diff handling that saves diffs > 100KB to temporary files - Add comprehensive test coverage (7 new tests) achieving 95% coverage - Support for MR ID/branch, color options, raw format, and repository selection - Include proper error handling and logging - Update README with usage examples and documentation - Follow project coding standards with type hints and docstrings Resolves large diff handling for LLM consumption while preserving full diff access. --- servers/gitlab_glab/tests/test_integration.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'servers/gitlab_glab/tests/test_integration.py') diff --git a/servers/gitlab_glab/tests/test_integration.py b/servers/gitlab_glab/tests/test_integration.py index 5d7eed5..b7d1005 100644 --- a/servers/gitlab_glab/tests/test_integration.py +++ b/servers/gitlab_glab/tests/test_integration.py @@ -28,7 +28,8 @@ class TestIntegration: # - find_project # - search_issues # - create_issue - assert mock_server.tool.call_count == 4 + # - get_mr_diff + assert mock_server.tool.call_count == 5 # Verify that the tool decorator was called with functions that have # working_directory parameter. We can't directly access the decorated functions -- cgit v1.2.3