feature(client): tool parameters raw input mode, togglegroup component #618
+254
−126
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.
Tool parameters raw input mode, togglegroup ui component
Hi guys! Needed this one for myself, created a PR)
Form/Raw switch for tool parameters input.
Raw mode uses
DynamicJsonForm
.Added
forceSimpleJson
property toDynamicJsonForm
to hide it's own form/json switch.New ui component
togglegroup
with similar styling as tabs used for switch, might be useful for other features.Written as simple as possible, in the project's codestyle.
Motivation and Context
In some tools, empty string instead null is required. But input form converts empty strings to nulls.
So the quick solution is to add DynamicJsonForm as the option to edit tool parameters.
This is usefull if you want more contorl over parameters value, bypass schema, use empty strings, etc.
How Has This Been Tested?
Manually, feature is very basic.
Breaking Changes
Nothing
Types of changes
Checklist
Additional context
Need advice for necessary docs and tests for this feature.