[UtkarshUsername]

Problem

Source Control settings show yellow warning dots for inactive providers, including providers whose CLI or credentials have never been configured. This makes optional, inactive integrations look like problems requiring attention.

Change

Use grey dots for unavailable or unauthenticated source control providers and a neutral Not authenticated badge. Available, authenticated providers retain green dots. Version control system indicators retain their existing behavior.

The component is shared by web and desktop. Mobile does not render this provider discovery list. No contracts, provider behavior, or connection handling change.

Scope and approval

This is a small, focused presentation fix for misleading warning indicators on inactive optional integrations.

Verification

Before:

After:

• vp lint apps/web/src/components/settings/SourceControlSettings.tsx passed.
• vp fmt --check apps/web/src/components/settings/SourceControlSettings.tsx passed.
• vp run typecheck in apps/web passed, with existing Effect suggestions in unrelated connection files.
• Reviewed the status branches against the read-only availability switches: available and authenticated stays green; other provider states become neutral.

Model: gpt-6.1-sol. Harness: Codex.

[macroscopeapp]

Approvability

Verdict: Approved at 5912057

Macroscope's review found this PR approvable — This is a narrowly scoped web UI fix that changes only badge and status-dot styling for inactive or unauthenticated source-control providers. It has no apparent impact on source-control operations, data, authentication behavior, or production configuration.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 9ebca5f
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 8a29ebb6-9562-444e-8c6d-19e002df707a

Commits

Reviewing files that changed from the base of the PR and between 5912057 and 1d977bc.

Files selected for processing: 1

• apps/web/src/components/settings/SourceControlSettings.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

Walkthrough

Source-control settings now use a secondary badge for unauthenticated status. Provider status dots show success only when the provider is available and authenticated.

Changes: Source-control status presentation

Layer / File(s)	Summary
Update authentication and provider status presentation apps/web/src/components/settings/SourceControlSettings.tsx	Unauthenticated status uses a secondary badge. Provider status dots use success only when available and authenticated; otherwise they use muted color. Unavailable non-provider VCS items retain the warning status.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: Minimal · up to 1d977

The provider status styling change is limited to neutral indicators for inactive or unauthenticated providers.

Architecture risk: Low · up to 59120

The change affects 1 system.

Changed systems: apps/web

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

• observed — apps/web (ui) was modified; 1 changed file maps to changed impact.

Before / after behavior

• observed — Modified behavior in apps/web/src/components/settings/SourceControlSettings.tsx: Changed unauthenticated authentication presentation from a warning badge to a secondary badge.
• observed — Modified behavior in apps/web/src/components/settings/SourceControlSettings.tsx: Provider items now use a success dot only when available and authenticated, and muted otherwise; the warning status branch remains for unavailable non-provider VCS items.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly and concisely describes the main change: inactive source control providers use neutral indicators.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Description check		The pull request description includes all required sections. It clearly states the problem, expected behavior, implementation change, scope rationale, verification steps, before/after screenshots, tes…

Full details: Description check

Explanation

The pull request description includes all required sections. It clearly states the problem, expected behavior, implementation change, scope rationale, verification steps, before/after screenshots, test results, limitations, and agent details.

Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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