chore(lint): fix pre-existing ESLint errors on main

src/tui/components/PowerlineThemeSelector.tsx

@@ -106,7 +106,7 @@ export const PowerlineThemeSelector: React.FC<PowerlineThemeSelectorProps> = ({
 }) => {
     const isRegular = mode === 'regular';
     const themes = useMemo(
-        () => (isRegular ? ['none', ...getPowerlineThemes().filter((name) => name !== 'custom')] : getPowerlineThemes()),
+        () => (isRegular ? ['none', ...getPowerlineThemes().filter(name => name !== 'custom')] : getPowerlineThemes()),
         [isRegular]
     );
     const currentTheme = isRegular
@@ -142,11 +142,11 @@ export const PowerlineThemeSelector: React.FC<PowerlineThemeSelectorProps> = ({
             ...(isRegular
                 ? { theme: themeName === 'none' ? undefined : themeName }
                 : {
-                        powerline: {
-                            ...latestSettingsRef.current.powerline,
-                            theme: themeName
-                        }
-                    })
+                    powerline: {
+                        ...latestSettingsRef.current.powerline,
+                        theme: themeName
+                    }
+                })
         });
     }, [selectedIndex, themes, isRegular]);
 

src/tui/components/__tests__/PowerlineThemeSelector.test.ts

@@ -180,7 +180,7 @@ describe('PowerlineThemeSelector helpers', () => {
     });
 
     it('writes settings.theme when navigating in regular mode', async () => {
-        const firstTheme = getPowerlineThemes().filter((name) => name !== 'custom')[0];
+        const firstTheme = getPowerlineThemes().find(name => name !== 'custom');
         expect(firstTheme).toBeDefined();
 
         const stdin = createMockStdin();

src/utils/__tests__/renderer-regular-theme.test.ts

@@ -154,7 +154,7 @@ describe('renderer regular-mode theme', () => {
     // Every shipped theme must behave identically in regular mode: same slot
     // rule, same bg[]-as-foreground palette, at all three color levels.
     describe('uniform cycle across all themes and levels', () => {
-        const themeNames = getPowerlineThemes().filter((name) => name !== 'custom');
+        const themeNames = getPowerlineThemes().filter(name => name !== 'custom');
         const levelName = (colorLevel: 1 | 2 | 3): 'ansi16' | 'ansi256' | 'truecolor' => (colorLevel === 1 ? 'ansi16' : colorLevel === 2 ? 'ansi256' : 'truecolor');
 
         it('has a non-empty bg palette at every level for every theme', () => {
@@ -167,24 +167,25 @@ describe('renderer regular-mode theme', () => {
             }
         });
 
-        it.each(themeNames.flatMap((name) => ([1, 2, 3] as const).map((colorLevel) => ({ name, colorLevel }))))(
+        it.each(themeNames.flatMap(name => ([1, 2, 3] as const).map(colorLevel => ({ name, colorLevel }))))(
             'cycles $name uniformly at colorLevel $colorLevel',
             ({ name, colorLevel }) => {
                 const theme = getPowerlineTheme(name);
                 expect(theme).toBeDefined();
                 const palette = (theme?.[String(colorLevel) as '1' | '2' | '3']?.bg ?? []).map(bgToFg);
                 const first = palette[0];
                 const second = palette[1];
-                expect(first).toBeDefined();
-                expect(second).toBeDefined();
+                if (first === undefined || second === undefined) {
+                    throw new Error(`theme ${name} at colorLevel ${colorLevel} must provide two palette entries`);
+                }
 
                 const line = renderWidgets(themedSettings(name, colorLevel), [
                     { id: '1', type: 'custom-text', customText: 'A' },
                     { id: '2', type: 'custom-text', customText: 'B' }
                 ]);
 
-                expect(line).toContain(fg(first as string, levelName(colorLevel)));
-                expect(line).toContain(fg(second as string, levelName(colorLevel)));
+                expect(line).toContain(fg(first, levelName(colorLevel)));
+                expect(line).toContain(fg(second, levelName(colorLevel)));
             }
         );
     });

src/utils/__tests__/separator-font-fallback.test.ts

@@ -6,7 +6,10 @@ import {
 } from 'vitest';
 
 import type { RenderContext } from '../../types/RenderContext';
-import { DEFAULT_SETTINGS, type Settings } from '../../types/Settings';
+import {
+    DEFAULT_SETTINGS,
+    type Settings
+} from '../../types/Settings';
 import type { WidgetItem } from '../../types/Widget';
 import {
     containsPowerlineGlyph,

src/utils/cli.ts

@@ -4,7 +4,6 @@ import type { WidgetItem } from '../types/Widget';
 import { WidgetItemSchema } from '../types/Widget';
 
 import { getPowerlineThemes } from './colors';
-
 import {
     getConfigLoadError,
     getConfigPath,
@@ -203,7 +202,7 @@ function getThemeWarnings(settings: Settings): string[] {
     if (!settings.theme || settings.theme === 'custom' || getPowerlineThemes().includes(settings.theme)) {
         return [];
     }
-    return [`unknown theme '${settings.theme}' (known: ${getPowerlineThemes().filter((name) => name !== 'custom').join(', ')})`];
+    return [`unknown theme '${settings.theme}' (known: ${getPowerlineThemes().filter(name => name !== 'custom').join(', ')})`];
 }
 
 interface SettingsLoad { settings: Settings }

src/utils/powerline.ts

@@ -169,9 +169,7 @@ export function setCachedPowerlineFontStatus(status: PowerlineFontStatus): void
 }
 
 export function getCachedPowerlineFontStatus(): PowerlineFontStatus {
-    if (cachedFontStatus === null) {
-        cachedFontStatus = checkPowerlineFonts();
-    }
+    cachedFontStatus ??= checkPowerlineFonts();
     return cachedFontStatus;
 }
 

src/utils/renderer.ts

@@ -38,8 +38,8 @@ import {
     isGradientSpec,
     parseGradientSpec
 } from './gradient';
-import { getTerminalWidth } from './terminal';
 import { fontSafeSeparator } from './powerline';
+import { getTerminalWidth } from './terminal';
 import {
     getWidget,
     widgetPreservesColors
@@ -163,12 +163,12 @@ function renderPowerlineStatusLine(
 
     // Get separator configuration; PUA glyphs degrade to '|' without a
     // Powerline/Nerd Font instead of rendering as mojibake
-    const separators = ((config.separators as string[] | undefined) ?? ['\uE0B0']).map((separator) => fontSafeSeparator(separator));
+    const separators = ((config.separators as string[] | undefined) ?? ['\uE0B0']).map(separator => fontSafeSeparator(separator));
     const invertBgs = (config.separatorInvertBackground as boolean[] | undefined) ?? separators.map(() => false);
 
     // Get caps arrays or fallback to empty arrays
-    const startCaps = ((config.startCaps as string[] | undefined) ?? []).map((cap) => fontSafeSeparator(cap));
-    const endCaps = ((config.endCaps as string[] | undefined) ?? []).map((cap) => fontSafeSeparator(cap));
+    const startCaps = ((config.startCaps as string[] | undefined) ?? []).map(cap => fontSafeSeparator(cap));
+    const endCaps = ((config.endCaps as string[] | undefined) ?? []).map(cap => fontSafeSeparator(cap));
 
     // Get the cap for this line (cycle through if more lines than caps)
     const capLineIndex = context.lineIndex ?? lineIndex;