chore(web): clear dead code so real findings are visible

21 unused-variable warnings, and working out that none of them was a bug meant
reading each one. That is the actual cost: `'message' is assigned but never
used` in BlockManager looked exactly like a swallowed error message, and
`'err' is defined but never used` in LiveTaskPanel looked exactly like a
swallowed exception. Neither was — but noise that has to be triaged one by one
is noise that hides the one that matters.

Removed: dead lucide imports across nine components, two dead type imports in
the API client, a `FileInfo` interface and a `STATUS_META` table in
FileInspector left behind by a refactor, a `message` string in BlockManager
fully superseded by the `confirm({title, description})` call two lines below
it, and two locals in LiveTaskPanel shadowed by the merge underneath them.

MarkdownEditor held a `doc` state that was written and never read, so the whole
useState goes rather than the value being renamed to satisfy the linter.

The two catch-related ones are kept and explained instead. LiveTaskPanel's
silent poll catch is correct and the reasoning is now in the code: a poll that
fails every couple of seconds must not spam the user, and a persistent failure
is already visible because lastPollTime only advances on success, so the
panel's "Updated N ago" goes stale and says so without a toast.

50 eslint warnings to 32, with the unused-vars class at zero.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UiLDxnEu1Gf8Q5hWAFhaAd

Author: claude

web/src/api/client.ts

@@ -16,12 +16,10 @@ import type {
   StartRunResponse,
   LatestRunResponse,
   InterruptedRun,
-  RunEvent,
   Skill,
   AgentSpec,
   PipelineView,
   PipelineConfig,
-  DynamicComposition,
   CompositionGetResponse,
   CompositionPreviewResponse,
   DocItem,

web/src/components/Agents/AgentManager.tsx

@@ -7,7 +7,6 @@ import {
   Trash2,
   Edit3,
   Check,
-  X,
   Cpu,
   Wrench,
   Thermometer,

web/src/components/Blocks/BlockEditor.tsx

@@ -23,7 +23,6 @@ import {
   AlertCircle,
   Archive,
   Bot,
-  CheckCircle2,
   ChevronDown,
   ChevronUp,
   Code2,

web/src/components/Blocks/BlockManager.tsx

@@ -127,9 +127,6 @@ export default function BlockManager() {
 
   const handleDelete = async (block: BlockCatalogEntry) => {
     const title = KIND_TITLES[block.kind] || block.kind;
-    const message = block.custom
-      ? `Delete ${title.toLowerCase()} "${block.name || block.id}"?`
-      : `Delete ${title.toLowerCase()} "${block.name || block.id}"? This removes the override of the builtin block.`;
     const ok = await confirm({
       title: `Delete ${title.toLowerCase()} "${block.name || block.id}"?`,
       description: block.custom ? undefined : 'This removes the override of the builtin block.',

web/src/components/Docs/MarkdownEditor.tsx

@@ -1,12 +1,10 @@
 import { useState, useEffect, useCallback } from 'react';
 import { useParams } from 'react-router-dom';
-import { getDoc, updateDoc, listDocs } from '@/api/client';
-import type { DocItem } from '@/types';
+import { getDoc, updateDoc } from '@/api/client';
 import { Save, FileText, Loader } from 'lucide-react';
 
 export default function MarkdownEditor() {
   const { docId } = useParams<{ docId: string }>();
-  const [doc, setDoc] = useState<DocItem | null>(null);
   const [content, setContent] = useState('');
   const [loading, setLoading] = useState(true);
   const [saving, setSaving] = useState(false);
@@ -17,7 +15,6 @@ export default function MarkdownEditor() {
     setLoading(true);
     try {
       const d = await getDoc(docId);
-      setDoc(d);
       setContent(d.content || '');
     } catch (e) {
       console.error('Failed to load doc:', e);

web/src/components/Files/FileInspector.tsx

@@ -5,10 +5,8 @@ import {
   FileMinus,
   FilePenLine,
   MessageSquare,
-  GitBranch,
   PlusCircle,
   Send,
-  Clock,
   Flag,
   X,
   CornerDownRight,
@@ -21,21 +19,14 @@ import {
   Eye,
   EyeOff,
 } from 'lucide-react';
-import { addTask, updateDoc, getDoc, getWorkspaceFile, getWorkspaceTree } from '@/api/client';
+import { addTask, getWorkspaceFile, getWorkspaceTree } from '@/api/client';
 import type { RunEvent } from '@/types';
 import clsx from 'clsx';
 
 // ── Types ──
 
 type FileStatus = 'changed' | 'created' | 'deleted' | 'unchanged';
 
-interface FileInfo {
-  path: string;
-  status: FileStatus;
-  events: RunEvent[];
-  lastEvent: RunEvent | null;
-}
-
 interface LineComment {
   id: string;
   line: number;
@@ -83,15 +74,6 @@ function extractModifiedPaths(events: RunEvent[]): Set<string> {
   return paths;
 }
 
-// ── Status helpers ──
-
-const STATUS_META: Record<FileStatus, { icon: typeof FileCode; label: string; cls: string; dotCls: string }> = {
-  changed:   { icon: FilePenLine, label: 'Modified',  cls: 'text-amber-600 dark:text-amber-400', dotCls: 'bg-amber-400' },
-  created:   { icon: FilePlus,    label: 'Created',   cls: 'text-emerald-600 dark:text-emerald-400', dotCls: 'bg-emerald-400' },
-  deleted:   { icon: FileMinus,   label: 'Deleted',   cls: 'text-red-600 dark:text-red-400', dotCls: 'bg-red-400' },
-  unchanged: { icon: FileCode,    label: '',           cls: 'text-gray-400 dark:text-gray-500', dotCls: 'bg-gray-300 dark:bg-gray-600' },
-};
-
 // ── Simple syntax highlighter ──
 
 interface Token { text: string; type: 'keyword' | 'string' | 'comment' | 'number' | 'function' | 'type' | 'plain'; }

web/src/components/Live/LiveFileInspector.tsx

@@ -5,7 +5,6 @@ import {
   GitBranch,
   PlusCircle,
   Flag,
-  Eye,
   ChevronDown,
   ChevronRight,
   Send,

web/src/components/Live/LiveTaskPanel.tsx

@@ -16,7 +16,6 @@ import type { Task, AgentSpec, Board } from '@/types';
 import clsx from 'clsx';
 import {
   Plus,
-  Edit3,
   Trash2,
   Save,
   X,
@@ -192,8 +191,6 @@ export default function LiveTaskPanel() {
       setOptimisticTasks((prev) => {
         if (!prev || !Array.isArray(prev)) return [];
         const tasks = b.tasks || [];
-        const cols = b.columns || [];
-        const byCol = b.by_column || {};
         const serverIds = new Set(tasks.map((t) => t.id));
         // Keep only optimistic tasks whose IDs are not yet in the server response
         const stillMissing = prev.filter((t) => !serverIds.has(t.id));
@@ -239,8 +236,11 @@ export default function LiveTaskPanel() {
       });
 
       setLastPollTime(Date.now());
-    } catch (err) {
-      // silently ignore polling errors
+    } catch {
+      // Deliberately silent. A poll that fails every couple of seconds must not
+      // spam the user, and a persistent failure is already visible: lastPollTime
+      // only advances on success, so the panel's "Updated N ago" goes stale and
+      // says so without a single toast.
     } finally {
       setTasksLoading(false);
       setRefreshing(false);

web/src/components/Review/ReviewView.tsx

@@ -13,7 +13,7 @@ import {
 import clsx from 'clsx';
 import { AppContext } from '@/App';
 import { applyPendingChanges, getPendingReview, rejectPendingChanges } from '@/api/client';
-import type { PendingChange, ReviewQueue } from '@/types';
+import type { ReviewQueue } from '@/types';
 import { useToast } from '@/components/ui/Toast';
 import { useConfirm } from '@/components/ui/Modal';
 import DiffView, { type DiffMode } from './DiffView';

web/src/components/Settings/SettingsPanel.tsx

@@ -10,7 +10,6 @@ import AutoConfigure from './AutoConfigure';
 import {
   Cpu,
   Key,
-  Sliders,
   ShieldCheck,
   Gauge,
   Wrench,