Skip to content

fix(macos): register the welcome account-choice parity surface - #4724

Merged
tyler-dane merged 2 commits into
mainfrom
fix/macos-parity-account-choice-surface
Oct 22, 2026
Merged

tyler-dane merged 2 commits into
mainfrom
fix/macos-parity-account-choice-surface

Conversation

@tyler-dane

Copy link
Copy Markdown
Contributor

What and why

Refs #4331

NativeParityCaptureTests.captureWelcomeOnboarding (from #4707) exports a second capture, onboarding-welcome-account-choice. That name was not a ParityCaptureSurface, so DesktopDebugCommand.parse returned nil, the app never wrote the PNG, and macos-capture timed out on every PR that touches CompassUI (seen on #4718 and #4720 once the earlier button-label failures were fixed).

  • Register the surface (one enum case; the exporter is generic).
  • Parse test for the new name.
  • Checklist changelog line.

Cloud session: no macOS runner or Swift toolchain here, so the Swift test and capture run on CI only.

Verify

bun run verify --strict

VERDICT: PASS


Generated by Claude Code

The onboarding capture exports onboarding-welcome-account-choice, but the
name was not a ParityCaptureSurface, so the debug URL never parsed and the
export timed out.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GKHjv2HDx1DFAnG4cfSxcr
@tyler-dane
tyler-dane marked this pull request as ready for review October 22, 2026 04:42
@tyler-dane tyler-dane added the agent-automerge Agent-loop finished; merge-guard may squash-merge label Oct 22, 2026 — with Claude
@tyler-dane
tyler-dane enabled auto-merge October 22, 2026 04:42
@tyler-dane tyler-dane added the parity-screenshots Run the web vs macOS parity screenshot workflow on this PR label Oct 22, 2026 — with Claude
@tyler-dane
tyler-dane added this pull request to the merge queue Oct 22, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Oct 22, 2026
…unt-choice-surface

# Conflicts:
#	docs/features/desktop-web-parity-checklist.md
@tyler-dane
tyler-dane enabled auto-merge October 22, 2026 06:07
@github-actions

Copy link
Copy Markdown
Contributor

Boot bundle: 512.1 KB gzip (+0.0 KB vs main)

@tyler-dane
tyler-dane added this pull request to the merge queue Oct 22, 2026
Merged via the queue into main with commit b14de34 Oct 22, 2026
26 checks passed
@tyler-dane
tyler-dane deleted the fix/macos-parity-account-choice-surface branch October 22, 2026 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-automerge Agent-loop finished; merge-guard may squash-merge parity-screenshots Run the web vs macOS parity screenshot workflow on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants