feat(audio): expose standalone transcription API - #4391
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Deterministic PR hygiene checks passed. |
✅ READY
Hygiene✅ Deterministic PR hygiene checks passed. |
리뷰 · 우선순위 58 / 80이 PR은 draft인 현재 원격 CI는 이 시각 기준 test 1/4·3/4·4/4가 실패했고, gates/hygiene/docker smoke 등은 src/server/audio-transcriptions.ts (upstream.ok ? 502 : status) - 업스트림이 성공인데 응답 본문 상한에 걸려 early return하면 finally의 recordOutcome이 잠깐 심어 둔 502로 남을 수 있다. 의도된 실패 기록이어도, 성공 HTTP를 먼저 넣고 실패 분기에서만 덮는 편이 읽기 쉽다. 메인테이너의 판단이 필요한 지점
너의 추천 이 댓글은 grok-bot이 작성했습니다 |
Summary
POST /v1/audio/transcriptionsfor external clients using an OpenCodex API key. ChatGPT Pool/Direct and the OpenAI API provider retain their separate upstream credentials; file uploads return JSON or plain transcript text.dev; feat(audio): expose streaming dictation and owned live calls #4392 targets this branch, followed by feat(dashboard): add dictation and live voice API controls #4395. Leave the chain open for review; no native GitHub stack registration.Verification
bun test tests/server/audio-transcriptions.test.ts: 24 pass, 0 fail.bun run test:changed: 8,138 pass, 2 skip, 0 fail across 326 selected files; two additional probe-release regressions subsequently passed in the focused run.bun test tests/test-layout.test.ts tests/test-layout-tooling.test.ts: 17 pass, 0 fail.--no-verifyand remote CI only.011f2dff5cin feat(audio): expose streaming dictation and owned live calls #4392, based on this PR's71e22d967f, passed all 24 transcription cases and remote gates in run 34687731903. Unrelated journal-restore assertions remain red and are recorded separately per owner scope; this PR stays draft, without a whole-CI-green claim.71e22d967ffafbd3492935c299623c9127eaf17balso passed all 24 transcription cases, endpoint cases and gates (1,979 dashboard tests) in run 34687369123. The screenshot/description gate is now successful after adding the artifact capture below.Checklist
Actual screenshot of this layer's own CI artifact (run 34687369123, merge d9771b3; GUI tree exactly matches
71e22d967f). This layer changes dashboard test fixtures and asset provenance only; audio controls belong to #4395. Synthetic management data only.