diff --git a/github-workflows/skills/gh-cli-patterns/SKILL.md b/github-workflows/skills/gh-cli-patterns/SKILL.md index 0767404..c0b1cb8 100644 --- a/github-workflows/skills/gh-cli-patterns/SKILL.md +++ b/github-workflows/skills/gh-cli-patterns/SKILL.md @@ -191,24 +191,24 @@ skill that emits a summary — do NOT define local output formats in individual ```text {Title} ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ - ✅ https://github.com///pull/42 Ready for review - 🟡 https://github.com///pull/43 CI pending - 🔴 https://github.com///pull/44 Conflicts | 3 open comments + ✅ #42 Ready for review + 🟡 #43 CI pending + 🔴 #44 Conflicts | 3 open comments ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ All Open PRs — / ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ - ✅ https://github.com///pull/42 Ready for review - 🟡 https://github.com///pull/43 CI pending - 🔴 https://github.com///pull/44 Conflicts | 3 open comments + ✅ #42 Ready for review + 🟡 #43 CI pending + 🔴 #44 Conflicts | 3 open comments ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ Ready to merge (1): /squash-merge-pr 42 (/) Blocked — needs human (2): - https://github.com///pull/43 — CI pending - https://github.com///pull/44 — Conflicts | 3 open comments + #43 — CI pending + #44 — Conflicts | 3 open comments ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ ``` @@ -228,7 +228,7 @@ Blocked — needs human (2): ### Status tags -Append after the URL, separated by ` | `. Omit when no issues exist ("Ready for review" suffices). +Append after the PR number, separated by ` | `. Omit when no issues exist ("Ready for review" suffices). | Tag | Section 1 trigger | Section 2 trigger | |-----|-------------------|-------------------| @@ -244,12 +244,6 @@ Append after the URL, separated by ` | `. Omit when no issues exist ("Ready for ### Data queries -**Fetch PR URL** (for Section 1 — current PRs): - -```bash -gh pr view --json url --jq '.url' -``` - **Fetch all open PRs** (for Section 2 — one GraphQL call per affected repo): `gh pr list --json` does NOT support `mergeStateStatus` — use GraphQL instead: @@ -260,7 +254,7 @@ gh api graphql -f query=' repository(owner:$owner,name:$repo){ pullRequests(states:OPEN,first:50){ nodes{ - number url title mergeable reviewDecision mergeStateStatus isDraft + number title mergeable reviewDecision mergeStateStatus isDraft commits(last:1){nodes{commit{statusCheckRollup{state}}}} } } diff --git a/github-workflows/skills/ship/SKILL.md b/github-workflows/skills/ship/SKILL.md index 3d0a69a..a8d9cdc 100644 --- a/github-workflows/skills/ship/SKILL.md +++ b/github-workflows/skills/ship/SKILL.md @@ -181,11 +181,7 @@ If any abort condition hits: re-invoke `/finalize-pr `, wait for comp then re-run both gates. Only list a PR as "Ready to merge" after both gates pass. Then emit the **Canonical PR Status Summary** as defined in /gh-cli-patterns, titled -`Ship Summary`. Affected repos = current repo. Fetch each PR's full URL via: - -```bash -gh pr view --json url --jq '.url' -``` +`Ship Summary`. Affected repos = current repo. Section 1 lists the PRs targeted by this `/ship` invocation. Section 2 lists all open PRs in the current repo (including unrelated ones).