Skip to content

add read_media #902

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

add read_media #902

wants to merge 4 commits into from

Conversation

s97712
Copy link
Contributor

@s97712 s97712 commented Jun 27, 2025

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Jun 27, 2025 •
edited
Loading

⚠️ No Changeset found

Latest commit: ab69de2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

s97712 added 2 commits June 28, 2025 11:19
@s97712
feat(tools): Refactor and improve readMediaTool
3485c83 
This commit introduces several improvements to the readMediaTool:

- **Use asynchronous file reading:** Switched from `readFileSync` to `fs.promises.readFile` to prevent blocking the event loop and improve performance with large media files.
- **Unify error response format:** The tool's error XML is now wrapped in a `<files>` tag, aligning its error reporting format with `readFileTool` for better consistency across the system.
- **Enhance documentation:** The tool's documentation has been updated to clarify its specific use case for multimodal recognition, distinguishing it from the `readFileTool`.
@s97712
feat: add auto-approval for read media operations
ab69de2
@chrarnoldus
Copy link
Contributor

Thank you for your pull request. The ability to read images was added to the read_file tool in https://github.com/Kilo-Org/kilocode/releases/tag/v4.73.0

@chrarnoldus chrarnoldus closed this Aug 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@s97712 @chrarnoldus