Skip to content

fix: surface nested target-Mac UI diagnostics - #55

Merged
daniele21 merged 2 commits into
devfrom
prs18/propagate-ui-diagnostic
Sep 10, 2026
Merged

fix: surface nested target-Mac UI diagnostics#55
daniele21 merged 2 commits into
devfrom
prs18/propagate-ui-diagnostic

Conversation

@daniele21

Copy link
Copy Markdown
Owner

Outcome

Ensure repeated target-Mac closedroom_window_missing failures expose the existing privacy-bounded AX/WindowServer diagnostic in the LOCAL REAL_ENVIRONMENT aggregate instead of losing it at the parent-runner boundary.

Root cause

real_environment_smoke.py owns the detailed UI failure and writes it to the UI evidence report. measured_release_target_mac.py and record_while_ai_busy_target_mac.py intentionally summarize that child failure as a failed check, so child_summary() was looking only at parent errors and never following ui_evidence_report / seed_ui_report.

Scope

  • follow only the two canonical nested UI report references;
  • require nested reports to resolve in the same evidence directory as the parent report;
  • reuse the existing strict diagnostic whitelist;
  • keep parent error counts and arbitrary payloads out of the aggregate;
  • add regression tests for measured, contention, out-of-directory references and arbitrary private error payloads.

Four axes

  • PRODUCT: PRODUCT_NONE — evidence propagation only.
  • DELIVERY: INTEGRATION into dev.
  • VALIDATION: repository selector authoritative; expected contained tooling scope.
  • EXECUTION: REMOTE_AUTOMATED for deterministic gates; REAL_ENVIRONMENT remains required for the actual diagnostic observation.

Invariants

  • no product/runtime/audio/UI behavior changes;
  • no cloud fallback or external meeting-data transfer;
  • no arbitrary child error payloads, UI labels/titles, transcript or meeting text copied into aggregate evidence;
  • release qualification remains unchanged.

@daniele21
daniele21 merged commit 3471ace into dev Sep 10, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant