fix(swift-ios): show cached source-control status immediately by saphid · Pull Request #5975 · pingdotgg/t3code

saphid · 2026-08-10T08:17:44Z

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 source-control status immediately in iOS before falling back to a full refresh

NativeFeatureClient.sourceControlStatus now races a VCS status stream against a 2-second timeout, returning cached/streamed status immediately when available and falling back to a full refresh only on timeout or error.
VcsStatusBroadcaster gains a generation counter per working directory, semaphore-serialized cache transitions, and coherent snapshot reads (readCoherentStatus) to prevent pairing stale remote status with a new local ref.
applyGitStatusStreamState (new) replaces applyGitStatusStreamEvent as the client-side accumulator, enforcing generation ordering and clearing remote state on ref changes.
GitManager.localStatus now includes coherenceToken and remoteAssociationToken (SHA-256 hashes over HEAD OID and remote association), enabling the broadcaster to detect ref changes without a full status read.
FeatureSourceControlView distinguishes between initial load (sourceControlStatus) and explicit refresh (refreshSourceControlStatus), so pull-to-refresh and the toolbar button always force a fresh fetch.
Risk: generation-mismatched remote updates are silently discarded; pollers that stall behind a generation change will trigger an immediate re-fetch, increasing short-term remote status request volume.

^{Macroscope summarized 7a1008e.}

Note

Medium Risk
Touches VCS cache coherency and concurrent refresh paths used by multiple clients; incorrect generation/token handling could show wrong ahead/behind or PR data, though coverage is heavy.

Overview
Adds generation-aware VCS status streaming so local and remote snapshots are not merged across branch/HEAD/remote changes, and wires fast cached source control on iOS on top of that stream.

Server:localStatus now exposes SHA-256 coherenceToken / remoteAssociationToken (HEAD, branch, upstream, normalized origin URL). VcsStatusBroadcaster tracks a per-repo generation, serializes cache updates, rejects stale remote writes, retries readCoherentStatus when identity shifts, and tags stream events with generation. HEAD-only moves can carry forward the last remote snapshot while forcing a remote refresh.

Clients: Contracts, shared reducers (applyGitStatusStreamState), and Swift decode the optional generation field. iOS sourceControlStatus waits up to 2s on the event stream via NativeSourceControlStatusAccumulator, then falls back to refreshSourceControlStatus; toolbar/pull-to-refresh use the explicit refresh path. Provider labels no longer fall back to a global provider list when the environment catalog is missing.

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

Delivery: direct
Validated against Theo commit: f98cab5
Depends on: none
Merge order: this PR only
Validation status: Mergeable with runnable checks green; requested review changes remain. The unrelated Vercel authorization failure is a maintainer-side gate.

coderabbitai · 2026-08-10T08:18:14Z

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: 1155e51f-feda-4356-8129-e9b8430f52de

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.}

saphid · 2026-08-10T08:50:19Z

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 · 2026-08-10T09:49:41Z

Approvability

Verdict: Needs human review

This PR introduces a new cache coherence system for VCS status with generation tracking across local/remote status. An unresolved review comment identifies a potential race condition where stale remote data from a previous branch could be paired with current local data after checkout. The scope and complexity of the caching logic changes, combined with the open correctness concern, warrant human review.

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

saphid · 2026-08-10T09:52:09Z

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.

t3-code

reviewed cached source-control event accumulation, timeout/fallback behavior, cancellation propagation, and explicit refresh preservation. no blocking issues found.

t3-code

independent review found a blocking coherence issue in the cached status path.

t3-code · 2026-08-12T02:09:09Z

+ local: VCSLocalStatus,
+ remote: VCSRemoteStatus?
+ ) -> FeatureSourceControlStatus? {
+ guard !local.hasPrimaryRemote || remote != nil else { return nil }


a non-nil remote value is not necessarily coherent with this local ref. the server cache updates local status independently and retains the prior remote half, so after checkout the accumulator can combine branch B local data with branch A ahead/behind and PR data. this one-shot caller then returns and unsubscribes before remoteUpdated(B) arrives. tie remote data to a local ref/generation, await a coherent remote refresh after ref changes, or otherwise reject the stale pair; add a localUpdated(A→B) regression test.

saphid · 2026-08-13T15:43:56Z

Addressed the review findings and completed the deferred native verification on exact head 3a20582c4bc43bbbf3fac478b0932b22d28167ed.

Corrections include generation-safe publication, compare-and-swap protection, identity probing, explicit origin errors, remote association tokens, working-tree token handling, and valid observation macro mutation.

Verification:

178 focused TypeScript tests passed
targeted lint, type, formatting, and Swift parse checks passed
focused native FeatureToolStateTests and WorkspaceContractTests: 26 passed, 0 failed
native result bundle: test_sim_2026-08-13T15-42-47-411Z_pid9862_21e5f4e8.xcresult
isolated DerivedData and one idle XCTest clone were removed; hygiene lease released

A prior direct Claude Opus 5 high review found seven actionable issues, all addressed in this head. A fresh review of the final head could not run because the Claude session quota remains exhausted until 04:50 AEST. No final-head independent review is claimed.

saphid · 2026-08-13T15:51:39Z

Follow-up for the hosted CI failures on exact head 5590d8c8d:

restored the legacy provider-catalog fallback used by snapshots without providersByEnvironment
added the required headOid: null to the unborn-repository server test fixture

Focused verification:

server typecheck passed
SourceControlRepositoryService.test.ts: 7 passed
HomeThreadMetadataTests: 5 passed
Swift parse and diff checks passed
native result bundle: test_sim_2026-08-13T15-50-30-445Z_pid9862_1ed6d520.xcresult
isolated Xcode cleanup completed

The Vercel marketing result is an authorization failure outside this PR's affected surface.

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 5590d8c. Configure here.}

saphid · 2026-08-15T11:02:53Z

Closing this version while I rebuild the upstream contribution set from the latest base. Clean versions are coming soon.

saphid · 2026-08-10T08:17:44Z

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 source-control status immediately in iOS before falling back to a full refresh

NativeFeatureClient.sourceControlStatus now races a VCS status stream against a 2-second timeout, returning cached/streamed status immediately when available and falling back to a full refresh only on timeout or error.
VcsStatusBroadcaster gains a generation counter per working directory, semaphore-serialized cache transitions, and coherent snapshot reads (readCoherentStatus) to prevent pairing stale remote status with a new local ref.
applyGitStatusStreamState (new) replaces applyGitStatusStreamEvent as the client-side accumulator, enforcing generation ordering and clearing remote state on ref changes.
GitManager.localStatus now includes coherenceToken and remoteAssociationToken (SHA-256 hashes over HEAD OID and remote association), enabling the broadcaster to detect ref changes without a full status read.
FeatureSourceControlView distinguishes between initial load (sourceControlStatus) and explicit refresh (refreshSourceControlStatus), so pull-to-refresh and the toolbar button always force a fresh fetch.
Risk: generation-mismatched remote updates are silently discarded; pollers that stall behind a generation change will trigger an immediate re-fetch, increasing short-term remote status request volume.

^{Macroscope summarized 7a1008e.}

Note

Medium Risk
Touches VCS cache coherency and concurrent refresh paths used by multiple clients; incorrect generation/token handling could show wrong ahead/behind or PR data, though coverage is heavy.

Overview
Adds generation-aware VCS status streaming so local and remote snapshots are not merged across branch/HEAD/remote changes, and wires fast cached source control on iOS on top of that stream.

Server:localStatus now exposes SHA-256 coherenceToken / remoteAssociationToken (HEAD, branch, upstream, normalized origin URL). VcsStatusBroadcaster tracks a per-repo generation, serializes cache updates, rejects stale remote writes, retries readCoherentStatus when identity shifts, and tags stream events with generation. HEAD-only moves can carry forward the last remote snapshot while forcing a remote refresh.

Clients: Contracts, shared reducers (applyGitStatusStreamState), and Swift decode the optional generation field. iOS sourceControlStatus waits up to 2s on the event stream via NativeSourceControlStatusAccumulator, then falls back to refreshSourceControlStatus; toolbar/pull-to-refresh use the explicit refresh path. Provider labels no longer fall back to a global provider list when the environment catalog is missing.

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

Delivery: direct
Validated against Theo commit: f98cab5
Depends on: none
Merge order: this PR only
Validation status: Mergeable with runnable checks green; requested review changes remain. The unrelated Vercel authorization failure is a maintainer-side gate.

coderabbitai · 2026-08-10T08:18:14Z

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: 1155e51f-feda-4356-8129-e9b8430f52de

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.}

saphid · 2026-08-10T08:50:19Z

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 · 2026-08-10T09:49:41Z

Approvability

Verdict: Needs human review

This PR introduces a new cache coherence system for VCS status with generation tracking across local/remote status. An unresolved review comment identifies a potential race condition where stale remote data from a previous branch could be paired with current local data after checkout. The scope and complexity of the caching logic changes, combined with the open correctness concern, warrant human review.

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

saphid · 2026-08-10T09:52:09Z

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.

t3-code

reviewed cached source-control event accumulation, timeout/fallback behavior, cancellation propagation, and explicit refresh preservation. no blocking issues found.

t3-code

independent review found a blocking coherence issue in the cached status path.

t3-code · 2026-08-12T02:09:09Z

+ local: VCSLocalStatus,
+ remote: VCSRemoteStatus?
+ ) -> FeatureSourceControlStatus? {
+ guard !local.hasPrimaryRemote || remote != nil else { return nil }


a non-nil remote value is not necessarily coherent with this local ref. the server cache updates local status independently and retains the prior remote half, so after checkout the accumulator can combine branch B local data with branch A ahead/behind and PR data. this one-shot caller then returns and unsubscribes before remoteUpdated(B) arrives. tie remote data to a local ref/generation, await a coherent remote refresh after ref changes, or otherwise reject the stale pair; add a localUpdated(A→B) regression test.

saphid · 2026-08-13T15:43:56Z

Addressed the review findings and completed the deferred native verification on exact head 3a20582c4bc43bbbf3fac478b0932b22d28167ed.

Corrections include generation-safe publication, compare-and-swap protection, identity probing, explicit origin errors, remote association tokens, working-tree token handling, and valid observation macro mutation.

Verification:

178 focused TypeScript tests passed
targeted lint, type, formatting, and Swift parse checks passed
focused native FeatureToolStateTests and WorkspaceContractTests: 26 passed, 0 failed
native result bundle: test_sim_2026-08-13T15-42-47-411Z_pid9862_21e5f4e8.xcresult
isolated DerivedData and one idle XCTest clone were removed; hygiene lease released

A prior direct Claude Opus 5 high review found seven actionable issues, all addressed in this head. A fresh review of the final head could not run because the Claude session quota remains exhausted until 04:50 AEST. No final-head independent review is claimed.

saphid · 2026-08-13T15:51:39Z

Follow-up for the hosted CI failures on exact head 5590d8c8d:

restored the legacy provider-catalog fallback used by snapshots without providersByEnvironment
added the required headOid: null to the unborn-repository server test fixture

Focused verification:

server typecheck passed
SourceControlRepositoryService.test.ts: 7 passed
HomeThreadMetadataTests: 5 passed
Swift parse and diff checks passed
native result bundle: test_sim_2026-08-13T15-50-30-445Z_pid9862_1ed6d520.xcresult
isolated Xcode cleanup completed

The Vercel marketing result is an authorization failure outside this PR's affected surface.

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 5590d8c. Configure here.}

saphid · 2026-08-15T11:02:53Z

Closing this version while I rebuild the upstream contribution set from the latest base. Clean versions are coming soon.

saphid · 2026-08-10T08:17:44Z

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 source-control status immediately in iOS before falling back to a full refresh

NativeFeatureClient.sourceControlStatus now races a VCS status stream against a 2-second timeout, returning cached/streamed status immediately when available and falling back to a full refresh only on timeout or error.
VcsStatusBroadcaster gains a generation counter per working directory, semaphore-serialized cache transitions, and coherent snapshot reads (readCoherentStatus) to prevent pairing stale remote status with a new local ref.
applyGitStatusStreamState (new) replaces applyGitStatusStreamEvent as the client-side accumulator, enforcing generation ordering and clearing remote state on ref changes.
GitManager.localStatus now includes coherenceToken and remoteAssociationToken (SHA-256 hashes over HEAD OID and remote association), enabling the broadcaster to detect ref changes without a full status read.
FeatureSourceControlView distinguishes between initial load (sourceControlStatus) and explicit refresh (refreshSourceControlStatus), so pull-to-refresh and the toolbar button always force a fresh fetch.
Risk: generation-mismatched remote updates are silently discarded; pollers that stall behind a generation change will trigger an immediate re-fetch, increasing short-term remote status request volume.

^{Macroscope summarized 7a1008e.}

Note

Medium Risk
Touches VCS cache coherency and concurrent refresh paths used by multiple clients; incorrect generation/token handling could show wrong ahead/behind or PR data, though coverage is heavy.

Overview
Adds generation-aware VCS status streaming so local and remote snapshots are not merged across branch/HEAD/remote changes, and wires fast cached source control on iOS on top of that stream.

Server:localStatus now exposes SHA-256 coherenceToken / remoteAssociationToken (HEAD, branch, upstream, normalized origin URL). VcsStatusBroadcaster tracks a per-repo generation, serializes cache updates, rejects stale remote writes, retries readCoherentStatus when identity shifts, and tags stream events with generation. HEAD-only moves can carry forward the last remote snapshot while forcing a remote refresh.

Clients: Contracts, shared reducers (applyGitStatusStreamState), and Swift decode the optional generation field. iOS sourceControlStatus waits up to 2s on the event stream via NativeSourceControlStatusAccumulator, then falls back to refreshSourceControlStatus; toolbar/pull-to-refresh use the explicit refresh path. Provider labels no longer fall back to a global provider list when the environment catalog is missing.

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

Delivery: direct
Validated against Theo commit: f98cab5
Depends on: none
Merge order: this PR only
Validation status: Mergeable with runnable checks green; requested review changes remain. The unrelated Vercel authorization failure is a maintainer-side gate.

coderabbitai · 2026-08-10T08:18:14Z

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: 1155e51f-feda-4356-8129-e9b8430f52de

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.}

saphid · 2026-08-10T08:50:19Z

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 · 2026-08-10T09:49:41Z

Approvability

Verdict: Needs human review

This PR introduces a new cache coherence system for VCS status with generation tracking across local/remote status. An unresolved review comment identifies a potential race condition where stale remote data from a previous branch could be paired with current local data after checkout. The scope and complexity of the caching logic changes, combined with the open correctness concern, warrant human review.

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

saphid · 2026-08-10T09:52:09Z

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.

t3-code

reviewed cached source-control event accumulation, timeout/fallback behavior, cancellation propagation, and explicit refresh preservation. no blocking issues found.

t3-code

independent review found a blocking coherence issue in the cached status path.

t3-code · 2026-08-12T02:09:09Z

+ local: VCSLocalStatus,
+ remote: VCSRemoteStatus?
+ ) -> FeatureSourceControlStatus? {
+ guard !local.hasPrimaryRemote || remote != nil else { return nil }


a non-nil remote value is not necessarily coherent with this local ref. the server cache updates local status independently and retains the prior remote half, so after checkout the accumulator can combine branch B local data with branch A ahead/behind and PR data. this one-shot caller then returns and unsubscribes before remoteUpdated(B) arrives. tie remote data to a local ref/generation, await a coherent remote refresh after ref changes, or otherwise reject the stale pair; add a localUpdated(A→B) regression test.

saphid · 2026-08-13T15:43:56Z

Addressed the review findings and completed the deferred native verification on exact head 3a20582c4bc43bbbf3fac478b0932b22d28167ed.

Corrections include generation-safe publication, compare-and-swap protection, identity probing, explicit origin errors, remote association tokens, working-tree token handling, and valid observation macro mutation.

Verification:

178 focused TypeScript tests passed
targeted lint, type, formatting, and Swift parse checks passed
focused native FeatureToolStateTests and WorkspaceContractTests: 26 passed, 0 failed
native result bundle: test_sim_2026-08-13T15-42-47-411Z_pid9862_21e5f4e8.xcresult
isolated DerivedData and one idle XCTest clone were removed; hygiene lease released

A prior direct Claude Opus 5 high review found seven actionable issues, all addressed in this head. A fresh review of the final head could not run because the Claude session quota remains exhausted until 04:50 AEST. No final-head independent review is claimed.

saphid · 2026-08-13T15:51:39Z

Follow-up for the hosted CI failures on exact head 5590d8c8d:

restored the legacy provider-catalog fallback used by snapshots without providersByEnvironment
added the required headOid: null to the unborn-repository server test fixture

Focused verification:

server typecheck passed
SourceControlRepositoryService.test.ts: 7 passed
HomeThreadMetadataTests: 5 passed
Swift parse and diff checks passed
native result bundle: test_sim_2026-08-13T15-50-30-445Z_pid9862_1ed6d520.xcresult
isolated Xcode cleanup completed

The Vercel marketing result is an authorization failure outside this PR's affected surface.

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 5590d8c. Configure here.}

saphid · 2026-08-15T11:02:53Z

Closing this version while I rebuild the upstream contribution set from the latest base. Clean versions are coming soon.

saphid · 2026-08-10T08:17:44Z

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 source-control status immediately in iOS before falling back to a full refresh

NativeFeatureClient.sourceControlStatus now races a VCS status stream against a 2-second timeout, returning cached/streamed status immediately when available and falling back to a full refresh only on timeout or error.
VcsStatusBroadcaster gains a generation counter per working directory, semaphore-serialized cache transitions, and coherent snapshot reads (readCoherentStatus) to prevent pairing stale remote status with a new local ref.
applyGitStatusStreamState (new) replaces applyGitStatusStreamEvent as the client-side accumulator, enforcing generation ordering and clearing remote state on ref changes.
GitManager.localStatus now includes coherenceToken and remoteAssociationToken (SHA-256 hashes over HEAD OID and remote association), enabling the broadcaster to detect ref changes without a full status read.
FeatureSourceControlView distinguishes between initial load (sourceControlStatus) and explicit refresh (refreshSourceControlStatus), so pull-to-refresh and the toolbar button always force a fresh fetch.
Risk: generation-mismatched remote updates are silently discarded; pollers that stall behind a generation change will trigger an immediate re-fetch, increasing short-term remote status request volume.

^{Macroscope summarized 7a1008e.}

Note

Medium Risk
Touches VCS cache coherency and concurrent refresh paths used by multiple clients; incorrect generation/token handling could show wrong ahead/behind or PR data, though coverage is heavy.

Overview
Adds generation-aware VCS status streaming so local and remote snapshots are not merged across branch/HEAD/remote changes, and wires fast cached source control on iOS on top of that stream.

Server:localStatus now exposes SHA-256 coherenceToken / remoteAssociationToken (HEAD, branch, upstream, normalized origin URL). VcsStatusBroadcaster tracks a per-repo generation, serializes cache updates, rejects stale remote writes, retries readCoherentStatus when identity shifts, and tags stream events with generation. HEAD-only moves can carry forward the last remote snapshot while forcing a remote refresh.

Clients: Contracts, shared reducers (applyGitStatusStreamState), and Swift decode the optional generation field. iOS sourceControlStatus waits up to 2s on the event stream via NativeSourceControlStatusAccumulator, then falls back to refreshSourceControlStatus; toolbar/pull-to-refresh use the explicit refresh path. Provider labels no longer fall back to a global provider list when the environment catalog is missing.

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

Delivery: direct
Validated against Theo commit: f98cab5
Depends on: none
Merge order: this PR only
Validation status: Mergeable with runnable checks green; requested review changes remain. The unrelated Vercel authorization failure is a maintainer-side gate.

coderabbitai · 2026-08-10T08:18:14Z

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: 1155e51f-feda-4356-8129-e9b8430f52de

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.}

saphid · 2026-08-10T08:50:19Z

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 · 2026-08-10T09:49:41Z

Approvability

Verdict: Needs human review

This PR introduces a new cache coherence system for VCS status with generation tracking across local/remote status. An unresolved review comment identifies a potential race condition where stale remote data from a previous branch could be paired with current local data after checkout. The scope and complexity of the caching logic changes, combined with the open correctness concern, warrant human review.

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

saphid · 2026-08-10T09:52:09Z

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.

t3-code

reviewed cached source-control event accumulation, timeout/fallback behavior, cancellation propagation, and explicit refresh preservation. no blocking issues found.

t3-code

independent review found a blocking coherence issue in the cached status path.

t3-code · 2026-08-12T02:09:09Z

+ local: VCSLocalStatus,
+ remote: VCSRemoteStatus?
+ ) -> FeatureSourceControlStatus? {
+ guard !local.hasPrimaryRemote || remote != nil else { return nil }


a non-nil remote value is not necessarily coherent with this local ref. the server cache updates local status independently and retains the prior remote half, so after checkout the accumulator can combine branch B local data with branch A ahead/behind and PR data. this one-shot caller then returns and unsubscribes before remoteUpdated(B) arrives. tie remote data to a local ref/generation, await a coherent remote refresh after ref changes, or otherwise reject the stale pair; add a localUpdated(A→B) regression test.

saphid · 2026-08-13T15:43:56Z

Addressed the review findings and completed the deferred native verification on exact head 3a20582c4bc43bbbf3fac478b0932b22d28167ed.

Corrections include generation-safe publication, compare-and-swap protection, identity probing, explicit origin errors, remote association tokens, working-tree token handling, and valid observation macro mutation.

Verification:

178 focused TypeScript tests passed
targeted lint, type, formatting, and Swift parse checks passed
focused native FeatureToolStateTests and WorkspaceContractTests: 26 passed, 0 failed
native result bundle: test_sim_2026-08-13T15-42-47-411Z_pid9862_21e5f4e8.xcresult
isolated DerivedData and one idle XCTest clone were removed; hygiene lease released

A prior direct Claude Opus 5 high review found seven actionable issues, all addressed in this head. A fresh review of the final head could not run because the Claude session quota remains exhausted until 04:50 AEST. No final-head independent review is claimed.

saphid · 2026-08-13T15:51:39Z

Follow-up for the hosted CI failures on exact head 5590d8c8d:

restored the legacy provider-catalog fallback used by snapshots without providersByEnvironment
added the required headOid: null to the unborn-repository server test fixture

Focused verification:

server typecheck passed
SourceControlRepositoryService.test.ts: 7 passed
HomeThreadMetadataTests: 5 passed
Swift parse and diff checks passed
native result bundle: test_sim_2026-08-13T15-50-30-445Z_pid9862_1ed6d520.xcresult
isolated Xcode cleanup completed

The Vercel marketing result is an authorization failure outside this PR's affected surface.

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 5590d8c. Configure here.}

saphid · 2026-08-15T11:02:53Z

Closing this version while I rebuild the upstream contribution set from the latest base. Clean versions are coming soon.

saphid · 2026-08-10T08:17:44Z

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 source-control status immediately in iOS before falling back to a full refresh

NativeFeatureClient.sourceControlStatus now races a VCS status stream against a 2-second timeout, returning cached/streamed status immediately when available and falling back to a full refresh only on timeout or error.
VcsStatusBroadcaster gains a generation counter per working directory, semaphore-serialized cache transitions, and coherent snapshot reads (readCoherentStatus) to prevent pairing stale remote status with a new local ref.
applyGitStatusStreamState (new) replaces applyGitStatusStreamEvent as the client-side accumulator, enforcing generation ordering and clearing remote state on ref changes.
GitManager.localStatus now includes coherenceToken and remoteAssociationToken (SHA-256 hashes over HEAD OID and remote association), enabling the broadcaster to detect ref changes without a full status read.
FeatureSourceControlView distinguishes between initial load (sourceControlStatus) and explicit refresh (refreshSourceControlStatus), so pull-to-refresh and the toolbar button always force a fresh fetch.
Risk: generation-mismatched remote updates are silently discarded; pollers that stall behind a generation change will trigger an immediate re-fetch, increasing short-term remote status request volume.

^{Macroscope summarized 7a1008e.}

Note

Medium Risk
Touches VCS cache coherency and concurrent refresh paths used by multiple clients; incorrect generation/token handling could show wrong ahead/behind or PR data, though coverage is heavy.

Overview
Adds generation-aware VCS status streaming so local and remote snapshots are not merged across branch/HEAD/remote changes, and wires fast cached source control on iOS on top of that stream.

Server:localStatus now exposes SHA-256 coherenceToken / remoteAssociationToken (HEAD, branch, upstream, normalized origin URL). VcsStatusBroadcaster tracks a per-repo generation, serializes cache updates, rejects stale remote writes, retries readCoherentStatus when identity shifts, and tags stream events with generation. HEAD-only moves can carry forward the last remote snapshot while forcing a remote refresh.

Clients: Contracts, shared reducers (applyGitStatusStreamState), and Swift decode the optional generation field. iOS sourceControlStatus waits up to 2s on the event stream via NativeSourceControlStatusAccumulator, then falls back to refreshSourceControlStatus; toolbar/pull-to-refresh use the explicit refresh path. Provider labels no longer fall back to a global provider list when the environment catalog is missing.

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

Delivery: direct
Validated against Theo commit: f98cab5
Depends on: none
Merge order: this PR only
Validation status: Mergeable with runnable checks green; requested review changes remain. The unrelated Vercel authorization failure is a maintainer-side gate.

coderabbitai · 2026-08-10T08:18:14Z

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: 1155e51f-feda-4356-8129-e9b8430f52de

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.}

saphid · 2026-08-10T08:50:19Z

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 · 2026-08-10T09:49:41Z

Approvability

Verdict: Needs human review

This PR introduces a new cache coherence system for VCS status with generation tracking across local/remote status. An unresolved review comment identifies a potential race condition where stale remote data from a previous branch could be paired with current local data after checkout. The scope and complexity of the caching logic changes, combined with the open correctness concern, warrant human review.

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

saphid · 2026-08-10T09:52:09Z

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.

t3-code

reviewed cached source-control event accumulation, timeout/fallback behavior, cancellation propagation, and explicit refresh preservation. no blocking issues found.

t3-code

independent review found a blocking coherence issue in the cached status path.

t3-code · 2026-08-12T02:09:09Z

+ local: VCSLocalStatus,
+ remote: VCSRemoteStatus?
+ ) -> FeatureSourceControlStatus? {
+ guard !local.hasPrimaryRemote || remote != nil else { return nil }


a non-nil remote value is not necessarily coherent with this local ref. the server cache updates local status independently and retains the prior remote half, so after checkout the accumulator can combine branch B local data with branch A ahead/behind and PR data. this one-shot caller then returns and unsubscribes before remoteUpdated(B) arrives. tie remote data to a local ref/generation, await a coherent remote refresh after ref changes, or otherwise reject the stale pair; add a localUpdated(A→B) regression test.

saphid · 2026-08-13T15:43:56Z

Addressed the review findings and completed the deferred native verification on exact head 3a20582c4bc43bbbf3fac478b0932b22d28167ed.

Corrections include generation-safe publication, compare-and-swap protection, identity probing, explicit origin errors, remote association tokens, working-tree token handling, and valid observation macro mutation.

Verification:

178 focused TypeScript tests passed
targeted lint, type, formatting, and Swift parse checks passed
focused native FeatureToolStateTests and WorkspaceContractTests: 26 passed, 0 failed
native result bundle: test_sim_2026-08-13T15-42-47-411Z_pid9862_21e5f4e8.xcresult
isolated DerivedData and one idle XCTest clone were removed; hygiene lease released

A prior direct Claude Opus 5 high review found seven actionable issues, all addressed in this head. A fresh review of the final head could not run because the Claude session quota remains exhausted until 04:50 AEST. No final-head independent review is claimed.

saphid · 2026-08-13T15:51:39Z

Follow-up for the hosted CI failures on exact head 5590d8c8d:

restored the legacy provider-catalog fallback used by snapshots without providersByEnvironment
added the required headOid: null to the unborn-repository server test fixture

Focused verification:

server typecheck passed
SourceControlRepositoryService.test.ts: 7 passed
HomeThreadMetadataTests: 5 passed
Swift parse and diff checks passed
native result bundle: test_sim_2026-08-13T15-50-30-445Z_pid9862_1ed6d520.xcresult
isolated Xcode cleanup completed

The Vercel marketing result is an authorization failure outside this PR's affected surface.

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 5590d8c. Configure here.}

saphid · 2026-08-15T11:02:53Z

Closing this version while I rebuild the upstream contribution set from the latest base. Clean versions are coming soon.

saphid · 2026-08-10T08:17:44Z

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 source-control status immediately in iOS before falling back to a full refresh

NativeFeatureClient.sourceControlStatus now races a VCS status stream against a 2-second timeout, returning cached/streamed status immediately when available and falling back to a full refresh only on timeout or error.
VcsStatusBroadcaster gains a generation counter per working directory, semaphore-serialized cache transitions, and coherent snapshot reads (readCoherentStatus) to prevent pairing stale remote status with a new local ref.
applyGitStatusStreamState (new) replaces applyGitStatusStreamEvent as the client-side accumulator, enforcing generation ordering and clearing remote state on ref changes.
GitManager.localStatus now includes coherenceToken and remoteAssociationToken (SHA-256 hashes over HEAD OID and remote association), enabling the broadcaster to detect ref changes without a full status read.
FeatureSourceControlView distinguishes between initial load (sourceControlStatus) and explicit refresh (refreshSourceControlStatus), so pull-to-refresh and the toolbar button always force a fresh fetch.
Risk: generation-mismatched remote updates are silently discarded; pollers that stall behind a generation change will trigger an immediate re-fetch, increasing short-term remote status request volume.

^{Macroscope summarized 7a1008e.}

Note

Medium Risk
Touches VCS cache coherency and concurrent refresh paths used by multiple clients; incorrect generation/token handling could show wrong ahead/behind or PR data, though coverage is heavy.

Overview
Adds generation-aware VCS status streaming so local and remote snapshots are not merged across branch/HEAD/remote changes, and wires fast cached source control on iOS on top of that stream.

Server:localStatus now exposes SHA-256 coherenceToken / remoteAssociationToken (HEAD, branch, upstream, normalized origin URL). VcsStatusBroadcaster tracks a per-repo generation, serializes cache updates, rejects stale remote writes, retries readCoherentStatus when identity shifts, and tags stream events with generation. HEAD-only moves can carry forward the last remote snapshot while forcing a remote refresh.

Clients: Contracts, shared reducers (applyGitStatusStreamState), and Swift decode the optional generation field. iOS sourceControlStatus waits up to 2s on the event stream via NativeSourceControlStatusAccumulator, then falls back to refreshSourceControlStatus; toolbar/pull-to-refresh use the explicit refresh path. Provider labels no longer fall back to a global provider list when the environment catalog is missing.

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

Delivery: direct
Validated against Theo commit: f98cab5
Depends on: none
Merge order: this PR only
Validation status: Mergeable with runnable checks green; requested review changes remain. The unrelated Vercel authorization failure is a maintainer-side gate.

coderabbitai · 2026-08-10T08:18:14Z

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: 1155e51f-feda-4356-8129-e9b8430f52de

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.}

saphid · 2026-08-10T08:50:19Z

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 · 2026-08-10T09:49:41Z

Approvability

Verdict: Needs human review

This PR introduces a new cache coherence system for VCS status with generation tracking across local/remote status. An unresolved review comment identifies a potential race condition where stale remote data from a previous branch could be paired with current local data after checkout. The scope and complexity of the caching logic changes, combined with the open correctness concern, warrant human review.

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

saphid · 2026-08-10T09:52:09Z

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.

t3-code

reviewed cached source-control event accumulation, timeout/fallback behavior, cancellation propagation, and explicit refresh preservation. no blocking issues found.

t3-code

independent review found a blocking coherence issue in the cached status path.

t3-code · 2026-08-12T02:09:09Z

+ local: VCSLocalStatus,
+ remote: VCSRemoteStatus?
+ ) -> FeatureSourceControlStatus? {
+ guard !local.hasPrimaryRemote || remote != nil else { return nil }


a non-nil remote value is not necessarily coherent with this local ref. the server cache updates local status independently and retains the prior remote half, so after checkout the accumulator can combine branch B local data with branch A ahead/behind and PR data. this one-shot caller then returns and unsubscribes before remoteUpdated(B) arrives. tie remote data to a local ref/generation, await a coherent remote refresh after ref changes, or otherwise reject the stale pair; add a localUpdated(A→B) regression test.

saphid · 2026-08-13T15:43:56Z

Addressed the review findings and completed the deferred native verification on exact head 3a20582c4bc43bbbf3fac478b0932b22d28167ed.

Corrections include generation-safe publication, compare-and-swap protection, identity probing, explicit origin errors, remote association tokens, working-tree token handling, and valid observation macro mutation.

Verification:

178 focused TypeScript tests passed
targeted lint, type, formatting, and Swift parse checks passed
focused native FeatureToolStateTests and WorkspaceContractTests: 26 passed, 0 failed
native result bundle: test_sim_2026-08-13T15-42-47-411Z_pid9862_21e5f4e8.xcresult
isolated DerivedData and one idle XCTest clone were removed; hygiene lease released

A prior direct Claude Opus 5 high review found seven actionable issues, all addressed in this head. A fresh review of the final head could not run because the Claude session quota remains exhausted until 04:50 AEST. No final-head independent review is claimed.

saphid · 2026-08-13T15:51:39Z

Follow-up for the hosted CI failures on exact head 5590d8c8d:

restored the legacy provider-catalog fallback used by snapshots without providersByEnvironment
added the required headOid: null to the unborn-repository server test fixture

Focused verification:

server typecheck passed
SourceControlRepositoryService.test.ts: 7 passed
HomeThreadMetadataTests: 5 passed
Swift parse and diff checks passed
native result bundle: test_sim_2026-08-13T15-50-30-445Z_pid9862_1ed6d520.xcresult
isolated Xcode cleanup completed

The Vercel marketing result is an authorization failure outside this PR's affected surface.

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 5590d8c. Configure here.}

saphid · 2026-08-15T11:02:53Z

Closing this version while I rebuild the upstream contribution set from the latest base. Clean versions are coming soon.

saphid · 2026-08-10T08:17:44Z

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 source-control status immediately in iOS before falling back to a full refresh

NativeFeatureClient.sourceControlStatus now races a VCS status stream against a 2-second timeout, returning cached/streamed status immediately when available and falling back to a full refresh only on timeout or error.
VcsStatusBroadcaster gains a generation counter per working directory, semaphore-serialized cache transitions, and coherent snapshot reads (readCoherentStatus) to prevent pairing stale remote status with a new local ref.
applyGitStatusStreamState (new) replaces applyGitStatusStreamEvent as the client-side accumulator, enforcing generation ordering and clearing remote state on ref changes.
GitManager.localStatus now includes coherenceToken and remoteAssociationToken (SHA-256 hashes over HEAD OID and remote association), enabling the broadcaster to detect ref changes without a full status read.
FeatureSourceControlView distinguishes between initial load (sourceControlStatus) and explicit refresh (refreshSourceControlStatus), so pull-to-refresh and the toolbar button always force a fresh fetch.
Risk: generation-mismatched remote updates are silently discarded; pollers that stall behind a generation change will trigger an immediate re-fetch, increasing short-term remote status request volume.

^{Macroscope summarized 7a1008e.}

Note

Medium Risk
Touches VCS cache coherency and concurrent refresh paths used by multiple clients; incorrect generation/token handling could show wrong ahead/behind or PR data, though coverage is heavy.

Overview
Adds generation-aware VCS status streaming so local and remote snapshots are not merged across branch/HEAD/remote changes, and wires fast cached source control on iOS on top of that stream.

Server:localStatus now exposes SHA-256 coherenceToken / remoteAssociationToken (HEAD, branch, upstream, normalized origin URL). VcsStatusBroadcaster tracks a per-repo generation, serializes cache updates, rejects stale remote writes, retries readCoherentStatus when identity shifts, and tags stream events with generation. HEAD-only moves can carry forward the last remote snapshot while forcing a remote refresh.

Clients: Contracts, shared reducers (applyGitStatusStreamState), and Swift decode the optional generation field. iOS sourceControlStatus waits up to 2s on the event stream via NativeSourceControlStatusAccumulator, then falls back to refreshSourceControlStatus; toolbar/pull-to-refresh use the explicit refresh path. Provider labels no longer fall back to a global provider list when the environment catalog is missing.

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

Delivery: direct
Validated against Theo commit: f98cab5
Depends on: none
Merge order: this PR only
Validation status: Mergeable with runnable checks green; requested review changes remain. The unrelated Vercel authorization failure is a maintainer-side gate.

coderabbitai · 2026-08-10T08:18:14Z

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: 1155e51f-feda-4356-8129-e9b8430f52de

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.}

saphid · 2026-08-10T08:50:19Z

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 · 2026-08-10T09:49:41Z

Approvability

Verdict: Needs human review

This PR introduces a new cache coherence system for VCS status with generation tracking across local/remote status. An unresolved review comment identifies a potential race condition where stale remote data from a previous branch could be paired with current local data after checkout. The scope and complexity of the caching logic changes, combined with the open correctness concern, warrant human review.

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

saphid · 2026-08-10T09:52:09Z

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.

t3-code

reviewed cached source-control event accumulation, timeout/fallback behavior, cancellation propagation, and explicit refresh preservation. no blocking issues found.

t3-code

independent review found a blocking coherence issue in the cached status path.

t3-code · 2026-08-12T02:09:09Z

+ local: VCSLocalStatus,
+ remote: VCSRemoteStatus?
+ ) -> FeatureSourceControlStatus? {
+ guard !local.hasPrimaryRemote || remote != nil else { return nil }


a non-nil remote value is not necessarily coherent with this local ref. the server cache updates local status independently and retains the prior remote half, so after checkout the accumulator can combine branch B local data with branch A ahead/behind and PR data. this one-shot caller then returns and unsubscribes before remoteUpdated(B) arrives. tie remote data to a local ref/generation, await a coherent remote refresh after ref changes, or otherwise reject the stale pair; add a localUpdated(A→B) regression test.

saphid · 2026-08-13T15:43:56Z

Addressed the review findings and completed the deferred native verification on exact head 3a20582c4bc43bbbf3fac478b0932b22d28167ed.

Corrections include generation-safe publication, compare-and-swap protection, identity probing, explicit origin errors, remote association tokens, working-tree token handling, and valid observation macro mutation.

Verification:

178 focused TypeScript tests passed
targeted lint, type, formatting, and Swift parse checks passed
focused native FeatureToolStateTests and WorkspaceContractTests: 26 passed, 0 failed
native result bundle: test_sim_2026-08-13T15-42-47-411Z_pid9862_21e5f4e8.xcresult
isolated DerivedData and one idle XCTest clone were removed; hygiene lease released

A prior direct Claude Opus 5 high review found seven actionable issues, all addressed in this head. A fresh review of the final head could not run because the Claude session quota remains exhausted until 04:50 AEST. No final-head independent review is claimed.

saphid · 2026-08-13T15:51:39Z

Follow-up for the hosted CI failures on exact head 5590d8c8d:

restored the legacy provider-catalog fallback used by snapshots without providersByEnvironment
added the required headOid: null to the unborn-repository server test fixture

Focused verification:

server typecheck passed
SourceControlRepositoryService.test.ts: 7 passed
HomeThreadMetadataTests: 5 passed
Swift parse and diff checks passed
native result bundle: test_sim_2026-08-13T15-50-30-445Z_pid9862_1ed6d520.xcresult
isolated Xcode cleanup completed

The Vercel marketing result is an authorization failure outside this PR's affected surface.

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 5590d8c. Configure here.}

saphid · 2026-08-15T11:02:53Z

Closing this version while I rebuild the upstream contribution set from the latest base. Clean versions are coming soon.

saphid · 2026-08-10T08:17:44Z

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 source-control status immediately in iOS before falling back to a full refresh

NativeFeatureClient.sourceControlStatus now races a VCS status stream against a 2-second timeout, returning cached/streamed status immediately when available and falling back to a full refresh only on timeout or error.
VcsStatusBroadcaster gains a generation counter per working directory, semaphore-serialized cache transitions, and coherent snapshot reads (readCoherentStatus) to prevent pairing stale remote status with a new local ref.
applyGitStatusStreamState (new) replaces applyGitStatusStreamEvent as the client-side accumulator, enforcing generation ordering and clearing remote state on ref changes.
GitManager.localStatus now includes coherenceToken and remoteAssociationToken (SHA-256 hashes over HEAD OID and remote association), enabling the broadcaster to detect ref changes without a full status read.
FeatureSourceControlView distinguishes between initial load (sourceControlStatus) and explicit refresh (refreshSourceControlStatus), so pull-to-refresh and the toolbar button always force a fresh fetch.
Risk: generation-mismatched remote updates are silently discarded; pollers that stall behind a generation change will trigger an immediate re-fetch, increasing short-term remote status request volume.

^{Macroscope summarized 7a1008e.}

Note

Medium Risk
Touches VCS cache coherency and concurrent refresh paths used by multiple clients; incorrect generation/token handling could show wrong ahead/behind or PR data, though coverage is heavy.

Overview
Adds generation-aware VCS status streaming so local and remote snapshots are not merged across branch/HEAD/remote changes, and wires fast cached source control on iOS on top of that stream.

Server:localStatus now exposes SHA-256 coherenceToken / remoteAssociationToken (HEAD, branch, upstream, normalized origin URL). VcsStatusBroadcaster tracks a per-repo generation, serializes cache updates, rejects stale remote writes, retries readCoherentStatus when identity shifts, and tags stream events with generation. HEAD-only moves can carry forward the last remote snapshot while forcing a remote refresh.

Clients: Contracts, shared reducers (applyGitStatusStreamState), and Swift decode the optional generation field. iOS sourceControlStatus waits up to 2s on the event stream via NativeSourceControlStatusAccumulator, then falls back to refreshSourceControlStatus; toolbar/pull-to-refresh use the explicit refresh path. Provider labels no longer fall back to a global provider list when the environment catalog is missing.

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

Delivery: direct
Validated against Theo commit: f98cab5
Depends on: none
Merge order: this PR only
Validation status: Mergeable with runnable checks green; requested review changes remain. The unrelated Vercel authorization failure is a maintainer-side gate.

coderabbitai · 2026-08-10T08:18:14Z

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: 1155e51f-feda-4356-8129-e9b8430f52de

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.}

saphid · 2026-08-10T08:50:19Z

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 · 2026-08-10T09:49:41Z

Approvability

Verdict: Needs human review

This PR introduces a new cache coherence system for VCS status with generation tracking across local/remote status. An unresolved review comment identifies a potential race condition where stale remote data from a previous branch could be paired with current local data after checkout. The scope and complexity of the caching logic changes, combined with the open correctness concern, warrant human review.

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

saphid · 2026-08-10T09:52:09Z

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.

t3-code

reviewed cached source-control event accumulation, timeout/fallback behavior, cancellation propagation, and explicit refresh preservation. no blocking issues found.

t3-code

independent review found a blocking coherence issue in the cached status path.

t3-code · 2026-08-12T02:09:09Z

+ local: VCSLocalStatus,
+ remote: VCSRemoteStatus?
+ ) -> FeatureSourceControlStatus? {
+ guard !local.hasPrimaryRemote || remote != nil else { return nil }


a non-nil remote value is not necessarily coherent with this local ref. the server cache updates local status independently and retains the prior remote half, so after checkout the accumulator can combine branch B local data with branch A ahead/behind and PR data. this one-shot caller then returns and unsubscribes before remoteUpdated(B) arrives. tie remote data to a local ref/generation, await a coherent remote refresh after ref changes, or otherwise reject the stale pair; add a localUpdated(A→B) regression test.

saphid · 2026-08-13T15:43:56Z

Addressed the review findings and completed the deferred native verification on exact head 3a20582c4bc43bbbf3fac478b0932b22d28167ed.

Corrections include generation-safe publication, compare-and-swap protection, identity probing, explicit origin errors, remote association tokens, working-tree token handling, and valid observation macro mutation.

Verification:

178 focused TypeScript tests passed
targeted lint, type, formatting, and Swift parse checks passed
focused native FeatureToolStateTests and WorkspaceContractTests: 26 passed, 0 failed
native result bundle: test_sim_2026-08-13T15-42-47-411Z_pid9862_21e5f4e8.xcresult
isolated DerivedData and one idle XCTest clone were removed; hygiene lease released

A prior direct Claude Opus 5 high review found seven actionable issues, all addressed in this head. A fresh review of the final head could not run because the Claude session quota remains exhausted until 04:50 AEST. No final-head independent review is claimed.

saphid · 2026-08-13T15:51:39Z

Follow-up for the hosted CI failures on exact head 5590d8c8d:

restored the legacy provider-catalog fallback used by snapshots without providersByEnvironment
added the required headOid: null to the unborn-repository server test fixture

Focused verification:

server typecheck passed
SourceControlRepositoryService.test.ts: 7 passed
HomeThreadMetadataTests: 5 passed
Swift parse and diff checks passed
native result bundle: test_sim_2026-08-13T15-50-30-445Z_pid9862_1ed6d520.xcresult
isolated Xcode cleanup completed

The Vercel marketing result is an authorization failure outside this PR's affected surface.

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 5590d8c. Configure here.}

saphid · 2026-08-15T11:02:53Z

Closing this version while I rebuild the upstream contribution set from the latest base. Clean versions are coming soon.

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 10, 2026

macroscopeappBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeWorkspaceMapper.swift

saphid marked this pull request as ready for review August 10, 2026 09:27

cursorBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeFeatureClient.swift

saphid mentioned this pull request Aug 10, 2026
feat(swift-ios): link thread rows to pull requests #5804
Closed
5 tasks

t3-codeBot approved these changes Aug 12, 2026
View reviewed changes

t3-codeBot requested changes Aug 12, 2026
View reviewed changes

saphid added 6 commits August 13, 2026 22:44

fix(vcs): keep cached status coherent across refs
dfeb177

fix(vcs): serialize cached status generations
73f6cdc

fix(vcs): close cached status race windows
02a33b7

fix(vcs): finish coherent status transitions
ca8ab77

fix(vcs): complete coherent status refreshes
d18afd6

fix(vcs): close status coherence races
3a20582

saphid force-pushed the saphid/swiftui-cached-status-v2 branch from c7e7dbd to 3a20582Compare August 13, 2026 15:43

github-actionsBot added size:XL 500-999 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Aug 13, 2026

saphid added 2 commits August 14, 2026 01:48

fix(swift-ios): preserve provider catalog fallback
dd2d3a7

test(server): include repository head identity
5590d8c

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/server/src/vcs/VcsStatusBroadcaster.ts

macroscopeappBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/WorkspaceView.swift Outdated

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/DailyUXModels.swift Outdated

saphid mentioned this pull request Aug 13, 2026
feat(swift-ios): make settle the primary thread swipe action #5974
Closed

fix(swift-ios): keep cached source control coherent
7a1008e

saphid closed this Aug 15, 2026

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 10, 2026

macroscopeappBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeWorkspaceMapper.swift

saphid marked this pull request as ready for review August 10, 2026 09:27

cursorBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeFeatureClient.swift

saphid mentioned this pull request Aug 10, 2026
feat(swift-ios): link thread rows to pull requests #5804
Closed
5 tasks

t3-codeBot approved these changes Aug 12, 2026
View reviewed changes

t3-codeBot requested changes Aug 12, 2026
View reviewed changes

saphid added 6 commits August 13, 2026 22:44

fix(vcs): keep cached status coherent across refs
dfeb177

fix(vcs): serialize cached status generations
73f6cdc

fix(vcs): close cached status race windows
02a33b7

fix(vcs): finish coherent status transitions
ca8ab77

fix(vcs): complete coherent status refreshes
d18afd6

fix(vcs): close status coherence races
3a20582

saphid force-pushed the saphid/swiftui-cached-status-v2 branch from c7e7dbd to 3a20582Compare August 13, 2026 15:43

github-actionsBot added size:XL 500-999 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Aug 13, 2026

saphid added 2 commits August 14, 2026 01:48

fix(swift-ios): preserve provider catalog fallback
dd2d3a7

test(server): include repository head identity
5590d8c

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/server/src/vcs/VcsStatusBroadcaster.ts

macroscopeappBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/WorkspaceView.swift Outdated

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/DailyUXModels.swift Outdated

saphid mentioned this pull request Aug 13, 2026
feat(swift-ios): make settle the primary thread swipe action #5974
Closed

fix(swift-ios): keep cached source control coherent
7a1008e

saphid closed this Aug 15, 2026

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 10, 2026

macroscopeappBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeWorkspaceMapper.swift

saphid marked this pull request as ready for review August 10, 2026 09:27

cursorBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeFeatureClient.swift

saphid mentioned this pull request Aug 10, 2026
feat(swift-ios): link thread rows to pull requests #5804
Closed
5 tasks

t3-codeBot approved these changes Aug 12, 2026
View reviewed changes

t3-codeBot requested changes Aug 12, 2026
View reviewed changes

saphid added 6 commits August 13, 2026 22:44

fix(vcs): keep cached status coherent across refs
dfeb177

fix(vcs): serialize cached status generations
73f6cdc

fix(vcs): close cached status race windows
02a33b7

fix(vcs): finish coherent status transitions
ca8ab77

fix(vcs): complete coherent status refreshes
d18afd6

fix(vcs): close status coherence races
3a20582

saphid force-pushed the saphid/swiftui-cached-status-v2 branch from c7e7dbd to 3a20582Compare August 13, 2026 15:43

github-actionsBot added size:XL 500-999 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Aug 13, 2026

saphid added 2 commits August 14, 2026 01:48

fix(swift-ios): preserve provider catalog fallback
dd2d3a7

test(server): include repository head identity
5590d8c

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/server/src/vcs/VcsStatusBroadcaster.ts

macroscopeappBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/WorkspaceView.swift Outdated

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/DailyUXModels.swift Outdated

saphid mentioned this pull request Aug 13, 2026
feat(swift-ios): make settle the primary thread swipe action #5974
Closed

fix(swift-ios): keep cached source control coherent
7a1008e

saphid closed this Aug 15, 2026

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 10, 2026

macroscopeappBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeWorkspaceMapper.swift

saphid marked this pull request as ready for review August 10, 2026 09:27

cursorBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeFeatureClient.swift

saphid mentioned this pull request Aug 10, 2026
feat(swift-ios): link thread rows to pull requests #5804
Closed
5 tasks

t3-codeBot approved these changes Aug 12, 2026
View reviewed changes

t3-codeBot requested changes Aug 12, 2026
View reviewed changes

saphid added 6 commits August 13, 2026 22:44

fix(vcs): keep cached status coherent across refs
dfeb177

fix(vcs): serialize cached status generations
73f6cdc

fix(vcs): close cached status race windows
02a33b7

fix(vcs): finish coherent status transitions
ca8ab77

fix(vcs): complete coherent status refreshes
d18afd6

fix(vcs): close status coherence races
3a20582

saphid force-pushed the saphid/swiftui-cached-status-v2 branch from c7e7dbd to 3a20582Compare August 13, 2026 15:43

github-actionsBot added size:XL 500-999 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Aug 13, 2026

saphid added 2 commits August 14, 2026 01:48

fix(swift-ios): preserve provider catalog fallback
dd2d3a7

test(server): include repository head identity
5590d8c

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/server/src/vcs/VcsStatusBroadcaster.ts

macroscopeappBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/WorkspaceView.swift Outdated

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/DailyUXModels.swift Outdated

saphid mentioned this pull request Aug 13, 2026
feat(swift-ios): make settle the primary thread swipe action #5974
Closed

fix(swift-ios): keep cached source control coherent
7a1008e

saphid closed this Aug 15, 2026

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 10, 2026

macroscopeappBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeWorkspaceMapper.swift

saphid marked this pull request as ready for review August 10, 2026 09:27

cursorBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeFeatureClient.swift

saphid mentioned this pull request Aug 10, 2026
feat(swift-ios): link thread rows to pull requests #5804
Closed
5 tasks

t3-codeBot approved these changes Aug 12, 2026
View reviewed changes

t3-codeBot requested changes Aug 12, 2026
View reviewed changes

saphid added 6 commits August 13, 2026 22:44

fix(vcs): keep cached status coherent across refs
dfeb177

fix(vcs): serialize cached status generations
73f6cdc

fix(vcs): close cached status race windows
02a33b7

fix(vcs): finish coherent status transitions
ca8ab77

fix(vcs): complete coherent status refreshes
d18afd6

fix(vcs): close status coherence races
3a20582

saphid force-pushed the saphid/swiftui-cached-status-v2 branch from c7e7dbd to 3a20582Compare August 13, 2026 15:43

github-actionsBot added size:XL 500-999 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Aug 13, 2026

saphid added 2 commits August 14, 2026 01:48

fix(swift-ios): preserve provider catalog fallback
dd2d3a7

test(server): include repository head identity
5590d8c

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/server/src/vcs/VcsStatusBroadcaster.ts

macroscopeappBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/WorkspaceView.swift Outdated

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/DailyUXModels.swift Outdated

saphid mentioned this pull request Aug 13, 2026
feat(swift-ios): make settle the primary thread swipe action #5974
Closed

fix(swift-ios): keep cached source control coherent
7a1008e

saphid closed this Aug 15, 2026

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 10, 2026

macroscopeappBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeWorkspaceMapper.swift

saphid marked this pull request as ready for review August 10, 2026 09:27

cursorBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeFeatureClient.swift

saphid mentioned this pull request Aug 10, 2026
feat(swift-ios): link thread rows to pull requests #5804
Closed
5 tasks

t3-codeBot approved these changes Aug 12, 2026
View reviewed changes

t3-codeBot requested changes Aug 12, 2026
View reviewed changes

saphid added 6 commits August 13, 2026 22:44

fix(vcs): keep cached status coherent across refs
dfeb177

fix(vcs): serialize cached status generations
73f6cdc

fix(vcs): close cached status race windows
02a33b7

fix(vcs): finish coherent status transitions
ca8ab77

fix(vcs): complete coherent status refreshes
d18afd6

fix(vcs): close status coherence races
3a20582

saphid force-pushed the saphid/swiftui-cached-status-v2 branch from c7e7dbd to 3a20582Compare August 13, 2026 15:43

github-actionsBot added size:XL 500-999 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Aug 13, 2026

saphid added 2 commits August 14, 2026 01:48

fix(swift-ios): preserve provider catalog fallback
dd2d3a7

test(server): include repository head identity
5590d8c

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/server/src/vcs/VcsStatusBroadcaster.ts

macroscopeappBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/WorkspaceView.swift Outdated

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/DailyUXModels.swift Outdated

saphid mentioned this pull request Aug 13, 2026
feat(swift-ios): make settle the primary thread swipe action #5974
Closed

fix(swift-ios): keep cached source control coherent
7a1008e

saphid closed this Aug 15, 2026

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 10, 2026

macroscopeappBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeWorkspaceMapper.swift

saphid marked this pull request as ready for review August 10, 2026 09:27

cursorBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeFeatureClient.swift

saphid mentioned this pull request Aug 10, 2026
feat(swift-ios): link thread rows to pull requests #5804
Closed
5 tasks

t3-codeBot approved these changes Aug 12, 2026
View reviewed changes

t3-codeBot requested changes Aug 12, 2026
View reviewed changes

saphid added 6 commits August 13, 2026 22:44

fix(vcs): keep cached status coherent across refs
dfeb177

fix(vcs): serialize cached status generations
73f6cdc

fix(vcs): close cached status race windows
02a33b7

fix(vcs): finish coherent status transitions
ca8ab77

fix(vcs): complete coherent status refreshes
d18afd6

fix(vcs): close status coherence races
3a20582

saphid force-pushed the saphid/swiftui-cached-status-v2 branch from c7e7dbd to 3a20582Compare August 13, 2026 15:43

github-actionsBot added size:XL 500-999 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Aug 13, 2026

saphid added 2 commits August 14, 2026 01:48

fix(swift-ios): preserve provider catalog fallback
dd2d3a7

test(server): include repository head identity
5590d8c

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/server/src/vcs/VcsStatusBroadcaster.ts

macroscopeappBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/WorkspaceView.swift Outdated

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/DailyUXModels.swift Outdated

saphid mentioned this pull request Aug 13, 2026
feat(swift-ios): make settle the primary thread swipe action #5974
Closed

fix(swift-ios): keep cached source control coherent
7a1008e

saphid closed this Aug 15, 2026

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 10, 2026

macroscopeappBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeWorkspaceMapper.swift

saphid marked this pull request as ready for review August 10, 2026 09:27

cursorBot reviewed Aug 10, 2026
View reviewed changes

Comment threadapps/swift-ios/App/NativeFeatureClient.swift

saphid mentioned this pull request Aug 10, 2026
feat(swift-ios): link thread rows to pull requests #5804
Closed
5 tasks

t3-codeBot approved these changes Aug 12, 2026
View reviewed changes

t3-codeBot requested changes Aug 12, 2026
View reviewed changes

saphid added 6 commits August 13, 2026 22:44

fix(vcs): keep cached status coherent across refs
dfeb177

fix(vcs): serialize cached status generations
73f6cdc

fix(vcs): close cached status race windows
02a33b7

fix(vcs): finish coherent status transitions
ca8ab77

fix(vcs): complete coherent status refreshes
d18afd6

fix(vcs): close status coherence races
3a20582

saphid force-pushed the saphid/swiftui-cached-status-v2 branch from c7e7dbd to 3a20582Compare August 13, 2026 15:43

github-actionsBot added size:XL 500-999 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Aug 13, 2026

saphid added 2 commits August 14, 2026 01:48

fix(swift-ios): preserve provider catalog fallback
dd2d3a7

test(server): include repository head identity
5590d8c

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/server/src/vcs/VcsStatusBroadcaster.ts

macroscopeappBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/WorkspaceView.swift Outdated

cursorBot reviewed Aug 13, 2026
View reviewed changes

Comment threadapps/swift-ios/Features/Workspace/DailyUXModels.swift Outdated

saphid mentioned this pull request Aug 13, 2026
feat(swift-ios): make settle the primary thread swipe action #5974
Closed

fix(swift-ios): keep cached source control coherent
7a1008e

saphid closed this Aug 15, 2026

Uh oh!

Conversation

saphid commented Aug 10, 2026• edited by macroscopeapp BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Verification

Scope

Show cached source-control status immediately in iOS before falling back to a full refresh

Uh oh!

coderabbitaiBot commented Aug 10, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

Uh oh!

saphid commented Aug 10, 2026

Uh oh!

macroscopeappBot commented Aug 10, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Approvability

Uh oh!

Uh oh!

saphid commented Aug 10, 2026

Uh oh!

t3-codeBot left a comment

Choose a reason for hiding this comment

Uh oh!

t3-codeBot left a comment

Choose a reason for hiding this comment

Uh oh!

t3-codeBotAug 12, 2026

Choose a reason for hiding this comment

Uh oh!

saphid commented Aug 13, 2026

Uh oh!

Uh oh!

saphid commented Aug 13, 2026

Uh oh!

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

saphid commented Aug 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

saphid commented Aug 10, 2026• edited by macroscopeapp BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Verification

Scope

Show cached source-control status immediately in iOS before falling back to a full refresh

Uh oh!

coderabbitaiBot commented Aug 10, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

Uh oh!

saphid commented Aug 10, 2026

Uh oh!

macroscopeappBot commented Aug 10, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Approvability

Uh oh!

Uh oh!

saphid commented Aug 10, 2026

Uh oh!

t3-codeBot left a comment

Choose a reason for hiding this comment

Uh oh!

t3-codeBot left a comment

Choose a reason for hiding this comment

saphid commented Aug 10, 2026•
edited by macroscopeapp Bot
Loading

coderabbitaiBot commented Aug 10, 2026•
edited
Loading

macroscopeappBot commented Aug 10, 2026•
edited
Loading

saphid commented Aug 10, 2026•
edited by macroscopeapp Bot
Loading

coderabbitaiBot commented Aug 10, 2026•
edited
Loading

macroscopeappBot commented Aug 10, 2026•
edited
Loading

saphid commented Aug 10, 2026•
edited by macroscopeapp Bot
Loading

coderabbitaiBot commented Aug 10, 2026•
edited
Loading

macroscopeappBot commented Aug 10, 2026•
edited
Loading