chore: remove dead code and tautological tests

playwright.config.ts

@@ -28,6 +28,11 @@ export default defineConfig({
   retries: process.env.CI ? 2 : 0,
   workers: process.env.CI ? 2 : undefined,
   reporter: process.env.CI ? [['github'], ['html', { open: 'never' }]] : 'list',
+  // The dev server transforms lazily-imported route modules on demand, so the
+  // first assertion after navigating to a lazy route races that waterfall. On a
+  // shared CI runner it has measured over 3.5s from document to the component
+  // that owns the heading, which overruns the 5s default.
+  expect: { timeout: process.env.CI ? 15_000 : 5_000 },
   use: {
     baseURL,
     trace: 'retain-on-failure',

src/lib/ai/providers/mock-provider.ts

@@ -508,5 +508,3 @@ export async function mockJudgeSparringSession(input: SparringScoreInput): Promi
 }
 
 // Names kept short for callers that treat the mock provider as a feature adapter.
-export const mockSparringResponse = mockGenerateSparringTurn;
-export const mockSparringScore = mockJudgeSparringSession;

src/lib/native.ts

@@ -9,11 +9,6 @@ export function isNativeApp(): boolean {
   return Capacitor.isNativePlatform();
 }
 
-export function getAppRedirectUrl(path = '/app'): string {
-  if (isNativeApp()) return `${NATIVE_URL_SCHEME}://${path.replace(/^\/+/, '')}`;
-  return `${window.location.origin}${path}`;
-}
-
 export function getWebAppRedirectUrl(path = '/app'): string {
   const origin = isNativeApp() ? PUBLIC_WEB_APP_ORIGIN : window.location.origin;
   return `${origin}${path}`;

src/lib/validation/url.ts

@@ -55,11 +55,3 @@ export function validateUrl(url?: string): string | undefined {
         return undefined;
     }
 }
-
-/**
- * Checks if a URL is safe without normalizing it.
- * Useful for validation-only scenarios.
- */
-export function isUrlSafe(url?: string): boolean {
-    return validateUrl(url) !== undefined;
-}

src/test/export-utils-extended.test.ts

@@ -225,29 +225,4 @@ describe('Export Utils - Extended', () => {
     });
   });
 
-  describe('PDF export functions exist', () => {
-    it('exportCaseToDocx is a function', () => {
-      expect(typeof exportCaseToDocx).toBe('function');
-    });
-
-    it('exportAttackToDocx is a function', () => {
-      expect(typeof exportAttackToDocx).toBe('function');
-    });
-
-    it('exportBrainstormToDocx is a function', () => {
-      expect(typeof exportBrainstormToDocx).toBe('function');
-    });
-
-    it('exportFactoidsToDocx is a function', () => {
-      expect(typeof exportFactoidsToDocx).toBe('function');
-    });
-
-    it('exportCaseToGoogleDocs is a function', () => {
-      expect(typeof exportCaseToGoogleDocs).toBe('function');
-    });
-
-    it('exportBrainstormToGoogleDocs is a function', () => {
-      expect(typeof exportBrainstormToGoogleDocs).toBe('function');
-    });
-  });
 });

src/test/i18n.test.ts

@@ -32,23 +32,6 @@ describe('i18n locale files', () => {
   const esKeys = flattenKeys(es);
   const huKeys = flattenKeys(hu);
 
-  it('en.json has keys', () => {
-    expect(enKeys.length).toBeGreaterThan(0);
-  });
-
-  it('es.json has keys', () => {
-    expect(esKeys.length).toBeGreaterThan(0);
-  });
-
-  it('hu.json has keys', () => {
-    expect(huKeys.length).toBeGreaterThan(0);
-  });
-
-  it('all 3 locale files have same number of keys', () => {
-    expect(enKeys.length).toBe(esKeys.length);
-    expect(enKeys.length).toBe(huKeys.length);
-  });
-
   it('all keys in en.json exist in es.json', () => {
     const esKeySet = new Set(esKeys);
     const missing = enKeys.filter(k => !esKeySet.has(k));
@@ -100,15 +83,9 @@ describe('i18n locale files', () => {
 });
 
 describe('i18n runtime', () => {
-  it('i18n.t returns correct English value for nav.home', () => {
+  it('i18n.t resolves English nav values', () => {
     expect(i18n.t('nav.home')).toBe('Home');
-  });
-
-  it('i18n.t returns correct English value for nav.generate', () => {
     expect(i18n.t('nav.generate')).toBe('Generate');
-  });
-
-  it('i18n.t returns correct English value for nav.settings', () => {
     expect(i18n.t('nav.settings')).toBe('Settings');
   });
 
@@ -124,11 +101,6 @@ describe('i18n runtime', () => {
     await i18n.changeLanguage('en');
   });
 
-  it('i18n.changeLanguage back to en', async () => {
-    await i18n.changeLanguage('en');
-    expect(i18n.language).toBe('en');
-  });
-
   it('interpolation works for limitModal.limitReachedDesc', () => {
     const result = i18n.t('limitModal.limitReachedDesc', { current: 5, limit: 10 });
     expect(result).toContain('5');

src/test/pages/HistoryPage.test.tsx

@@ -114,9 +114,4 @@ describe('HistoryPage', () => {
     const { container } = render(<HistoryPage />);
     expect(container.querySelector('.max-w-5xl')).not.toBeNull();
   });
-
-  it('renders without crashing with empty arrays', () => {
-    const { container } = render(<HistoryPage />);
-    expect(container).toBeDefined();
-  });
 });