Expand agent operating policy: commit/PR workflow and code-review guidance#139
Merged
Conversation
Add explicit `Committing and Pushing` and `Creating a Pull Request` procedures, a `Bug Fixes` section that requires fixing the root cause and covering it with a test, and revise the clarification-question rules to ask one focused question at a time. Authorization and version/report requirements defer to their owning sections, and branch guidance documents the kebab-case convention rather than only prohibiting agent identifiers. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Clarify that review is read-only by default, add a dedicated PR review procedure, flag workarounds that mask a root cause, require a regression test only when behavior changes, and inspect all checks rather than the required-only filter. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
armiol
approved these changes
Jul 24, 2026
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
2 participants
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.
Summary
Formalizes the agent-facing operating policy so commits, pull requests, bug
fixes, and code reviews follow explicit, consistent procedures.
Changes
AGENTS.mdCommitting and Pushingprocedure: per-turn authorization (deferringto
Commit and History Safety), branch offmasterin the repository'skebab-case style, verify the
chordsVersionbump and regenerate thereports per
Versioning and Reports, commit in one or more logical stepswith the established
Bump version —>message, push, and offer a PR.Creating a Pull Requestprocedure: draft PR, basemaster,self-assign,
## Summary/## Changesbody, and closing keywords.Bug Fixessection: fix the root cause over a workaround, and coverthe fix with a failing-first test.
Safety Rulesbullet prohibiting references to dependent privateprojects in PRs and issues (project names, domain/business logic, linked
issues/PRs/repositories, or internal identifiers).
Planning and Questionsto ask one focused clarification questionper message.
.agents/skills/code-reviewer/SKILL.mdRunning Checksnote (review is read-only by default; read existingCI results instead of re-running the pipeline).
PR Review Procedurecovering PR context, scope confirmation,reading files at the PR head, and check-status handling that inspects all
checks (not just required).
focus on workarounds that mask a root cause.
Reviewer notes
The
pom.xmlanddependencies.mddiffs are limited to the version-header andgeneration-timestamp lines (
.92→.93); no dependencies changed. Theversion was bumped to
2.0.0-SNAPSHOT.93and the reports regenerated via./gradlew generateLicenseReport mergeAllLicenseReports generatePom.