Redesign personal AI connection management and setup - #324
Merged
Maneek21 merged 3 commits intoSep 9, 2026
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
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.
Personal AI connections previously mixed setup, permissions, tokens, OAuth grants and developer information into one dense page. This change gives the page a searchable connection list and a separate setup flow, with access and activity available on demand.
Connections have inline revoke confirmation and recoverable error states. Setup presents a plain-language access selector, client-specific instructions, once-only token acknowledgement and verification tied to the selected connection. Drafts survive Back and Resume; completing setup clears the acknowledged credential. Partial refresh failures cannot silently appear as an empty or current inventory.
Validation: web typecheck and full web lint passed; three settings navigation tests passed; smoke-script syntax and git diff checks passed. Rendered all seven client paths at 390 and 1440 CSS px, plus expanded revocation at 320 px, and inspected dark and light layouts. Synthetic UI checks covered search, empty/loading/error recovery, token and grant revocation, failed revoke/retry, draft retention, permission validation, acknowledgement, local copy feedback, pending/observed verification and focus restoration.
The updated production smoke adds disposable seeded token issuance and revocation; its execution remains pending. Local screenshots and mutation checks used synthetic fixture data, not live OAuth or persistent credentials. Browser zoom, screen-reader acceptance, real integration evidence and independent usability review remain open.
Stacked on #323. Do not merge before user review. No merge or auto-merge is authorized.