Polish: zero lint warnings, test coverage, CI improvements

- Suppress intentional console.error with eslint-disable comments
- Suppress mount-only useEffect dependency warnings
- Suppress no-img-element for external SVG badge
- Add @vitest/coverage-v8 with coverage report in CI
- Add test:coverage script

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: caishengold

.github/workflows/ci.yml

@@ -29,7 +29,7 @@ jobs:
           node-version: 20
           cache: npm
       - run: npm ci
-      - run: npm test
+      - run: npm run test:coverage
 
   build:
     name: Build

.gitignore

@@ -25,3 +25,6 @@ sdk/js/node_modules/
 
 # Test database
 data/test.db*
+
+# Coverage reports
+coverage/

app/agents/client.tsx

@@ -392,6 +392,7 @@ function ShareBar({ agentId, agentName }: { agentId: string; agentName: string }
           className="px-3 py-1.5 rounded-lg text-xs bg-white/5 hover:bg-white/10 text-white/50 hover:text-white transition-all">Social Card</a>
       </div>
       <div className="mt-3">
+        {/* eslint-disable-next-line @next/next/no-img-element */}
         <img src={badgeUrl} alt={`${agentName} badge`} className="mx-auto max-h-8 opacity-60 hover:opacity-100 transition-opacity" />
       </div>
     </div>

app/api/[...path]/route.ts

@@ -53,6 +53,7 @@ async function safeHandle(req: NextRequest, params: Promise<{ path: string[] }>)
     const { path } = await params;
     return await handle(req, path);
   } catch (e: any) {
+    // eslint-disable-next-line no-console
     console.error("API Error:", e);
     return json({ error: e.message || "Internal Server Error" }, 500);
   }

app/confessions/client.tsx

@@ -49,6 +49,7 @@ export default function ConfessionsClient({ initialConfessions, initialTotal }:
     setLoading(false);
   }, [confessions]);
 
+  // eslint-disable-next-line react-hooks/exhaustive-deps
   useEffect(() => { if (initialConfessions.length === 0) load('new', 0); }, []);
   const changeSort = (s: string) => { setSort(s); setPage(0); load(s, 0, query); };
   const doSearch = () => { setPage(0); load(sort, 0, query); };

app/leaderboard/client.tsx

@@ -17,6 +17,7 @@ export default function LeaderboardClient({ initialAgents, initialCouples, initi
   const [agents, setAgents] = useState<any[]>(initialAgents);
   const [loading, setLoading] = useState(false);
 
+  // eslint-disable-next-line react-hooks/exhaustive-deps
   useEffect(() => { if (initialAgents.length === 0) changeTab('popular'); }, []);
   const changeTab = async (t: string) => {
     setTab(t);

lib/api-server.ts

@@ -12,11 +12,13 @@ export async function apiFetch<T = any>(path: string, revalidateSec = 120): Prom
   try {
     const res = await fetch(url, { next: { revalidate: revalidateSec } });
     if (!res.ok) {
+      // eslint-disable-next-line no-console
       console.error(`[apiFetch] ${url} → ${res.status}`);
       return null;
     }
     return res.json();
   } catch (err) {
+    // eslint-disable-next-line no-console
     console.error(`[apiFetch] ${url} → error:`, err);
     return null;
   }

package-lock.json

@@ -8,6 +8,7 @@
     "start": "next start",
     "lint": "next lint",
     "test": "vitest run",
+    "test:coverage": "vitest run --coverage",
     "test:watch": "vitest",
     "format": "prettier --write .",
     "format:check": "prettier --check ."
@@ -27,6 +28,7 @@
     "@tailwindcss/postcss": "^4.1.18",
     "@types/node": "^20.14.2",
     "@types/react": "^18.3.3",
+    "@vitest/coverage-v8": "^4.0.18",
     "autoprefixer": "^10.4.24",
     "eslint": "^8.57.1",
     "eslint-config-next": "^14.2.5",

package.json

@@ -10,6 +10,12 @@ export default defineConfig({
     testTimeout: 15000,
     fileParallelism: false,
     sequence: { shuffle: false },
+    coverage: {
+      provider: "v8",
+      reporter: ["text", "lcov"],
+      include: ["lib/**", "middleware.ts"],
+      reportsDirectory: "./coverage",
+    },
   },
   resolve: {
     alias: {