@abaplint/cli 2.113.139 → 2.113.141

package/build/cli.js

@@ -7580,7 +7580,7 @@ class SelectLoop extends combi_1.Expression {
         const tab = (0, combi_1.seq)(_1.SQLIntoTable, (0, combi_1.alt)(pack, (0, combi_1.seq)(_1.SQLFrom, pack), (0, combi_1.seq)(pack, _1.SQLFrom)));
         const packTab = (0, combi_1.seq)(pack, _1.SQLIntoTable);
         const into = (0, combi_1.altPrio)(sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList);
-        const perm = (0, combi_1.per)(_1.SQLFrom, where, sql_up_to_1.SQLUpTo, sql_order_by_1.SQLOrderBy, sql_having_1.SQLHaving, _1.SQLClient, bypass, _1.SQLGroupBy, _1.SQLForAllEntries, (0, combi_1.alt)(tab, into, packTab));
+        const perm = (0, combi_1.per)(_1.SQLFrom, where, sql_up_to_1.SQLUpTo, sql_order_by_1.SQLOrderBy, sql_having_1.SQLHaving, _1.SQLClient, bypass, _1.SQLGroupBy, _1.SQLForAllEntries, (0, combi_1.alt)(tab, sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList, packTab));
         const strict = (0, combi_1.seq)(_1.SQLFrom, (0, combi_1.ver)(version_1.Version.v750, sql_fields_loop_1.SQLFieldsLoop), (0, combi_1.optPrio)(_1.SQLForAllEntries), (0, combi_1.optPrio)((0, combi_1.seq)(where, (0, combi_1.optPrio)(sql_order_by_1.SQLOrderBy), into, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo))));
         const aggr = (0, combi_1.seq)(_1.SQLAggregation, into, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), _1.SQLFrom, where, _1.SQLGroupBy);
         const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), sql_field_list_loop_1.SQLFieldListLoop, perm), strict, aggr), (0, combi_1.optPrio)(sql_hints_1.SQLHints));
@@ -8226,11 +8226,10 @@ const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src
 const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
 const sql_function_1 = __webpack_require__(/*! ./sql_function */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_function.js");
-const simple_field_chain_1 = __webpack_require__(/*! ./simple_field_chain */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/simple_field_chain.js");
 const sql_path_1 = __webpack_require__(/*! ./sql_path */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_path.js");
 class SQLField extends combi_1.Expression {
     getRunnable() {
-        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), simple_field_chain_1.SimpleFieldChain));
+        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.SimpleFieldChain2));
         const as = (0, combi_1.seq)("AS", _1.SQLAsName);
         const field = (0, combi_1.altPrio)(_1.SQLAggregation, _1.SQLCase, sql_function_1.SQLFunction, sql_path_1.SQLPath, _1.SQLFieldName, abap, _1.Constant);
         const sub = (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/", "&&"), (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenLeftW)), field, (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenRightW))));
@@ -8318,7 +8317,7 @@ class SQLFieldListLoop extends combi_1.Expression {
         const comma = (0, combi_1.opt)((0, combi_1.ver)(version_1.Version.v740sp05, ","));
         const as = (0, combi_1.seq)("AS", _1.SQLAsName);
         const someField = (0, combi_1.seq)(_1.SQLField, comma);
-        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.SimpleFieldChain));
+        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.SimpleFieldChain2));
         const fieldList = (0, combi_1.seq)((0, combi_1.star)(someField), (0, combi_1.alt)(_1.SQLFieldName, abap, sql_path_1.SQLPath, _1.Constant), (0, combi_1.optPrio)(as), comma, (0, combi_1.star)(someField));
         const fields = (0, combi_1.alt)("*", _1.Dynamic, fieldList);
         return fields;
@@ -15966,7 +15965,7 @@ const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@a
 class Ranges {
     getMatcher() {
         const occurs = (0, combi_1.seq)("OCCURS", expressions_1.Source);
-        const ret = (0, combi_1.seq)("RANGES", expressions_1.SimpleName, "FOR", expressions_1.SimpleFieldChain2, (0, combi_1.optPrio)(occurs), (0, combi_1.optPrio)(expressions_1.FieldLength));
+        const ret = (0, combi_1.seq)("RANGES", expressions_1.DefinitionName, "FOR", expressions_1.SimpleFieldChain2, (0, combi_1.optPrio)(occurs), (0, combi_1.optPrio)(expressions_1.FieldLength));
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
     }
 }
@@ -33227,7 +33226,7 @@ const assert_error_1 = __webpack_require__(/*! ../assert_error */ "./node_module
 class Ranges {
     runSyntax(node, input) {
         var _a;
-        const nameToken = (_a = node.findFirstExpression(Expressions.SimpleName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+        const nameToken = (_a = node.findFirstExpression(Expressions.DefinitionName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
         const typeExpression = node.findFirstExpression(Expressions.SimpleFieldChain2);
         if (typeExpression === undefined) {
             throw new assert_error_1.AssertError("Ranges, unexpected node");
@@ -54575,7 +54574,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.139";
+        return "2.113.141";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -77413,8 +77412,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);
@@ -77423,6 +77423,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");
@@ -77893,9 +77896,6 @@ class IncludeGraph {
         this.graph = new Graph();
         this.build();
     }
-    getIssues() {
-        return this.issues;
-    }
     listMainForInclude(filename) {
         const ret = [];
         if (filename === undefined) {
@@ -77915,6 +77915,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;
     }
     ///////////////////////////////
@@ -77955,21 +77966,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)) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.139",
+    "version": "2.113.141",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,11 +38,11 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.139",
+        "@abaplint/core": "^2.113.141",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.0.10",
+        "@types/node": "^24.0.12",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",