Skip to content
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 17 additions & 14 deletions .agents/skills/babysit/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,15 +47,18 @@ formal review list is not a completed review. Check the PR body and trigger
comments too. If review fails to start, trigger it once; do not duplicate an
active request.

Read every finding the completed review delivered, sort the whole set, then
push one batch. Before acting on any finding, apply the repository's complexity
tripwire; when it fires, pause and report the resumable state instead of
opening another round.
Apply the repository's complexity tripwire before acting on any finding; when
it fires, pause and report the resumable state instead of opening another
round. Then read every finding the completed review delivered, sort the whole
set, and push one batch.
Comment thread
lastobelus marked this conversation as resolved.
Outdated

Sort each finding into one verb:

- **fix** — it breaks correctness, user data, or a stated acceptance criterion
in supported use. Fix it this round.
- **fold** — it improves the changed code, breaks nothing, and stays inside the
slice. Fold it into the batch this round is already pushing; when the round
has no batch, defer it rather than opening a round for polish.
Comment thread
lastobelus marked this conversation as resolved.
- **narrow** — it holds only because the change claims an open-ended property.
Remove or narrow the claim instead of building machinery to satisfy it.
- **defer** — real value, outside this slice. Record it in the reply and in the
Expand All @@ -66,7 +69,7 @@ Sort each finding into one verb:
outside.

Reply with evidence and resolve every thread you handled, including every
narrow, defer, and decline.
fold, narrow, defer, and decline.

Diagnose red CI without waiting for the review to complete: fix code failures
and rerun transient infrastructure failures. Rebase when behind or required,
Expand All @@ -79,9 +82,9 @@ resolutions.

The pull request is green when the current head has green CI, zero unresolved
threads, a clean mergeable state, and a completed current-head review whose
findings are all dispositioned. A narrow that changed no file, a defer, and a
decline disposition a finding without a new head, so they need no further
review.
findings are all dispositioned. A narrow that changed no file, a folded finding
carried in an existing batch, a defer, and a decline disposition a finding
without a new head, so they need no further review.

Open at most three finding-bearing rounds against one boundary. Every fix and
every file-changing narrow creates a new head and therefore opens the next
Expand All @@ -100,9 +103,9 @@ before the merge command.

## 6. Report

Report the pull request and head, rounds used, the fixes and the narrowed,
deferred, and declined findings with their reasons, rebases, validation, CI and
review results, and the merge result when applicable. In merge mode, include
the prepared next steps or archive readiness. If interrupted or paused, stop
reads and mutations immediately, report the same state as a resumable
checkpoint, and resume from a fresh snapshot.
Report the pull request and head, rounds used, the fixes and the folded,
narrowed, deferred, and declined findings with their reasons, rebases,
validation, CI and review results, and the merge result when applicable. In
merge mode, include the prepared next steps or archive readiness. If
interrupted or paused, stop reads and mutations immediately, report the same
state as a resumable checkpoint, and resume from a fresh snapshot.
16 changes: 10 additions & 6 deletions .agents/skills/babysit/references/merge.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,15 @@ URL. Run `done` so every matching local review reaches Done; zero matching
reviews is success. Report a lookup or Markover failure without weakening the
verified GitHub merge result.

Then refresh the issue conversation for comments added during the merge before
recommending next steps or closing it, and account for every remaining adjunct
item. If work remains, prepare concrete next steps for the report. If the issue
is complete, perform the remaining state-only housekeeping, verify the issue
and its trackers reflect completion, and prepare an archive-ready conclusion.
The merged pull request's work-intent claim completes with the merge, so set
its phase to `completed`. Then refresh the issue conversation for comments
Comment thread
lastobelus marked this conversation as resolved.
Outdated
added during the merge before recommending next steps or closing it, and
account for every remaining adjunct item. If work remains, prepare concrete
next steps for the report; the issue's own claim stays as it is. If the issue
is complete, perform the remaining state-only housekeeping, set its claim to
`completed` as well, verify the issue and its trackers reflect completion, and
prepare an archive-ready conclusion.

**Complete when:** the merge is verified, matching Markover reviews are Done,
and the issue and its trackers match the real state.
every completed claim says so, and the issue and its trackers match the real
state.
Loading