Drop redundant guards and dead variables flagged by CodeQL

Each guard sits behind an earlier early-return that already enforces
truthiness, so the inner check can never fire false. The visible
ternaries always rendered the truthy branch. The dead UnifiedInputContainer
inside the council-mode arm was a duplicate of the real one in the
non-council branch. Plus four script imports nothing referenced,
chat.ts seedId not consumed downstream, MiniPlayer imageKey overwritten
on every branch, ErrorBoundary errorInfo only ever written, and
SpeechPanel's OptionButtons prop was declared but never destructured.

Author: michael-chipmates

client/scripts/extract-public-data.mjs

@@ -4,8 +4,8 @@
 // Reads full seed data, voice profiles, figure translations, and story files
 // Outputs lightweight summaries for prerendering and client display
 
-import { readFileSync, writeFileSync, mkdirSync, existsSync, readdirSync } from 'fs';
-import { join, basename } from 'path';
+import { readFileSync, writeFileSync, mkdirSync, existsSync } from 'fs';
+import { join } from 'path';
 
 const CLIENT_DIR = join(import.meta.dirname, '..');
 const SEEDS_DIR = join(CLIENT_DIR, 'src/assets/translations/seeds');

client/scripts/prerender.mjs

@@ -4,8 +4,8 @@
 // Phase 1: HTML shell pages with meta tags + JSON-LD for crawlers
 // Phase 2 (future): Full React SSR rendering with react-dom/server
 
-import { readFileSync, writeFileSync, mkdirSync, existsSync } from 'fs';
-import { join, dirname } from 'path';
+import { readFileSync, writeFileSync, mkdirSync } from 'fs';
+import { join } from 'path';
 
 const BUILD_DIR = join(import.meta.dirname, '..', 'build');
 const SITE_URL = 'https://agoracosmica.org';

client/src/components/AudioLibrary/MiniPlayer.tsx

@@ -63,9 +63,9 @@ const MiniPlayer: FC<MiniPlayerProps> = ({ story, figure, onExpand, audioService
       if (playback.figureName) {
         // Extract imageKey from figure name for image loading
         const figureNameLower = playback.figureName.toLowerCase();
-        let imageKey = 'default';
-        
-        // Extract the key from the figure name
+        // Extract the key from the figure name. All branches below assign,
+        // so no initial fallback is needed.
+        let imageKey: string;
         if (figureNameLower.includes('leonardo') || figureNameLower.includes('vinci')) {
           imageKey = 'vinci';
         } else if (figureNameLower.includes('king jr')) {

client/src/components/AudioLibrary/NowPlayingView.tsx

@@ -632,7 +632,7 @@ const NowPlayingView: FC<NowPlayingViewProps> = ({ story, figure, audioService,
             priority={true}
             withBlurUp={true}
             className="now-playing-view__figure-image"
-            alt={tString('audioLibrary.nowPlaying.imageOf', `Image of ${currentFigure && currentFigure.name ? getFigureDisplayName(currentFigure.name) : ''}`).replace('{{figureName}}', currentFigure && currentFigure.name ? getFigureDisplayName(currentFigure.name) : '')}
+            alt={tString('audioLibrary.nowPlaying.imageOf', `Image of ${currentFigure.name ? getFigureDisplayName(currentFigure.name) : ''}`).replace('{{figureName}}', currentFigure.name ? getFigureDisplayName(currentFigure.name) : '')}
           />
         )}
 

client/src/components/CosmicCouncil/CouncilSetupModal.tsx

@@ -211,7 +211,8 @@ const CouncilSetupModal: FC<CouncilSetupModalProps> = ({
     }
 
     if (selectedFigures.length < 3) {
-      const allFigures = moderator ? [moderator, ...selectedFigures] : selectedFigures;
+      // moderator is non-null here — the !moderator branch above returned.
+      const allFigures = [moderator, ...selectedFigures];
       const genderCheck = checkGenderLimit(allFigures, figure);
       if (!genderCheck.canAdd) {
         console.warn(genderCheck.reason);

client/src/components/ErrorBoundary.tsx

@@ -13,7 +13,6 @@ interface ErrorBoundaryProps extends ErrorBoundaryWrapperProps {
 interface ErrorBoundaryState {
   hasError: boolean;
   error: Error | null;
-  errorInfo: ErrorInfo | null;
 }
 
 // Wrapper component to use hooks with class component
@@ -23,10 +22,9 @@ function ErrorBoundaryWrapper(props: ErrorBoundaryWrapperProps): React.ReactElem
 }
 
 class ErrorBoundary extends React.Component<ErrorBoundaryProps, ErrorBoundaryState> {
-  state: ErrorBoundaryState = { 
-    hasError: false, 
-    error: null, 
-    errorInfo: null 
+  state: ErrorBoundaryState = {
+    hasError: false,
+    error: null
   };
 
   static getDerivedStateFromError(): Partial<ErrorBoundaryState> {
@@ -35,7 +33,7 @@ class ErrorBoundary extends React.Component<ErrorBoundaryProps, ErrorBoundarySta
 
   componentDidCatch(error: Error, errorInfo: ErrorInfo): void {
     console.error('Story Mode Error:', error, errorInfo);
-    this.setState({ error, errorInfo });
+    this.setState({ error });
   }
 
   render(): ReactNode {

client/src/components/HomePage/MainContent.tsx

@@ -284,14 +284,6 @@ const MainContent: FC<MainContentProps> = ({
                   isReviewMode={isReviewMode}
                   isAudioPlaying={isAudioPlaying}
                 />
-
-                {/* Hide input field during council mode - autonomous conversation doesn't need user input */}
-                {!isCouncilMode && (
-                  <UnifiedInputContainer
-                    selectedFigure={selectedFigure?.name || 'Unknown'}
-                    onSubmitMessage={onSubmitMessage}
-                  />
-                )}
               </>
             ) : !conversationStartedFinal ? (
               // Show empty state while mode selector or language wheel should appear

client/src/components/QuestVerdictCard/PostQuestVerdictCard.tsx

@@ -44,7 +44,7 @@ const PostQuestVerdictCard: FC<PostQuestVerdictCardProps> = ({
 
   return (
     <div
-      className={`quest-verdict-card ${visible ? 'visible' : ''}`}
+      className="quest-verdict-card visible"
       role="status"
       aria-live="polite"
       onTouchStart={handleTouchStart}

client/src/components/StoryPlayer.tsx

@@ -586,7 +586,7 @@ const StoryPlayer: FC<StoryPlayerProps> = ({
   const { audioUrl } = storyData;
 
   // Don't show loading if we have valid story data
-  const isLoading = status === 'loading' && (!storyData || !storyData.text);
+  const isLoading = status === 'loading' && !storyData.text;
 
   // Don't show the story player at all if we're in translation mode
   if (storyData.needsTranslation) {

client/src/components/WisdomMapModal/PostDialogueBloomInvite.tsx

@@ -67,7 +67,7 @@ const PostDialogueBloomInvite: FC<PostDialogueBloomInviteProps> = ({ count, onTa
 
   return (
     <div
-      className={`bloom-invite ${visible ? 'visible' : ''}`}
+      className="bloom-invite visible"
       onClick={handleTap}
       onTouchStart={handleTouchStart}
       onTouchEnd={handleTouchEnd}