fix: -o/--output now disables speaker playback by default

Previously, using -o to save audio to a file would ALSO play through
speakers (since --play defaults to true). This was unexpected - if
you're saving to file, you probably don't want it blasting through
speakers.

Now -o implies --play=false unless --play is explicitly provided.
To get the old behavior (save AND play), use: sag -o file.mp3 --play

Author: steipete

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+### Fixed
+- `-o/--output` now disables speaker playback by default unless `--play` is explicitly set. Previously `-o` saved to file AND played through speakers, which was confusing.
+
 ## 0.2.0 - 2025-12-19
 ### Added
 - Voice control flags: `--stability`, `--similarity`/`--similarity-boost`, `--style`, `--speaker-boost`/`--no-speaker-boost`.

cmd/speak.go

@@ -101,6 +101,10 @@ func init() {
 				if inferred := inferFormatFromExt(opts.outputPath); inferred != "" {
 					opts.outputFmt = inferred
 				}
+				// Disable playback when -o is set, unless --play was explicitly provided
+				if !cmd.Flags().Changed("play") {
+					opts.play = false
+				}
 			}
 
 			ctx, cancel := context.WithTimeout(cmd.Context(), 90*time.Second)
@@ -137,7 +141,7 @@ func init() {
 	cmd.Flags().StringVar(&opts.voiceID, "voice-id", "", "Voice ID to use (ELEVENLABS_VOICE_ID)")
 	cmd.Flags().StringVarP(&opts.voiceID, "voice", "v", opts.voiceID, "Alias for --voice-id; accepts name or ID; use '?' to list voices")
 	cmd.Flags().StringVar(&opts.modelID, "model-id", opts.modelID, "Model ID (default: eleven_v3). Common: eleven_multilingual_v2 (stable), eleven_flash_v2_5 (fast/cheap), eleven_turbo_v2_5 (balanced).")
-	cmd.Flags().StringVarP(&opts.outputPath, "output", "o", "", "Write audio to this file (in addition to playback)")
+	cmd.Flags().StringVarP(&opts.outputPath, "output", "o", "", "Write audio to file (disables playback unless --play is also set)")
 	cmd.Flags().StringVar(&opts.outputFmt, "format", opts.outputFmt, "Output format (e.g. mp3_44100_128)")
 	cmd.Flags().BoolVar(&opts.stream, "stream", opts.stream, "Stream audio while generating")
 	cmd.Flags().BoolVar(&opts.play, "play", opts.play, "Play audio through speakers")