Skip to content

Fix macOS voice meter source assertions#546

Merged
basnijholt merged 1 commit into
mainfrom
codex/fix-main-ci
May 29, 2026
Merged

Fix macOS voice meter source assertions#546
basnijholt merged 1 commit into
mainfrom
codex/fix-main-ci

Conversation

@basnijholt
Copy link
Copy Markdown
Owner

Summary

  • Update macOS voice meter source-shape assertions for the AVAudioEngine spectrum implementation
  • Keep the no-audio-file guarantee by asserting AVAudioRecorder is absent

Test Plan

  • uv run pytest tests/test_macos_app.py::test_macos_app_voice_level_meter_uses_live_audio_engine_without_saving_audio tests/test_macos_app.py::test_macos_app_voice_level_meter_smooths_fast_meter_changes -q
  • uv run --all-extras pytest
  • pre-commit run --all-files
  • swift test --package-path macos/AgentCLI --enable-xctest (local build completed; XCTest unavailable with Command Line Tools xcrun PlatformPath error)

@basnijholt basnijholt merged commit 9c5892b into main May 29, 2026
9 of 10 checks passed
@basnijholt basnijholt deleted the codex/fix-main-ci branch May 29, 2026 05:35
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.

1 participant