meta(agents): Remove Claude co-author guidance#16803
Conversation
These lines cause AI agents, other than Claude Code, to incorrectly attribute Claude in commit messages.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
|
||
| Use **pnpm**: `pnpm install`, `pnpm dev`, `pnpm build`, `pnpm test` | ||
|
|
||
| ## Commit Attribution |
There was a problem hiding this comment.
There is an explicit guideline to add this to every commit message, see:
https://github.com/getsentry/skills/blob/main/plugins/sentry-skills/skills/commit/SKILL.md#ai-generated-changes
We should simply improve it, so the note is not hardcoded to Claude
There was a problem hiding this comment.
Not all AI agents have a specific commit email address, e.g. Codex lacks one as far as I am aware. I guess this suggestion would work for Cursor, though.
Maybe we can register a bot account to indicate commits generated by other AI agents? Something like ai-agent-<xyz>@noreply.sentry.io?
There was a problem hiding this comment.
Also, more fundamentally @stephanie-anderson -- if we already have this line in the skill (as you noted), then why should it also be included in AGENTS.md?
I think it is actually a bit redundant, and this guidance should just belong in the skill, and not be repeated here. Repeating here also means additional tokens/context are being used.
I don't feel super strongly about this, just want to question fundamental assumptions 😄
There was a problem hiding this comment.
Yeah, I think the AGENTS.md should hint to use the commit skill instead
|
@sergical I commented on #16597 My main complaint with that PR is it's adding a lot of text, which means the AGENTS.md will consume more context. And, this info is potentially redundant with the commit skill, as I discussed with @stephanie-anderson in the inline comment above |
|
@szokeasaurusrex i agree that we want to minimize the default context loaded in every agent session, but removing all of attribution guidelines breaks it for Cursor, and Claude Code which are the defacto supported agents at Sentry. Maybe the silver line is just having the 3 lines that are in https://github.com/getsentry/sentry-for-ai/blob/main/AGENTS.md#commit-attribution |
| - At the end of each plan, give me a list of unresolved questions to answer, if any. | ||
|
|
||
| ## Pull Request generation | ||
| ## Commit and Pull Request generation |
|
@sergical I have updated this PR to reference using the Sentry commit and PR skills which already contain the attribution line guidelines -- as folks should already be using that, I think this may be sufficient? |
|
@sergical see my comment on that PR, but in general, I think that's a sensible direction 👍 |
3 participants
DESCRIBE YOUR PR
These lines cause AI agents, other than Claude Code, to incorrectly attribute Claude in commit messages.
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.