Skip to content

fix: mark manual cron runs active#78245

Closed
bryce-d-greybeard wants to merge 1 commit into
openclaw:mainfrom
bryce-d-greybeard:bryce/issue-78233-cron-manual-active
Closed

fix: mark manual cron runs active#78245
bryce-d-greybeard wants to merge 1 commit into
openclaw:mainfrom
bryce-d-greybeard:bryce/issue-78233-cron-manual-active

Conversation

@bryce-d-greybeard
Copy link
Copy Markdown
Contributor

Closes #78233

Summary

  • mark manual cron run executions active in the process-local cron runtime tracker after the durable running marker is persisted
  • clear the active marker in finishPreparedManualRun with finally, including missing-job or persist-failure exits
  • add a regression test covering manual-run active tracking while the isolated job is still running

Verification

  • PATH="/tank/development/linus/openclaw/node_modules/.bin:/tmp/openclaw-pnpm-shim:$PATH" pnpm format:check src/cron/service/ops.ts src/cron/service/ops.regression.test.ts
  • git diff --check
  • PATH="/tank/development/linus/openclaw/node_modules/.bin:/tmp/openclaw-pnpm-shim:$PATH" node scripts/check-changed.mjs ⚠️ blocked by unrelated existing core typecheck errors, primarily missing @openclaw/fs-safe/* packages plus strictness diagnostics outside this patch
  • targeted vitest for src/cron/service/ops.regression.test.ts -t "tracks manual cron.run as an active cron job" ⚠️ attempted with parent node_modules; Vitest produced no test output and was killed by the 120s timeout before running/reporting tests in this worktree

@openclaw-barnacle openclaw-barnacle Bot added size: S r: too-many-prs Auto-close: author has more than twenty active PRs. triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup. labels May 6, 2026
@openclaw-barnacle
Copy link
Copy Markdown

Closing this PR because the author has more than 20 active PRs in this repo. Please reduce the active PR queue and reopen or resubmit once it is back under the limit. You can close your own PRs to get back under the limit.

@openclaw-barnacle openclaw-barnacle Bot closed this May 6, 2026
@openclaw-barnacle
Copy link
Copy Markdown

Closing this PR because the author has more than 20 active PRs in this repo. Please reduce the active PR queue and reopen or resubmit once it is back under the limit. You can close your own PRs to get back under the limit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

r: too-many-prs Auto-close: author has more than twenty active PRs. size: S triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cron: transient 'lost' marker on long-running manual runs before sweeper recovery

1 participant