Skip to content

fix(hyperframes): resolve render deadlock, panic risk, and over-const… - #1

Merged
iMyth merged 2 commits into
mainfrom
develop
Jul 18, 2026
Merged

fix(hyperframes): resolve render deadlock, panic risk, and over-const…#1
iMyth merged 2 commits into
mainfrom
develop

Conversation

@iMyth

@iMyth iMyth commented Jul 18, 2026

Copy link
Copy Markdown
Owner

…rained prompt

  • Fix stdout pipe deadlock: change Stdio::piped() to Stdio::null() for render process stdout in section_video.rs and render.rs. The piped stdout was never consumed, causing the OS pipe buffer (64KB) to fill up and deadlock the Chrome render process.

  • Replace unwrap() calls in timeline.rs with safe match patterns to eliminate panic risk from TOCTOU race between has_audio check and frag_map lookup in concurrent batch generation.

  • Rewrite agent prompt: convert 6 negative constraints (禁止/不要/不能) to positive creative direction. Add role framing, restructure into clear sections (role, content, visuals, rhythm, technical), and move render engine requirements to a dedicated numbered list with rationale.

…rained prompt

- Fix stdout pipe deadlock: change Stdio::piped() to Stdio::null() for
  render process stdout in section_video.rs and render.rs. The piped
  stdout was never consumed, causing the OS pipe buffer (64KB) to fill
  up and deadlock the Chrome render process.

- Replace unwrap() calls in timeline.rs with safe match patterns to
  eliminate panic risk from TOCTOU race between has_audio check and
  frag_map lookup in concurrent batch generation.

- Rewrite agent prompt: convert 6 negative constraints (禁止/不要/不能)
  to positive creative direction. Add role framing, restructure into
  clear sections (role, content, visuals, rhythm, technical), and move
  render engine requirements to a dedicated numbered list with rationale.
@iMyth
iMyth merged commit 8d44bb0 into main Jul 18, 2026
3 checks 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