Skip to content

ci(review): install ffmpeg for video proof inspection#146

Merged
Takhoffman merged 1 commit into
mainfrom
codex/install-ffmpeg-review-runner
May 20, 2026
Merged

ci(review): install ffmpeg for video proof inspection#146
Takhoffman merged 1 commit into
mainfrom
codex/install-ffmpeg-review-runner

Conversation

@Takhoffman
Copy link
Copy Markdown
Contributor

Summary

  • add a shared setup-media-proof-tools action that ensures ffmpeg and ffprobe are available
  • run it in both exact event re-review and review shard jobs before video proof inspection
  • fail fast on non-apt runners that do not already provide ffmpeg/ffprobe, instead of silently producing ENOENT media artifacts

Validation

  • pnpm run check:active-surface
  • pnpm exec oxfmt --check .github/workflows/sweep.yml .github/actions/setup-media-proof-tools/action.yml
  • pnpm run check

Root cause

The ffmpeg video proof code was deployed, but GitHub review runners did not install ffprobe/ffmpeg before ClawSweeper prepared video proof artifacts, so media extraction reported spawnSync ffprobe ENOENT.

@Takhoffman Takhoffman merged commit 6a5bd62 into main May 20, 2026
7 checks passed
@Takhoffman Takhoffman deleted the codex/install-ffmpeg-review-runner branch May 20, 2026 02:53
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