-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
chore(ACI): Remove references to SentryAppIdentifier #107887
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.
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
static/app/views/automations/components/automationBuilderContext.tsx
Outdated
Show resolved
Hide resolved
static/app/views/automations/components/automationBuilderContext.tsx
Outdated
Show resolved
Hide resolved
static/app/views/automations/components/automationBuilderContext.tsx
Outdated
Show resolved
Hide resolved
| sentryAppIdentifier === SentryAppIdentifier.SENTRY_APP_INSTALLATION_UUID && | ||
| targetIdentifier === sentryApp?.installationUuid; | ||
| return isMatchingAppId || isMatchingInstallationUuid; | ||
| return targetIdentifier === sentryApp?.id; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Creating new Sentry App actions will fail because the frontend no longer sends the required sentry_app_identifier field, which the backend still expects for validation.
Severity: HIGH
Suggested Fix
Update the getDefaultConfig() function in automationBuilderContext.tsx to include the sentryAppIdentifier field in the returned configuration object for new Sentry App actions. This will align the frontend payload with the backend's required schema and allow new actions to be saved successfully.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.
Location: static/app/views/automations/components/actionNodeList.tsx#L51
Potential issue: When creating a new Sentry App action, the frontend configuration
generated by `getDefaultConfig()` in `automationBuilderContext.tsx` no longer includes
the `sentryAppIdentifier` field. However, the backend schema defined in
`sentry_app_handler.py` still lists `sentry_app_identifier` as a required field.
Consequently, any attempt to save a new Sentry App action from the user interface will
result in a backend validation error, preventing users from successfully creating these
automations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will not be merged until #107174 is merged first.
Now that we've migrated the
Actiondata to only ever have a sentry app id (and not an installation uuid) we can remove references to the sentry app identifier as it was only used to differentiate between actions with a sentry app id and actions with a sentry app installation uuid.Closes ISWF-2034