Skip to content

Fix: audio export path resolution#40

Merged
jamiepine merged 3 commits intojamiepine:mainfrom
Spyabo:fix/audio-export-path-resolution
Feb 2, 2026
Merged

Fix: audio export path resolution#40
jamiepine merged 3 commits intojamiepine:mainfrom
Spyabo:fix/audio-export-path-resolution

Conversation

@Spyabo
Copy link
Contributor

@Spyabo Spyabo commented Feb 2, 2026

Summary

  • Fixed audio export failing when Tauri save dialog returns object instead of string path
  • Added validation to prevent writing with invalid paths
  • Changed from writeBinaryFile to writeFile API

Test plan

  • Built Windows installer
  • Tested audio export from both Generate and Story tabs
  • Verified files save correctly to chosen location

Note

Needs macOS testing - Please verify this doesn't break file saving on macOS before merging.

Copy link
Owner

@jamiepine jamiepine left a comment

Choose a reason for hiding this comment

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

works on macOS, LGTM!

@jamiepine jamiepine merged commit 8197f07 into jamiepine:main Feb 2, 2026
@jamiepine
Copy link
Owner

sorry if I merged that too soon haha

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.

2 participants