docs(gmeet-parity): row 15 codec-gap status post #1251 build infra (#1223)#1282
Conversation
…ild infra (tinyhumansai#1223) Reflect that the Chrome-branded CEF build infrastructure landed in scripts/cef-with-codecs/ via tinyhumansai#1251. Code-side work for the proprietary codec gap is complete; remaining blockers are operational (license clearance + binary hosting), not code. Sign-off action items updated to track tinyhumansai#1223 as the umbrella for the codec-built CEF rollout (license + hosting + tauri-cef re-pin) instead of the now-stale tinyhumansai#1053 reference. Closes tinyhumansai#1223
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Google Meet parity QA documentation was updated to reflect Phase A diagnostic results and revised follow-up actions. The Background effects section expanded from a partial-status note to a detailed narrative covering codec limitations and build-infra guidance. The Sign-off section was refreshed with updated results, dates, and explicit next-step references. ChangesQA Documentation Update
Estimated Code Review Effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
Summary
docs/qa/GMEET-PARITY.mdrow 15 to reflect that the Chrome-branded CEF build infrastructure landed inscripts/cef-with-codecs/via feat(cef-build): proprietary-codec build infra for Gmeet dynamic backgrounds (#1223) #1251.tauri-cefre-pin.#1053reference to#1223as the umbrella for the codec-built CEF rollout.Problem
After #1251 merged the
scripts/cef-with-codecs/build scripts (Chrome-branded FFmpeg viaautomate-git.py,install-local.sh, license posture memo), the GMEET-PARITY audit doc was still telling readers that the next step was "filing a follow-up againsttinyhumansai/tauri-cef" and that the row-15 work was "out of scope for openhuman code". Both statements were accurate on 2026-05-05 and stale on 2026-05-06 — the openhuman-side build infra now exists; what's missing is the legal sign-off and a binary host.Issue #1223 has been open since 2026-05-05 because PR #1251 shipped without a
Closeskeyword. The umbrella tracker for the codec-built CEF rollout therefore still read as "needs investigation" to anyone scanning the issue list, even though the code-side investigation work is done.Solution
Doc-only update to row 15 of
docs/qa/GMEET-PARITY.md:scripts/cef-with-codecs/README.md, list the two operational blockers (license clearance, binary hosting), and define the smoke acceptance criterion (MediaSource.isTypeSupported('video/mp4; codecs="avc1.42E01E"') === true) for flipping the row to ✅ once the codec-built binary lands.Auto-memory
feedback_cef_runtime_gaps.mdgap #3 was updated in parallel (outside this repo, in the local memory store) so future sessions surface the new status; that file is not part of the diff.No code, scripts, build config, or vendored-binary changes — this is purely a documentation correctness fix that retires a stale instruction.
Submission Checklist
docs/TESTING-STRATEGY.mddiff-cover) meet the gate enforced by.github/workflows/coverage.yml. Runpnpm test:coverageandpnpm test:rustlocally; PRs below 80% on changed lines will not merge.docs/TEST-COVERAGE-MATRIX.mdreflect this change (orN/A: behaviour-only change)## Relateddocs/TESTING-STRATEGY.md)docs/RELEASE-MANUAL-SMOKE.md)Closes #NNNin the## RelatedsectionImpact
docs/qa/GMEET-PARITY.mdrow 15 or auditing the sign-off block now sees the post-feat(cef-build): proprietary-codec build infra for Gmeet dynamic backgrounds (#1223) #1251 status (build infra shipped, awaits license + hosting) instead of the stale "follow-up to be filed" wording. Closes the dangling Vendored CEF lacks proprietary codec support — Gmeet dynamic video backgrounds fail with FFmpegDemuxer error #1223 umbrella so the issue board reflects code-side completion.Related
tauri-cefre-pin to a hosted Chrome-branded archive (operational, tracked outside the repo); rerunscripts/diagnose-cef-runtime.mjs probeagainst the new binary to captureMediaSource.isTypeSupported('video/mp4; codecs="avc1.42E01E"') === trueas the row-15 ✅ smoke artefact.AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
Validation Run
pnpm --filter openhuman-app format:checkpnpm typecheckValidation Blocked
command:N/Aerror:N/Aimpact:N/ABehavior Changes
scripts/cef-with-codecs/and the operational blockers list.Parity Contract
Summary by CodeRabbit