fix: fix location for template literals

Author: fisker

package.json

@@ -27,10 +27,10 @@
     "release": "yarn build && standard-version"
   },
   "devDependencies": {
-    "@angular/compiler": "19.2.1",
+    "@angular/compiler": "19.2.2",
     "@babel/code-frame": "7.26.2",
-    "@babel/parser": "7.26.9",
-    "@babel/types": "7.26.9",
+    "@babel/parser": "7.26.10",
+    "@babel/types": "7.26.10",
     "@types/babel__code-frame": "7.0.6",
     "@types/node": "22.13.10",
     "@vitest/coverage-v8": "3.0.8",
@@ -50,7 +50,7 @@
     "vitest": "3.0.8"
   },
   "peerDependencies": {
-    "@angular/compiler": "^19.0.0"
+    "@angular/compiler": ">=19.2.2"
   },
   "engines": {
     "node": ">= 20"

src/transform-node.ts

@@ -555,24 +555,13 @@ class Transformer extends Source {
     }
 
     if (node instanceof angular.TemplateLiteralElement) {
-      const templateLiteral = options!.parent! as angular.TemplateLiteral;
-      const elementIndex = templateLiteral.elements.indexOf(node);
+      const { elements } = options!.parent! as angular.TemplateLiteral;
+      const elementIndex = elements.indexOf(node);
       const isFirst = elementIndex === 0;
-      const isLast = elementIndex === templateLiteral.elements.length - 1;
-
-      // The `TemplateLiteralElement` don't have correct location information
-      const start = isFirst
-        ? templateLiteral.sourceSpan.start + 1
-        : node.sourceSpan.start;
-      let end;
-      if (isLast) {
-        end = templateLiteral.sourceSpan.end - 1;
-      } else {
-        const nextExpression = templateLiteral.expressions[elementIndex];
-        // TODO: Support search multiple characters in `getCharacterLastIndex()`
-        // FIXME: Search `${` instead
-        end = this.getCharacterLastIndex('$', nextExpression.sourceSpan.start);
-      }
+      const isLast = elementIndex === elements.length - 1;
+
+      const end = node.sourceSpan.end - (isLast ? 1 : 0);
+      const start = node.sourceSpan.start + (isFirst ? 1 : 0);
       const raw = this.text.slice(start, end);
 
       return this.#create<babel.TemplateElement>(

tests/transform.test.ts

@@ -96,6 +96,9 @@ describe.each`
   ${'PrefixNot'}        | ${'UnaryExpression'}          | ${' ! ( typeof {} === "number" ) '} | ${true}     | ${true}      | ${true}            | ${true}
   ${'TemplateLiteral'}  | ${'TemplateLiteral'}          | ${' ` a ${ b } \u0063 ` '}          | ${true}     | ${true}      | ${true}            | ${true}
   ${'TemplateLiteral'}  | ${'TemplateLiteral'}          | ${' ( ` a ${ b } \u0063 ` ) '}      | ${true}     | ${true}      | ${true}            | ${true}
+  ${'TemplateLiteral'}  | ${'TemplateLiteral'}          | ${' `  \u0063  ` '}                 | ${true}     | ${true}      | ${true}            | ${true}
+  ${'TemplateLiteral'}  | ${'TemplateLiteral'}          | ${' `   ` '}                        | ${true}     | ${true}      | ${true}            | ${true}
+  ${'TemplateLiteral'}  | ${'TemplateLiteral'}          | ${' `` '}                           | ${true}     | ${true}      | ${true}            | ${true}
 `('($expectedAngularType -> $expectedEstreeType)', (fields) => {
   for (const method of PARSE_METHODS) {
     testSection(method, fields);

yarn.lock

@@ -15,17 +15,17 @@ __metadata:
   languageName: node
   linkType: hard
 
-"@angular/compiler@npm:19.2.1":
-  version: 19.2.1
-  resolution: "@angular/compiler@npm:19.2.1"
+"@angular/compiler@npm:19.2.2":
+  version: 19.2.2
+  resolution: "@angular/compiler@npm:19.2.2"
   dependencies:
     tslib: "npm:^2.3.0"
   peerDependencies:
-    "@angular/core": 19.2.1
+    "@angular/core": 19.2.2
   peerDependenciesMeta:
     "@angular/core":
       optional: true
-  checksum: 10/24d094e6f6653e20bd554a1b38488b6e6f752ec65491dd77135fc8b7ac03020f64ed967e21fd9ea38a5e644e7e90e84cf8da24d31ef905944cbafa34c05a31bb
+  checksum: 10/2c427e4e4696f8ce357e432a517d12a0730f9cbdb1306b5be49f9d4f7bd54b76442cfaba8f82579152bc814402e5f666bb5092c994f43c3022bb9a93e8d64db6
   languageName: node
   linkType: hard
 
@@ -54,7 +54,18 @@ __metadata:
   languageName: node
   linkType: hard
 
-"@babel/parser@npm:7.26.9, @babel/parser@npm:^7.25.4":
+"@babel/parser@npm:7.26.10":
+  version: 7.26.10
+  resolution: "@babel/parser@npm:7.26.10"
+  dependencies:
+    "@babel/types": "npm:^7.26.10"
+  bin:
+    parser: ./bin/babel-parser.js
+  checksum: 10/3f87781f46795ba72448168061d9e99c394fdf9cd4aa3ddf053a06334247da4d25d0923ccc89195937d3360d384cee181e99711763c1e8fe81d4f17ee22541fc
+  languageName: node
+  linkType: hard
+
+"@babel/parser@npm:^7.25.4":
   version: 7.26.9
   resolution: "@babel/parser@npm:7.26.9"
   dependencies:
@@ -65,7 +76,17 @@ __metadata:
   languageName: node
   linkType: hard
 
-"@babel/types@npm:7.26.9, @babel/types@npm:^7.25.4, @babel/types@npm:^7.26.9":
+"@babel/types@npm:7.26.10, @babel/types@npm:^7.26.10":
+  version: 7.26.10
+  resolution: "@babel/types@npm:7.26.10"
+  dependencies:
+    "@babel/helper-string-parser": "npm:^7.25.9"
+    "@babel/helper-validator-identifier": "npm:^7.25.9"
+  checksum: 10/6b4f24ee77af853c2126eaabb65328cd44a7d6f439685131cf929c30567e01b6ea2e5d5653b2c304a09c63a5a6199968f0e27228a007acf35032036d79a9dee6
+  languageName: node
+  linkType: hard
+
+"@babel/types@npm:^7.25.4, @babel/types@npm:^7.26.9":
   version: 7.26.9
   resolution: "@babel/types@npm:7.26.9"
   dependencies:
@@ -989,10 +1010,10 @@ __metadata:
   version: 0.0.0-use.local
   resolution: "angular-estree-parser@workspace:."
   dependencies:
-    "@angular/compiler": "npm:19.2.1"
+    "@angular/compiler": "npm:19.2.2"
     "@babel/code-frame": "npm:7.26.2"
-    "@babel/parser": "npm:7.26.9"
-    "@babel/types": "npm:7.26.9"
+    "@babel/parser": "npm:7.26.10"
+    "@babel/types": "npm:7.26.10"
     "@types/babel__code-frame": "npm:7.0.6"
     "@types/node": "npm:22.13.10"
     "@vitest/coverage-v8": "npm:3.0.8"
@@ -1011,7 +1032,7 @@ __metadata:
     typescript-eslint: "npm:8.26.1"
     vitest: "npm:3.0.8"
   peerDependencies:
-    "@angular/compiler": ^19.0.0
+    "@angular/compiler": ^19.2.2
   languageName: unknown
   linkType: soft