Skip to content

Preserve hidden GC data in saved sessions - #44

Merged
happykhan merged 1 commit into
codex/shared-plot-scenefrom
codex/preserve-session-gc-data
Sep 5, 2026
Merged

happykhan merged 1 commit into
codex/shared-plot-scenefrom
codex/preserve-session-gc-data

Conversation

@happykhan

Copy link
Copy Markdown
Owner

Summary

  • keep raw GC content and skew arrays in saved editable sessions even when their display controls are off
  • retain the display choice in session parameters
  • verify that a saved session reloads with GC hidden and can restore the ring without recomputation

Why

Visibility is presentation state. The previous save path removed hidden GC arrays from the embedded plot, permanently preventing users from turning those rings back on after reloading a session.

Stack

This is intentionally separate from and stacked on #43 so the renderer refactor and session-format behaviour can be reviewed independently.

Verification

  • focused Playwright save, reload, re-enable, and SVG export regression
  • npm test (181 passed)
  • npm run build
  • npm run test:e2e (25 passed)
  • npm run test:e2e:dev (1 passed)

@happykhan
happykhan force-pushed the codex/preserve-session-gc-data branch from b585dcc to 82301f0 Compare September 4, 2026 07:36
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 4, 2026 •

Copy link
Copy Markdown

Deploying brigx with  Cloudflare Pages  Cloudflare Pages

Latest commit: 82301f0
Status: ✅  Deploy successful!
Preview URL: https://37f2183c.brigx.pages.dev
Branch Preview URL: https://codex-preserve-session-gc-da.brigx.pages.dev

View logs

@happykhan
happykhan merged commit b74ce2a into codex/shared-plot-scene Sep 5, 2026
1 check 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