fix: address code review feedback

CodebaseIntelligence.tsx:
- Fix guard to work with deps/nodes instead of requiring metrics
- Fix startHere to find critical file that's also entry point
- Fix totalFunctions to only use real data, show N/A when missing
- Fix HealthIndicator to handle 0 values (not treat as falsy)

DashboardLayout.tsx:
- Wrap localStorage operations in try/catch

DashboardStats.tsx:
- Rename 'Functions Indexed' to 'Files Indexed' (correct metric)

TopNav.tsx:
- Add aria-label to theme toggle for accessibility

RepoList.tsx:
- Change 'Functions' label to 'Files' (matches file_count)

RepoSummaryCard.tsx:
- Fix markdown asterisks - use JSX <strong> for repo name
- Fix primaryLang to sort by dominant language value

ResultCard.tsx:
- Use resolvedTheme instead of theme for system theme support

Author: DevanshuNEU

frontend/src/components/CodebaseIntelligence.tsx

@@ -32,10 +32,11 @@ export function CodebaseIntelligence({ repo, apiKey, onTabChange }: CodebaseInte
 
   // Derive intelligence from raw data
   const intelligence = useMemo(() => {
-    if (!deps?.metrics) return null
-
-    const nodes = deps.nodes || []
-    const edges = deps.edges || []
+    const nodes = deps?.nodes || []
+    const edges = deps?.edges || []
+    
+    // Gate on deps existence or having graph data
+    if (!deps && nodes.length === 0) return null
 
     // Calculate in-degree and out-degree from edges
     const inDegree: Record<string, number> = {}
@@ -90,15 +91,21 @@ export function CodebaseIntelligence({ repo, apiKey, onTabChange }: CodebaseInte
     })
 
     // Find "start here" file - the most critical file that's also an entry point
-    const startHere = criticalFiles[0]?.file || entryPoints[0]?.file || null
+    const criticalEntryPoint = criticalFiles.find(cf => 
+      entryPoints.some(ep => ep.file === cf.file)
+    )
+    const startHere = criticalEntryPoint?.file || entryPoints[0]?.file || null
 
     // Generate smart summary
-    const totalFiles = deps.total_files || nodes.length
-    const totalFunctions = style?.summary?.total_functions || repo.file_count || 0
+    const totalFiles = deps?.total_files || nodes.length
+    const totalFunctions: number | null = style?.summary?.total_functions ?? null
 
-    let sizeDesc = 'compact'
-    if (totalFunctions > 1000) sizeDesc = 'large'
-    else if (totalFunctions > 200) sizeDesc = 'medium-sized'
+    let sizeDesc: string | null = null
+    if (typeof totalFunctions === 'number') {
+      if (totalFunctions > 1000) sizeDesc = 'large'
+      else if (totalFunctions > 200) sizeDesc = 'medium-sized'
+      else sizeDesc = 'compact'
+    }
 
     // Detect patterns
     const hasMiddleware = nodes.some((n: any) => 
@@ -168,9 +175,11 @@ export function CodebaseIntelligence({ repo, apiKey, onTabChange }: CodebaseInte
             </h3>
 
             <p className="text-foreground leading-relaxed mb-4">
-              <strong>{repo.name}</strong> is a {intelligence.summary.size} {intelligence.summary.language} codebase 
-              with <span className="text-primary font-semibold">{intelligence.summary.totalFunctions.toLocaleString()}</span> functions 
-              across {intelligence.summary.totalFiles} files.
+              <strong>{repo.name}</strong> is a {intelligence.summary.size ? `${intelligence.summary.size} ` : ''}{intelligence.summary.language} codebase 
+              {intelligence.summary.totalFunctions != null ? (
+                <> with <span className="text-primary font-semibold">{intelligence.summary.totalFunctions.toLocaleString()}</span> functions</>
+              ) : null}
+              {' '}across {intelligence.summary.totalFiles} files.
               {intelligence.summary.pattern && (
                 <> Uses {intelligence.summary.pattern}.</>
               )}
@@ -367,8 +376,8 @@ export function CodebaseIntelligence({ repo, apiKey, onTabChange }: CodebaseInte
   )
 }
 
-function HealthIndicator({ label, value, threshold }: { label: string, value: string | null, threshold: number }) {
-  if (!value) {
+function HealthIndicator({ label, value, threshold }: { label: string, value: string | number | null, threshold: number }) {
+  if (value == null) {
     return (
       <div className="p-3 bg-muted/50 rounded-lg">
         <p className="text-[10px] text-muted-foreground uppercase tracking-wide mb-1">{label}</p>
@@ -377,15 +386,16 @@ function HealthIndicator({ label, value, threshold }: { label: string, value: st
     )
   }
 
-  const numValue = parseFloat(value)
-  const isGood = numValue >= threshold
+  const numValue = typeof value === 'number' ? value : parseFloat(value)
+  const isGood = !isNaN(numValue) && numValue >= threshold
+  const displayValue = typeof value === 'number' ? `${value}%` : value
 
   return (
     <div className="p-3 bg-muted/50 rounded-lg">
       <p className="text-[10px] text-muted-foreground uppercase tracking-wide mb-1">{label}</p>
       <p className={`text-sm font-semibold flex items-center gap-1 ${isGood ? 'text-green-600 dark:text-green-400' : 'text-foreground'}`}>
         {isGood && <CheckCircle2 className="w-3.5 h-3.5" />}
-        {value}
+        {displayValue}
       </p>
     </div>
   )

frontend/src/components/RepoList.tsx

@@ -83,7 +83,7 @@ const RepoCard = ({ repo, index, onSelect }: {
         {/* Stats */}
         <div className="pt-4 border-t border-border">
           <div className="flex items-center justify-between">
-            <span className="text-sm text-muted-foreground">Functions</span>
+            <span className="text-sm text-muted-foreground">Files</span>
             <span className="text-2xl font-bold text-primary">
               {(repo.file_count || 0).toLocaleString()}
             </span>

frontend/src/components/RepoSummaryCard.tsx

@@ -53,7 +53,9 @@ export function RepoSummaryCard({ repo, apiKey }: RepoSummaryCardProps) {
             <span className="text-[10px] px-1.5 py-0.5 bg-primary/20 text-primary rounded font-normal">Auto-generated</span>
           </h3>
 
-          <p className="text-foreground leading-relaxed mb-4">{summary.main}</p>
+          <p className="text-foreground leading-relaxed mb-4">
+            <strong>{repo.name}</strong> {summary.main}
+          </p>
 
           {/* Quick Stats Pills */}
           <div className="flex flex-wrap gap-2">
@@ -99,7 +101,19 @@ function generateSummary(repo: Repository, insights: any, style: any) {
   const fileCount = insights?.total_files || repo.file_count || 0
   const functionCount = style?.summary?.total_functions || insights?.total_functions || 0
   const languages = insights?.languages || style?.language_distribution || {}
-  const primaryLang = Object.keys(languages)[0] || 'Unknown'
+  
+  // Get primary language by sorting entries by value descending
+  const langEntries = Object.entries(languages)
+  let primaryLang = 'Unknown'
+  if (langEntries.length > 0) {
+    const sorted = langEntries.sort((a, b) => {
+      const valA = typeof a[1] === 'number' ? a[1] : (a[1] as any)?.percentage ?? (a[1] as any)?.count ?? 0
+      const valB = typeof b[1] === 'number' ? b[1] : (b[1] as any)?.percentage ?? (b[1] as any)?.count ?? 0
+      return Number(valB) - Number(valA)
+    })
+    primaryLang = sorted[0][0]
+  }
+  
   const asyncAdoption = style?.summary?.async_adoption || null
   const typeHints = style?.summary?.type_hints_usage || null
 
@@ -125,8 +139,8 @@ function generateSummary(repo: Repository, insights: any, style: any) {
     stats.push({ icon: Code2, label: 'Async', value: asyncAdoption })
   }
 
-  // Generate main summary text
-  let main = `**${repo.name}** is a `
+  // Generate main summary text (repo name rendered separately in JSX)
+  let main = `is a `
 
   if (functionCount > 500) main += 'large '
   else if (functionCount > 100) main += 'medium-sized '

frontend/src/components/dashboard/DashboardLayout.tsx

@@ -17,13 +17,21 @@ export function DashboardLayout({ children }: DashboardLayoutProps) {
   const { theme } = useTheme()
 
   const [sidebarCollapsed, setSidebarCollapsed] = useState(() => {
-    const stored = localStorage.getItem(SIDEBAR_STORAGE_KEY)
-    return stored ? JSON.parse(stored) : false
+    try {
+      const stored = localStorage.getItem(SIDEBAR_STORAGE_KEY)
+      return stored ? JSON.parse(stored) : false
+    } catch {
+      return false
+    }
   })
   const [commandPaletteOpen, setCommandPaletteOpen] = useState(false)
 
   useEffect(() => {
-    localStorage.setItem(SIDEBAR_STORAGE_KEY, JSON.stringify(sidebarCollapsed))
+    try {
+      localStorage.setItem(SIDEBAR_STORAGE_KEY, JSON.stringify(sidebarCollapsed))
+    } catch {
+      // Ignore storage errors
+    }
   }, [sidebarCollapsed])
 
   useKeyboardShortcut(SHORTCUTS.COMMAND_PALETTE, () => {

frontend/src/components/dashboard/DashboardStats.tsx

@@ -29,12 +29,12 @@ const useAnimatedCounter = (end: number, duration: number = 1200) => {
 export function DashboardStats({ repos }: DashboardStatsProps) {
   const totalRepos = repos.length
   const indexedRepos = repos.filter(r => r.status === 'indexed').length
-  const totalFunctions = repos.reduce((acc, r) => acc + (r.file_count || 0), 0)
+  const totalFiles = repos.reduce((acc, r) => acc + (r.file_count || 0), 0)
   const indexingCount = repos.filter(r => r.status === 'indexing' || r.status === 'cloning').length
 
   const animatedTotal = useAnimatedCounter(totalRepos)
   const animatedIndexed = useAnimatedCounter(indexedRepos)
-  const animatedFunctions = useAnimatedCounter(totalFunctions)
+  const animatedFiles = useAnimatedCounter(totalFiles)
 
   const stats = [
     {
@@ -56,8 +56,8 @@ export function DashboardStats({ repos }: DashboardStatsProps) {
       ) : null,
     },
     {
-      label: 'Functions Indexed',
-      value: animatedFunctions,
+      label: 'Files Indexed',
+      value: animatedFiles,
       icon: Code2,
       suffix: '',
       format: true,

frontend/src/components/dashboard/TopNav.tsx

@@ -82,6 +82,7 @@ export function TopNav({ onToggleSidebar, sidebarCollapsed, onOpenCommandPalette
             size="icon"
             onClick={toggleTheme}
             className="text-muted-foreground hover:text-foreground"
+            aria-label={theme === 'dark' ? 'Switch to light mode' : 'Switch to dark mode'}
           >
             {theme === 'dark' ? (
               <Sun className="w-5 h-5" />

frontend/src/components/search/ResultCard.tsx

@@ -19,7 +19,7 @@ export function ResultCard({ result, rank, isExpanded: initialExpanded = false,
   const [expanded, setExpanded] = useState(initialExpanded);
   const contentRef = useRef<HTMLDivElement>(null);
   const [contentHeight, setContentHeight] = useState<number | undefined>(initialExpanded ? undefined : 0);
-  const { theme } = useTheme();
+  const { resolvedTheme } = useTheme();
 
   const matchPercent = Math.round(result.score * 100);
   const isTopResult = rank === 1;
@@ -90,7 +90,7 @@ export function ResultCard({ result, rank, isExpanded: initialExpanded = false,
           <div className="relative">
             <SyntaxHighlighter
               language={result.language || 'text'}
-              style={theme === 'dark' ? oneDark : oneLight}
+              style={resolvedTheme === 'dark' ? oneDark : oneLight}
               customStyle={{ margin: 0, borderRadius: 0, fontSize: '0.75rem', lineHeight: '1.6', padding: '1rem' }}
               showLineNumbers
               startingLineNumber={result.line_start}