@abaplint/cli 2.102.56 → 2.102.58

package/build/cli.js

@@ -295,6 +295,10 @@ class Formatter {
                 return new Formatters.CodeFrame().output(issues, fileCount);
             case "checkstyle":
                 return new Formatters.Checkstyle().output(issues, fileCount);
+            case "sonarqube":
+                return new Formatters.Sonarqube().output(issues, fileCount);
+            case "codeclimate":
+                return new Formatters.CodeClimate().output(issues, fileCount);
             default:
                 return new Formatters.Standard().output(issues, fileCount);
         }
@@ -359,6 +363,51 @@ exports.Checkstyle = Checkstyle;
 
 /***/ }),
 
+/***/ "./build/src/formatters/codeclimate.js":
+/*!*********************************************!*\
+  !*** ./build/src/formatters/codeclimate.js ***!
+  \*********************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.CodeClimate = void 0;
+const node_crypto_1 = __webpack_require__(/*! node:crypto */ "node:crypto");
+function md5(content) {
+    return (0, node_crypto_1.createHash)("md5").update(content).digest("hex");
+}
+class CodeClimate {
+    output(issues, _fileCount) {
+        const out = [];
+        const defaultSeverity = "info";
+        const severityArray = ["info", "minor", "major", "critical", "blocker"];
+        for (const issue of issues) {
+            const single = {
+                type: "issue",
+                check_name: issue.getKey(),
+                description: issue.getMessage(),
+                categories: ["Code Quality"],
+                location: {
+                    path: issue.getFilename(),
+                    lines: {
+                        begin: issue.getStart().getRow(),
+                        end: issue.getEnd().getRow(),
+                    },
+                },
+                severity: (severityArray.includes(issue.getSeverity().toLowerCase())) ? issue.getSeverity().toLowerCase() : defaultSeverity,
+                fingerprint: md5(issue.getKey() + issue.getMessage() + issue.getFilename() + issue.getStart().getRow() + issue.getEnd().getRow()),
+            };
+            out.push(single);
+        }
+        return JSON.stringify(out) + "\n";
+    }
+}
+exports.CodeClimate = CodeClimate;
+//# sourceMappingURL=codeclimate.js.map
+
+/***/ }),
+
 /***/ "./build/src/formatters/codeframe.js":
 /*!*******************************************!*\
   !*** ./build/src/formatters/codeframe.js ***!
@@ -472,6 +521,8 @@ __exportStar(__webpack_require__(/*! ./standard */ "./build/src/formatters/stand
 __exportStar(__webpack_require__(/*! ./total */ "./build/src/formatters/total.js"), exports);
 __exportStar(__webpack_require__(/*! ./codeframe */ "./build/src/formatters/codeframe.js"), exports);
 __exportStar(__webpack_require__(/*! ./checkstyle */ "./build/src/formatters/checkstyle.js"), exports);
+__exportStar(__webpack_require__(/*! ./sonarqube */ "./build/src/formatters/sonarqube.js"), exports);
+__exportStar(__webpack_require__(/*! ./codeclimate */ "./build/src/formatters/codeclimate.js"), exports);
 //# sourceMappingURL=index.js.map
 
 /***/ }),
@@ -615,6 +666,57 @@ exports.Junit = Junit;
 
 /***/ }),
 
+/***/ "./build/src/formatters/sonarqube.js":
+/*!*******************************************!*\
+  !*** ./build/src/formatters/sonarqube.js ***!
+  \*******************************************/
+/***/ ((__unused_webpack_module, exports) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.Sonarqube = void 0;
+class Sonarqube {
+    output(issues, _fileCount) {
+        const out = {};
+        const issueArray = [];
+        const defaultSeverity = "INFO";
+        const severityArray = ["INFO", "MINOR", "MAJOR", "CRITICAL", "BLOCKER"];
+        for (const issue of issues) {
+            const startOffset = issue.getStart().getCol() - 1;
+            const endOffset = issue.getEnd().getCol() - 2;
+            const single = {
+                engineId: "abaplint",
+                ruleId: issue.getKey(),
+                severity: (severityArray.includes(issue.getSeverity().toUpperCase())) ? issue.getSeverity().toUpperCase() : defaultSeverity,
+                type: "CODE_SMELL",
+                primaryLocation: {
+                    message: issue.getMessage(),
+                    filePath: issue.getFilename(),
+                    textRange: {
+                        startLine: issue.getStart().getRow(),
+                        endLine: issue.getEnd().getRow(),
+                        startColumn: startOffset,
+                        endColumn: endOffset,
+                    },
+                },
+                effortMinutes: 10,
+            };
+            if (startOffset >= endOffset) {
+                delete single.primaryLocation.textRange.startColumn;
+                delete single.primaryLocation.textRange.endColumn;
+            }
+            issueArray.push(single);
+        }
+        out.issues = issueArray;
+        return JSON.stringify(out) + "\n";
+    }
+}
+exports.Sonarqube = Sonarqube;
+//# sourceMappingURL=sonarqube.js.map
+
+/***/ }),
+
 /***/ "./build/src/formatters/standard.js":
 /*!******************************************!*\
   !*** ./build/src/formatters/standard.js ***!
@@ -49204,7 +49306,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.56";
+        return "2.102.58";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -65664,6 +65766,10 @@ const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modu
 const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
 const basic_1 = __webpack_require__(/*! ../abap/types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 class SelectAddOrderByConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        this.skipForAllEntries = false;
+    }
 }
 exports.SelectAddOrderByConf = SelectAddOrderByConf;
 class SelectAddOrderBy {
@@ -65695,7 +65801,7 @@ If the target is a sorted/hashed table, no issue is reported`,
         this.conf = conf;
     }
     run(obj) {
-        var _a;
+        var _a, _b;
         const issues = [];
         if (!(obj instanceof _abap_object_1.ABAPObject) || obj.getType() === "INTF") {
             return [];
@@ -65709,13 +65815,16 @@ If the target is a sorted/hashed table, no issue is reported`,
             const selects = stru.findAllStatements(Statements.Select);
             selects.push(...stru.findAllStatements(Statements.SelectLoop));
             for (const s of selects) {
-                const c = s.concatTokens().toUpperCase();
-                if (c.startsWith("SELECT SINGLE ")) {
+                const concat = s.concatTokens().toUpperCase();
+                if (concat.startsWith("SELECT SINGLE ")) {
+                    continue;
+                }
+                else if (((_a = this.getConfig()) === null || _a === void 0 ? void 0 : _a.skipForAllEntries) === true && concat.includes(" FOR ALL ENTRIES ")) {
                     continue;
                 }
                 // skip COUNT(*)
                 const list = s.findAllExpressions(Expressions.SQLField);
-                if (list.length === 1 && ((_a = list[0].getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.SQLAggregation) {
+                if (list.length === 1 && ((_b = list[0].getFirstChild()) === null || _b === void 0 ? void 0 : _b.get()) instanceof Expressions.SQLAggregation) {
                     continue;
                 }
                 else if (s.findFirstExpression(Expressions.SQLOrderBy)) {
@@ -81815,6 +81924,17 @@ module.exports = require("fs");
 
 /***/ }),
 
+/***/ "node:crypto":
+/*!******************************!*\
+  !*** external "node:crypto" ***!
+  \******************************/
+/***/ ((module) => {
+
+"use strict";
+module.exports = require("node:crypto");
+
+/***/ }),
+
 /***/ "node:os":
 /*!**************************!*\
   !*** external "node:os" ***!

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.56",
+    "version": "2.102.58",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,24 +38,24 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.56",
-        "@types/chai": "^4.3.6",
+        "@abaplint/core": "^2.102.58",
+        "@types/chai": "^4.3.8",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.3",
         "@types/mocha": "^10.0.2",
-        "@types/node": "^20.8.2",
+        "@types/node": "^20.8.6",
         "@types/progress": "^2.0.5",
         "chai": "^4.3.10",
         "chalk": "^5.3.0",
-        "eslint": "^8.50.0",
+        "eslint": "^8.51.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
-        "memfs": "^4.5.0",
+        "memfs": "^4.6.0",
         "minimist": "^1.2.8",
         "mocha": "^10.2.0",
         "progress": "^2.0.3",
         "typescript": "^5.2.2",
-        "webpack": "^5.88.2",
+        "webpack": "^5.89.0",
         "webpack-cli": "^5.1.4",
         "xml-js": "^1.6.11"
     },