From cce352adcaddaa9893e7ee755554289c2e6a09a5 Mon Sep 17 00:00:00 2001 From: QianEric <307361214+qianeric-backup@users.noreply.github.com> Date: Thu, 29 Oct 2026 22:53:09 +0800 Subject: [PATCH] fix(lint): recognize the Admin UI Ext global The Admin UI JavaScript is written against ExtJS, which the admin UI pages load via ext-base.js / ext-all.js before any of the application scripts run. The ESLint configuration for `**/*.js` never declared `Ext`, so every such reference was reported as `no-undef` and eslint could not be run cleanly on the admin UI sources. Declaring `Ext` as a read-only global alongside the other bundled libraries removes that noise. Repository-wide `eslint .` drops from 5653 errors to 1360; the remaining findings are unrelated (beefrtcs, $jEncoder, globalrtc, no-unused-vars, and so on). Refs #3666 --- eslint.config.js | 1 + 1 file changed, 1 insertion(+) diff --git a/eslint.config.js b/eslint.config.js index 44cee5d4ff..145c8fee17 100644 --- a/eslint.config.js +++ b/eslint.config.js @@ -33,6 +33,7 @@ module.exports = [ $j: "readonly", // Libraries + Ext: "readonly", MobileEsp: "readonly", evercookie: "readonly", swfobject: "readonly",