docs(ci): document workflow buckets and history#253
Conversation
Groups the 11 workflows into four buckets (CI, auto-release, release, packaging publish, packaging validation) with triggers and intent. Captures the non-obvious history: PR #224 removing the pull_request trigger from packaging test workflows, PR #100 simplifying chocolatey packaging per reviewer feedback, and the GoReleaser tag-rename dance. Closes #252
monit-reviewer
left a comment
monit-reviewer
left a comment
There was a problem hiding this comment.
Automated PR Review
Reviewed commit: f45b347
Approved with 2 non-blocking suggestions below. Address at your discretion.
Summary
| Reviewer | Findings |
|---|---|
| harness-engineering:architecture-reviewer | 1 |
| harness-engineering:knowledge-reviewer | 1 |
harness-engineering:architecture-reviewer (1 findings)
💡 Suggestion - .github/workflows/CLAUDE.md:8
PR description says '4 buckets' but the table has 5 distinct bucket rows (CI, auto-release, release, packaging publish, packaging validation). Minor inconsistency between PR description and actual content.
harness-engineering:knowledge-reviewer (1 findings)
💡 Suggestion - .github/workflows/CLAUDE.md:16
History section states packaging test workflows fired on
pull_request'since Jan 2026, PR #11'. PR #11 is implausibly low for a January 2026 addition to a repo with 250+ PRs — likely a typo (possibly PR #211 or similar). Incorrect PR references undermine the document's value as an authoritative history source.
Completed in 30s | $0.31
| Field | Value |
|---|---|
| Reviewers | hybrid-synthesis, harness-engineering:knowledge-reviewer, harness-engineering:enforcement-reviewer, harness-engineering:architecture-reviewer |
| Engine | claude |
| Reviewed by | pr-review-daemon · monit-pr-reviewer |
| Duration | 30s wall · 46s compute (Reviewers: 12s · Synthesis: 15s) |
| Cost | $0.31 |
| Tokens | 69.1k in / 2.4k out |
| Turns | 4 |
monit-reviewer
left a comment
monit-reviewer
left a comment
There was a problem hiding this comment.
Automated PR Review
Reviewed commit: 3d16d29 | Previous: f45b347 (incremental)
Summary
No issues found.
2 PR discussion threads considered.
Completed in 49s | $0.30
| Field | Value |
|---|---|
| Mode | Re-review · Cycle 2 · Session resumed |
| Reviewers | hybrid-synthesis, harness-engineering:knowledge-reviewer, harness-engineering:enforcement-reviewer, harness-engineering:architecture-reviewer |
| Engine | claude |
| Reviewed by | pr-review-daemon · monit-pr-reviewer |
| Duration | 49s wall · 60s compute (Reviewers: 29s · Synthesis: 9s) |
| Cost | $0.30 |
| Tokens | 107.2k in / 3.1k out |
| Turns | 5 |
2 participants
Summary
Adds
.github/workflows/CLAUDE.mdthat:pull_requesttrigger from packaging test workflows, PR feat(chocolatey): simplify packaging per reviewer feedback #100 simplifying chocolatey packaging per reviewer feedback, and the GoReleaser tag-rename pattern.pull_requestto the packaging test workflows; new packaging integrations must account for the tag rename; publish workflows are manual.Docs only — no workflow behavior changes.
Test plan
ls .github/workflows/Closes #252