Repository navigation
linux: work-589 - #951
linux: work-589#951
Conversation
Agent-Team: linux Worker: codex
PM note: how this PR will be handledWhat it is for: linux: work-589 Rules that apply: touches build scripts. Things to watch (checked by rule):
What happens next:
A living status comment below is kept current. Nothing here is a merge decision. |
Where this PR standsStage: Done: approved What the PM needs before approving (ticks as it is met; the full CI runs only at the end, after the reviews are clean):
Rules that apply: touches build scripts. Reviews at the current commit (
13 earlier reviews are for older commits and will be redone. Latest fixes comment: https://github.com/fizzyedit/work/issues/592#issuecomment-6466335253 (4 findings answered) Updated 2026-10-29 04:02 UTC by the PM. This comment is edited in place. |
Agent-Team: linux Worker: codex
Agent-Team: linux Worker: claude
Agent-Team: linux Worker: codex
Agent-Team: linux Worker: codex
Part of fizzyedit/work#589
Rework: work#687
In plain words
This resolves the test conflict between the shared motion preferences work and current main. It keeps checks for shared preferences alongside the newer browser keyboard and fullscreen behavior.
What you will notice
Nothing visible: internal merge conflict resolution.
How this serves fizzy
Goal: shared motion preferences, preserving cross-platform behavior and evidence over opinion while bringing the branch up to date.
Goal
Resolve the harness-started merge of origin/main into PR #951 for fizzyedit/work#687.
What changed and why
Resolved web/tests/embed.test.mjs only: retained both storageApis and insets tracking, main's fullscreen-stack assertions, and the branch's shared preference/storage/focus tests. Preserved main's worker startup and keyboard inset coverage.
How it was verified
The combined embed test passes. Formatting, lint, native build (85/85), unit tests (191/191), web check (45/45), and SDK-version check (35/35) pass. Integration reaches 97/100 steps; all 19 failed tests encounter sandbox socket PermissionDenied. GATES.md names each denied test.
Not done / follow-ups
The harness must record the merge commit, push, verify mergeability, and post the single FIXES comment on work#592 naming web/tests/embed.test.mjs. Git writes and publishing are unavailable in this worker sandbox. No additional review findings were changed.
Changes (12 files, +459/-20)
Gates: 6 of 6 passed
scripts/check-fmt.sh(exit 0)scripts/zb build --summary all(exit 0)scripts/zb build test --summary all(exit 0)scripts/zb build test-integration --summary all(exit 0)scripts/zb build check-web --summary all(exit 0)scripts/zb build test-sdk-version --summary all(exit 0)