feat: describegpt add TOON format support
#3205
Merged
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
- use our patched fork until our PR is merged
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds TOON (Tabular Object Oriented Notation) format support to the describegpt command, providing a compact, human-readable alternative to JSON output that's optimized for LLM prompts.
Key Changes:
- Adds TOON as a new output format option alongside Markdown, TSV, and JSON
- Integrates the
toon-formatcrate (v0.4) with a patched fork to gate CLI dependencies - Updates help text and error messages to include TOON in format options
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/cmd/describegpt.rs | Adds TOON enum variant, imports toon-format library, implements TOON output logic in dictionary and general response handling, updates format validation messages |
| docs/nyc311-describegpt.toon | Provides example TOON-formatted output demonstrating the new format on NYC 311 dataset |
| README.md | Documents TOON format availability and links to toonformat.dev for more information |
| Cargo.toml | Adds toon-format dependency and patch directive for forked version |
| Cargo.lock | Locks toon-format crate to specific commit on patched branch |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
resolves #3204