Sync PRODUCT docs with Alt+A models-only connect - #473
Conversation
PRODUCT, IMPLEMENTATION, and operator-facing auth error strings still taught bare-c / Ctrl+A connect after the picker became models-only with Alt+A. Sync them to the shipped contract: named OAuth and API-key instances, Custom from Alt+A, no printable Connect key under type-to-filter.
TheGreatAxios
left a comment
There was a problem hiding this comment.
Reviewer-of-record
Lens: branch vs origin/main, commit style, full diff
Verdict: approve
Summary: Docs and operator-facing auth strings match the shipped models-only /model + Alt+A connect stack; no code behavior change beyond message text.
Findings: none
Gate notes: single commit a5739ec, subject plain-English imperative under 72 chars, full gate green in worktree (typecheck/build/test 4547).
TheGreatAxios
left a comment
There was a problem hiding this comment.
Critique
Lens: behavioral accuracy and maintainability of user-facing copy
Verdict: approve
Summary: PRODUCT and IMPLEMENTATION now describe models-only list, Alt+A add-provider, named OAuth/API-key instances, and Custom without teaching printable Connect keys under type-to-filter.
Findings: none
TheGreatAxios
left a comment
There was a problem hiding this comment.
Greybeard
Lens: architecture/docs consistency after CL-5898/5899/5900
Verdict: approve
Summary: Documentation and auth-failure guidance track the multi-instance connect contract that already landed on main; no speculative prose.
Findings: none
TheGreatAxios
left a comment
There was a problem hiding this comment.
gaasbot
Lens: public OSS release readiness
Verdict: approve
Summary: Operator docs and error strings are suitable for public users; no secrets, machine paths, or internal-only assumptions introduced.
Findings: none
Reviewer-of-recordLens: branch vs origin/main, commit style, full diff Findings: none CritiqueLens: behavioral accuracy of user-facing copy Findings: none GreybeardLens: architecture/docs consistency after CL-5898/5899/5900 Findings: none gaasbotLens: public OSS release readiness Findings: none |
Summary
Align product and implementation docs plus operator-facing auth messages with the connect stack that just landed (CL-5898/5899/5900):
/modelis models-only (connected accounts); Alt+A is the add-provider pathc/ Ctrl+A / in-list “connect →” instructionsLinear: CL-5901
Verification
bun run typecheckbun run buildbun run test(4547 pass)