Mangle optimizations

Author: tolyo

.husky/pre-commit

@@ -1,5 +1,5 @@
 make lint
 make check
-make pretty
+make format
 make types
 git add ./

Makefile

@@ -56,7 +56,7 @@ size-html:
 version:
 	@node utils/version.cjs	
 
-pretty:
+format:
 	@npx prettier ./ --write --cache --log-level=silent
 
 lint:
@@ -84,7 +84,7 @@ serve:
 	node --watch ./utils/express.js & \
 	wait
 
-prepare-release: build test check types doc pretty gzip version size-html
+prepare-release: build test check types doc format gzip version size-html
 
 PLAYWRIGHT_TEST := npx playwright test
 
@@ -110,4 +110,3 @@ coverage-open:
 
 hugo:
 	hugo server --source=docs --disableFastRender
-

src/core/parse/ast/ast.ts

@@ -203,7 +203,7 @@ export class AST {
     while ((token = this._expect("==", "!=", "===", "!=="))) {
       left = {
         _type: ASTType._BinaryExpression,
-        _operator: /** @type {Token} */ token.text,
+        _operator: /** @type {Token} */ token._text,
         _left: left,
         _right: this._relational(),
       };
@@ -224,7 +224,7 @@ export class AST {
     while ((token = this._expect("<", ">", "<=", ">="))) {
       left = {
         _type: ASTType._BinaryExpression,
-        _operator: /** @type {Token} */ token.text,
+        _operator: /** @type {Token} */ token._text,
         _left: left,
         _right: this._additive(),
       };
@@ -245,7 +245,7 @@ export class AST {
     while ((token = this._expect("+", "-"))) {
       left = {
         _type: ASTType._BinaryExpression,
-        _operator: /** @type {Token} */ token.text,
+        _operator: /** @type {Token} */ token._text,
         _left: left,
         _right: this._multiplicative(),
       };
@@ -266,7 +266,7 @@ export class AST {
     while ((token = this._expect("*", "/", "%"))) {
       left = {
         _type: ASTType._BinaryExpression,
-        _operator: token.text,
+        _operator: token._text,
         _left: left,
         _right: this._unary(),
       };
@@ -299,7 +299,7 @@ export class AST {
 
       return {
         _type: ASTType._UpdateExpression,
-        _operator: /** @type {Token} */ token.text,
+        _operator: /** @type {Token} */ token._text,
         _prefix: true,
         _argument: argument,
       };
@@ -309,7 +309,7 @@ export class AST {
     if ((token = this._expect("+", "-", "!"))) {
       return {
         _type: ASTType._UnaryExpression,
-        _operator: /** @type {Token} */ token.text,
+        _operator: /** @type {Token} */ token._text,
         _prefix: true,
         _argument: this._unary(),
       };
@@ -339,7 +339,7 @@ export class AST {
 
       expr = {
         _type: ASTType._UpdateExpression,
-        _operator: /** @type {Token} */ token.text,
+        _operator: /** @type {Token} */ token._text,
         _prefix: false,
         _argument: expr,
       };
@@ -364,18 +364,18 @@ export class AST {
       primary = this._arrayDeclaration();
     } else if (this._expect("{")) {
       primary = this._object();
-    } else if (hasOwn(this._selfReferential, (peekToken as Token).text)) {
+    } else if (hasOwn(this._selfReferential, (peekToken as Token)._text)) {
       primary = cloneSelfReferentialNode(
-        this._selfReferential[this._consume().text],
+        this._selfReferential[this._consume()._text],
       );
-    } else if (hasOwn(literals, (peekToken as Token).text)) {
+    } else if (hasOwn(literals, (peekToken as Token)._text)) {
       primary = {
         _type: ASTType._Literal,
-        _value: literals[this._consume().text as keyof typeof literals],
+        _value: literals[this._consume()._text as keyof typeof literals],
       };
-    } else if ((peekToken as Token).identifier) {
+    } else if ((peekToken as Token)._identifier) {
       primary = this._identifier();
-    } else if ((peekToken as Token).constant) {
+    } else if ((peekToken as Token)._constant) {
       primary = this._constant();
     } else {
       this._throwError("not a primary expression", this._peek() as Token);
@@ -384,22 +384,22 @@ export class AST {
     let next: Token | false;
 
     while ((next = this._expect("(", "[", "."))) {
-      if (next.text === "(") {
+      if (next._text === "(") {
         primary = {
           _type: ASTType._CallExpression,
           _callee: primary,
           _arguments: this._parseArguments(),
         };
         this._consume(")");
-      } else if (next.text === "[") {
+      } else if (next._text === "[") {
         primary = {
           _type: ASTType._MemberExpression,
           _object: primary,
           _property: this._assignment(),
           _computed: true,
         };
         this._consume("]");
-      } else if (next.text === ".") {
+      } else if (next._text === ".") {
         primary = {
           _type: ASTType._MemberExpression,
           _object: primary,
@@ -443,7 +443,7 @@ export class AST {
   _parseArguments(): ASTNode[] {
     const args: ASTNode[] = [];
 
-    if (this._peekToken().text !== ")") {
+    if (this._peekToken()._text !== ")") {
       do {
         args.push(this._filterChain());
       } while (this._expect(","));
@@ -459,11 +459,11 @@ export class AST {
   _identifier(): ASTNode {
     const token = this._consume();
 
-    if (!token.identifier) {
+    if (!token._identifier) {
       this._throwError("is not a valid identifier", token);
     }
 
-    return { _type: ASTType._Identifier, _name: token.text };
+    return { _type: ASTType._Identifier, _name: token._text };
   }
 
   /**
@@ -472,7 +472,7 @@ export class AST {
    */
   _constant(): ASTNode {
     // TODO check that it is a constant
-    return { _type: ASTType._Literal, _value: this._consume().value };
+    return { _type: ASTType._Literal, _value: this._consume()._value };
   }
 
   /**
@@ -482,7 +482,7 @@ export class AST {
   _arrayDeclaration(): ASTNode {
     const elements: ASTNode[] = [];
 
-    if (this._peekToken().text !== "]") {
+    if (this._peekToken()._text !== "]") {
       do {
         if (this._peek("]")) {
           // Support trailing commas per ES5.1.
@@ -505,7 +505,7 @@ export class AST {
 
     let property: ObjectPropertyNode;
 
-    if (this._peekToken().text !== "}") {
+    if (this._peekToken()._text !== "}") {
       do {
         if (this._peek("}")) {
           // Support trailing commas per ES5.1.
@@ -520,12 +520,12 @@ export class AST {
           _computed: false,
         };
 
-        if (nextToken.constant) {
+        if (nextToken._constant) {
           property._key = this._constant();
           property._computed = false;
           this._consume(":");
           property._value = this._assignment();
-        } else if (nextToken.identifier) {
+        } else if (nextToken._identifier) {
           property._key = this._identifier();
           property._computed = false;
 
@@ -562,11 +562,11 @@ export class AST {
     throw $parseMinErr(
       "syntax",
       "Syntax Error: Token '{0}' {1} at column {2} of the expression [{3}] starting at [{4}].",
-      token.text,
+      token._text,
       msg,
-      token.index + 1,
+      token._index + 1,
       this._text,
-      this._text?.substring(token.index),
+      this._text?.substring(token._index),
     );
   }
 
@@ -626,7 +626,7 @@ export class AST {
 
     if (!j) return token;
 
-    const txt = token.text;
+    const txt = token._text;
 
     if (expected.length === 1) return expected[0] === txt ? token : false;
 

src/core/parse/interpreter.ts

@@ -61,7 +61,7 @@ export class ASTInterpreter {
    * @param {ASTNode} ast - The AST to compile.
    * @returns {CompiledExpression}
    */
-  compile(ast: ASTNode): CompiledExpression {
+  _compile(ast: ASTNode): CompiledExpression {
     const decoratedNode = findConstantAndWatchExpressions(ast, this._$filter);
 
     const { _body: body } = decoratedNode as BodyNode;
@@ -160,7 +160,7 @@ export class ASTInterpreter {
 
     switch (ast._type) {
       case ASTType._Literal:
-        return this.value((ast as LiteralNode)._value, context);
+        return this._value((ast as LiteralNode)._value, context);
       case ASTType._UnaryExpression:
         right = this._recurse((ast as ExpressionNode)._argument as ASTNode);
 
@@ -183,7 +183,7 @@ export class ASTInterpreter {
           context,
         );
       case ASTType._Identifier:
-        return self.identifier(
+        return self._identifier(
           (ast as LiteralNode)._name as string,
           context,
           create,
@@ -210,7 +210,7 @@ export class ASTInterpreter {
               context,
               create,
             )
-          : this.nonComputedMember(left, right as string, context, create);
+          : this._nonComputedMember(left, right as string, context, create);
       case ASTType._CallExpression:
         args = [];
         const callArguments = (ast as ExpressionNode)._arguments as ASTNode[];
@@ -798,7 +798,7 @@ export class ASTInterpreter {
    * @param {Object} [context] - The context.
    * @returns {CompiledExpressionFunction} The function returning the literal value.
    */
-  value(value: any, context?: LinkContext): CompiledExpressionFunction {
+  _value(value: any, context?: LinkContext): CompiledExpressionFunction {
     return () =>
       context ? { context: undefined, name: undefined, value } : value;
   }
@@ -810,7 +810,7 @@ export class ASTInterpreter {
    * @param {boolean|1} [create] - Whether to create the identifier if it does not exist.
    *  @returns {CompiledExpressionFunction}  The function returning the identifier value.
    */
-  identifier(
+  _identifier(
     name: string,
     context?: LinkContext,
     create?: CreateFlag,
@@ -889,7 +889,7 @@ export class ASTInterpreter {
    * @param {boolean|1} [create] - Whether to create the member if it does not exist.
    * @returns {CompiledExpressionFunction}  The function returning the non-computed member value.
    */
-  nonComputedMember(
+  _nonComputedMember(
     left: CompiledExpressionFunction,
     right: string,
     context?: LinkContext,