Skip to content

feat: add RHIZA_SYNC_SCHEDULE variable to override default sync cron#955

Open
Copilot wants to merge 3 commits intomainfrom
copilot/enable-user-sync-overrides
Open

feat: add RHIZA_SYNC_SCHEDULE variable to override default sync cron#955
Copilot wants to merge 3 commits intomainfrom
copilot/enable-user-sync-overrides

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 30, 2026

Summary

Adds RHIZA_SYNC_SCHEDULE Makefile variable so users can override the default weekly sync cron (0 0 * * 1) without their change being clobbered on every make sync.

Changes

  • Add RHIZA_SYNC_SCHEDULE ?= 0 0 * * 1 to .rhiza/rhiza.mk
  • Add _apply-sync-schedule target that patches .github/workflows/rhiza_sync.yml via sed after uvx rhiza sync . completes but before post-sync hooks
  • Use portable sed -i.bak && rm .bak for macOS/Linux compatibility
  • Document the override in docs/CUSTOMIZATION.md with cron examples
  • Add 8 tests covering default value, env/Makefile override, patching, skip-when-default, missing-file, and integration with sync target

Usage — set before the include line in your root Makefile:

RHIZA_SYNC_SCHEDULE = 0 9 * * 1-5   # weekdays at 9 AM UTC

include .rhiza/rhiza.mk

GitLab CI schedules are configured in the UI, so this variable only affects GitHub Actions.

Testing

  • make test passes locally
  • make fmt has been run
  • New tests added (or explain why not needed)

Checklist

  • Commit messages follow the Conventional Commits format
  • Documentation updated if behaviour changed
  • make deptry passes (no unused or missing dependencies)

Copilot AI and others added 2 commits March 30, 2026 13:03
…SYNC_SCHEDULE

Agent-Logs-Url: https://github.com/Jebel-Quant/rhiza/sessions/f54e4610-8776-40f5-8deb-40060b706f79

Co-authored-by: markrichardson <5681211+markrichardson@users.noreply.github.com>
… realism

Agent-Logs-Url: https://github.com/Jebel-Quant/rhiza/sessions/f54e4610-8776-40f5-8deb-40060b706f79

Co-authored-by: markrichardson <5681211+markrichardson@users.noreply.github.com>
Copilot AI changed the title [WIP] Add mechanism for users to override default rhiza sync times feat: add RHIZA_SYNC_SCHEDULE variable to override default sync cron Mar 30, 2026
Copilot AI requested a review from markrichardson March 30, 2026 13:07
@markrichardson markrichardson marked this pull request as ready for review March 30, 2026 13:36
@HarryCampion
Copy link
Copy Markdown
Collaborator

This is neat @markrichardson. Is it overridden via an env var? Or does user need to override the rhiza.mk?

@HarryCampion
Copy link
Copy Markdown
Collaborator

This is neat @markrichardson. Is it overridden via an env var? Or does user need to override the rhiza.mk?

Ignore this I see it tested. Looks great

@tschm
Copy link
Copy Markdown
Member

tschm commented Mar 30, 2026

Rhiza is no longer overwriting files (you have modified) with a hammer. So, the patching exercise should be obsolete. Let's first verify that this is indeed correct and if not we should do as sketched here?

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.

Enable users to override default rhiza sync times

4 participants