@abaplint/cli 2.119.49 → 2.119.51

package/build/cli.js

@@ -3078,6 +3078,69 @@ exports.ExpandMacros = ExpandMacros;
 
 /***/ },
 
+/***/ "../core/build/src/abap/2_statements/expressions/_derived_types.js"
+/*!*************************************************************************!*\
+  !*** ../core/build/src/abap/2_statements/expressions/_derived_types.js ***!
+  \*************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.commonDerivedTypes = commonDerivedTypes;
+exports.derivedTypesAlt = derivedTypesAlt;
+const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
+const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
+function commonDerivedTypes() {
+    const entity = (0, combi_1.alt)(_1.TypeName, _1.EntityAssociation);
+    // more-specific keyword phrases (e.g. "FAILED LATE") must precede less-specific ones ("FAILED")
+    return [
+        (0, combi_1.ver)(version_1.Release.v774, (0, combi_1.seq)("FAILED LATE", entity)),
+        (0, combi_1.ver)(version_1.Release.v774, (0, combi_1.seq)("MAPPED LATE", entity)),
+        (0, combi_1.ver)(version_1.Release.v774, (0, combi_1.seq)("REPORTED LATE", entity)),
+        (0, combi_1.ver)(version_1.Release.v777, (0, combi_1.seq)("FAILED EARLY", entity)),
+        (0, combi_1.ver)(version_1.Release.v777, (0, combi_1.seq)("MAPPED EARLY", entity)),
+        (0, combi_1.ver)(version_1.Release.v777, (0, combi_1.seq)("REPORTED EARLY", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("FAILED", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("MAPPED", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("REPORTED", entity)),
+        (0, combi_1.ver)(version_1.Release.v773, (0, combi_1.seq)("FUNCTION IMPORT", entity)),
+        (0, combi_1.ver)(version_1.Release.v773, (0, combi_1.seq)("FUNCTION RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v773, (0, combi_1.seq)("ACTION RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE AUTHORIZATION KEY", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE AUTHORIZATION REQUEST", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE AUTHORIZATION RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE FEATURES KEY", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE FEATURES REQUEST", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE FEATURES RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v780, (0, combi_1.seq)("AUTHORIZATION RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("AUTHORIZATION KEY", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("FEATURES RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("FEATURES KEY", entity)),
+        (0, combi_1.ver)(version_1.Release.v780, (0, combi_1.seq)("PERMISSIONS KEY", entity)),
+        (0, combi_1.seq)("READ IMPORT", entity),
+        (0, combi_1.seq)("READ RESULT", entity),
+        (0, combi_1.ver)(version_1.Release.v915, (0, combi_1.seq)("READ CHANGES", entity)),
+        (0, combi_1.seq)("CREATE", entity),
+        (0, combi_1.seq)("DELETE", entity),
+        (0, combi_1.seq)("UPDATE", entity),
+        (0, combi_1.ver)(version_1.Release.v777, (0, combi_1.seq)("LOCK", entity)),
+        (0, combi_1.ver)(version_1.Release.v775, (0, combi_1.seq)("KEY OF", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("DETERMINATION", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("VALIDATION", entity)),
+        (0, combi_1.ver)(version_1.Release.v778, (0, combi_1.seq)("CHANGE", entity)),
+    ];
+}
+function derivedTypesAlt(...extra) {
+    const all = [...extra, ...commonDerivedTypes()];
+    const [first, second, ...rest] = all;
+    return (0, combi_1.altPrio)(first, second, ...rest);
+}
+//# sourceMappingURL=_derived_types.js.map
+
+/***/ },
+
 /***/ "../core/build/src/abap/2_statements/expressions/_dynamic_access.js"
 /*!**************************************************************************!*\
   !*** ../core/build/src/abap/2_statements/expressions/_dynamic_access.js ***!
@@ -10653,24 +10716,16 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.TypeStructure = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
 const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
+const _derived_types_1 = __webpack_require__(/*! ./_derived_types */ "../core/build/src/abap/2_statements/expressions/_derived_types.js");
 class TypeStructure extends combi_1.Expression {
     getRunnable() {
-        // todo, add version,
-        const hier = (0, combi_1.seq)("HIERARCHY", _1.NamespaceSimpleName);
-        const create = (0, combi_1.seq)("CREATE", (0, combi_1.alt)(_1.NamespaceSimpleName, _1.EntityAssociation));
-        const update = (0, combi_1.seq)("UPDATE", (0, combi_1.alt)(_1.NamespaceSimpleName, _1.EntityAssociation));
-        const readResult = (0, combi_1.seq)("READ RESULT", (0, combi_1.alt)(_1.NamespaceSimpleName, _1.EntityAssociation));
-        const readLink = (0, combi_1.seq)("READ LINK", _1.EntityAssociation);
-        const action = (0, combi_1.seq)("ACTION IMPORT", _1.Source);
-        const permissionsRequest = (0, combi_1.seq)("PERMISSIONS REQUEST", _1.NamespaceSimpleName);
-        const evt = (0, combi_1.seq)("EVENT", _1.EventName);
-        const failedEarly = (0, combi_1.seq)("FAILED EARLY", _1.NamespaceSimpleName);
-        const mappedEarly = (0, combi_1.seq)("MAPPED EARLY", _1.NamespaceSimpleName);
-        const reportedEarly = (0, combi_1.seq)("REPORTED EARLY", _1.NamespaceSimpleName);
-        const structure = (0, combi_1.seq)("STRUCTURE FOR", (0, combi_1.altPrio)(hier, evt, create, update, action, permissionsRequest, readLink, readResult));
-        const response = (0, combi_1.seq)("RESPONSE FOR", (0, combi_1.altPrio)(failedEarly, mappedEarly, reportedEarly));
-        const request = (0, combi_1.seq)("REQUEST FOR CHANGE", _1.NamespaceSimpleName);
-        return (0, combi_1.seq)("TYPE", (0, combi_1.altPrio)(structure, response, request));
+        const entity = (0, combi_1.alt)(_1.TypeName, _1.EntityAssociation);
+        const derivedTypes = (0, _derived_types_1.derivedTypesAlt)((0, combi_1.ver)(version_1.Release.v779, (0, combi_1.seq)("FUNCTION REQUEST", entity)), (0, combi_1.ver)(version_1.Release.v779, (0, combi_1.seq)("ACTION REQUEST", entity)), (0, combi_1.seq)("ACTION IMPORT", _1.Source), (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("GLOBAL AUTHORIZATION REQUEST", entity)), (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("GLOBAL AUTHORIZATION RESULT", entity)), (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("GLOBAL FEATURES REQUEST", entity)), (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("GLOBAL FEATURES RESULT", entity)), (0, combi_1.ver)(version_1.Release.v780, (0, combi_1.seq)("AUTHORIZATION REQUEST", entity)), (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("FEATURES REQUEST", entity)), (0, combi_1.seq)("PERMISSIONS REQUEST", _1.NamespaceSimpleName), (0, combi_1.ver)(version_1.Release.v780, (0, combi_1.seq)("PERMISSIONS RESULT", entity)), (0, combi_1.seq)("READ LINK", _1.EntityAssociation), (0, combi_1.seq)("HIERARCHY", _1.NamespaceSimpleName), (0, combi_1.seq)("EVENT", _1.EventName));
+        const structure = (0, combi_1.ver)(version_1.Release.v774, (0, combi_1.seq)("STRUCTURE FOR", derivedTypes));
+        const response = (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("RESPONSE FOR", (0, combi_1.altPrio)((0, combi_1.seq)("FAILED EARLY", _1.NamespaceSimpleName), (0, combi_1.seq)("FAILED LATE", _1.NamespaceSimpleName), (0, combi_1.seq)("FAILED", _1.NamespaceSimpleName), (0, combi_1.seq)("MAPPED EARLY", _1.NamespaceSimpleName), (0, combi_1.seq)("MAPPED LATE", _1.NamespaceSimpleName), (0, combi_1.seq)("MAPPED", _1.NamespaceSimpleName), (0, combi_1.seq)("REPORTED EARLY", _1.NamespaceSimpleName), (0, combi_1.seq)("REPORTED LATE", _1.NamespaceSimpleName), (0, combi_1.seq)("REPORTED", _1.NamespaceSimpleName))));
+        const request = (0, combi_1.ver)(version_1.Release.v778, (0, combi_1.seq)("REQUEST FOR", (0, combi_1.alt)("CHANGE", "DELETE"), _1.NamespaceSimpleName));
+        return (0, combi_1.seq)("TYPE", (0, combi_1.altPrio)(structure, response, request), (0, combi_1.optPrio)("VALUE IS INITIAL"));
     }
 }
 exports.TypeStructure = TypeStructure;
@@ -10693,6 +10748,7 @@ const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/exp
 const field_chain_1 = __webpack_require__(/*! ./field_chain */ "../core/build/src/abap/2_statements/expressions/field_chain.js");
 const type_table_key_1 = __webpack_require__(/*! ./type_table_key */ "../core/build/src/abap/2_statements/expressions/type_table_key.js");
 const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
+const _derived_types_1 = __webpack_require__(/*! ./_derived_types */ "../core/build/src/abap/2_statements/expressions/_derived_types.js");
 class TypeTable extends combi_1.Expression {
     getRunnable() {
         const header = "WITH HEADER LINE";
@@ -10706,7 +10762,9 @@ class TypeTable extends combi_1.Expression {
         // "WITH" is not allowed as a field name in keys
         const typetable = (0, combi_1.alt)(generic, (0, combi_1.seq)(normal1, (0, combi_1.alt)((0, combi_1.opt)((0, combi_1.per)(header, initial, (0, combi_1.plusPrio)(type_table_key_1.TypeTableKey))), (0, combi_1.seq)((0, combi_1.plus)(type_table_key_1.TypeTableKey), (0, combi_1.optPrio)(initial))), (0, combi_1.optPrio)("VALUE IS INITIAL")));
         const occurs = (0, combi_1.seq)("OCCURS", (0, combi_1.altPrio)(_1.Integer, field_chain_1.FieldChain));
-        const derived = (0, combi_1.ver)(version_1.Release.v754, (0, combi_1.seq)("TABLE FOR", (0, combi_1.altPrio)("ACTION IMPORT", "ACTION RESULT", "CREATE", "EVENT", "REPORTED EARLY", "READ IMPORT", "FAILED EARLY", "FAILED", "LOCK", "DETERMINATION", "READ RESULT", "UPDATE", "DELETE"), (0, combi_1.alt)(_1.TypeName, _1.EntityAssociation)), { also: combi_1.AlsoIn.OpenABAP });
+        const entity = (0, combi_1.alt)(_1.TypeName, _1.EntityAssociation);
+        const derivedTypes = (0, _derived_types_1.derivedTypesAlt)((0, combi_1.ver)(version_1.Release.v773, (0, combi_1.seq)("ACTION IMPORT", entity)), (0, combi_1.seq)("READ LINK", entity), (0, combi_1.seq)("EVENT", entity));
+        const derived = (0, combi_1.ver)(version_1.Release.v770, (0, combi_1.seq)("TABLE FOR", derivedTypes, (0, combi_1.optPrio)("VALUE IS INITIAL")), { also: combi_1.AlsoIn.OpenABAP });
         const oldType = (0, combi_1.seq)((0, combi_1.opt)("REF TO"), _1.TypeName, (0, combi_1.alt)((0, combi_1.seq)(occurs, (0, combi_1.opt)(header)), header));
         const oldLike = (0, combi_1.seq)((0, combi_1.opt)("REF TO"), field_chain_1.FieldChain, (0, combi_1.alt)((0, combi_1.seq)(occurs, (0, combi_1.opt)(header)), header));
         const typeLine = (0, combi_1.seq)("LINE OF", _1.TypeName, occurs, header);
@@ -28374,25 +28432,12 @@ class BasicTypes {
         return name.includes("\\") || name.includes("~");
     }
     isRAPTableFor(text) {
-        return text.startsWith("TYPE TABLE FOR ACTION IMPORT ")
-            || text.startsWith("TYPE TABLE FOR ACTION RESULT ")
-            || text.startsWith("TYPE TABLE FOR CREATE ")
-            || text.startsWith("TYPE TABLE FOR DELETE ")
-            || text.startsWith("TYPE TABLE FOR DETERMINATION ")
-            || text.startsWith("TYPE TABLE FOR EVENT ")
-            || text.startsWith("TYPE TABLE FOR FAILED ")
-            || text.startsWith("TYPE TABLE FOR FAILED EARLY ")
-            || text.startsWith("TYPE TABLE FOR LOCK ")
-            || text.startsWith("TYPE TABLE FOR READ ")
-            || text.startsWith("TYPE TABLE FOR READ IMPORT ")
-            || text.startsWith("TYPE TABLE FOR READ RESULT ")
-            || text.startsWith("TYPE TABLE FOR REPORTED EARLY ")
-            || text.startsWith("TYPE TABLE FOR UPDATE ");
+        return text.startsWith("TYPE TABLE FOR ");
     }
     isRAPTypeStructure(text) {
         return text.startsWith("TYPE STRUCTURE FOR ")
             || text.startsWith("TYPE RESPONSE FOR ")
-            || text.startsWith("TYPE REQUEST FOR CHANGE ");
+            || text.startsWith("TYPE REQUEST FOR ");
     }
     // todo, rewrite this method
     resolveTypeChain(expr) {
@@ -67943,7 +67988,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.119.49";
+        return "2.119.51";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -69913,6 +69958,8 @@ TEST-SEAMS: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md
 
 BREAK points`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `DESCRIBE TABLE items LINES count.`,
+            goodExample: `count = lines( items ).`,
         };
     }
     getDescription(statement) {
@@ -72439,6 +72486,14 @@ class ClassAttributeNames extends _abap_rule_1.ABAPRule {
             title: "Class attributes naming",
             shortDescription: `Allows you to enforce a pattern, such as a prefix, for class variable names.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
+            badExample: `CLASS zcl_foo DEFINITION PUBLIC.
+  PUBLIC SECTION.
+    DATA counter TYPE i.
+ENDCLASS.`,
+            goodExample: `CLASS zcl_foo DEFINITION PUBLIC.
+  PUBLIC SECTION.
+    DATA mv_counter TYPE i.
+ENDCLASS.`,
         };
     }
     getDescription(actual, expected) {
@@ -73159,6 +73214,14 @@ This only applies to global classes.
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#if-your-global-class-is-create-private-leave-the-constructor-public
 https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abeninstance_constructor_guidl.htm`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `CLASS zcl_foo DEFINITION PUBLIC CREATE PRIVATE.
+  PRIVATE SECTION.
+    METHODS constructor.
+ENDCLASS.`,
+            goodExample: `CLASS zcl_foo DEFINITION PUBLIC CREATE PRIVATE.
+  PUBLIC SECTION.
+    METHODS constructor.
+ENDCLASS.`,
         };
     }
     getMessage() {
@@ -73603,6 +73666,8 @@ class DangerousStatement extends _abap_rule_1.ABAPRule {
             extendedInformation: `Dynamic SQL: Typically ABAP logic does not need dynamic SQL,
 dynamic SQL can potentially create SQL injection problems`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Security],
+            badExample: `SELECT * FROM (lv_table_name) INTO TABLE @DATA(lt_rows).`,
+            goodExample: `SELECT * FROM zknown_table INTO TABLE @DATA(lt_rows).`,
         };
     }
     getDescription(statement) {
@@ -73761,6 +73826,17 @@ class DbOperationInLoop extends _abap_rule_1.ABAPRule {
             title: "Database operation in loop",
             shortDescription: `Database operation in LOOP/DO/WHILE`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Performance],
+            badExample: `LOOP AT lt_items INTO DATA(ls_item).
+  SELECT SINGLE name FROM zcustomer INTO @DATA(lv_name) WHERE id = @ls_item-customer_id.
+ENDLOOP.`,
+            goodExample: `ASSERT lines( lt_items ) > 0.
+SELECT id, name FROM zcustomer INTO TABLE @DATA(lt_customers)
+  FOR ALL ENTRIES IN @lt_items
+  WHERE id = @lt_items-customer_id.
+
+LOOP AT lt_items INTO DATA(ls_item).
+  READ TABLE lt_customers INTO DATA(ls_customer) WITH KEY id = ls_item-customer_id.
+ENDLOOP.`,
         };
     }
     getConfig() {
@@ -78147,6 +78223,12 @@ Use RETURN to leave procesing blocks instead.`,
 https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abapcheck_processing_blocks.htm
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#check-vs-return`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
+            badExample: `CHECK is_valid = abap_true.
+do_something( ).`,
+            goodExample: `IF is_valid = abap_false.
+  RETURN.
+ENDIF.
+do_something( ).`,
         };
     }
     getConfig() {
@@ -78929,6 +79011,10 @@ class FormTablesObsolete extends _abap_rule_1.ABAPRule {
             shortDescription: `Checks for TABLES parameters in forms.`,
             extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapform_tables.htm`,
             tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `FORM update_items TABLES items.
+ENDFORM.`,
+            goodExample: `FORM update_items CHANGING items TYPE ty_items.
+ENDFORM.`,
         };
     }
     getMessage() {
@@ -79295,6 +79381,10 @@ class FunctionModuleRecommendations extends _abap_rule_1.ABAPRule {
             shortDescription: `Suggests replacements for various function modules`,
             extendedInformation: `https://docs.abapopenchecks.org/checks/53/`,
             tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `CALL FUNCTION 'GUID_CREATE'
+  IMPORTING
+    ev_guid_32 = guid.`,
+            goodExample: `guid = cl_system_uuid=>create_uuid_c32_static( ).`,
         };
     }
     getConfig() {
@@ -82650,6 +82740,9 @@ class LineLength extends _abap_rule_1.ABAPRule {
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#stick-to-a-reasonable-line-length
 https://docs.abapopenchecks.org/checks/04/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: "DATA long_text TYPE string VALUE 'this line keeps going until it is longer than the configured maximum'.",
+            goodExample: `DATA long_text TYPE string.
+long_text = 'split long expressions across multiple lines'.`,
         };
     }
     getConfig() {
@@ -82827,6 +82920,10 @@ class LocalClassNaming extends _abap_rule_1.ABAPRule {
             title: "Local class naming conventions",
             shortDescription: `Allows you to enforce a pattern, such as a prefix, for local class names.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
+            badExample: `CLASS helper DEFINITION.
+ENDCLASS.`,
+            goodExample: `CLASS lcl_helper DEFINITION.
+ENDCLASS.`,
         };
     }
     getDescription(expected, actual) {
@@ -84398,6 +84495,12 @@ class MethodParameterNames {
             title: "Method parameter naming conventions",
             shortDescription: `Allows you to enforce a pattern, such as a prefix, for method parameter names`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
+            badExample: `METHODS get_name
+  IMPORTING customer TYPE string
+  RETURNING VALUE(name) TYPE string.`,
+            goodExample: `METHODS get_name
+  IMPORTING iv_customer TYPE string
+  RETURNING VALUE(rv_name) TYPE string.`,
         };
     }
     getDescription(expected, actual) {
@@ -85275,6 +85378,13 @@ class NoAliases extends _abap_rule_1.ABAPRule {
             title: "No ALIASES",
             shortDescription: `Detects use of the ALIAS statement`,
             tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `INTERFACE lif_foo.
+  METHODS long_method_name.
+  ALIASES short FOR long_method_name.
+ENDINTERFACE.`,
+            goodExample: `INTERFACE lif_foo.
+  METHODS long_method_name.
+ENDINTERFACE.`,
         };
     }
     getConfig() {
@@ -85479,8 +85589,22 @@ class NoCommentsBetweenMethods extends _abap_rule_1.ABAPRule {
         return {
             key: "no_comments_between_methods",
             title: "No comments between methods in global classes",
-            shortDescription: `Its not possible to have comments between methods in global classes.`,
+            shortDescription: `It's not possible to have comments between methods in global classes.`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Syntax],
+            badExample: `CLASS zcl_foo IMPLEMENTATION.
+  METHOD first.
+  ENDMETHOD.
+  " comment
+  METHOD second.
+  ENDMETHOD.
+ENDCLASS.`,
+            goodExample: `CLASS zcl_foo IMPLEMENTATION.
+  METHOD first.
+  ENDMETHOD.
+
+  METHOD second.
+  ENDMETHOD.
+ENDCLASS.`,
         };
     }
     getConfig() {
@@ -86187,6 +86311,14 @@ class NoPublicAttributes extends _abap_rule_1.ABAPRule {
 Exceptions are excluded from this rule.`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#members-private-by-default-protected-only-if-needed`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `CLASS zcl_foo DEFINITION PUBLIC.
+  PUBLIC SECTION.
+    DATA counter TYPE i.
+ENDCLASS.`,
+            goodExample: `CLASS zcl_foo DEFINITION PUBLIC.
+  PRIVATE SECTION.
+    DATA counter TYPE i.
+ENDCLASS.`,
         };
     }
     getDescription(name) {
@@ -87864,7 +87996,7 @@ See recognized syntax at https://syntax.abaplint.org`,
                         }
                         else {
                             const tok = statement.getFirstToken();
-                            const message = "Statement does not exist in ABAP " + this.reg.getConfig().getRelease().name + " (or a parser error), \"" + tok.getStr() + "\"";
+                            const message = "Statement does not exist in the configured ABAP version(or a parser error), \"" + tok.getStr() + "\"";
                             const issue = issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity);
                             issues.push(issue);
                         }
@@ -89503,6 +89635,14 @@ class PrefixIsCurrentClass extends _abap_rule_1.ABAPRule {
             // eslint-disable-next-line max-len
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#omit-the-self-reference-me-when-calling-an-instance-attribute-or-method`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
+            badExample: `METHOD run.
+  zcl_current=>helper( ).
+  me->do_work( ).
+ENDMETHOD.`,
+            goodExample: `METHOD run.
+  helper( ).
+  do_work( ).
+ENDMETHOD.`,
         };
     }
     getConfig() {
@@ -92105,6 +92245,12 @@ class StaticCallViaInstance extends _abap_rule_1.ABAPRule {
             shortDescription: `Static method call via instance variable`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-call-static-methods-through-instance-variables`,
             tags: [_irule_1.RuleTag.Styleguide],
+            badExample: `METHOD run.
+  NEW lcl_helper( )->static_method( ).
+ENDMETHOD.`,
+            goodExample: `METHOD run.
+  lcl_helper=>static_method( ).
+ENDMETHOD.`,
         };
     }
     getConfig() {

package/package.json

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