Repository navigation
Unify git_log output format between filtered and unfiltered paths - #4681
sachinsharma3191 wants to merge 2 commits into
Conversation
…ered calls The filtered branch (with start/end timestamps) used raw git log --format with %s (subject only), while the unfiltered branch used Python repr (!r) formatting on commit fields. This produced inconsistent output schemas: - Filtered: raw hash, author name only, subject only - Unfiltered: quoted hash, git.Actor repr, quoted message with escaped newlines Now both paths use repo.iter_commits() with after/before kwargs, producing a unified clean format: raw hash, "Name <email>" author, full message body. Fixes modelcontextprotocol#4469
…hema # Conflicts: # src/git/src/mcp_server_git/server.py
|
Hi maintainers, I've merged the latest |
cliffhall
added
the
closed-superseded-v2
|
Thank you for this pull request, and apologies for the long wait for a response. Since it was opened, the reference servers have been rebuilt for the 2026-07-28 MCP specification (#4857) and released as v2 (2.0.0 on npm for the TypeScript servers, 2026.10.19 on PyPI for the Python servers). The code this PR changes has been replaced, and the behavior it addresses has either been fixed in v2 or no longer exists in the same form. This is not a judgment on the change or on you; it's a consequence of the code it targets having been replaced. If the problem still happens on v2, please open a fresh issue with the version you're running and steps to reproduce. Note that this repository now accepts contributions as issues rather than pull requests from anyone but its maintainers; see This is part of the backlog triage in #4875. |
Summary
Fixes #4469
repo.iter_commits()call withafter/beforekwargs!rrepr formatting — output now uses clean hex SHA,Name <email>author format, and stripped commit messages in both casesrepo.git.log()path that produced different output formattest_git_log_consistent_formattest verifying both paths produce identical output structureTest plan
git_logcalls produce identical output formatName <email>consistently