Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

Integrate markdown converter for supported binary file types#129

Merged
ryanhoangt merged 13 commits into
mainfrom
file-viewer-gaia
Jun 2, 2025
Merged

Integrate markdown converter for supported binary file types#129
ryanhoangt merged 13 commits into
mainfrom
file-viewer-gaia

Conversation

@ryanhoangt
Copy link
Copy Markdown
Contributor

Description

Ported from #126.

@ryanhoangt ryanhoangt requested a review from xingyaoww May 27, 2025 11:19
Comment thread openhands_aci/editor/editor.py
Comment thread openhands_aci/editor/md_converter.py Outdated
Comment thread openhands_aci/editor/md_converter.py Outdated
FILE_CONTENT_TRUNCATED_NOTICE: str = '<response clipped><NOTE>Due to the max output limit, only part of this file has been shown to you. You should retry this tool after you have searched inside the file with `grep -n` in order to find the line numbers of what you are looking for.</NOTE>'
TEXT_FILE_CONTENT_TRUNCATED_NOTICE: str = '<response clipped><NOTE>Due to the max output limit, only part of this file has been shown to you. You should retry this tool after you have searched inside the file with `grep -n` in order to find the line numbers of what you are looking for.</NOTE>'

BINARY_FILE_CONTENT_TRUNCATED_NOTICE: str = '<response clipped><NOTE>Due to the max output limit, only part of this file has been shown to you. Please use Python libraries to view the entire file or search for specific content within the file.</NOTE>'
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I've been thinking - we should be able to potentially add pagination for this.. but probably a future PR

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah that makes sense, I created an issue to track this: #130

Comment thread tests/unit/test_file_validation.py
@ryanhoangt ryanhoangt requested a review from xingyaoww June 2, 2025 14:58
@ryanhoangt ryanhoangt merged commit 6bd03e6 into main Jun 2, 2025
4 checks passed
@ryanhoangt ryanhoangt deleted the file-viewer-gaia branch June 2, 2025 15:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants