Skip to content

Capture direct native frames and verify both installed rendering paths - #168

Draft
proggeramlug wants to merge 1 commit into
codex/windows-jolt-packagefrom
codex/direct-2d-capture
Draft

Capture direct native frames and verify both installed rendering paths#168
proggeramlug wants to merge 1 commit into
codex/windows-jolt-packagefrom
codex/direct-2d-capture

Conversation

@proggeramlug

@proggeramlug proggeramlug commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

A native game using setDirect2DMode(true) renders and exits without producing its queued PNG. The direct frame path now services output capture after drawing through the existing framebuffer readback implementation. Render-target-only frames keep the output request pending, and ordinary frames retain their submission behavior.

Extract the direct frame method into its own module and extend the installed-package gate to compile and run the same Jolt physics fixture through both scene and direct-2D rendering. Add a renderer regression for uncaptured frames, capture deferral across a smaller render texture and successive captures with different exact pixels.

Local validation: all four installed fixture captures match all 16,384 expected pixels on Radeon DX12 and Vulkan. Each backend also passes all five render-target tests and all 93 image goldens, with four existing ignored cases. Contracts, formatting, strict Clippy, WASM compilation and blank/physics-failure image controls pass. Image baselines and thresholds are unchanged. Original missing-capture evidence and the WASM guard correction are retained.

Stacked on #167. Hosted Tests run 34559712338 passes all 22 jobs, including both installed rendering modes with all expected pixels, native engine build and all 20 example links. Hosted source trees match the PR head. This qualifies native headless capture with the validated Perry 0.5.1220 runtime profile; local DX12 uses SDK compiler DLLs on PATH. Visible presentation, browser startup/capture, packaged DXC and the complete starter/lifecycle remain separate. Relates to #140, #142 and #145.

Published report and evidence retain the original missing-capture and WASM failures, exact final source, fresh installed scene/direct-2D checks and all final hosted results. ZIP SHA-256: f0f2a03b41d1f926c900a9901831f2a96a5311cab55f22c63e60f0b7f542dd34.

@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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