laskoviymishka commented on code in PR #17671: URL: https://github.com/apache/iceberg/pull/17671#discussion_r3816992401
########## AGENTS.md: ########## @@ -197,6 +197,25 @@ Add the following block at the bottom of PR descriptions: - Prompt Summary: [brief description of the task] ``` +### AI-Assisted Review Comments + +- A human **must** read, understand, and endorse any AI-derived review comment before it is posted, and posts it under their own account. If a point cannot be verified, it is not posted. +- Keep each comment to the actionable point: **what** is wrong, **where** (file/line/symbol), and **the suggested fix**. Drop multi-paragraph rationale dumps. +- **Automated bot reviewers** (CI jobs, GitHub Actions, bots, or app integrations): **at most one** per PR. Do not stack multiple bot review threads. Bots **must not** post AI-generated comments directly; a human must read, endorse, and post them under their own account. +- That "at most one" limit applies only to automated bots. It does **not** limit how many human reviewers may use AI. + +### Crediting AI Tools + +- Disclose AI assistance with the `Generated-by: <tool>` commit token, **not** with `Co-authored-by:` trailers. Review Comment: Why is this needed? ########## AGENTS.md: ########## @@ -206,3 +225,4 @@ Add the following block at the bottom of PR descriptions: - **Never** commit secrets, credentials, or cloud-specific tokens. - **Ask first** before adding new third-party dependencies (license compatibility matters). - **Ask first** before promoting package-private classes/methods to public. +- **AI-authored PRs** must pass the same review, test, and security/static-analysis gates as human-authored PRs. Review Comment: This is not needed, we don't treat PR-s differently here. ########## AGENTS.md: ########## @@ -197,6 +197,25 @@ Add the following block at the bottom of PR descriptions: - Prompt Summary: [brief description of the task] ``` +### AI-Assisted Review Comments + +- A human **must** read, understand, and endorse any AI-derived review comment before it is posted, and posts it under their own account. If a point cannot be verified, it is not posted. +- Keep each comment to the actionable point: **what** is wrong, **where** (file/line/symbol), and **the suggested fix**. Drop multi-paragraph rationale dumps. Review Comment: Each reviewer decides on their own how to structure review. This is not needed. ########## AGENTS.md: ########## @@ -197,6 +197,25 @@ Add the following block at the bottom of PR descriptions: - Prompt Summary: [brief description of the task] ``` +### AI-Assisted Review Comments + +- A human **must** read, understand, and endorse any AI-derived review comment before it is posted, and posts it under their own account. If a point cannot be verified, it is not posted. Review Comment: Last sentence is confusing, can you expand. ########## AGENTS.md: ########## @@ -197,6 +197,25 @@ Add the following block at the bottom of PR descriptions: - Prompt Summary: [brief description of the task] ``` +### AI-Assisted Review Comments + +- A human **must** read, understand, and endorse any AI-derived review comment before it is posted, and posts it under their own account. If a point cannot be verified, it is not posted. +- Keep each comment to the actionable point: **what** is wrong, **where** (file/line/symbol), and **the suggested fix**. Drop multi-paragraph rationale dumps. +- **Automated bot reviewers** (CI jobs, GitHub Actions, bots, or app integrations): **at most one** per PR. Do not stack multiple bot review threads. Bots **must not** post AI-generated comments directly; a human must read, endorse, and post them under their own account. Review Comment: Bots (and AI assisted reviews) can chain comment threads. Having this context for each review iteration is helpful. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
