chore: version + harden the devops subagent definition#18
Merged
Conversation
Track .claude/agents/devops.md so the agent definition is versioned and can evolve through reviewable PRs, instead of living only in the local worktree. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Encode the lessons from the #16 PR so the agent needs less hand-holding: - Workflow defaults: fetch + verify sync, branch off main, PR back to main, one issue = one PR, deliverable is a draft PR it creates, keep PRs scoped, and confirm CI is green before reporting done. - CI checks: state the invariant a check guarantees and on which OS it runs; prefer checks correct regardless of runner OS (e.g. multi-platform lock enforcement via `providers lock` + `git diff`, not single-OS readonly init). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
.claude/agents/devops.mdlived only in the local worktree. This versions it so the agent definition evolves through reviewable PRs, and folds in the lessons from #16 / PR #17 (multi-platform Terraform lock).Changes
ae353d8— track the currentdevops.mdas-is (baseline, no behaviour change).48fc522— harden it:git fetch+ verify sync first, branch offmain→ PR back tomain, one issue = one PR, deliverable is a draft PR the agent creates itself, keep PRs scoped, and confirm CI is green (gh run list) before reporting done.providers lock+git diff --exit-code, notinit -lockfile=readonlyon a single OS.Why
During #16 the agent shipped a CI lock-check that only validated
linux_amd64(ubuntu runner), missing adarwin_arm64-only regression; it also worked off an unverified remote state. These defaults close both gaps so the agent needs less hand-holding on small tasks.How to verify
No runtime impact — docs/config only. Read the diff of
.claude/agents/devops.md.🤖 Generated with Claude Code