From 3f1a1b4868aff1b36f3b69f4d4c429373e28bc05 Mon Sep 17 00:00:00 2001
From: Pedro Filho <pedro@filho.me>
Date: Fri, 21 Aug 2026 13:04:27 -0300
Subject: [PATCH] chore: remove eslint-disable comments

The disables were papering over nextjs/no-img-element firing outside its scope. Docs now uses next/image (unoptimized, SVG logos); demo-vite gets a scoped oxlint override since next/image does not exist in a Vite app; the landing array-index-key disable was dead weight.
---
 apps/demo-vite/src/root.tsx                  | 2 --
 apps/docs/lib/layout.shared.tsx              | 9 +++++----
 apps/landing/app/_components/chart-panel.tsx | 7 +------
 oxlint.config.ts                             | 9 +++++++++
 4 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/apps/demo-vite/src/root.tsx b/apps/demo-vite/src/root.tsx
index b4b1da3..636d135 100644
--- a/apps/demo-vite/src/root.tsx
+++ b/apps/demo-vite/src/root.tsx
@@ -81,7 +81,6 @@ const RootLayout = (): ReactNode => (
   <div className="flex h-dvh w-full flex-col overflow-hidden bg-neutral-50 text-neutral-900 dark:bg-neutral-950 dark:text-neutral-100">
     <header className="flex shrink-0 items-center gap-3 border-b border-neutral-200 bg-white px-3 py-1.5 sm:px-4 dark:border-neutral-800 dark:bg-neutral-900">
       <Link aria-label="Homepage" className="shrink-0 pr-1" to="/">
-        {/* eslint-disable-next-line @next/next/no-img-element */}
         <img
           alt="dashfoo"
           className="block h-5 w-auto dark:hidden"
@@ -89,7 +88,6 @@ const RootLayout = (): ReactNode => (
           src="/dashfoo-logo-light.svg"
           width={105}
         />
-        {/* eslint-disable-next-line @next/next/no-img-element */}
         <img
           alt="dashfoo"
           className="hidden h-5 w-auto dark:block"
diff --git a/apps/docs/lib/layout.shared.tsx b/apps/docs/lib/layout.shared.tsx
index c022681..1397109 100644
--- a/apps/docs/lib/layout.shared.tsx
+++ b/apps/docs/lib/layout.shared.tsx
@@ -1,4 +1,5 @@
 import type { BaseLayoutProps } from "fumadocs-ui/layouts/shared";
+import Image from "next/image";
 
 const GITHUB_URL = "https://github.com/pedroapfilho/dashfoo";
 
@@ -21,20 +22,20 @@ export const baseOptions = (): BaseLayoutProps => ({
   nav: {
     title: (
       <>
-        {/* eslint-disable-next-line @next/next/no-img-element */}
-        <img
+        <Image
           alt="dashfoo"
           className="block h-5 w-auto dark:hidden"
           height={20}
           src="/dashfoo-logo-light.svg"
+          unoptimized
           width={105}
         />
-        {/* eslint-disable-next-line @next/next/no-img-element */}
-        <img
+        <Image
           alt="dashfoo"
           className="hidden h-5 w-auto dark:block"
           height={20}
           src="/dashfoo-logo-dark.svg"
+          unoptimized
           width={105}
         />
       </>
diff --git a/apps/landing/app/_components/chart-panel.tsx b/apps/landing/app/_components/chart-panel.tsx
index e8208b2..1f9b2c8 100644
--- a/apps/landing/app/_components/chart-panel.tsx
+++ b/apps/landing/app/_components/chart-panel.tsx
@@ -10,12 +10,7 @@ const ChartPanel = (): ReactNode => (
     </div>
     <div className="flex min-h-0 flex-1 items-end gap-1.5">
       {CHART_BARS.map((h, i) => (
-        <div
-          className="bg-accent flex-1 rounded-t-sm"
-          // eslint-disable-next-line react/no-array-index-key
-          key={i}
-          style={{ height: `${h}%` }}
-        />
+        <div className="bg-accent flex-1 rounded-t-sm" key={i} style={{ height: `${h}%` }} />
       ))}
     </div>
   </div>
diff --git a/oxlint.config.ts b/oxlint.config.ts
index 93362d3..d978b4a 100644
--- a/oxlint.config.ts
+++ b/oxlint.config.ts
@@ -21,6 +21,15 @@ export default defineConfig({
       },
     },
 
+    {
+      // demo-vite is a Vite app: next/image does not exist there, so the Next-specific
+      // img ban cannot be satisfied.
+      files: ["apps/demo-vite/**/*.tsx"],
+      rules: {
+        "nextjs/no-img-element": "off",
+      },
+    },
+
     {
       files: ["packages/react/src/hooks/responsive.ts"],
       rules: {