@abaplint/core 2.113.135 → 2.113.137

package/build/abaplint.d.ts

@@ -5570,6 +5570,7 @@ export declare class Registry implements IRegistry {
     private readonly ddicReferences;
     private readonly msagReferences;
     private readonly macroReferences;
+    private errorNamespace;
     private conf;
     constructor(conf?: IConfiguration);
     static abaplintVersion(): string;
@@ -5710,6 +5711,7 @@ declare class RollbackEntities implements IStatement {
 
 export declare class RulesRunner {
     private readonly reg;
+    private readonly syntaxPerformance;
     constructor(reg: IRegistry);
     objectsToCheck(objects: Iterable<IObject>): readonly IObject[];
     runRules(objects: Iterable<IObject>, input?: IRunInput): readonly Issue[];

package/build/src/abap/2_statements/expressions/message_class.js

@@ -6,7 +6,7 @@ const combi_1 = require("../combi");
 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;

package/build/src/abap/2_statements/statements/read_table.js

@@ -7,7 +7,7 @@ const version_1 = require("../../../version");
 const transporting_fields_1 = require("../expressions/transporting_fields");
 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)));

package/build/src/abap/5_syntax/_type_utils.js

@@ -203,6 +203,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;

package/build/src/abap/5_syntax/expressions/source.js

@@ -111,7 +111,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);

package/build/src/registry.js

@@ -60,14 +60,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;
@@ -145,12 +145,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]);

package/build/src/rules_runner.js

@@ -6,9 +6,35 @@ const artifacts_rules_1 = require("./artifacts_rules");
 const _abap_object_1 = require("./objects/_abap_object");
 const skip_logic_1 = require("./skip_logic");
 const excludeHelper_1 = require("./utils/excludeHelper");
+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 = [];
@@ -33,12 +59,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/core",
-    "version": "2.113.135",
+    "version": "2.113.137",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -53,9 +53,9 @@
         "@microsoft/api-extractor": "^7.52.8",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.0.8",
+        "@types/node": "^24.0.10",
         "chai": "^4.5.0",
-        "eslint": "^9.30.0",
+        "eslint": "^9.30.1",
         "mocha": "^11.7.1",
         "c8": "^10.1.3",
         "source-map-support": "^0.5.21",