Skip to content

fix(file-tools): move dedup hints out of content#14092

Open
LeonSGP43 wants to merge 1 commit into
NousResearch:mainfrom
LeonSGP43:fix/read-file-dedup-stub-13079
Open

fix(file-tools): move dedup hints out of content#14092
LeonSGP43 wants to merge 1 commit into
NousResearch:mainfrom
LeonSGP43:fix/read-file-dedup-stub-13079

Conversation

@LeonSGP43

Copy link
Copy Markdown
Contributor

Summary

  • stop returning read-file dedup hints in the content field
  • return the dedup notice as an error payload so downstream callers do not treat it as file contents
  • lock the contract with a regression test for the dedup stub shape

Closes #13079.

Testing

  • pytest -o addopts= tests/tools/test_file_read_guards.py

@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists comp/tools Tool registry, model_tools, toolsets tool/file File tools (read, write, patch, search) duplicate This issue or pull request already exists labels Apr 22, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Likely duplicate of #13098 — same fix for #13079 (move dedup hint to error field). Also duplicates #13210.

@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #13098.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tools Tool registry, model_tools, toolsets duplicate This issue or pull request already exists P2 Medium — degraded but workaround exists tool/file File tools (read, write, patch, search) type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: read_file tool's dedup mechanism pollutes file content with cache hint text

2 participants