docs: clarify read_command_output search param should be omitted when not filtering

[hannesrudolph]

Summary

Adds explicit guidance to the read_command_output tool description telling models to omit the search parameter entirely when not filtering, rather than passing null or empty string.

This prevents models from mistakenly passing the literal string "null" as a search pattern, which causes the tool to search for lines containing the text "null" instead of returning all content.

Changes

• Updated the search parameter description in the tool's overall description to include: "Omit this parameter entirely if you don't need to filter - do not pass null or empty string."
• Updated the SEARCH_DESCRIPTION constant with the same guidance

Context

Investigation found that models sometimes pass "search": "null" when they intend "no search filter". The tool was designed with strict: false specifically so models could omit optional parameters. This documentation update teaches models the correct behavior.

---

Important

Clarifies search parameter usage in read_command_output.ts to omit it when not filtering.

• Documentation:
  • Updated search parameter description in read_command_output.ts to instruct omitting it if not filtering.
  • Updated SEARCH_DESCRIPTION constant with the same guidance.

^{This description was created by for c2f6d00. You can customize this summary. It will automatically update as commits are pushed.}

[roomote]

Oroocle   See task on Roo Cloud

Review complete. No issues flagged.

• No issues to address

_{Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.}