@abaplint/cli 2.113.137 → 2.113.139

package/build/cli.js

@@ -49079,12 +49079,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;
                     }
                 }
             }
@@ -54573,7 +54575,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.137";
+        return "2.113.139";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -77836,6 +77838,7 @@ const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./nod
 const severity_1 = __webpack_require__(/*! ../severity */ "./node_modules/@abaplint/core/build/src/severity.js");
 // todo, check for cycles/circular dependencies, method findTop
 // todo, add configurable error for multiple use includes
+const FMXXINCLUDE = /^(\/\w+\/)?L.+XX$/;
 function getABAPObjects(reg) {
     const ret = [];
     for (const o of reg.getObjects()) {
@@ -77847,40 +77850,35 @@ function getABAPObjects(reg) {
 }
 class Graph {
     constructor() {
-        this.vertices = [];
-        this.edges = [];
+        this.verticesIncludenameIndex = {};
+        this.verticesFilenameIndex = {};
+        this.edges = {};
     }
     addVertex(vertex) {
-        this.vertices.push(vertex);
-    }
-    findInclude(includeName) {
-        for (const v of this.vertices) {
-            if (v.includeName.toUpperCase() === includeName.toUpperCase()) {
-                return v;
-            }
+        if (vertex.includeName !== undefined) {
+            this.verticesIncludenameIndex[vertex.includeName.toUpperCase()] = vertex;
         }
-        return undefined;
+        this.verticesFilenameIndex[vertex.filename.toUpperCase()] = vertex;
     }
-    findVertex(filename) {
-        for (const v of this.vertices) {
-            if (v.filename.toUpperCase() === filename.toUpperCase()) {
-                return v;
-            }
-        }
-        return undefined;
+    findVertexViaIncludename(includeName) {
+        return this.verticesIncludenameIndex[includeName.toUpperCase()];
+    }
+    findVertexByFilename(filename) {
+        return this.verticesFilenameIndex[filename.toUpperCase()];
     }
     addEdge(from, toFilename) {
-        this.edges.push({ from: from.filename, to: toFilename });
+        if (this.edges[from.filename] === undefined) {
+            this.edges[from.filename] = [];
+        }
+        this.edges[from.filename].push(toFilename);
     }
     findTop(filename) {
         const ret = [];
-        for (const e of this.edges) {
-            if (e.from === filename) {
-                ret.push(...this.findTop(e.to));
-            }
+        for (const to of this.edges[filename] || []) {
+            ret.push(...this.findTop(to));
         }
         if (ret.length === 0) {
-            const found = this.findVertex(filename);
+            const found = this.findVertexByFilename(filename);
             if (found !== undefined) {
                 ret.push(found);
             }
@@ -77924,19 +77922,23 @@ class IncludeGraph {
         this.addVertices();
         for (const o of getABAPObjects(this.reg)) {
             for (const f of o.getABAPFiles()) {
+                if (f.getFilename().includes(".prog.screen_") || f.getFilename().includes(".fugr.screen_")) {
+                    // skip dynpro files
+                    continue;
+                }
                 for (const s of f.getStatements()) {
                     if (s.get() instanceof statements_1.Include) {
-                        const ifFound = s.concatTokens().toUpperCase().includes("IF FOUND");
                         const iexp = s.findFirstExpression(expressions_1.IncludeName);
                         if (iexp === undefined) {
                             throw new Error("unexpected Include node");
                         }
                         const name = iexp.getFirstToken().getStr().toUpperCase();
-                        if (name.match(/^(\/\w+\/)?L.+XX$/)) { // function module XX includes, possibily namespaced
+                        if (name.match(FMXXINCLUDE)) { // function module XX includes, possibily namespaced
                             continue;
                         }
-                        const found = this.graph.findInclude(name);
+                        const found = this.graph.findVertexViaIncludename(name);
                         if (found === undefined) {
+                            const ifFound = s.concatTokens().toUpperCase().includes("IF FOUND");
                             if (ifFound === false) {
                                 const issue = issue_1.Issue.atStatement(f, s, "Include " + name + " not found", new check_include_1.CheckInclude().getMetadata().key, severity_1.Severity.Error);
                                 this.issues.push(issue);
@@ -77956,7 +77958,7 @@ class IncludeGraph {
         this.findUnusedIncludes();
     }
     findUnusedIncludes() {
-        for (const v of this.graph.vertices) {
+        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);
@@ -77986,7 +77988,7 @@ class IncludeGraph {
                 if (file) {
                     this.graph.addVertex({
                         filename: file.getFilename(),
-                        includeName: o.getName(),
+                        includeName: undefined,
                         include: false
                     });
                 }
@@ -77995,7 +77997,7 @@ class IncludeGraph {
                 for (const f of o.getSequencedFiles()) {
                     this.graph.addVertex({
                         filename: f.getFilename(),
-                        includeName: o.getName(),
+                        includeName: undefined,
                         include: false
                     });
                 }
@@ -78012,7 +78014,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/cli",
-    "version": "2.113.137",
+    "version": "2.113.139",
     "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.137",
+        "@abaplint/core": "^2.113.139",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",