Skip to content

fix(web): stop phrasing unknown provider auth as a verification failure - #7934

Closed
Swayam2004 wants to merge 1 commit into
pingdotgg:mainfrom
Swayam2004:fix/grok-auth-status-copy
Closed

fix(web): stop phrasing unknown provider auth as a verification failure#7934
Swayam2004 wants to merge 1 commit into
pingdotgg:mainfrom
Swayam2004:fix/grok-auth-status-copy

Conversation

@Swayam2004

@Swayam2004 Swayam2004 commented Aug 22, 2026

Copy link
Copy Markdown

Providers whose health check fully succeeds but never reports an auth result render this detail on every cycle: "Installed and ready, but authentication could not be verified." Nothing failed. The status is unknown, and the copy reads as an auth failure. Grok hits this every five minutes (#7932).

Reworded the final fallback in getProviderSummary. Server-supplied messages still take precedence, and all other branches are untouched.

Before: Installed and ready, but authentication could not be verified.
After: Installed and ready. Authentication status was not reported.

Adds a focused test covering the fallback branch. Happy to attach real before/after captures if wanted.

Work done by ox-alpha (opencode harness).


Note

Low Risk
UI copy-only change in provider status text plus unit tests; no auth or backend behavior changes.

Overview
Ready providers with unknown auth no longer look like they failed verification. The fallback detail in getProviderSummary now says authentication was not reported instead of implying a check failed.

Server-supplied messages still win. Adds tests for the fallback, message override, and authenticated headline.

Reviewed by Cursor Bugbot for commit 03d7fe1. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Reword unknown provider auth detail in getProviderSummary to not imply verification failure

Changes the fallback detail string for available-but-unverified providers from 'Installed and ready, but authentication could not be verified.' to 'Installed and ready. Authentication status was not reported.' Adds tests in providerStatus.test.ts covering the fallback case, server-supplied message override, and authenticated provider headline.

Macroscope summarized 03d7fe1.

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 341567ad-4d5a-4a52-a1de-617c7156d9e6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Aug 22, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Skipped

Macroscope did not run approvability analysis for this PR. Macroscope could not determine whether this PR modifies its approvability configuration, so the PR was not approved automatically. A PR that may change the rules that govern approval is never approved automatically.

@t3dotgg

t3dotgg commented Aug 27, 2026

Copy link
Copy Markdown
Member

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

Closing this duplicate wording change in favor of #7943. Both distinguish a ready provider with unreported authentication from an actual authentication failure. Keep the regression test in the retained PR. The Grok reliability merge does not by itself remove the fallback for every provider, so the status-copy fix still needs review.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed. If GitHub does not let you reopen it, leave a comment here and we'll take another look.

@t3dotgg t3dotgg closed this Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants