chore(producer): resync stale committed puppeteer-core symlinks to lockfile version

[vanceingalls]

Status: superseded, empty diff

This PR originally resynced 3 stale puppeteer-core/puppeteer git-tracked symlinks under packages/producer/node_modules/. That fix is no longer needed: a separate PR (fix(producer): stop tracking node_modules symlinks, part of #2144) untracked those files entirely and merged into main first — a cleaner fix than this one's resync.

After restacking this branch onto current main, the diff is now empty (0 files) — confirmed by GitHub refusing a base-branch update with "no new commits between base branch 'main' and head branch."

Earlier review flagged a diff far broader than the stated 3-file resync (including .DS_Store and other tracked artifacts). That was drift from comparing against a stale pre-merge base — those files were removed by the same #2144 cleanup, independently of this PR, before this branch's comparison base caught up.

No action needed from a reviewer; this PR can be closed once confirmed.

[vanceingalls]

• fix(cli): surface video frame-injection failures in snapshot output #2255
• fix(shader-transitions): pin cloned scene roots to composition dimensions #2254
• fix(cli): fold canvas pixel hashes into the frozen-sweep fingerprint #2253
• chore(producer): resync stale committed puppeteer-core symlinks to lockfile version #2259 👈 (View in Graphite)
• fix(producer): disk-headroom gate before hdr raw frame pre-extraction #2256
• feat(cli): preview --browser-no-gpu for gpu-unstable hosts #2252
• fix(cli): settle post-seek font subset loads before audit screenshots #2251
• fix(engine): densify drawelement self-verify with parallel worker count #2250
• fix(producer): withhold de parallel router below a machine ram floor #2249
• fix(producer,engine): gate drawelement on ancestor background-image + tail verify sample #2247
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[miga-heygen]

#2259 — decouple statfsSync error handling from headroom gate

Verdict: LGTM 🟢

Incremental on top of #2256. Refactors the awkward re-throw pattern I noted in my #2256 review: the statfsSync try/catch was wrapping the headroom check too, requiring a err.message.includes("HDR pre-extraction needs") re-throw to let the headroom error propagate while swallowing the statfs error.

Now cleanly separated: statfsSync gets its own early-return try/catch (unsupported platform → skip the gate), and the headroom check + warning run outside any try/catch with a guaranteed freeBytes value. Same behavior, clearer control flow, no string-matching re-throw.

Review by Miga

[miguel-heygen]

Blockers on current head c20e202: (1) required regression check is failing and additional shards are pending; (2) the actual PR diff is not limited to the three Puppeteer symlink resyncs described in the body — it includes tracked packages/producer/node_modules contents and .DS_Store plus broad stacked changes, so the branch is not an atomic canonical fix. Rebase/split onto the intended base, remove generated artifacts and unrelated files, remove the AI-generated trailer from the public PR description, then rerun CI and request re-review.

[miguel-heygen]

Correction: my prior CHANGES_REQUESTED was based on the stale Graphite base file list. Comparing the head tree to current main shows identical tree SHA (3e0019c87f78) and no real delta; the refactor already landed via #2256. This PR has no code delta to review or stamp and should be closed as landed, or replaced with a fresh atomic symlink-only PR if that work is still intended.