@abaplint/core 2.79.18 → 2.79.22

package/README.md

@@ -1,7 +1,7 @@
-# @abaplint/core
-
-[abaplint](https://abaplint.org/) core library
-
-Exposes functionallity like the parser and rules, which can be used in other projects.
-
+# @abaplint/core
+
+[abaplint](https://abaplint.org/) core library
+
+Exposes functionallity like the parser and rules, which can be used in other projects.
+
 For more information see https://github.com/abaplint/abaplint

package/build/abaplint.d.ts

@@ -2402,6 +2402,7 @@ declare interface IDDICReferences {
     addUsing(obj: IObject, using: IObjectAndToken | undefined): void;
     clear(obj: IObject): void;
     listUsing(obj: IObject): readonly IObjectAndToken[];
+    listByFilename(filename: string, line: number): readonly IObjectAndToken[];
     listWhereUsed(obj: IObject): {
         type: string;
         name: string;

package/build/src/abap/5_syntax/_builtin.js

@@ -222,92 +222,92 @@ class BuiltIn {
     /////////////////////////////
     buildSY() {
         const components = [];
-        components.push({ name: "abcde", type: new basic_1.CharacterType(26) });
-        components.push({ name: "batch", type: new basic_1.CharacterType(1) });
-        components.push({ name: "binpt", type: new basic_1.CharacterType(1) });
-        components.push({ name: "calld", type: new basic_1.CharacterType(1) });
-        components.push({ name: "callr", type: new basic_1.CharacterType(8) });
+        components.push({ name: "abcde", type: new basic_1.CharacterType(26, "sy-abcde") });
+        components.push({ name: "batch", type: new basic_1.CharacterType(1, "sy-batch") });
+        components.push({ name: "binpt", type: new basic_1.CharacterType(1, "sy-binpt") });
+        components.push({ name: "calld", type: new basic_1.CharacterType(1, "sy-calld") });
+        components.push({ name: "callr", type: new basic_1.CharacterType(8, "sy-callr") });
         components.push({ name: "colno", type: new basic_1.IntegerType() });
         components.push({ name: "cpage", type: new basic_1.IntegerType() });
-        components.push({ name: "cprog", type: new basic_1.CharacterType(40) });
+        components.push({ name: "cprog", type: new basic_1.CharacterType(40, "sy-cprog") });
         components.push({ name: "cucol", type: new basic_1.IntegerType() });
         components.push({ name: "curow", type: new basic_1.IntegerType() });
-        components.push({ name: "datar", type: new basic_1.CharacterType(1) });
+        components.push({ name: "datar", type: new basic_1.CharacterType(1, "sy-datar") });
         components.push({ name: "datlo", type: new basic_1.DateType() });
         components.push({ name: "datum", type: new basic_1.DateType() });
         components.push({ name: "dayst", type: new basic_1.CharacterType(1) });
         components.push({ name: "dbcnt", type: new basic_1.IntegerType() });
-        components.push({ name: "dbnam", type: new basic_1.CharacterType(20) });
-        components.push({ name: "dbsys", type: new basic_1.CharacterType(10) });
-        components.push({ name: "dyngr", type: new basic_1.CharacterType(4) });
-        components.push({ name: "dynnr", type: new basic_1.CharacterType(4) });
+        components.push({ name: "dbnam", type: new basic_1.CharacterType(20, "sy-dbnam") });
+        components.push({ name: "dbsys", type: new basic_1.CharacterType(10, "sy-dbsys") });
+        components.push({ name: "dyngr", type: new basic_1.CharacterType(4, "sy-dyngr") });
+        components.push({ name: "dynnr", type: new basic_1.CharacterType(4, "sy-dynnr") });
         components.push({ name: "fdayw", type: new basic_1.IntegerType() });
         components.push({ name: "fdpos", type: new basic_1.IntegerType() });
         components.push({ name: "fleng", type: new basic_1.IntegerType() });
         components.push({ name: "folen", type: new basic_1.IntegerType() });
-        components.push({ name: "host", type: new basic_1.CharacterType(32) });
+        components.push({ name: "host", type: new basic_1.CharacterType(32, "sy-host") });
         components.push({ name: "index", type: new basic_1.IntegerType() });
-        components.push({ name: "langu", type: new basic_1.CharacterType(1) });
-        components.push({ name: "ldbpg", type: new basic_1.CharacterType(40) });
-        components.push({ name: "pdest", type: new basic_1.CharacterType(4) });
+        components.push({ name: "langu", type: new basic_1.CharacterType(1, "sy-langu") });
+        components.push({ name: "ldbpg", type: new basic_1.CharacterType(40, "sy-ldbpg") });
+        components.push({ name: "pdest", type: new basic_1.CharacterType(4, "sy-pdest") });
         components.push({ name: "lilli", type: new basic_1.IntegerType() });
         components.push({ name: "linct", type: new basic_1.IntegerType() });
         components.push({ name: "linno", type: new basic_1.IntegerType() });
         components.push({ name: "linsz", type: new basic_1.IntegerType() });
-        components.push({ name: "lisel", type: new basic_1.CharacterType(255) });
+        components.push({ name: "lisel", type: new basic_1.CharacterType(255, "sy-lisel") });
         components.push({ name: "listi", type: new basic_1.IntegerType() });
         components.push({ name: "loopc", type: new basic_1.IntegerType() });
         components.push({ name: "lsind", type: new basic_1.IntegerType() });
         components.push({ name: "macol", type: new basic_1.IntegerType() });
-        components.push({ name: "mandt", type: new basic_1.CharacterType(3) });
+        components.push({ name: "mandt", type: new basic_1.CharacterType(3, "sy-mandt") });
         components.push({ name: "marow", type: new basic_1.IntegerType() });
         components.push({ name: "modno", type: new basic_1.IntegerType() });
-        components.push({ name: "msgid", type: new basic_1.CharacterType(20) });
-        components.push({ name: "msgli", type: new basic_1.CharacterType(60) });
-        components.push({ name: "msgno", type: new basic_1.NumericType(3) });
-        components.push({ name: "msgty", type: new basic_1.CharacterType(1) });
-        components.push({ name: "msgv1", type: new basic_1.CharacterType(50) });
-        components.push({ name: "msgv2", type: new basic_1.CharacterType(50) });
-        components.push({ name: "msgv3", type: new basic_1.CharacterType(50) });
-        components.push({ name: "msgv4", type: new basic_1.CharacterType(50) });
-        components.push({ name: "opsys", type: new basic_1.CharacterType(10) });
+        components.push({ name: "msgid", type: new basic_1.CharacterType(20, "sy-msgid") });
+        components.push({ name: "msgli", type: new basic_1.CharacterType(60, "sy-msgli") });
+        components.push({ name: "msgno", type: new basic_1.NumericType(3, "sy-msgno") });
+        components.push({ name: "msgty", type: new basic_1.CharacterType(1, "sy-msgty") });
+        components.push({ name: "msgv1", type: new basic_1.CharacterType(50, "sy-msgv1") });
+        components.push({ name: "msgv2", type: new basic_1.CharacterType(50, "sy-msgv2") });
+        components.push({ name: "msgv3", type: new basic_1.CharacterType(50, "sy-msgv3") });
+        components.push({ name: "msgv4", type: new basic_1.CharacterType(50, "sy-msgv4") });
+        components.push({ name: "opsys", type: new basic_1.CharacterType(10, "sy-opsys") });
         components.push({ name: "pagno", type: new basic_1.IntegerType() });
-        components.push({ name: "pfkey", type: new basic_1.CharacterType(20) });
-        components.push({ name: "repid", type: new basic_1.CharacterType(40) });
-        components.push({ name: "saprl", type: new basic_1.CharacterType(4) });
+        components.push({ name: "pfkey", type: new basic_1.CharacterType(20, "sy-pfkey") });
+        components.push({ name: "repid", type: new basic_1.CharacterType(40, "sy-repid") });
+        components.push({ name: "saprl", type: new basic_1.CharacterType(4, "sy-saprl") });
         components.push({ name: "scols", type: new basic_1.IntegerType() });
-        components.push({ name: "slset", type: new basic_1.CharacterType(14) });
-        components.push({ name: "spono", type: new basic_1.CharacterType(10) });
+        components.push({ name: "slset", type: new basic_1.CharacterType(14, "sy-slset") });
+        components.push({ name: "spono", type: new basic_1.CharacterType(10, "sy-spono") });
         components.push({ name: "srows", type: new basic_1.IntegerType() });
         components.push({ name: "staco", type: new basic_1.IntegerType() });
         components.push({ name: "staro", type: new basic_1.IntegerType() });
         components.push({ name: "stepl", type: new basic_1.IntegerType() });
         components.push({ name: "subrc", type: new basic_1.IntegerType() });
-        components.push({ name: "sysid", type: new basic_1.CharacterType(3) });
+        components.push({ name: "sysid", type: new basic_1.CharacterType(3, "sy-sysid") });
         components.push({ name: "tabix", type: new basic_1.IntegerType() });
-        components.push({ name: "tcode", type: new basic_1.CharacterType(20) });
+        components.push({ name: "tcode", type: new basic_1.CharacterType(20, "sy-tcode") });
         components.push({ name: "tfill", type: new basic_1.IntegerType() });
         components.push({ name: "timlo", type: new basic_1.TimeType() });
-        components.push({ name: "title", type: new basic_1.CharacterType(70) });
+        components.push({ name: "title", type: new basic_1.CharacterType(70, "sy-title") });
         components.push({ name: "tleng", type: new basic_1.IntegerType() });
-        components.push({ name: "tvar0", type: new basic_1.CharacterType(20) });
-        components.push({ name: "tvar1", type: new basic_1.CharacterType(20) });
-        components.push({ name: "tvar2", type: new basic_1.CharacterType(20) });
-        components.push({ name: "tvar3", type: new basic_1.CharacterType(20) });
-        components.push({ name: "tvar4", type: new basic_1.CharacterType(20) });
-        components.push({ name: "tvar5", type: new basic_1.CharacterType(20) });
-        components.push({ name: "tvar6", type: new basic_1.CharacterType(20) });
-        components.push({ name: "tvar7", type: new basic_1.CharacterType(20) });
-        components.push({ name: "tvar8", type: new basic_1.CharacterType(20) });
-        components.push({ name: "tvar9", type: new basic_1.CharacterType(20) });
+        components.push({ name: "tvar0", type: new basic_1.CharacterType(20, "sy-tvar0") });
+        components.push({ name: "tvar1", type: new basic_1.CharacterType(20, "sy-tvar1") });
+        components.push({ name: "tvar2", type: new basic_1.CharacterType(20, "sy-tvar2") });
+        components.push({ name: "tvar3", type: new basic_1.CharacterType(20, "sy-tvar3") });
+        components.push({ name: "tvar4", type: new basic_1.CharacterType(20, "sy-tvar4") });
+        components.push({ name: "tvar5", type: new basic_1.CharacterType(20, "sy-tvar5") });
+        components.push({ name: "tvar6", type: new basic_1.CharacterType(20, "sy-tvar6") });
+        components.push({ name: "tvar7", type: new basic_1.CharacterType(20, "sy-tvar7") });
+        components.push({ name: "tvar8", type: new basic_1.CharacterType(20, "sy-tvar8") });
+        components.push({ name: "tvar9", type: new basic_1.CharacterType(20, "sy-tvar9") });
         components.push({ name: "tzone", type: new basic_1.IntegerType() });
-        components.push({ name: "ucomm", type: new basic_1.CharacterType(70) });
-        components.push({ name: "uline", type: new basic_1.CharacterType(255) });
-        components.push({ name: "uname", type: new basic_1.CharacterType(12) });
+        components.push({ name: "ucomm", type: new basic_1.CharacterType(70, "sy-ucomm") });
+        components.push({ name: "uline", type: new basic_1.CharacterType(255, "sy-uline") });
+        components.push({ name: "uname", type: new basic_1.CharacterType(12, "sy-uname") });
         components.push({ name: "uzeit", type: new basic_1.TimeType() });
-        components.push({ name: "vline", type: new basic_1.CharacterType(1) });
-        components.push({ name: "wtitl", type: new basic_1.CharacterType(1) });
-        components.push({ name: "zonlo", type: new basic_1.CharacterType(6) });
+        components.push({ name: "vline", type: new basic_1.CharacterType(1, "sy-vline") });
+        components.push({ name: "wtitl", type: new basic_1.CharacterType(1, "sy-wtitl") });
+        components.push({ name: "zonlo", type: new basic_1.CharacterType(6, "sy-zonlo") });
         const type = new basic_1.StructureType(components);
         const id1 = new tokens_1.Identifier(new position_1.Position(this.row++, 1), "sy");
         const sy = new _typed_identifier_1.TypedIdentifier(id1, BuiltIn.filename, type, ["read_only" /* ReadOnly */, "built-in" /* BuiltIn */]);

package/build/src/abap/5_syntax/basic_types.js

@@ -14,6 +14,8 @@ const basic_1 = require("../types/basic");
 const field_chain_1 = require("./expressions/field_chain");
 const types_1 = require("../types");
 const expressions_1 = require("../2_statements/expressions");
+const _builtin_1 = require("./_builtin");
+const position_1 = require("../../position");
 class BasicTypes {
     constructor(filename, scope) {
         this.filename = filename;
@@ -45,6 +47,10 @@ class BasicTypes {
         if (id && (lookup === null || lookup === void 0 ? void 0 : lookup.type)) {
             return new _typed_identifier_1.TypedIdentifier(id.getToken(), id.getFilename(), lookup.type);
         }
+        const builtin = this.scope.getDDIC().lookupBuiltinType(name);
+        if (builtin) {
+            return new _typed_identifier_1.TypedIdentifier(new identifier_1.Identifier(new position_1.Position(1, 1), name), _builtin_1.BuiltIn.filename, builtin);
+        }
         return undefined;
     }
     resolveLikeName(node, headerLogic = true) {
@@ -469,7 +475,7 @@ class BasicTypes {
             foundType = found === null || found === void 0 ? void 0 : found.getType();
             if (foundType === undefined) {
                 const f = this.scope.getDDIC().lookupTableOrView(subs[0]);
-                this.scope.getDDICReferences().addUsing(this.scope.getParentObj(), { object: f.object });
+                this.scope.getDDICReferences().addUsing(this.scope.getParentObj(), { object: f.object, filename: this.filename, token: expr.getFirstToken() });
                 if (f.type instanceof _typed_identifier_1.TypedIdentifier) {
                     foundType = f.type.getType();
                 }

package/build/src/abap/5_syntax/expressions/method_source.js

@@ -8,6 +8,7 @@ const method_call_chain_1 = require("./method_call_chain");
 const basic_1 = require("../../types/basic");
 const types_1 = require("../../types");
 const _reference_1 = require("../_reference");
+const _object_oriented_1 = require("../_object_oriented");
 class MethodSource {
     runSyntax(node, scope, filename) {
         const context = new method_call_chain_1.MethodCallChain().runSyntax(node, scope, filename);
@@ -20,12 +21,17 @@ class MethodSource {
             if (context instanceof basic_1.ObjectReferenceType) {
                 const id = context.getIdentifier();
                 if (id instanceof types_1.ClassDefinition) {
-                    const method = id.getMethodDefinitions().getByName(last.concatTokens());
+                    const methodName = last.concatTokens().toUpperCase();
+                    const helper = new _object_oriented_1.ObjectOriented(scope);
+                    const { method: foundMethod } = helper.searchMethodName(id, methodName);
+                    if (foundMethod === undefined && methodName !== "CONSTRUCTOR") {
+                        throw new Error(`MethodSource, method not found \"${methodName}\"`);
+                    }
                     const extra = {
                         ooName: id.getName(),
                         ooType: "CLAS"
                     };
-                    scope.addReference(last.getFirstToken(), method, _reference_1.ReferenceType.MethodReference, filename, extra);
+                    scope.addReference(last.getFirstToken(), foundMethod, _reference_1.ReferenceType.MethodReference, filename, extra);
                 }
             }
             else if (context instanceof basic_1.VoidType) {

package/build/src/abap/5_syntax/statements/tables.js

@@ -17,7 +17,7 @@ class Tables {
         }
         const found = (_b = scope.getDDIC()) === null || _b === void 0 ? void 0 : _b.lookupTableOrView(name);
         if (found) {
-            scope.getDDICReferences().addUsing(scope.getParentObj(), { object: found.object });
+            scope.getDDICReferences().addUsing(scope.getParentObj(), { object: found.object, filename: filename, token: nameToken });
             scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, filename, found.type));
             return;
         }

package/build/src/ddic.js

@@ -46,13 +46,13 @@ class DDIC {
     lookupBuiltinType(name, length, decimals) {
         switch (name) {
             case "STRING":
-                return new Types.StringType();
+                return new Types.StringType(name);
             case "XSTRING":
-                return new Types.XStringType();
+                return new Types.XStringType(name);
             case "D":
-                return new Types.DateType();
+                return new Types.DateType(name);
             case "T":
-                return new Types.TimeType();
+                return new Types.TimeType(name);
             case "XSEQUENCE":
                 return new Types.XSequenceType();
             case "CLIKE":
@@ -79,9 +79,9 @@ class DDIC {
                 return new Types.CSequenceType();
             case "I":
             case "INT8": // todo, take version into account
-                return new Types.IntegerType();
+                return new Types.IntegerType(name);
             case "F":
-                return new Types.FloatType();
+                return new Types.FloatType(name);
             case "P":
                 if (length && decimals) {
                     return new Types.PackedType(length, decimals);

package/build/src/ddic_references.js

@@ -3,38 +3,66 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.DDICReferences = void 0;
 class DDICReferences {
     constructor() {
-        this.index = {};
+        this.nameTypeIndex = {};
+        this.filenameIndex = {};
     }
     setUsing(obj, using) {
-        const newName = obj.getName().toUpperCase();
-        const newType = obj.getType();
-        if (this.index[newName] === undefined) {
-            this.index[newName] = {};
+        this.clear(obj);
+        for (const u of using) {
+            this.addUsing(obj, u);
         }
-        this.index[newName][newType] = using;
     }
     addUsing(obj, using) {
         if (using === undefined) {
             return;
         }
+        // add to name and type index
         const newName = obj.getName().toUpperCase();
         const newType = obj.getType();
-        if (this.index[newName] === undefined) {
-            this.index[newName] = {};
+        if (this.nameTypeIndex[newName] === undefined) {
+            this.nameTypeIndex[newName] = {};
         }
-        if (this.index[newName][newType] === undefined) {
-            this.index[newName][newType] = [];
+        if (this.nameTypeIndex[newName][newType] === undefined) {
+            this.nameTypeIndex[newName][newType] = [];
+        }
+        this.nameTypeIndex[newName][newType].push(using);
+        // add to filename index
+        if (using.filename && using.token) {
+            if (this.filenameIndex[using.filename] === undefined) {
+                this.filenameIndex[using.filename] = {};
+            }
+            if (this.filenameIndex[using.filename][using.token.getRow()] === undefined) {
+                this.filenameIndex[using.filename][using.token.getRow()] = [];
+            }
+            this.filenameIndex[using.filename][using.token.getRow()].push(using);
         }
-        this.index[newName][newType].push(using);
     }
     clear(obj) {
-        this.setUsing(obj, []);
+        var _a, _b;
+        // remove from filenameIndex first
+        for (const u of this.listUsing(obj)) {
+            if (u.filename && u.token) {
+                const found = (_a = this.filenameIndex[u.filename]) === null || _a === void 0 ? void 0 : _a[u.token.getRow()];
+                if (found) {
+                    found.pop(); // TODODOD, this assumes there is max one reference on each line
+                }
+            }
+        }
+        const newName = obj.getName().toUpperCase();
+        const newType = obj.getType();
+        if ((_b = this.nameTypeIndex[newName]) === null || _b === void 0 ? void 0 : _b[newType]) {
+            this.nameTypeIndex[newName][newType] = [];
+        }
+    }
+    listByFilename(filename, line) {
+        var _a;
+        return ((_a = this.filenameIndex[filename]) === null || _a === void 0 ? void 0 : _a[line]) || [];
     }
     listUsing(obj) {
         var _a;
         const newName = obj.getName().toUpperCase();
         const newType = obj.getType();
-        const found = (_a = this.index[newName]) === null || _a === void 0 ? void 0 : _a[newType];
+        const found = (_a = this.nameTypeIndex[newName]) === null || _a === void 0 ? void 0 : _a[newType];
         if (found !== undefined) {
             return found;
         }
@@ -47,9 +75,9 @@ class DDICReferences {
         const ret = [];
         const searchName = obj.getName().toUpperCase();
         const searchType = obj.getType();
-        for (const name in this.index) {
-            for (const type in this.index[name]) {
-                for (const f of this.index[name][type]) {
+        for (const name in this.nameTypeIndex) {
+            for (const type in this.nameTypeIndex[name]) {
+                for (const f of this.nameTypeIndex[name][type]) {
                     if (f.object && f.object.getType() === searchType && f.object.getName() === searchName) {
                         ret.push({ type, name, token: f.token, filename: f.filename });
                     }

package/build/src/lsp/_lookup.js

@@ -64,14 +64,20 @@ class LSPLookup {
             }
             return { hover, definition: location, implementation: location, definitionId: variable, scope: bottomScope };
         }
+        let hoverValue = "";
+        const ddicRefs = reg.getDDICReferences().listByFilename(cursor.identifier.getFilename(), cursor.identifier.getStart().getRow());
+        for (const d of ddicRefs) {
+            if (d.object && d.token && d.token.getStart().equals(cursor.identifier.getStart())) {
+                hoverValue += `DDIC: ${d.object.getType()} ${d.object.getName()}`;
+            }
+        }
         const refs = this.searchReferences(bottomScope, cursor.token);
         if (refs.length > 0) {
-            let value = "";
             for (const ref of refs) {
-                if (value !== "") {
-                    value += "\n_________________\n";
+                if (hoverValue !== "") {
+                    hoverValue += "\n_________________\n";
                 }
-                value += this.referenceHover(ref, bottomScope, reg);
+                hoverValue += this.referenceHover(ref, bottomScope, reg);
             }
             let definition = undefined;
             if (refs[0].resolved) {
@@ -80,7 +86,10 @@ class LSPLookup {
                     definition = undefined;
                 }
             }
-            return { hover: value, definition, definitionId: refs[0].resolved, scope: bottomScope };
+            return { hover: hoverValue, definition, definitionId: refs[0].resolved, scope: bottomScope };
+        }
+        if (hoverValue !== "") {
+            return { hover: hoverValue, scope: bottomScope };
         }
         return undefined;
     }

package/build/src/lsp/help.js

@@ -9,13 +9,13 @@ const dump_scope_1 = require("./dump_scope");
 class Help {
     static find(reg, textDocument, position) {
         let content = "";
-        content = `
-    <a href="#_tokens" rel="no-refresh">Tokens</a> |
-    <a href="#_statements" rel="no-refresh">Statements</a> |
-    <a href="#_structure" rel="no-refresh">Structure</a> |
-    <a href="#_files" rel="no-refresh">Files</a> |
-    <a href="#_info" rel="no-refresh">Info Dump</a>
-    <hr>
+        content = `
+    <a href="#_tokens" rel="no-refresh">Tokens</a> |
+    <a href="#_statements" rel="no-refresh">Statements</a> |
+    <a href="#_structure" rel="no-refresh">Structure</a> |
+    <a href="#_files" rel="no-refresh">Files</a> |
+    <a href="#_info" rel="no-refresh">Info Dump</a>
+    <hr>
     ` +
             "<tt>" + textDocument.uri + " (" +
             (position.line + 1) + ", " +

package/build/src/objects/assignment_service_to_authorization_group.js

@@ -8,7 +8,7 @@ class AssignmentServiceToAuthorizationGroup extends _abstract_object_1.AbstractO
     }
     getAllowedNaming() {
         return {
-            maxLength: 30,
+            maxLength: 32,
             allowNamespace: true,
         };
     }

package/build/src/registry.js

@@ -68,7 +68,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.79.18";
+        return "2.79.22";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/abapdoc.js

@@ -23,7 +23,7 @@ class Abapdoc extends _abap_rule_1.ABAPRule {
         return {
             key: "abapdoc",
             title: "Check abapdoc",
-            shortDescription: `Various checks regarding abapdoc.
+            shortDescription: `Various checks regarding abapdoc.
 Base rule checks for existence of abapdoc for public class methods and all interface methods.`,
             tags: [_irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/ambiguous_statement.js

@@ -20,14 +20,14 @@ class AmbiguousStatement extends _abap_rule_1.ABAPRule {
         return {
             key: "ambiguous_statement",
             title: "Check for ambigious statements",
-            shortDescription: `Checks for ambiguity between deleting or modifying from internal and database table
-Add "TABLE" keyword or "@" for escaping SQL variables
-
+            shortDescription: `Checks for ambiguity between deleting or modifying from internal and database table
+Add "TABLE" keyword or "@" for escaping SQL variables
+
 Only works if the target version is 740sp05 or above`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `DELETE foo FROM bar.
+            badExample: `DELETE foo FROM bar.
 MODIFY foo FROM bar.`,
-            goodExample: `DELETE foo FROM @bar.
+            goodExample: `DELETE foo FROM @bar.
 MODIFY TABLE foo FROM bar.`,
         };
     }

package/build/src/rules/avoid_use.js

@@ -35,12 +35,12 @@ class AvoidUse extends _abap_rule_1.ABAPRule {
             key: "avoid_use",
             title: "Avoid use of certain statements",
             shortDescription: `Detects usage of certain statements.`,
-            extendedInformation: `DEFAULT KEY: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-default-key
-
-Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
-
-STATICS: use CLASS-DATA instead
-
+            extendedInformation: `DEFAULT KEY: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-default-key
+
+Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
+
+STATICS: use CLASS-DATA instead
+
 DESCRIBE TABLE LINES: use lines() instead (quickfix exists)`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/begin_end_names.js

@@ -23,11 +23,11 @@ class BeginEndNames extends _abap_rule_1.ABAPRule {
             title: "Check BEGIN END names",
             shortDescription: `Check BEGIN OF and END OF names match, plus there must be statements between BEGIN and END`,
             tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
-            badExample: `DATA: BEGIN OF stru,
-      field TYPE i,
+            badExample: `DATA: BEGIN OF stru,
+      field TYPE i,
     END OF structure_not_the_same.`,
-            goodExample: `DATA: BEGIN OF stru,
-      field TYPE i,
+            goodExample: `DATA: BEGIN OF stru,
+      field TYPE i,
     END OF stru.`,
         };
     }

package/build/src/rules/begin_single_include.js

@@ -21,19 +21,19 @@ class BeginSingleInclude extends _abap_rule_1.ABAPRule {
             title: "BEGIN contains single INCLUDE",
             shortDescription: `Finds TYPE BEGIN with just one INCLUDE TYPE, and DATA with single INCLUDE STRUCTURE`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `TYPES: BEGIN OF dummy1.
-  INCLUDE TYPE dselc.
-TYPES: END OF dummy1.
-
-DATA BEGIN OF foo.
-INCLUDE STRUCTURE syst.
-DATA END OF foo.
-
-STATICS BEGIN OF bar.
-INCLUDE STRUCTURE syst.
+            badExample: `TYPES: BEGIN OF dummy1.
+  INCLUDE TYPE dselc.
+TYPES: END OF dummy1.
+
+DATA BEGIN OF foo.
+INCLUDE STRUCTURE syst.
+DATA END OF foo.
+
+STATICS BEGIN OF bar.
+INCLUDE STRUCTURE syst.
 STATICS END OF bar.`,
-            goodExample: `DATA BEGIN OF foo.
-INCLUDE STRUCTURE dselc.
+            goodExample: `DATA BEGIN OF foo.
+INCLUDE STRUCTURE dselc.
 DATA END OF foo.`,
         };
     }

package/build/src/rules/chain_mainly_declarations.js

@@ -45,10 +45,10 @@ class ChainMainlyDeclarations extends _abap_rule_1.ABAPRule {
             key: "chain_mainly_declarations",
             title: "Chain mainly declarations",
             shortDescription: `Chain mainly declarations, allows chaining for the configured statements, reports errors for other statements.`,
-            extendedInformation: `
-https://docs.abapopenchecks.org/checks/23/
-
-https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenchained_statements_guidl.htm
+            extendedInformation: `
+https://docs.abapopenchecks.org/checks/23/
+
+https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenchained_statements_guidl.htm
 `,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             badExample: `CALL METHOD: bar.`,

package/build/src/rules/check_abstract.js

@@ -23,8 +23,8 @@ class CheckAbstract extends _abap_rule_1.ABAPRule {
         return {
             key: "check_abstract",
             title: "Check abstract methods and classes",
-            shortDescription: `Checks abstract methods and classes:
-- class defined as abstract and final,
+            shortDescription: `Checks abstract methods and classes:
+- class defined as abstract and final,
 - non-abstract class contains abstract methods`,
             extendedInformation: `If a class defines only constants, use an interface instead`,
             tags: [_irule_1.RuleTag.SingleFile],

package/build/src/rules/check_comments.js

@@ -27,9 +27,9 @@ class CheckComments extends _abap_rule_1.ABAPRule {
         return {
             key: "check_comments",
             title: "Check Comments",
-            shortDescription: `
-Various checks for comment usage.
-
+            shortDescription: `
+Various checks for comment usage.
+
 * End of line comments. Comments starting with "#EC" or "##" are ignored`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#put-comments-before-the-statement-they-relate-to`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],

package/build/src/rules/check_include.js

@@ -17,9 +17,9 @@ class CheckInclude {
             key: "check_include",
             title: "Check INCLUDEs",
             shortDescription: `Checks INCLUDE statements`,
-            extendedInformation: `
-* Reports unused includes
-* Errors if the includes are not found
+            extendedInformation: `
+* Reports unused includes
+* Errors if the includes are not found
 * Error if including a main program`,
             tags: [_irule_1.RuleTag.Syntax],
         };

package/build/src/rules/check_no_handler_pragma.js

@@ -21,15 +21,15 @@ class CheckNoHandlerPragma extends _abap_rule_1.ABAPRule {
             title: "Check if NO_HANDLER can be removed",
             shortDescription: `Checks NO_HANDLER pragmas that can be removed`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `TRY.
-    ...
-  CATCH zcx_abapgit_exception ##NO_HANDLER.
-    RETURN. " it has a handler
+            badExample: `TRY.
+    ...
+  CATCH zcx_abapgit_exception ##NO_HANDLER.
+    RETURN. " it has a handler
 ENDTRY.`,
-            goodExample: `TRY.
-    ...
-  CATCH zcx_abapgit_exception.
-    RETURN.
+            goodExample: `TRY.
+    ...
+  CATCH zcx_abapgit_exception.
+    RETURN.
 ENDTRY.`,
         };
     }