Fix PR comments

Author: mujacica

static/app/components/events/interfaces/frame/context.tsx

@@ -168,7 +168,7 @@ export function Context({
       : {}
   );
 
-  const fileExtension = getFileExtension(frame.filename || '') ?? '';
+  const fileExtension = getFileExtension(frame.filename || frame.absPath || '') ?? '';
   const lines = usePrismTokensSourceContext({
     contextLines,
     lineNo: frame.lineNo,

static/app/components/events/interfaces/frame/deprecatedLine.tsx

@@ -14,6 +14,7 @@ import {LeadHint} from 'sentry/components/events/interfaces/frame/leadHint';
 import {StacktraceLink} from 'sentry/components/events/interfaces/frame/stacktraceLink';
 import type {FrameSourceMapDebuggerData} from 'sentry/components/events/interfaces/sourceMapsDebuggerModal';
 import {SourceMapsDebuggerModal} from 'sentry/components/events/interfaces/sourceMapsDebuggerModal';
+import {useStacktraceContext} from 'sentry/components/events/interfaces/stackTraceContext';
 import {getThreadById} from 'sentry/components/events/interfaces/utils';
 import {StrictClick} from 'sentry/components/strictClick';
 import {IconChevron, IconFix, IconRefresh} from 'sentry/icons';
@@ -26,9 +27,7 @@ import type {
 import type {PlatformKey} from 'sentry/types/project';
 import type {StacktraceType} from 'sentry/types/stacktrace';
 import {trackAnalytics} from 'sentry/utils/analytics';
-import {useDetailedProject} from 'sentry/utils/project/useDetailedProject';
 import {useOrganization} from 'sentry/utils/useOrganization';
-import {useProjects} from 'sentry/utils/useProjects';
 import {withSentryAppComponents} from 'sentry/utils/withSentryAppComponents';
 import {SectionKey} from 'sentry/views/issueDetails/streamline/context';
 
@@ -112,21 +111,11 @@ function DeprecatedLine({
   components,
 }: Props) {
   const organization = useOrganization();
-  const {projects} = useProjects();
-  const project = useMemo(
-    () => projects.find(p => p.id === event.projectID),
-    [projects, event]
-  );
+  const {hasScmSourceContext} = useStacktraceContext();
   const [isHovering, setIsHovering] = useState(false);
   const [isExpanded, setIsExpanded] = useState(initialExpanded ?? false);
   const platform = getPlatform(data.platform, propPlatform ?? 'other');
   const leadsToApp = !data.inApp && (nextFrame?.inApp || !nextFrame);
-  const hasScmFeature = organization.features.includes('scm-source-context');
-  const {data: detailedProject} = useDetailedProject(
-    {orgSlug: organization.slug, projectSlug: project?.slug ?? ''},
-    {enabled: hasScmFeature && !!project}
-  );
-  const hasScmSourceContext = hasScmFeature && !!detailedProject?.scmSourceContextEnabled;
 
   const isExpandable = useMemo((): boolean => {
     return !!(

static/app/components/events/interfaces/nativeFrame.tsx

@@ -1,5 +1,5 @@
 import type {MouseEvent} from 'react';
-import {Fragment, useMemo, useState} from 'react';
+import {Fragment, useState} from 'react';
 import styled from '@emotion/styled';
 
 import {Tag} from '@sentry/scraps/badge';
@@ -42,9 +42,6 @@ import type {
 import type {PlatformKey} from 'sentry/types/project';
 import {StackView, type StacktraceType} from 'sentry/types/stacktrace';
 import {defined} from 'sentry/utils';
-import {useDetailedProject} from 'sentry/utils/project/useDetailedProject';
-import {useOrganization} from 'sentry/utils/useOrganization';
-import {useProjects} from 'sentry/utils/useProjects';
 import {useSyncedLocalStorageState} from 'sentry/utils/useSyncedLocalStorageState';
 import {withSentryAppComponents} from 'sentry/utils/withSentryAppComponents';
 import {SectionKey, useIssueDetails} from 'sentry/views/issueDetails/streamline/context';
@@ -107,7 +104,7 @@ function NativeFrame({
   const isDartAsyncSuspensionFrame =
     frame.filename === '<asynchronous suspension>' ||
     frame.absPath === '<asynchronous suspension>';
-  const {displayOptions, stackView} = useStacktraceContext();
+  const {displayOptions, stackView, hasScmSourceContext} = useStacktraceContext();
 
   const {sectionData} = useIssueDetails();
   const debugSectionConfig = sectionData[SectionKey.DEBUGMETA];
@@ -133,19 +130,6 @@ function NativeFrame({
     // We know the debug section is rendered (only once streamline ui is enabled)
     (hasStreamlinedUI ? !!debugSectionConfig : true);
 
-  const organization = useOrganization();
-  const {projects} = useProjects();
-  const project = useMemo(
-    () => projects.find(p => p.id === event.projectID),
-    [projects, event]
-  );
-  const hasScmFeature = organization.features.includes('scm-source-context');
-  const {data: detailedProject} = useDetailedProject(
-    {orgSlug: organization.slug, projectSlug: project?.slug ?? ''},
-    {enabled: hasScmFeature && defined(project)}
-  );
-  const hasScmSourceContext = hasScmFeature && !!detailedProject?.scmSourceContextEnabled;
-
   const leadsToApp = !frame.inApp && (nextFrame?.inApp || !nextFrame);
   const expandable = isExpandable({
     frame,

static/app/components/events/interfaces/stackTraceContext.tsx

@@ -1,6 +1,8 @@
 import {createContext, useContext, useMemo, useState} from 'react';
 
 import {StackType, StackView} from 'sentry/types/stacktrace';
+import {defined} from 'sentry/utils';
+import {useDetailedProject} from 'sentry/utils/project/useDetailedProject';
 import {useLocalStorageState} from 'sentry/utils/useLocalStorageState';
 import {useOrganization} from 'sentry/utils/useOrganization';
 
@@ -41,6 +43,10 @@ interface StacktraceContextType {
    * This should only be used to control the full/relevant toggle.
    */
   forceFullStackTrace: boolean;
+  /**
+   * Whether the project has SCM source context enabled
+   */
+  hasScmSourceContext: boolean;
   /**
    * Display full stack trace or filter to relevant frames.
    * This should only be used to control the full/relevant toggle.
@@ -79,6 +85,7 @@ const IssueStacktraceContext = createContext<StacktraceContextType>({
   stackType: StackType.ORIGINAL,
   displayOptions: [],
   setDisplayOptions: () => {},
+  hasScmSourceContext: false,
   isFullStackTrace: true,
   forceFullStackTrace: false,
   setIsFullStackTrace: () => {},
@@ -94,6 +101,13 @@ export function StacktraceContext({
   defaultIsNewestFramesFirst = true,
 }: StackTraceContextOptions) {
   const organization = useOrganization();
+  const hasScmFeature = organization.features.includes('scm-source-context');
+  const {data: detailedProject} = useDetailedProject(
+    {orgSlug: organization.slug, projectSlug: projectSlug ?? ''},
+    {enabled: hasScmFeature && defined(projectSlug)}
+  );
+  const hasScmSourceContext = hasScmFeature && !!detailedProject?.scmSourceContextEnabled;
+
   const [isFullStackTrace, setIsFullStackTrace] = useState(false);
   const [isNewestFramesFirst, setIsNewestFramesFirst] = useState(
     defaultIsNewestFramesFirst
@@ -123,6 +137,7 @@ export function StacktraceContext({
       setIsNewestFramesFirst,
       displayOptions,
       setDisplayOptions,
+      hasScmSourceContext,
       stackView,
       stackType,
       forceFullStackTrace,
@@ -133,6 +148,7 @@ export function StacktraceContext({
       isNewestFramesFirst,
       displayOptions,
       setDisplayOptions,
+      hasScmSourceContext,
       stackView,
       stackType,
     ]