Copilot code review agent fails on file paths containing spaces (autofind tab-parsing bug) #191014
Replies: 1 comment
-
|
Thank you for your interest in contributing to our community! We currently only accept discussions created through the GitHub UI using our provided discussion templates. Please re-submit your discussion by navigating to the appropriate category and using the template provided. This discussion has been closed because it was not submitted through the expected format. If you believe this was a mistake, please reach out to the maintainers. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Description
The Copilot code review agent (
autofind) fails with the following error when reviewing PRs that contain files in directories with spaces:Root Cause
Git's unified diff format adds a trailing tab character (
\t) after file paths in---and+++header lines. This is standard git behavior — the tab separates the filename from an optional timestamp.Example from
git diff | od -c:The
autofindbinary appears to include this trailing tab as part of the filename, producingAIRootViewController.swift\tinstead ofAIRootViewController.swift, which then fails the file-existence check.Reproduction
ViewControllers/2 - Power/Environment
Expected Behavior
autofindshould strip trailing whitespace (tab/space) from filenames parsed out of unified diff headers, matching standard git diff format.Workaround
None available on the repo side without renaming directories (breaking change for a large project). Currently the Copilot code review agent is non-functional for any PR touching files in paths with spaces.
Beta Was this translation helpful? Give feedback.
All reactions