fix(lint): clear remaining biome errors (unused code, optional chains, non-null assertions, effect deps)

apps/api/src/plugins/auth.ts

@@ -326,7 +326,7 @@ export async function authRoutes(app: FastifyInstance): Promise<void> {
 
       const audit = auditFromRequest(request);
 
-      if (!user || !user.passwordHash) {
+      if (!user?.passwordHash) {
         authAttempts.inc({ method: "password", result: "failure" });
         await audit("LOGIN_FAILED", {
           username: sanitizeAuditInput(body.username),

apps/api/src/plugins/saml.ts

@@ -108,7 +108,7 @@ export async function registerSaml(app: FastifyInstance): Promise<void> {
       return redirectToLogin(reply, "saml_auth_failed");
     }
 
-    if (!profile || !profile.nameID) {
+    if (!profile?.nameID) {
       request.log.warn("SAML callback: no profile or nameID in assertion");
       authAttempts.inc({ method: "saml", result: "failure" });
       await audit("SAML_LOGIN_FAILED", { reason: "missing_profile" });

apps/api/src/routes/batch.ts

@@ -456,7 +456,8 @@ export async function registerBatchRoutes(app: FastifyInstance): Promise<void> {
       // Append results from object storage in original upload order
       try {
         for (const entry of successEntries) {
-          const stream = await getObjectStream(entry.outputRef!);
+          if (!entry.outputRef) continue;
+          const stream = await getObjectStream(entry.outputRef);
           archive.append(stream, { name: entry.filename });
         }
 

apps/api/src/routes/enterprise/siem.ts

@@ -4,7 +4,7 @@ import { z } from "zod";
 import { env } from "../../config.js";
 import { db, schema } from "../../db/index.js";
 import { auditFromRequest } from "../../lib/audit.js";
-import { encrypt, isEncrypted } from "../../lib/encryption.js";
+import { encrypt } from "../../lib/encryption.js";
 import { requirePermission } from "../../permissions.js";
 
 const configSchema = z.object({

apps/api/src/routes/pipeline.ts

@@ -1161,7 +1161,8 @@ export async function registerPipelineRoutes(app: FastifyInstance): Promise<void
     // Append results from object storage in original upload order
     try {
       for (const entry of successEntries) {
-        const stream = await getObjectStream(entry.outputRef!);
+        if (!entry.outputRef) continue;
+        const stream = await getObjectStream(entry.outputRef);
         archive.append(stream, { name: entry.filename });
       }
 

apps/api/src/routes/progress.ts

@@ -387,7 +387,7 @@ export async function registerProgressRoutes(app: FastifyInstance): Promise<void
       if (!sseListeners.has(jobId)) {
         sseListeners.set(jobId, new Set());
       }
-      sseListeners.get(jobId)!.add(callback);
+      sseListeners.get(jobId)?.add(callback);
 
       // Clean up on client disconnect
       request.raw.on("close", () => {

apps/api/src/routes/tools/html-to-image.ts

@@ -88,9 +88,10 @@ export function registerHtmlToImage(app: FastifyInstance) {
           isMobile: preset.isMobile,
         };
 
+        // Zod refine guarantees either url or html is present
         const buffer = settings.html
           ? await captureHtml(settings.html, captureOpts)
-          : await capturePage(settings.url!, captureOpts);
+          : await capturePage(settings.url ?? "", captureOpts);
 
         const jobId = randomUUID();
         const ext = settings.format;

apps/web/src/components/common/bottom-sheet.tsx

@@ -45,7 +45,7 @@ export function BottomSheet({
   }, [onClose]);
 
   const bind = useDrag(
-    ({ movement: [, my], last, cancel }) => {
+    ({ movement: [, my], last }) => {
       // Only allow downward dragging
       if (my < 0) {
         setTranslateY(0);

apps/web/src/components/common/dropzone.tsx

@@ -151,7 +151,7 @@ export function Dropzone({
       setUrlError(t.dropzone.urlFetchFailed);
     }
     setUrlLoading(false);
-  }, [urlInput, importSingleUrl, onUrlImport, checkFile, acceptDescription]);
+  }, [urlInput, importSingleUrl, onUrlImport, checkFile, acceptDescription, t]);
 
   const handleDrag = useCallback((e: DragEvent) => {
     e.preventDefault();

apps/web/src/components/common/non-native-preview.tsx

@@ -109,7 +109,7 @@ export function NonNativePreview({
     } finally {
       stopMessageRotation();
     }
-  }, [file, filename, previewUrl, startMessageRotation, stopMessageRotation]);
+  }, [file, src, filename, previewUrl, startMessageRotation, stopMessageRotation]);
 
   const ext = filename.split(".").pop()?.toUpperCase() ?? "";
   const IconComponent = modality === "audio" ? Volume2 : Video;

apps/web/src/components/common/route-announcer.tsx

@@ -6,6 +6,7 @@ export function RouteAnnouncer() {
   const isFirstRender = useRef(true);
   const announcerRef = useRef<HTMLDivElement>(null);
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: pathname is the intentional trigger for re-announcing on route change
   useEffect(() => {
     if (isFirstRender.current) {
       isFirstRender.current = false;

apps/web/src/components/editor/options/crop-options.tsx

@@ -23,7 +23,7 @@ export function CropOptions() {
       if (!cropState) return;
 
       const preset = ASPECT_RATIOS.find((p) => p.label === label);
-      if (!preset || !preset.value) {
+      if (!preset?.value) {
         setCropState({ ...cropState, aspectRatio: null });
         return;
       }

apps/web/src/components/editor/tools/crop-tool.tsx

@@ -71,7 +71,7 @@ export function useCropTool(): CropToolApi {
     (label: string) => {
       setAspectRatioState(label);
       const preset = ASPECT_RATIOS.find((p) => p.label === label);
-      if (!preset || !preset.value || !cropState) return;
+      if (!preset?.value || !cropState) return;
       const ratio = preset.value;
 
       let w = cropState.width;

apps/web/src/components/files/file-list.tsx

@@ -1,6 +1,5 @@
 import { Download, Search, Trash2 } from "lucide-react";
 import { useCallback, useEffect, useRef, useState } from "react";
-import { useNavigate } from "react-router-dom";
 import { useTranslation } from "@/contexts/i18n-context";
 import { getFileDownloadUrl } from "@/lib/api";
 import { format } from "@/lib/format";
@@ -22,7 +21,6 @@ export function FileList({ filterMimePrefix }: { filterMimePrefix?: string }) {
     setSearchQuery,
   } = useFilesPageStore();
 
-  const navigate = useNavigate();
   const [inputValue, setInputValue] = useState("");
   const debounceRef = useRef<ReturnType<typeof setTimeout> | null>(null);
   const listRef = useRef<HTMLDivElement>(null);

apps/web/src/components/tools/barcode-read-settings.tsx

@@ -1,9 +1,7 @@
 import { Check, Copy, Download, Search } from "lucide-react";
 import { useRef, useState } from "react";
 import { ProgressCard } from "@/components/common/progress-card";
-import { useTranslation } from "@/contexts/i18n-context";
 import { formatHeaders } from "@/lib/api";
-import { format } from "@/lib/format";
 import { copyToClipboard } from "@/lib/utils";
 import { useFileStore } from "@/stores/file-store";
 
@@ -108,7 +106,6 @@ function scanOneFile(
 }
 
 export function BarcodeReadSettings() {
-  const { t } = useTranslation();
   const { files, processing, error, setProcessing, setError } = useFileStore();
 
   const [tryHarder, setTryHarder] = useState(false);

apps/web/src/components/tools/border-settings.tsx

@@ -2,9 +2,7 @@ import { Download } from "lucide-react";
 import type React from "react";
 import { useCallback, useEffect, useRef, useState } from "react";
 import { ProgressCard } from "@/components/common/progress-card";
-import { useTranslation } from "@/contexts/i18n-context";
 import { useToolProcessor } from "@/hooks/use-tool-processor";
-import { format } from "@/lib/format";
 import { useFileStore } from "@/stores/file-store";
 
 // ── Presets ──────────────────────────────────────────────────────────
@@ -227,7 +225,6 @@ export function BorderControls({
   onChange,
   onImageStyle,
 }: BorderControlsProps) {
-  const { t } = useTranslation();
   const [selectedPreset, setSelectedPreset] = useState<string | null>(null);
   const [borderWidth, setBorderWidth] = useState(10);
   const [borderColor, setBorderColor] = useState("#000000");

apps/web/src/components/tools/collage-preview.tsx

@@ -51,7 +51,6 @@ function displayUrl(img: CollageImage): string {
 }
 
 export function CollagePreview() {
-  const { t } = useTranslation();
   const images = useCollageStore((s) => s.images);
   const templateId = useCollageStore((s) => s.templateId);
   const phase = useCollageStore((s) => s.phase);

apps/web/src/components/tools/collage-settings.tsx

@@ -1,7 +1,6 @@
 import { Download, Loader2 } from "lucide-react";
 import { useCallback } from "react";
 import { CollapsibleSection } from "@/components/common/collapsible-section";
-import { useTranslation } from "@/contexts/i18n-context";
 import { formatHeaders } from "@/lib/api";
 import {
   COLLAGE_TEMPLATES,
@@ -38,7 +37,6 @@ const BG_PRESETS = [
 ];
 
 export function CollageSettings() {
-  const { t } = useTranslation();
   const store = useCollageStore();
   const {
     images,

apps/web/src/components/tools/compare-settings.tsx

@@ -1,10 +1,8 @@
 import { Download, Loader2, Upload } from "lucide-react";
 import { useRef, useState } from "react";
-import { useTranslation } from "@/contexts/i18n-context";
 import { formatHeaders } from "@/lib/api";
 import { useFileStore } from "@/stores/file-store";
 export function CompareSettings() {
-  const { t } = useTranslation();
   const { files, processing, error, setProcessing, setError, setProcessedUrl } = useFileStore();
   const [secondFile, setSecondFile] = useState<File | null>(null);
   const [similarity, setSimilarity] = useState<number | null>(null);

apps/web/src/stores/features-store.ts

@@ -54,7 +54,7 @@ export const useFeaturesStore = create<FeaturesState>((set, get) => {
       try {
         await refreshBundles();
         const updated = get().bundles.find((b) => b.id === bundleId);
-        if (!updated || updated.status !== "installing") {
+        if (updated?.status !== "installing") {
           clearInterval(pollRefs[bundleId]);
           delete pollRefs[bundleId];