Skip to content

Conversation

@alwx
Copy link
Contributor

@alwx alwx commented Jan 5, 2026

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Adds AGENTS.md, fixes #5499
The added file is partially based on https://github.com/getsentry/sentry-cocoa/blob/main/AGENTS.md

#skip-changelog

@alwx alwx self-assigned this Jan 5, 2026
@lucas-zimerman
Copy link
Collaborator

is this only internal or clients will also use it?
Should we mark it with #skip-changelog and with the label ready-to-merge?

@antonis
Copy link
Contributor

antonis commented Jan 7, 2026

is this only internal or clients will also use it?

My understanding is that this would be used both internally and externally since it is open.

@lucas-zimerman
Copy link
Collaborator

lucas-zimerman commented Jan 7, 2026

My understanding is that this would be used both internally and externally since it is open.

Does it make sense to add it to the changelog?

@lucas-zimerman
Copy link
Collaborator

@sentry review

@antonis
Copy link
Contributor

antonis commented Jan 7, 2026

My understanding is that this would be used both internally and externally since it is open.

Does it make sense to add it to the changelog?

I do not have a strong opinion but I think not since it is targeting agents and not humans. Other platforms skipped the changelog too

Comment on lines +435 to +440
- **Expo**: Managed workflow support
- **Sentry Cocoa**: iOS native SDK
- **Sentry Java/Android**: Android native SDK
- **Flutter**: Flutter SDK depends on native SDKs
- **.NET**: .NET MAUI depends on native SDKs
- **Unity**: Unity SDK depends on native SDKs
Copy link
Collaborator

Choose a reason for hiding this comment

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

Q: shouldn't we only consider the impact on React Native and maybe cocoa/java?

Copy link
Contributor

Choose a reason for hiding this comment

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

+1 on this. We may also need to add the Javascript SDK here

@lucas-zimerman lucas-zimerman added the ready-to-merge Triggers the full CI test suite label Jan 7, 2026
Copy link
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

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

Looking good! Just a nit that I would avoid mentioning platforms unrelated to React Native.

Also, I am not familiar if the file size maters here. Other than that, thank youfor the PR!

Copy link
Contributor

@antonis antonis left a comment

Choose a reason for hiding this comment

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

LGTM 🙇

@alwx alwx merged commit 2462ed9 into main Jan 12, 2026
53 of 57 checks passed
@alwx alwx deleted the alwx/agents branch January 12, 2026 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Triggers the full CI test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add AGENTS.md with instructions for AI agents

4 participants