ts-const-value-transformer 0.8.0 → 0.8.2

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## v0.8.2
+
+- Fix to use internal functions to reduce memory usage
+
+## v0.8.1
+
+- Fix to parenthesize expression with multi-line
+- Reduce mappings to only have position changings
+- Minor changes in development environment
+
 ## v0.8.0
 
 - Add `cacheResult` option (enable by default but disable by default for webpack loader)

package/dist/transform.mjs

@@ -174,10 +174,17 @@ function visitNodeAndReplaceIfNeeded(node, parent, sourceFile, program, options,
         else {
             return node;
         }
-        const result = ts.addSyntheticTrailingComment(newNode, ts.SyntaxKind.MultiLineCommentTrivia, ` ${node.getText(sourceFile)} `);
+        const originalSource = node.getText(sourceFile);
+        const comment = ` ${originalSource.replace(/\/\*/g, ' *').replace(/\*\//g, '* ')} `;
+        let result = ts.addSyntheticTrailingComment(newNode, ts.SyntaxKind.MultiLineCommentTrivia, comment);
+        newSource = `${newSource} /*${comment}*/`;
+        if (/[\r\n]/m.test(originalSource)) {
+            result = ts.factory.createParenthesizedExpression(result);
+            newSource = `(${newSource})`;
+        }
         ts.setTextRange(result, node);
         result[SYMBOL_ORIGINAL_NODE_DATA] = [
-            node.getText(sourceFile),
+            originalSource,
             newSource,
             node.pos,
             node.end,
@@ -288,24 +295,6 @@ function getNameFromElementAccessExpression(node, typeChecker) {
     }
     return null;
 }
-function getMemberName(m, tsInstance) {
-    if (!m || !m.name) {
-        return '';
-    }
-    const name = m.name;
-    if (tsInstance.isIdentifier(name)) {
-        return name.escapedText;
-    }
-    else if (tsInstance.isPrivateIdentifier(name)) {
-        return name.escapedText;
-    }
-    else if (tsInstance.isStringLiteral(name)) {
-        return name.text;
-    }
-    else {
-        return '';
-    }
-}
 function isReadonlyPropertyAccess(a, typeChecker, tsInstance) {
     const ts = tsInstance;
     const type = typeChecker.getTypeAtLocation(a.expression);
@@ -316,32 +305,36 @@ function isReadonlyPropertyAccess(a, typeChecker, tsInstance) {
         return false;
     }
     if (type.getFlags() & ts.TypeFlags.Object) {
-        const dummyTypeNode = typeChecker.typeToTypeNode(type, a, ts.NodeBuilderFlags.NoTruncation);
-        if (dummyTypeNode && ts.isTypeLiteralNode(dummyTypeNode)) {
-            for (let i = 0; i < dummyTypeNode.members.length; ++i) {
-                const m = dummyTypeNode.members[i];
-                if (m &&
-                    getMemberName(m, ts) === memberName &&
-                    ts.isPropertySignature(m)) {
-                    if (m.modifiers?.some((m) => m.kind === ts.SyntaxKind.ReadonlyKeyword)) {
-                        return true;
-                    }
+        const prop = type.getProperty(memberName);
+        if (prop) {
+            // Use internal but exported function to improve memory performance
+            if ('getCheckFlags' in ts &&
+                'CheckFlags' in ts &&
+                ts.CheckFlags.Readonly != null) {
+                const checkFlags = ts.getCheckFlags(prop);
+                if (checkFlags & ts.CheckFlags.Readonly) {
+                    return true;
                 }
             }
-        }
-        const prop = type.getProperty(memberName);
-        if (prop && prop.declarations && prop.declarations.length > 0) {
-            const decl = prop.declarations[0];
-            if (ts.isPropertySignature(decl) &&
-                decl.modifiers?.some((m) => m.kind === ts.SyntaxKind.ReadonlyKeyword)) {
-                return true;
+            if ('getDeclarationModifierFlagsFromSymbol' in ts) {
+                const modifierFlags = ts.getDeclarationModifierFlagsFromSymbol(prop);
+                if (modifierFlags & ts.ModifierFlags.Readonly) {
+                    return true;
+                }
             }
-            if (ts.isVariableDeclaration(decl) &&
-                // eslint-disable-next-line @typescript-eslint/strict-boolean-expressions
-                decl.parent &&
-                ts.isVariableDeclarationList(decl.parent) &&
-                decl.parent.flags & ts.NodeFlags.Const) {
-                return true;
+            if (prop.declarations && prop.declarations.length > 0) {
+                const decl = prop.declarations[0];
+                if (ts.isPropertySignature(decl) &&
+                    decl.modifiers?.some((m) => m.kind === ts.SyntaxKind.ReadonlyKeyword)) {
+                    return true;
+                }
+                if (ts.isVariableDeclaration(decl) &&
+                    // eslint-disable-next-line @typescript-eslint/strict-boolean-expressions
+                    decl.parent &&
+                    ts.isVariableDeclarationList(decl.parent) &&
+                    decl.parent.flags & ts.NodeFlags.Const) {
+                    return true;
+                }
             }
         }
     }
@@ -438,13 +431,7 @@ function printSourceImpl(tsInstance, sourceFile, originalSourceName, mapGenerato
 function printNode(tsInstance, baseSource, sourceFile, node, posContext, originalSourceName, mapGenerator) {
     const originalNodeData = node[SYMBOL_ORIGINAL_NODE_DATA];
     if (originalNodeData) {
-        let result = originalNodeData[1];
-        const comments = tsInstance.getSyntheticTrailingComments(node);
-        if (comments) {
-            for (const comment of comments) {
-                result += ` /*${comment.text}*/`;
-            }
-        }
+        const result = originalNodeData[1];
         const old = originalNodeData[0];
         const oldFull = baseSource.substring(originalNodeData[2], originalNodeData[3]);
         const i = oldFull.lastIndexOf(old);
@@ -452,22 +439,12 @@ function printNode(tsInstance, baseSource, sourceFile, node, posContext, origina
         const newText = i < 0
             ? result
             : oldFull.substring(0, i) + result + oldFull.substring(i + old.length);
-        if (mapGenerator) {
-            if (posContext.pos < node.pos) {
-                addMappingForCurrent();
-            }
-            posContext.pos = node.pos;
-            if (leadingUnchanged > 0) {
-                addMappingForCurrent();
-            }
-            posContext.pos = node.pos + leadingUnchanged;
-            addMappingForCurrent(old);
-        }
+        posContext.pos = node.pos + leadingUnchanged;
+        addMappingForCurrent(old);
         posContext.diff += result.length - old.length;
         posContext.pos += old.length;
         addMappingForCurrent();
         posContext.pos = node.end;
-        addMappingForCurrent();
         return newText;
     }
     let output = '';
@@ -479,14 +456,12 @@ function printNode(tsInstance, baseSource, sourceFile, node, posContext, origina
             if (child.pos > node.pos) {
                 const text = baseSource.substring(node.pos, child.pos);
                 output += text;
-                addMappingForCurrent();
                 posContext.pos = child.pos;
             }
         }
         else if (child.pos > lastChildPos) {
             const text = baseSource.substring(lastChildPos, child.pos);
             output += text;
-            addMappingForCurrent();
             posContext.pos = child.pos;
         }
         output += printNode(tsInstance, baseSource, sourceFile, child, posContext, originalSourceName, mapGenerator);
@@ -495,29 +470,19 @@ function printNode(tsInstance, baseSource, sourceFile, node, posContext, origina
     }, void 0);
     if (!headPrinted) {
         output = baseSource.substring(node.pos, node.end);
-        addMappingForCurrent();
         posContext.pos = node.end;
     }
     else if (lastChildPos < node.end) {
         const text = baseSource.substring(lastChildPos, node.end);
         output += text;
-        addMappingForCurrent();
         posContext.pos = node.end;
     }
-    addMappingForCurrent();
     return output;
     function addMappingForCurrent(name) {
         const original = positionToLineAndColumn(sourceFile, posContext.pos, 0);
         if (original.line !== posContext.lastLine) {
             posContext.diff = 0;
             posContext.lastLine = original.line;
-            if (mapGenerator && original.column > 0) {
-                mapGenerator.addMapping({
-                    original: { line: original.line, column: 0 },
-                    generated: { line: original.line, column: 0 },
-                    source: originalSourceName,
-                });
-            }
         }
         if (mapGenerator) {
             mapGenerator.addMapping({

package/dist/version.d.mts

@@ -1,2 +1,2 @@
-declare const _default: "0.8.0";
+declare const _default: "0.8.2";
 export default _default;

package/dist/version.mjs

@@ -1 +1 @@
-export default '0.8.0';
+export default '0.8.2';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ts-const-value-transformer",
-  "version": "0.8.0",
+  "version": "0.8.2",
   "engines": {
     "node": ">=20.19.3"
   },
@@ -90,10 +90,13 @@
     "version": "node ./tools/updateVersion.mjs ./src/main/version.mts && git add -A ./src/main/version.mts"
   },
   "devDependencies": {
+    "@babel/preset-typescript": "^7.28.5",
+    "@swc/core": "^1.15.10",
     "@types/jest": "^30.0.0",
     "@types/node": "~20.19.22",
     "@typescript-eslint/eslint-plugin": "^8.48.0",
     "@typescript-eslint/parser": "^8.48.0",
+    "babel-loader": "^10.0.0",
     "eslint": "^9.39.1",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-import-x": "^4.16.1",
@@ -103,6 +106,7 @@
     "memfs": "^4.51.0",
     "neostandard": "^0.12.2",
     "prettier": "^3.6.2",
+    "swc-loader": "^0.2.7",
     "ts-jest": "^29.4.5",
     "ts-loader": "^9.5.4",
     "ts-patch": "^3.3.0",