ref(✂️): remove unused code from explore (#90147)

Author: TkDodo

static/app/components/modals/explore/saveQueryModal.tsx

@@ -1,5 +1,4 @@
 import {Fragment, useCallback, useState} from 'react';
-import {css} from '@emotion/react';
 import styled from '@emotion/styled';
 import * as Sentry from '@sentry/react';
 
@@ -209,11 +208,6 @@ const StyledButtonBar = styled(ButtonBar)`
   }
 `;
 
-export const modalCss = css`
-  max-width: 700px;
-  margin: 70px auto;
-`;
-
 const SectionHeader = styled('h6')`
   font-size: ${p => p.theme.form.md.fontSize};
   margin-bottom: ${space(0.5)};

static/app/components/quickTrace/utils.tsx

@@ -18,7 +18,6 @@ import type {
   TracePerformanceIssue,
 } from 'sentry/utils/performance/quickTrace/types';
 import {getTraceTimeRangeFromEvent} from 'sentry/utils/performance/quickTrace/utils';
-import {MutableSearch} from 'sentry/utils/tokenizeSearch';
 import {hasDatasetSelector} from 'sentry/views/dashboards/utils';
 import type {TraceViewSources} from 'sentry/views/performance/newTraceDetails/traceHeader/breadcrumbs';
 import type {TraceTree} from 'sentry/views/performance/newTraceDetails/traceModels/traceTree';
@@ -85,41 +84,6 @@ export function generateSingleErrorTarget(
   }
 }
 
-export function generateMultiTransactionsTarget(
-  currentEvent: Event,
-  events: EventLite[],
-  organization: Organization,
-  groupType: 'Ancestor' | 'Children' | 'Descendant'
-): LocationDescriptor {
-  const queryResults = new MutableSearch([]);
-  const eventIds = events.map(child => child.event_id);
-  for (let i = 0; i < eventIds.length; i++) {
-    queryResults.addOp(i === 0 ? '(' : 'OR');
-    queryResults.addFreeText(`id:${eventIds[i]}`);
-    if (i === eventIds.length - 1) {
-      queryResults.addOp(')');
-    }
-  }
-
-  const {start, end} = getTraceTimeRangeFromEvent(currentEvent);
-  const traceEventView = EventView.fromSavedQuery({
-    id: undefined,
-    name: `${groupType} Transactions of Event ID ${currentEvent.id}`,
-    fields: ['transaction', 'project', 'trace.span', 'transaction.duration', 'timestamp'],
-    orderby: '-timestamp',
-    query: queryResults.formatString(),
-    projects: [...new Set(events.map(child => child.project_id))],
-    version: 2,
-    start,
-    end,
-  });
-  return traceEventView.getResultsViewUrlTarget(
-    organization,
-    false,
-    hasDatasetSelector(organization) ? SavedQueryDatasets.TRANSACTIONS : undefined
-  );
-}
-
 const timestampsFieldCandidates = [
   'dateCreated',
   'startTimestamp',

static/app/views/explore/components/table.tsx

@@ -13,9 +13,6 @@ import {
   GridHead,
   GridHeadCell,
   GridRow,
-  Header,
-  HeaderButtonContainer,
-  HeaderTitle,
 } from 'sentry/components/gridEditable/styles';
 import {space} from 'sentry/styles/space';
 import {defined} from 'sentry/utils';
@@ -164,9 +161,6 @@ export const TableRow = GridRow;
 export const TableBodyCell = GridBodyCell;
 
 export const TableHead = GridHead;
-export const TableHeader = Header;
-export const TableHeaderActions = HeaderButtonContainer;
-export const TableHeaderTitle = HeaderTitle;
 export const TableHeadCell = styled(GridHeadCell)<{align?: Alignments}>`
   ${p => p.align && `justify-content: ${p.align};`}
 `;

static/app/views/explore/contexts/traceItemAttributeContext.tsx

@@ -23,7 +23,7 @@ type TypedTraceItemAttributesStatus = {
 type TypedTraceItemAttributesResult = TypedTraceItemAttributes &
   TypedTraceItemAttributesStatus;
 
-export const TraceItemAttributeContext = createContext<
+const TraceItemAttributeContext = createContext<
   TypedTraceItemAttributesResult | undefined
 >(undefined);
 
@@ -120,13 +120,6 @@ export function useTraceItemAttributes(type?: 'number' | 'string') {
   };
 }
 
-export function useTraceItemAttribute(key: string) {
-  const {attributes: numberAttributes} = useTraceItemAttributes('number');
-  const {attributes: stringAttributes} = useTraceItemAttributes('string');
-
-  return stringAttributes[key] ?? numberAttributes[key] ?? null;
-}
-
 function getDefaultStringAttributes(itemType: TraceItemDataset) {
   if (itemType === TraceItemDataset.SPANS) {
     return SENTRY_SPAN_STRING_TAGS;

static/app/views/explore/logs/styles.tsx

@@ -16,7 +16,7 @@ import {
 } from 'sentry/views/explore/components/table';
 import {SeverityLevel} from 'sentry/views/explore/logs/utils';
 
-export const StyledPanel = styled(Panel)`
+const StyledPanel = styled(Panel)`
   margin-bottom: 0;
 `;
 
@@ -25,27 +25,6 @@ export const LogDetailPanelItem = styled(PanelItem)`
   display: auto;
 `;
 
-export const StyledPanelItem = styled(PanelItem)<{
-  align?: 'left' | 'center' | 'right';
-  overflow?: boolean;
-  span?: number;
-}>`
-  align-items: center;
-  padding: ${space(1)} ${space(1)};
-  ${p => (p.align === 'left' ? 'justify-content: flex-start;' : null)}
-  ${p => (p.align === 'right' ? 'justify-content: flex-end;' : null)}
-  ${p => (p.overflow ? p.theme.overflowEllipsis : null)};
-  ${p =>
-    p.align === 'center'
-      ? `
-  justify-content: space-around;`
-      : p.align === 'left' || p.align === 'right'
-        ? `text-align: ${p.align};`
-        : undefined}
-  ${p => p.span && `grid-column: auto / span ${p.span};`}
-  white-space: nowrap;
-`;
-
 export const LogTableRow = styled(TableRow)`
   &:not(thead > &) {
     cursor: pointer;
@@ -117,12 +96,6 @@ export const DetailsContent = styled(StyledPanel)`
   padding: ${space(1)} ${space(2)};
 `;
 
-export const LogDetailsTitle = styled('div')`
-  font-size: ${p => p.theme.fontSizeLarge};
-  font-weight: ${p => p.theme.fontWeightBold};
-  user-select: none;
-`;
-
 export const LogFirstCellContent = styled('div')`
   display: flex;
   align-items: center;

static/app/views/explore/logs/utils.tsx

@@ -3,7 +3,6 @@ import * as Sentry from '@sentry/react';
 import {t} from 'sentry/locale';
 import type {TagCollection} from 'sentry/types/group';
 import {defined} from 'sentry/utils';
-import type {TableDataRow} from 'sentry/utils/discover/discoverQuery';
 import type {EventsMetaType} from 'sentry/utils/discover/eventView';
 import {
   type ColumnValueType,
@@ -13,7 +12,6 @@ import {
   prettifyTagKey,
 } from 'sentry/utils/discover/fields';
 import type {MutableSearch} from 'sentry/utils/tokenizeSearch';
-import type {TableColumn} from 'sentry/views/discover/table/types';
 import {LogAttributesHumanLabel} from 'sentry/views/explore/logs/constants';
 import {
   type LogAttributeItem,
@@ -165,7 +163,7 @@ export function getTableHeaderLabel(
   );
 }
 
-export function isLogAttributeUnit(unit: string | null): unit is LogAttributeUnits {
+function isLogAttributeUnit(unit: string | null): unit is LogAttributeUnits {
   return (
     unit === null ||
     Object.values(DurationUnit).includes(unit as DurationUnit) ||
@@ -205,21 +203,6 @@ export function getLogAttributeItem(
   };
 }
 
-export function logRowItemToTableColumn(
-  item: LogRowItem
-): TableColumn<keyof TableDataRow> {
-  return {
-    key: item.fieldKey,
-    name: item.fieldKey,
-    column: {
-      field: item.fieldKey,
-      kind: 'field',
-    },
-    isSortable: false,
-    type: item.metaFieldType,
-  };
-}
-
 export function adjustLogTraceID(traceID: string) {
   return traceID.replace(/-/g, '');
 }

static/app/views/explore/multiQueryMode/queryVisualizations/chart.tsx

@@ -53,8 +53,6 @@ interface MultiQueryChartProps {
   samplingMode?: SamplingMode;
 }
 
-export const EXPLORE_CHART_GROUP = 'multi-query-charts_group';
-
 export function MultiQueryModeChart({
   index,
   query: queryParts,

static/app/views/explore/tables/tracesTable/fieldRenderers.tsx

@@ -26,7 +26,6 @@ import type {SpanResult} from 'sentry/views/explore/hooks/useTraceSpans';
 import type {SpanIndexedField, SpanIndexedResponse} from 'sentry/views/insights/types';
 import {TraceViewSources} from 'sentry/views/performance/newTraceDetails/traceHeader/breadcrumbs';
 import {getTraceDetailsUrl} from 'sentry/views/performance/traceDetails/utils';
-import {transactionSummaryRouteWithQuery} from 'sentry/views/performance/transactionSummary/utils';
 
 import type {Field} from './data';
 import {getShortenedSdkName, getStylingSliceName} from './utils';
@@ -171,7 +170,7 @@ interface ProjectRendererProps {
   hideName?: boolean;
 }
 
-export function ProjectRenderer({projectSlug, hideName}: ProjectRendererProps) {
+function ProjectRenderer({projectSlug, hideName}: ProjectRendererProps) {
   const organization = useOrganization();
 
   return (
@@ -469,32 +468,6 @@ export function TraceIdRenderer({
   );
 }
 
-interface TransactionRendererProps {
-  projectSlug: string;
-  transaction: string;
-}
-
-export function TransactionRenderer({
-  projectSlug,
-  transaction,
-}: TransactionRendererProps) {
-  const location = useLocation();
-  const organization = useOrganization();
-  const {projects} = useProjects({slugs: [projectSlug]});
-
-  const target = transactionSummaryRouteWithQuery({
-    organization,
-    transaction,
-    query: {
-      ...location.query,
-      query: undefined,
-    },
-    projectID: String(projects[0]?.id ?? ''),
-  });
-
-  return <Link to={target}>{transaction}</Link>;
-}
-
 export function SpanTimeRenderer({
   timestamp,
   tooltipShowSeconds,
@@ -544,7 +517,7 @@ const OMITTED_SPAN_STATUS = ['unknown'];
 /**
  * This display a tag for the status (not to be confused with 'status_code' which has values like '200', '429').
  */
-export function StatusTag({status, onClick}: {status: string; onClick?: () => void}) {
+function StatusTag({status, onClick}: {status: string; onClick?: () => void}) {
   const tagType = statusToTagType(status);
 
   if (!tagType) {

static/app/views/explore/tables/tracesTable/utils.tsx

@@ -1,18 +1,7 @@
-import type {TraceResult} from 'sentry/views/explore/hooks/useTraces';
 import type {SpanResult} from 'sentry/views/explore/hooks/useTraceSpans';
 
 import type {Field} from './data';
 
-export function normalizeTraces(traces: TraceResult[] | undefined) {
-  if (!traces) {
-    return traces;
-  }
-  return traces.sort(
-    // Only sort name == null to the end, the rest leave in the original order.
-    (t1, t2) => (t1.name ? '0' : '1').localeCompare(t2.name ? '0' : '1')
-  );
-}
-
 export function getStylingSliceName(
   sliceName: string | null,
   sliceSecondaryName: string | null
@@ -25,21 +14,6 @@ export function getStylingSliceName(
   return sliceName;
 }
 
-export function areQueriesEmpty(queries: string[]): boolean {
-  if (queries.length > 1) {
-    return false;
-  }
-  if (queries.length === 0) {
-    return true;
-  }
-
-  if (queries.length === 1) {
-    return queries[0]!.length === 0;
-  }
-
-  return false;
-}
-
 export function getSecondaryNameFromSpan(span: SpanResult<Field>) {
   return span['sdk.name'];
 }

static/app/views/explore/utils.tsx

@@ -126,7 +126,7 @@ export function getExploreUrlFromSavedQueryUrl({
   });
 }
 
-export function getExploreMultiQueryUrl({
+function getExploreMultiQueryUrl({
   organization,
   selection,
   interval,
@@ -274,13 +274,6 @@ export function limitMaxPickableDays(organization: Organization): {
   };
 }
 
-export function showConfidence(isSampled: boolean | null | undefined) {
-  if (defined(isSampled) && isSampled === false) {
-    return false;
-  }
-  return true;
-}
-
 export function getDefaultExploreRoute(organization: Organization) {
   if (organization.features.includes('performance-trace-explorer')) {
     return 'traces';

static/app/views/traces/index.tsx

@@ -5,8 +5,6 @@ import Redirect from 'sentry/components/redirect';
 import useOrganization from 'sentry/utils/useOrganization';
 import {useRedirectNavV2Routes} from 'sentry/views/nav/useRedirectNavV2Routes';
 
-export const TRACE_EXPLORER_DOCS_URL = 'https://docs.sentry.io/product/explore/traces/';
-
 interface Props {
   children: React.ReactNode;
 }