🤖 fix: auto-correct redundant absolute paths with warning #503
Conversation
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
Instead of rejecting absolute paths within the workspace, automatically
convert them to relative paths and include a warning to the agent.
**Problem:**
- Oct 31 nightly run showed 55% of terminal-bench tasks (44/80) hit
'Redundant path prefix' errors
- Agent wastes tool calls retrying with relative paths
- Causes confusion and burns time approaching timeout
- Particularly affects QEMU (80% fail), Git (83% fail), Security (75% fail)
**Solution:**
- validateNoRedundantPrefix now returns {correctedPath, warning} instead of {error}
- Tools (file_read, file_edit_*) auto-correct the path and include warning in response
- Agent still gets feedback to use relative paths, but operation succeeds
- Preserves token-saving goal while removing friction
**Expected impact:**
- Reduce wasted tool calls in 44/80 tasks
- Estimated +5-8 percentage point improvement in pass rate (48-51% vs current 43%)
- Particularly helps QEMU/Git/Security task categories
**Testing:**
- Updated all tests to expect auto-correction instead of errors
- All 26 fileCommon tests passing
_Generated with `cmux`_
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Add optional warning field to FileReadToolResult and FileEditDiffSuccessBase to support auto-correction warnings. Also update file_read test to expect auto-correction instead of rejection.
Eliminates duplication across file_read, file_edit_insert, and file_edit_operation by creating validateAndCorrectPath() wrapper in fileCommon.ts.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Problem
Terminal-bench agents hit "Redundant path prefix" errors on 55% of tasks (44/80) when using absolute paths like
/app/file.txt. This wastes tool calls as agents retry with relative paths—particularly affecting QEMU (80% fail), Git (83% fail), and Security (75% fail) categories.Solution
Changed
validateNoRedundantPrefixto auto-correct paths while keeping the warning. Tools now use the corrected path and include:"Using relative paths like 'file.txt' instead of '/app/file.txt' saves tokens. The path has been auto-corrected for you."Expected impact: +5-8pp pass rate improvement. Combined with merged 30-min timeout, targeting 60-65% pass rate.
Generated with
cmux