Skip to content

fix(speech-to-text): correct output storage path#1086

Open
IzaakGough wants to merge 1 commit into
mainfrom
@invertase/fix-speech-to-text-output-storage-path
Open

fix(speech-to-text): correct output storage path#1086
IzaakGough wants to merge 1 commit into
mainfrom
@invertase/fix-speech-to-text-output-storage-path

Conversation

@IzaakGough

@IzaakGough IzaakGough commented May 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes #754

  • derive the transcoded upload destination from the source object name instead of the local tmp path
  • normalize OUTPUT_STORAGE_PATH before constructing the destination key
  • add regression tests covering nested input paths and matching output prefixes

Testing

npm test -- --runInBand util.test.ts
npm run build

Manual verification

  • deployed the extension and uploaded myDirectory/test-audio.m4a
  • verified outputs were created under OUTPUT_STORAGE_PATH/ without nested source folders or tmp path leakage

@IzaakGough

Copy link
Copy Markdown
Collaborator Author

Note that failing checks are unrelated to this PR

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[speech-to-text] extension creates unwanted nested directory

2 participants