Enable no-require-imports that was waiting for us to switch to flat config file

Author: canova

eslint.config.mjs

@@ -165,8 +165,6 @@ export default [
           'ts-check': false, // allow even without description
         },
       ],
-      // TODO: Re-enable for src when we update to eslint and switch to the new
-      // config format
       '@typescript-eslint/no-require-imports': 'off',
     },
     linterOptions: {
@@ -179,6 +177,14 @@ export default [
     },
   },
 
+  // Source files - enable stricter TypeScript rules
+  {
+    files: ['src/**/*.ts', 'src/**/*.tsx'],
+    rules: {
+      '@typescript-eslint/no-require-imports': 'error',
+    },
+  },
+
   // Test files override
   {
     files: ['src/test/**/*'],

…/utils/__mocks__/node-worker-contents.js …utils/__mocks__/node-worker-contents.mjssrc/utils/__mocks__/node-worker-contents.js renamed to src/utils/__mocks__/node-worker-contents.mjs src/utils/__mocks__/node-worker-contents.js renamed to src/utils/__mocks__/node-worker-contents.mjs

@@ -2,9 +2,9 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 //
-const { parentPort, workerData } = require('worker_threads');
-const fs = require('fs');
-const vm = require('vm');
+import { parentPort, workerData } from 'worker_threads';
+import fs from 'fs';
+import vm from 'vm';
 
 if (typeof workerData !== 'string') {
   throw new Error(`Please pass a file name using the 'workerData' property.`);

src/utils/__mocks__/worker-factory.ts

@@ -9,7 +9,7 @@ class NodeWorker {
   onmessage: ((event: MessageEvent) => unknown) | null;
 
   constructor(file: string) {
-    const worker = new Worker(__dirname + '/node-worker-contents.js', {
+    const worker = new Worker(__dirname + '/node-worker-contents.mjs', {
       workerData: file,
     });
     worker.on('message', this.onMessage);