Repository navigation
Correct transcription language guidance - #17
Merged
Merged
Conversation
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.
Why
The transcription skill told agents to use
--target-languagefor translation, but this flag only hints at the spoken language for Replicate. Following that advice can silently produce an untranslated transcript.Document full lowercase English language names such as
frenchfor Replicate, BCP-47 codes such asfr-FRwith--source-languagefor AWS/GCP, theen-USdefault for AWS/GCP, and automatic detection when the Replicate hint is omitted. For translation, transcribe first and process the transcript with/text/translate.The companion SDK help and regression tests are in node-sdk #562 — Clarify the transcription language hint.
Validation
corepack yarn checkpasses. TypeScript lint and Biome were also rerun on tracked files to exclude the pre-existing untracked VS Code workspace; its original checksum is preserved.en-USdefault. Round two found no issues.3253426, including both integration E2E jobs; latest main is included.Closes transloadit/node-sdk#561.
🤖 Claimed by kvz on
studio1/node-sdk, studio1/skillssince 2026-10-15 · resume:codex resume 01a13cd4-bc97-7ca1-869d-37e2ee01d908