File manager zip file support, folder support and clean up #273
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
-update file manager skill to handle extracting .zip files and compressing files and folders to zip -update file manager skill to read all supported file types in a designated folder - this should help with coding projects so that the AI can actually understand how a project of files relates to each other, as well as potentially for a forthcoming "WingmanAI Helper / Tutorial Wingman" as this skill would allow the helper or tutorial to actually read the files or logs in its own directory without specifying the actual names of each file -Clean up - make variables that perform materially similar functions in the various tools the same, both to avoid AI confusion and to allow consolidation of fetching all possible relevant parameters early in the execute_tool function to avoid unnecessary repetition throughout -Clean-up: implement some measures in default_config.yaml skills description and how the tool parameters are worded intended to mitigate times when AI will respond it needs a specific file to read out loud (per testers of reading aloud capability) by also allowing the wingman to read designated text aloud not just files. -Clean-up: add some additional possible text file formats originally left out in original such as .yml as another version of .yaml -The zip file functionality should be tested on mac to be sure it works (assuming zipping files works on macs at all); tested with Windows and it works fine. It apparently depends on having "zlib" according to zipfile documentation - I'm assuming that comes with Wingman MacOS too but worth a test or two to be sure.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
-update file manager skill to handle extracting .zip files and compressing files and folders to zip
-update file manager skill to read all supported file types in a designated folder - this should help with coding projects so that the AI can actually understand how a project of files relates to each other, as well as potentially for a forthcoming "WingmanAI Helper / Tutorial Wingman" as this skill would allow the helper or tutorial to actually read the files or logs in its own directory without specifying the actual names of each file
-Clean up - make variables that perform materially similar functions in the various tools the same, both to avoid AI confusion and to allow consolidation of fetching all possible relevant parameters early in the execute_tool function to avoid unnecessary repetition throughout
-Clean-up: implement some measures in default_config.yaml skills description and how the tool parameters are worded intended to mitigate times when AI will respond it needs a specific file to read out loud (per testers of reading aloud capability) by also allowing the wingman to read designated text aloud not just files.
-Clean-up: add some additional possible text file formats originally left out in original such as .yml as another version of .yaml
-The zip file functionality should be tested on mac to be sure it works (assuming zipping files works on macs at all); tested with Windows and it works fine. It apparently depends on having "zlib" according to zipfile documentation - I'm assuming that comes with Wingman MacOS too but worth a test or two to be sure.