fix(media): reconcile Gallery total and sanitize transcripts (#108) - #122
Merged
Conversation
Kaltura can leave an enabled no-op Load More control after every displayed entry is already present. Treat a cumulative page at its displayed total as exhausted instead of turning truthful 2/2 discovery red. Assisted-by: GPT-5 Co-authored-by: Codex <noreply@openai.com>
Keep local formatter prompts and CLI envelopes out of visible transcript derivatives, and reject any prompt that reaches artifact validation. Assisted-by: GPT-5 Co-authored-by: Codex <noreply@openai.com>
Keep Gallery pagination fail-closed until cumulative mode is confirmed, preserve fenced transcript content, and distinguish prompt leakage from legitimate speech. Assisted-by: GPT-5 Co-authored-by: Codex <noreply@openai.com>
jerome-queck
force-pushed
the
codex/issue-108-gallery-exhaustion
branch
from
August 22, 2026 12:39
dff2929 to
378708c
Compare
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.
Closes #108
Summary
Production evidence
npm run verify -- all: 830/830 present.~/Library/Logs; an explicit manual production run across all eight active courses and four provider lanes exited 0. The resulting active digest remains 374 total, 50 complete, 324 explicit red/unsupported, and zero queued or active.--manualstarted from launchd, processed all eight active courses and four provider lanes, and exited 0; the normal daily 00:05 schedule was restored afterward.Validation
node --test test/media-gallery-browser.test.mjs test/media-formatter.test.mjs test/media-transcript.test.mjs: 28/28 pass.npm run lint: pass.npm run format:check: pass.node --test test/media-worker.test.mjs: 17/17 pass, including the 04:00 checkpoint/resume case.git diff --check: pass.npm test: 365/366 pass. The existing environment-sensitivemedia-setupCLI test invokes the real ignored configuration and receives sandboxEPERMwriting/Volumes/RAID0/Media/.runtime; unrelated to this diff.Review
Standards
No findings.
Spec
No findings. Gallery pagination now requires explicit cumulative metadata before early exhaustion; formatter cleanup is anchored to a complete wrapper; prompt validation compares output occurrences against source speech.
Drift
#108 — Activate current-semester recording coverage: 10 of 10 criteria delivered, nothing beyond the brief.
Assisted-by: GPT-5
Co-authored-by: Codex noreply@openai.com