Skip to content

fix(scheduler): the timer run stands aside when a scan will trigger the steps - #54

Merged
PiratesIRC merged 1 commit into
mainfrom
fix/timer-stands-aside-for-trigger
Sep 14, 2026
Merged

PiratesIRC merged 1 commit into
mainfrom
fix/timer-stands-aside-for-trigger

Conversation

@PiratesIRC

Copy link
Copy Markdown
Owner

Measured on the first morning the IPTV Checker trigger fired (2026-09-14): the 05:05 run, which had been waiting for that scan under Wait for IPTV Checker Completion, saw the scan end and sorted the same 256 channels one second after the triggered run, in a different worker, while that run was still writing. No damage measured, but two delete-and-recreate passes over the same rows at once is not a state to leave open.

  • The timer path now checks, before it waits, whether a scan is running and Run After IPTV Checker Scan is on; if both, it logs that it is standing aside and returns skipped.
  • _iptv_checker_is_running fails open to False on a missing or unreadable progress file, so an installation without IPTV Checker keeps its schedule exactly as before.
  • The triggered path never consults the check. The scheduled-run timestamp is not written for a run that stood aside.
  • Six new tests in tests/test_iptv_checker_trigger.py; suite 1534 passed.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WJpo1EzMNmJiSiNpbwBi6G

…he steps

Measured 2026-09-14: with Run After IPTV Checker Scan on, the 05:05 run that
had been waiting for the checker and the run the checker then triggered sorted
the same channels concurrently, one second apart, in two workers. The timer
path now checks, before it waits, whether a scan is running and the trigger
setting is on; if so it logs that it is standing aside and returns "skipped".
A missing or unreadable progress file counts as not running, so the schedule
is unchanged where IPTV Checker is absent. The scheduled-run timestamp is not
written for a run that stood aside.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WJpo1EzMNmJiSiNpbwBi6G
@PiratesIRC
PiratesIRC merged commit da282a0 into main Sep 14, 2026
6 checks passed
@PiratesIRC
PiratesIRC deleted the fix/timer-stands-aside-for-trigger branch September 14, 2026 11:55
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