Skip to content

[KS] Driver runs: pin TZ, mask architecture, follow-ups (#433) - #454

Merged
sethwebster merged 1 commit into
seth/kitchen-sink-gatefrom
seth/ks-433-driver-observers-2
Oct 13, 2026
Merged

sethwebster merged 1 commit into
seth/kitchen-sink-gatefrom
seth/ks-433-driver-observers-2

Conversation

@sethwebster

Copy link
Copy Markdown
Collaborator

Refs #433

Follow-up to #451.

What changed

  • state-visual.yaml: also masks system-integration-state-architecture, since arm64 shows at the golden's bottom edge.
  • scripts/e2e/kitchen-sink.ts: the driven process now launches with TZ=UTC (PINNED_ENV, applied before the flow's own env, so a flow can still override it). JavaScript's Date/Intl and NSTimeZone now match the fixed timeZone. tests/e2e-kitchen-sink-launch.test.ts checks the fake app receives it. Documented in test-driver.md and system.md.
  • Simulated powerChanged: rather than adding a batteryLevel field, system.md now says that onBattery is simulated and batteryLevel stays null.
  • SparkSystemState.mm: NSCAssert(overrides) before simulateSystemEvent writes the driver state.
  • RNWindowManager.mm: the comment now points to docs/displays.md#simulated-displays.

Verification (run in this session)

  • bun run typecheck: pass.
  • bun run e2e:lint: 0 problems.
  • bun run test: 1981 passed, 3 failed:
    • node-runtime (asdf shim) and the motion-native frames check (display asleep): the accepted host failures.
    • media-native playback timing (media.native.mm:174): passed when rerun on its own, and this PR doesn't touch media.
  • Rebuilt the Kitchen Sink, then ran system-integration and infra/catalog-driver with --backend driver under ks-lock native: 41 passed, 0 failed, 4 skipped (Windows-only).
  • With TZ=UTC and the extra mask, system-integration-state (4 variants) and infra-shell-catalog matched with 0 different pixels. No golden was re-approved.

Launch the driven process with TZ=UTC so Date and Intl match the fixed
timeZone; mask the architecture row in state-visual; document that a
simulated powerChanged leaves batteryLevel null; assert the driver state
exists before simulateSystemEvent writes it; fix a doc reference.
@sethwebster
sethwebster merged commit 4f01076 into seth/kitchen-sink-gate Oct 13, 2026
2 checks passed
@sethwebster
sethwebster deleted the seth/ks-433-driver-observers-2 branch October 13, 2026 06:18
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