Skip to content

[codex] Fence Web correction preview ownership and background lifecycle - #40

Merged
atomicdeploy merged 2 commits into
mainfrom
codex/web-preview-ownership
Oct 30, 2026
Merged

atomicdeploy merged 2 commits into
mainfrom
codex/web-preview-ownership

Conversation

@atomicdeploy

Copy link
Copy Markdown
Owner

Scope

Related to #3 and #2. Preserve the single canonical Web source, existing editor
and shared HTTP owner. The already-integrated preview-on-edit adapter now keeps
the original connection-generation pin for acquire, renewal, readback and release
instead of obtaining fresh generic command admission. No named WebSocket preview
command exists, so none is invented.

Hidden-document/pagehide events retire the owned RAM lease. Foreground/BFCache
return is passive; disposal removes listeners. Explicit Save remains the only
persistent correction writer. Rules, indices and diagnostic numbering are unchanged.

Verification

  • Pre-fix regression reproduced; real-module editor DOM, controller and shared
    HTTP tests pass, including captured-pin identity, background/page departure,
    passive restoration, same-board connection replacement and listener disposal.
  • The fixture now uses unique lease IDs per acquisition rather than reusing7.
  • Single-source, app-contract, dialog, control-presentation, living-contract and
    hash-bound comparison checks pass. Only reviewed changed-file references were
    rebound; donor counts and pending gaps are not promoted.
  • Full/onboard low-write builds and no-write checks match revision
    aa518a96eea2dd7d787cb0a31d5191c78886e2e31605eda64458a0cea728835d:
    HTML546206 bytes / gzip154031 bytes.
  • Existing read-only simulated preview serves the updated source with HTTP200;
    this is not rendered-browser or hardware acceptance.

Remaining boundaries

Explicit workspace-panel hiding, rendered/RTL/touch lifecycle and actual board
lease/output acceptance remain open. Windows browser URL-policy enforcement
still blocks visual automation; no alternate route bypassed it. No board,
production client, saved configuration, firmware upload or actuator was contacted
or changed. The previously blocked/deferred generated bytecode file is untouched.

CI remains a separate exact-head gate. GitHub connector creation returned403
because its integration lacks write access; authenticated gh is the documented
publishing fallback.

@atomicdeploy
atomicdeploy marked this pull request as ready for review October 30, 2026 13:22
@atomicdeploy
atomicdeploy merged commit d5008bd into main Oct 30, 2026
14 checks passed
@atomicdeploy
atomicdeploy deleted the codex/web-preview-ownership branch October 30, 2026 14:00
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.

2 participants