Skip to content

Sync PRODUCT docs with Alt+A models-only connect - #473

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-5901-sync-product-docs-with-alta-models-only-connect
Aug 10, 2026
Merged

Sync PRODUCT docs with Alt+A models-only connect#473
TheGreatAxios merged 1 commit into
mainfrom
cl-5901-sync-product-docs-with-alta-models-only-connect

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

Align product and implementation docs plus operator-facing auth messages with the connect stack that just landed (CL-5898/5899/5900):

  • /model is models-only (connected accounts); Alt+A is the add-provider path
  • Remove stale bare-c / Ctrl+A / in-list “connect →” instructions
  • Document named OAuth accounts and named API-key multi-instance behavior as shipped
  • Point auth-failure guidance at Alt+A

Linear: CL-5901

Verification

  • bun run typecheck
  • bun run build
  • bun run test (4547 pass)

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.
@linear-code

linear-code Bot commented Aug 10, 2026

Copy link
Copy Markdown

CL-5901

@TheGreatAxios TheGreatAxios left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 TheGreatAxios left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 TheGreatAxios left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 TheGreatAxios left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@TheGreatAxios

Copy link
Copy Markdown
Collaborator Author

Reviewer-of-record

Lens: branch vs origin/main, commit style, full diff
Verdict: clean
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

Critique

Lens: behavioral accuracy of user-facing copy
Verdict: clean
Summary: PRODUCT and IMPLEMENTATION 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

Greybeard

Lens: architecture/docs consistency after CL-5898/5899/5900
Verdict: clean
Summary: Documentation and auth-failure guidance track the multi-instance connect contract on main.

Findings: none

gaasbot

Lens: public OSS release readiness
Verdict: clean
Summary: Operator docs and error strings are suitable for public users; no secrets or internal artifacts.

Findings: none

@TheGreatAxios
TheGreatAxios merged commit 2c4effc into main Aug 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant