@abaplint/transpiler-cli 2.10.28 → 2.10.30

package/build/bundle.js

@@ -78257,6 +78257,7 @@ const core_1 = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abapli
 const traversal_1 = __webpack_require__(/*! ../traversal */ "./node_modules/@abaplint/transpiler/build/src/traversal.js");
 const method_call_param_1 = __webpack_require__(/*! ./method_call_param */ "./node_modules/@abaplint/transpiler/build/src/expressions/method_call_param.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 MethodCallTranspiler {
     transpile(node, traversal) {
         const nameToken = node.findDirectExpression(core_1.Expressions.MethodName)?.getFirstToken();
@@ -78274,6 +78275,11 @@ class MethodCallTranspiler {
         if (m?.name && traversal.isBuiltinMethod(nameToken) === false) {
             name = m.name.toLowerCase() + "(";
         }
+        if (feature_flags_1.FEATURE_FLAGS.private === true
+            && m?.def.getVisibility() === core_1.Visibility.Private
+            && m?.def.isStatic() === false) {
+            name = "#" + name;
+        }
         const step = node.findDirectExpression(core_1.Expressions.MethodCallParam);
         if (step === undefined) {
             throw new Error("MethodCallTranspiler, unexpected node");
@@ -79907,6 +79913,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: 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 ***!
@@ -81762,6 +81785,7 @@ static INTERNAL_TYPE = 'CLAS';
 static INTERNAL_NAME = '${traversal.buildInternalName(token.getStr(), def)}';
 static IMPLEMENTED_INTERFACES = [${this.findImplementedByClass(traversal, def, scope).map(e => `"` + e.toUpperCase() + `"`).join(",")}];
 static ATTRIBUTES = {${traversal.buildAttributes(def, scope).join(",\n")}};
+static FRIENDS_ACCESS_STATIC = {}; // todo
 static METHODS = {${traversal.buildMethods(def, scope).join(",\n")}};`, node, traversal);
     }
     findImplementedInterface(traversal, def, scope) {
@@ -84373,6 +84397,7 @@ const traversal_1 = __webpack_require__(/*! ../traversal */ "./node_modules/@aba
 const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
 const unique_identifier_1 = __webpack_require__(/*! ../unique_identifier */ "./node_modules/@abaplint/transpiler/build/src/unique_identifier.js");
+const feature_flags_1 = __webpack_require__(/*! ../feature_flags */ "./node_modules/@abaplint/transpiler/build/src/feature_flags.js");
 class MethodImplementationTranspiler {
     transpile(node, traversal) {
         const token = node.findFirstExpression(abaplint.Expressions.MethodName).getFirstToken();
@@ -84481,15 +84506,22 @@ class MethodImplementationTranspiler {
                 methodName = a.getComponent().replace("~", "$").toLowerCase();
             }
         }
+        // https://github.com/tc39/proposal-class-fields
+        let isPrivate = "";
+        if (feature_flags_1.FEATURE_FLAGS.private === true
+            && method?.getVisibility() === abaplint.Visibility.Private
+            && method.isStatic() === false) {
+            isPrivate = "#";
+        }
         if (method && method.isStatic()) {
             // in ABAP static methods can be called with instance arrows, "->"
             const className = scope.getParent()?.getIdentifier().sname?.toLowerCase();
-            staticMethod = "async " + traversal_1.Traversal.escapeNamespace(methodName) + "(" + unique + ") {\n" +
+            staticMethod = "async " + isPrivate + traversal_1.Traversal.escapeNamespace(methodName) + "(" + unique + ") {\n" +
                 "return " + traversal_1.Traversal.escapeNamespace(className) + "." + traversal_1.Traversal.escapeNamespace(methodName) + "(" + unique + ");\n" +
                 "}\n" + "static ";
         }
         unique_identifier_1.UniqueIdentifier.resetIndexBackup();
-        const str = staticMethod + "async " + traversal_1.Traversal.escapeNamespace(methodName) + "(" + unique + ") {" + after;
+        const str = staticMethod + "async " + isPrivate + traversal_1.Traversal.escapeNamespace(methodName) + "(" + unique + ") {" + after;
         return new chunk_1.Chunk().append(str, node, traversal);
     }
     /////////////////////////////
@@ -86959,6 +86991,7 @@ class ${className?.toLowerCase()} {
   static IMPLEMENTED_INTERFACES = [];
   static INTERNAL_NAME = 'ABSTRACT_CLASS_INTERNAL_NAME';
   static ATTRIBUTES = {};
+  static FRIENDS_ACCESS_STATIC = {};
   async constructor_() {
     this.me = new abap.types.ABAPObject();
     this.me.set(this);
@@ -88076,6 +88109,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;
@@ -88443,15 +88477,8 @@ class Traversal {
         }
         return undefined;
     }
-    buildConstructorContents(scope, def) {
+    buildThisAttributes(def, cName) {
         let ret = "";
-        if (def.getSuperClass() !== undefined || def.getName().toUpperCase() === "CX_ROOT") {
-            ret += "super();\n";
-        }
-        const cName = Traversal.escapeNamespace(def.getName().toLowerCase());
-        ret += `this.me = new abap.types.ABAPObject();
-this.me.set(this);
-this.INTERNAL_ID = abap.internalIdCounter++;\n`;
         for (const a of def.getAttributes()?.getAll() || []) {
             const escaped = Traversal.escapeNamespace(a.getName().toLowerCase());
             if (a.getMeta().includes("static" /* abaplint.IdentifierMeta.Static */) === true) {
@@ -88462,6 +88489,39 @@ this.INTERNAL_ID = abap.internalIdCounter++;\n`;
             ret += name + " = " + new transpile_types_1.TranspileTypes().toType(a.getType()) + ";\n";
             ret += this.setValues(a, name);
         }
+        return ret;
+    }
+    buildFriendsAccess(def, hasSuperClass) {
+        let ret = "this.FRIENDS_ACCESS_INSTANCE = {\n";
+        if (hasSuperClass === true) {
+            ret += `"SUPER": sup.FRIENDS_ACCESS_INSTANCE,\n`;
+        }
+        for (const a of def.getMethodDefinitions()?.getAll() || []) {
+            const name = a.getName().toLowerCase();
+            if (name === "constructor" || a.isStatic() === true) {
+                continue;
+            }
+            let privateHash = "";
+            if (feature_flags_1.FEATURE_FLAGS.private === true && a.getVisibility() === abaplint.Visibility.Private) {
+                privateHash = "#";
+            }
+            const methodName = privateHash + Traversal.escapeNamespace(name.replace("~", "$"));
+            ret += `"${name.replace("~", "$")}": this.${methodName}.bind(this),\n`;
+        }
+        ret += "};\n";
+        return ret;
+    }
+    buildConstructorContents(scope, def) {
+        let ret = "";
+        if (def.getSuperClass() !== undefined || def.getName().toUpperCase() === "CX_ROOT") {
+            ret += "const sup = super();\n";
+        }
+        const cName = Traversal.escapeNamespace(def.getName().toLowerCase());
+        ret += `this.me = new abap.types.ABAPObject();
+this.me.set(this);
+this.INTERNAL_ID = abap.internalIdCounter++;\n`;
+        ret += this.buildFriendsAccess(def, def.getSuperClass() !== undefined);
+        ret += this.buildThisAttributes(def, cName);
         // attributes from directly implemented interfaces(not interfaces implemented in super classes)
         for (const i of def.getImplementing()) {
             ret += this.dataFromInterfaces(i.name, scope, cName);
@@ -89027,10 +89087,13 @@ async function run() {
                     continue;
                 }
                 ret += `      {\n        const test = await (new ${lc}()).constructor_();\n`;
+                // todo, some refactoring here,
                 ret += `        if (test.setup) await test.setup();\n`;
+                ret += `        if (test.FRIENDS_ACCESS_INSTANCE.setup) await test.FRIENDS_ACCESS_INSTANCE.setup();\n`;
+                ret += `        if (test.FRIENDS_ACCESS_INSTANCE.SUPER && test.FRIENDS_ACCESS_INSTANCE.SUPER.setup) await test.FRIENDS_ACCESS_INSTANCE.SUPER.setup();\n`;
                 ret += `        console.log("${st.obj.getName()}: running ${lc}->${m}");\n`;
                 ret += `        meth = locl.addMethod("${m}");\n`;
-                ret += `        await test.${m}();\n`;
+                ret += `        await test.FRIENDS_ACCESS_INSTANCE.${m}();\n`;
                 ret += `        meth.pass();\n`;
                 ret += `        if (test.teardown) await test.teardown();\n`;
                 ret += `      }\n`;
@@ -89038,59 +89101,6 @@ async function run() {
             ret += `      if (${lc}.class_teardown) await ${lc}.class_teardown();\n`;
             ret += `    }\n`;
         }
-        /*
-            for (const obj of reg.getObjects()) {
-              if (reg.isDependency(obj) || !(obj instanceof abaplint.Objects.Class)) {
-                continue;
-              }
-              ret += `// --------------------------------------------\n`;
-              ret += `    clas = unit.addObject("${obj.getName()}");\n`;
-              for (const file of obj.getABAPFiles()) {
-                for (const def of file.getInfo().listClassDefinitions()) {
-                  if (def.isForTesting === false
-                      || def.isGlobal === true
-                      || def.methods.length === 0
-                      || def.isAbstract === true) {
-                    // todo, fix, there might be global test methods
-                    continue;
-                  }
-                  const hasTestFile = obj.getFiles().some(f => { return f.getFilename().includes(".testclasses."); });
-                  if (hasTestFile === false) {
-                    break;
-                  }
-                  ret += `    {
-              const {${def.name}} = await import("./${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}.testclasses.mjs");
-              locl = clas.addTestClass("${def.name}");
-              if (${def.name}.class_setup) await ${def.name}.class_setup();\n`;
-        
-                  for (const m of def.methods) {
-                    if (m.isForTesting === false) {
-                      continue;
-                    }
-                    const skipThis = (skip || []).some(a => a.object === obj.getName() && a.class === def.name && a.method === m.name);
-                    if (skipThis) {
-                      ret += `  console.log('${obj.getName()}: running ${def.name}->${m.name}, skipped');\n`;
-                      ret += `  meth = locl.addMethod("${m.name}");\n`;
-                      ret += `  meth.skip();\n`;
-                      continue;
-                    }
-        
-                    ret += `      {\n        const test = await (new ${def.name}()).constructor_();\n`;
-                    ret += `        if (test.setup) await test.setup();\n`;
-                    ret += `        console.log("${obj.getName()}: running ${def.name}->${m.name}");\n`;
-                    ret += `        meth = locl.addMethod("${m.name}");\n`;
-                    ret += `        await test.${m.name}();\n`;
-                    ret += `        meth.pass();\n`;
-                    ret += `        if (test.teardown) await test.teardown();\n`;
-                    ret += `      }\n`;
-                  }
-        
-                  ret += `      if (${def.name}.class_teardown) await ${def.name}.class_teardown();\n`;
-                  ret += `    }\n`;
-                }
-              }
-            }
-            */
         ret += `// -------------------END-------------------
     fs.writeFileSync(__dirname + path.sep + "_output.xml", unit.xUnitXML());
   } catch (e) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.10.28",
+  "version": "2.10.30",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -28,7 +28,7 @@
   "license": "MIT",
   "devDependencies": {
     "@abaplint/core": "^2.113.107",
-    "@abaplint/transpiler": "^2.10.28",
+    "@abaplint/transpiler": "^2.10.30",
     "@types/glob": "^8.1.0",
     "@types/node": "^22.13.13",
     "@types/progress": "^2.0.7",