@abaplint/core 2.113.138 → 2.113.140

package/build/src/objects/function_group.js

@@ -80,12 +80,14 @@ class FunctionGroup extends _abap_object_1.ABAPObject {
                 }
                 if ((i.startsWith("L") || namespaced) && f.getFilename().includes(search.toLowerCase() + ".")) {
                     ret.push({ file: f, name: i });
+                    break;
                 }
                 // fix for URL encoded? Uris
                 if (namespaced) {
                     search = i.replace(/\//g, "%23");
                     if (f.getFilename().includes(search.toLowerCase() + ".")) {
                         ret.push({ file: f, name: i });
+                        break;
                     }
                 }
             }

package/build/src/registry.js

@@ -67,7 +67,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.138";
+        return "2.113.140";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules_runner.js

@@ -72,8 +72,9 @@ class RulesRunner {
         }
         (_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) {
+            const start = Date.now();
             if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {
-                process.stderr.write("Initializing rule " + rule.getMetadata().key + "\n");
+                process.stderr.write("Initializing rule " + rule.getMetadata().key);
             }
             else {
                 (_d = input === null || input === void 0 ? void 0 : input.progress) === null || _d === void 0 ? void 0 : _d.tick("Initialize Rules - " + rule.getMetadata().key);
@@ -82,6 +83,9 @@ class RulesRunner {
                 throw new Error(rule.getMetadata().key + " missing initialize method");
             }
             rule.initialize(this.reg);
+            if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {
+                process.stderr.write(", " + (Date.now() - start) + "ms\n");
+            }
             rulePerformance[rule.getMetadata().key] = 0;
         }
         (_e = input === null || input === void 0 ? void 0 : input.progress) === null || _e === void 0 ? void 0 : _e.set(check.length, "Finding Issues");

package/build/src/utils/include_graph.js

@@ -28,7 +28,9 @@ class Graph {
         this.edges = {};
     }
     addVertex(vertex) {
-        this.verticesIncludenameIndex[vertex.includeName.toUpperCase()] = vertex;
+        if (vertex.includeName !== undefined) {
+            this.verticesIncludenameIndex[vertex.includeName.toUpperCase()] = vertex;
+        }
         this.verticesFilenameIndex[vertex.filename.toUpperCase()] = vertex;
     }
     findVertexViaIncludename(includeName) {
@@ -64,9 +66,6 @@ class IncludeGraph {
         this.graph = new Graph();
         this.build();
     }
-    getIssues() {
-        return this.issues;
-    }
     listMainForInclude(filename) {
         const ret = [];
         if (filename === undefined) {
@@ -86,6 +85,17 @@ class IncludeGraph {
                 ret.push(i);
             }
         }
+        const v = this.graph.findVertexByFilename(file.getFilename());
+        if (v !== undefined
+            && v.include === true
+            && this.listMainForInclude(v.filename).length === 0) {
+            const f = this.reg.getFileByName(v.filename);
+            if (f === undefined) {
+                throw new Error("findUnusedIncludes internal error");
+            }
+            const issue = issue_1.Issue.atPosition(f, new position_1.Position(1, 1), "INCLUDE not used anywhere", new check_include_1.CheckInclude().getMetadata().key, severity_1.Severity.Error);
+            ret.push(issue);
+        }
         return ret;
     }
     ///////////////////////////////
@@ -126,21 +136,6 @@ class IncludeGraph {
                 }
             }
         }
-        this.findUnusedIncludes();
-    }
-    findUnusedIncludes() {
-        for (const v of Object.values(this.graph.verticesFilenameIndex)) {
-            if (v.include === true) {
-                if (this.listMainForInclude(v.filename).length === 0) {
-                    const f = this.reg.getFileByName(v.filename);
-                    if (f === undefined) {
-                        throw new Error("findUnusedIncludes internal error");
-                    }
-                    const issue = issue_1.Issue.atPosition(f, new position_1.Position(1, 1), "INCLUDE not used anywhere", new check_include_1.CheckInclude().getMetadata().key, severity_1.Severity.Error);
-                    this.issues.push(issue);
-                }
-            }
-        }
     }
     addVertices() {
         for (const o of getABAPObjects(this.reg)) {
@@ -159,7 +154,7 @@ class IncludeGraph {
                 if (file) {
                     this.graph.addVertex({
                         filename: file.getFilename(),
-                        includeName: o.getName(),
+                        includeName: undefined,
                         include: false
                     });
                 }
@@ -168,7 +163,7 @@ class IncludeGraph {
                 for (const f of o.getSequencedFiles()) {
                     this.graph.addVertex({
                         filename: f.getFilename(),
-                        includeName: o.getName(),
+                        includeName: undefined,
                         include: false
                     });
                 }
@@ -185,7 +180,7 @@ class IncludeGraph {
                 if (file) {
                     this.graph.addVertex({
                         filename: file.getFilename(),
-                        includeName: o.getName(),
+                        includeName: undefined, // this is the SAPL program
                         include: false
                     });
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.113.138",
+    "version": "2.113.140",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -53,7 +53,7 @@
         "@microsoft/api-extractor": "^7.52.8",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.0.10",
+        "@types/node": "^24.0.12",
         "chai": "^4.5.0",
         "eslint": "^9.30.1",
         "mocha": "^11.7.1",