Skip to content

docs: give scheduled Pickup runs a head start over Architect runs - #426

Merged
JacobStephens2 merged 1 commit into
mainfrom
issue-345
Oct 5, 2026
Merged

JacobStephens2 merged 1 commit into
mainfrom
issue-345

Conversation

@JacobStephens2

@JacobStephens2 JacobStephens2 commented Oct 5, 2026 •

Copy link
Copy Markdown
Owner

Summary

Give Pickup runs a 20-second head start when they share scheduled minutes with Architect runs. Show the delay in the combined crontab and explain where to add it.

-*/5 * * * * cd ~/repos/widgets && thirdshift architect base main --plan-only >> ~/.thirdshift/logs/cron.log 2>&1
+*/5 * * * * cd ~/repos/widgets && sleep 20 && thirdshift architect base main --plan-only >> ~/.thirdshift/logs/cron.log 2>&1

Closes #345

Evidence

  • Before: The combined crontab started Architect and Pickup commands on the same scheduled minute without a delay.
    After: Shell execution with stubs confirmed cd -> sleep 20 -> thirdshift architect base main --plan-only, in the Launch directory. A failed cd starts neither command.
  • All eight README crontab commands passed /bin/sh -n.
  • cargo fmt --check and cargo clippy --all-targets -- -D warnings passed.
  • cargo test --test command_surface: 29 passed.
  • cargo test: 1086 passed, 0 failed, 0 ignored across 34 test binaries.

Merge Danger

Door: two-way

Revert the README change.

Blast Radius: documentation

Users copying the combined crontab delay each Architect pass by 20 seconds; Pickup timing is unchanged.

Unaddressed findings

Standards

None. The review against main reported no Standards findings.

Spec

None. The review against main reported no Spec findings.

Built with codex · gpt-6.1-sol · xhigh

@JacobStephens2
JacobStephens2 merged commit 60fa979 into main Oct 5, 2026
13 checks passed
@JacobStephens2
JacobStephens2 deleted the issue-345 branch October 5, 2026 12:39
This was referenced Oct 6, 2026
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.

put && sleep 20 suggestion in crontab example for thirdshift architect runs

1 participant