ember-eslint-parser 0.11.0 → 0.11.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ember-eslint-parser",
-  "version": "0.11.0",
+  "version": "0.11.2",
   "description": "",
   "keywords": [],
   "repository": {
@@ -31,6 +31,8 @@
   },
   "devDependencies": {
     "@babel/core": "^7.23.6",
+    "@babel/eslint-parser": "^7.28.6",
+    "@babel/plugin-proposal-decorators": "^7.25.9",
     "@typescript-eslint/parser": "^7.1.0",
     "@typescript-eslint/scope-manager": "^7.1.0",
     "@typescript-eslint/visitor-keys": "^7.1.0",
@@ -53,9 +55,13 @@
     "vitest": "^1.2.2"
   },
   "peerDependencies": {
+    "@babel/eslint-parser": "^7.28.6",
     "@typescript-eslint/parser": "*"
   },
   "peerDependenciesMeta": {
+    "@babel/eslint-parser": {
+      "optional": true
+    },
     "@typescript-eslint/parser": {
       "optional": true
     }

package/src/parser/gjs-gts-parser.js

@@ -4,10 +4,26 @@ import { registerParsedFile } from '../preprocessor/noop.js';
 import { patchTs, replaceExtensions, syncMtsGtsSourceFiles, typescriptParser } from './ts-patch.js';
 import { buildGlimmerVisitors } from './transforms.js';
 import { toTree } from 'ember-estree';
-import * as eslintScope from 'eslint-scope';
 
 const require = createRequire(import.meta.url);
 
+// Resolved once at module load via createRequire — top-level `await import()`
+// would mark the module as async, and the bench harness loads parsers
+// synchronously through `createRequire`. `@babel/eslint-parser` is an optional
+// peer (TS-only setups don't need it), so a missing dep yields null and the JS
+// path below emits a targeted error.
+//
+// The experimental-worker entry point runs babel in a worker and matches the
+// parser ember-cli wires up for plain `.js` files in JS-only apps, so config
+// discovery (decorators & friends) lines up with how the rest of the project
+// is being parsed.
+let babelParser = null;
+try {
+  babelParser = require('@babel/eslint-parser/experimental-worker');
+} catch {
+  // optional peer; left null
+}
+
 /**
  * implements https://eslint.org/docs/latest/extend/custom-parsers
  *
@@ -185,18 +201,25 @@ export function parseForESLint(code, options) {
             return tsResult;
           }
         : (placeholderJS) => {
-            // JS path: parse with oxc, create scope manager from placeholder AST
-            const { parseSync } = require('oxc-parser');
-            const oxcResult = parseSync(filePath || 'input.js', placeholderJS);
-            const program = oxcResult.program;
-            program.tokens = oxcResult.tokens || [];
-            program.comments = oxcResult.comments || [];
-            scopeManager = eslintScope.analyze(program, {
-              ecmaVersion: 2022,
-              sourceType: 'module',
-              range: true,
+            // JS path: defer to @babel/eslint-parser, which discovers the
+            // consuming app's babel config and applies its plugins (notably
+            // decorators) when parsing. That keeps gjs aligned with how the
+            // app's plain `.js` files are parsed, so syntax accepted there
+            // is accepted here without a second parser config.
+            if (!babelParser) {
+              throw new Error(
+                'ember-eslint-parser: linting `.gjs`/`.gts` files in a JS-only ' +
+                  'project requires `@babel/eslint-parser`. Install it as a ' +
+                  'dev dependency, or install `@typescript-eslint/parser` to ' +
+                  'opt into the TypeScript-based path.'
+              );
+            }
+            const babelResult = babelParser.parseForESLint(placeholderJS, {
+              ...options,
+              filePath,
             });
-            return { ast: program, scopeManager };
+            scopeManager = babelResult.scopeManager;
+            return babelResult;
           },
       // Factory form: scopeManager is set by the parser callback before this
       // runs. Returns null when unavailable so ember-estree skips the walk.