Skip to content

fix: add Linux sample paths, expand CI to macOS + Windows (Phase 4)#23

Merged
geoffmcc merged 3 commits into
mainfrom
fix/phase4-platform-parity
Jul 19, 2026
Merged

fix: add Linux sample paths, expand CI to macOS + Windows (Phase 4)#23
geoffmcc merged 3 commits into
mainfrom
fix/phase4-platform-parity

Conversation

@geoffmcc

Copy link
Copy Markdown
Owner

Summary

Adds platform parity: Linux sample search support, cross-platform CI testing, and correct macOS build matrix.

Changes

  • src/alscan/search.py: Added _linux_sample_dirs() function with ~/.ableton/ paths, refactored platform detection into dedicated _windows_sample_dirs() / _macos_sample_dirs() / _linux_sample_dirs() functions
  • .github/workflows/ci.yml:
    • Test matrix: added macos-latest and windows-latest (single Python 3.12 each to control CI cost)
    • Build matrix: added macos-15-intel for x86_64 macOS builds
    • Replaced deprecated macos-14 with macos-15 for ARM64 builds

Audit Findings Fixed

  • F12: Added Linux support to known_sample_dirs()
  • F13: Expanded CI to test on macOS and Windows ✅
  • F15: Build matrix now produces all 4 platform binaries (Linux x86_64, macOS ARM64, macOS x86_64, Windows x86_64) ✅

Build Matrix (4 binaries)

Binary Runner
alscan-x86_64-linux.tar.gz ubuntu-latest
alscan-aarch64-macos.tar.gz macos-15
alscan-x86_64-macos.tar.gz macos-15-intel
alscan-x86_64-windows.zip windows-latest

Testing

  • 765 tests passed
  • 2 pre-existing failures (unrelated script-as-module tests)

Phase 4 of 6

This is Phase 4 of the audit remediation roadmap.

geoffmcc added 3 commits July 18, 2026 22:49
…dit remediation)

- Add Linux support to known_sample_dirs() in search.py
- Add macos-latest and windows-latest to CI test matrix
- Add macos-15-intel to build matrix for x86_64 macOS
- Replace deprecated macos-14 with macos-15 for ARM64 builds

Fixes F12, F13, F15 from audit intake.
- Add macos-15-intel to test matrix
- Test Python 3.14 (not 3.12) on macOS and Windows
- Keep full Python range (3.12, 3.13, 3.14) on Linux
@geoffmcc
geoffmcc merged commit f36974d into main Jul 19, 2026
8 checks passed
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