Skip to content

refactor: construct ScheduleStore internally in Scheduler#142

Merged
alecthomas merged 1 commit intomainfrom
aat/scheduler-store-internal
Feb 22, 2026
Merged

refactor: construct ScheduleStore internally in Scheduler#142
alecthomas merged 1 commit intomainfrom
aat/scheduler-store-internal

Conversation

@alecthomas
Copy link
Copy Markdown
Collaborator

Move store construction from external injection into the Scheduler
constructor. Add SchedulerDB config field with default
${CACHEW_STATE}/scheduler.db. Extract jobKey() helper to unify
key format across logging and store operations.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

Move store construction from external injection into the Scheduler
constructor. Add SchedulerDB config field with default
${CACHEW_STATE}/scheduler.db. Extract jobKey() helper to unify
key format across logging and store operations.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@alecthomas alecthomas requested a review from a team as a code owner February 22, 2026 23:21
@alecthomas alecthomas requested review from js-murph and removed request for a team February 22, 2026 23:21
@alecthomas alecthomas enabled auto-merge (squash) February 22, 2026 23:21
@alecthomas alecthomas merged commit 2df6683 into main Feb 22, 2026
5 checks passed
@alecthomas alecthomas deleted the aat/scheduler-store-internal branch February 22, 2026 23:22
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