feat(dashboard): 'Connect your agent' panel (API token UI) — landed post-#11#13
Merged
Merged
Conversation
Settings gains a 'Connect your agent' card (team mode only): lists the signed-in member's per-user API tokens, creates one (label + one-time token shown once with a ready-to-paste 'Authorization: Bearer ...' line), and revokes. Wired into loadSettings(). This UI was developed on feat/agent-connect but did not land in the #11 merge (only the backend did), so main had the agent-connect API with no UI to mint tokens. This adds the fixed version (renderTokList so the one-time token is not wiped on re-render; correct /api/remember + /api/recall snippet URLs).
Owner
Author
Council Gate Verdict — APPROVE_WITH_NITSRisk level: high | Agreement: 0.5 | Council session: Per-model scores (rubric: correctness / safety / style / tests / complexity)
Engraphis referenceCouncil verdict persisted as
|
Coding-Dev-Tools
added
the
council-approved
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.
What
The agent-connect backend (per-user tokens,
/api/remember, gate) merged in #11, but the frontend panel commit (b30b81c) was not in that merge — somainhas the API with no UI to mint tokens. This PR adds the fixed panel.What it adds (Settings → new 'Connect your agent' card, team-mode only)
Authorization: Bearer …line.Includes the two fixes I made during the accuracy review
POST <base>/api/remember/GET <base>/api/recall(the earlier version dropped/api).renderTokList()split out so creating a token doesn't re-render the panel and wipe the one-time token before the user can copy it.Verified
node --checkon the inline JS: OK. LF line endings (no CRLF pollution).test_dashboard_v2.py+test_agent_connect.py: 46 passed. Dashboard serves 200 with the card present.Notes
feat/agent-connect(b30b81c+ fix2f9d474), applied to currentmain(whoseindex.htmlhad no panel). The backend it calls (/api/auth/token,/api/auth/connect-info,/api/remember) is already onmainfrom feat(agent-connect): per-user API tokens + Team-gated /api/remember #11.