Skip to content

meta(agents): Remove Claude co-author guidance #16803

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
szokeasaurusrex merged 4 commits into from
Mar 5, 2026
Merged

meta(agents): Remove Claude co-author guidance #16803

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
d6c6678
meta(agents): Remove Claude co-author guidance
szokeasaurusrex Mar 5, 2026
67fb13c
Merge branch 'master' into szokeasaurusrex/remove-claude-attribution
szokeasaurusrex Mar 5, 2026
d9ad7b2
Update AGENTS.md
szokeasaurusrex Mar 5, 2026
d3d26cc
Update AGENTS.md
szokeasaurusrex Mar 5, 2026
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
14 changes: 4 additions & 10 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,6 @@

Use **pnpm**: `pnpm install`, `pnpm dev`, `pnpm build`, `pnpm test`

## Commit Attribution
Copy link
Contributor

@stephanie-anderson stephanie-anderson Mar 5, 2026

Choose a reason for hiding this comment

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

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

Copy link
Member Author

@szokeasaurusrex szokeasaurusrex Mar 5, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

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?

Copy link
Member Author

@szokeasaurusrex szokeasaurusrex Mar 5, 2026

Choose a reason for hiding this comment

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

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 😄

Copy link
Contributor

@stephanie-anderson stephanie-anderson Mar 5, 2026

Choose a reason for hiding this comment

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

Yeah, I think the AGENTS.md should hint to use the commit skill instead


AI commits MUST include:

```
Co-Authored-By: Claude <noreply@anthropic.com>
```

## Development

- `pnpm dev` - Start dev server with Sentry sidecar
Expand Down Expand Up @@ -80,6 +72,8 @@ When writing requirements in `develop-docs/`:
- Make the plan extremely concise. Sacrifice grammar for the sake of concision.
- 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
Copy link
Member Author

@szokeasaurusrex szokeasaurusrex Mar 5, 2026

Choose a reason for hiding this comment

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

@stephanie-anderson does this look good?


When committing changes, use the Sentry commit skill if available.

Use .github/PULL_REQUEST_TEMPLATE.md and add Co-Authored-By: Claude
For PRs, use .github/PULL_REQUEST_TEMPLATE.md. Also, use the Sentry pull request skill if available.
Loading