Repository navigation
chore: install agent contract and chore workflows - #189
Conversation
Installed by scripts/quick-setup.sh from gominimal/min-aw@v0.2.4.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (2)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThis PR establishes a comprehensive agentic workflow infrastructure by adding GitHub issue templates (bug, feature, chore, feedback), defining labels and agent conventions, creating worker workflows that execute remote agents, configuring Copilot instructions, and documenting the agent contract via AGENTS.md. ChangesAgentic Workflow Foundation
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related issues
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/ISSUE_TEMPLATE/chore.md:
- Around line 29-37: Update the "Agent Label (if applicable)" list to include
the two missing labels by adding `agent:api-drift` and `agent:not-gating-audit`
alongside the existing `agent:doc-drift`, `agent:coverage`, and
`agent:dep-drift` entries; ensure the list under that header (and any mention
that points readers to `AGENTS.md`) remains consistent with the AGENTS.md
taxonomy so the template and docs match.
In @.github/labels.yml:
- Around line 6-25: The file is missing two required agent labels; add entries
for name: "agent:api-drift" and name: "agent:not-gating-audit" mirroring the
existing label format (include a hex color and a short description each) so they
appear alongside the other agent:* labels (e.g., like "agent:doc-drift" and
"agent:coverage"); ensure the descriptions state they are opened/updated by the
api-drift agent and the not-gating-audit agent respectively and follow the same
YAML structure as the other label entries.
In @.github/workflows/sync-from-source-of-truth.yml:
- Around line 86-90: The workflow's SOURCE_REF validation rejects the current
default 'v0.2.4', so either make the default a distinct placeholder token (e.g.,
set SOURCE_REF to "__SOURCE_REF__" in the template and have quick-setup.sh
replace that with "v0.2.4") or delete/disable the validation block that checks
for the unsubstituted "v0.2.4"; locate the variable SOURCE_REF and the
validation if-block that compares it to "v0.2.4" and implement one of these two
fixes, updating quick-setup.sh substitution logic if you choose the placeholder
approach.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 166e28fc-96a6-409e-a618-ff57d1ea7e40
📒 Files selected for processing (12)
.github/ISSUE_TEMPLATE/bug.md.github/ISSUE_TEMPLATE/chore.md.github/ISSUE_TEMPLATE/feature.md.github/ISSUE_TEMPLATE/feedback.md.github/agents/.gitkeep.github/agents/README.md.github/copilot-instructions.md.github/labels.yml.github/workflows/sync-from-source-of-truth.yml.github/workflows/worker-fix.yml.github/workflows/worker-iterate.ymlAGENTS.md
Pulls in the three CodeRabbit-flagged template fixes:
1. sync-from-source-of-truth.yml: drops the unsubstituted-token guard
that had become a self-rejecting tautology after sed substituted
{{SOURCE_REF}} -> v0.2.4 globally. The regex check already rejects
the unsubstituted form. Source_ref default bumped v0.2.4 -> v0.2.5.
2. labels.yml: adds agent:api-drift and agent:not-gating-audit (were
referenced in chore.md / agent-taxonomy comment but never defined).
3. chore.md: adds the same two labels to the agent-label enumeration.
Source-of-truth pin: gominimal/min-aw@v0.2.5
(https://github.com/gominimal/min-aw/releases/tag/v0.2.5)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Caution Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted. Error details |
TL;DR
gominimal/min-aw@v0.2.4. Chores file issues / open draft PRs against this repo on a fuzzy schedule. Caps: max 1 issue per chore per run; max 1 PR per chore per run; auto-merge opt-in per chore (onlytrivial-dep-bumpenables it, and only for patch-level Cargo.lock changes)..github/workflows/<chore>.lock.ymlafter sync, OR halt all chores by removing the sync workflow itself. No retaliatory state on the source-of-truth side.What this PR installs (markdown-only)
AGENTS.md— agent context document for human + AI tooling.github/copilot-instructions.md— Copilot instructions referencing gh-aw-fragments.github/labels.yml— agent label set (agent:*,kind:*,priority:*).github/agents/— agent definition directory.github/ISSUE_TEMPLATE/— issue templates (bug, feature, chore, feedback).github/workflows/sync-from-source-of-truth.yml— weekly sync workflow that pulls compiled.lock.ymlfiles from min-awSource-of-truth pin:
gominimal/min-aw@v0.2.4. The pin is either a release tag (vX.Y.Z) or a full 40-char commit SHA — both link via/tree/so the link works regardless of which form was passed toquick-setup.sh.What runs after merge
The sync workflow lands compiled chore
.lock.ymlfiles on a weekly schedule. Each chore is governed by its own caps and triggers; full details inworkflows/on min-aw. For minimal (LANGUAGE=Rust) per ADR 0003, the applicable chores include:docs-patrol— files an issue when documentation drifts from the source code (1 issue/run, fuzzy weekly Monday).worker-fix— picks up openagent:*issues and drafts a fix PR (1 PR/run, fuzzy daily). Bot-authored PRs auto-trigger CodeRabbit review.worker-iterate— addresses CodeRabbit (and human) inline review-comment feedback ongominimal-aw-bot[bot]-authored[worker: ...PRs. Triggered per CodeRabbit review submission; pushes commits to the PR branch and replies inline. Caps: 3 comments/run, 5 runs/PR. (#83)not-gating-audit— weekly audit that branch protection onmainrequires only your CI (nogh-awchecks added). Files an issue if drift is detected.dependency-review,api-surface-drift,test-coverage-detector,trivial-dep-bump) — install only if LANGUAGE=rust today; per-language overlays for Go/Node/Nickel land in #69 phase 1.What this does NOT do
gh-awcheck is ever added torequired_status_checks. Your CI remains the only gate. Thenot-gating-auditchore actively defends this; it files an issue if anygh-awcheck appears in branch protection.main. All chore output flows through the gh-aw safe-output gate (PRs and issues, never direct pushes).Review checklist
AGENTS.md+.github/markdown/yaml files; no source code touchedAGENTS.mdmatches the active agent set for this repo.github/workflows/sync-from-source-of-truth.ymlpins to a release tag (notmain)mainwas made (this is a PR)mainis unchanged (verify viagh api repos/gominimal/minimal/branches/main/protection)Hand-off
This PR is opened as draft. The min-aw operator will not toggle ready-for-review or enable auto-merge. When you are satisfied:
To push back or roll back, see the rollback procedure in
docs/rollout/07-target-repo-rollout.md. Or comment on this PR — feedback shapes the next sync.Summary by CodeRabbit
Release Notes
Documentation
Chores