Skip to content

decision(speech-to-text): transcoded .wav lands in a different Storage location #3140

Description

@cabljac

The extension derived the transcoded .wav Storage destination from the local /tmp path. The kit derives it from the object name and normalises the prefix separator. Migrated users find their transcoded audio in a different location.

Extension:

storagePath: config.outputStoragePath
  ? `${config.outputStoragePath}${transcodeResult.outputPath}` // "transcriptions" + "/tmp/a.mp3.wav"
  : transcodeResult.outputPath.slice(1);                       // "tmp/a.mp3.wav"

Kit:

const transcodedObjectName = `${filePath}.wav`;
storagePath: config.outputStoragePath
  ? `${config.outputStoragePath.replace(/\/$/, "")}/${transcodedObjectName}`
  : transcodedObjectName;

For an input object named a.mp3:

OUTPUT_STORAGE_PATH Extension writes Kit writes
unset tmp/a.mp3.wav a.mp3.wav
transcriptions transcriptions/tmp/a.mp3.wav transcriptions/a.mp3.wav
transcriptions/ transcriptions//tmp/a.mp3.wav transcriptions/a.mp3.wav

The .txt transcript path had a related remnant (replace("tmp/", "")), fixed in #3073 (#3026). The .wav location itself is still open.

Decide one of:

  • Parity: reproduce the extension's tmp/-prefixed paths, double slash included, so existing consumers keep finding files where they were.
  • Keep the kit's path: the extension's tmp/ segment is an artefact of the local download path, not a design. Document the change and the old-vs-new matrix in the kit README and the Notes entries (docs(kits): write the Notes entries for accepted divergences #3035).

Parity ledger: #2974, speech-to-text §1.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions