- Notifications
You must be signed in to change notification settings - Fork3.1k
improve response/feedback loop with better error messages#1414
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this 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 PR improves error messaging for theget_file_contents tool to better guide LLM users when they encounter the "file content SHA is nil" error. The enhanced message now explicitly instructs users to add a trailing slash (/) when requesting directories, addressing confusion that occurred when GPT-5.1-Codex ignored these instructions.
- Enhanced error message to include actionable guidance about trailing slashes for directory paths
💡Add Copilot custom instructions for smarter, more guided reviews.Learn how to get started.
88a594e intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
GTP-5.1-Codex experienced a problem with
get_file_contents. It ignored the instruction to add trailing slashes to directories. It looped multiple times before it recovered and added the trailing slashes.The follow up error:
file content SHA is nilusually means we are using the logic to look for a file but provided a path in reality. This improved error message should improve the feedback loop drastically for llms.Closes: