Skip to content

Conversation

@roomote
Copy link

@roomote roomote bot commented Oct 27, 2025

This PR attempts to address Issue #8858. Feedback and guidance are welcome.

Problem

The read_file tool documentation was misleading for models that use the simplified single-file format (like grok-code-fast-1). The documentation incorrectly suggested these models could read multiple files when they actually cannot.

Solution

  • Updated getSimpleReadFileDescription() to explicitly state that only ONE file can be read at a time
  • Added clear documentation about using separate tool calls for multiple files
  • This ensures users of models like grok-code-fast-1 understand the limitation upfront

Testing

  • All existing tests pass
  • Linting and type checks pass

Fixes #8858


Important

Clarifies single-file limitation in getSimpleReadFileDescription() for specific models in simple-read-file.ts.

  • Documentation:
    • Updated getSimpleReadFileDescription() in simple-read-file.ts to clarify that only one file can be read at a time.
    • Added note on using separate tool calls for multiple files.
  • Testing:
    • All existing tests pass.
    • Linting and type checks pass.

This description was created by Ellipsis for fe29629. You can customize this summary. It will automatically update as commits are pushed.

- Updated getSimpleReadFileDescription to explicitly state it only supports reading one file at a time
- Added note about using separate tool calls for multiple files
- This fixes the documentation mismatch for models like grok-code-fast-1 that use the simplified format

Fixes #8858
@roomote roomote bot requested review from cte, jr and mrubens as code owners October 27, 2025 07:56
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. documentation Improvements or additions to documentation labels Oct 27, 2025
@roomote
Copy link
Author

roomote bot commented Oct 27, 2025

Review Complete ✅

No issues found. The PR correctly clarifies the single-file limitation in the simplified read_file tool documentation.

Follow Along on Roo Code Cloud

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Oct 27, 2025
Copy link
Author

@roomote roomote bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found.

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

Labels

documentation Improvements or additions to documentation Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[BUG] System prompt's read_file tool failing for reading multiple files.

3 participants