feat: update specification schema to mark optional fields with '-' in…#134
Merged
feat: update specification schema to mark optional fields with '-' in…#134
Conversation
…stead of '—' fix: adjust tool-renderer prop types to use '-' for optional fields feat: enhance UI integration by wrapping GeneratedCommand in Card component refactor: simplify command generation logic using generateCommand utility fix: improve tool loading logic to handle local storage correctly feat: add description and URL fields to new tool creation dialog test: add defaultTool utility for consistent tool object creation in tests test: enhance command-tree tests for rendering behavior with commands test: update command-dialog tests to use defaultTool utility test: improve parameter-details-dialog tests for parameter validation test: add tests for saved-commands-dialog to check long command rendering test: enhance help-menu tests to verify tool preview generation test: update parameter-list tests to use defaultTool utility test: improve preview-tabs tests for long flag toggle visibility test: enhance tool-editor tests for clipboard functionality and separator cleanup test: update tools test to use defaultTool utility for consistency Co-authored-by: Copilot <copilot@github.com>
…ad of 'name' Co-authored-by: Copilot <copilot@github.com>
… theme switcher functionality
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
fix: improve tool loading logic to handle local storage correctly
This pull request refactors and extends the command generation logic for the Commandly UI, making the code more modular, maintainable, and feature-rich. The main improvements are the extraction of command generation into a reusable utility function, the introduction of a context-based compound component API for the
GeneratedCommandcomponent, and the addition of a user preference for long or short flags in generated commands. The changes also include utility and cleanup improvements for handling tool definitions.Component refactor and new features:
GeneratedCommandinto a compound component with context, splitting out UI elements (header, toolbar, output, actions, flag preference switch, etc.) and enabling more flexible composition and extension.generateCommand, which supports the long/short flag preference and is now used in the component. [1] [2]Utility and codebase improvements:
generateCommandutility function toutils/flat.ts, supporting options for long/short flags and selected commands, and improved parameter handling.fixToolfunction for normalizing tool definitions.feat: add description and URL fields to new tool creation dialog