@abaplint/transpiler-cli 2.10.65 → 2.10.67

package/build/bundle.js

@@ -47305,8 +47305,12 @@ class DataElement extends _abstract_object_1.AbstractObject {
             references.push({ object: lookup.object });
         }
         reg.getDDICReferences().setUsing(this, references);
-        this.parsedType = lookup.type;
-        return this.parsedType;
+        if (!(lookup.type instanceof Types.UnknownType)) {
+            // the referenced type might not exist or contain syntax errors(for CLAS)
+            // so dont cache it, expect the user to fix it
+            this.parsedType = lookup.type;
+        }
+        return lookup.type;
     }
     parse() {
         var _a, _b, _c;
@@ -53581,7 +53585,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.148";
+        return "2.113.149";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -80807,6 +80811,23 @@ exports.TargetTranspiler = TargetTranspiler;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/transpiler/build/src/feature_flags.js":
+/*!**********************************************************************!*\
+  !*** ./node_modules/@abaplint/transpiler/build/src/feature_flags.js ***!
+  \**********************************************************************/
+/***/ ((__unused_webpack_module, exports) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.FEATURE_FLAGS = void 0;
+exports.FEATURE_FLAGS = {
+    PRIVATE_ATTRIBUTES: true,
+};
+//# sourceMappingURL=feature_flags.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/transpiler/build/src/handlers/handle_abap.js":
 /*!*****************************************************************************!*\
   !*** ./node_modules/@abaplint/transpiler/build/src/handlers/handle_abap.js ***!
@@ -88164,6 +88185,7 @@ const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abap
 const traversal_1 = __webpack_require__(/*! ../traversal */ "./node_modules/@abaplint/transpiler/build/src/traversal.js");
 const transpile_types_1 = __webpack_require__(/*! ../transpile_types */ "./node_modules/@abaplint/transpiler/build/src/transpile_types.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
+const feature_flags_1 = __webpack_require__(/*! ../feature_flags */ "./node_modules/@abaplint/transpiler/build/src/feature_flags.js");
 class ClassImplementationTranspiler {
     transpile(node, traversal) {
         const ret = new chunk_1.Chunk();
@@ -88232,6 +88254,22 @@ class ClassImplementationTranspiler {
         }
         return ret;
     }
+    buildPrivate(node, traversal) {
+        if (node === undefined || feature_flags_1.FEATURE_FLAGS.PRIVATE_ATTRIBUTES === false) {
+            return "";
+        }
+        const cdef = traversal.getClassDefinition(node.getFirstToken());
+        if (cdef === undefined) {
+            return "ERROR_CDEF_NOT_FOUND";
+        }
+        let ret = "";
+        for (const attr of cdef.getAttributes().getAll()) {
+            if (attr.getVisibility() === abaplint.Visibility.Private) {
+                ret += "#" + traversal_1.Traversal.escapeNamespace(attr.getName().toLowerCase()) + ";\n";
+            }
+        }
+        return ret;
+    }
     /** this builds the part after the class, containing the static variables/constants */
     buildStatic(node, traversal) {
         if (node === undefined) {
@@ -88303,8 +88341,9 @@ class ClassImplementationTranspiler {
         if (ret === "") {
             return ret;
         }
+        const privates = this.buildPrivate(node.findFirstExpression(abaplint.Expressions.ClassName), traversal);
         // note: for CALL TRANSFORMATION, its nice that the values are initialized by the JS constructor,
-        return "constructor() {\n" + ret + "}\n";
+        return privates + "constructor() {\n" + ret + "}\n";
     }
 }
 exports.ClassImplementationTranspiler = ClassImplementationTranspiler;
@@ -89297,6 +89336,7 @@ const transpile_types_1 = __webpack_require__(/*! ./transpile_types */ "./node_m
 const chunk_1 = __webpack_require__(/*! ./chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
 const expressions_1 = __webpack_require__(/*! ./expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 const keywords_1 = __webpack_require__(/*! ./keywords */ "./node_modules/@abaplint/transpiler/build/src/keywords.js");
+const feature_flags_1 = __webpack_require__(/*! ./feature_flags */ "./node_modules/@abaplint/transpiler/build/src/feature_flags.js");
 class Traversal {
     constructor(spaghetti, file, obj, reg, options) {
         this.scopeCache = undefined;
@@ -89412,6 +89452,22 @@ class Traversal {
         }
         return undefined;
     }
+    isPrivateAttribute(token) {
+        const scope = this.findCurrentScopeByToken(token);
+        if (scope === undefined) {
+            throw new Error("isPrivateAttribute, unable to lookup position");
+        }
+        const name = token.getStr();
+        if (name.toLowerCase() === "me") {
+            return false;
+        }
+        const found = scope.findVariable(name);
+        if (found instanceof abaplint.Types.ClassAttribute
+            && found.getVisibility() === abaplint.Visibility.Private) {
+            return true;
+        }
+        return false;
+    }
     isClassAttribute(token) {
         const scope = this.findCurrentScopeByToken(token);
         if (scope === undefined) {
@@ -89453,7 +89509,11 @@ class Traversal {
             return name;
         }
         else if (this.isClassAttribute(t)) {
-            name = "this." + Traversal.escapeNamespace(name);
+            let escaped = Traversal.escapeNamespace(name);
+            if (feature_flags_1.FEATURE_FLAGS.PRIVATE_ATTRIBUTES === true && this.isPrivateAttribute(t)) {
+                escaped = "#" + escaped;
+            }
+            name = "this." + escaped;
         }
         else if (this.isBuiltinVariable(t)) {
             name = "abap.builtin." + name.toLowerCase().replace("%", "$");
@@ -89668,14 +89728,19 @@ class Traversal {
     buildThisAttributes(def, cName) {
         let ret = "";
         for (const a of def.getAttributes()?.getAll() || []) {
-            const escaped = Traversal.escapeNamespace(a.getName().toLowerCase());
+            let escaped = Traversal.escapeNamespace(a.getName().toLowerCase());
             if (a.getMeta().includes("static" /* abaplint.IdentifierMeta.Static */) === true) {
                 ret += "this." + escaped + " = " + cName + "." + escaped + ";\n";
-                continue;
             }
-            const name = "this." + escaped;
-            ret += name + " = " + new transpile_types_1.TranspileTypes().toType(a.getType()) + ";\n";
-            ret += this.setValues(a, name);
+            else {
+                if (feature_flags_1.FEATURE_FLAGS.PRIVATE_ATTRIBUTES === true
+                    && a.getVisibility() === abaplint.Visibility.Private) {
+                    escaped = "#" + escaped;
+                }
+                const name = "this." + escaped;
+                ret += name + " = " + new transpile_types_1.TranspileTypes().toType(a.getType()) + ";\n";
+                ret += this.setValues(a, name);
+            }
         }
         return ret;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.10.65",
+  "version": "2.10.67",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,8 +27,8 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.113.148",
-    "@abaplint/transpiler": "^2.10.65",
+    "@abaplint/core": "^2.113.149",
+    "@abaplint/transpiler": "^2.10.67",
     "@types/glob": "^8.1.0",
     "@types/node": "^22.16.5",
     "@types/progress": "^2.0.7",