Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 8 additions & 4 deletions .agents/rules/pr.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,10 +41,14 @@ Before drafting any pull request description, strictly adhere to the rules in
individual file modifications, test execution logs/checklists, or AI
disclosure metadata in PR descriptions or commit messages.
* **AI Disclosure & Comment Etiquette**: Keep the PR description clean for use
as the merge commit message. Disclose human oversight level in a separate PR
comment after creation. Never post comments for every CI run or narrate build
IDs, commit SHAs, line counts, or trivial diff details (e.g.,
"comment-only"); comment `PTAL` only when ready for re-review.
as the merge commit message. After creating a PR, post a separate comment so
maintainers understand who/what they are interacting with: state the level of
human oversight and explain how comments on the PR will be handled (whether
an agent autonomously monitors and replies, whether a human reviews/approves
agent responses before posting, or whether the human responds directly).
Never post comments for every CI run or narrate build IDs, commit SHAs, line
counts, or trivial diff details (e.g., "comment-only"); comment `PTAL` only
when ready for re-review.
* High-level overview only: state *why* the change is made and *how* at a
conceptual level. Link related issues (e.g. `Work towards #<issue>`).
* **Conciseness & Style (Strunk & White)**: Omit needless words. Use clear,
Expand Down
8 changes: 6 additions & 2 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,12 @@ PR Instructions/requirements
* Update docs and tests, and run `bazel test` and `buildifier` locally prior
to pushing.
* AI-assisted PRs:
* After creating the PR, post a separate comment disclosing the level of
human oversight (e.g., human-reviewed, semi-autonomous, or autonomous).
* After creating the PR, post a separate comment so maintainers understand
who/what they are interacting with: state the level of human oversight
(e.g., human-reviewed, semi-autonomous, or autonomous) and explain how
comments on the PR will be handled (e.g., whether an agent automatically
monitors and replies, whether a human reviews/approves agent replies before
posting, or whether the human responds directly).
* Do not post comments for every CI run or narrate build IDs, commit SHAs,
or trivial diff details. Reply `PTAL` once CI is passing and ready for
review.
Expand Down
8 changes: 6 additions & 2 deletions AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,12 @@ execution logs, or check checklists in the PR description.
and `buildifier` locally before opening or updating a PR.
* **Post AI disclosure as a PR comment (not in the PR description):** When an
AI agent creates or interacts on a PR, post a single comment after the PR is
created disclosing the level of human oversight (e.g., `human-reviewed`,
`semi-autonomous`, or `autonomous`).
created so maintainers understand who/what they are interacting with. State
the level of human oversight (e.g., `human-reviewed`, `semi-autonomous`, or
`autonomous`) and explain how comments on the PR will be handled (e.g.,
whether an agent automatically monitors and replies to review comments,
whether a human reviews and approves agent responses/changes before posting,
or whether the human author handles comments directly).
* **No CI status spam or commit narration:** Do not post comments for every CI
run, and do not narrate build IDs, commit SHAs, patch line/file counts, or
trivial diff details (e.g., stating a push is "comment-only"). GitHub
Expand Down
13 changes: 9 additions & 4 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -380,10 +380,15 @@ had written it entirely themselves. This means:
* **Vetting the correctness and functionality:** You are responsible for
thoroughly testing and verifying that the code is correct, functional, and
meets all project requirements and standards.
* **Disclosing human oversight in a PR comment:** Keep the PR description
clean for use as the merge commit message. After creating the PR, post a
comment disclosing the level of human oversight (e.g., closely
human-directed/reviewed, semi-autonomous, or fully autonomous).
* **Disclosing AI usage and comment handling in a PR comment:** Keep the PR
description clean for use as the merge commit message. After creating the
PR, post a comment so maintainers understand who or what they are
interacting with: disclose the level of human oversight (e.g., closely
human-directed/reviewed, semi-autonomous, or fully autonomous) and explain
how comments on the PR will be handled (e.g., whether an agent
automatically monitors and replies to comments, whether a human reviews and
approves agent responses before posting, or whether the human author
responds directly).
* **Avoiding bot comment spam:** Do not post comments for every CI run or
commit push, and do not narrate build IDs, commit SHAs, line counts, or
trivial diff details (such as noting a push is "comment-only"). Only post
Expand Down
Loading