[eslint] Add import/first rule.

I was sure we already had this rule, but somehow we didn't. We should of
course strive to have `import` statements first in all files.

Bug: none
Change-Id: Ic583710adc6cb3b7e5c0ae4bd82cd1f31fc9442b
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6219315
Reviewed-by: Nikolay Vitkov <[email protected]>
Auto-Submit: Benedikt Meurer <[email protected]>
Commit-Queue: Benedikt Meurer <[email protected]>

Author: bmeurer

eslint.config.mjs

@@ -249,6 +249,8 @@ export default [
       'no-implicit-globals': 'off',
       'no-unused-private-class-members': 'error',
 
+      // Sort imports first
+      'import/first': 'error',
       // Closure does not properly typecheck default exports
       'import/no-default-export': 'error',
       /**

test/e2e/panels/panels_test.ts

@@ -8,6 +8,7 @@ import {
   getBrowserAndPages,
   waitFor,
 } from '../../shared/helper.js';
+import {runCommandWithQuickOpen} from '../helpers/quick_open-helpers.js';
 
 // Ideally we want to have all panels within this list,
 // but introducing shadow doms for all panels currently leads
@@ -16,8 +17,6 @@ const PANEL_NAMES = [
   'Lighthouse',
 ];
 
-import {runCommandWithQuickOpen} from '../helpers/quick_open-helpers.js';
-
 describe('DevTools panels', function() {
   PANEL_NAMES.forEach(panel => {
     it(`${panel} doesn't increase number of adopted style sheets on the document`, async () => {