Skip to content

Preserve canvas pixels in rendered clones#55

Open
Valid-Systems wants to merge 2 commits into
tamnd:mainfrom
Valid-Systems:fix/blitzgraph-assets
Open

Preserve canvas pixels in rendered clones#55
Valid-Systems wants to merge 2 commits into
tamnd:mainfrom
Valid-Systems:fix/blitzgraph-assets

Conversation

@Valid-Systems

Copy link
Copy Markdown
Contributor

Summary

  • preserve rendered canvas pixels by converting readable canvases to inline PNG images before serializing the DOM
  • skip huge canvases before toDataURL to avoid worst-case HTML bloat
  • add a regression test that decodes the snapshot PNG and verifies the drawn red pixel
  • reuse an external Chrome DevTools endpoint for Windows render tests to avoid the existing Rod leakless launcher issue

Verification

  • go test ./browser -run TestRenderPreservesCanvasPixels -count=1
  • rtk go test ./browser -run 'TestRenderPreservesCanvasPixels|TestRenderCapturesFinalDOM|TestRenderRoutesNonHTML' -count=1
  • rtk go test ./browser -count=1
  • rtk go test -race ./browser -count=1
  • rtk go build ./cmd/kage
  • kage clone https://blitzgraph.com/ --control-url <local Chrome DevTools ws> --max-pages 1 --force -o D:\tmp\opencode\kage-blitzgraph\out-canvas-final, then served the clone and inspected it with Playwright: inlineCanvasImages=1, canvasTags=0, scriptTags=0, cloneFailed=0, cloneBadResponses=0

Known Windows main-branch failures

  • rtk go test ./... still fails on Windows in pack.TestBuildAppDir (AppRun is not executable) and clone render tests (fork/exec D:\tmp\leakless-...). The same focused failures reproduce from a clean main worktree at 105e68f.

@tamnd

tamnd commented Jun 21, 2026

Copy link
Copy Markdown
Owner

There are some conflicts with the current main branch. Could you please help resolve them?

DeepWiki Dev and others added 2 commits June 21, 2026 01:17
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@Valid-Systems Valid-Systems force-pushed the fix/blitzgraph-assets branch from 5d89619 to 943e119 Compare June 21, 2026 05:20
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