Skip to content

fix(video-bridge): validate structural segment sampling - #11381

Merged
diegosouzapw merged 2 commits into
release/v3.8.50from
fix/v3850-video-fu07-segment-aware
Aug 24, 2026
Merged

diegosouzapw merged 2 commits into
release/v3.8.50from
fix/v3850-video-fu07-segment-aware

Conversation

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Summary

  • make the opt-in segment_aware Video Bridge sampler use one bounded, local-only FFmpeg structural pass for scene, freeze, blur, exposure, and SI/TI evidence
  • allocate the existing 1-16 frame budget away from frozen spans while preserving long-tail timeline coverage and the legacy allocation when structural evidence is unavailable
  • fail open to deterministic uniform sampling on malformed evidence, missing filters, detector failure, or timeout; caller aborts still terminate extraction and clean the private temporary tree
  • add a deterministic real-media evaluation harness and document exactly which results are measured versus held

TDD and real-media evidence

  • focused affected matrix: 53/53 PASS
  • FU-07 evaluator: 5/5 fixtures PASS
  • real frozen-to-motion fixture: frame budget reallocates 1:3
  • static fixture after production dedup: caption calls 2 -> 1 (one incremental call avoided)
  • long-tail, gradual-fade false-positive, blur/exposure, SI/TI, timeout, abort, and cleanup regressions are covered

Deliberate HOLDs

This PR does not claim semantic caption-model quality or a real monetary gain. No authorized caption endpoint, model credentials, frozen judge, or provider cost receipt was configured. CPU/RSS measurement also remains a HOLD when /usr/bin/time is unavailable. The harness reports those states explicitly instead of fabricating values.

Validation

  • npm run typecheck:core — PASS
  • focused ESLint + Prettier — PASS
  • docs/changelog/cycles/file-size/tracked-artifacts/diff checks — PASS
  • typecheck:noimplicit:core — BASE-RED only in unchanged open-sse/translator/response/openai-responses/pureHelpers.ts; no FU-07 diagnostic

Base

Validated against exact release tip dafb4ae808305c9287c751cf1ce16fe2234ba073.

The PR remains draft for the broader release campaign; no merge, tag, release, or deployment is requested.

@diegosouzapw

Copy link
Copy Markdown
Collaborator Author

CI classification snapshot — 2026-08-24

This note does not claim the full CI is green. The PR remains OPEN/DRAFT; no rerun, waiver, ready-for-review, auto-merge, or merge was performed.

@diegosouzapw
diegosouzapw marked this pull request as ready for review August 24, 2026 12:24
@diegosouzapw
diegosouzapw merged commit 9fedc1c into release/v3.8.50 Aug 24, 2026
26 of 30 checks passed
@diegosouzapw

Copy link
Copy Markdown
Collaborator Author

Merged via consolidated batch validation (worktree .claude/worktrees/merge-11381, resolved against tip after sibling #11344 landed on the shared videoBridgeRuntime.ts/GUARDRAILS.md seam — one docs conflict, combined additively). Opt-in segment_aware structural sampler: one bounded local-only FFmpeg pre-analysis pass (scene/freeze/blur/exposure/SI-TI), budget reallocation away from frozen spans while preserving uniform fallback, deterministic real-media evaluation harness. Static gates green; own regression suite (videoBridgeFu07StructuralSampling.test.ts, videoBridgeSampler.test.ts) 21/21 pass (2 real-FFmpeg cases correctly skip — optional runtime dependency) + typecheck:core clean. Thanks!

diegosouzapw pushed a commit that referenced this pull request Aug 24, 2026
…11382/#11383 growth

These entries were already validated in an earlier merge-batch worktree but
never reached origin (worktree discarded before pushing). Re-adding them
here since #11355's test/route.ts growth (1215->1237) is now live on
origin/release/v3.8.50 and fails the frozen cap otherwise.
@diegosouzapw
diegosouzapw deleted the fix/v3850-video-fu07-segment-aware branch August 25, 2026 02:37
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…per-inference#11344/cheaper-inference#11381/cheaper-inference#11362/cheaper-inference#11382/cheaper-inference#11383 growth

These entries were already validated in an earlier merge-batch worktree but
never reached origin (worktree discarded before pushing). Re-adding them
here since cheaper-inference#11355's test/route.ts growth (1215->1237) is now live on
origin/release/v3.8.50 and fails the frozen cap otherwise.
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