zevairouter 1.0.103 → 1.0.105

package/src/app/(dashboard)/dashboard/providers/page.js

@@ -142,6 +142,7 @@ export default function ProvidersPage() {
   }, [registerSearch, unregisterSearch]);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- pagination reset: returns to page 1 when search query or page size changes
     setApikeyPage(1);
   }, [searchQuery, apikeyPageSize]);
 
@@ -947,7 +948,13 @@ function AddOpenAICompatibleModal({ isOpen, onClose, onCreated }) {
 
   useEffect(() => {
     const defaultBaseUrl = "https://api.openai.com/v1";
-    setFormData((prev) => ({ ...prev, baseUrl: defaultBaseUrl }));
+    // Only reset baseUrl when it was not customized — the previous effect body
+    // stomped user input on every apiType change.
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- syncs baseUrl default when apiType changes; functional update bails out (returns prev) when the user customized the URL
+    setFormData((prev) => {
+      if (prev.baseUrl && prev.baseUrl !== defaultBaseUrl) return prev;
+      return { ...prev, baseUrl: defaultBaseUrl };
+    });
   }, [formData.apiType]);
 
   const handleSubmit = async () => {
@@ -1136,6 +1143,7 @@ function AddAnthropicCompatibleModal({ isOpen, onClose, onCreated }) {
 
   useEffect(() => {
     if (isOpen) {
+      // eslint-disable-next-line react-hooks/set-state-in-effect -- reset-on-open: clears validation state each time the modal is reopened
       setValidationResult(null);
       setCheckKey("");
       setCheckModelId("");

package/src/app/(dashboard)/dashboard/proxy-pools/page.js

@@ -85,6 +85,7 @@ export default function ProxyPoolsPage() {
   }, []);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- mount data fetch; fetchProxyPools sets state only after await, which the rule flags conservatively
     fetchProxyPools();
   }, [fetchProxyPools]);
 
@@ -342,10 +343,12 @@ export default function ProxyPoolsPage() {
 
   // Cleanup selectedIds when pools change
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- selection pruning: drops selected ids that no longer exist after pools refresh
     setSelectedIds((prev) => prev.filter((id) => proxyPools.some((p) => p.id === id)));
   }, [proxyPools]);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- pagination clamp: keeps page within range when the pool list shrinks
     setPage((current) => Math.min(current, Math.max(1, Math.ceil(proxyPools.length / pageSize))));
   }, [proxyPools.length, pageSize]);
 
@@ -924,7 +927,7 @@ export default function ProxyPoolsPage() {
             value={cloudflareForm.apiToken}
             onChange={(e) => setCloudflareForm((prev) => ({ ...prev, apiToken: e.target.value }))}
             placeholder="your-cloudflare-api-token"
-            hint={<>Requires "Workers Scripts: Edit" permission. <a href="https://dash.cloudflare.com/profile/api-tokens" target="_blank" rel="noopener noreferrer" className="text-primary hover:underline">Get token →</a></>}
+            hint={<>Requires &quot;Workers Scripts: Edit&quot; permission. <a href="https://dash.cloudflare.com/profile/api-tokens" target="_blank" rel="noopener noreferrer" className="text-primary hover:underline">Get token →</a></>}
             type="password"
           />
           <Input

package/src/app/(dashboard)/dashboard/pxpipe/PxpipeClient.js

@@ -87,6 +87,7 @@ export default function PxpipeClient() {
   }, []);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- mount data fetch; refresh() sets state only after await, which the rule flags conservatively
     refresh();
   }, [refresh]);
 

package/src/app/(dashboard)/dashboard/terminal/TerminalPageClient.js

@@ -65,6 +65,7 @@ export default function TerminalPageClient() {
   }, []);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- mount gate check; checkGate sets state only after await, which the rule flags conservatively
     checkGate();
   }, [checkGate]);
 

package/src/app/(dashboard)/dashboard/token-saver/TokenSaverClient.js

@@ -62,6 +62,7 @@ export default function TokenSaverClient() {
   const { copied, copy } = useCopyToClipboard();
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- mount init from locale cookie (external store) + subscription to locale-change events
     setLocale(getCurrentLocale());
     return onLocaleChange(() => setLocale(getCurrentLocale()));
   }, []);
@@ -74,7 +75,9 @@ export default function TokenSaverClient() {
   useEffect(() => {
     const current = CAVEMAN_LEVELS.find((lvl) => lvl.id === cavemanLevel);
     if (current?.wenyan && !isWenyanLocale) {
+      // eslint-disable-next-line react-hooks/set-state-in-effect -- syncs persisted setting to a valid value: wenyan-only level is invalid after switching to a non-wenyan locale
       setCavemanLevel("ultra");
+      // eslint-disable-next-line react-hooks/immutability -- false positive: these are local fetch+setState helpers invoked once on effect fire (not during render); defined with const below because all invocations are deferred (effects/events), so no TDZ issue at runtime
       patchSetting({ cavemanLevel: "ultra" });
     }
   }, [isWenyanLocale, cavemanLevel]);

package/src/app/(dashboard)/dashboard/usage/components/ProviderLimits/QuotaTable.js

@@ -109,10 +109,12 @@ export default function QuotaTable({
   const totalPages = Math.max(1, Math.ceil(sortedQuotas.length / PAGE_SIZE));
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- pagination reset: returns to page 1 when sort mode or data changes
     setPage(1);
   }, [sortMode, quotas]);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- pagination clamp: keeps page within range when total pages shrink
     setPage((currentPage) => Math.min(currentPage, totalPages));
   }, [totalPages]);
 

package/src/app/(dashboard)/dashboard/usage/components/ProviderTopology.js

@@ -1,4 +1,5 @@
 "use client";
+/* eslint-disable @next/next/no-img-element -- dynamic/blob/external image src with onError fallback; next/image requires static dims + domain allowlist */
 
 import { useMemo, useState, useEffect, useCallback, useRef } from "react";
 import PropTypes from "prop-types";
@@ -14,6 +15,7 @@ import { AI_PROVIDERS } from "@/shared/constants/providers";
 // Force-stop FE animation if a provider stays active longer than this
 const FE_ACTIVE_TIMEOUT_MS = 60000;
 const FE_ACTIVE_TICK_MS = 1000;
+const FIT_OPTS = { padding: 0.2, duration: 200 };
 
 function getProviderConfig(providerId) {
   return AI_PROVIDERS[providerId] || { color: "#6b7280", name: providerId };
@@ -230,18 +232,22 @@ export default function ProviderTopology({ providers = [], activeRequests = [],
   }, [rawActiveSet]);
 
   const activeSet = useMemo(() => {
+    void tick; // tick only re-triggers this memo so time-based expiry is re-evaluated on each interval
+    // eslint-disable-next-line react-hooks/purity -- intentional time-window filter: memo is re-evaluated every FE_ACTIVE_TICK_MS via tick, so each evaluation needs the current time
     const now = Date.now();
     const filtered = new Set();
+    /* eslint-disable react-hooks/refs -- firstSeenRef is written in an effect and read here only to re-derive a time-windowed view on each tick; intentional */
     for (const p of rawActiveSet) {
       const ts = firstSeenRef.current[p];
       if (!ts || now - ts < FE_ACTIVE_TIMEOUT_MS) filtered.add(p);
     }
+    /* eslint-enable react-hooks/refs */
     return filtered;
   }, [rawActiveSet, tick]);
 
   const { nodes, edges } = useMemo(
     () => buildLayout(providers, activeSet, lastSet, errorSet),
-    [providers, activeSet, lastKey, errorKey]
+    [providers, activeSet, lastSet, errorSet]
   );
 
   // Stable key — only remount when provider list changes
@@ -252,10 +258,9 @@ export default function ProviderTopology({ providers = [], activeRequests = [],
 
   const rfInstance = useRef(null);
   const containerRef = useRef(null);
-  const fitOpts = { padding: 0.2, duration: 200 };
   const onInit = useCallback((instance) => {
     rfInstance.current = instance;
-    setTimeout(() => instance.fitView(fitOpts), 50);
+    setTimeout(() => instance.fitView(FIT_OPTS), 50);
   }, []);
 
   // Re-fit on container resize
@@ -263,7 +268,7 @@ export default function ProviderTopology({ providers = [], activeRequests = [],
     const el = containerRef.current;
     if (!el) return;
     const ro = new ResizeObserver(() => {
-      if (rfInstance.current) rfInstance.current.fitView(fitOpts);
+      if (rfInstance.current) rfInstance.current.fitView(FIT_OPTS);
     });
     ro.observe(el);
     return () => ro.disconnect();
@@ -272,7 +277,7 @@ export default function ProviderTopology({ providers = [], activeRequests = [],
   // Re-fit when node count/layout changes
   useEffect(() => {
     if (rfInstance.current) {
-      const id = setTimeout(() => rfInstance.current.fitView(fitOpts), 50);
+      const id = setTimeout(() => rfInstance.current.fitView(FIT_OPTS), 50);
       return () => clearTimeout(id);
     }
   }, [nodes.length]);

package/src/app/(dashboard)/dashboard/usage/components/RequestDetailsTab.js

@@ -144,10 +144,12 @@ export default function RequestDetailsTab() {
   }, [pagination.page, pagination.pageSize, filters]);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- mount data fetch; fetchProviders sets state only after await, which the rule flags conservatively
     fetchProviders();
   }, [fetchProviders]);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- data fetch on filter/page change; fetchDetails sets state only after await, which the rule flags conservatively
     fetchDetails();
   }, [fetchDetails]);
 

package/src/app/(dashboard)/dashboard/usage/components/UsageChart.js

@@ -43,6 +43,7 @@ export default function UsageChart({ period = "7d" }) {
   }, [period]);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- data fetch on period change; fetchData sets state only after await, which the rule flags conservatively
     fetchData();
   }, [fetchData]);
 

package/src/app/(dashboard)/dashboard/usage/components/UsageTable.js

@@ -110,6 +110,7 @@ export default function UsageTable({
   useEffect(() => {
     try {
       const saved = localStorage.getItem(storageKey);
+      // eslint-disable-next-line react-hooks/set-state-in-effect -- one-time mount initialization from localStorage (external store), intentionally client-only
       if (saved) setExpanded(new Set(JSON.parse(saved)));
     } catch (e) {
       console.error(`Failed to load ${storageKey}:`, e);

package/src/app/api/changelog/route.js

@@ -0,0 +1,48 @@
+import { NextResponse } from "next/server";
+import fs from "fs";
+import path from "path";
+
+export const dynamic = "force-dynamic";
+
+/**
+ * Serve the bundled CHANGELOG.md from the installed package root.
+ * This avoids depending on raw.githubusercontent.com, which 404s for
+ * private repos and requires internet access — self-hosted installs
+ * should always be able to show their own changelog.
+ */
+function findChangelog() {
+  const candidates = [
+    // Standalone: server runs from <pkg>/cli/app → package root is ../..
+    path.resolve(process.cwd(), "..", "..", "CHANGELOG.md"),
+    // Dev: cwd is repo/package root
+    path.join(process.cwd(), "CHANGELOG.md"),
+    // One level up (some standalone layouts)
+    path.resolve(process.cwd(), "..", "CHANGELOG.md"),
+    // Bundled copy inside cli/app (if build copied it)
+    path.resolve(process.cwd(), "cli", "app", "CHANGELOG.md"),
+  ];
+  for (const p of candidates) {
+    try {
+      if (fs.existsSync(p)) return p;
+    } catch { /* keep looking */ }
+  }
+  return null;
+}
+
+export async function GET() {
+  const file = findChangelog();
+  if (!file) {
+    return NextResponse.json({ error: "CHANGELOG.md not found" }, { status: 404 });
+  }
+  try {
+    const content = fs.readFileSync(file, "utf8");
+    return new NextResponse(content, {
+      headers: {
+        "Content-Type": "text/markdown; charset=utf-8",
+        "Cache-Control": "public, max-age=300",
+      },
+    });
+  } catch (e) {
+    return NextResponse.json({ error: e.message }, { status: 500 });
+  }
+}

package/src/app/api/oauth/cursor/auto-import/route.js

@@ -78,7 +78,6 @@ const normalize = (value) => {
  */
 function extractTokensViaBetterSqlite(dbPath) {
   // Dynamic require so the route stays importable even if native bindings fail
-  // eslint-disable-next-line @typescript-eslint/no-require-imports
   const Database = require("better-sqlite3");
   const db = new Database(dbPath, { readonly: true, fileMustExist: true });
 

package/src/app/callback/page.js

@@ -75,6 +75,7 @@ function CallbackContent() {
       return;
     }
 
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- OAuth popup relay: status transitions are the entire point of this one-shot callback effect
     setStatus("success");
     setTimeout(() => {
       window.close();

package/src/app/dashboard/settings/pricing/page.js

@@ -12,6 +12,7 @@ export default function PricingSettingsPage() {
   const [loading, setLoading] = useState(true);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/immutability -- false positive: these are local fetch+setState helpers invoked once on effect fire (not during render); defined with const below because all invocations are deferred (effects/events), so no TDZ issue at runtime
     loadPricing();
   }, []);
 

package/src/lib/db/migrations/001-initial.js

@@ -2,7 +2,7 @@
 // For existing DB at version 0 (legacy unstamped), it's idempotent (IF NOT EXISTS).
 import { TABLES, buildCreateTableSql } from "../schema.js";
 
-export default {
+const M001Initial = {
   version: 1,
   name: "initial",
   up(db) {
@@ -12,3 +12,6 @@ export default {
     }
   },
 };
+
+
+export default M001Initial;

package/src/shared/components/AddCustomEmbeddingModal.js

@@ -22,6 +22,7 @@ export default function AddCustomEmbeddingModal({ isOpen, onClose, onCreated, on
 
   useEffect(() => {
     if (!isOpen) return;
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- reset-on-open + sync-from-props hybrid: reinitializes the form whenever the modal opens or the edited node changes
     setValidationResult(null);
     setCheckKey("");
     setCheckModelId("");

package/src/shared/components/BulkAccountAutomationModal.js

@@ -110,6 +110,7 @@ export default function BulkAccountAutomationModal({
       const saved = window.localStorage.getItem(draftKey);
       if (saved) {
         bulkTextRef.current = saved;
+        // eslint-disable-next-line react-hooks/set-state-in-effect -- one-time restore of draft text from localStorage (external store) on mount
         setBulkTextState(saved);
       }
     } catch { /* ignore */ }
@@ -160,9 +161,10 @@ export default function BulkAccountAutomationModal({
     if (typeof window !== "undefined") {
       window.localStorage.removeItem(storageKey);
     }
-  }, [storageKey, defaultEngine]);
+  }, [storageKey, defaultEngine, setBulkText]);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- sync-from-props: resets engine selection when the provider/default engine changes
     setEngine(defaultEngine || DEFAULT_ENGINE);
   }, [defaultEngine, provider]);
 

package/src/shared/components/ChangelogModal.js

@@ -16,13 +16,24 @@ export default function ChangelogModal({ isOpen, onClose }) {
 
   useEffect(() => {
     if (!isOpen || html) return;
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- fetch-on-open: loading must flip synchronously before the async fetch resolves
     setLoading(true);
     setError("");
-    fetch(`${GITHUB_CONFIG.changelogUrl}?t=${Date.now()}`, { cache: "no-store" })
+    // Prefer the locally-bundled CHANGELOG (works offline + private repos).
+    // Fall back to GitHub raw only if the local file isn't reachable.
+    const tryLocal = fetch(`/api/changelog?t=${Date.now()}`, { cache: "no-store" })
       .then((res) => {
-        if (!res.ok) throw new Error(`HTTP ${res.status}`);
+        if (!res.ok) throw new Error(`local ${res.status}`);
         return res.text();
-      })
+      });
+    tryLocal
+      .catch(() =>
+        fetch(`${GITHUB_CONFIG.changelogUrl}?t=${Date.now()}`, { cache: "no-store" })
+          .then((res) => {
+            if (!res.ok) throw new Error(`HTTP ${res.status}`);
+            return res.text();
+          })
+      )
       .then((md) => setHtml(marked.parse(md)))
       .catch((err) => setError(err.message || "Failed to load"))
       .finally(() => setLoading(false));

package/src/shared/components/CursorAuthModal.js

@@ -46,6 +46,7 @@ export default function CursorAuthModal({ isOpen, onSuccess, onClose }) {
   // Auto-detect tokens when modal opens
   useEffect(() => {
     if (!isOpen) return;
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- fetch-on-open: runAutoDetect sets state only after await, which the rule flags conservatively
     runAutoDetect();
   }, [isOpen]);
 

package/src/shared/components/EditConnectionModal.js

@@ -29,6 +29,7 @@ export default function EditConnectionModal({ isOpen, connection, proxyPools, on
 
   useEffect(() => {
     if (connection) {
+      // eslint-disable-next-line react-hooks/set-state-in-effect -- sync-from-props hybrid: pre-fills form from the connection being edited; user edits fields independently afterwards
       setFormData({
         name: connection.name || "",
         priority: connection.priority || 1,

package/src/shared/components/HeaderLanguage.js

@@ -19,6 +19,7 @@ export default function HeaderLanguage() {
   const [locale, setLocale] = useState("en");
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- syncs locale from cookie (external store) on mount and whenever the language modal closes
     setLocale(getLocaleFromCookie());
   }, [open]);
 

package/src/shared/components/KiroAuthModal.js

@@ -144,6 +144,7 @@ export default function KiroAuthModal({
   useEffect(() => {
     if (!isOpen) return;
     if (!initialSelectedMethod) return;
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- sync-from-props: applies the requested initial auth method when the modal flow changes
     setSelectedMethod(initialSelectedMethod);
     setImportMode(initialImportMode || "single-token");
     setBulkResult(null);

package/src/shared/components/KiroOAuthWrapper.js

@@ -27,6 +27,7 @@ export default function KiroOAuthWrapper({
   useEffect(() => {
     if (!isOpen || !initialFlow) return;
     if (initialFlow.method === "builder-id") {
+      // eslint-disable-next-line react-hooks/set-state-in-effect -- sync-from-props: restores the requested auth flow when the modal is opened with an initialFlow
       setAuthMethod("builder-id");
       return;
     }

package/src/shared/components/LanguageSwitcher.js

@@ -71,6 +71,7 @@ export default function LanguageSwitcher({ className = "", isOpen: controlledOpe
   };
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- one-time mount initialization from locale cookie (external store), intentionally client-only
     setLocale(getLocaleFromCookie());
   }, []);
 
@@ -85,6 +86,7 @@ export default function LanguageSwitcher({ className = "", isOpen: controlledOpe
       document.addEventListener("mousedown", handleClickOutside);
       return () => document.removeEventListener("mousedown", handleClickOutside);
     }
+    // eslint-disable-next-line react-hooks/exhaustive-deps -- react only to isOpen; setIsOpen only ever closes (idempotent) so a fresh closure isn't required
   }, [isOpen]);
 
   const handleSetLocale = async (nextLocale) => {

package/src/shared/components/McpMarketplaceModal.js

@@ -29,7 +29,7 @@ export default function McpMarketplaceModal({ isOpen, onClose, onAdd, addedNames
       })
       .catch((e) => setError(e.message))
       .finally(() => setLoading(false));
-  }, [isOpen]);
+  }, [isOpen, servers.length]);
 
   const addedSet = useMemo(() => new Set(addedNames), [addedNames]);
 

package/src/shared/components/ModelSelectModal.js

@@ -59,6 +59,7 @@ export default function ModelSelectModal({
   };
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- fetch-on-open: fetchCombos sets state only after await, which the rule flags conservatively
     if (isOpen) fetchCombos();
   }, [isOpen]);
 
@@ -75,6 +76,7 @@ export default function ModelSelectModal({
   };
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- fetch-on-open: fetchProviderNodes sets state only after await, which the rule flags conservatively
     if (isOpen) fetchProviderNodes();
   }, [isOpen]);
 
@@ -91,6 +93,7 @@ export default function ModelSelectModal({
   };
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- fetch-on-open: fetchCustomModels sets state only after await, which the rule flags conservatively
     if (isOpen) fetchCustomModels();
   }, [isOpen]);
 
@@ -107,6 +110,7 @@ export default function ModelSelectModal({
   };
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- fetch-on-open: fetchDisabledModels sets state only after await, which the rule flags conservatively
     if (isOpen) fetchDisabledModels();
   }, [isOpen]);
 
@@ -319,16 +323,16 @@ export default function ModelSelectModal({
   }, [combos, searchQuery, kindFilter]);
 
   // Sort models alphabetically, with added models floated to top
-  const sortModels = (models) => {
-    const added = models.filter(m => addedModelValues.includes(m.value)).sort((a, b) => a.name.localeCompare(b.name));
-    const rest = models.filter(m => !addedModelValues.includes(m.value)).sort((a, b) => a.name.localeCompare(b.name));
-    return [...added, ...rest];
-  };
-
   // Filter models by search query
   const filteredGroups = useMemo(() => {
     const query = searchQuery.trim().toLowerCase();
 
+    const sortModels = (models) => {
+      const added = models.filter(m => addedModelValues.includes(m.value)).sort((a, b) => a.name.localeCompare(b.name));
+      const rest = models.filter(m => !addedModelValues.includes(m.value)).sort((a, b) => a.name.localeCompare(b.name));
+      return [...added, ...rest];
+    };
+
     const filtered = {};
     Object.entries(groupedModels).forEach(([providerId, group]) => {
       let models = group.models;

package/src/shared/components/OAuthModal.js

@@ -30,6 +30,7 @@ export default function OAuthModal({ isOpen, provider, providerInfo, onSuccess,
   // Detect if running on localhost (client-side only)
   useEffect(() => {
     if (typeof window !== "undefined") {
+      // eslint-disable-next-line react-hooks/set-state-in-effect -- one-time mount initialization from window.location (external source, client-only, avoids hydration mismatch)
       setIsLocalhost(
         window.location.hostname === "localhost" || window.location.hostname === "127.0.0.1"
       );
@@ -64,7 +65,7 @@ export default function OAuthModal({ isOpen, provider, providerInfo, onSuccess,
       setError(err.message);
       setStep("error");
     }
-  }, [authData, provider, onSuccess]);
+  }, [authData, provider, onSuccess, oauthMeta]);
 
   const completeXaiManualCode = useCallback(async (code) => {
     if (!authData?.state) return;
@@ -310,6 +311,7 @@ export default function OAuthModal({ isOpen, provider, providerInfo, onSuccess,
   // Reset state and start OAuth when modal opens
   useEffect(() => {
     if (isOpen && provider) {
+      // eslint-disable-next-line react-hooks/set-state-in-effect -- reset-on-open: clears previous flow state before starting a new OAuth flow
       setAuthData(null);
       setCallbackUrl("");
       setError(null);

package/src/shared/components/PricingModal.js

@@ -10,6 +10,7 @@ export default function PricingModal({ isOpen, onClose, onSave }) {
 
   useEffect(() => {
     if (isOpen) {
+      // eslint-disable-next-line react-hooks/immutability -- false positive: these are local fetch+setState helpers invoked once on effect fire (not during render); defined with const below because all invocations are deferred (effects/events), so no TDZ issue at runtime
       loadPricing();
     }
   }, [isOpen]);

package/src/shared/components/ProviderIcon.js

@@ -1,6 +1,7 @@
 "use client";
+/* eslint-disable @next/next/no-img-element -- dynamic/blob/external image src with onError fallback; next/image requires static dims + domain allowlist */
 
-import { useEffect, useState } from "react";
+import { useState } from "react";
 import PropTypes from "prop-types";
 
 export default function ProviderIcon({
@@ -14,10 +15,14 @@ export default function ProviderIcon({
   const [errored, setErrored] = useState(false);
   const [imageSrc, setImageSrc] = useState(src);
 
-  useEffect(() => {
+  // Reset error/fallback state when `src` changes by adjusting state during
+  // render (React docs pattern) instead of an effect + extra re-render.
+  const [prevSrc, setPrevSrc] = useState(src);
+  if (prevSrc !== src) {
+    setPrevSrc(src);
     setErrored(false);
     setImageSrc(src);
-  }, [src]);
+  }
 
   if (!imageSrc || errored) {
     return (

package/src/shared/components/RequestLogger.js

@@ -9,6 +9,7 @@ export default function RequestLogger() {
   const [autoRefresh, setAutoRefresh] = useState(true);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/immutability -- false positive: these are local fetch+setState helpers invoked once on effect fire (not during render); defined with const below because all invocations are deferred (effects/events), so no TDZ issue at runtime
     fetchLogs();
   }, []);
 

package/src/shared/components/Sidebar.js

@@ -84,6 +84,7 @@ export default function Sidebar({ onClose }) {
   }, []);
 
   useEffect(() => {
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- route-driven UI sync: auto-expands the Advanced section when navigating to an advanced page; user can still collapse it
     if (advancedActive) setAdvancedOpen(true);
   }, [advancedActive]);