chore: fix lint warnings

Author: vekexasia

web/public/sw.js

@@ -1,2 +1,2 @@
 // Empty service worker - placeholder to prevent 500 errors
-self.addEventListener('fetch', function(event) {});
+self.addEventListener('fetch', function() {});

web/src/app/[locale]/menu/MenuPageClient.test.tsx

@@ -21,8 +21,12 @@ vi.mock('@/lib/api', () => ({
 
 vi.mock('next/image', () => ({
   default: (props: Record<string, unknown>) => {
-    // eslint-disable-next-line @next/next/no-img-element, jsx-a11y/alt-text
-    const { src, alt, fill: _fill, priority: _priority, sizes: _sizes, unoptimized: _unoptimized, ...rest } = props;
+    const { src, alt, ...rest } = props;
+    delete rest.fill;
+    delete rest.priority;
+    delete rest.sizes;
+    delete rest.unoptimized;
+    // eslint-disable-next-line @next/next/no-img-element
     return <img src={src as string} alt={alt as string} {...rest} />;
   },
 }));

web/src/app/[locale]/menu/MenuPageClient.tsx

@@ -126,12 +126,6 @@ export default function MenuPageClient() {
     return publishedMenus[0];
   }, [data, typeParam, publishedMenus]);
 
-  const isVisible = (entry: MenuEntry) => {
-    if (!currentMenu) return false;
-    if (entry.hidden) return false;
-    return entry.menuIds.includes(currentMenu.id);
-  };
-
   const noticeConfig = data?.info?.menuNotice;
   const noticeEnabled = noticeConfig?.enabled !== false;
   const defaultNoticeText = `${t("allergyWarning")}\n\n${t("frozenIngredientsNote")}`;
@@ -140,6 +134,10 @@ export default function MenuPageClient() {
 
   const filteredCategories = useMemo(() => {
     if (!currentMenu) return [];
+    const isVisible = (entry: MenuEntry) => {
+      if (entry.hidden) return false;
+      return entry.menuIds.includes(currentMenu.id);
+    };
     return categories
       .map((cat) => ({
         ...cat,

web/src/app/admin/AdminContent.tsx

@@ -271,6 +271,7 @@ export default function AdminContent({
         <aside className="adm-sidebar" style={{ width: 230, background: "#fff", borderRight: "1px solid #E7E5E4", display: "flex", flexDirection: "column", padding: "14px 0", flexShrink: 0, overflowY: "auto" }}>
           {data?.headerImage && (
             <div style={{ margin: "0 10px 14px", borderRadius: 8, overflow: "hidden", position: "relative", height: 90, flexShrink: 0 }}>
+              {/* eslint-disable-next-line @next/next/no-img-element */}
               <img
                 src={data.headerImage}
                 alt={restaurantName}

web/src/components/admin/pages/CategoriesPage.tsx

@@ -105,7 +105,11 @@ export default function CategoriesPage() {
   const toggleSelect = (id: string) => {
     setSelected((prev) => {
       const n = new Set(prev);
-      n.has(id) ? n.delete(id) : n.add(id);
+      if (n.has(id)) {
+        n.delete(id);
+      } else {
+        n.add(id);
+      }
       return n;
     });
   };

web/src/components/admin/pages/MenusPage.tsx

@@ -1,6 +1,6 @@
 "use client";
 
-import { useEffect, useState } from "react";
+import { useCallback, useEffect, useState } from "react";
 import {
   ApiError,
   createMenu,
@@ -55,19 +55,19 @@ export default function MenusPage() {
   const [deleting, setDeleting] = useState<AdminMenu | null>(null);
   const [deletingId, setDeletingId] = useState<string | null>(null);
 
-  const refresh = async () => {
+  const refresh = useCallback(async () => {
     setLoadError(null);
     try {
       const res = await fetchMenus();
       setMenus(res.menus);
     } catch (err) {
       setLoadError(err instanceof Error ? err.message : t("menus.loadError"));
     }
-  };
+  }, [t]);
 
   useEffect(() => {
     refresh().finally(() => setLoading(false));
-  }, []);
+  }, [refresh]);
 
   // Compute per-menu entry counts using the public catalog the store already loaded.
   // (Memberships drive these counts, so categories alone aren't enough.)

web/src/components/admin/pages/SettingsPage.tsx

@@ -398,7 +398,8 @@ export default function SettingsPage({ section }: { section?: SettingsSection }
 
   const openLabelModal = (label: AdminLabel, tab: string) => {
     const raw = (label.i18n as Record<string, Record<string, string>>) ?? {};
-    const { [primaryLocale]: _omit, ...rest } = raw;
+    const rest = { ...raw };
+    delete rest[primaryLocale];
     setTranslatingLabel(label);
     setTranslatingLabelName(label.name);
     setTranslatingLabelColor(label.color);
@@ -409,7 +410,8 @@ export default function SettingsPage({ section }: { section?: SettingsSection }
   const handleSaveTranslations = async () => {
     if (!translatingLabel) return;
     try {
-      const { [primaryLocale]: _omit, ...i18nToSave } = translatingLabelI18n;
+      const i18nToSave = { ...translatingLabelI18n };
+      delete i18nToSave[primaryLocale];
       await updateLabel(translatingLabel.id, { name: translatingLabelName.trim(), color: translatingLabelColor, i18n: i18nToSave });
       setLabelsList((prev) => prev.map((l) => l.id === translatingLabel.id ? { ...l, name: translatingLabelName.trim(), color: translatingLabelColor, i18n: i18nToSave } : l));
       setTranslatingLabel(null);

web/src/components/ui/Flag.tsx

@@ -15,11 +15,13 @@ export function Flag({ code, customUrl, label, className, decorative }: FlagProp
   const ariaHidden = decorative ? true : undefined;
 
   if (customUrl) {
+    // eslint-disable-next-line @next/next/no-img-element
     return <img src={customUrl} alt={alt} className={cls} aria-hidden={ariaHidden} />;
   }
 
   const bundled = getBundledFlagSvg(code);
   if (bundled) {
+    // eslint-disable-next-line @next/next/no-img-element
     return <img src={svgToDataUrl(bundled)} alt={alt} className={cls} aria-hidden={ariaHidden} />;
   }
 

web/workers/chat/scripts/test-chat.ts

@@ -13,7 +13,6 @@ const RESTAURANT_ID = process.env.TEST_RESTAURANT_ID || 'demo-restaurant';
 // ANSI colours
 const RED = '\x1b[31m';
 const GREEN = '\x1b[32m';
-const YELLOW = '\x1b[33m';
 const CYAN = '\x1b[36m';
 const BOLD = '\x1b[1m';
 const RESET = '\x1b[0m';

web/workers/chat/src/index.ts

@@ -14,7 +14,7 @@ function json(data: unknown, status: number, corsHeaders: Record<string, string>
   });
 }
 
-export default {
+const worker = {
   async fetch(request: Request, env: Env, ctx: ExecutionContext): Promise<Response> {
     const url = new URL(request.url);
 
@@ -107,3 +107,5 @@ export default {
     return new Response('Not Found', { status: 404 });
   },
 };
+
+export default worker;