Conversation
This was referenced Sep 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Latest main compatibility
Current main
5f233f1is merged at head37312b9.dashboard-attention-unavailablereferences were removed with main; the replacementdashboard-trust-unavailablematrix was regenerated from the combined source.Research and scenario validation
The copy hierarchy follows Apple's guidance to write clearly, lead with the most important information, keep alerts actionable, and use disclosure for secondary detail (Writing, Alerts, Disclosure controls). Error states identify the problem and offer a recovery action in line with WCAG guidance (Error identification, Error suggestion); status changes remain available without turning raw diagnostics into the primary message (Status messages).
This was challenged against 100 simulated usage scenarios across three specialist review passes plus a final whole-diff audit: novice/expert readers, keyboard and large-text use, conflicting or absent evidence, stale/failed services, dense receipts, old/future timestamps, and provider-limit edge cases. This is role-play evaluation, not 100 human participants or a usability-study claim.
Review order
src/agentacct/receipt.pyandtests/test_receipt.py: deterministic project-scope and producer copy contracts.TemporalText.swift,TemporalTextTests.swift, and temporal call sites: the shared time-state policy.SourcesPane.swift,SourcesSnapshotHarness.swift, and Sources tests: failure precedence, retained-data disclosure, evidence roles, and privacy wording.DashboardPane.swift,DashboardStore.swift, andUsageCapacity.swift: signal-rail architecture plus typed source failures and centralized reset copy.Test plan at current head
37312b9python -m pytest -q— 2,667 passed; one third-party Starlette deprecation warning.swift test— 224 passed; seven visual wrappers skipped by design and exercised by the next command../Scripts/visual-snapshots record DashboardVisualRegressionTests MenuVisualRegressionTests SetupVisualRegressionTests SourcesVisualRegressionTests UsageVisualRegressionTests WorkVisualRegressionTests— all six suites recorded and immediately verified onmacos-26.6-25G72-xcode-26.6-17F113-arm64-2x.swift build -c releasepassed at exact head.AppBuildIdentityTests: 4 passed at exact head.git diff --checkis clean.Fresh GitHub CI is running for this head.
Safety checklist
--jsonis used.Docs
Manual follow-up boundary
CI and deterministic native rendering can verify the code and reference matrix. They do not substitute for a human VoiceOver pass or a signed installed-app session against a live daemon; this PR does not claim those manual checks were performed.