Skip to content

Carry the coding-agents shared rules in AGENTS.md - #62

Merged
slominskir merged 1 commit into
mainfrom
shared-rules-6.1.0
Oct 17, 2026
Merged

slominskir merged 1 commit into
mainfrom
shared-rules-6.1.0

Conversation

@slominskir-coding-agent

Copy link
Copy Markdown
Contributor

What changed

AGENTS.md now carries the coding-agents shared rules block, version 6.1.0, between its BEGIN/END markers. This is the one-time catch-up from the hand-made copy of the starter's generic rules to the versioned block, so later releases can replace the block as a whole. It brings in the rules this file lacked: task authorization, the Prepared-by: footer on agent-filed issues, the review conventions, rewording the first message, and commit trailers.

  • Development: unchanged, except for three lines the block now covers: the intro about reporting what you ran, "add or update tests with each change", and "never commit secrets".
  • Commit identity, Commits, Branches and pull requests: replaced by the block.
  • Project rules (after the block): the VERSION release rule and the organization project note, as before, and a GitHub note: where the shared rules say merge request, read pull request, and for its pipeline, its checks (build and integration from CI, and CodeQL).

No deployment steps.

Checks

  • The text between the markers is identical to the block in coding-agents' v6.1.0 starter, and the sync script's marker check (block() in scripts/sync-shared-rules.py) parses the file: one block, version 6.1.0.
  • No Gradle checks run: Spotless covers only Java, and only AGENTS.md changed.

Decisions

  • 6.1.0, not 6.2.0: coding-agents' main has a 6.2.0 block that hasn't been released yet, and the sync script uses the latest release, as the other projects got.
  • "Add or update tests with each change" is dropped in favor of the shared rule, which asks for tests in proportion to the change's risk. Say if you want the stricter rule kept under Project rules.
  • A GitHub App PR, not a hand edit: coding-agents says GitHub projects get the block by hand, because the VM's App can't open PRs on projects it isn't installed on. It is installed on JeffersonLab, so it can open this one.

Part of acc/iac/docs/coding-agents#27 (https://code.jlab.org/acc/iac/docs/coding-agents/-/issues/27).

🤖 Generated with Claude Code

AGENTS.md held a hand-made copy of the starter's generic rules, from
before the starter had a versioned shared block, so it lacked the
rules added since: task authorization, the Prepared-by footer on
issues, the review conventions, and rewording the first message.
Replace those generic rules with the shared rules block of
coding-agents 6.1.0, between its markers, so later releases can
replace it as a whole.

Keep the project's own rules after it, under Project rules: the
VERSION release rule, the organization project note, and which checks
a pull request waits for. Add a line saying the shared rules' merge
requests and pipelines are pull requests and their checks here. The
Development section is unchanged except for the lines the block now
covers (report what you ran, add tests, never commit secrets).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@slominskir
slominskir merged commit 7ea8c7a into main Oct 17, 2026
6 checks passed
@slominskir
slominskir deleted the shared-rules-6.1.0 branch October 17, 2026 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

source::ai Work done by an AI agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant