Skip to content

Add AGENTS.md and AI-assisted contribution guidance - #3366

Draft
mjcarroll wants to merge 1 commit into
rollingfrom
mjcarroll/add-agents-md
Draft

mjcarroll wants to merge 1 commit into
rollingfrom
mjcarroll/add-agents-md

Conversation

@mjcarroll

@mjcarroll mjcarroll commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Description

Adds an AGENTS.md aimed at AI agents and a short section in CONTRIBUTING.md linking the ROS guidelines for AI-assisted contributions, Making a PR and the Developer Guide.

The main points for agents: don't open a PR unless the human has built, tested and read the diff; never claim verification that wasn't run; use the PR template; sign-off and disclosure rules.

AGENTS.md is deliberately not rclcpp-specific so it can be reused across ROS 2 repos. Opening as a draft to collect feedback on wording and on whether we want to adopt it org-wide, along with a PR template check for closing non-conforming PRs.

Is this user-facing behavior change?

No, docs only.

Did you use Generative AI?

Claude (Sonnet 5.5).

Additional Information

Add AGENTS.md for AI agents and point to the ROS guidelines for AI-assisted contributions from CONTRIBUTING.md.

Signed-off-by: Michael Carroll <mjcarroll.oss@gmail.com>
@github-actions

Copy link
Copy Markdown

ABI Compliance Check

✅ Verdict: compatible

Library Verdict Summary
libcomponent_manager.so ✅ compatible No ABI changes detected.
librclcpp.so ✅ compatible No ABI changes detected.
librclcpp_action.so ✅ compatible No ABI changes detected.
librclcpp_lifecycle.so ✅ compatible No ABI changes detected.
✅ libcomponent_manager.so — full abidiff report

Compared:

  • Base: lib-base/libcomponent_manager.so
  • Head: lib-pr/libcomponent_manager.so @ 61484a3
(empty report — no differences printed by abidiff)
✅ librclcpp.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp.so
  • Head: lib-pr/librclcpp.so @ 61484a3
(empty report — no differences printed by abidiff)
✅ librclcpp_action.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp_action.so
  • Head: lib-pr/librclcpp_action.so @ 61484a3
(empty report — no differences printed by abidiff)
✅ librclcpp_lifecycle.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp_lifecycle.so
  • Head: lib-pr/librclcpp_lifecycle.so @ 61484a3
(empty report — no differences printed by abidiff)

Updated for commit 61484a3 · check run · suppressions: /home/runner/work/_temp/ros2-abi-suppressions.txt

@christophebedard christophebedard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I do think we should try this out. And we should try it out in a single repo for some time first

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants