TidyLinker.com
TypeScript
Pull Request Merged
PR #149 merged: fix: register Horizon supervisors on staging
Summary
- Surfaced during PR #146 smoke testing on dev.tidylinker.com: queued jobs (
DirectBookingRequestNotification,LogUserAuthActivity,NewSupportRequestNotification) all sat in Redis forever - Root cause:
config/horizon.phponly definedproductionandlocalenvironments, but staging runsAPP_ENV=staging, so Horizon spawned zero supervisors - Adds a
stagingenvironment block (3 max processes — modest, matches the lighter staging traffic) - Adds a Pest test asserting every env we actually run in has a supervisor block, plus a sanity check that
horizon:listruns cleanly underAPP_ENV=staging
Production was unaffected by the original bug.
Test plan
-
php artisan test tests/Feature/Config/HorizonEnvironmentsTest.php→ 4 passed - Pint clean
- After merge + redeploy of dev.tidylinker.com: confirm
php artisan horizon:listshows a running supervisor, and queued notifications drain automatically (previously had to drain manually withqueue:work --once)
+17
additions
-0
deletions
2
files changed