Skip to content

docs: add AGENTS.md, fix v0.1 language matrix, clarify versioning#41

Merged
boorad merged 3 commits into
mainfrom
docs/agents-md-and-versioning
May 4, 2026
Merged

docs: add AGENTS.md, fix v0.1 language matrix, clarify versioning#41
boorad merged 3 commits into
mainfrom
docs/agents-md-and-versioning

Conversation

@boorad
Copy link
Copy Markdown
Contributor

@boorad boorad commented May 4, 2026

Summary

Three docs fixes bundled into a single branch:

  • AGENTS.md as canonical agent doc — adds AGENTS.md (the convention used by Codex/Aider/Cursor) and converts CLAUDE.md to a relative symlink so Claude Code keeps working without duplicating content.
  • Correct v0.1 language matrixREADME.md (and the new AGENTS.md) now reflect that Python and Go structural support shipped in v0.1.5; planned languages (C#, Kotlin, Ruby, PHP) move from v0.3 → v0.2.
  • Versioning section — explains Cargo's 0.x SemVer convention (minor acts as major pre-1.0), with a bump table that maps feat: / feat!: / fix: to actual bumps both pre- and post-1.0. Placeholders use 0.M.p (pre-1.0) and m.n.p (post-1.0) so the same letter never means two different positions. Calls out that a plain feat: on 0.1.x will not produce 0.2.0feat!: or a BREAKING CHANGE: footer is required.
  • /address-pr-feedback clarification — Amazon Q does not auto-trigger on push; users must post /q review as a PR comment to request a (re-)review.

Changes

  • AGENTS.md — new, canonical agent instructions
  • CLAUDE.md — symlink → AGENTS.md
  • README.md — language matrix corrected, status callout updated
  • .claude/commands/address-pr-feedback.md — explicit /q review trigger guidance

Test plan

  • cat CLAUDE.md resolves to AGENTS.md contents (symlink works locally)
  • Render AGENTS.md on GitHub — bump table renders cleanly with the new placeholders
  • Render README.md on GitHub — supported-languages table shows Go as yes (v0.1)
  • Confirm release-plz still picks up the next feat: / feat!: correctly on its next run (no config changes here, but the doc claims should hold)

Summary by CodeRabbit

  • Documentation
    • Updated Amazon Q bot instructions to clarify that it does not auto-trigger and requires /q review command to initiate reviews.
    • Added comprehensive developer guidelines document covering setup, build requirements, architecture, design principles, and contribution standards.
    • Updated project status to reflect Go structural scanning as now available (previously planned).
    • Expanded language roadmap with additional planned language support.

boorad added 3 commits May 4, 2026 11:46
Python and Go structural support landed in v0.1.5 via plain `feat:` PRs,
but README and CLAUDE.md still listed Go as planned for v0.2 and Python's
status was inconsistent. Make AGENTS.md the canonical agent-instructions
file (so non-Claude agents pick it up) and point CLAUDE.md at it via
symlink.

Also document the Cargo 0.x SemVer carve-out: while < 1.0.0, plain `feat:`
is a patch bump; `feat!:` is required to cut 0.2.0. The next language
batch (C#, Kotlin, Ruby, PHP) is shifted to v0.2 and will land with
`feat!:` so release-plz actually bumps the minor.
The previous wording told users to "wait for AQ's review" if none was
present, but AQ doesn't auto-trigger on push (or at all, absent the org
integration on initial open). Waiting alone never produces a review.

Spell out the actual lever: post `/q review` as a PR comment. Apply the
same clarification to the commit-lag section — if a user wants AQ to
re-review the latest commit, they have to ask explicitly; AQ will not
catch up on its own.
Pre-1.0 and post-1.0 columns now use distinct placeholders (`0.M.p`
vs `m.n.p`) so the same letter never refers to two different version
positions. Also add a "see bump table" pointer to the trigger-prefix
bullet list, since on Cargo 0.x `fix:` and `feat:` both land as patch
bumps and the bullet list alone is misleading.
@boorad boorad self-assigned this May 4, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 4801e37e-8dde-4a1c-bf2e-bb08aae424a2

📥 Commits

Reviewing files that changed from the base of the PR and between e072c49 and e91a605.

📒 Files selected for processing (5)
  • .claude/commands/address-pr-feedback.md
  • AGENTS.md
  • CLAUDE.md
  • CLAUDE.md
  • README.md

📝 Walkthrough

Walkthrough

This PR updates project documentation and bot guidance: it introduces a new AGENTS.md file with agent setup and architecture instructions, references it from CLAUDE.md, clarifies Amazon Q bot behavior in .claude/commands/address-pr-feedback.md to indicate manual trigger requirements, and updates the README to mark Go as structurally supported and list planned languages.

Changes

Documentation and Agent Guidance Updates

Layer / File(s) Summary
New Agent Documentation
AGENTS.md
Comprehensive agent setup guide covering pre-commit hooks, build/test/lint commands, architecture overview (scanner, rules, Rego, output), design principles (two-pass scanning, schema consistency), language support status, roadmap, and Conventional Commits with release-plz versioning rules.
Documentation References
CLAUDE.md
Added reference to the new AGENTS.md file as the canonical agent instructions.
Bot Behavior Clarification
.claude/commands/address-pr-feedback.md
Clarified that Amazon Q does not auto-trigger; users must post /q review as a PR comment if no activity is detected. Updated guidance that commit_id mismatch should not block processing and that fresh reviews require explicit /q review posting.
Project Status Updates
README.md
Updated Status badge to include Go among structurally scanning ready languages. Marked Go as structurally supported in v0.1 (instead of planned) and added planned entries for C#, Kotlin, Ruby, and PHP with v0.2 structural support timelines.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • EnforceAuth/mono#179: Updates Claude PR-review-bot documentation with Amazon Q review behavior and /q review interaction guidance.
  • EnforceAuth/mono#469: Modifies .claude/commands/address-pr-feedback.md to adjust bot reply timing and commit SHA verification instructions for post-push scenarios.
  • feat: add CI/CD workflows and Claude Code commands ea-commerce#1: Introduces the same command file that this PR updates, establishing the bot feedback handling foundation.

Poem

🐰 A burrow of docs, now neat and bright,
Agents shall find their path with delight,
Amazon Q waits for a kind /q review,
Go joins the fold in structurally-scanned hue,
With Conventional Commits and release-plz's might! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately captures the three main changes: adding AGENTS.md, fixing the v0.1 language matrix, and clarifying versioning guidance—all of which are reflected in the raw_summary and PR objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Review rate limit: 4/5 reviews remaining, refill in 12 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@amazon-q-developer amazon-q-developer Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation updates look good. The changes correctly reflect v0.1 Go support, clarify Amazon Q's manual trigger requirement, and establish AGENTS.md as the canonical agent instructions file with CLAUDE.md as a symlink. No defects found that block merge.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

Copy link
Copy Markdown

@amazon-q-developer amazon-q-developer Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation updates look good. The changes correctly reflect v0.1 Go support, clarify Amazon Q's manual trigger requirement, and establish AGENTS.md as the canonical agent instructions file with CLAUDE.md as a symlink. No defects found that block merge.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@boorad boorad merged commit aa046d9 into main May 4, 2026
3 checks passed
@boorad boorad deleted the docs/agents-md-and-versioning branch May 4, 2026 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant