Skip to content

fix(providers): show a No Auth provider's real requirement on its banner - #15363

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.52from
HouMinXi:fix/noauth-banner
Oct 6, 2026
Merged

diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.52from
HouMinXi:fix/noauth-banner

Conversation

@HouMinXi

@HouMinXi HouMinXi commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The No Auth section showed every provider the same banner, saying it was ready to use with no signup and no API key. That is wrong for providers that need a local component, a browser binary or a matching client.
  • The real requirement was already written in each provider's notice in noauth.ts. The banner now appends that notice when one exists, and a provider without one keeps the original banner.
  • No new copy was written.

Related Issues

Validation

  • Change type: other (dashboard banner)
  • Focused tests: node --import tsx/esm --test tests/unit/noauth-banner-requirement.test.ts (3 pass)
  • eslint on the changed files, with the repo suppression file
  • Reconciled with release/v3.8.52 @ 23a1148486
  • New test covers the change
  • Defect injection: removing the two resolver calls turns the cloudflare and opencode cases red while the duckduckgo case stays green, and the change was restored

Tests Added Or Updated

  • tests/unit/noauth-banner-requirement.test.ts (added)

Coverage Notes

  • The test checks a provider with a notice, a provider without one, and the account card path.

Reviewer Notes

  • authHint is not used. Only notice.text is appended, because that is the field that states a hard prerequisite.

@HouMinXi
HouMinXi requested a review from diegosouzapw as a code owner October 2, 2026 17:15
The No Auth section told every provider it was ready to use with no
signup and no API key. Several of them need a local component, a browser
binary or a matching client, and that requirement was already written in
each provider's notice. The banner now appends that notice when one
exists and stays unchanged when it does not.

Signed-off-by: Minxi Hou <houminxi@gmail.com>
@diegosouzapw
diegosouzapw merged commit a45b818 into diegosouzapw:release/v3.8.52 Oct 6, 2026
42 of 51 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

2 participants