Repository navigation
Declare each provider's defaults once, in plugin.toml - #65
Merged
Merged
Conversation
thesiti92
force-pushed
the
feat/provider-defaults
branch
from
October 1, 2026 14:22
79c3b5a to
ccba959
Compare
thesiti92
added this pull request to stack #66
October 1, 2026 14:23
sidkmenon
approved these changes
Oct 1, 2026
An option's default can follow another option's value through x-default-by, which the host fills in like any default and the manifest check validates for every choice. The summarizer's model now follows its provider, and its endpoints, key variables and keyless rule come from a provider_details option instead of code. Provider titles ship as x-enum-titles, so settings screens need no copy of any of it. Table pruning now tries every table, since an object option is only valid whole.
fill_defaults repeats until a round fills nothing, so a chain resolves whatever the declaration order, and the manifest check rejects an x-default-by cycle.
thesiti92
force-pushed
the
feat/provider-defaults
branch
from
October 1, 2026 14:56
ccba959 to
bb5bb0d
Compare
sidkmenon
approved these changes
Oct 1, 2026
Merged
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
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.
Stacked on #62. Settings screens such as Whiteboard's no longer need their own copy of any summarizer default.
Changes
x-default-by. An option's default can depend on another option's value:{ key = "provider", values = { gemini = …, openai = …, anthropic = … } }.config showand pruning treat them the same.modeldefaults per provider:gemini-3.8-flash,gpt-6-luna,claude-haiku-4-5. A CLI user who sets onlyprovider = "openai"getsgpt-6-luna, and a model equal to its provider's default is pruned.default.tomlno longer pinsmodel.provider_details. This new option, derived the same way, holds each provider's default endpoint, key variables, and whether a custom endpoint may go without a key. The summarizer reads these from its options instead of code.x-enum-titles. Provider display names ship in the schema.Verified
cargo testandcargo xtask test-pluginspass.providerset.