@abaplint/cli 2.113.135 → 2.113.137

package/build/cli.js

@@ -6460,7 +6460,7 @@ const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/co
 class MessageClass extends combi_1.Expression {
     getRunnable() {
         // "&1" can be used for almost anything(field names, method names etc.) in macros
-        return (0, combi_1.seq)((0, combi_1.regex)(/^>?[\w\/]+#?@?\/?!?&?>?\$?\??<?$/), (0, combi_1.starPrio)((0, combi_1.tok)(tokens_1.Plus)), (0, combi_1.starPrio)((0, combi_1.tok)(tokens_1.PlusW)), (0, combi_1.starPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.Dash), (0, combi_1.optPrio)((0, combi_1.regex)(/^\w+$/)))), (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.DashW)));
+        return (0, combi_1.seq)((0, combi_1.regex)(/^>?[\w\/]+#?@?\/?!?&?>?\$?\??<?§?~?$/), (0, combi_1.starPrio)((0, combi_1.tok)(tokens_1.Plus)), (0, combi_1.starPrio)((0, combi_1.tok)(tokens_1.PlusW)), (0, combi_1.starPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.Dash), (0, combi_1.optPrio)((0, combi_1.regex)(/^\w+$/)))), (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.DashW)));
     }
 }
 exports.MessageClass = MessageClass;
@@ -16101,7 +16101,7 @@ const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@a
 const transporting_fields_1 = __webpack_require__(/*! ../expressions/transporting_fields */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/transporting_fields.js");
 class ReadTable {
     getMatcher() {
-        const comparing = (0, combi_1.seq)("COMPARING", (0, combi_1.alt)((0, combi_1.plus)(expressions_1.FieldSub), expressions_1.Dynamic));
+        const comparing = (0, combi_1.seq)("COMPARING", (0, combi_1.alt)((0, combi_1.plus)(expressions_1.FieldSub), (0, combi_1.plus)(expressions_1.Dynamic)));
         const index = (0, combi_1.seq)("INDEX", expressions_1.Source);
         const components = (0, combi_1.seq)((0, combi_1.alt)(expressions_1.Field, expressions_1.Dynamic), "COMPONENTS", expressions_1.ComponentCompareSimple);
         const key = (0, combi_1.seq)((0, combi_1.altPrio)("WITH KEY", "WITH TABLE KEY"), (0, combi_1.alt)(expressions_1.ComponentCompareSimple, components, (0, combi_1.seq)((0, combi_1.optPrio)("="), expressions_1.Source)));
@@ -23426,6 +23426,13 @@ class TypeUtils {
         }
         return ret;
     }
+    isConvable(source, target) {
+        if (source instanceof basic_1.TableType && target instanceof basic_1.TableType) {
+            // table key conversion, eg DEFAULT to EMPTY
+            return this.isAssignable(source.getRowType(), target.getRowType());
+        }
+        return this.isAssignable(source, target);
+    }
     isCastable(_source, _target) {
         // todo
         return true;
@@ -28051,7 +28058,7 @@ class Source {
                     {
                         const foundType = this.determineType(node, input, targetType);
                         const bodyType = conv_body_1.ConvBody.runSyntax(node.findDirectExpression(Expressions.ConvBody), input);
-                        if (new _type_utils_1.TypeUtils(input.scope).isAssignable(foundType, bodyType) === false) {
+                        if (new _type_utils_1.TypeUtils(input.scope).isConvable(foundType, bodyType) === false) {
                             const message = `CONV: Types not compatible, ${foundType === null || foundType === void 0 ? void 0 : foundType.constructor.name}, ${bodyType === null || bodyType === void 0 ? void 0 : bodyType.constructor.name}`;
                             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                             return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
@@ -54559,14 +54566,14 @@ class Registry {
         this.objects = {};
         this.objectsByType = {};
         this.dependencies = {};
-        this.conf = conf ? conf : config_1.Config.getDefault();
+        this.setConfig(conf ? conf : config_1.Config.getDefault());
         this.ddicReferences = new ddic_references_1.DDICReferences();
         this.msagReferences = new msag_references_1.MSAGReferences();
         this.macroReferences = new macro_references_1.MacroReferences();
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.135";
+        return "2.113.137";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -54644,12 +54651,11 @@ class Registry {
             obj.setDirty();
         }
         this.conf = conf;
+        this.errorNamespace = new RegExp(this.getConfig().getSyntaxSetttings().errorNamespace, "i");
         return this;
     }
     inErrorNamespace(name) {
-        // todo: performance? cache regexp?
-        const reg = new RegExp(this.getConfig().getSyntaxSetttings().errorNamespace, "i");
-        return reg.test(name);
+        return this.errorNamespace.test(name);
     }
     addFile(file) {
         return this.addFiles([file]);
@@ -77339,9 +77345,35 @@ const artifacts_rules_1 = __webpack_require__(/*! ./artifacts_rules */ "./node_m
 const _abap_object_1 = __webpack_require__(/*! ./objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
 const skip_logic_1 = __webpack_require__(/*! ./skip_logic */ "./node_modules/@abaplint/core/build/src/skip_logic.js");
 const excludeHelper_1 = __webpack_require__(/*! ./utils/excludeHelper */ "./node_modules/@abaplint/core/build/src/utils/excludeHelper.js");
+class SyntaxPerformance {
+    constructor() {
+        this.results = [];
+    }
+    push(obj, runtime) {
+        if (runtime < 100) {
+            return;
+        }
+        this.results.push({
+            runtime: runtime,
+            name: obj.getType() + " " + obj.getName(),
+        });
+    }
+    output() {
+        const MAX = 10;
+        this.results.sort((a, b) => { return b.runtime - a.runtime; });
+        for (let i = 0; i < MAX; i++) {
+            const row = this.results[i];
+            if (row === undefined) {
+                break;
+            }
+            process.stderr.write(`\t${row.runtime}ms\t${row.name}\n`);
+        }
+    }
+}
 class RulesRunner {
     constructor(reg) {
         this.reg = reg;
+        this.syntaxPerformance = new SyntaxPerformance();
     }
     objectsToCheck(objects) {
         const check = [];
@@ -77366,12 +77398,25 @@ class RulesRunner {
         for (const obj of check) {
             (_b = input === null || input === void 0 ? void 0 : input.progress) === null || _b === void 0 ? void 0 : _b.tick("Run Syntax - " + obj.getName());
             if (obj instanceof _abap_object_1.ABAPObject) {
+                const start = Date.now();
                 new syntax_1.SyntaxLogic(this.reg, obj).run();
+                if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {
+                    this.syntaxPerformance.push(obj, Date.now() - start);
+                }
             }
         }
+        if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {
+            process.stderr.write("Syntax Performance:\n");
+            this.syntaxPerformance.output();
+        }
         (_c = input === null || input === void 0 ? void 0 : input.progress) === null || _c === void 0 ? void 0 : _c.set(rules.length, "Initialize Rules");
         for (const rule of rules) {
-            (_d = input === null || input === void 0 ? void 0 : input.progress) === null || _d === void 0 ? void 0 : _d.tick("Initialize Rules - " + rule.getMetadata().key);
+            if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {
+                process.stderr.write("Initializing rule " + rule.getMetadata().key + "\n");
+            }
+            else {
+                (_d = input === null || input === void 0 ? void 0 : input.progress) === null || _d === void 0 ? void 0 : _d.tick("Initialize Rules - " + rule.getMetadata().key);
+            }
             if (rule.initialize === undefined) {
                 throw new Error(rule.getMetadata().key + " missing initialize method");
             }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.135",
+    "version": "2.113.137",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.135",
+        "@abaplint/core": "^2.113.137",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
@@ -47,7 +47,7 @@
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",
         "chalk": "^5.4.1",
-        "eslint": "^9.30.0",
+        "eslint": "^9.30.1",
         "glob": "^11.0.3",
         "json5": "^2.2.3",
         "memfs": "^4.17.2",