@abaplint/cli 2.113.201 → 2.113.203

package/build/cli.js

@@ -1250,19 +1250,17 @@ const virtual_position_1 = __webpack_require__(/*! ../../virtual_position */ "./
 const tokens_1 = __webpack_require__(/*! ./tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
 const lexer_buffer_1 = __webpack_require__(/*! ./lexer_buffer */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/lexer_buffer.js");
 const lexer_stream_1 = __webpack_require__(/*! ./lexer_stream */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/lexer_stream.js");
+const ModeNormal = 1;
+const ModePing = 2;
+const ModeStr = 3;
+const ModeTemplate = 4;
+const ModeComment = 5;
+const ModePragma = 6;
 class Lexer {
-    constructor() {
-        this.ModeNormal = 1;
-        this.ModePing = 2;
-        this.ModeStr = 3;
-        this.ModeTemplate = 4;
-        this.ModeComment = 5;
-        this.ModePragma = 6;
-    }
     run(file, virtual) {
         this.virtual = virtual;
         this.tokens = [];
-        this.m = this.ModeNormal;
+        this.m = ModeNormal;
         this.process(file.getRaw());
         return { file, tokens: this.tokens };
     }
@@ -1288,13 +1286,13 @@ class Lexer {
                 pos = new virtual_position_1.VirtualPosition(this.virtual, pos.getRow(), pos.getCol());
             }
             let tok = undefined;
-            if (this.m === this.ModeComment) {
+            if (this.m === ModeComment) {
                 tok = new tokens_1.Comment(pos, s);
             }
-            else if (this.m === this.ModePing || this.m === this.ModeStr) {
+            else if (this.m === ModePing || this.m === ModeStr) {
                 tok = new tokens_1.StringToken(pos, s);
             }
-            else if (this.m === this.ModeTemplate) {
+            else if (this.m === ModeTemplate) {
                 const first = s.charAt(0);
                 const last = s.charAt(s.length - 1);
                 if (first === "|" && last === "|") {
@@ -1487,35 +1485,35 @@ class Lexer {
             const buf = this.buffer.add(current);
             const ahead = this.stream.nextChar();
             const aahead = this.stream.nextNextChar();
-            if (this.m === this.ModeNormal) {
+            if (this.m === ModeNormal) {
                 if (splits[ahead]) {
                     this.add();
                 }
                 else if (ahead === "'") {
                     // start string
                     this.add();
-                    this.m = this.ModeStr;
+                    this.m = ModeStr;
                 }
                 else if (ahead === "|" || ahead === "}") {
                     // start template
                     this.add();
-                    this.m = this.ModeTemplate;
+                    this.m = ModeTemplate;
                 }
                 else if (ahead === "`") {
                     // start ping
                     this.add();
-                    this.m = this.ModePing;
+                    this.m = ModePing;
                 }
                 else if (aahead === "##") {
                     // start pragma
                     this.add();
-                    this.m = this.ModePragma;
+                    this.m = ModePragma;
                 }
                 else if (ahead === "\""
                     || (ahead === "*" && current === "\n")) {
                     // start comment
                     this.add();
-                    this.m = this.ModeComment;
+                    this.m = ModeComment;
                 }
                 else if (ahead === "@" && buf.trim().length === 0) {
                     this.add();
@@ -1536,12 +1534,12 @@ class Lexer {
                     this.add();
                 }
             }
-            else if (this.m === this.ModePragma && (ahead === "," || ahead === ":" || ahead === "." || ahead === " " || ahead === "\n")) {
+            else if (this.m === ModePragma && (ahead === "," || ahead === ":" || ahead === "." || ahead === " " || ahead === "\n")) {
                 // end of pragma
                 this.add();
-                this.m = this.ModeNormal;
+                this.m = ModeNormal;
             }
-            else if (this.m === this.ModePing
+            else if (this.m === ModePing
                 && buf.length > 1
                 && current === "`"
                 && aahead !== "``"
@@ -1550,26 +1548,26 @@ class Lexer {
                 // end of ping
                 this.add();
                 if (ahead === `"`) {
-                    this.m = this.ModeComment;
+                    this.m = ModeComment;
                 }
                 else {
-                    this.m = this.ModeNormal;
+                    this.m = ModeNormal;
                 }
             }
-            else if (this.m === this.ModeTemplate
+            else if (this.m === ModeTemplate
                 && buf.length > 1
                 && (current === "|" || current === "{")
                 && (this.stream.prevChar() !== "\\" || this.stream.prevPrevChar() === "\\\\")) {
                 // end of template
                 this.add();
-                this.m = this.ModeNormal;
+                this.m = ModeNormal;
             }
-            else if (this.m === this.ModeTemplate
+            else if (this.m === ModeTemplate
                 && ahead === "}"
                 && current !== "\\") {
                 this.add();
             }
-            else if (this.m === this.ModeStr
+            else if (this.m === ModeStr
                 && current === "'"
                 && buf.length > 1
                 && aahead !== "''"
@@ -1578,17 +1576,17 @@ class Lexer {
                 // end of string
                 this.add();
                 if (ahead === "\"") {
-                    this.m = this.ModeComment;
+                    this.m = ModeComment;
                 }
                 else {
-                    this.m = this.ModeNormal;
+                    this.m = ModeNormal;
                 }
             }
-            else if (ahead === "\n" && this.m !== this.ModeTemplate) {
+            else if (ahead === "\n" && this.m !== ModeTemplate) {
                 this.add();
-                this.m = this.ModeNormal;
+                this.m = ModeNormal;
             }
-            else if (this.m === this.ModeTemplate && current === "\n") {
+            else if (this.m === ModeTemplate && current === "\n") {
                 this.add();
             }
             if (!this.stream.advance()) {
@@ -53392,7 +53390,7 @@ class Table extends _abstract_object_1.AbstractObject {
     }
     getAllowedNaming() {
         let length = 30;
-        const regex = /^((\/[A-Z_\d]{3,8}\/)|[a-zA-Z0-9]{3})\w+$/;
+        const regex = /^((\/[A-Z_\d]{3,8}\/)|[a-zA-Z0-9]{3}|CI_)\w+$/;
         if (this.getTableCategory() === TableCategory.Transparent) {
             length = 16;
         }
@@ -55001,7 +54999,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.201";
+        return "2.113.203";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -57705,6 +57703,7 @@ class CheckCommentsConf extends _basic_rule_config_1.BasicRuleConfig {
         super(...arguments);
         /** Allows the use of end-of-line comments. */
         this.allowEndOfLine = false;
+        this.maxIssuesPerFile = 10;
     }
 }
 exports.CheckCommentsConf = CheckCommentsConf;
@@ -57750,6 +57749,10 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#put-comment
         if (this.conf.allowEndOfLine === true) {
             return [];
         }
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         const commentRows = [];
         for (let i = 0; i < rows.length; i++) {
             const row = rows[i];
@@ -57766,6 +57769,9 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#put-comment
                     continue;
                 }
                 issues.push(issue_1.Issue.atStatement(file, statement, this.getDescription(IssueType.EndOfLine), this.getMetadata().key, this.conf.severity));
+                if (issues.length >= max) {
+                    break;
+                }
             }
         }
         return issues;
@@ -66026,6 +66032,7 @@ class IndentationConf extends _basic_rule_config_1.BasicRuleConfig {
         this.globalClassSkipFirst = false;
         this.ignoreGlobalClassDefinition = false;
         this.ignoreGlobalInterface = false;
+        this.maxIssuesPerFile = 10;
     }
 }
 exports.IndentationConf = IndentationConf;
@@ -66068,7 +66075,10 @@ ENDCLASS.`,
     }
     runParsed(file, obj) {
         var _a, _b;
-        const MAX_ISSUES = 100;
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         let skip = false;
         if (file.getStructure() === undefined) {
             return []; // syntax error in file
@@ -66140,7 +66150,7 @@ ENDCLASS.`,
                 const message = "Indentation problem, expected " + expected + " spaces";
                 const issue = issue_1.Issue.atPosition(file, position, message, this.getMetadata().key, this.conf.severity, fix);
                 ret.push(issue);
-                if (ret.length >= MAX_ISSUES) {
+                if (ret.length >= max) {
                     break;
                 }
             }
@@ -66810,6 +66820,7 @@ class KeywordCaseConf extends _basic_rule_config_1.BasicRuleConfig {
         this.ignoreGlobalClassBoundaries = false;
         /** A list of keywords to be ignored */
         this.ignoreKeywords = [];
+        this.maxIssuesPerFile = 10;
     }
 }
 exports.KeywordCaseConf = KeywordCaseConf;
@@ -66920,6 +66931,10 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
                 return [];
             }
         }
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         const skip = new Skip(this.getConfig());
         let prev = undefined;
         for (const statement of file.getStatements()) {
@@ -66942,6 +66957,9 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
                 }
                 prev = result[0].token;
             }
+            if (issues.length >= max) {
+                break;
+            }
         }
         return issues;
     }
@@ -67209,6 +67227,7 @@ class LineLengthConf extends _basic_rule_config_1.BasicRuleConfig {
         super(...arguments);
         /** Maximum line length in characters, trailing whitespace ignored */
         this.length = 120;
+        this.maxIssuesPerFile = 10;
     }
 }
 exports.LineLengthConf = LineLengthConf;
@@ -67237,6 +67256,10 @@ https://docs.abapopenchecks.org/checks/04/`,
         const issues = [];
         // maximum line length in abap files
         const maxLineLength = 255;
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         const array = file.getRawRows();
         for (let rowIndex = 0; rowIndex < array.length; rowIndex++) {
             const row = array[rowIndex].replace("\r", "");
@@ -67248,6 +67271,9 @@ https://docs.abapopenchecks.org/checks/04/`,
                 const message = `Reduce line length to max ${this.conf.length}, currently ${row.length}`;
                 issues.push(issue_1.Issue.atRow(file, rowIndex + 1, message, this.getMetadata().key, this.conf.severity));
             }
+            if (issues.length >= max) {
+                break;
+            }
         }
         return issues;
     }
@@ -69692,6 +69718,7 @@ class NoPrefixesConf extends _basic_rule_config_1.BasicRuleConfig {
         /** importing, exporting, returning and changing parameters, case insensitive regex */
         this.methodParameters = "^[ICER].?_";
         this.allowIsPrefixBoolean = true;
+        this.maxIssuesPerFile = 10;
         // todo, public localClass: string = "";
         // todo, public localInterface: string = "";
         // todo, public functionModuleParameters: string = "";
@@ -69738,21 +69765,40 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/sub-sections/AvoidEncodi
             // syntax error, skip
             return [];
         }
+        let max = config.maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         if (config.data !== undefined && config.data !== "") {
             ret.push(...this.checkData(structure, new RegExp(config.data, "i"), file));
         }
+        if (ret.length >= max) {
+            return ret;
+        }
         if (config.statics !== undefined && config.statics !== "") {
             ret.push(...this.checkStatics(structure, new RegExp(config.statics, "i"), file));
         }
+        if (ret.length >= max) {
+            return ret;
+        }
         if (config.fieldSymbols !== undefined && config.fieldSymbols !== "") {
             ret.push(...this.checkFieldSymbols(structure, new RegExp(config.fieldSymbols, "i"), file));
         }
+        if (ret.length >= max) {
+            return ret;
+        }
         if (config.constants !== undefined && config.constants !== "") {
             ret.push(...this.checkConstants(structure, new RegExp(config.constants, "i"), file));
         }
+        if (ret.length >= max) {
+            return ret;
+        }
         if (config.types !== undefined && config.types !== "") {
             ret.push(...this.checkTypes(structure, new RegExp(config.types, "i"), file));
         }
+        if (ret.length >= max) {
+            return ret;
+        }
         if (config.methodParameters !== undefined && config.methodParameters !== "") {
             ret.push(...this.checkMethodParameters(structure, new RegExp(config.methodParameters, "i"), file));
         }
@@ -75678,6 +75724,10 @@ const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/c
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 class UnnecessaryChainingConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        this.maxIssuesPerFile = 10;
+    }
 }
 exports.UnnecessaryChainingConf = UnnecessaryChainingConf;
 class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
@@ -75704,6 +75754,10 @@ class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
     }
     runParsed(file) {
         const issues = [];
+        let max = this.getConfig().maxIssuesPerFile;
+        if (max === undefined || max < 1) {
+            max = 10;
+        }
         const statements = file.getStatements();
         for (let i = 0; i < statements.length; i++) {
             const colon = statements[i].getColon();
@@ -75734,6 +75788,9 @@ class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
             const message = "Unnecessary chaining";
             const issue = issue_1.Issue.atToken(file, colon, message, this.getMetadata().key, this.conf.severity, fix);
             issues.push(issue);
+            if (issues.length >= max) {
+                break;
+            }
         }
         return issues;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.201",
+    "version": "2.113.203",
     "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.201",
+        "@abaplint/core": "^2.113.203",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",