Skip to content

fix: harden silence processing, API, packaging, and tests - #15

Merged
BTawaifi merged 1 commit into
mainfrom
review/full-hardening-2026-08-11
Aug 11, 2026
Merged

BTawaifi merged 1 commit into
mainfrom
review/full-hardening-2026-08-11

Conversation

@BTawaifi

Copy link
Copy Markdown
Owner

Summary

  • compute silent ranges as the full complement of normalized non-silent ranges
  • preserve trailing silence and fully silent files
  • return a structured processing result and let Python callers receive failures
  • make CLI failures return non-zero status
  • derive collision-resistant output names from the input and support configurable output formats/stems
  • modernize package build metadata, Python support declaration, and package exports
  • exclude tests from built distributions
  • replace mock-only coverage with 26 regression tests including real pydub tone/silence detection
  • expand CI across Python 3.10–3.13 and verify built wheel contents/API/CLI

This PR is intentionally broad because it closes the correctness and verification gaps found in a full code review.

@BTawaifi
BTawaifi force-pushed the review/full-hardening-2026-08-11 branch from 53b5449 to d35735a Compare August 11, 2026 02:55
@BTawaifi
BTawaifi merged commit 62e35b2 into main Aug 11, 2026
5 checks passed
@BTawaifi
BTawaifi deleted the review/full-hardening-2026-08-11 branch August 11, 2026 03:06
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