Skip to content

Offer Custom endpoint from the Alt+A add-provider selector - #471

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-5899-offer-custom-endpoint-from-the-alta-add-provider-selector
Aug 10, 2026
Merged

Offer Custom endpoint from the Alt+A add-provider selector#471
TheGreatAxios merged 1 commit into
mainfrom
cl-5899-offer-custom-endpoint-from-the-alta-add-provider-selector

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Alt+A add-provider selector includes Custom.
  • Custom uses the existing full manual form; successful connect refreshes the catalog and focuses the new default model.
  • Escape returns safely to the selector/model list.

Closes CL-5899.

Verification

  • bun run typecheck
  • bun run build
  • bun run test (4539 pass, 0 fail)

@linear-code

linear-code Bot commented Aug 10, 2026

Copy link
Copy Markdown

CL-5899

@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 correctness of Alt+A Custom entry
Verdict: approve
Summary: Custom is listed in the Alt+A selector, uses the existing full manual form, and post-connect focus still goes through the shared connect path.

Findings: none.

Greybeard

Lens: architecture and documentation alignment
Verdict: approve
Summary: Selector construction is centralized in addProviderSelectorChoices; TUI/IMPLEMENTATION/CHANGELOG note Custom is reachable mid-session.

Findings: none.

gaasbot

Lens: public OSS / release readiness
Verdict: approve
Summary: Small, clear surface change with tests; safe for public release.

Findings: none.

The add-provider list now includes Custom alongside first-class kinds so
operators can connect a free-form endpoint without leaving the model picker.
@TheGreatAxios
TheGreatAxios force-pushed the cl-5899-offer-custom-endpoint-from-the-alta-add-provider-selector branch from f21d305 to c0c6d01 Compare August 10, 2026 18:37
@TheGreatAxios
TheGreatAxios merged commit aaa5209 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