diff --git a/.agents/skills/iterate-pr/SKILL.md b/.agents/skills/iterate-pr/SKILL.md index 1c7151f11..55183aafb 100644 --- a/.agents/skills/iterate-pr/SKILL.md +++ b/.agents/skills/iterate-pr/SKILL.md @@ -4,11 +4,16 @@ description: Iterate on a PR until CI passes. Optionally merge or merge and publ argument-hint: "[--merge] [--release]" metadata: author: leecalcote - version: "2.3.0" + version: "2.4.0" --- # Iterate on PR Until CI Passes +> **2.4.0** - `fetch_pr_feedback.py` no longer reports an empty result as a successful +> fetch. Every path where a feedback channel could come back empty because the request +> did not actually deliver now fails loudly, and a server-side count probe backstops any +> path not yet known. See [Fetch Integrity](#fetch-integrity). + Continuously iterate on the current branch until all CI checks pass and review feedback is addressed. **Requires**: GitHub CLI (`gh`) authenticated. @@ -19,6 +24,11 @@ Continuously iterate on the current branch until all CI checks pass and review f ### Running the Bundled Scripts +Skills live in `.agents/skills/` - that is the canonical path, and the only one used in +this document. Some repos also expose `.claude/skills` as a symlink to it, but that +symlink is not guaranteed: it does not survive a Windows checkout with +`core.symlinks=false`, and not every repo carries it. Always invoke via `.agents/`. + Every script below accepts either runner - they only use the standard library, so no dependency installation is needed either way: @@ -39,23 +49,66 @@ fi The rest of this document shows invocations in the shorter `python3