mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-09-17 20:32:25 +03:00
3f36e8b03a
Task review found a cache-version skew: descriptionRedacted was added to VideoResultCacheMetadata without bumping VIDEO_BRIDGE_RESULT_CACHE_VERSION, so a stale pre-diff cache entry could be served post-diff with descriptionRedacted silently undefined -> videoBridgeObserved: false for a video that DOES have a transcript, silently disabling redaction on repeat requests. - videoBridgePipeline.ts: bump VIDEO_BRIDGE_RESULT_CACHE_VERSION "v5" -> "v6", documenting the descriptionRedacted addition as the reason (v6 (#12150), alongside the existing v5 (#11652) precedent comment). - videoBridgeTranscriptCacheIdentity.test.ts: strengthen the contract-version guard from notEqual("v4") (forward-compatible with any bump, so it would not have caught a stale v5 entry) to equal("v6") -- strictly tighter, not weaker. Renamed to reflect what it now asserts. - videoBridgeResultCache.test.ts: found via a repo-wide sweep for the old "v5" literal (not in the original review list) -- one more real end-to-end test asserted the exact cached value and broke the same way; fixed to "v6". Four other "v5" occurrences in that file are synthetic corrupt/invalid- metadata fixtures whose failure path is caught before identity matching runs, so they are untouched (confirmed still passing pre- and post-bump). - Two trivial hand-formatted Prettier nits (videoBridge.ts spread collapsed to one line; a test's "as UnionType" cast re-wrapped to match this repo's existing pattern) since eslint/prettier cannot run here (@eslint/compat missing repo-wide, pre-existing). Refs #12150