ember-eslint-parser 0.2.0 → 0.2.2

package/README.md

@@ -1 +1,33 @@
 # ember-eslint-parser
+
+This is the eslint parser for ember's gjs and gts files (using `<template>`).
+
+It is meant to be used with [eslint-plugin-ember](https://github.com/ember-cli/eslint-plugin-ember), which provides nice defaults for all the different file types in ember projects.
+
+It's recommended to only use _overrides_ when defining your eslint config, so using this parser would look like this:
+```js 
+    {
+      files: ['**/*.gjs'],
+      parser: 'ember-eslint-parser',
+      plugins: ['ember'],
+      extends: [
+        'eslint:recommended',
+        'plugin:ember/recommended',
+        'plugin:ember/recommended-gjs',
+      ],
+    },
+    {
+      files: ['**/*.gts'],
+      parser: 'ember-eslint-parser',
+      plugins: ['ember'],
+      extends: [
+        'eslint:recommended',
+        'plugin:ember/recommended',
+        'plugin:ember/recommended-gts',
+      ],
+    },
+```
+
+## Support
+
+eslint-plugin-ember is the primary consumer of this parser library, so SemVer _may_ not be respected for other consumers.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ember-eslint-parser",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "",
   "keywords": [],
   "repository": {
@@ -28,6 +28,8 @@
     "eslint-plugin-n": "^16.4.0",
     "eslint-plugin-prettier": "^5.0.1",
     "eslint-plugin-promise": "^6.0.0",
+    "execa": "^8.0.1",
+    "fs-extra": "^11.2.0",
     "jest": "^29.7.0",
     "prettier": "^3.1.1",
     "publint": "^0.2.6",
@@ -51,7 +53,6 @@
     "typescript": "^5.3.3"
   },
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1",
     "lint": "concurrently \"npm:lint:*(!fix)\" --names \"lint:\"",
     "lint:js": "eslint .",
     "lint:package": "pnpm publint",

package/src/parser/transforms.js

@@ -48,6 +48,7 @@ function findVarInParentScopes(scopeManager, nodePath, name) {
   if (!scope) {
     return { scope: currentScope };
   }
+
   return { scope: currentScope, variable: scope.set.get(name) };
 }
 
@@ -266,7 +267,11 @@ module.exports.preprocessGlimmerTemplates = function preprocessGlimmerTemplates(
         let start = n.range[0];
         let codeSlice = code.slice(...n.range);
         for (const part of n.tag.split('.')) {
-          const regex = new RegExp(`\\b${part}\\b`);
+          let pattern = `\\b${part}\\b`;
+          if (part.startsWith(':')) {
+            pattern = `${part}\\b`;
+          }
+          const regex = new RegExp(pattern);
           const match = codeSlice.match(regex);
           const range = [start + match.index, 0];
           range[1] = range[0] + part.length;
@@ -423,7 +428,16 @@ module.exports.convertAst = function convertAst(result, preprocessedResult, visi
       // the whole tag + children
       const n = node.parts[0];
       const { scope, variable } = findVarInParentScopes(result.scopeManager, path, n.name) || {};
+      /*
+      register a node in scope if we found a variable
+      we ignore named-blocks as we know that it doesn't reference anything in current scope
+      if we do not find a variable we register it with a missing variable if
+        * it starts with upper case, it should be a component with a reference
+        * it includes a dot, it's a path which should have a reference
+        * it's NOT a standard html tag, it should have a referenced variable
+      */
       if (
+        !n.name.startsWith(':') &&
         scope &&
         (variable ||
           isUpperCase(n.name[0]) ||