Merge pull request #19786 from nicolo-ribaudo/linting-export

Lint against `export [some declaration]`

Author: Snuffleupagus

eslint.config.mjs

@@ -306,6 +306,16 @@ export default [
           selector: "NewExpression[callee.name='Ref']",
           message: "Use `Ref.get()` rather than `new Ref()`.",
         },
+        {
+          selector: "ExportNamedDeclaration[declaration]",
+          message:
+            "Separate the declaration and the export statement, using `export { ... }`.",
+        },
+        {
+          selector: "ExportDefaultDeclaration:has(> :declaration)",
+          message:
+            "Separate the declaration and the export statement, using `export default <variable name>`.",
+        },
       ],
       "no-unneeded-ternary": "error",
       "operator-assignment": "error",

src/pdf.sandbox.external.js

@@ -16,6 +16,7 @@
 // In mozilla-central, this file is loaded as non-module script,
 // so it mustn't have any dependencies.
 
+// eslint-disable-next-line no-restricted-syntax
 export class SandboxSupportBase {
   /**
    * @param {DOMWindow} - win