fix: post-React 19 cleanup — remove dead deps, wire up ESLint plugins, modernize entry

.github/workflows/ci.yml

@@ -92,8 +92,7 @@ jobs:
     - name: Run type checking
       run: |
         cd frontend
-        echo "Skipping TypeScript type checking for now - will fix type issues in a separate PR"
-        # npx tsc --noEmit
+        npx tsc --noEmit
 
     - name: Run tests
       run: |
@@ -137,8 +136,7 @@ jobs:
     - name: Run npm audit
       run: |
         cd frontend
-        echo "Skipping npm audit for now - will address security issues in a separate PR"
-        # npm audit --audit-level=moderate
+        npm audit --audit-level=moderate
 
   dependency-check:
     runs-on: ubuntu-latest

frontend/eslint.config.js

@@ -1,6 +1,8 @@
 import js from '@eslint/js'
 import typescript from '@typescript-eslint/eslint-plugin'
 import typescriptParser from '@typescript-eslint/parser'
+import reactHooks from 'eslint-plugin-react-hooks'
+import reactRefresh from 'eslint-plugin-react-refresh'
 
 export default [
   js.configs.recommended,
@@ -28,11 +30,16 @@ export default [
       }
     },
     plugins: {
-      '@typescript-eslint': typescript
+      '@typescript-eslint': typescript,
+      'react-hooks': reactHooks,
+      'react-refresh': reactRefresh
     },
     rules: {
       'no-unused-vars': 'off',
-      '@typescript-eslint/no-unused-vars': 'warn'
+      '@typescript-eslint/no-unused-vars': 'warn',
+      'react-hooks/rules-of-hooks': 'error',
+      'react-hooks/exhaustive-deps': 'warn',
+      'react-refresh/only-export-components': ['warn', { allowConstantExport: true }]
     }
   },
   {

frontend/package.json

@@ -18,7 +18,6 @@
     "axios": "^1.13.2",
     "react": "^19.2.5",
     "react-dom": "^19.2.5",
-    "react-router-dom": "^7.14.2",
     "tailwindcss": "^4.1.17"
   },
   "devDependencies": {
@@ -31,7 +30,6 @@
     "@typescript-eslint/parser": "^8.59.1",
     "@vitejs/plugin-react": "^6.0.1",
     "@vitest/coverage-v8": "^4.0.10",
-    "autoprefixer": "^10.4.16",
     "eslint": "^10.2.1",
     "eslint-plugin-react-hooks": "^7.1.1",
     "eslint-plugin-react-refresh": "^0.5.2",

frontend/postcss.config.js

@@ -1,6 +1,5 @@
 export default {
   plugins: {
     '@tailwindcss/postcss': {},
-    autoprefixer: {},
   },
 }

frontend/src/main.tsx

@@ -1,10 +1,10 @@
-import React from 'react'
-import ReactDOM from 'react-dom/client'
+import { StrictMode } from 'react'
+import { createRoot } from 'react-dom/client'
 import App from './App.tsx'
 import './index.css'
 
-ReactDOM.createRoot(document.getElementById('root')!).render(
-  <React.StrictMode>
+createRoot(document.getElementById('root')!).render(
+  <StrictMode>
     <App />
-  </React.StrictMode>,
+  </StrictMode>,
 )

frontend/src/services/api.ts

@@ -9,7 +9,7 @@ import type {
   ScriptureSearch
 } from '../types/scripture';
 
-const API_BASE_URL = (import.meta as any).env?.VITE_API_URL || 'http://localhost:8000';
+const API_BASE_URL = import.meta.env.VITE_API_URL || 'http://localhost:8000';
 
 const api = axios.create({
   baseURL: API_BASE_URL,

frontend/src/vite-env.d.ts

@@ -1 +1,9 @@
 /// <reference types="vite/client" />
+
+interface ImportMetaEnv {
+  readonly VITE_API_URL: string
+}
+
+interface ImportMeta {
+  readonly env: ImportMetaEnv
+}