Skip to content

Fix startup auto-route race#32

Merged
bhack merged 1 commit into
mainfrom
fix-startup-auto-route-race
May 23, 2026
Merged

Fix startup auto-route race#32
bhack merged 1 commit into
mainfrom
fix-startup-auto-route-race

Conversation

@bhack
Copy link
Copy Markdown
Owner

@bhack bhack commented May 23, 2026

Summary

  • keep hidden --background --auto-route startup alive briefly when the requested PipeWire output is not available yet
  • retry controller startup until the sink appears, then route system audio
  • close partially initialized PipeWire backends when controller construction fails
  • add a nested live UI startup-race simulation for the login race

Root Cause

At login, the Flatpak autostart command could run before WirePlumber had restored or exposed the requested output sink. Mini EQ treated that missing sink as a fatal startup error, so the background process exited before it could enable system-wide EQ.

Validation

  • .venv/bin/python -m ruff check .
  • .venv/bin/python -m ruff format --check .
  • .venv/bin/python -m pytest tests/test_mini_eq_app.py tests/test_mini_eq_routing.py tests/test_check_live_ui_runtime.py -q
  • .venv/bin/python tools/check_live_ui_runtime.py --timeout 35 --cycles 1 --audio-duration 45 --retries 0 --simulate-startup-routing-race

@bhack bhack force-pushed the fix-startup-auto-route-race branch 2 times, most recently from 4676465 to 02a6c87 Compare May 23, 2026 15:33
@bhack bhack force-pushed the fix-startup-auto-route-race branch from 02a6c87 to db5352a Compare May 23, 2026 16:35
@bhack bhack linked an issue May 23, 2026 that may be closed by this pull request
2 tasks
@bhack bhack marked this pull request as ready for review May 23, 2026 18:45
@bhack bhack merged commit 7a9bd9c into main May 23, 2026
18 of 23 checks passed
@bhack bhack deleted the fix-startup-auto-route-race branch May 23, 2026 18:45
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.

[Bug]: System-wide EQ not applied at startup

1 participant