@abaplint/cli 2.102.55 → 2.102.57

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 ***!
@@ -16498,9 +16600,10 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.TruncateDataset = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
 const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 class TruncateDataset {
     getMatcher() {
-        return (0, combi_1.seq)("TRUNCATE DATASET", expressions_1.Source, "AT CURRENT POSITION");
+        return (0, combi_1.verNot)(version_1.Version.Cloud, (0, combi_1.seq)("TRUNCATE DATASET", expressions_1.Source, "AT CURRENT POSITION"));
     }
 }
 exports.TruncateDataset = TruncateDataset;
@@ -21873,6 +21976,7 @@ class TypeUtils {
         else if (source instanceof basic_1.XStringType) {
             if (target instanceof basic_1.CLikeType
                 || target instanceof basic_1.IntegerType
+                || target instanceof basic_1.StringType
                 || target instanceof basic_1.ObjectReferenceType
                 || target instanceof basic_1.HexType) {
                 return false;
@@ -24672,6 +24776,7 @@ class MethodCallChain {
         if (first.get() instanceof Expressions.MethodCall) {
             children.unshift(first);
         }
+        let previous = "";
         while (children.length > 0) {
             const current = children.shift();
             if (current === undefined) {
@@ -24692,6 +24797,9 @@ class MethodCallChain {
                     }
                 }
                 else {
+                    if (previous === "=>" && (method === null || method === void 0 ? void 0 : method.isStatic()) === false) {
+                        throw new Error("Method \"" + methodName + "\" not static");
+                    }
                     const extra = {
                         ooName: foundDef === null || foundDef === void 0 ? void 0 : foundDef.getName(),
                         ooType: foundDef instanceof class_definition_1.ClassDefinition ? "CLAS" : "INTF"
@@ -24729,6 +24837,7 @@ class MethodCallChain {
             else if (current instanceof nodes_1.ExpressionNode && current.get() instanceof Expressions.AttributeName) {
                 context = new attribute_name_1.AttributeName().runSyntax(context, current, scope, filename);
             }
+            previous = current.concatTokens();
         }
         return context;
     }
@@ -26389,7 +26498,8 @@ class StringTemplate {
             if (type === undefined) {
                 throw new Error("No target type determined");
             }
-            else if (typeUtils.isCharLike(type) === false && typeUtils.isHexLike(type) === false) {
+            else if ((typeUtils.isCharLike(type) === false && typeUtils.isHexLike(type) === false)
+                || type instanceof basic_1.StructureType) {
                 throw new Error("Not character like, " + type.constructor.name);
             }
             const format = templateSource.findDirectExpression(Expressions.StringTemplateFormatting);
@@ -27436,9 +27546,10 @@ const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_module
 const fstarget_1 = __webpack_require__(/*! ../expressions/fstarget */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/fstarget.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js");
+const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
 class Assign {
     runSyntax(node, scope, filename) {
-        var _a, _b;
+        var _a;
         const assignSource = node.findDirectExpression(Expressions.AssignSource);
         const sources = (assignSource === null || assignSource === void 0 ? void 0 : assignSource.findDirectExpressions(Expressions.Source)) || [];
         const theSource = sources[sources.length - 1];
@@ -27457,10 +27568,18 @@ class Assign {
         else {
             sourceType = new source_1.Source().runSyntax(theSource, scope, filename);
         }
-        if (sourceType === undefined || ((_a = node.findDirectExpression(Expressions.AssignSource)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(Expressions.Dynamic))) {
+        if ((assignSource === null || assignSource === void 0 ? void 0 : assignSource.getChildren().length) === 5
+            && ((_a = assignSource === null || assignSource === void 0 ? void 0 : assignSource.getFirstChild()) === null || _a === void 0 ? void 0 : _a.concatTokens()) === "COMPONENT") {
+            const componentSource = sources[sources.length - 2];
+            const componentType = new source_1.Source().runSyntax(componentSource, scope, filename);
+            if (new _type_utils_1.TypeUtils(scope).isAssignable(componentType, new basic_1.CharacterType(30)) === false) {
+                throw new Error("component name must be charlike");
+            }
+        }
+        if (sourceType === undefined || (assignSource === null || assignSource === void 0 ? void 0 : assignSource.findDirectExpression(Expressions.Dynamic))) {
             sourceType = new basic_1.AnyType();
         }
-        for (const d of ((_b = node.findDirectExpression(Expressions.AssignSource)) === null || _b === void 0 ? void 0 : _b.findAllExpressions(Expressions.Dynamic)) || []) {
+        for (const d of (assignSource === null || assignSource === void 0 ? void 0 : assignSource.findAllExpressions(Expressions.Dynamic)) || []) {
             new dynamic_1.Dynamic().runSyntax(d, scope, filename);
         }
         const target = node.findDirectExpression(Expressions.FSTarget);
@@ -49187,7 +49306,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.55";
+        return "2.102.57";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -81798,6 +81917,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.55",
+    "version": "2.102.57",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,19 +38,19 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.55",
-        "@types/chai": "^4.3.6",
+        "@abaplint/core": "^2.102.57",
+        "@types/chai": "^4.3.7",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.3",
         "@types/mocha": "^10.0.2",
-        "@types/node": "^20.8.2",
+        "@types/node": "^20.8.4",
         "@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",