🛠️ apply pre-commit fixes

getsentry · Aug 9, 2024 · 967d72f · 967d72f
1 parent ef20f80
commit 967d72f
Show file tree

Hide file tree

Showing 37 changed files with 213 additions and 194 deletions.
diff --git a/static/app/components/charts/baseChart.tsx b/static/app/components/charts/baseChart.tsx
@@ -597,7 +597,7 @@ function BaseChartUnwrapped({
   //
   const eventsMap = useMemo(
     () =>
-      (({
+      ({
         click: (props, instance) => {
           handleClick(props, instance);
           onClick?.(props, instance);
@@ -616,8 +616,8 @@ function BaseChartUnwrapped({
 
         brush: (props, instance) => onBrushStart?.(props, instance),
         brushend: (props, instance) => onBrushEnd?.(props, instance),
-        brushselected: (props, instance) => onBrushSelected?.(props, instance)
-      }) as ReactEchartProps['onEvents']),
+        brushselected: (props, instance) => onBrushSelected?.(props, instance),
+      }) as ReactEchartProps['onEvents'],
     [
       onClick,
       onHighlight,

diff --git a/static/app/components/charts/utils.tsx b/static/app/components/charts/utils.tsx
@@ -230,7 +230,9 @@ export function isMultiSeriesStats(
 ): data is MultiSeriesEventsStats {
   return (
     // the isSingleSeriesStats check is for topN queries returning null data
-    (defined(data) && ((data.data === undefined && data.totals === undefined) || (defined(isTopN) && isTopN && defined(data) && !isSingleSeriesStats(data))))
+    defined(data) &&
+    ((data.data === undefined && data.totals === undefined) ||
+      (defined(isTopN) && isTopN && defined(data) && !isSingleSeriesStats(data)))
   );
 }
 

diff --git a/static/app/components/clippedBox.spec.tsx b/static/app/components/clippedBox.spec.tsx
@@ -40,14 +40,14 @@ class MockResizeObserver {
 
 function mockGetBoundingClientRect({height}: {height: number}) {
   window.HTMLDivElement.prototype.getBoundingClientRect = () =>
-    (({
+    ({
       bottom: 0,
       height,
       left: 0,
       right: 0,
       top: 0,
-      width: 0
-    }) as DOMRect);
+      width: 0,
+    }) as DOMRect;
 }
 
 function clearMockGetBoundingClientRect() {

diff --git a/static/app/components/deprecatedforms/form.tsx b/static/app/components/deprecatedforms/form.tsx
@@ -132,7 +132,7 @@ class Form<
     const nonFieldErrors = this.state.errors?.non_field_errors;
 
     return (
-      (<FormContext.Provider value={this.getContext()}>
+      <FormContext.Provider value={this.getContext()}>
         <StyledForm
           onSubmit={this.onSubmit}
           className={this.props.className}
@@ -150,7 +150,7 @@ class Form<
                   <ul>
                     {nonFieldErrors.map((e, i) => (
                       // TODO(TS): Objects cannot be rendered to dom
-                      (<li key={i}>{e as any}</li>)
+                      <li key={i}>{e as any}</li>
                     ))}
                   </ul>
                 </div>
@@ -182,7 +182,7 @@ class Form<
             {this.props.extraButton}
           </div>
         </StyledForm>
-      </FormContext.Provider>)
+      </FormContext.Provider>
     );
   }
 }

diff --git a/static/app/components/events/eventReplay/index.spec.tsx b/static/app/components/events/eventReplay/index.spec.tsx
@@ -25,9 +25,9 @@ jest.mock('sentry/utils/replays/hooks/useReplayOnboarding');
 jest.mock(
   'sentry/components/events/eventReplay/replayClipPreview',
   () =>
-    (function() {
+    function () {
       return <div data-test-id="replay-clip" />;
-    })
+    }
 );
 
 const mockEventTimestamp = new Date('2022-09-22T16:59:41Z');

diff --git a/static/app/components/events/interfaces/crashContent/exception/utils.tsx b/static/app/components/events/interfaces/crashContent/exception/utils.tsx
@@ -22,9 +22,10 @@ export function isFrameFilenamePathlike(frame: Frame): boolean {
   return (
     // If all filenames are anonymous, we do not want to show this alert
     // If all absolute paths do not have a file extension, we do not want to show this alert
-    ((frame.filename === '<anonymous>' && frame.inApp) ||
+    (frame.filename === '<anonymous>' && frame.inApp) ||
     // If all function names are on the blocklist, we do not want to show this alert
-    fileNameBlocklist.includes(frame.function ?? '') || (!!frame.absPath && !getFileExtension(filename)))
+    fileNameBlocklist.includes(frame.function ?? '') ||
+    (!!frame.absPath && !getFileExtension(filename))
   );
 }
 

diff --git a/static/app/components/events/interfaces/spans/newTraceDetailsSpanTree.tsx b/static/app/components/events/interfaces/spans/newTraceDetailsSpanTree.tsx
@@ -800,13 +800,13 @@ class NewTraceDetailsSpanTree extends Component<PropType> {
       spanTree.push({type: SpanTreeNodeType.MESSAGE, element: limitExceededMessage});
 
     return (
-      (<TraceViewContainer>
+      <TraceViewContainer>
         <WindowScroller onScroll={this.throttledOnScroll}>
           {({height, isScrolling, onChildScroll, scrollTop, registerChild}) => (
             <AutoSizer disableHeight>
               {({width}) => (
                 // TODO(TS): registerChild expects a ReactNode instead of a HTML ref
-                (<div ref={el => registerChild(el as any)}>
+                <div ref={el => registerChild(el as any)}>
                   <ReactVirtualizedList
                     autoHeight
                     isScrolling={isScrolling}
@@ -820,12 +820,12 @@ class NewTraceDetailsSpanTree extends Component<PropType> {
                     rowRenderer={props => this.renderRow(props, spanTree)}
                     ref={listRef}
                   />
-                </div>)
+                </div>
               )}
             </AutoSizer>
           )}
         </WindowScroller>
-      </TraceViewContainer>)
+      </TraceViewContainer>
     );
   }
 }

diff --git a/static/app/components/events/interfaces/spans/scrollbarManager.tsx b/static/app/components/events/interfaces/spans/scrollbarManager.tsx
@@ -546,7 +546,7 @@ export const Consumer = ScrollbarManagerContext.Consumer;
 export const withScrollbarManager = <P extends ScrollbarManagerChildrenProps>(
   WrappedComponent: React.ComponentType<P>
 ) =>
-  (class extends Component<
+  class extends Component<
     Omit<P, keyof ScrollbarManagerChildrenProps> & Partial<ScrollbarManagerChildrenProps>
   > {
     static displayName = `withScrollbarManager(${getDisplayName(WrappedComponent)})`;
@@ -565,4 +565,4 @@ export const withScrollbarManager = <P extends ScrollbarManagerChildrenProps>(
         </ScrollbarManagerContext.Consumer>
       );
     }
-  });
+  };
diff --git a/static/app/components/feedback/list/useMailboxCounts.tsx b/static/app/components/feedback/list/useMailboxCounts.tsx
@@ -77,7 +77,7 @@ export default function useMailboxCounts({
 
   return useMemo(
     () =>
-      (({
+      ({
         ...result,
 
         data: result.data
@@ -86,8 +86,8 @@ export default function useMailboxCounts({
               resolved: result.data[MAILBOX.resolved],
               ignored: result.data[MAILBOX.ignored],
             }
-          : undefined
-      }) as UseApiQueryResult<HookReturnType, RequestError>),
+          : undefined,
+      }) as UseApiQueryResult<HookReturnType, RequestError>,
     [result, MAILBOX.ignored, MAILBOX.resolved, MAILBOX.unresolved]
   );
 }
diff --git a/static/app/components/forms/fields/radioField.tsx b/static/app/components/forms/fields/radioField.tsx
@@ -22,21 +22,21 @@ function handleChange(
 
 function RadioField(props: RadioFieldProps) {
   return (
-    (<FormField {...props}>
+    <FormField {...props}>
       {({id, onChange, onBlur, value, disabled, orientInline, ...fieldProps}) => (
         // XXX: The label must be present on the role="radiogroup" element. The
         // `htmlFor` attribute on the Field label does NOT link to the group.
-        (<RadioGroup
+        <RadioGroup
           id={id}
           choices={fieldProps.choices}
           disabled={disabled}
           orientInline={orientInline}
           label={fieldProps.label}
           value={value === '' ? null : value}
           onChange={(v, e) => handleChange(v, onChange, onBlur, e)}
-        />)
+        />
       )}
-    </FormField>)
+    </FormField>
   );
 }
 

diff --git a/static/app/components/group/pluginActions.tsx b/static/app/components/group/pluginActions.tsx
@@ -164,7 +164,7 @@ class PluginActionsModal extends Component<ModalProps, ModalState> {
     const {actionType} = this.state;
 
     return (
-      (<Fragment>
+      <Fragment>
         <Header closeButton>
           <h4>{tct('[name] Issue', {name: plugin.name || plugin.title})}</h4>
         </Header>
@@ -179,7 +179,7 @@ class PluginActionsModal extends Component<ModalProps, ModalState> {
         {actionType && (
           // need the key here so React will re-render
           // with new action prop
-          (<Body key={actionType}>
+          <Body key={actionType}>
             {plugins.get(plugin).renderGroupActions({
               plugin,
               group,
@@ -188,9 +188,9 @@ class PluginActionsModal extends Component<ModalProps, ModalState> {
               actionType,
               onSuccess,
             })}
-          </Body>)
+          </Body>
         )}
-      </Fragment>)
+      </Fragment>
     );
   }
 }

diff --git a/static/app/components/hookOrDefault.spec.tsx b/static/app/components/hookOrDefault.spec.tsx
@@ -28,9 +28,9 @@ describe('HookOrDefault', () => {
     HookStore.add(
       'component:replay-onboarding-cta',
       () =>
-        (function({organization}) {
+        function ({organization}) {
           return <div data-test-id="hook-component">{organization.slug}</div>;
-        })
+        }
     );
 
     const Component = HookOrDefault({

diff --git a/static/app/components/modals/recoveryOptionsModal.tsx b/static/app/components/modals/recoveryOptionsModal.tsx
@@ -60,7 +60,7 @@ function RecoveryOptionsModal({
   }
 
   return (
-    (<Fragment>
+    <Fragment>
       <Header closeButton>{t('Two-Factor Authentication Enabled')}</Header>
       <Body>
         <TextBlock>
@@ -72,22 +72,22 @@ function RecoveryOptionsModal({
 
         {displaySmsPrompt ? (
           // set up backup phone number
-          (<Alert type="warning">
+          <Alert type="warning">
             {t('We recommend adding a phone number as a backup 2FA method.')}
-          </Alert>)
+          </Alert>
         ) : (
           // get recovery codes
-          (<Alert type="warning">
+          <Alert type="warning">
             {t(
               `Recovery codes are the only way to access your account if you lose
                   your device and cannot receive two-factor authentication codes.`
             )}
-          </Alert>)
+          </Alert>
         )}
       </Body>
       {displaySmsPrompt ? (
         // set up backup phone number
-        (<Footer>
+        <Footer>
           <Button onClick={handleSkipSms} name="skipStep" autoFocus>
             {t('Skip this step')}
           </Button>
@@ -101,10 +101,10 @@ function RecoveryOptionsModal({
           >
             {t('Add a Phone Number')}
           </Button>
-        </Footer>)
+        </Footer>
       ) : (
         // get recovery codes
-        (<Footer>
+        <Footer>
           <Button
             priority="primary"
             onClick={closeModal}
@@ -118,9 +118,9 @@ function RecoveryOptionsModal({
           >
             {t('Get Recovery Codes')}
           </Button>
-        </Footer>)
+        </Footer>
       )}
-    </Fragment>)
+    </Fragment>
   );
 }
 

diff --git a/static/app/components/modals/widgetViewerModal.tsx b/static/app/components/modals/widgetViewerModal.tsx
@@ -632,7 +632,7 @@ function WidgetViewerModal(props: Props) {
     const links = parseLinkHeader(pageLinks ?? null);
     const isFirstPage = links.previous?.results === false;
     return (
-      (<Fragment>
+      <Fragment>
         <GridEditable
           isLoading={loading}
           data={tableResults?.[0]?.data ?? []}
@@ -682,7 +682,7 @@ function WidgetViewerModal(props: Props) {
               }}
             />
           )}
-      </Fragment>)
+      </Fragment>
     );
   };