Repository navigation
fix(stats): preserve whitespace in filenames - #2280
Merged
Byron merged 2 commits intoOct 12, 2026
Merged
Byron merged 2 commits into
Byron merged 2 commits into
Conversation
Copy link
Copy Markdown
Contributor
Sorry, something went wrong.
Commit.stats strips leading and trailing whitespace from filenames after splitting the diff output into records. This merges distinct paths such as `name`, ` name`, and `name `, overwrites their per-file statistics, and turns an all-space filename into an empty key. Keep the filename field intact. Add an integration regression covering root commits, later modifications, empty commits, and deletions. Store blobs directly to avoid relying on trailing-space worktree filenames. The regression fails before this change and passes afterward. On Linux with Python 3.12, the full suite passes: 1690 passed, 89 skipped, 1 xfailed, and 64 subtests passed. Pre-commit, mypy, and basedpyright also pass. Prepared by OpenAI Codex (dot); human review is pending.
The stats regression's root and modification assertions pass on Windows, but IndexFile.remove normalizes the all-space test path to `.` and makes `git rm` fail before testing deletion statistics. Clear and write the fixture's in-memory index instead, just as the test already stores its initial blobs directly. The resulting empty-tree commit still exercises deletion statistics for every original filename. The full Linux suite again passes: 1690 passed, 89 skipped, 1 xfailed, and 64 subtests. Focused commit/util/diff tests, pre-commit, mypy, and basedpyright pass. Windows CI will verify the portability correction. Prepared by OpenAI Codex (dot); human review is pending.
Copy link
Copy Markdown
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The targeted fix is correct and thoroughly covered by regression tests.
0 open findings
What changed in this PRPreserves leading and trailing whitespace in filenames reported by Commit.stats.
Changes:
- Stops stripping filename whitespace during statistics parsing.
- Adds regression coverage for additions, modifications, empty commits, and deletions.
| File | Description |
|---|---|
| git/util.py | Retains filenames exactly when building per-file statistics. |
| test/test_commit.py | Tests distinct whitespace-sensitive filenames across commit operations. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Sorry, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Commit.stats strips leading and trailing spaces from filenames, so distinct paths such as name, name, and name overwrite each other's per-file statistics. An all-space filename becomes an empty dictionary key. Git's --numstat output preserves these spaces; record newlines have already been removed by splitlines().
This keeps the filename field intact. The regression builds five distinct paths directly from blobs/index entries, then checks root commits, subsequent modifications, an empty commit, and deletions. This avoids creating trailing-space filenames in the working tree.
Verification
Tab/newline filename quoting is unchanged. CI on 963ca1c7 now passes all 47 jobs across all five workflows, including Windows, macOS, Ubuntu, Cygwin, Alpine, lint, and dependency checks. The previously failing Alpine network-dependent clone test passed on rerun; no source change was needed.
Rechecked on 2026-10-12: upstream main remains 804bc807 and still strips filename whitespace. No matching superseding fix was found among recent PRs or stats/whitespace searches. The earlier Git 3 migration PRs #2262 and #2273 are closed without merging. The focused commit-stats tests also pass locally (2 passed).
This PR was prepared and submitted by OpenAI Codex (dot), an AI agent operating through v1shay's account. Human review is pending.