@abaplint/cli 2.115.16 → 2.115.18

package/build/cli.js

@@ -30453,8 +30453,9 @@ const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 class Assign {
     runSyntax(node, input) {
-        var _a, _b;
+        var _a;
         const assignSource = node.findDirectExpression(Expressions.AssignSource);
+        const isComponent = ((_a = assignSource === null || assignSource === void 0 ? void 0 : assignSource.getFirstChild()) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase()) === "COMPONENT";
         const sources = (assignSource === null || assignSource === void 0 ? void 0 : assignSource.findDirectExpressionsMulti([Expressions.Source, Expressions.SimpleSource3])) || [];
         const theSource = sources[sources.length - 1];
         let sourceType = undefined;
@@ -30472,10 +30473,10 @@ class Assign {
             sourceType = basic_1.VoidType.get("Dynamic");
         }
         else {
-            sourceType = source_1.Source.runSyntax(theSource, input, undefined, false, true);
+            sourceType = source_1.Source.runSyntax(theSource, input, undefined, false, isComponent === false);
         }
         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().toUpperCase()) === "COMPONENT") {
+            && isComponent) {
             const componentSource = sources[sources.length - 2];
             const componentType = source_1.Source.runSyntax(componentSource, input);
             if (new _type_utils_1.TypeUtils(input.scope).isAssignable(componentType, new basic_1.CharacterType(30)) === false) {
@@ -30492,7 +30493,7 @@ class Assign {
         }
         const target = node.findDirectExpression(Expressions.FSTarget);
         if (target) {
-            if (((_b = assignSource === null || assignSource === void 0 ? void 0 : assignSource.getFirstChild()) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase()) === "COMPONENT") {
+            if (isComponent) {
                 fstarget_1.FSTarget.runSyntax(target, input, basic_1.AnyType.get());
             }
             else {
@@ -40580,7 +40581,7 @@ exports.InterfaceDefinition = InterfaceDefinition;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Message = void 0;
 class Message {
-    constructor(number, message) {
+    constructor(number, message, className) {
         this.number = number;
         if (this.number === undefined) {
             this.number = "";
@@ -40589,6 +40590,13 @@ class Message {
         if (this.message === undefined) {
             this.message = "";
         }
+        this.className = className;
+        if (this.className === undefined) {
+            this.className = "";
+        }
+    }
+    getMessageClass() {
+        return this.className;
     }
     getNumber() {
         return this.number;
@@ -51008,15 +51016,15 @@ const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@aba
 class MessageClass extends _abstract_object_1.AbstractObject {
     constructor() {
         super(...arguments);
-        this.parsedMessages = undefined;
+        this.xml = undefined;
     }
     getType() {
         return "MSAG";
     }
     getDescription() {
+        var _a;
         this.parseXML();
-        // todo
-        return undefined;
+        return (_a = this.xml) === null || _a === void 0 ? void 0 : _a.description;
     }
     getAllowedNaming() {
         return {
@@ -51024,13 +51032,18 @@ class MessageClass extends _abstract_object_1.AbstractObject {
             allowNamespace: true,
         };
     }
+    getParsed() {
+        this.parseXML();
+        return this.xml;
+    }
     setDirty() {
-        this.parsedMessages = undefined;
+        this.xml = undefined;
         super.setDirty();
     }
     getMessages() {
+        var _a;
         this.parseXML();
-        const msg = this.parsedMessages;
+        const msg = (_a = this.xml) === null || _a === void 0 ? void 0 : _a.parsedMessages;
         return msg ? msg : [];
     }
     getByNumber(num) {
@@ -51045,21 +51058,27 @@ class MessageClass extends _abstract_object_1.AbstractObject {
     }
     /////////////////////////////////
     parseXML() {
-        var _a, _b;
-        if (this.parsedMessages !== undefined) {
+        var _a, _b, _c, _d, _e, _f, _g, _h;
+        if (this.xml !== undefined) {
             return;
         }
-        this.parsedMessages = [];
+        this.xml = {
+            topName: undefined,
+            description: undefined,
+            parsedMessages: [],
+        };
         const parsed = super.parseRaw2();
         if (parsed === undefined) {
             return;
         }
-        const t100 = (_b = (_a = parsed === null || parsed === void 0 ? void 0 : parsed.abapGit) === null || _a === void 0 ? void 0 : _a["asx:abap"]["asx:values"]) === null || _b === void 0 ? void 0 : _b.T100;
+        this.xml.topName = (_c = (_b = (_a = parsed === null || parsed === void 0 ? void 0 : parsed.abapGit) === null || _a === void 0 ? void 0 : _a["asx:abap"]["asx:values"]) === null || _b === void 0 ? void 0 : _b.T100A) === null || _c === void 0 ? void 0 : _c.ARBGB;
+        this.xml.description = (_f = (_e = (_d = parsed === null || parsed === void 0 ? void 0 : parsed.abapGit) === null || _d === void 0 ? void 0 : _d["asx:abap"]["asx:values"]) === null || _e === void 0 ? void 0 : _e.T100A) === null || _f === void 0 ? void 0 : _f.STEXT;
+        const t100 = (_h = (_g = parsed === null || parsed === void 0 ? void 0 : parsed.abapGit) === null || _g === void 0 ? void 0 : _g["asx:abap"]["asx:values"]) === null || _h === void 0 ? void 0 : _h.T100;
         if (t100 === undefined) {
             return;
         }
         for (const msg of (0, xml_utils_1.xmlToArray)(t100.T100)) {
-            this.parsedMessages.push(new message_1.Message(msg.MSGNR, (0, xml_utils_1.unescape)(msg.TEXT)));
+            this.xml.parsedMessages.push(new message_1.Message(msg.MSGNR, (0, xml_utils_1.unescape)(msg.TEXT), msg.ARBGB));
         }
     }
 }
@@ -55438,7 +55457,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.115.16";
+        return "2.115.18";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -55652,7 +55671,7 @@ class Registry {
             return this;
         }
         ParsingPerformance.clear();
-        (_a = input === null || input === void 0 ? void 0 : input.progress) === null || _a === void 0 ? void 0 : _a.set(this.getObjectCount().normal, "Lexing and parsing");
+        (_a = input === null || input === void 0 ? void 0 : input.progress) === null || _a === void 0 ? void 0 : _a.set(this.getObjectCount().total, "Lexing and parsing");
         for (const o of this.getObjects()) {
             await ((_b = input === null || input === void 0 ? void 0 : input.progress) === null || _b === void 0 ? void 0 : _b.tick("Lexing and parsing(" + this.conf.getVersion() + ") - " + o.getType() + " " + o.getName()));
             this.parsePrivate(o);
@@ -69833,6 +69852,7 @@ class MSAGConsistency {
         return this;
     }
     run(obj) {
+        var _a, _b;
         const issues = [];
         if (!(obj instanceof objects_1.MessageClass)) {
             return [];
@@ -69881,6 +69901,18 @@ class MSAGConsistency {
                     }
                 }
             }
+            if (message.getMessageClass() !== obj.getName()) {
+                const text = `Message class "${message.getMessageClass()}" does not match filename "${obj.getName()}" for message ${message.getNumber()}`;
+                const position = new position_1.Position(1, 1);
+                const issue = issue_1.Issue.atPosition(obj.getFiles()[0], position, text, this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
+        }
+        if (((_a = obj.getParsed()) === null || _a === void 0 ? void 0 : _a.topName) !== obj.getName()) {
+            const text = `Message class name "${(_b = obj.getParsed()) === null || _b === void 0 ? void 0 : _b.topName}" does not match filename "${obj.getName()}"`;
+            const position = new position_1.Position(1, 1);
+            const issue = issue_1.Issue.atPosition(obj.getFiles()[0], position, text, this.getMetadata().key, this.conf.severity);
+            issues.push(issue);
         }
         return issues;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.115.16",
+    "version": "2.115.18",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,11 +38,11 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.115.16",
+        "@abaplint/core": "^2.115.18",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.10.4",
+        "@types/node": "^24.10.9",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",
@@ -50,7 +50,7 @@
         "eslint": "^9.39.2",
         "glob": "^11.0.3",
         "json5": "^2.2.3",
-        "memfs": "^4.51.1",
+        "memfs": "^4.56.10",
         "minimist": "^1.2.8",
         "mocha": "^11.7.5",
         "progress": "^2.0.3",