Skip to content
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.

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

feat:Add support zip, tar.gz and git repo uploading as files #144

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

sjurajpuchky
Copy link

Adding support of uploading zip and tar.gz files, files are extracted, sanitized filenames and filtred by supported extensions. Also folders like venv, vendor, node_modules are ignored. Support of upload files directly from git repository is also added. There is limit of 10 files to be uploaded at once.

@paulmikulskis
Copy link

this is sweet, can we get this merged in VRSEN?

@sjurajpuchky
Copy link
Author

sjurajpuchky commented Oct 2, 2024

this is sweet, can we get this merged in VRSEN?

sorry for delay, I did get question right now and have to tell you what a lovely idea i would be pleased 👍

@sjurajpuchky
Copy link
Author

@paulmikulskis let's continue on work there are lot of stuff what realy disturbs me when i use framework for my work pains me so much to see that kind of non senses 💯

@VRSEN
Copy link
Owner

VRSEN commented Oct 23, 2024

Sorry for a delay guys, @sjurajpuchky, can you rebase this on main?

@sjurajpuchky
Copy link
Author

sjurajpuchky commented Nov 4, 2024

Sorry for a delay guys, @sjurajpuchky, can you rebase this on main

@VRSEN conflicts are resolved, waits for approval

print(f"Uploaded file ID: {file.id}")

if file_path.endswith('.zip'):
extracted_files.extend(extract_zip(file_path, to_folder))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Thank you for your contribution, @sjurajpuchky ! Could you explain why you consider this functionality part of the core agent orchestration framework? Could it be implemented separately, either as a tool or a standalone service?

Copy link
Author

Choose a reason for hiding this comment

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

Hi bonk1t, like to happen. Main reason was to availability upload more files at once with zip file since, there is a limit of 10 files (what i would like to solve in future) and one zip file is more confortable.

In date of PR, was zip files not supported.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@sjurajpuchky Could you update the PR to include only the minimal necessary changes? I’ll review and merge it. Cheers!

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.

5 participants