Conversation
|
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. |
Does it make sense to add it to the changelog? |
|
@sentry review |
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 |
| - **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 |
There was a problem hiding this comment.
Q: shouldn't we only consider the impact on React Native and maybe cocoa/java?
There was a problem hiding this comment.
+1 on this. We may also need to add the Javascript SDK here
lucas-zimerman
left a comment
lucas-zimerman
left a comment
There was a problem hiding this comment.
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!
3 participants
📢 Type of change
📜 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