@abaplint/cli 2.120.58 → 2.120.60

package/build/cli.js

@@ -6355,7 +6355,9 @@ const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_st
 const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
 class MethodDefExceptions extends combi_1.Expression {
     getRunnable() {
-        const exceptions = (0, combi_1.seq)("EXCEPTIONS", (0, combi_1.plusPrio)(_1.NamespaceSimpleName));
+        // parameter section keywords cannot be exception names
+        const name = (0, combi_1.seq)((0, combi_1.stopBefore1)("IMPORTING", "EXPORTING", "CHANGING", "RETURNING", "RAISING", "EXCEPTIONS"), _1.NamespaceSimpleName);
+        const exceptions = (0, combi_1.seq)("EXCEPTIONS", (0, combi_1.plusPrio)(name));
         return exceptions;
     }
 }
@@ -6427,7 +6429,9 @@ const class_name_1 = __webpack_require__(/*! ./class_name */ "../core/build/src/
 class MethodDefRaising extends combi_1.Expression {
     getRunnable() {
         const resumable = (0, combi_1.seq)("RESUMABLE", (0, combi_1.tok)(tokens_1.ParenLeft), class_name_1.ClassName, (0, combi_1.tok)(tokens_1.ParenRightW));
-        const raising = (0, combi_1.seq)("RAISING", (0, combi_1.plus)((0, combi_1.altPrio)(resumable, class_name_1.ClassName)));
+        // parameter section keywords cannot be exception names
+        const name = (0, combi_1.seq)((0, combi_1.stopBefore1)("IMPORTING", "EXPORTING", "CHANGING", "RETURNING", "RAISING", "EXCEPTIONS"), class_name_1.ClassName);
+        const raising = (0, combi_1.seq)("RAISING", (0, combi_1.plus)((0, combi_1.altPrio)(resumable, name)));
         return raising;
     }
 }
@@ -16731,7 +16735,7 @@ class MethodDef {
         (0, combi_1.seq)("VALIDATE ON SAVE IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("MODIFY", (0, combi_1.opt)("IMPORTING"), (0, combi_1.plus)((0, combi_1.seq)(expressions_1.MethodParamName, modify))), (0, combi_1.seq)("PRECHECK IMPORTING", expressions_1.MethodParamName, modify), (0, combi_1.seq)("DETERMINATION", expressions_1.TypeName, "IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("VALIDATION", expressions_1.TypeName, "IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("NUMBERING IMPORTING", expressions_1.MethodParamName, modify), (0, combi_1.seq)("READ IMPORTING", expressions_1.MethodParamName, (0, combi_1.altPrio)(forRead, forfunction)), (0, combi_1.seq)("FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.NamespaceSimpleName, "FOR", expressions_1.NamespaceSimpleName, result), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR CREATE", expressions_1.TypeName, expressions_1.MethodParamName, "FOR UPDATE", expressions_1.TypeName, expressions_1.MethodParamName, "FOR DELETE", expressions_1.TypeName), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR READ", expressions_1.TypeName, result), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR UPDATE", expressions_1.TypeName), (0, combi_1.seq)((0, combi_1.alt)("BEHAVIOR", "LOCK"), "IMPORTING", expressions_1.MethodParamName, "FOR LOCK", expressions_1.TypeName), (0, combi_1.seq)("DETERMINE", (0, combi_1.alt)("ON MODIFY", "ON SAVE"), "IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("GLOBAL AUTHORIZATION IMPORTING REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)("GLOBAL FEATURES IMPORTING REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)((0, combi_1.seq)((0, combi_1.opt)("INSTANCE"), "AUTHORIZATION IMPORTING"), expressions_1.MethodParamName, "REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)("INSTANCE FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result));
         // todo, this is only from version something
         // AMDP OPTIONS clause blocked in KeyUser
-        const amdp = (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("AMDP OPTIONS", (0, combi_1.optPrio)("READ-ONLY"), "CDS SESSION CLIENT", (0, combi_1.alt)("CURRENT", "DEPENDENT"), (0, combi_1.optPrio)(expressions_1.MethodDefImporting), (0, combi_1.optPrio)(expressions_1.MethodDefExporting), (0, combi_1.optPrio)(expressions_1.MethodDefRaising)));
+        const amdp = (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("AMDP OPTIONS", (0, combi_1.per)("READ-ONLY", (0, combi_1.seq)("CDS SESSION CLIENT", (0, combi_1.alt)("CURRENT", "DEPENDENT"))), (0, combi_1.optPrio)(expressions_1.MethodDefImporting), (0, combi_1.optPrio)(expressions_1.MethodDefExporting), (0, combi_1.optPrio)(expressions_1.MethodDefRaising)));
         // FOR DDL OBJECT / FOR TABLE FUNCTION / FOR SCALAR FUNCTION blocked in KeyUser
         const regular = (0, combi_1.seq)((0, combi_1.altPrio)("CLASS-METHODS", "METHODS"), expressions_1.MethodName, (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.optPrio)(expressions_1.Abstract), (0, combi_1.optPrio)(def), expressions_1.EventHandler), parameters, testing, (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("FOR", behavior)), amdp, "NOT AT END OF MODE", (0, combi_1.optPrio)(expressions_1.Redefinition)));
         // RAP event handlers are instance methods; CLASS-METHODS is not allowed.
@@ -16762,7 +16766,8 @@ class MethodImplementation {
         const name = (0, combi_1.regex)(/[\w~]+/);
         const kernel = (0, combi_1.seq)("KERNEL MODULE", (0, combi_1.plus)(name), (0, combi_1.optPrio)((0, combi_1.altPrio)("FAIL", "IGNORE")));
         const using = (0, combi_1.seq)("USING", (0, combi_1.plus)(expressions_1.SimpleFieldChain));
-        const database = (0, combi_1.seq)("DATABASE", (0, combi_1.alt)("PROCEDURE", "FUNCTION", "GRAPH WORKSPACE"), "FOR HDB", expressions_1.Language, (0, combi_1.opt)("OPTIONS READ-ONLY"), (0, combi_1.opt)(using));
+        const options = (0, combi_1.seq)("OPTIONS", (0, combi_1.per)("READ-ONLY", "SUPPRESS SYNTAX ERRORS", "DETERMINISTIC"));
+        const database = (0, combi_1.seq)("DATABASE", (0, combi_1.alt)("PROCEDURE", "FUNCTION", "GRAPH WORKSPACE"), "FOR HDB", expressions_1.Language, (0, combi_1.opt)(options), (0, combi_1.opt)(using));
         // BY DATABASE and BY KERNEL both blocked in KeyUser
         const by = (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("BY", (0, combi_1.alt)(kernel, database)));
         return (0, combi_1.seq)("METHOD", expressions_1.MethodName, (0, combi_1.optPrio)(by));
@@ -34781,6 +34786,11 @@ class Source {
                             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                             return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                         }
+                        if ((foundType === null || foundType === void 0 ? void 0 : foundType.isGeneric()) && bodyType !== undefined) {
+                            this.addIfInferred(node, input, bodyType);
+                            this.traverseRemainingChildren(children, input);
+                            return bodyType;
+                        }
                         this.addIfInferred(node, input, foundType);
                         this.traverseRemainingChildren(children, input);
                         return foundType;
@@ -39448,10 +39458,22 @@ const basic_types_1 = __webpack_require__(/*! ../basic_types */ "../core/build/s
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js");
 const _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifier */ "../core/build/src/abap/types/_typed_identifier.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "../core/build/src/abap/5_syntax/_reference.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 class CreateData {
     runSyntax(node, input) {
+        const likeLineOf = node.concatTokens().toUpperCase().includes(" LIKE LINE OF ");
         for (const s of node.findDirectExpressions(Expressions.Source)) {
-            source_1.Source.runSyntax(s, input);
+            const sourceType = source_1.Source.runSyntax(s, input);
+            if (likeLineOf === true
+                && sourceType !== undefined
+                && !(sourceType instanceof basic_1.TableType)
+                && !(sourceType instanceof basic_1.VoidType)
+                && !(sourceType instanceof basic_1.UnknownType)
+                && !(sourceType instanceof basic_1.AnyType)) {
+                const message = `"${s.concatTokens()}" is not an internal table`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                return;
+            }
         }
         for (const t of node.findDirectExpressions(Expressions.Target)) {
             target_1.Target.runSyntax(t, input);
@@ -39636,6 +39658,11 @@ class CreateObject {
             ooName = found.getVoided();
         }
         input.scope.addReference(t === null || t === void 0 ? void 0 : t.getFirstToken(), cdef, _reference_1.ReferenceType.ConstructorReference, input.filename, { ooName: ooName });
+        if (type && type.get() instanceof Expressions.Dynamic) {
+            // the class is only known at runtime, cdef is the static type of the target
+            this.validateParameters(undefined, node, input);
+            return;
+        }
         if (cdef !== undefined) {
             const err = CreateObject.checkInstantiationAllowed(cdef, input);
             if (err) {
@@ -70685,7 +70712,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.58";
+        return "2.120.60";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -71318,12 +71345,13 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#abap-doc-on
         return issues;
     }
     getAbapdoc(rows, pos) {
+        var _a, _b, _c;
         let previousRow = pos.getRow() - 2;
-        let rowText = rows[previousRow].trim().toUpperCase();
+        let rowText = ((_a = rows[previousRow]) !== null && _a !== void 0 ? _a : "").trim().toUpperCase();
         const text = [];
         if (rowText === "METHODS" || rowText === "CLASS-METHODS") {
             previousRow--;
-            rowText = rows[previousRow].trim().toUpperCase();
+            rowText = ((_b = rows[previousRow]) !== null && _b !== void 0 ? _b : "").trim().toUpperCase();
         }
         text.push(rowText);
         //we need to push the first row despite if it is actually an abapdoc or not
@@ -71331,7 +71359,7 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#abap-doc-on
         if (rowText.trim().substring(0, 2) === "\"!") {
             while (previousRow >= 0) {
                 previousRow--;
-                rowText = rows[previousRow].trim().toUpperCase();
+                rowText = ((_c = rows[previousRow]) !== null && _c !== void 0 ? _c : "").trim().toUpperCase();
                 if (rowText.trim().substring(0, 2) !== "\"!") {
                     break;
                 }
@@ -85123,6 +85151,7 @@ __exportStar(__webpack_require__(/*! ./types_naming */ "../core/build/src/rules/
 __exportStar(__webpack_require__(/*! ./uncaught_exception */ "../core/build/src/rules/uncaught_exception.js"), exports);
 __exportStar(__webpack_require__(/*! ./unknown_types */ "../core/build/src/rules/unknown_types.js"), exports);
 __exportStar(__webpack_require__(/*! ./unnecessary_chaining */ "../core/build/src/rules/unnecessary_chaining.js"), exports);
+__exportStar(__webpack_require__(/*! ./unnecessary_loop_guard */ "../core/build/src/rules/unnecessary_loop_guard.js"), exports);
 __exportStar(__webpack_require__(/*! ./unnecessary_pragma */ "../core/build/src/rules/unnecessary_pragma.js"), exports);
 __exportStar(__webpack_require__(/*! ./unnecessary_return */ "../core/build/src/rules/unnecessary_return.js"), exports);
 __exportStar(__webpack_require__(/*! ./unreachable_code */ "../core/build/src/rules/unreachable_code.js"), exports);
@@ -85135,6 +85164,7 @@ __exportStar(__webpack_require__(/*! ./unused_variables */ "../core/build/src/ru
 __exportStar(__webpack_require__(/*! ./use_bool_expression */ "../core/build/src/rules/use_bool_expression.js"), exports);
 __exportStar(__webpack_require__(/*! ./use_class_based_exceptions */ "../core/build/src/rules/use_class_based_exceptions.js"), exports);
 __exportStar(__webpack_require__(/*! ./use_line_exists */ "../core/build/src/rules/use_line_exists.js"), exports);
+__exportStar(__webpack_require__(/*! ./use_message_class */ "../core/build/src/rules/use_message_class.js"), exports);
 __exportStar(__webpack_require__(/*! ./use_new */ "../core/build/src/rules/use_new.js"), exports);
 __exportStar(__webpack_require__(/*! ./when_others_last */ "../core/build/src/rules/when_others_last.js"), exports);
 __exportStar(__webpack_require__(/*! ./whitespace_end */ "../core/build/src/rules/whitespace_end.js"), exports);
@@ -98871,6 +98901,151 @@ exports.UnnecessaryChaining = UnnecessaryChaining;
 
 /***/ },
 
+/***/ "../core/build/src/rules/unnecessary_loop_guard.js"
+/*!*********************************************************!*\
+  !*** ../core/build/src/rules/unnecessary_loop_guard.js ***!
+  \*********************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.UnnecessaryLoopGuard = exports.UnnecessaryLoopGuardConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "../core/build/src/issue.js");
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "../core/build/src/abap/2_statements/statements/index.js"));
+const Structures = __importStar(__webpack_require__(/*! ../abap/3_structures/structures */ "../core/build/src/abap/3_structures/structures/index.js"));
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "../core/build/src/rules/_abap_rule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "../core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "../core/build/src/rules/_irule.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "../core/build/src/edit_helper.js");
+class UnnecessaryLoopGuardConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.UnnecessaryLoopGuardConf = UnnecessaryLoopGuardConf;
+class UnnecessaryLoopGuard extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new UnnecessaryLoopGuardConf();
+    }
+    getMetadata() {
+        return {
+            key: "unnecessary_loop_guard",
+            title: "Unnecessary loop guard",
+            shortDescription: `Detects unnecessary IS [NOT] INITIAL check before LOOP AT`,
+            extendedInformation: `LOOP AT iterates zero times on an empty table, so an IS [NOT] INITIAL guard is redundant.`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
+            badExample: `IF lt_data IS NOT INITIAL.
+  LOOP AT lt_data INTO DATA(ls_item).
+    WRITE ls_item-name.
+  ENDLOOP.
+ENDIF.`,
+            goodExample: `LOOP AT lt_data INTO DATA(ls_item).
+  WRITE ls_item-name.
+ENDLOOP.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        const stru = file.getStructure();
+        if (stru === undefined) {
+            return issues;
+        }
+        for (const ifStru of stru.findAllStructuresRecursive(Structures.If)) {
+            if (ifStru.findDirectStructures(Structures.ElseIf).length > 0
+                || ifStru.findDirectStructures(Structures.Else).length > 0) {
+                continue;
+            }
+            const ifStatement = ifStru.findDirectStatement(Statements.If);
+            if (ifStatement === undefined) {
+                continue;
+            }
+            const cond = ifStatement.findDirectExpression(Expressions.Cond);
+            if (cond === undefined) {
+                continue;
+            }
+            const compareNodes = cond.findDirectExpressions(Expressions.Compare);
+            if (compareNodes.length !== 1) {
+                continue;
+            }
+            const m = compareNodes[0].concatTokens().toUpperCase().match(/^(.+?)\s+IS\s+(?:NOT\s+)?INITIAL\s*$/);
+            if (!m) {
+                continue;
+            }
+            const tableName = m[1];
+            const bodyStructures = ifStru.findDirectStructures(Structures.Body);
+            if (bodyStructures.length !== 1) {
+                continue;
+            }
+            const normalStructures = bodyStructures[0].findDirectStructures(Structures.Normal);
+            if (normalStructures.length !== 1) {
+                continue;
+            }
+            const loopStructures = normalStructures[0].findDirectStructures(Structures.Loop);
+            if (loopStructures.length !== 1) {
+                continue;
+            }
+            const loopStatement = loopStructures[0].findDirectStatement(Statements.Loop);
+            if (loopStatement === undefined) {
+                continue;
+            }
+            const loopSource = loopStatement.findFirstExpression(Expressions.LoopSource);
+            if (loopSource === undefined) {
+                continue;
+            }
+            if (loopSource.concatTokens().toUpperCase() !== tableName) {
+                continue;
+            }
+            const endifStatement = ifStru.findDirectStatement(Statements.EndIf);
+            const fix = edit_helper_1.EditHelper.merge(edit_helper_1.EditHelper.deleteStatement(file, ifStatement), edit_helper_1.EditHelper.deleteStatement(file, endifStatement));
+            issues.push(issue_1.Issue.atStatement(file, ifStatement, "Unnecessary IS [NOT] INITIAL check before LOOP AT", this.getMetadata().key, this.conf.severity, fix));
+        }
+        return issues;
+    }
+}
+exports.UnnecessaryLoopGuard = UnnecessaryLoopGuard;
+//# sourceMappingURL=unnecessary_loop_guard.js.map
+
+/***/ },
+
 /***/ "../core/build/src/rules/unnecessary_pragma.js"
 /*!*****************************************************!*\
   !*** ../core/build/src/rules/unnecessary_pragma.js ***!
@@ -100945,6 +101120,103 @@ exports.UseLineExists = UseLineExists;
 
 /***/ },
 
+/***/ "../core/build/src/rules/use_message_class.js"
+/*!****************************************************!*\
+  !*** ../core/build/src/rules/use_message_class.js ***!
+  \****************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.UseMessageClass = exports.UseMessageClassConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "../core/build/src/issue.js");
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "../core/build/src/rules/_abap_rule.js");
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "../core/build/src/abap/2_statements/statements/index.js"));
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "../core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "../core/build/src/rules/_irule.js");
+class UseMessageClassConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.UseMessageClassConf = UseMessageClassConf;
+class UseMessageClass extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new UseMessageClassConf();
+    }
+    getMetadata() {
+        return {
+            key: "use_message_class",
+            title: "Use message class",
+            shortDescription: `Define messages in SE91 instead of using inline text or variables`,
+            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `MESSAGE 'Something went wrong' TYPE 'E'.`,
+            goodExample: `MESSAGE e001(bc_msg).`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, _obj) {
+        const structure = file.getStructure();
+        if (structure === undefined) {
+            return [];
+        }
+        const issues = [];
+        const key = this.getMetadata().key;
+        for (const statement of structure.findAllStatements(Statements.Message)) {
+            // The parser populates MessageSource only for the class-based forms (e001(bc_msg) or ID/TYPE/NUMBER).
+            // The text form (MESSAGE <source> TYPE <source>) uses MessageSourceSource directly with no MessageSource child.
+            // Absence of MessageSource is therefore a reliable discriminator between the two forms.
+            if (statement.findDirectExpression(Expressions.MessageSource) !== undefined) {
+                continue;
+            }
+            issues.push(issue_1.Issue.atStatement(file, statement, "Use a message class (SE91) instead of inline text or variable", key, this.conf.severity));
+        }
+        return issues;
+    }
+}
+exports.UseMessageClass = UseMessageClass;
+//# sourceMappingURL=use_message_class.js.map
+
+/***/ },
+
 /***/ "../core/build/src/rules/use_new.js"
 /*!******************************************!*\
   !*** ../core/build/src/rules/use_new.js ***!

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.120.58",
+    "version": "2.120.60",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -39,7 +39,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.120.58",
+        "@abaplint/core": "^2.120.60",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",