Skip to content

Commit 2fbfbbe

Browse files
authored
Release 0.3.12
1 parent 9e6f4a5 commit 2fbfbbe

51 files changed

Lines changed: 1095 additions & 578 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.archon/commands/maintainer-review-gate.md

Lines changed: 0 additions & 255 deletions
This file was deleted.
Lines changed: 10 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -1,32 +1,25 @@
11
---
2-
description: Produce the final summary across all branches of maintainer-review-pr (review / decline / unclear) for the workflow log
2+
description: Produce the final summary of a maintainer-review-pr run for the workflow log
33
argument-hint: (no arguments — reads upstream artifacts)
44
---
55

66
# Maintainer Review — Final Report
77

8-
You are the final reporter. The workflow has finished one of three branches (review / decline / unclear). Your job: produce a one-screen summary that tells the maintainer what just happened and what's pending.
8+
You are the final reporter. The workflow has finished the deep review. Your job: produce a one-screen summary that tells the maintainer what just happened and what's pending.
99

1010
**Workflow ID**: $WORKFLOW_ID
1111

1212
---
1313

14-
## Phase 1: DETECT WHICH BRANCH RAN
15-
16-
Check what artifacts exist:
14+
## Phase 1: LOAD ARTIFACTS
1715

1816
```bash
1917
PR_NUMBER=$(cat $ARTIFACTS_DIR/.pr-number 2>/dev/null)
2018
ls $ARTIFACTS_DIR/
2119
ls $ARTIFACTS_DIR/review/ 2>/dev/null
22-
cat $ARTIFACTS_DIR/gate-decision.md 2>/dev/null | head -30
2320
```
2421

25-
Three possibilities:
26-
27-
1. **Review branch ran**: `$ARTIFACTS_DIR/review/synthesis.md` exists.
28-
2. **Decline branch ran**: `$ARTIFACTS_DIR/decline-comment.md` exists with non-placeholder content; the post-decline bash node already posted to GitHub.
29-
3. **Unclear branch ran**: gate verdict was `unclear` and the maintainer was prompted to decide manually.
22+
`$ARTIFACTS_DIR/review/synthesis.md` should exist and contain the synthesized verdict + findings.
3023

3124
---
3225

@@ -37,37 +30,19 @@ Write `$ARTIFACTS_DIR/final-report.md`:
3730
```markdown
3831
# Maintainer Review — PR #<n> — Final
3932

40-
## Branch taken
41-
<review | decline | needs_split | unclear>
42-
43-
## Gate decision
44-
<one-line summary from gate-decision.md: verdict + direction + scope + template>
45-
4633
## Outcome
4734

48-
### If review branch:
4935
- Synthesized verdict: <ready-to-merge | minor-fixes-needed | blocking-issues>
5036
- Findings: <N CRITICAL / N HIGH / N MEDIUM / N LOW>
5137
- Aspects run: <list>
52-
- **Draft comment**: $ARTIFACTS_DIR/review/review-comment.md (copy-paste or edit before posting to PR)
38+
- **Draft comment**: $ARTIFACTS_DIR/review/review-comment.md (already posted to PR; copy-paste if you want to edit and re-post)
5339
- **Full synthesis**: $ARTIFACTS_DIR/review/synthesis.md
5440

55-
### If decline branch:
56-
- Decline categories: <list>
57-
- Cited direction clauses: <list>
58-
- Comment posted to PR: yes
59-
- Reply window: <YYYY-MM-DD>
60-
- Awaiting-author label added: read `$ARTIFACTS_DIR/.label-applied` — value is `applied` or `skipped`. If `skipped`, surface why by reading `$ARTIFACTS_DIR/.label-error` (gh stderr) and include a one-line explanation. **Do not say `yes` if the file says `skipped`** — say `no, label add failed: <reason>` so the maintainer can decide whether to add it manually.
61-
62-
### If unclear branch:
63-
- Gate could not classify confidently.
64-
- Maintainer prompted manually — outcome recorded in approval-gate response.
65-
6641
## Next steps for the maintainer
6742
<2-3 short bullets. e.g.:
68-
- "Read $ARTIFACTS_DIR/review/review-comment.md and post to PR."
69-
- "Wait for contributor reply by <DATE>; if no reply, close PR."
70-
- "Update direction.md to address the open question this PR raised: <topic>".>
43+
- "Open PR #<n> and confirm the posted review reads well."
44+
- "If blocking-issues: wait for contributor reply; check back in N days."
45+
- "If ready-to-merge: merge when CI is green.">
7146
```
7247

7348
---
@@ -77,10 +52,10 @@ Write `$ARTIFACTS_DIR/final-report.md`:
7752
Return a single-line outcome:
7853

7954
```
80-
PR #<n> — branch=<review|decline|needs_split|unclear>, verdict=<gate verdict>, action=<posted-comment|drafted-review|awaiting-manual-decision>.
55+
PR #<n> — verdict=<synthesized verdict>, action=posted-review-comment.
8156
```
8257

8358
### CHECKPOINT
8459
- [ ] `$ARTIFACTS_DIR/final-report.md` written.
85-
- [ ] Correctly identifies which branch ran (don't pretend the review branch ran when it didn't).
60+
- [ ] Numbers in the report match `$ARTIFACTS_DIR/review/synthesis.md` (don't invent finding counts).
8661
- [ ] Lists concrete next steps for the maintainer.

.archon/commands/maintainer-review-synthesize.md

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,6 @@ Then read each one:
3232

3333
Some files may be missing — that's expected. Don't error.
3434

35-
### Read the gate decision (for context)
36-
```bash
37-
cat $ARTIFACTS_DIR/gate-decision.md
38-
```
39-
40-
The gate may have noted things ("template was empty — nudge in synthesis"). Carry those notes forward.
41-
4235
---
4336

4437
## Phase 2: AGGREGATE + DEDUPLICATE
@@ -88,9 +81,6 @@ Write `$ARTIFACTS_DIR/review/synthesis.md`:
8881
## CLAUDE.md compliance
8982
<bullet list of any violations carried forward from code-review.>
9083

91-
## Gate-decision notes
92-
<any non-review notes the gate flagged for synthesis — e.g. "template was empty, nudge contributor".>
93-
9484
## Aspects run
9585
- code-review: <yes/no, summary line>
9686
- error-handling: <yes/no, summary line>

0 commit comments

Comments
 (0)