fix(cron): record interrupted startup runs#71547
Conversation
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Greptile SummaryThis PR changes gateway restart behavior so that cron jobs found mid-run at startup are recorded as failed ( Confidence Score: 4/5Safe to merge; logic is well-tested and no critical defects found. No P0 or P1 issues found. The interrupted-run marking, skip-in-runMissedJobs, emit-inside-lock, and flat-schedule-key recovery all follow established patterns in the codebase and are covered by the updated test suite. No files require special attention. Reviews (1): Last reviewed commit: "fix(cron): record interrupted startup ru..." | Re-trigger Greptile |
Summary
runningAtMson gateway startup as interrupted failed runs instead of replaying or silently clearing themcron,tz,staggerMs, aliases) in the cron tool before gateway validationFixes #59056
Fixes #61343
Fixes #63657
Fixes #59301
Supersedes #57640
Validation
pnpm test:serial src/cron/service.restart-catchup.test.ts src/agents/tools/cron-tool.flat-params.test.ts src/agents/tools/cron-tool.test.tspassed before the clean rebaseOPENCLAW_LOCAL_CHECK_MODE=throttled pnpm check:changedattempted twice; both attempts timed out waiting for unrelated local heavy-check locks beforetsgo:corecould start