fix(eslint-plugin-fiori-tools): support ESLint 10 by upgrading @babel/eslint-parser to v8 (#4751)

* fix(eslint-plugin-fiori-tools): support ESLint 10

* fix comment

Author: devinea

.changeset/eslint-plugin-fiori-tools-eslint-10-support.md

@@ -0,0 +1,5 @@
+---
+'@sap-ux/eslint-plugin-fiori-tools': minor
+---
+
+fix: support ESLint 10 by upgrading `@babel/eslint-parser` and `@babel/core` to `8.0.0-rc.6` and adding `@babel/parser@8.0.0-rc.6` as a runtime dependency. This avoids `@babel/eslint-parser`'s `createRequire` failing to load `@babel/parser@8` (pure ESM) under pnpm's strict `node_modules` isolation, where `@babel/parser` is otherwise not visible from the consumer's resolution paths.

packages/eslint-plugin-fiori-tools/eslint.config.mjs

@@ -6,7 +6,12 @@ const __dirname = import.meta.dirname;
 
 export default [
     {
-        ignores: ['config/**/eslintrc*.js', 'test/global-setup.js', 'jest.resolver.cjs']
+        ignores: [
+            'config/**/eslintrc*.js',
+            'test/global-setup.js',
+            'jest.resolver.cjs',
+            'test/babel-eslint-parser.transformer.cjs'
+        ]
     },
     ...base,
     {

packages/eslint-plugin-fiori-tools/jest.config.mjs

@@ -12,6 +12,19 @@ export default {
         // which deadlocks under Jest's --experimental-vm-modules.
         '^synckit$': '<rootDir>/test/__mocks__/synckit.cjs'
     },
+    transform: {
+        // Patch @babel/eslint-parser's index.js (see transformer header for rationale).
+        // This rule must come before ts-jest's default `^.+\\.[jt]s$` so it wins for the
+        // matched path; Jest applies the first matching transform.
+        '@babel[\\\\/]eslint-parser[\\\\/]lib[\\\\/]index\\.js$':
+            '<rootDir>/test/babel-eslint-parser.transformer.cjs',
+        ...baseConfig.transform
+    },
+    transformIgnorePatterns: [
+        // Override jest.base's pattern to additionally allow @babel/eslint-parser through
+        // the transform pipeline so our patcher above runs on it.
+        'node_modules/(?!(?:.*?/)?(@sap-ux|@sap-ux-private|@sap/ux-cds-compiler-facade|@sap-devx[+/]yeoman-ui-types|@babel[+/]eslint-parser)/)'
+    ],
     coveragePathIgnorePatterns: [
         'src/types.ts',
         'src/language/annotations/types.ts',

packages/eslint-plugin-fiori-tools/package.json

@@ -21,7 +21,8 @@
         "format": "prettier --write '**/*.{js,json,ts,yaml,yml}' --ignore-path ../../.prettierignore",
         "lint": "eslint",
         "lint:fix": "eslint --fix",
-        "test": "cross-env \"NODE_OPTIONS=--experimental-vm-modules\" jest --ci --forceExit --detectOpenHandles --colors --maxWorkers=50%"
+        "test": "cross-env \"NODE_OPTIONS=--experimental-vm-modules\" jest --ci --forceExit --detectOpenHandles --colors --maxWorkers=50%",
+        "tgz:package": "pnpm pack"
     },
     "devDependencies": {
         "@jest/globals": "30.3.0",
@@ -32,8 +33,9 @@
         "@types/semver": "7.7.1"
     },
     "dependencies": {
-        "@babel/core": "7.29.0",
-        "@babel/eslint-parser": "^7.28.5",
+        "@babel/core": "8.0.0-rc.6",
+        "@babel/eslint-parser": "8.0.0-rc.6",
+        "@babel/parser": "8.0.0-rc.6",
         "@eslint/js": "10.0.1",
         "@eslint/json": "0.14.0",
         "@eslint/core": "1.1.1",

packages/eslint-plugin-fiori-tools/test/babel-eslint-parser.transformer.cjs

@@ -0,0 +1,28 @@
+'use strict';
+
+// Jest transformer that string-patches @babel/eslint-parser@8.0.0-rc.6/lib/index.js so
+// it imports @babel/parser statically instead of via createRequire(). The shipped parser
+// uses createRequire(import.meta.url).require('@babel/parser') which throws under Jest's
+// --experimental-vm-modules because the VM-modules runtime cannot satisfy a sync CJS-style
+// require for a pure-ESM module like @babel/parser@8. Production runtime is unaffected:
+// Node 22.12+ supports require(esm) natively, so the unpatched parser loads fine there.
+module.exports = {
+    process(sourceText) {
+        const importReplaced = sourceText.replace(
+            "import { createRequire } from 'node:module';",
+            "import * as babelParser from '@babel/parser';"
+        );
+        const requireRemoved = importReplaced.replace(
+            /const require\$1 = createRequire\(import\.meta\.url\);\s*const babelParser = require\$1\(require\$1\.resolve\("@babel\/parser",\s*\{\s*paths:\s*\[require\$1\.resolve\("@babel\/core\/package\.json"\)\]\s*\}\)\);/,
+            ''
+        );
+        if (importReplaced === sourceText || requireRemoved === importReplaced) {
+            throw new Error(
+                '[babel-eslint-parser.transformer] Failed to patch @babel/eslint-parser/lib/index.js: ' +
+                    'expected createRequire-based @babel/parser load not found. ' +
+                    'The upstream source has likely changed — update the transformer.'
+            );
+        }
+        return { code: requireRemoved };
+    }
+};