Repository navigation
docs(video): reconcile Video Bridge canonical state and false 9/9 closure (#11661) - #12001
Merged
diegosouzapw merged 1 commit intoAug 29, 2026
Merged
Conversation
…lared (#11661) The transcript source field (client/embedded/audio-bridge) is validated against an enum but is not yet server-verified — OmniRoute does not confirm that an embedded/audio-bridge label actually came from a server-owned extraction. GUARDRAILS.md previously documented the enum without flagging this gap, which is exactly what issue #11652 exists to close. This is a documentation correction only; no runtime behavior changes. Refs #11661
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
…lared (cheaper-inference#11661) (cheaper-inference#12001) Reconciles the Video Bridge FU-01..09 backlog docs against verified code and GitHub state (ground truth established first, per this repo's Documentation accuracy rule), correcting a real gap in GUARDRAILS.md: the transcript source field was documented as validated without noting OmniRoute didn't yet verify server-side extraction — exactly the gap cheaper-inference#11652 (now merged as cheaper-inference#12009) closes. Refs cheaper-inference#11661, not Closes — truthfully closing it needs the sibling PRs' actual landed state folded back in, left as an explicit follow-up.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Reconciles the Video Bridge follow-up backlog (
VB-FU-01..09, umbrella#9760) documentationagainst verified code and GitHub state, and corrects a real accuracy gap this audit found in the
public guardrail docs.
Ground truth was established first (per this repo's Documentation accuracy rule), then docs
were corrected to match — not the other way around. Method, evidence, and every command run are
recorded in the artifacts below.
What this PR actually does
docs/security/GUARDRAILS.md): the transcriptsourcefield(
client/embedded/audio-bridge) was documented as a validated enum without stating thatOmniRoute does not yet verify the label came from a real server-side extraction. Any caller can
set
source: "embedded"today — confirmed by readingnormalizeVideoTranscriptinsrc/lib/guardrails/videoBridgeHelpers.ts:111-172. This is exactly the gap issue#11652existsto close. Added one clarifying paragraph; no other public doc (
docs/openapi.yaml,docs/security/ROUTE_GUARD_TIERS.md,CHANGELOG.md) needed correction — they already stateVB-FU-07/VB-FU-09promotion asHOLDandVB-FU-08as not wired to production, accurately.changelog.d/maintenance/11661-video-bridge-transcript-provenance-doc.md._tasksrepo (pushed, not part of this PR's diff):_tasks/research/2026-08-29-video-bridge-current-ledger.md— the one CURRENT authority forVB-FU-01..09, each item'simplementation_state/verification_evidence/promotion_evidencetraced to an exact merged commit (withgit merge-base --is-ancestorchecks) and to test runs performed in this session. No VPS receipts are claimed — this session
has no VPS access, and every field that would require one says
ABSENT._tasks/research/2026-08-17-video-bridge-skill-gap-matrix.md— markedhistorical/superseded with a banner identifying exactly which of its claims were false and
why; original text preserved, nothing deleted.
_tasks/postmortems/2026-08-29-video-bridge-false-nine-of-nine-closure.md— root-causepostmortem on how a 2026-08-18 update wrongly declared
VB-FU-01..09"9 of 9 IMPLEMENTADO".Ground-truth findings (the actual audit)
761d38f4ancestor of HEAD; test passes04d2a603ancestor of HEAD; test passes7715825c(not the stacked-branch merge SHAghreports) — ancestor of HEAD; test passesd23bfefeancestor of HEAD; tests passnormalizeVideoTranscript); PR #11680 (embedded subtitle + redaction) is still an open draft, not merged46b8e7a9, ancestor of HEAD) fixes fusion timeline ordering only, per its own PR body — real Audio Bridge/STT orchestration (#11654's actual ask) does not exist in the codebase (extractAudio/PCM/16 kHz— zero matches)9fedc1c4); model-quality promotion isHOLDper#11656, already stated accurately in GUARDRAILS.mdc83116e6); producer/consumer lifecycle and resolution variants absent (#11655), already stated accurately in GUARDRAILS.mdc8ad44e0); promotionHOLDper#11656, same as FU-07#11657); zero matches forvideoBridgePipeline/processVideoPart/port interfacesTDD / validation evidence
This is a docs-only PR in the main repo — no production code changed, so no new test was written.
Existing tests were run to verify the ledger's claims, not to prove a code change:
Total: 213 passed, 0 failed, 2 skipped across the full local Video Bridge test surface.
Gates run (this PR's diff — docs + changelog fragment only)
npm run check:docs-allcheck:fabricated-docs --strict: "No fabricated API/env/CLI/hook/file references found"npm run typecheck:corenpx eslint --suppressions-location config/quality/eslint-suppressions.json docs/security/GUARDRAILS.mdnode scripts/check/check-test-discovery.mjscheck:complexity-ratchets,check:cycles, andcheck:file-sizewere not run — nosrc/,open-sse/, orelectron/file was touched.356 for providers. Unrelated to this PR's diff.
Pending live validation
This ticket's acceptance criteria ("every implementation PR merged and ancestral to
release/v3.8.51," "exact merged SHA passes the required VPS and CI matrix," "no required check
red/stale/cancelled/skipped/unattributed," "settings and temporary VPS artifacts restored") cannot
be truthfully claimed today:
#11652,#11654,#11655,#11656,#11657,#11658,#11659are all OPEN as of2026-08-29 and are being worked in parallel by other sessions per this ticket's own dispatch
context.
#11680(would close#11658/#11659) is an open draft, not merged, and its own bodysays it does not authorize merge.
was performed or is claimed anywhere in this PR or in the private ledger — this session has no
VPS access.
gh pr viewreports for the cited PRs was collected.This PR therefore uses
Refs #11661, notCloses— closing#11661truthfully requires thesibling PRs above to merge first, then a follow-up pass to update the CURRENT ledger with their
actual landed state and, where available, real VPS evidence.
Refs #11661