Add ESLint config enforcing no-undef and eqeqeq

Without a config file, ESLint was running with no-undef disabled, meaning undefined variable references in static/app.js were silently ignored. Add .eslintrc.json with no-undef: error and eqeqeq: error so CI actually catches JS bugs. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Fix inspector auto-refresh ignoring 'Off' setting on page load
2026-05-13 15:33:26 -04:00 · 2026-05-13 13:20:42 -04:00 · 2026-05-13 13:19:44 -04:00
4 changed files with 24 additions and 3 deletions
@@ -0,0 +1,21 @@
+{
+  "env": {
+    "browser": true,
+    "es2021": true
+  },
+  "globals": {
+    "lt": "readonly",
+    "GANDALF_CONFIG": "readonly",
+    "CSS": "readonly"
+  },
+  "rules": {
+    "no-undef": "error",
+    "no-unused-vars": ["warn", { "argsIgnorePattern": "^_", "varsIgnorePattern": "^_" }],
+    "no-console": "off",
+    "eqeqeq": ["error", "always", { "null": "ignore" }]
+  },
+  "parserOptions": {
+    "ecmaVersion": 2021,
+    "sourceType": "script"
+  }
+}
@@ -469,7 +469,7 @@
 {% block scripts %}
 <script>
  // Start auto-refresh using saved settings interval (default 30 s)
-  const _savedInterval = (window.gandalfSettings && window.gandalfSettings.refreshInterval) || 30;
+  const _savedInterval = window.gandalfSettings?.refreshInterval ?? 30;
  if (_savedInterval > 0) lt.autoRefresh.start(refreshAll, _savedInterval * 1000);

  // When settings change, restart auto-refresh with new interval
@@ -473,7 +473,7 @@ async function loadInspector() {
 }

 loadInspector();
-const _inspInterval = (window.gandalfSettings && window.gandalfSettings.refreshInterval) || 60;
+const _inspInterval = window.gandalfSettings?.refreshInterval ?? 60;
 if (_inspInterval > 0) lt.autoRefresh.start(loadInspector, Math.max(_inspInterval, 15) * 1000);

 window.onGandalfSettingsChanged = function(s) {
@@ -571,7 +571,7 @@ async function loadLinks() {
 }

 loadLinks();
-const _linksInterval = (window.gandalfSettings && window.gandalfSettings.refreshInterval) || 60;
+const _linksInterval = window.gandalfSettings?.refreshInterval ?? 60;
 if (_linksInterval > 0) lt.autoRefresh.start(loadLinks, Math.max(_linksInterval, 15) * 1000);

 window.onGandalfSettingsChanged = function(s) {
Author	SHA1	Message	Date
jared	faa0707f79	Add ESLint config enforcing no-undef and eqeqeq Lint / Python (flake8) (push) Successful in 53s Details Lint / JS (eslint) (push) Successful in 12s Details Security / Python Security (bandit) (push) Successful in 1m44s Details Test / Python Tests (pytest) (push) Successful in 59s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 3s Details Without a config file, ESLint was running with no-undef disabled, meaning undefined variable references in static/app.js were silently ignored. Add .eslintrc.json with no-undef: error and eqeqeq: error so CI actually catches JS bugs. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-13 15:33:26 -04:00
jared	9c52e4ad1a	Fix inspector auto-refresh ignoring 'Off' setting on page load Lint / Python (flake8) (push) Successful in 41s Details Lint / JS (eslint) (push) Successful in 8s Details Security / Python Security (bandit) (push) Successful in 1m0s Details Test / Python Tests (pytest) (push) Successful in 50s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 2s Details Same ?? / \|\| issue as the previous fix in index.html and links.html. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-13 13:20:42 -04:00
jared	156ef97667	Fix auto-refresh ignoring 'Off' setting on page load Lint / Python (flake8) (push) Successful in 40s Details Lint / JS (eslint) (push) Successful in 7s Details Security / Python Security (bandit) (push) Successful in 39s Details Test / Python Tests (pytest) (push) Successful in 53s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 2s Details Using \|\| 30 / \|\| 60 as a fallback treats refreshInterval=0 (Off) as falsy and replaces it with the default, causing auto-refresh to start even when the user saved 'Off'. Replace with nullish coalescing (??) so only null/undefined triggers the default. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-13 13:19:44 -04:00