Skip to content

Add Tests and Publish workflows#2

Merged
AzisK merged 3 commits intomainfrom
Add-Tests-and-Publish-workflows
Feb 9, 2026
Merged

Add Tests and Publish workflows#2
AzisK merged 3 commits intomainfrom
Add-Tests-and-Publish-workflows

Conversation

@AzisK
Copy link
Owner

@AzisK AzisK commented Feb 9, 2026

No description provided.

@AzisK AzisK force-pushed the Add-Tests-and-Publish-workflows branch from 7976e5c to 76c94eb Compare February 9, 2026 23:01
@AzisK AzisK force-pushed the Add-Tests-and-Publish-workflows branch from 76c94eb to 31a29e7 Compare February 9, 2026 23:05
@AzisK AzisK merged commit e3b50c5 into main Feb 9, 2026
3 checks passed
@AzisK AzisK deleted the Add-Tests-and-Publish-workflows branch February 15, 2026 16:57
AzisK added a commit that referenced this pull request Feb 15, 2026
* refactor: abstract platform-specific audio playback and clipboard commands

- Add _default_play_cmd(): detects afplay (macOS), paplay/aplay/ffplay (Linux)
- Add _default_clipboard_cmd(): detects pbpaste (macOS), wl-paste/xclip/xsel (Linux)
- Inject 'run' dependency into get_text() for clipboard path (was calling
  subprocess.run directly, bypassing the DI pattern used everywhere else)
- Update speak_text() to use _default_play_cmd() instead of hardcoded 'afplay'
- Add 14 new tests for platform detection and clipboard run injection
- All 44 tests pass

* docs: update README and CHANGELOG for macOS + Linux platform support

* docs: add no co-authored-by rule to AGENTS.md

* fix: monkeypatch platform in tests that depend on play/clipboard commands

* Fix monkeypatching
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