Repository navigation
Discover viewed settings and reuse the advertised session form - #1755
Merged
Merged
Conversation
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
|
debt-src at a9965dc blocks on 15 lines: |
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.
Viewed settings: discovery and original writers
The sidebar used local-override eligibility to decide which settings existed. Files therefore exposed four UI fields and hid every other settings group. Native session advertisements were also confined to the model popup.
The dropdown now uses FormBridge's visible parameter discovery. Each field keeps its original declaration. Local fields still use the same ObjectState parent and explicit override storage. Global fields write and save the original SavedSettings object; their controls say they apply to every view and do not offer a false local override button. Hidden declarations remain hidden through FormBridge.
Native session sources supply AgentProperties. Mailbox sources supply no ACP configuration; channel and file views do not borrow the last agent's state. AgentProperties renders SessionForm over the selected session's actual advertisements and listens to its original event stream, including binding changes. The existing model popup inherits the same form, adding only popup dismissal. All edits and provider-login requests return through the original prompt and its guarded backend writer.
Applied through retained NRA plans. No manual product edits or new agent.
Executed controls
Actual physical inspection and installation are recorded below after execution; these checks do not establish them.
Limits
This exposes the existing visible Toad settings declarations and the session's actual ACP advertisements. It does not invent advertisements for backend settings that the backend has not declared, or make app-owned operations into per-file overrides.