Fix PIPE location

Author: fisker

src/transform-node.ts

@@ -178,12 +178,11 @@ class Transformer extends Source {
     }
 
     if (node instanceof angular.BindingPipe) {
-      const { exp: expressionNode, name, args: originalArguments } = node;
+      const { name } = node;
       const left = this.transform<babel.Expression>(
-        expressionNode,
+        node.exp,
         childTransformOptions,
       );
-      const start = getOuterStart(left);
       const leftEnd = getOuterEnd(left);
       const rightStart = this.getCharacterIndex(
         /\S/,
@@ -198,20 +197,16 @@ class Transformer extends Source {
         },
         ancestors,
       );
-      const argumentNodes = originalArguments.map<babel.Expression>((node) =>
+      const arguments_ = node.args.map<babel.Expression>((node) =>
         this.transform(node, childTransformOptions),
       );
       return this.#create<NGPipeExpression>(
         {
           type: 'NGPipeExpression',
           left,
           right,
-          arguments: argumentNodes,
-          start,
-          end: getOuterEnd(
-            // TODO[@fisker]: End seems not correct, since there should be `()`
-            argumentNodes.length === 0 ? right : argumentNodes.at(-1)!,
-          ),
+          arguments: arguments_,
+          ...node.sourceSpan,
         },
         ancestors,
       );

src/utils.ts

@@ -1,23 +1,5 @@
 import type { LocationInformation, RawNGSpan } from './types.ts';
 
-function expandSurroundingSpaces(
-  { start: startIndex, end: endIndex }: RawNGSpan,
-  text: string,
-) {
-  let start = startIndex;
-  let end = endIndex;
-
-  while (end !== text.length && /\s/.test(text[end])) {
-    end++;
-  }
-
-  while (start !== 0 && /\s/.test(text[start - 1])) {
-    start--;
-  }
-
-  return { start, end };
-}
-
 function getCharacterSearchTestFunction(pattern: RegExp | string) {
   if (typeof pattern === 'string') {
     return (character: string) => character === pattern;

tests/__snapshots__/transform.test.ts.snap

@@ -13,7 +13,7 @@ NGPipeExpression {
   "comments": []
 }
 > 1 |  ( ( ( ( a ) ) |  b  )) 
-    |      ^^^^^^^^^^^^^^^^^^
+    |      ^^^^^^^^^^^^^^
 --------------------------------------------------------------------------------
 Identifier {
   "name": "a",
@@ -82,7 +82,7 @@ NGPipeExpression {
   "comments": []
 }
 > 1 |  ( ( ( ( a ) ) |  b  )) 
-    |      ^^^^^^^^^^^^^^^^^^
+    |      ^^^^^^^^^^^^^^
 --------------------------------------------------------------------------------
 Identifier {
   "name": "a",