Skip to content

672 from dev now#810

Merged
TimRl merged 19 commits intodevfrom
672-from-dev-now
Mar 31, 2026
Merged

672 from dev now#810
TimRl merged 19 commits intodevfrom
672-from-dev-now

Conversation

@Luke-Bilhorn
Copy link
Copy Markdown
Contributor

@Luke-Bilhorn Luke-Bilhorn commented Mar 27, 2026

Note

Medium Risk
Changes export output paths and introduces optional zipping and combined text+audio exports, which could break downstream expectations about folder layout. Adds a new archiving dependency and deletes the unzipped export directory after creating the zip, so failures could impact user data if paths are wrong.

Overview
Export behavior is reworked to produce a dedicated wrapper output folder per run (name includes project, format/multi, and date, with collision-avoidance), and exports now write into subfolders when combining text + audio.

Adds new ExportOptions (zipOutput, includeAudio, includeTimestamps) and updates the command wiring to use the shared type. When zipOutput is enabled, the wrapper folder is zipped via new zipDirectory() (archiver-based) and the unzipped folder is removed.

Audio export no longer creates its own export/<project>-<timestamp> tree; it now writes directly to the provided destination folder, aligning with the new wrapper-folder approach.

The Project Export webview is updated to remember the last export folder, add a Zip output toggle, add explicit audio-mode choices (with/without timestamps), add a USFM no-validation variant, and reorganize the format selection UI.

Written by Cursor Bugbot for commit 2c48c28. This will update automatically on new commits. Configure here.

…ted to include audiofiles."

This reverts commit 3038f56.
Now, when downloading audio with text, it has type "multi" instead of "HTML" or "plaintext" or "wav", and it has "audio" and "html" folders inside, for example.

Now, downloads are always wrapped in a new folder but the option to zip remains.
…n, making it its own button.

I just generaly sat down and cleaned up the export options page and gave it a unified, consistent structure and spacing that should hopefully be less confusing to the user.
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Luke-Bilhorn and others added 5 commits March 29, 2026 23:50
…ieval

- Added `initializeAudioExporter` function to set the extension context for audio exporting.
- Updated the FFmpeg path retrieval in the audio export process to utilize the context, enhancing the functionality of the audio exporter.
- Implemented syncing of sourceLanguage and targetLanguage from metadata to configuration, ensuring updates only occur when necessary.
- Improved type definitions for languages in ProjectMetadata to utilize LanguageMetadata, enhancing clarity and type safety.
@TimRl TimRl self-requested a review March 30, 2026 19:40
TimRl
TimRl previously approved these changes Mar 30, 2026
@TimRl TimRl requested a review from LeviXIII March 31, 2026 16:35
@TimRl TimRl merged commit 62d0c32 into dev Mar 31, 2026
3 checks passed
@TimRl TimRl deleted the 672-from-dev-now branch March 31, 2026 16:44
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.

3 participants