fix: address Copilot review feedback

- Fix tabNames mapping in overview mode (was logging 'code' instead of 'specification')
- Remove redundant onClick handlers from Tab components (handleTabChange handles toggle)
- Document regex limitation in strip_noqa_comments docstring

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: MarkusNeusinger

app/src/components/SpecTabs.tsx

@@ -178,7 +178,10 @@ export function SpecTabs({
       onTrackEvent?.('tab_collapse', { library: libraryId });
     } else {
       setTabIndex(newValue);
-      const tabNames = ['code', 'specification', 'implementation', 'quality'];
+      // In overview mode, only Spec tab exists at index 0
+      const tabNames = overviewMode
+        ? ['specification']
+        : ['code', 'specification', 'implementation', 'quality'];
       onTrackEvent?.('tab_click', { tab: tabNames[newValue], library: libraryId });
     }
   };
@@ -242,33 +245,17 @@ export function SpecTabs({
           }}
         >
           {!overviewMode && (
-            <Tab
-              icon={<CodeIcon sx={{ fontSize: '1.1rem' }} />}
-              iconPosition="start"
-              label="Code"
-              onClick={() => tabIndex === 0 && setTabIndex(null)}
-            />
+            <Tab icon={<CodeIcon sx={{ fontSize: '1.1rem' }} />} iconPosition="start" label="Code" />
           )}
-          <Tab
-            icon={<DescriptionIcon sx={{ fontSize: '1.1rem' }} />}
-            iconPosition="start"
-            label="Spec"
-            onClick={() => tabIndex === specTabIndex && setTabIndex(null)}
-          />
+          <Tab icon={<DescriptionIcon sx={{ fontSize: '1.1rem' }} />} iconPosition="start" label="Spec" />
           {!overviewMode && (
-            <Tab
-              icon={<ImageIcon sx={{ fontSize: '1.1rem' }} />}
-              iconPosition="start"
-              label="Impl"
-              onClick={() => tabIndex === 2 && setTabIndex(null)}
-            />
+            <Tab icon={<ImageIcon sx={{ fontSize: '1.1rem' }} />} iconPosition="start" label="Impl" />
           )}
           {!overviewMode && (
             <Tab
               icon={<StarIcon sx={{ fontSize: '1.1rem', color: tabIndex === 3 ? '#3776AB' : '#f59e0b' }} />}
               iconPosition="start"
               label={qualityScore ? `${Math.round(qualityScore)}` : 'Quality'}
-              onClick={() => tabIndex === 3 && setTabIndex(null)}
             />
           )}
         </Tabs>

core/utils.py

@@ -4,7 +4,13 @@
 
 
 def strip_noqa_comments(code: str | None) -> str | None:
-    """Remove # noqa: comments from code for cleaner user display."""
+    """Remove # noqa: comments from code for cleaner user display.
+
+    Note:
+        Uses regex-based matching, which will also remove noqa patterns inside
+        string literals. This is acceptable for plot implementation code where
+        noqa comments in strings are extremely rare.
+    """
     if not code:
         return code
     return re.sub(r"\s*# noqa:[^\n]*", "", code)