Skip to content

refactor(dev): babysit-prs — progressive disclosure into references/ - #11

Merged
zublenko merged 1 commit into
babysit-prs-skillfrom
babysit-prs-references
Aug 6, 2026
Merged

refactor(dev): babysit-prs — progressive disclosure into references/#11
zublenko merged 1 commit into
babysit-prs-skillfrom
babysit-prs-references

Conversation

@zublenko

@zublenko zublenko commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Implements point 2 of the babysit-prs improvement plan: progressive disclosure. The two long per-PR procedures move out of SKILL.md into one reference file per axis.

Stacked on babysit-prs-skill (point 1, #10).

What changed

SKILL.md goes 574 → 311 lines. The orchestrator half, the worker contract (routing table + confidence rule) and Safety stay; the procedures leave:

  • references/conflict-procedure.md — the merge-conflict axis, steps 1–7
  • references/ci-procedure.md — the failing-checks axis, steps 1–5

Both bodies moved verbatim. The only text changes are the three cross-references that stopped resolving in-file: "(step 5)""(step 5 below)", "Step 5 asserts""Step 5 below asserts", and CI step 4's mention of the conflict procedure now naming the file plus its own step 7. Verified mechanically by diffing the old sections against the new files (164→164 and 77→78 non-blank lines, 3 hunks, all three of them the above).

Each reference file opens with a short header instead of duplicating shared rules: what axis it is, which WORKTREE_DIR literal it means, that <...> values come from the worker prompt, that its step numbers are its own (not the orchestrator's Steps 1–5), and a pointer back to ../SKILL.md for the confidence rule and Safety. The conflict file ends by handing off to the CI file when both axes are flagged.

Wiring

  • Step 1 resolves SKILL_DIR next to SKILL_FILE, tables the two procedure paths, and says explicitly that the orchestrator must not read them — it only passes the paths on. That's where the win lands: ~300 lines of git surgery and CI archaeology never enter a context that, under /loop, accumulates pass after pass.
  • Step 4's worker prompt now carries a Procedure: path per flagged axis (absolute, literal — same rule as every other value in that template), with an instruction to read only the flagged ones and only on reaching that axis. A conflicts-only worker never loads the CI procedure.
  • The inline fallback (no Agent tool) is called out as the one path that reads the procedure files itself, one axis at a time.

Verification

  • Procedure bodies diffed against the pre-split sections; every command line accounted for.
  • No stale #conflict-procedure / #ci-procedure anchor links anywhere in the repo.
  • grep -iE 'hoopit|BAC-|WEB-|FA-' clean across the skill dir — repo-agnostic per create-hoopit-skill Rule 1.
  • Frontmatter description unchanged; scripts/gen-skills-readme.sh regenerates README.md with no diff (pre-commit passes).

Not in this PR

Later points of the plan: flake retry, cross-pass state, stale-worktree sweep.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Point 2 of the babysit-prs improvement plan: progressive disclosure. The two
long per-PR procedures move out of SKILL.md into one file per axis under
references/, leaving SKILL.md at 311 lines (was 574).

- references/conflict-procedure.md — the merge-conflict axis (steps 1-7)
- references/ci-procedure.md — the failing-checks axis (steps 1-5)

Both bodies move verbatim; the only text changes are cross-references that no
longer resolve in-file ("step 5" -> "step 5 below", and CI step 4's pointer at
the conflict procedure's own step 7). Each file opens by naming its worktree dir
and its placeholders, and defers to SKILL.md for the confidence rule and Safety
so neither is duplicated.

Wiring:
- Step 1 resolves SKILL_DIR alongside SKILL_FILE and states that the
  orchestrator must not read either procedure file — it only passes the paths on.
- The Step 4 worker prompt carries a procedure path per flagged axis, with an
  explicit instruction to read only the flagged ones, as each axis is reached.
- The inline (no Agent tool) fallback is called out as the one path that reads
  the procedure files itself.

Frontmatter description is unchanged, so the generated README is untouched.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d226df3b-9335-4b0c-b180-4612220a4ee7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@zublenko

zublenko commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #12, which targets main directly. This PR was based on the babysit-prs-skill feature branch, which never landed on main#12 replays point 1's commits (reviewed on #10) plus this progressive-disclosure split onto main, with byte-identical skill files.

@zublenko zublenko closed this Aug 6, 2026
@zublenko
zublenko merged commit e62c0d1 into babysit-prs-skill Aug 6, 2026
2 checks passed
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