Skip to content

fix: stop scode-shepherd plans forbidding the worker's report - #511

Merged
scode merged 1 commit into
mainfrom
shepherd-report-write-rule
Nov 7, 2026
Merged

scode merged 1 commit into
mainfrom
shepherd-report-write-rule

Conversation

@scode

@scode scode commented Nov 7, 2026

Copy link
Copy Markdown
Owner

The scode-shepherd skill told the supervisor that workers never write in the run directory. That is false: the runner
has every worker session write its report file there, and the skill's spec says that report is the only thing a worker
writes there. On a real run the supervisor copied the false sentence into the plan as a rule for every worker, "Never
write files in the run directory yourself", which contradicts the runner's own instruction to write the report. For a
plan rule that conflicts with one of its own, the runner tells the worker to follow the runner's rule and report
replan, which stops the run and wakes the supervisor to fix the plan. That worker wrote its report and carried on, but
a worker that followed the precedence rule could halt the run over this one sentence.

The skill now says what is true: a worker writes only its own report in the run directory. The supervisor must not tell
a worker to write anything else there, which is what the old sentence was for (workers logging decisions in the run's
journal, which the runner fills from their reports), and must not ban writes there altogether either.

The scode-shepherd skill told the supervisor that workers never write in the run directory. That is false: the runner
has every worker session write its report file there, and the skill's spec says that report is the only thing a worker
writes there. On a real run the supervisor copied the false sentence into the plan as a rule for every worker, "Never
write files in the run directory yourself", which contradicts the runner's own instruction to write the report. For a
plan rule that conflicts with one of its own, the runner tells the worker to follow the runner's rule and report
replan, which stops the run and wakes the supervisor to fix the plan. That worker wrote its report and carried on, but
a worker that followed the precedence rule could halt the run over this one sentence.

The skill now says what is true: a worker writes only its own report in the run directory. The supervisor must not tell
a worker to write anything else there, which is what the old sentence was for (workers logging decisions in the run's
journal, which the runner fills from their reports), and must not ban writes there altogether either.
@scode
scode marked this pull request as ready for review November 7, 2026 16:21
@scode
scode merged commit b3fa197 into main Nov 7, 2026
5 checks passed
@scode
scode deleted the shepherd-report-write-rule branch November 7, 2026 16: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