Replace imports check presubmit with a working eslint rule.

Author: goodov

build/commands/PRESUBMIT.py

@@ -0,0 +1,27 @@
+// Copyright (c) 2026 The Brave Authors. All rights reserved.
+// This Source Code Form is subject to the terms of the Mozilla Public
+// License, v. 2.0. If a copy of the MPL was not distributed with this file,
+// You can obtain one at https://mozilla.org/MPL/2.0/.
+
+import { execSync } from 'child_process'
+
+// Run the TypeScript compiler.
+run('tsc --project build/commands')
+
+// Run the ESLint linter. This is required to catch possible runtime errors
+// that might occur because of broken imports.
+run('eslint --quiet build/commands')
+
+// Run the Jest test runner. Pass through any extra arguments to it.
+const extraArgs = process.argv.slice(2).join(' ')
+run(`jest build/commands${extraArgs ? ` ${extraArgs}` : ''}`)
+
+// Run a command and exit cleanly without exception noise.
+function run(cmd: string): void {
+  console.log(`> ${cmd}`)
+  try {
+    execSync(cmd, { stdio: 'inherit' })
+  } catch (e: any) {
+    process.exit(e.status ?? 1)
+  }
+}

build/commands/PRESUBMIT_test.py

@@ -6,6 +6,7 @@
 import { defineConfig, globalIgnores } from 'eslint/config'
 
 import jest from 'eslint-plugin-jest'
+import importPlugin from 'eslint-plugin-import'
 import licenses from 'eslint-plugin-licenses'
 import noUnsanitized from 'eslint-plugin-no-unsanitized'
 import reactHooks from 'eslint-plugin-react-hooks'
@@ -287,6 +288,19 @@ export default defineConfig([
       '@typescript-eslint/no-require-imports': 'off',
     },
   },
+  {
+    // Enforce import resolution for build/commands scripts. When running
+    // TypeScript natively via Node.js (no transpilation), imports must use the
+    // real file extension. For example, if a file is named config.ts, importing
+    // it as './config.js' is an error.
+    files: ['build/commands/**/*.{js,cjs,mjs,ts,cts,mts}'],
+    plugins: {
+      import: importPlugin,
+    },
+    rules: {
+      'import/no-unresolved': 'error',
+    },
+  },
   {
     // Test data files; skips some non-important rules.
     files: ['test/data/**/*.js'],

build/commands/scripts/testScripts.ts

@@ -31,7 +31,7 @@
     "docs": "node ./build/commands/scripts/commands.js docs",
     "test": "node ./build/commands/scripts/commands.js test",
     "list_affected_tests": "node ./build/commands/scripts/commands.js list_affected_tests",
-    "test:scripts": "tsc --project build/commands && jest build/commands",
+    "test:scripts": "node ./build/commands/scripts/testScripts.ts",
     "tslint": "npm run eslint",
     "eslint": "eslint --quiet .",
     "pep8": "pycodestyle --max-line-length 120 -r script",