Skip to content

Add fixture-backed transcription router media plan#54

Merged
WalksWithASwagger merged 1 commit into
mainfrom
codex/issue-50-router-media-normalization
May 22, 2026
Merged

Add fixture-backed transcription router media plan#54
WalksWithASwagger merged 1 commit into
mainfrom
codex/issue-50-router-media-normalization

Conversation

@WalksWithASwagger
Copy link
Copy Markdown
Owner

Summary

  • Add opt-in ffprobe media inspection and planned-only FFmpeg normalization metadata.
  • Expose router output contracts plus privacy and cost receipts without changing runtime transcription defaults.
  • Add fixture-backed coverage for cloud, local/private, chunked, timestamped, diarization-capable, video, and no-default-probe paths.

Verification

  • PYTHON=/Users/kk/Code/spektorAI/venv/bin/python make lint
  • PYTHON=/Users/kk/Code/spektorAI/venv/bin/python make test -> 288 passed
  • PYTHON=/Users/kk/Code/spektorAI/venv/bin/python /Users/kk/Code/spektorAI/venv/bin/python -m pytest tests/test_audio.py -q -> 19 passed
  • git diff --check

Closes #50

@github-actions
Copy link
Copy Markdown

Agentic PR Review

Verdict: needs-human

Checks:

  • Linked issue via Closes #...: yes
  • Issue acceptance criteria found: 4
  • PR verification section present: yes
  • PR acceptance self-check present: no
  • PR self-check covers issue acceptance: no
  • Diff within v1 limits: no
  • Missing acceptance coverage:
    • Large audio/video inputs can be inspected and normalized through an FFmpeg-backed plan before transcription.
    • Router fixtures cover local/private, cloud, chunked, timestamped, and diarization-capable paths.
    • Runtime defaults remain unchanged until the router has explicit privacy and cost receipts.
    • Docs connect the provider matrix to the implemented router behavior.

This PR is not blocked from human review, but the agentic gate is not satisfied.

@github-actions github-actions Bot added the needs-human Agentic automation stopped for human judgment. label May 22, 2026
@WalksWithASwagger WalksWithASwagger merged commit 9792bd4 into main May 22, 2026
1 check passed
@WalksWithASwagger WalksWithASwagger deleted the codex/issue-50-router-media-normalization branch May 22, 2026 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-human Agentic automation stopped for human judgment.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add fixture-backed media normalization for the transcription router

1 participant