fix(addie): make agent registration self-explanatory#3781
Merged
Conversation
1513635 to
ab69b1d
Compare
- Rename dashboard CTA "+ Add agent" → "+ Register agent" - Replace seed prompt sent to Addie. Old: "I want to add an agent for compliance monitoring." (compliance framing was confusing for first-time registration). New: "I want to register an agent in the AAO registry. What information do you require?" Updated in dashboard-agents.html (header CTA, empty-state CTA) and org-health.ts (org-health quick action) - Reframe Addie's save_agent description around AAO registry registration; surface all four auth modes (none / static bearer-or-basic / OAuth client credentials / OAuth user) and state explicitly that agent type is auto-detected — never asked - Teach Addie to drive a structured intake (URL → auth method → matching fields → protocol) before calling save_agent - Add a "Quickstart: register your agent (members)" section to docs/registry/registering-an-agent.mdx with the explicit sign-up → /dashboard/agents → "+ Register agent" path
The bare /login route returns 404; the actual entry point is /auth/login, which redirects to WorkOS AuthKit and handles both sign-in and sign-up. Caught by check:owned-links in CI.
Remove the duplicate "Quickstart" section that sat above the four-paths list and fold its content into "1. Self-registration (members only)" so there's one canonical place a member learns the click path. Also fixes the ?org parameter (id, not slug), aligns tier wording with the live gate, and describes type resolution accurately (capability snapshot, not live probe).
#3780 removed `source: registered/discovered` from the public registry surface and collapsed the four-paths model. Update the changeset description to match — the docs change now expands the one enrollment path that remains, not "section 1" of a four-section list.
c135671 to
1b6a478
Compare
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.
Summary
Make member agent registration self-explanatory.
Four issues with the existing flow surfaced from a real user attempt:
+ Add agentdid not match the registry concept of enrolled agents.I want to add an agent for compliance monitoring.to Addie — compliance monitoring is a downstream benefit, not the reason to register.docs/registry/registering-an-agent.mdxdid not show a member how to actually get from the dashboard to a registered agent.This PR makes the path obvious end-to-end:
+ Add agent→+ Register agenton/dashboard/agents(header + empty state).dashboard-agents.htmlx2,services/org-health.ts) now sendHelp me register my agent.— natural first-person phrasing instead of compliance jargon. Centralized as aREGISTER_AGENT_SEED_PROMPTconstant indashboard-agents.html, mirrored inorg-health.ts.save_agentto lead with AAO registry registration, list the four persistable auth modes (none / static bearer / static basic / OAuth client credentials), and explicitly state that agent type is server-resolved from a capability snapshot — never asked. Trimmed by ~45% by pushing the intake script intobehaviors.md.behaviors.mdwalks Addie through a structured intake (URL → auth method choice → matching fields → protocol) before callingsave_agent. Includes a paste-it-all shortcut for power users, an explicit override of the "act immediately on a pasted URL" rule when intent is registration, a no-echo-secrets rule, and an error-recovery branch (probe timeout, auth rejected, validation, permission denied).None, then click Authorize on the agent card.save_agentno longer advertises a fifth auth mode it can't actually persist.docs/registry/registering-an-agent.mdxwith the explicit click path: sign in →/dashboard/agents→+ Register agent→ what Addie will ask. Adopts the post-feat(registry): remove discovered agents from public surface #3780 single-enrollment-path framing (nosource: registered/discovered, no four-paths model — those are gone from the public registry surface). Aligns tier wording with the live gate (Professional tier or higher) and describes type resolution as a capability-snapshot read, not a live probe./loginwas 404ing; replaced with/auth/login(AuthKit handles both sign-in and sign-up via that route). Caught bycheck:owned-linksin CI.Rebase notes
Rebased on
mainafter #3780 merged. The four-paths /source: registered|discoveredframing this PR originally extended is gone from the public registry surface, so the docs change now adapts the click-path walkthrough into main's single-enrollment-path structure rather than restructuring around a "section 1" that no longer exists.Test plan
/dashboard/agents, the header CTA reads+ Register agentand the empty-state CTA readsRegister your first agent./chatwith Addie pre-loaded withHelp me register my agent..save_agent.org-healthquick actions, the "Register a buyer/seller agent" link uses the same updated seed prompt.## How agents end up in the registrycontains the numbered click-path; no stalesource: registered/discoveredor four-paths language remains./auth/loginlinks in the docs and Addie behaviors resolve (no 404).