Default to capturing mic + all system audio. - #39
Open
aaugustin wants to merge 1 commit into
Open
Conversation
Most users record meetings they're participating in, so mic=true and capture_mode="all" out of the box avoids missing your own audio and skips the source picker prompt. --no-mic allows opting-out. With --no-mic, --mic-device is rejected, as there's no sensible behavior for this combination. Fix paberr#36.
There was a problem hiding this comment.
Pull request overview
Updates ownscribe’s out-of-the-box audio capture behavior to better match the primary “record a meeting you’re in” use case by enabling microphone capture and recording all system audio without prompting by default, while adding an explicit CLI opt-out.
Changes:
- Change default audio config to
mic = trueandcapture_mode = "all"(avoids missing the user’s own voice and avoids the source picker prompt). - Update CLI to support
--mic/--no-micand to reject the--no-mic+--mic-devicecombination. - Refresh tests and README examples to reflect the new defaults and flags.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/test_pipeline.py | Updates recorder-construction expectations to match new default mic + capture mode. |
| tests/test_config.py | Adjusts default-config assertions for mic and capture_mode. |
| tests/test_cli.py | Updates CLI parsing tests for --no-mic and adds coverage for invalid flag combinations. |
| src/ownscribe/config.py | Changes default audio settings in both TOML template and dataclass defaults. |
| src/ownscribe/cli.py | Implements --mic/--no-mic tri-state override behavior and validates incompatible combinations. |
| README.md | Updates usage examples and config snippet to match the new default capture behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+101
to
107
| if mic is False and mic_device: | ||
| raise click.UsageError("--no-mic and --mic-device cannot be used together.") | ||
| if mic is not None: | ||
| config.audio.mic = mic | ||
| if mic_device: | ||
| config.audio.mic = True | ||
| config.audio.mic_device = mic_device |
| 1. Capture system audio and your microphone until you press Ctrl+C (or auto-stop after 5 minutes of silence) | ||
| 2. Transcribe with WhisperX | ||
| 3. Summarize with your local LLM | ||
| 4. Save everything to `~/ownscribe/YYYY-MM-DD_HHMMSS/` |
Comment on lines
+33
to
+39
| def test_no_mic_flag(self): | ||
| runner = CliRunner() | ||
| with _mock_config(), mock.patch("ownscribe.pipeline.run_pipeline") as mock_run: | ||
| result = runner.invoke(cli, ["--mic"]) | ||
| result = runner.invoke(cli, ["--no-mic"]) | ||
| assert result.exit_code == 0 | ||
| config = mock_run.call_args[0][0] | ||
| assert config.audio.mic is True | ||
| assert config.audio.mic is False |
paberr
requested changes
Jul 21, 2026
paberr
left a comment
paberr
left a comment
Owner
There was a problem hiding this comment.
Thanks a lot for the work!
Besides the Copilot & my comments, I think the README needs some updating:
- line 45 still mentions the old default and
--mic - I'm not sure if the permissions described in 68 are still correct or if there's a microphone prompt too?
- The mute hint "press 'm'" is on the
--no-micexample where it doesn't actually apply.
Comment on lines
+101
to
+104
| if mic is False and mic_device: | ||
| raise click.UsageError("--no-mic and --mic-device cannot be used together.") | ||
| if mic is not None: | ||
| config.audio.mic = mic |
Owner
There was a problem hiding this comment.
Blocking: I think --no-mic doesn't actually disable the mic when mic_device is set in config.toml:
--no-micsetsconfig.audio.mic = False, butconfig.audio.mic_devicekeeps its value from the config file.CoreAudioRecorder.start()re-enables the mic from the device name alone (if self._mic or self._mic_device: cmd.append("--mic")).- So the helper is still launched with
--mic --mic-device "USB Mic"and we record the mic anyway.
Possible fix: clear config.audio.mic_device when --no-mic is given, e.g.
if mic is not None:
config.audio.mic = mic
if mic is False:
config.audio.mic_device = ""(same as CoPilot's comment below I just noticed)
Comment on lines
+33
to
+39
| def test_no_mic_flag(self): | ||
| runner = CliRunner() | ||
| with _mock_config(), mock.patch("ownscribe.pipeline.run_pipeline") as mock_run: | ||
| result = runner.invoke(cli, ["--mic"]) | ||
| result = runner.invoke(cli, ["--no-mic"]) | ||
| assert result.exit_code == 0 | ||
| config = mock_run.call_args[0][0] | ||
| assert config.audio.mic is True | ||
| assert config.audio.mic is False |
Owner
There was a problem hiding this comment.
Two small things on the tests here, non-blocking:
- With
test_mic_flagrenamed, we no longer test that an explicit--micoverridesmic = falsefrom the config file. Maybe keep a--mictest next to the--no-micone? - There's also no test that
--mic-devicealone impliesmic = True— that implication now sits on its own line incli.pyand could regress silently.
Once the mic_device interaction above is fixed, a test for config-set mic_device + --no-mic would be a nice-to-have too.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Most users record meetings they're participating in, so mic=true and
capture_mode="all" out of the box avoids missing your own audio and
skips the source picker prompt.
--no-mic allows opting-out. With --no-mic, --mic-device is rejected,
as there's no sensible behavior for this combination.
Fix #36.