You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "kevinjqliu (via GitHub)" <gi...@apache.org> on 2026/03/25 05:36:03 UTC
[PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
kevinjqliu opened a new pull request, #15758:
URL: https://github.com/apache/iceberg/pull/15758
Follow up to #15529
Add a section for self-disclosure, esp when PRs themselves are created by an agent.
I have seen several newly opened PR that seem to be fully generated; it would be nice to have some kind of indication on the PR description.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4128435526
I generally don't have strong feelings about this in Agent.MD
On the con side, everything we put in here is essentially a cost in tokens for all users so I want to make sure it's useful
On the pro side, it would nice to auto mark
I would recommend we go over the text though and have less "if then" comments and more straight instructions.
No need to say "if you are doing this with ai" just say "PR's include this disclosure box after the description"
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "huaxingao (via GitHub)" <gi...@apache.org>.
huaxingao commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4128027544
Thanks @kevinjqliu for the PR! Should we also talk about AI generated reviews? Maybe add something like "Similarly, if you use AI tools to review PRs, please review the comments for accuracy before posting. Auto-posting AI-generated comments without human review is discouraged."
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989899160
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
Review Comment:
Another full replacement here
```When creating a PR description, always include an AI Disclosure block at the bottom. Use the template below and fill in the fields you know (Model, Platform/Tool). For fields you cannot determine (Human Oversight), use [unknown - human to fill in].```
Or something like that. Explicit instructions. Minimal branching or ambiguous conditions
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "nssalian (via GitHub)" <gi...@apache.org>.
nssalian commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4685617475
@RussellSpitzer @huaxingao @jbonofre PTAL
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989905792
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
Review Comment:
Here would be sometihng like "Set Model to your Model Identifier" or whnot
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "qzyu999 (via GitHub)" <gi...@apache.org>.
qzyu999 commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4414272312
> > is possible to add a (nearly empty) CLAUDE.md
>
> i remember `AGENTS.md` was chosen to be vendor agnostic, i think claude can also read from `AGENTS.md`
Hi @kevinjqliu, from their own docs, https://code.claude.com/docs/en/memory#agents-md, they mention that Claude won't read it automatically:
> Claude Code reads CLAUDE.md, not AGENTS.md. If your repository already uses AGENTS.md for other coding agents, create a CLAUDE.md that imports it so both tools read the same instructions without duplicating them. You can also add Claude-specific instructions below the import. Claude loads the imported file at session start, then appends the rest:
for example, CLAUDE.md:
> @AGENTS.md
> ## Claude Code
>
> Use plan mode for changes under `src/billing/`.
I don't use Claude Code directly myself, but it sounds to me like Claude Code users would benefit from a `CLAUDE.md` file that simply has the line: `@AGENTS.md` and it would solve the problem for them.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "qzyu999 (via GitHub)" <gi...@apache.org>.
qzyu999 commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4414164072
Hi @kevinjqliu, thanks for the AGENTS.md update. A bit off-topic, but it is possible to add a (nearly empty) `CLAUDE.md` and have it symlink (via `ln -s AGENTS.md CLAUDE.md`) so Claude code users can benefit, the same way it's done in Airflow: https://github.com/apache/airflow/blob/main/CLAUDE.md. This may deserve a separate issue in itself though.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989756746
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
+
+### Optional Fields
+
+- **Prompt Summary**: A brief description of the prompt or task given to the agent.
+- **Agent Version**: Version of the orchestration tool or agent framework, if applicable.
+- **Iteration Count**: How many agent iterations or attempts were used to produce the final result.
+- **Context Window**: Any notable context limitations that may affect completeness.
+
+### PR Description Template
+
+Add the following block at the bottom of AI-generated PR descriptions:
Review Comment:
I'm not sure how any of this will reliably work _especially filling in model name and version
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989924886
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
Review Comment:
Pick one synonym I think. I don't have the context on this, but I can only imagine it would be harder for a model with synonyms rather than just using a single term over and over
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4199153518
That's good. I think the ash header/copyright should be documented here if the code is AI generated. See the asf guidelines.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4665449699
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989783876
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
+
+### Optional Fields
+
+- **Prompt Summary**: A brief description of the prompt or task given to the agent.
+- **Agent Version**: Version of the orchestration tool or agent framework, if applicable.
+- **Iteration Count**: How many agent iterations or attempts were used to produce the final result.
+- **Context Window**: Any notable context limitations that may affect completeness.
+
+### PR Description Template
+
+Add the following block at the bottom of AI-generated PR descriptions:
Review Comment:
Also you could just remove the "AI-generated" here
"Add the following block to the bottom of PR descriptions"
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989918591
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
+
+### Optional Fields
Review Comment:
Probably we don't need to add Optional category. This is for a robot, just tell it to include everything if we think it's important with instructions for missing fields.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "nssalian (via GitHub)" <gi...@apache.org>.
nssalian commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989548856
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
+
+### Optional Fields
Review Comment:
I'm curious if these fields are useful to add. What information here would help the community?
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
+
+### Optional Fields
+
+- **Prompt Summary**: A brief description of the prompt or task given to the agent.
+- **Agent Version**: Version of the orchestration tool or agent framework, if applicable.
+- **Iteration Count**: How many agent iterations or attempts were used to produce the final result.
+- **Context Window**: Any notable context limitations that may affect completeness.
+
+### PR Description Template
+
+Add the following block at the bottom of AI-generated PR descriptions:
Review Comment:
We should add this as a section in the PR template so users are prompted to add these if needed.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r3045158579
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
Review Comment:
If the code is generated, the copyright doesn't apply.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4413154674
thanks for the reviews!
picking this back up, i've edited the instructions based on reviews. Added a sentence on PR review as well.
I'll create a separate PR for the pull request template
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4402222010
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989758660
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
+
+### Optional Fields
+
+- **Prompt Summary**: A brief description of the prompt or task given to the agent.
+- **Agent Version**: Version of the orchestration tool or agent framework, if applicable.
+- **Iteration Count**: How many agent iterations or attempts were used to produce the final result.
+- **Context Window**: Any notable context limitations that may affect completeness.
+
+### PR Description Template
+
+Add the following block at the bottom of AI-generated PR descriptions:
+
+```
+---
+**AI Disclosure**
+- AI-Generated: Yes
Review Comment:
This isn't needed, either this section is here or not IMHO
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989911008
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
Review Comment:
Prompt the user to fill this in or leave it as unreviewed
Or something like that
We don't need to describe, we need to instruct
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "manuzhang (via GitHub)" <gi...@apache.org>.
manuzhang commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4131538252
Why don't we add such a section to the PR template like https://github.com/apache/spark/blob/master/.github/PULL_REQUEST_TEMPLATE?
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4199166225
For reference, here's the updated guidelines https://www.apache.org/legal/generative-tooling.html
Included the Generated-By mention
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4413177032
added pr template in https://github.com/apache/iceberg/pull/16267
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4414255993
> is possible to add a (nearly empty) CLAUDE.md
i remember `AGENTS.md` was chosen to be vendor agnostic, i think claude can also read from `AGENTS.md`
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4737461138
merging this, we can keep iterating on it. thanks everyone!
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu merged PR #15758:
URL: https://github.com/apache/iceberg/pull/15758
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989777430
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
+
+### Optional Fields
+
+- **Prompt Summary**: A brief description of the prompt or task given to the agent.
+- **Agent Version**: Version of the orchestration tool or agent framework, if applicable.
+- **Iteration Count**: How many agent iterations or attempts were used to produce the final result.
+- **Context Window**: Any notable context limitations that may affect completeness.
Review Comment:
I'm not sure how 179 or 180 would accurately produced by some agents (or whether that output could be trusted) without an explicit skill or tool to fetch the information
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989902990
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
Review Comment:
I don't think we need a **AI-Generated** field, if the disclosure is here we can assume it was
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989770260
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
Review Comment:
The grammar here is a bit off.
```suggestion
Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** be labeled as "AI-Generated". Include the following information in the PR description:
```
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989913150
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
+
+### Optional Fields
+
+- **Prompt Summary**: A brief description of the prompt or task given to the agent.
+- **Agent Version**: Version of the orchestration tool or agent framework, if applicable.
+- **Iteration Count**: How many agent iterations or attempts were used to produce the final result.
+- **Context Window**: Any notable context limitations that may affect completeness.
Review Comment:
Needs more instructions less description
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "nssalian (via GitHub)" <gi...@apache.org>.
nssalian commented on code in PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#discussion_r2989548856
##########
AGENTS.md:
##########
@@ -161,6 +161,38 @@ The `api/` module has the strongest stability guarantees — breaking changes ar
- Commit messages describe the *what* and *why*, not implementation details.
- Apache License header required on all new files (enforced by spotless pre-commit hook).
+## AI-Generated PR Disclosure
+
+Pull requests that are authored or substantially generated by AI, LLM, or automated agents **must** clearly identify themselves. Include the following information in the PR description:
+
+### Required Fields
+
+- **AI-Generated**: `Yes` — explicitly state that the PR was generated with AI/LLM assistance.
+- **Model**: The specific model used (e.g., `Claude Opus 4.6`, `GPT-4o`, `Gemini 2.5 Pro`). Include the version or snapshot date if available.
+- **Platform/Tool**: The tool or platform that orchestrated the generation (e.g., `GitHub Copilot`, `Cursor`, `Aider`, `Claude Code`, `custom script`).
+- **Human Oversight**: Describe the level of human review applied (e.g., `fully reviewed`, `partially reviewed`, `unreviewed`).
+
+### Optional Fields
Review Comment:
I'm curious if these fields are useful to add. Would this information here help the community? I feel like we could remove these
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] docs: add a "AI-Generated PR Disclosure" section to AGENTS.md [iceberg]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on PR #15758:
URL: https://github.com/apache/iceberg/pull/15758#issuecomment-4737285437
made an edit to include the ASF policy
> See the [ASF generative tooling guidelines](https://www.apache.org/legal/generative-tooling.html) for additional policy on AI-generated contributions. Generated files must still follow Iceberg's Apache License header and license check requirements unless explicitly excluded by project policy. For commits authored using generative AI tooling, include a `Generated-by: <tool>` token in the commit message.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org