👷 fix MODULE_TYPELESS_PACKAGE_JSON warning (#3906)

Co-authored-by: Benoît Zugmeyer <[email protected]>

Author: thomas-lebeau

developer-extension/tsconfig.webpack.json

@@ -3,12 +3,14 @@
 
   "compilerOptions": {
     "baseUrl": ".",
-    "module": "ES6",
+    "module": "preserve",
     "target": "esnext",
     "jsx": "react",
     "lib": ["ES2022", "DOM"],
     "types": ["chrome", "react", "react-dom", "jasmine"],
-    "skipLibCheck": true
+    "skipLibCheck": true,
+    "allowImportingTsExtensions": true,
+    "noEmit": true
   },
 
   "include": ["src"]

developer-extension/webpack.config.ts renamed to developer-extension/webpack.config.mts

@@ -1,4 +1,4 @@
-const { SymbolFlags } = require('typescript')
+import { SymbolFlags } from 'typescript'
 
 /**
  * This rule forbids exporting 'enums'.  It serves two purposes:
@@ -34,7 +34,7 @@ const { SymbolFlags } = require('typescript')
  * [1]: https://www.typescriptlang.org/tsconfig#isolatedModules
  * [2]: https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-8.html#type-only-imports-and-export
  */
-module.exports = {
+export default {
   meta: {
     docs: {
       description: 'Disallow export of enums.',

eslint-local-rules/disallowEnumExports.js

@@ -1,6 +1,6 @@
-const path = require('path')
+import path from 'path'
 
-module.exports = {
+export default {
   meta: {
     docs: {
       description: 'Disallow the use of too generic utility file names',

eslint-local-rules/disallowGenericUtils.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   meta: {
     docs: {
       description: 'Disallow JS files that are not used as a "script"',

eslint-local-rules/disallowNonScripts.js

@@ -1,10 +1,15 @@
-const path = require('path')
-const { minimatch } = require('minimatch')
-const resolve = require('eslint-module-utils/resolve').default
-const moduleVisitor = require('eslint-module-utils/moduleVisitor').default
-const importType = require('eslint-plugin-import/lib/core/importType').default
+import path from 'node:path'
+import { minimatch } from 'minimatch'
 
-module.exports = {
+import resolvePackage from 'eslint-module-utils/resolve.js'
+import moduleVisitorPackage from 'eslint-module-utils/moduleVisitor.js'
+import importTypePackage from 'eslint-plugin-import/lib/core/importType.js'
+
+const moduleVisitor = moduleVisitorPackage.default
+const importType = importTypePackage.default
+const resolve = resolvePackage.default
+
+export default {
   meta: {
     docs: {
       description:

eslint-local-rules/disallowProtectedDirectoryImport.js

@@ -1,6 +1,6 @@
-const path = require('path')
+import path from 'path'
 
-module.exports = {
+export default {
   meta: {
     docs: {
       description:
@@ -22,7 +22,7 @@ module.exports = {
   },
 }
 
-const packagesRoot = path.resolve(__dirname, '..', 'packages')
+const packagesRoot = path.resolve(import.meta.dirname, '..', 'packages')
 
 // Those modules are known to have side effects when evaluated
 const pathsWithSideEffect = new Set([

eslint-local-rules/disallowSideEffects.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   meta: {
     docs: {
       description: 'Disallow importing spec file code to avoid to execute the tests from the imported spec file twice',

eslint-local-rules/disallowSpecImport.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   meta: {
     docs: {
       description:

eslint-local-rules/disallowTestImportExportFromSrc.js

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   meta: {
     docs: {
       description: 'Disallow problematic URL constructor patched values.',