Skip to content

Conversation

@zhutao100
Copy link
Contributor

The functionResponse should contain the content of the function response, not the generic message for default/unhandled case.

This change fixes commonly used tools like ReadManyFiles, making them correctly return the file contents to LLM.

TLDR

This change fixes commonly used tools like ReadManyFiles, making them correctly return the file contents to LLM.

Reviewer Test Plan

The difference before/after the change can be directly verified by

  • open qwen-code in a directory containing some files.
  • Write instructions like "read @file1 and @FILE2 to understand them".
  • Catch and view the tool request to the server.

Testing Matrix

🍏 🪟 🐧
npm run 👍
npx 👍
Docker
Podman - -
Seatbelt - -

The functionResponse should contain the content of the function response, not the generic message for default/unhandled case.

This change fixes commonly used tools like ReadManyFiles, making them correctly return the file contents to LLM.
@zhutao100 zhutao100 force-pushed the pr-functionResponseArray branch from a6360af to f6b4b41 Compare November 1, 2025 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant