codemod(flex): apply changes for @getsentry/alerts-create-issues #106840
Conversation
github-actions
bot
added
the
Scope: Frontend
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| <Container marginBottom="2xl"> | ||
| <HeaderTitleLegend>{t('Alerts Triggered')}</HeaderTitleLegend> | ||
| </ChartHeader> | ||
| </Container> |
There was a problem hiding this comment.
Spacing value changed from 20px to 24px
Low Severity
The codemod changed the chart header's bottom margin from space(3) (20px) to marginBottom="2xl" (24px), a 4px increase. While this may be intentional as part of the design system migration, the PR description doesn't mention visual changes, and this spacing difference could unintentionally affect the chart's layout.
| @@ -1,4 +1,4 @@ | |||
| import styled from '@emotion/styled'; | |||
| import {Container} from '@sentry/scraps/layout'; | |||
There was a problem hiding this comment.
Inconsistent import paths for Container component
Low Severity
The codemod introduces Container imports using @sentry/scraps/layout in email.tsx and assignedTo.tsx, but the majority of files in views/automations (7 of 11 files) use sentry/components/core/layout for layout imports. Sibling files like discord.tsx and slack.tsx use sentry/components/core/layout, creating an inconsistent import pattern that reduces code maintainability.
Applies the flex codemod to files owned by @getsentry/alerts-create-issues.