Skip to content

Commit e4ebdd9

Browse files
committed
fix: PRs open as draft, mark ready after coding complete
- daily-intelligence.md: draft: true in safe-outputs, gh pr ready in body - aw-daily SKILL.md: --draft flag on gh pr create, gh pr ready step - aw-merge command: --draft on gh pr create, gh pr ready before CI wait - All agent instruction files updated with draft PR convention - Recompiled daily-intelligence.lock.yml (0 errors)
1 parent 3f2f3c9 commit e4ebdd9

7 files changed

Lines changed: 40 additions & 19 deletions

File tree

.github/copilot-instructions.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,7 @@ The 9 files in `skills/aw-author/references/` are the canonical gh-aw specificat
9292
7. **`${{ }}` expressions in fenced code blocks are NOT interpolated** — use env vars instead
9393
8. **Knowledge base is append-only** — mark superseded entries with `[SUPERSEDED by YYYY-MM-DD]`, never delete
9494
9. **Section headers are anchors, not line numbers** — the daily pipeline uses headers to locate edit targets because line numbers shift between runs
95+
10. **PRs are created as drafts** — open with `--draft` or `draft: true`, then mark ready with `gh pr ready` after all changes are pushed and verified
9596

9697
## GitHub Discussion Category
9798

.github/workflows/daily-intelligence.lock.yml

Lines changed: 16 additions & 16 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/daily-intelligence.md

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ safe-outputs:
3939
create-pull-request:
4040
title-prefix: "docs(references): "
4141
labels: [automated, reference-update]
42-
draft: false
42+
draft: true
4343
base-branch: develop
4444
max: 1
4545
push-to-pull-request-branch:
@@ -181,12 +181,18 @@ For each gap, edit the target reference file. Use section headers as anchors (no
181181

182182
### Phase 7: PR Creation
183183

184-
Create a PR to `develop` using the `create-pull-request` safe-output:
184+
Create a **draft** PR to `develop` using the `create-pull-request` safe-output:
185185
- Title: `docs(references): daily intelligence update {TODAY}`
186186
- Body: summary of gaps addressed, links to issues (using `Closes #NNN`), link to Discussion
187187
- Base: `develop`
188188
- Labels: `automated`, `reference-update`
189189

190+
The PR is created as a draft (frontmatter `draft: true`). After all changes are pushed and verified, mark it ready for review:
191+
192+
```bash
193+
gh pr ready <PR_NUMBER>
194+
```
195+
190196
### Phase 8: Summary
191197

192198
Report what was accomplished: searches run, findings count, gaps identified, issues created, files changed, PR URL. If any phase was skipped or failed, note it clearly.

AGENTS.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ The **aw-author** plugin (v1.3.0) for Claude Code. It provides skills, commands,
3333
6. **Discussion posts go to the "Project News" category** (ID: `DIC_kwDORSXBr84C61Lr`).
3434
7. **The daily pipeline is idempotent.** It checks for existing Discussions, PRs, and issues before creating new ones. Re-running on the same day is safe.
3535
8. **When editing reference files, verify the change compiles.** Run `gh aw compile` after modifying any `.md` file in `.github/workflows/`.
36+
9. **PRs are created as drafts.** Open with `--draft` or `draft: true` in safe-outputs, then mark ready with `gh pr ready` after all changes are pushed and verified.
3637

3738
## gh-aw Spec Essentials
3839

CLAUDE.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ After editing any `.github/workflows/*.md` file, always recompile with `gh aw co
4747
- **`plugins:` is deprecated** — use `dependencies:`, run `gh aw fix --write`
4848
- **Knowledge base is append-only** — mark old entries `[SUPERSEDED by YYYY-MM-DD]`
4949
- **Use section headers as edit anchors** — not line numbers (they shift between runs)
50+
- **PRs are created as drafts** — use `--draft` on `gh pr create`, mark ready with `gh pr ready` after changes are complete
5051
- **Discussion category ID**: `DIC_kwDORSXBr84C61Lr` (Project News)
5152

5253
## gh-aw Compiler

commands/aw-merge.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ If `--dry-run`: show `git log --oneline origin/main..origin/develop` and **exit*
4141

4242
```bash
4343
gh pr create \
44+
--draft \
4445
--repo zircote/github-agentic-workflows \
4546
--base main \
4647
--head develop \
@@ -59,6 +60,11 @@ $(git log --oneline origin/main..origin/develop)
5960
_Automated by /aw-merge_"
6061
```
6162

63+
Mark PR ready:
64+
```bash
65+
gh pr ready <PR_NUMBER>
66+
```
67+
6268
### Phase 3: CI and merge
6369

6470
1. Wait for CI checks: poll `gh pr checks` every 15 seconds, max 5 minutes

skills/aw-daily/SKILL.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -302,6 +302,7 @@ If `--dry-run`: show the diff but do not commit or push.
302302

303303
```bash
304304
gh pr create \
305+
--draft \
305306
--repo zircote/github-agentic-workflows \
306307
--base develop \
307308
--head daily-intelligence-{TODAY} \
@@ -326,7 +327,12 @@ Automated reference file updates from daily intelligence sweep.
326327
_Automated by /aw-daily_"
327328
```
328329

329-
Store the PR URL.
330+
Store the PR URL and number.
331+
332+
**Mark PR ready** after all changes are pushed:
333+
```bash
334+
gh pr ready {PR_NUMBER}
335+
```
330336

331337
**Error mode:** If PR creation fails, report error. Leave branch for manual inspection. Switch back to `ORIGINAL_BRANCH`.
332338

0 commit comments

Comments
 (0)