fix(swift-ios): show cached source-control status immediately

[saphid]

Summary

Uses the VCS subscription snapshot for fast cached status, falls back to an explicit refresh on timeout/subscription failure, and preserves toolbar/pull-to-refresh semantics.

Verification

• apps/swift-ios/Scripts/ci-test.sh — 219 tests passed
• Fresh independent Claude Opus 5 high review completed; all actionable findings were addressed and the final repair review found no blockers.
• Simulator visual evidence will be attached before marking this PR ready for review.

Scope

Targets the active native SwiftUI owner branch (#5178).

Note

Show cached VCS status immediately in source control view before falling back to a full refresh

• NativeFeatureClient.sourceControlStatus now reads from the VCS status event stream via NativeSourceControlStatusAccumulator, returning a cached status within ~2 seconds before falling back to a full refreshVCSStatus call.
• A new refreshSourceControlStatus method (and matching FeatureClient protocol requirement) forces a fresh fetch, used by the toolbar reload button and pull-to-refresh in FeatureSourceControlView.
• The initial view load uses the fast cached path; explicit refresh actions use the new forced path.
• NativeWorkspaceMapper.sourceControl(local:remote:) defers emitting a status for repos with a primary remote until remote data arrives, preventing incomplete status display.
• Behavioral Change: the source control view's initial load now returns as soon as cached local+remote state is available rather than always waiting for a full network round-trip.

^{Macroscope summarized c7e7dbd.}

---

Note

Low Risk
UX and read-path changes for source-control display; explicit refresh behavior is preserved for user-initiated reloads, with tests covering remote-wait semantics.

Overview
Source control on iOS now prefers fast cached/subscription data on first open instead of always hitting a full VCS refresh.

sourceControlStatus races a ~2s listen on vcsStatusEvents (via NativeSourceControlStatusAccumulator merging local/remote VCSStatusEvents) against a timeout; if nothing usable arrives, it falls back to refreshVCSStatus as before. Repos with a primary remote do not surface UI status until remote data is present, avoiding misleading ahead/behind counts.

A new refreshSourceControlStatus always performs the full refresh; the toolbar reload and pull-to-refresh call that path, while the screen’s initial .task uses the cached path. FeatureClient gains the matching protocol method (stub delegates to sourceControlStatus for unavailable clients).

^{Reviewed by Cursor Bugbot for commit c7e7dbd. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

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: d90367c1-d22f-43d0-9c63-c21171dd9d56

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[saphid]

Integrated simulator verification completed after the focused cache tests and passing native CI. The home screen remained responsive while connected to the disposable backend and continued rendering the workspace/thread source-control summary.

The cache refresh/fallback behavior itself is covered by the exact-head native tests so this proof does not rely on timing a transient network failure in a screenshot.

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR introduces new async task coordination with timeouts and event accumulation to show cached source control status immediately. While self-contained and well-tested, the new loading strategy pattern represents a meaningful runtime behavior change that warrants human review.

^{You can customize Macroscope's approvability policy. Learn more.}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit d10233b. Configure here.}

[saphid]

Additional integrated simulator proof on iOS 26.5: the home row resolved the real upstream PR from the cached source-control status and visibly rendered the green #5975 ↗ badge. The semantic UI value independently reported Pull request 5975, open.

This follows the focused 221-test pass and the green native CI run.