T8 #1633
Merged
T8 #1633
Conversation
This file contains hidden or 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
… response handling
nmaguiar
added
bug
enhancement
dependencies
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request adds support for the TOON format (a structured data serialization format) as both an input and output option throughout the OpenAF ecosystem. The changes introduce the TOON library, integrate it into the OpenAF core functionality, and update completion scripts to recognize the new format.
Changes:
- Added toon.js library (v2.1.0) as an external dependency
- Integrated TOON format support into OpenAF core APIs (AF.toTOON, AF.fromTOON, $path functions)
- Updated oafp tool to support TOON as input (in=toon) and output (out=toon) format
- Added TOON format to shell completion scripts and metadata
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| versionsAndDeps.json | Adds toon.js library as external dependency with version 2.1.0 |
| js/toon.js | New minified TOON library file for encoding/decoding TOON format |
| js/repack.js | Adds loadTOON() call to shared archive initialization sequence |
| js/owrap.template.js | Adds $toTOON helper function to Handlebars templates |
| js/owrap.oJob.js | Adds "python" to supported job languages list |
| js/owrap.ai.js | Adds jsonPromptWithStatsRaw and promptJSONWithStatsRaw methods |
| js/openaf.js | Implements AF.toTOON/fromTOON methods, loadTOON function, $path integration, and $output support |
| js/oafp.js | Adds TOON input/output format handling and version update |
| complete/completion_oafp.yaml | Adds in=toon and out=toon completion metadata |
| complete/completion_oafp.sh | Adds in=toon and out=toon shell completion support |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
github-project-automation
bot
moved this from Backlog
to Done
in Continuous Enhancement
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.
This pull request adds support for a new "TOON" format as both an input and output option in the completion scripts and their metadata. The changes ensure that the
toonformat is recognized, handled, and documented consistently alongside other supported formats.Input and output format support:
toonas a supported input format, including flag handling, argument parsing, and help text incompletion_oafp.sh. [1] [2] [3]toonas a supported output format, including flag handling, argument parsing, and help text incompletion_oafp.sh. [1] [2] [3]Metadata/documentation updates:
in=toonandout=toonoptions in the completion metadata filecompletion_oafp.yaml, including descriptions for both. [1] [2]