@abaplint/transpiler-cli 2.12.5 → 2.12.7

package/abap_transpile

@@ -1,2 +1,2 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 require("./build/bundle");

package/build/bundle.js

@@ -22879,9 +22879,30 @@ class TypeUtils {
                 return true;
             }
             else if (source instanceof basic_1.StructureType) {
-                if (this.structureContainsString(target) && !this.structureContainsString(source)) {
+                const targetDeep = this.structureContainsString(target);
+                const sourceDeep = this.structureContainsString(source);
+                if (targetDeep && !sourceDeep) {
                     return false;
                 }
+                const targetComponents = target.getComponents();
+                const sourceComponents = source.getComponents();
+                if (targetComponents.length !== sourceComponents.length) {
+                    if (targetDeep === true || sourceDeep === true) {
+                        return false;
+                    }
+                }
+                for (let i = 0; i < targetComponents.length; i++) {
+                    if (sourceComponents[i] === undefined) {
+                        continue;
+                    }
+                    // hmm
+                    if (sourceComponents[i].type instanceof basic_1.StringType && !(targetComponents[i].type instanceof basic_1.StringType)) {
+                        return false;
+                    }
+                    else if (!(sourceComponents[i].type instanceof basic_1.StringType) && targetComponents[i].type instanceof basic_1.StringType) {
+                        return false;
+                    }
+                }
                 return true;
             }
             else if (target.containsVoid() === true) {
@@ -24877,7 +24898,7 @@ class FieldAssignment {
                 const text = c.concatTokens();
                 if (text !== "-" && context instanceof basic_1.StructureType) {
                     context = context.getComponentByName(text);
-                    if (context === undefined && targetType.containsVoid() === false) {
+                    if (context === undefined) {
                         const message = `field ${text} does not exist in structure`;
                         input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                         return;
@@ -32031,6 +32052,7 @@ const _object_oriented_1 = __webpack_require__(/*! ../_object_oriented */ "./nod
 const _scope_type_1 = __webpack_require__(/*! ../_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+const visibility_1 = __webpack_require__(/*! ../../4_file_information/visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
 class MethodImplementation {
     runSyntax(node, input) {
         const helper = new _object_oriented_1.ObjectOriented(input.scope);
@@ -32044,6 +32066,13 @@ class MethodImplementation {
             return;
         }
         const { method: methodDefinition } = helper.searchMethodName(classDefinition, methodName);
+        if (classDefinition.isForTesting()
+            && (methodDefinition === null || methodDefinition === void 0 ? void 0 : methodDefinition.getVisibility()) !== visibility_1.Visibility.Private
+            && ["SETUP", "TEARDOWN", "CLASS_SETUP", "CLASS_TEARDOWN"].includes(methodName.toUpperCase())) {
+            const message = "Special test method \"" + methodName + "\" must be private";
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+            return;
+        }
         const start = node.getFirstToken().getStart();
         if ((methodDefinition === null || methodDefinition === void 0 ? void 0 : methodDefinition.isStatic()) === false) {
             input.scope.push(_scope_type_1.ScopeType.MethodInstance, methodName, start, input.filename);
@@ -32884,6 +32913,11 @@ class ReadTable {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
             }
+            else if (sourceType instanceof basic_1.TableType && sourceType.getAccessType() === basic_1.TableAccessType.hashed) {
+                const message = "INDEX on hashed table not possible";
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                return;
+            }
         }
         const fromSource = node.findExpressionAfterToken("FROM");
         if (fromSource) {
@@ -53637,7 +53671,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.232";
+        return "2.113.233";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -80408,6 +80442,15 @@ class MethodSourceTranspiler {
                 }
                 call += traversal.traverse(child).getCode();
             }
+            else if (child.get() instanceof core_1.Expressions.SourceFieldSymbol) {
+                call += traversal.traverse(child).getCode();
+            }
+            else if (child.get() instanceof core_1.Expressions.ComponentName) {
+                call += `["${child.concatTokens().toLowerCase()}"]`;
+            }
+            else if (child.get() instanceof core_1.Tokens.Dash) {
+                call += '.get()';
+            }
             else {
                 ret.appendString("MethodSourceTranspiler-" + child.get().constructor.name + "-todo");
             }
@@ -84600,7 +84643,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 = {${Array.from(traversal.buildAttributes(def, scope)).join(",\n")}};
-static METHODS = {${traversal.buildMethods(def, scope).join(",\n")}};`, node, traversal);
+static METHODS = {${traversal.buildMethods(def).join(",\n")}};`, node, traversal);
     }
     findImplementedInterface(traversal, def, scope) {
         if (def === undefined || scope === undefined) {
@@ -90736,7 +90779,7 @@ class InterfaceTranspiler {
                 ret += `static INTERNAL_TYPE = 'INTF';\n`;
                 ret += `static INTERNAL_NAME = '${traversal.buildInternalName(token.getStr(), def)}';\n`;
                 ret += `static ATTRIBUTES = {${Array.from(traversal.buildAttributes(def, scope)).join(",\n")}};\n`;
-                ret += `static METHODS = {${traversal.buildMethods(def, scope).join(",\n")}};\n`;
+                ret += `static METHODS = {${traversal.buildMethods(def).join(",\n")}};\n`;
                 // todo, add IMPLEMENTED_INTERFACES ?
             }
             else if (c instanceof abaplint.Nodes.StatementNode && c.get() instanceof abaplint.Statements.EndInterface) {
@@ -91632,7 +91675,7 @@ class Traversal {
         }
         return undefined;
     }
-    buildMethods(def, _scope) {
+    buildMethods(def) {
         const methods = [];
         if (def === undefined) {
             return methods;
@@ -103651,32 +103694,36 @@ class Progress {
 }
 async function loadLib(config) {
     const files = [];
-    for (const l of config.libs || []) {
+    for (const lib of config.libs || []) {
         let dir = "";
         let cleanupFolder = false;
-        if (l.folder !== undefined && l.folder !== "" && fs.existsSync(process.cwd() + l.folder)) {
-            console.log("From folder: " + l.folder);
-            dir = process.cwd() + l.folder;
+        if (lib.folder !== undefined && lib.folder !== "" && fs.existsSync(process.cwd() + lib.folder)) {
+            console.log("From folder: " + lib.folder);
+            dir = process.cwd() + lib.folder;
         }
         else {
-            console.log("Clone: " + l.url);
+            console.log("Clone: " + lib.url);
             dir = fs.mkdtempSync(path.join(os.tmpdir(), "abap_transpile-"));
-            childProcess.execSync("git clone --quiet --depth 1 " + l.url + " .", { cwd: dir, stdio: "inherit" });
+            childProcess.execSync("git clone --quiet --depth 1 " + lib.url + " .", { cwd: dir, stdio: "inherit" });
             cleanupFolder = true;
         }
         let patterns = ["/src/**"];
-        if (l.files !== undefined && typeof l.files === "string" && l.files !== "") {
-            patterns = [l.files];
+        if (lib.files !== undefined && typeof lib.files === "string" && lib.files !== "") {
+            patterns = [lib.files];
         }
-        else if (Array.isArray(l.files)) {
-            patterns = l.files;
+        else if (Array.isArray(lib.files)) {
+            patterns = lib.files;
         }
+        const excludeFilters = (lib.exclude_filter ?? []).map(pattern => new RegExp(pattern, "i"));
         const filesToRead = [];
         for (const pattern of patterns) {
             for (const filename of glob.sync(dir + pattern, { nosort: true, nodir: true })) {
                 if (filename.endsWith(".clas.testclasses.abap")) {
                     continue;
                 }
+                else if (excludeFilters.length > 0 && excludeFilters.some(a => a.test(filename)) === true) {
+                    continue;
+                }
                 filesToRead.push(filename);
             }
         }

package/build/types.d.ts

@@ -1,15 +1,23 @@
 import { ITranspilerOptions } from "@abaplint/transpiler";
 export interface ITranspilerConfig {
     input_folder: string | string[];
-    /** list of regex, case insensitive, empty gives all files, positive list */
+    /** list of regex, case insensitive, empty gives all files, positive list
+     * @uniqueItems true
+     */
     input_filter?: string[];
-    /** list of regex, case insensitive */
+    /** list of regex, case insensitive
+     * @uniqueItems true
+     */
     exclude_filter?: string[];
     output_folder: string;
     libs?: {
         url?: string;
         folder?: string;
         files?: string | string[];
+        /** list of regex, case insensitive
+         * @uniqueItems true
+         */
+        exclude_filter?: string[];
     }[];
     write_unit_tests?: boolean;
     write_source_map?: boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.12.5",
+  "version": "2.12.7",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,10 +27,10 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.113.232",
-    "@abaplint/transpiler": "^2.12.5",
+    "@abaplint/core": "^2.113.233",
+    "@abaplint/transpiler": "^2.12.7",
     "@types/glob": "^8.1.0",
-    "@types/node": "^24.8.1",
+    "@types/node": "^24.9.1",
     "@types/progress": "^2.0.7",
     "glob": "=7.2.0",
     "progress": "^2.0.3",

package/schema.json

@@ -0,0 +1,219 @@
+{
+  "$ref": "#/definitions/ITranspilerConfig",
+  "$schema": "http://json-schema.org/draft-07/schema#",
+  "definitions": {
+    "ITranspilerConfig": {
+      "additionalProperties": false,
+      "properties": {
+        "exclude_filter": {
+          "description": "list of regex, case insensitive",
+          "items": {
+            "type": "string"
+          },
+          "type": "array",
+          "uniqueItems": true
+        },
+        "input_filter": {
+          "description": "list of regex, case insensitive, empty gives all files, positive list",
+          "items": {
+            "type": "string"
+          },
+          "type": "array",
+          "uniqueItems": true
+        },
+        "input_folder": {
+          "anyOf": [
+            {
+              "type": "string"
+            },
+            {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            }
+          ]
+        },
+        "libs": {
+          "items": {
+            "additionalProperties": false,
+            "properties": {
+              "exclude_filter": {
+                "description": "list of regex, case insensitive",
+                "items": {
+                  "type": "string"
+                },
+                "type": "array",
+                "uniqueItems": true
+              },
+              "files": {
+                "anyOf": [
+                  {
+                    "type": "string"
+                  },
+                  {
+                    "items": {
+                      "type": "string"
+                    },
+                    "type": "array"
+                  }
+                ]
+              },
+              "folder": {
+                "type": "string"
+              },
+              "url": {
+                "type": "string"
+              }
+            },
+            "type": "object"
+          },
+          "type": "array"
+        },
+        "options": {
+          "$ref": "#/definitions/ITranspilerOptions"
+        },
+        "output_folder": {
+          "type": "string"
+        },
+        "write_source_map": {
+          "type": "boolean"
+        },
+        "write_unit_tests": {
+          "type": "boolean"
+        }
+      },
+      "required": [
+        "input_folder",
+        "output_folder",
+        "options"
+      ],
+      "type": "object"
+    },
+    "ITranspilerOptions": {
+      "additionalProperties": false,
+      "properties": {
+        "addCommonJS": {
+          "description": "adds common js modules",
+          "type": "boolean"
+        },
+        "addFilenames": {
+          "description": "adds filenames as comments in the output js",
+          "type": "boolean"
+        },
+        "ignoreSourceMap": {
+          "description": "ignore source map",
+          "type": "boolean"
+        },
+        "ignoreSyntaxCheck": {
+          "description": "ignore syntax check, used for internal testing",
+          "type": "boolean"
+        },
+        "importProg": {
+          "description": "import programs",
+          "type": "boolean"
+        },
+        "keywords": {
+          "description": "list of keywords to rename, if not supplied default will be used",
+          "items": {
+            "type": "string"
+          },
+          "type": "array"
+        },
+        "populateTables": {
+          "additionalProperties": false,
+          "description": "populate tables, all tables are populated if undefined and they exist",
+          "properties": {
+            "reposrc": {
+              "description": "insert into REPOSRC, skips if equals false",
+              "type": "boolean"
+            },
+            "seosubco": {
+              "description": "insert into SEOSUBCO, skips if equals false",
+              "type": "boolean"
+            },
+            "seosubcodf": {
+              "description": "insert into SEOSUBCODF, skips if equals false",
+              "type": "boolean"
+            },
+            "seosubcotx": {
+              "description": "insert into SEOSUBCOTX, skips if equals false",
+              "type": "boolean"
+            }
+          },
+          "type": "object"
+        },
+        "setup": {
+          "additionalProperties": false,
+          "description": "extra setup script to be executed during initialization",
+          "properties": {
+            "filename": {
+              "type": "string"
+            },
+            "postFunction": {
+              "type": "string"
+            },
+            "preFunction": {
+              "type": "string"
+            }
+          },
+          "required": [
+            "filename"
+          ],
+          "type": "object"
+        },
+        "skip": {
+          "$ref": "#/definitions/TestMethodList",
+          "description": "list of unit tests to skip"
+        },
+        "skipVersionCheck": {
+          "description": "skips version check, not recommended",
+          "type": "boolean"
+        },
+        "unknownTypes": {
+          "$ref": "#/definitions/UnknownTypesEnum",
+          "description": "sets behavior for unknown types, either fail at compile- or run-time"
+        }
+      },
+      "type": "object"
+    },
+    "TestMethodList": {
+      "items": {
+        "additionalProperties": false,
+        "properties": {
+          "class": {
+            "pattern": "^[a-zA-Z0-9_\\/]+$",
+            "type": "string"
+          },
+          "method": {
+            "pattern": "^[a-zA-Z0-9_\\/]+$",
+            "type": "string"
+          },
+          "note": {
+            "type": "string"
+          },
+          "object": {
+            "pattern": "^[a-zA-Z0-9_\\/]+$",
+            "type": "string"
+          }
+        },
+        "required": [
+          "object",
+          "class",
+          "method"
+        ],
+        "type": "object"
+      },
+      "type": "array",
+      "uniqueItems": true
+    },
+    "UnknownTypesEnum": {
+      "enum": [
+        "compileError",
+        "runtimeError"
+      ],
+      "type": "string"
+    }
+  }
+}
+