@abaplint/transpiler-cli 2.8.24 → 2.8.25

package/build/bundle.js

@@ -535,6 +535,7 @@ class LexerStream {
             this.row = this.row + 1;
         }
         if (this.offset === this.raw.length) {
+            this.col = this.col - 1;
             return false;
         }
         this.col = this.col + 1;
@@ -7831,9 +7832,8 @@ class Target extends combi_1.Expression {
         const attr = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.InstanceArrow), attribute_name_1.AttributeName);
         const comp = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.Dash), _1.ComponentName);
         const something = (0, combi_1.starPrio)((0, combi_1.altPrio)(dereference_1.Dereference, attr, comp, _1.TableExpression));
-        const cast = (0, combi_1.seq)((0, combi_1.altPrio)(_1.Cast, _1.NewObject), _1.Arrow, _1.FieldAll);
         const clas = (0, combi_1.seq)(_1.ClassName, (0, combi_1.tok)(tokens_1.StaticArrow), attribute_name_1.AttributeName);
-        const start = (0, combi_1.altPrio)(cast, clas, _1.TargetField, _1.TargetFieldSymbol);
+        const start = (0, combi_1.altPrio)(_1.Cast, _1.NewObject, clas, _1.TargetField, _1.TargetFieldSymbol);
         const fields = (0, combi_1.seq)((0, combi_1.optPrio)(_1.FieldOffset), (0, combi_1.optPrio)(_1.FieldLength));
         const optional = (0, combi_1.altPrio)(_1.TableBody, fields);
         return (0, combi_1.altPrio)(_1.InlineData, _1.InlineFS, (0, combi_1.seq)(start, something, optional));
@@ -10826,9 +10826,9 @@ const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/
 const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 class EditorCall {
     getMatcher() {
-        const title = (0, combi_1.seq)("TITLE", expressions_1.Source);
+        const title = (0, combi_1.seq)("TITLE", expressions_1.SimpleSource3);
         const options = (0, combi_1.per)("DISPLAY-MODE", title);
-        const ret = (0, combi_1.seq)("EDITOR-CALL FOR", (0, combi_1.opt)("REPORT"), expressions_1.Source, (0, combi_1.opt)(options));
+        const ret = (0, combi_1.seq)("EDITOR-CALL FOR", (0, combi_1.optPrio)("REPORT"), expressions_1.Source, (0, combi_1.optPrio)(options));
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
     }
 }
@@ -21617,6 +21617,16 @@ class TypeUtils {
                 return false;
             }
         }
+        else if (source instanceof basic_1.TableType) {
+            if (target instanceof basic_1.TableType) {
+                return this.isAssignableStrict(source.getRowType(), target.getRowType());
+            }
+            else if (target instanceof basic_1.UnknownType
+                || target instanceof basic_1.AnyType
+                || target instanceof basic_1.VoidType) {
+                return true;
+            }
+        }
         return this.isAssignable(source, target);
     }
     isAssignable(source, target) {
@@ -22849,6 +22859,7 @@ const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ ".
 const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
 const basic_types_1 = __webpack_require__(/*! ../basic_types */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/basic_types.js");
+const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 class Cast {
     runSyntax(node, scope, targetType, filename) {
         const sourceNode = node.findDirectExpression(Expressions.Source);
@@ -22875,6 +22886,7 @@ class Cast {
                 const found = scope.findObjectDefinition(typeName);
                 if (found) {
                     tt = new basic_1.ObjectReferenceType(found, { qualifiedName: typeName });
+                    scope.addReference(typeExpression.getFirstToken(), found, _reference_1.ReferenceType.ObjectOrientedReference, filename);
                 }
             }
             else {
@@ -25432,6 +25444,7 @@ const _scope_type_1 = __webpack_require__(/*! ../_scope_type */ "./node_modules/
 const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js");
 const sql_compare_1 = __webpack_require__(/*! ./sql_compare */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_compare.js");
 const sql_order_by_1 = __webpack_require__(/*! ./sql_order_by */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_order_by.js");
+const dynamic_1 = __webpack_require__(/*! ./dynamic */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js");
 const isSimple = /^\w+$/;
 class Select {
     runSyntax(node, scope, filename, skipImplicitInto = false) {
@@ -25442,7 +25455,7 @@ class Select {
         if (from === undefined) {
             throw new Error(`Missing FROM`);
         }
-        const fields = this.findFields(node);
+        const fields = this.findFields(node, scope, filename);
         if (fields.length === 0
             && node.findDirectExpression(Expressions.SQLFieldListLoop) === undefined) {
             throw new Error(`fields missing`);
@@ -25600,17 +25613,20 @@ class Select {
         }
         return new basic_1.VoidType("SELECT_todo");
     }
-    findFields(node) {
-        var _a;
+    findFields(node, scope, filename) {
+        var _a, _b;
         let expr = undefined;
         const ret = [];
         expr = node.findFirstExpression(Expressions.SQLFieldList);
         if (expr === undefined) {
             expr = node.findDirectExpression(Expressions.SQLFieldListLoop);
         }
+        if (((_a = expr === null || expr === void 0 ? void 0 : expr.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Dynamic) {
+            new dynamic_1.Dynamic().runSyntax(expr.getFirstChild(), scope, filename);
+        }
         for (const field of (expr === null || expr === void 0 ? void 0 : expr.findDirectExpressionsMulti([Expressions.SQLField, Expressions.SQLFieldName])) || []) {
             let code = field.concatTokens().toUpperCase();
-            const as = ((_a = field.findDirectExpression(Expressions.SQLAsName)) === null || _a === void 0 ? void 0 : _a.concatTokens()) || "";
+            const as = ((_b = field.findDirectExpression(Expressions.SQLAsName)) === null || _b === void 0 ? void 0 : _b.concatTokens()) || "";
             if (as !== "") {
                 code = code.replace(" AS " + as, "");
             }
@@ -26052,13 +26068,18 @@ exports.SQLCompare = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const nodes_1 = __webpack_require__(/*! ../../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const dynamic_1 = __webpack_require__(/*! ./dynamic */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js");
 const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js");
 class SQLCompare {
     runSyntax(node, scope, filename, tables) {
-        var _a;
+        var _a, _b;
         let sourceType;
         let token;
+        if (((_a = node.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Dynamic) {
+            new dynamic_1.Dynamic().runSyntax(node.getFirstChild(), scope, filename);
+            return;
+        }
         for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }
@@ -26084,7 +26105,7 @@ class SQLCompare {
                 }
             }
         }
-        const fieldName = (_a = node.findDirectExpression(Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens();
+        const fieldName = (_b = node.findDirectExpression(Expressions.SQLFieldName)) === null || _b === void 0 ? void 0 : _b.concatTokens();
         if (fieldName && sourceType && token) {
             // check compatibility for rule sql_value_conversion
             const targetType = this.findType(fieldName, tables, scope);
@@ -26397,6 +26418,7 @@ const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@a
 const table_expression_1 = __webpack_require__(/*! ./table_expression */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/table_expression.js");
 const expressions_1 = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const field_length_1 = __webpack_require__(/*! ./field_length */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/field_length.js");
+const cast_1 = __webpack_require__(/*! ./cast */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/cast.js");
 class Target {
     runSyntax(node, scope, filename) {
         const concat = node.concatTokens();
@@ -26528,6 +26550,13 @@ class Target {
                 return new unknown_type_1.UnknownType(name + " unknown, Target");
             }
         }
+        else if (node.get() instanceof Expressions.Cast && node instanceof nodes_1.ExpressionNode) {
+            const ret = new cast_1.Cast().runSyntax(node, scope, undefined, filename);
+            if (ret instanceof unknown_type_1.UnknownType) {
+                throw new Error("CAST, uknown type");
+            }
+            return ret;
+        }
         return new unknown_type_1.UnknownType("unknown target type");
     }
 }
@@ -28823,6 +28852,33 @@ exports.Do = Do;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/editor_call.js":
+/*!***************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/editor_call.js ***!
+  \***************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.EditorCall = void 0;
+const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
+class EditorCall {
+    runSyntax(node, scope, filename) {
+        for (const s of node.findDirectExpressions(Expressions.Source)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
+        for (const t of node.findDirectExpressions(Expressions.SimpleSource3)) {
+            new source_1.Source().runSyntax(t, scope, filename);
+        }
+    }
+}
+exports.EditorCall = EditorCall;
+//# sourceMappingURL=editor_call.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/else_if.js":
 /*!***********************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/else_if.js ***!
@@ -32636,6 +32692,7 @@ const close_dataset_1 = __webpack_require__(/*! ./statements/close_dataset */ ".
 const get_run_time_1 = __webpack_require__(/*! ./statements/get_run_time */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_run_time.js");
 const update_database_1 = __webpack_require__(/*! ./statements/update_database */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/update_database.js");
 const add_1 = __webpack_require__(/*! ./statements/add */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/add.js");
+const editor_call_1 = __webpack_require__(/*! ./statements/editor_call */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/editor_call.js");
 const subtract_1 = __webpack_require__(/*! ./statements/subtract */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/subtract.js");
 const add_corresponding_1 = __webpack_require__(/*! ./statements/add_corresponding */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/add_corresponding.js");
 const subtract_corresponding_1 = __webpack_require__(/*! ./statements/subtract_corresponding */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/subtract_corresponding.js");
@@ -32811,6 +32868,7 @@ if (Object.keys(map).length === 0) {
     addToMap(new translate_1.Translate());
     addToMap(new modify_internal_1.ModifyInternal());
     addToMap(new read_textpool_1.ReadTextpool());
+    addToMap(new editor_call_1.EditorCall());
 }
 // -----------------------------------
 class SyntaxLogic {
@@ -40177,8 +40235,8 @@ exports.MemoryFile = MemoryFile;
 "use strict";
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.Visibility = exports.Info = exports.Rename = exports.PrettyPrinter = exports.Position = exports.CurrentScope = exports.ABAPFile = exports.RulesRunner = exports.SpaghettiScope = exports.SyntaxLogic = exports.ABAPObject = exports.Tokens = exports.ExpressionsCDS = exports.CDSParser = exports.LanguageServerTypes = exports.DDLParser = exports.applyEditList = exports.applyEditSingle = exports.SpaghettiScopeNode = exports.AbstractFile = exports.Token = exports.ScopeType = exports.BasicTypes = exports.TypedIdentifier = exports.AbstractType = exports.VirtualPosition = exports.Comment = exports.Unknown = exports.Empty = exports.Identifier = exports.Nodes = exports.Types = exports.Expressions = exports.Statements = exports.Structures = exports.SkipLogic = exports.Objects = exports.ArtifactsRules = exports.ArtifactsObjects = exports.ArtifactsABAP = exports.BuiltIn = exports.MethodLengthStats = exports.LanguageServer = exports.Registry = exports.CyclomaticComplexityStats = exports.ReferenceType = exports.Version = exports.Config = exports.Issue = exports.MemoryFile = void 0;
-exports.RuleTag = exports.Severity = void 0;
+exports.Info = exports.Diagnostics = exports.Rename = exports.PrettyPrinter = exports.Position = exports.CurrentScope = exports.ABAPFile = exports.RulesRunner = exports.SpaghettiScope = exports.SyntaxLogic = exports.ABAPObject = exports.Tokens = exports.ExpressionsCDS = exports.CDSParser = exports.LanguageServerTypes = exports.DDLParser = exports.applyEditList = exports.applyEditSingle = exports.SpaghettiScopeNode = exports.AbstractFile = exports.Token = exports.ScopeType = exports.BasicTypes = exports.TypedIdentifier = exports.AbstractType = exports.VirtualPosition = exports.Comment = exports.Unknown = exports.Empty = exports.Identifier = exports.Nodes = exports.Types = exports.Expressions = exports.Statements = exports.Structures = exports.SkipLogic = exports.Objects = exports.ArtifactsRules = exports.ArtifactsObjects = exports.ArtifactsABAP = exports.BuiltIn = exports.MethodLengthStats = exports.LanguageServer = exports.Registry = exports.CyclomaticComplexityStats = exports.ReferenceType = exports.Version = exports.Config = exports.Issue = exports.MemoryFile = void 0;
+exports.LSPEdit = exports.RuleTag = exports.Severity = exports.Visibility = void 0;
 const issue_1 = __webpack_require__(/*! ./issue */ "./node_modules/@abaplint/core/build/src/issue.js");
 Object.defineProperty(exports, "Issue", ({ enumerable: true, get: function () { return issue_1.Issue; } }));
 const config_1 = __webpack_require__(/*! ./config */ "./node_modules/@abaplint/core/build/src/config.js");
@@ -40279,6 +40337,10 @@ const cyclomatic_complexity_stats_1 = __webpack_require__(/*! ./utils/cyclomatic
 Object.defineProperty(exports, "CyclomaticComplexityStats", ({ enumerable: true, get: function () { return cyclomatic_complexity_stats_1.CyclomaticComplexityStats; } }));
 const skip_logic_1 = __webpack_require__(/*! ./skip_logic */ "./node_modules/@abaplint/core/build/src/skip_logic.js");
 Object.defineProperty(exports, "SkipLogic", ({ enumerable: true, get: function () { return skip_logic_1.SkipLogic; } }));
+const diagnostics_1 = __webpack_require__(/*! ./lsp/diagnostics */ "./node_modules/@abaplint/core/build/src/lsp/diagnostics.js");
+Object.defineProperty(exports, "Diagnostics", ({ enumerable: true, get: function () { return diagnostics_1.Diagnostics; } }));
+const _edit_1 = __webpack_require__(/*! ./lsp/_edit */ "./node_modules/@abaplint/core/build/src/lsp/_edit.js");
+Object.defineProperty(exports, "LSPEdit", ({ enumerable: true, get: function () { return _edit_1.LSPEdit; } }));
 //# sourceMappingURL=index.js.map
 
 /***/ }),
@@ -40311,8 +40373,8 @@ class Issue {
             severity,
         });
     }
-    static atStatement(file, statement, message, key, severity, fix) {
-        return this.atRange(file, statement.getStart(), statement.getEnd(), message, key, severity, fix);
+    static atStatement(file, statement, message, key, severity, fix, alternativeFixes) {
+        return this.atRange(file, statement.getStart(), statement.getEnd(), message, key, severity, fix, alternativeFixes);
     }
     static atPosition(file, start, message, key, severity, fix) {
         const row = start.getRow();
@@ -40324,7 +40386,7 @@ class Issue {
             key,
             start,
             end,
-            fix,
+            defaultFix: fix,
             severity,
         });
     }
@@ -40338,11 +40400,11 @@ class Issue {
             key,
             start,
             end,
-            fix,
+            defaultFix: fix,
             severity,
         });
     }
-    static atRange(file, start, end, message, key, severity, fix) {
+    static atRange(file, start, end, message, key, severity, fix, alternativeFixes) {
         severity = severity !== null && severity !== void 0 ? severity : severity_1.Severity.Error;
         return new Issue({
             filename: file.getFilename(),
@@ -40350,8 +40412,9 @@ class Issue {
             key,
             start,
             end,
-            fix,
+            defaultFix: fix,
             severity,
+            alternativeFixes,
         });
     }
     static atToken(file, token, message, key, severity, fix) {
@@ -40363,7 +40426,7 @@ class Issue {
             start: token.getStart(),
             end: token.getEnd(),
             severity,
-            fix,
+            defaultFix: fix,
         });
     }
     static atIdentifier(identifier, message, key, severity, fix) {
@@ -40375,14 +40438,14 @@ class Issue {
             start: identifier.getStart(),
             end: identifier.getEnd(),
             severity,
-            fix,
+            defaultFix: fix,
         });
     }
     constructor(data) {
         this.data = data;
         if (this.data.start instanceof virtual_position_1.VirtualPosition) {
             // no quick fixes inside macros
-            this.data.fix = undefined;
+            this.data.defaultFix = undefined;
         }
         if (this.data.start.getCol() < 1) {
             throw new Error("issue, start col < 1");
@@ -40409,8 +40472,11 @@ class Issue {
     getFilename() {
         return this.data.filename;
     }
-    getFix() {
-        return this.data.fix;
+    getDefaultFix() {
+        return this.data.defaultFix;
+    }
+    getAlternativeFixes() {
+        return this.data.alternativeFixes;
     }
     getSeverity() {
         return this.data.severity;
@@ -40445,22 +40511,6 @@ class LSPEdit {
         }
         return workspace;
     }
-    /*
-      public static mapEditsDocument(edit: IEdit): LServer.WorkspaceEdit {
-        const workspace: LServer.WorkspaceEdit = {documentChanges: []};
-        for (const filename in edit) {
-          const doc: LServer.VersionedTextDocumentIdentifier = {uri: filename, version: 1};
-    
-          const e = LServer.TextDocumentEdit.create(doc, this.mapText(edit[filename]));
-          workspace.documentChanges?.push(e);
-        }
-        // @ts-ignore
-        console.dir(workspace.documentChanges![0].edits[0]);
-        // @ts-ignore
-        console.dir(workspace.documentChanges![0].edits[1]);
-        return workspace;
-      }
-    */
     static mapEdit(edit) {
         const workspace = { changes: {} };
         for (const filename in edit) {
@@ -40963,27 +41013,37 @@ class CodeActions {
         const shown = new Set();
         const ret = [];
         for (const i of issues) {
-            const fix = i.getFix();
-            if (fix === undefined) {
-                continue;
-            }
-            if (totals[i.getKey()] === undefined) {
-                totals[i.getKey()] = 1;
-            }
-            else {
-                totals[i.getKey()]++;
+            const fix = i.getDefaultFix();
+            if (fix !== undefined) {
+                if (totals[i.getKey()] === undefined) {
+                    totals[i.getKey()] = 1;
+                }
+                else {
+                    totals[i.getKey()]++;
+                }
+                if (this.inRange(i, params.range) === true) {
+                    ret.push({
+                        title: "Apply fix, " + i.getKey(),
+                        kind: LServer.CodeActionKind.QuickFix,
+                        diagnostics: [diagnostics_1.Diagnostics.mapDiagnostic(i)],
+                        isPreferred: true,
+                        edit: _edit_1.LSPEdit.mapEdit(fix),
+                    });
+                    shown.add(i.getKey());
+                }
             }
-            if (this.inRange(i, params.range) === false) {
-                continue;
+            for (const alternative of i.getAlternativeFixes() || []) {
+                if (this.inRange(i, params.range) === true) {
+                    ret.push({
+                        title: alternative.description,
+                        kind: LServer.CodeActionKind.QuickFix,
+                        diagnostics: [diagnostics_1.Diagnostics.mapDiagnostic(i)],
+                        isPreferred: true,
+                        edit: _edit_1.LSPEdit.mapEdit(alternative.edit),
+                    });
+                    shown.add(i.getKey());
+                }
             }
-            ret.push({
-                title: "Apply fix, " + i.getKey(),
-                kind: LServer.CodeActionKind.QuickFix,
-                diagnostics: [diagnostics_1.Diagnostics.mapDiagnostic(i)],
-                isPreferred: true,
-                edit: _edit_1.LSPEdit.mapEdit(fix),
-            });
-            shown.add(i.getKey());
         }
         for (const s of shown) {
             if (totals[s] > 1) {
@@ -40998,7 +41058,7 @@ class CodeActions {
         const diagnostics = [];
         const fixes = [];
         for (const i of issues) {
-            const fix = i.getFix();
+            const fix = i.getDefaultFix();
             if (fix === undefined) {
                 continue;
             }
@@ -44793,15 +44853,42 @@ class EnhancementImplementation extends _abstract_object_1.AbstractObject {
     getType() {
         return "ENHO";
     }
+    setDirty() {
+        this.parsedXML = undefined;
+        super.setDirty();
+    }
     getAllowedNaming() {
         return {
             maxLength: 30,
             allowNamespace: true,
         };
     }
+    getClassName() {
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.className;
+    }
     getDescription() {
-        // todo
-        return undefined;
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.description;
+    }
+    parse() {
+        if (this.parsedXML) {
+            return { updated: false, runtime: 0 };
+        }
+        const start = Date.now();
+        this.parsedXML = {};
+        const parsed = super.parseRaw2();
+        if (parsed === undefined
+            || parsed.abapGit === undefined
+            || parsed.abapGit["asx:abap"]["asx:values"] === undefined) {
+            return { updated: false, runtime: 0 };
+        }
+        this.parsedXML.className = parsed.abapGit["asx:abap"]["asx:values"].CLASS;
+        this.parsedXML.description = parsed.abapGit["asx:abap"]["asx:values"].SHORTTEXT;
+        const end = Date.now();
+        return { updated: true, runtime: end - start };
     }
 }
 exports.EnhancementImplementation = EnhancementImplementation;
@@ -44941,6 +45028,10 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ExtensionIndex = void 0;
 const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
 class ExtensionIndex extends _abstract_object_1.AbstractObject {
+    constructor() {
+        super(...arguments);
+        this.parsedXML = undefined;
+    }
     getType() {
         return "XINX";
     }
@@ -44950,9 +45041,38 @@ class ExtensionIndex extends _abstract_object_1.AbstractObject {
             allowNamespace: true,
         };
     }
+    setDirty() {
+        this.parsedXML = undefined;
+        super.setDirty();
+    }
     getDescription() {
-        // todo
-        return undefined;
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.ddtext;
+    }
+    getTableName() {
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.sqltab;
+    }
+    parse() {
+        var _a, _b, _c, _d, _e;
+        if (this.parsedXML !== undefined) {
+            return { updated: false, runtime: 0 };
+        }
+        const start = Date.now();
+        this.parsedXML = {};
+        const parsed = super.parseRaw2();
+        if (parsed === undefined) {
+            return { updated: false, runtime: 0 };
+        }
+        const xinx = (_c = (_b = (_a = parsed.abapGit) === null || _a === void 0 ? void 0 : _a["asx:abap"]) === null || _b === void 0 ? void 0 : _b["asx:values"]) === null || _c === void 0 ? void 0 : _c.XINX;
+        this.parsedXML = {
+            sqltab: (_d = xinx === null || xinx === void 0 ? void 0 : xinx.DD12V) === null || _d === void 0 ? void 0 : _d.SQLTAB,
+            ddtext: (_e = xinx === null || xinx === void 0 ? void 0 : xinx.DD12V) === null || _e === void 0 ? void 0 : _e.DDTEXT,
+        };
+        const end = Date.now();
+        return { updated: true, runtime: end - start };
     }
 }
 exports.ExtensionIndex = ExtensionIndex;
@@ -48910,12 +49030,14 @@ class Table extends _abstract_object_1.AbstractObject {
     }
     getAllowedNaming() {
         let length = 30;
+        const regex = /^((\/[A-Z_\d]{3,8}\/)|[a-zA-Z0-9]{3})\w+$/;
         if (this.getTableCategory() === TableCategory.Transparent) {
             length = 16;
         }
         return {
             maxLength: length,
             allowNamespace: true,
+            customRegex: regex,
         };
     }
     setDirty() {
@@ -49427,15 +49549,43 @@ class Transaction extends _abstract_object_1.AbstractObject {
     getType() {
         return "TRAN";
     }
+    setDirty() {
+        this.parsedXML = undefined;
+        super.setDirty();
+    }
     getAllowedNaming() {
         return {
             maxLength: 20,
             allowNamespace: true,
         };
     }
+    getProgramName() {
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.programName;
+    }
     getDescription() {
-        // todo
-        return undefined;
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.description;
+    }
+    parse() {
+        var _a, _b;
+        if (this.parsedXML) {
+            return { updated: false, runtime: 0 };
+        }
+        const start = Date.now();
+        this.parsedXML = {};
+        const parsed = super.parseRaw2();
+        if (parsed === undefined
+            || parsed.abapGit === undefined
+            || parsed.abapGit["asx:abap"]["asx:values"] === undefined) {
+            return { updated: false, runtime: 0 };
+        }
+        this.parsedXML.description = (_a = parsed.abapGit["asx:abap"]["asx:values"].TSTCT) === null || _a === void 0 ? void 0 : _a.TTEXT;
+        this.parsedXML.programName = (_b = parsed.abapGit["asx:abap"]["asx:values"].TSTC) === null || _b === void 0 ? void 0 : _b.PGMNA;
+        const end = Date.now();
+        return { updated: true, runtime: end - start };
     }
 }
 exports.Transaction = Transaction;
@@ -50383,7 +50533,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.106.5";
+        return "2.108.8";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -50705,6 +50855,8 @@ https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abencharacter_set_gui
 
 Checkes files with extensions ".abap" and ".asddls"`,
             tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `WRITE '뽑'.`,
+            goodExample: `WRITE cl_abap_conv_in_ce=>uccp( 'BF51' ).`,
         };
     }
     initialize(_reg) {
@@ -51401,6 +51553,273 @@ exports.AlignParameters = AlignParameters;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/align_pseudo_comments.js":
+/*!******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/align_pseudo_comments.js ***!
+  \******************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.AlignPseudoComments = exports.AlignPseudoCommentsConf = void 0;
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
+const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+class AlignPseudoCommentsConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.AlignPseudoCommentsConf = AlignPseudoCommentsConf;
+class AlignPseudoComments extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new AlignPseudoCommentsConf();
+    }
+    getMetadata() {
+        return {
+            key: "align_pseudo_comments",
+            title: "Align pseudo comments",
+            shortDescription: `Align code inspector pseudo comments in statements`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix],
+            badExample: `WRITE 'sdf'. "#EC sdf`,
+            goodExample: `WRITE 'sdf'.                                                "#EC sdf`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        let previousEnd = undefined;
+        for (const statement of file.getStatements()) {
+            if (!(statement.get() instanceof _statement_1.Comment)) {
+                previousEnd = statement.getLastToken().getEnd();
+                continue;
+            }
+            const commentLength = statement.concatTokens().length;
+            const firstCommentToken = statement.getFirstToken();
+            if (firstCommentToken.getStr().startsWith(`"#`) === false) {
+                continue;
+            }
+            else if (previousEnd === undefined) {
+                continue;
+            }
+            let expectedColumn = 61;
+            if (commentLength > 10) {
+                expectedColumn = 72 - commentLength;
+            }
+            const col = firstCommentToken.getStart().getCol();
+            if (previousEnd.getCol() < expectedColumn && col !== expectedColumn) {
+                let fix = undefined;
+                if (col < expectedColumn) {
+                    fix = edit_helper_1.EditHelper.insertAt(file, firstCommentToken.getStart(), " ".repeat(expectedColumn - col));
+                }
+                else {
+                    const from = new position_1.Position(firstCommentToken.getStart().getRow(), expectedColumn);
+                    fix = edit_helper_1.EditHelper.deleteRange(file, from, firstCommentToken.getStart());
+                }
+                const message = "Align pseudo comment to column " + expectedColumn;
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, fix));
+            }
+        }
+        return issues;
+    }
+}
+exports.AlignPseudoComments = AlignPseudoComments;
+//# sourceMappingURL=align_pseudo_comments.js.map
+
+/***/ }),
+
+/***/ "./node_modules/@abaplint/core/build/src/rules/align_type_expressions.js":
+/*!*******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/align_type_expressions.js ***!
+  \*******************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.AlignTypeExpressions = exports.AlignTypeExpressionsConf = void 0;
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const ddic_1 = __webpack_require__(/*! ../ddic */ "./node_modules/@abaplint/core/build/src/ddic.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
+const Expressions = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const Structures = __webpack_require__(/*! ../abap/3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+class AlignTypeExpressionsConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** Ignore global exception classes */
+        this.ignoreExceptions = true;
+    }
+}
+exports.AlignTypeExpressionsConf = AlignTypeExpressionsConf;
+class AlignTypeExpressions extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new AlignTypeExpressionsConf();
+    }
+    getMetadata() {
+        return {
+            key: "align_type_expressions",
+            title: "Align TYPE expressions",
+            shortDescription: `Align TYPE expressions in statements`,
+            extendedInformation: `
+Currently works for METHODS + BEGIN OF
+
+If BEGIN OF has an INCLUDE TYPE its ignored
+
+Also note that clean ABAP does not recommend aligning TYPE clauses:
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-align-type-clauses`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix],
+            badExample: `
+TYPES: BEGIN OF foo,
+         bar TYPE i,
+         foobar TYPE i,
+       END OF foo.
+
+INTERFACE lif.
+  METHODS bar
+    IMPORTING
+      foo TYPE i
+      foobar TYPE i.
+ENDINTERFACE.`,
+            goodExample: `
+TYPES: BEGIN OF foo,
+         bar    TYPE i,
+         foobar TYPE i,
+       END OF foo.
+
+INTERFACE lif.
+  METHODS bar
+    IMPORTING
+      foo    TYPE i
+      foobar TYPE i.
+ENDINTERFACE.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, obj) {
+        const issues = [];
+        const stru = file.getStructure();
+        if (stru === undefined) {
+            return issues; // parser error
+        }
+        const ddic = new ddic_1.DDIC(this.reg);
+        if (obj instanceof objects_1.Class) {
+            const definition = obj.getClassDefinition();
+            if (definition === undefined) {
+                return [];
+            }
+            else if (this.conf.ignoreExceptions && ddic.isException(definition, obj)) {
+                return [];
+            }
+        }
+        issues.push(...this.checkTypes(stru, file));
+        issues.push(...this.checkMethods(stru, file));
+        return issues;
+    }
+    check(fields, column, file) {
+        const issues = [];
+        const rows = new Set();
+        for (const f of fields) {
+            const row = f.after.getRow();
+            if (rows.has(row)) {
+                return [];
+            }
+            rows.add(row);
+        }
+        for (const f of fields) {
+            if (f.after.getCol() === column) {
+                continue;
+            }
+            let fix = undefined;
+            if (f.after.getCol() < column) {
+                fix = edit_helper_1.EditHelper.insertAt(file, f.after, " ".repeat(column - f.after.getCol()));
+            }
+            else {
+                fix = edit_helper_1.EditHelper.deleteRange(file, new position_1.Position(f.after.getRow(), column), f.after);
+            }
+            const message = `Align TYPE expressions to column ${column}`;
+            const issue = issue_1.Issue.atPosition(file, f.after, message, this.getMetadata().key, this.conf.severity, fix);
+            issues.push(issue);
+        }
+        return issues;
+    }
+    checkMethods(stru, file) {
+        const issues = [];
+        const methods = stru.findAllStatements(Statements.MethodDef);
+        for (const m of methods) {
+            const fields = [];
+            const params = m.findAllExpressions(Expressions.MethodParam);
+            let column = 0;
+            for (const p of params) {
+                const children = p.getChildren();
+                const name = children[children.length - 2];
+                fields.push({
+                    nameEnd: name.getLastToken().getEnd(),
+                    after: p.findFirstExpression(Expressions.TypeParam).getFirstToken().getStart()
+                });
+                column = Math.max(column, name.getFirstToken().getEnd().getCol() + 1);
+            }
+            const ret = m.findFirstExpression(Expressions.MethodDefReturning);
+            if (ret) {
+                const children = ret.getChildren();
+                const name = children[children.length - 2];
+                fields.push({
+                    nameEnd: name.getLastToken().getEnd(),
+                    after: ret.findFirstExpression(Expressions.TypeParam).getFirstToken().getStart()
+                });
+                column = Math.max(column, name.getLastToken().getEnd().getCol() + 1);
+            }
+            issues.push(...this.check(fields, column, file));
+        }
+        return issues;
+    }
+    checkTypes(stru, file) {
+        const issues = [];
+        const types = stru.findAllStructuresRecursive(Structures.Types);
+        for (const t of types) {
+            if (t.findDirectStatement(Statements.IncludeType)) {
+                continue;
+            }
+            const fields = [];
+            let column = 0;
+            const st = t.findDirectStatements(Statements.Type);
+            for (const s of st) {
+                const name = s.getChildren()[1];
+                fields.push({
+                    nameEnd: name.getLastToken().getEnd(),
+                    after: s.getChildren()[2].getFirstToken().getStart()
+                });
+                column = Math.max(column, name.getFirstToken().getEnd().getCol() + 1);
+            }
+            issues.push(...this.check(fields, column, file));
+        }
+        return issues;
+    }
+}
+exports.AlignTypeExpressions = AlignTypeExpressions;
+//# sourceMappingURL=align_type_expressions.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/allowed_object_naming.js":
 /*!******************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/allowed_object_naming.js ***!
@@ -51932,7 +52351,8 @@ STATICS BEGIN OF bar.
 INCLUDE STRUCTURE syst.
 STATICS END OF bar.`,
             goodExample: `DATA BEGIN OF foo.
-INCLUDE STRUCTURE dselc.
+  DATA field TYPE i.
+  INCLUDE STRUCTURE dselc.
 DATA END OF foo.`,
         };
     }
@@ -52413,7 +52833,7 @@ https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenchained_statement
             }
             let prevFix;
             if (previousRow === colon.getStart().getRow()) {
-                prevFix = (_a = issues.pop()) === null || _a === void 0 ? void 0 : _a.getFix();
+                prevFix = (_a = issues.pop()) === null || _a === void 0 ? void 0 : _a.getDefaultFix();
             }
             const fix = this.getFix(file, statement, statementNode, prevFix);
             const message = "Chain mainly declarations";
@@ -52942,6 +53362,7 @@ const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 class CheckSubrcConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -52988,6 +53409,12 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
     setConfig(conf) {
         this.conf = conf;
     }
+    buildFix(file, statement) {
+        return {
+            description: "Add ##SUBRC_OK",
+            edit: edit_helper_1.EditHelper.insertAt(file, statement.getLastToken().getStart(), " ##SUBRC_OK"),
+        };
+    }
     runParsed(file) {
         const issues = [];
         const statements = file.getStatements();
@@ -53002,11 +53429,13 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
             if (config.openDataset === true
                 && statement.get() instanceof Statements.OpenDataset
                 && this.isChecked(i, statements) === false) {
+                // it doesnt make sense to ignore the subrc for open dataset, so no quick fix
                 issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
             }
             else if (config.authorityCheck === true
                 && statement.get() instanceof Statements.AuthorityCheck
                 && this.isChecked(i, statements) === false) {
+                // it doesnt make sense to ignore the subrc for authority checks, so no quick fix
                 issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
             }
             else if (config.selectSingle === true
@@ -53018,7 +53447,8 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
                 if (concat.startsWith("SELECT SINGLE @ABAP_TRUE FROM ")) {
                     continue;
                 }
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.selectTable === true
                 && statement.get() instanceof Statements.Select
@@ -53027,42 +53457,49 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
                 && statement.concatTokens().toUpperCase().startsWith("SELECT COUNT(*) ") === false
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.updateDatabase === true
                 && statement.get() instanceof Statements.UpdateDatabase
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.insertDatabase === true
                 && statement.get() instanceof Statements.InsertDatabase
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.modifyDatabase === true
                 && statement.get() instanceof Statements.ModifyDatabase
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.readTable === true
                 && statement.get() instanceof Statements.ReadTable
                 && this.isChecked(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.assign === true
                 && statement.get() instanceof Statements.Assign
                 && this.isSimpleAssign(statement) === false
                 && this.isChecked(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.find === true
                 && statement.get() instanceof Statements.Find
                 && this.isExemptedFind(statement) === false
                 && this.isChecked(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
         }
         return issues;
@@ -53787,6 +54224,7 @@ class CommentedCode extends _abap_rule_1.ABAPRule {
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#delete-code-instead-of-commenting-it
 https://docs.abapopenchecks.org/checks/14/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
+            badExample: `* WRITE 'hello world'.`,
         };
     }
     getMessage() {
@@ -54098,6 +54536,8 @@ class ContainsTab extends _abap_rule_1.ABAPRule {
 https://docs.abapopenchecks.org/checks/09/
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#indent-and-snap-to-tab`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `\tWRITE 'hello world'.`,
+            goodExample: `  WRITE 'hello world'.`,
         };
     }
     getMessage() {
@@ -55567,6 +56007,10 @@ Make sure to test the downported code, it might not always be completely correct
         if (found) {
             return found;
         }
+        found = this.downportMessageSource(high, lowFile, highSyntax);
+        if (found) {
+            return found;
+        }
         found = this.downportMessage(high, lowFile, highSyntax);
         if (found) {
             return found;
@@ -55921,6 +56365,22 @@ ${indentation}${uniqueName} = ${source.concatTokens()}.\n${indentation}`);
         }
         return undefined;
     }
+    downportMessageSource(high, lowFile, highSyntax) {
+        if (!(high.get() instanceof Statements.Message)) {
+            return undefined;
+        }
+        const source = high.findExpressionAfterToken("MESSAGE");
+        if ((source === null || source === void 0 ? void 0 : source.get()) instanceof Expressions.Source) {
+            const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
+            const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
+            const firstToken = high.getFirstToken();
+            const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, firstToken.getStart(), `DATA(${uniqueName}) = ${source.concatTokens()}.\n` + indentation);
+            const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, source.getFirstToken().getStart(), source.getLastToken().getEnd(), uniqueName);
+            const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
+            return issue_1.Issue.atToken(lowFile, high.getFirstToken(), "Outline message source", this.getMetadata().key, this.conf.severity, fix);
+        }
+        return undefined;
+    }
     replaceInsertExpression(high, lowFile, highSyntax) {
         if (!(high.get() instanceof Statements.InsertInternal)) {
             return undefined;
@@ -58255,6 +58715,8 @@ class EmptyStatement extends _abap_rule_1.ABAPRule {
             title: "Remove empty statement",
             shortDescription: `Checks for empty statements (an empty statement is a single dot)`,
             tags: [_irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
+            badExample: `WRITE 'hello world'..`,
+            goodExample: `WRITE 'hello world'.`,
         };
     }
     getConfig() {
@@ -58305,6 +58767,8 @@ class EmptyStructureConf extends _basic_rule_config_1.BasicRuleConfig {
         super(...arguments);
         /** Checks for empty LOOP blocks */
         this.loop = true;
+        /** Allow empty LOOP if subrc is checked after the loop */
+        this.loopAllowIfSubrc = true;
         /** Checks for empty IF blocks */
         this.if = true;
         /** Checks for empty WHILE blocks */
@@ -58337,6 +58801,14 @@ class EmptyStructure extends _abap_rule_1.ABAPRule {
             shortDescription: `Checks that the code does not contain empty blocks.`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#no-empty-if-branches`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `IF foo = bar.
+ENDIF.
+
+DO 2 TIMES.
+ENDDO.`,
+            goodExample: `LOOP AT itab WHERE qty = 0 OR date > sy-datum.
+ENDLOOP.
+result = xsdbool( sy-subrc = 0 ).`,
         };
     }
     getDescription(name) {
@@ -58354,15 +58826,13 @@ class EmptyStructure extends _abap_rule_1.ABAPRule {
         if (stru === undefined) {
             return [];
         }
-        for (const statement of file.getStatements()) {
+        const statements = file.getStatements();
+        for (const statement of statements) {
             if (statement.get() instanceof _statement_1.Unknown) {
                 return []; // contains parser errors
             }
         }
         const candidates = [];
-        if (this.getConfig().loop === true) {
-            candidates.push(...stru.findAllStructuresRecursive(Structures.Loop));
-        }
         if (this.getConfig().while === true) {
             candidates.push(...stru.findAllStructuresRecursive(Structures.While));
         }
@@ -58398,6 +58868,22 @@ class EmptyStructure extends _abap_rule_1.ABAPRule {
                 }
             }
         }
+        if (this.getConfig().loop === true) {
+            const loops = stru.findAllStructuresRecursive(Structures.Loop);
+            for (const loop of loops) {
+                if (loop.getChildren().length === 2) {
+                    const endloopStatement = loop.getLastChild();
+                    const endloopIndex = statements.findIndex((s) => s === endloopStatement);
+                    const afterEndloop = statements[endloopIndex + 1];
+                    if (afterEndloop !== undefined && afterEndloop.concatTokens().toUpperCase().includes("SY-SUBRC")) {
+                        continue;
+                    }
+                    const token = loop.getFirstToken();
+                    const issue = issue_1.Issue.atToken(file, token, this.getDescription(loop.get().constructor.name), this.getMetadata().key, this.conf.severity);
+                    issues.push(issue);
+                }
+            }
+        }
         if (this.getConfig().if === true) {
             const tries = stru.findAllStructuresRecursive(Structures.If)
                 .concat(stru.findAllStructuresRecursive(Structures.Else))
@@ -60643,6 +61129,24 @@ class Indentation extends _abap_rule_1.ABAPRule {
             title: "Indentation",
             shortDescription: `Checks indentation`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
+            badExample: `CLASS lcl DEFINITION.
+PRIVATE SECTION.
+METHODS constructor.
+ENDCLASS.
+
+CLASS lcl IMPLEMENTATION.
+METHOD constructor.
+ENDMETHOD.
+ENDCLASS.`,
+            goodExample: `CLASS lcl DEFINITION.
+  PRIVATE SECTION.
+    METHODS constructor.
+ENDCLASS.
+
+CLASS lcl IMPLEMENTATION.
+  METHOD constructor.
+  ENDMETHOD.
+ENDCLASS.`,
         };
     }
     getConfig() {
@@ -60754,6 +61258,8 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 __exportStar(__webpack_require__(/*! ./7bit_ascii */ "./node_modules/@abaplint/core/build/src/rules/7bit_ascii.js"), exports);
 __exportStar(__webpack_require__(/*! ./abapdoc */ "./node_modules/@abaplint/core/build/src/rules/abapdoc.js"), exports);
 __exportStar(__webpack_require__(/*! ./align_parameters */ "./node_modules/@abaplint/core/build/src/rules/align_parameters.js"), exports);
+__exportStar(__webpack_require__(/*! ./align_pseudo_comments */ "./node_modules/@abaplint/core/build/src/rules/align_pseudo_comments.js"), exports);
+__exportStar(__webpack_require__(/*! ./align_type_expressions */ "./node_modules/@abaplint/core/build/src/rules/align_type_expressions.js"), exports);
 __exportStar(__webpack_require__(/*! ./allowed_object_naming */ "./node_modules/@abaplint/core/build/src/rules/allowed_object_naming.js"), exports);
 __exportStar(__webpack_require__(/*! ./allowed_object_types */ "./node_modules/@abaplint/core/build/src/rules/allowed_object_types.js"), exports);
 __exportStar(__webpack_require__(/*! ./ambiguous_statement */ "./node_modules/@abaplint/core/build/src/rules/ambiguous_statement.js"), exports);
@@ -61455,6 +61961,8 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
             shortDescription: `Checks that keywords have the same case. Non-keywords must be lower case.`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-your-pretty-printer-team-settings`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
+            badExample: `write 'hello world'.`,
+            goodExample: `WRITE 'hello world'.`,
         };
     }
     getConfig() {
@@ -61542,8 +62050,11 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
     /** returns a list of tokens which violates the keyword_case rule */
     traverse(s, parent) {
         let ret = [];
-        for (const child of s.getChildren()) {
+        const children = s.getChildren();
+        for (let i = 0; i < children.length; i++) {
+            const child = children[i];
             if (child instanceof nodes_1.TokenNodeRegex) {
+                const next = children[i + 1];
                 if (this.conf.ignoreLowerClassImplmentationStatement
                     && parent instanceof Statements.ClassImplementation) {
                     continue;
@@ -61551,7 +62062,7 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
                 const str = child.get().getStr();
                 const upper = str.toUpperCase();
                 // todo, this is a hack, the parser should recongize OTHERS/TEXT as a keyword
-                if (upper === "OTHERS" || upper === "TEXT") {
+                if (upper === "OTHERS" || (upper === "TEXT" && (next === null || next === void 0 ? void 0 : next.concatTokens()) === "-")) {
                     continue;
                 }
                 if (this.conf.ignoreFunctionModuleName === true
@@ -62116,6 +62627,12 @@ class LocalVariableNames extends _abap_rule_1.ABAPRule {
 Allows you to enforce a pattern, such as a prefix, for local variables, constants and field symbols.
 Regexes are case-insensitive.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
+            badExample: `FORM bar.
+  DATA foo.
+ENDFORM.`,
+            goodExample: `FORM bar.
+  DATA lv_foo.
+ENDFORM.`,
         };
     }
     getDescription(expected, actual) {
@@ -63089,6 +63606,15 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obscu
 
 Interface method names are ignored`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
+            badExample: `CLASS lcl DEFINITION.
+  PUBLIC SECTION.
+    METHODS matches.
+ENDCLASS.
+
+CLASS lcl IMPLEMENTATION.
+  METHOD matches.
+  ENDMETHOD.
+ENDCLASS.`,
         };
     }
     getConfig() {
@@ -63278,6 +63804,13 @@ class MixReturning extends _abap_rule_1.ABAPRule {
             // eslint-disable-next-line max-len
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-either-returning-or-exporting-or-changing-but-not-a-combination`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `CLASS lcl DEFINITION.
+  PUBLIC SECTION.
+    METHODS
+      foobar
+        EXPORTING foo TYPE i
+        RETURNING VALUE(rv_string) TYPE string.
+ENDCLASS.`,
         };
     }
     getMessage() {
@@ -64109,6 +64642,7 @@ class NoPrefixesConf extends _basic_rule_config_1.BasicRuleConfig {
         this.types = "^TY_";
         /** importing, exporting, returning and changing parameters, case insensitive regex */
         this.methodParameters = "^[ICER].?_";
+        this.allowIsPrefixBoolean = true;
         // todo, public localClass: string = "";
         // todo, public localInterface: string = "";
         // todo, public functionModuleParameters: string = "";
@@ -64253,12 +64787,18 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/sub-sections/AvoidEncodi
         return ret;
     }
     checkMethodParameters(topNode, regex, file) {
+        var _a, _b;
         const ret = [];
         for (const method of topNode.findAllStatements(Statements.MethodDef)) {
-            for (const def of method.findAllExpressions(Expressions.MethodParamName)) {
-                const name = def.concatTokens();
-                if (name !== "" && name.match(regex)) {
-                    const issue = issue_1.Issue.atToken(file, def.getFirstToken(), MESSAGE, this.getMetadata().key, this.conf.severity);
+            for (const param of method.findAllExpressionsMulti([Expressions.MethodDefReturning, Expressions.MethodParam])) {
+                const nameToken = param === null || param === void 0 ? void 0 : param.findFirstExpression(Expressions.MethodParamName);
+                const type = (_b = (_a = param === null || param === void 0 ? void 0 : param.findFirstExpression(Expressions.TypeParam)) === null || _a === void 0 ? void 0 : _a.concatTokens()) === null || _b === void 0 ? void 0 : _b.toUpperCase();
+                if (this.getConfig().allowIsPrefixBoolean === true && (type === null || type === void 0 ? void 0 : type.endsWith("TYPE ABAP_BOOL"))) {
+                    continue;
+                }
+                const name = nameToken === null || nameToken === void 0 ? void 0 : nameToken.concatTokens();
+                if (nameToken && name && name !== "" && name.match(regex)) {
+                    const issue = issue_1.Issue.atToken(file, nameToken.getFirstToken(), MESSAGE, this.getMetadata().key, this.conf.severity);
                     ret.push(issue);
                 }
             }
@@ -66567,6 +67107,10 @@ class PreferReturningToExporting extends _abap_rule_1.ABAPRule {
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-returning-to-exporting
 https://docs.abapopenchecks.org/checks/44/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `CLASS lcl DEFINITION.
+  PUBLIC SECTION.
+    METHODS test EXPORTING ev_foo TYPE i.
+ENDCLASS.`,
         };
     }
     getConfig() {
@@ -66734,6 +67278,10 @@ class PreferredCompareOperator extends _abap_rule_1.ABAPRule {
             title: "Preferred compare operator",
             shortDescription: `Configure undesired operator variants`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
+            badExample: `IF foo EQ bar.
+ENDIF.`,
+            goodExample: `IF foo = bar.
+ENDIF.`,
         };
     }
     getDescription(operator) {
@@ -67447,6 +67995,8 @@ add ORDER BY PRIMARY KEY if in doubt
 
 If the target is a sorted/hashed table, no issue is reported`,
             tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `SELECT * FROM db INTO TABLE @DATA(tab).`,
+            goodExample: `SELECT * FROM db INTO TABLE @DATA(tab) ORDER BY PRIMARY KEY.`,
         };
     }
     getConfig() {
@@ -69464,6 +70014,8 @@ class TypesNaming extends _abap_rule_1.ABAPRule {
             shortDescription: `Allows you to enforce a pattern for TYPES definitions`,
             extendedInformation: `Does not run for TYPE POOLS`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
+            badExample: `TYPES foo TYPE i.`,
+            goodExample: `TYPES ty_foo TYPE i.`,
         };
     }
     getConfig() {
@@ -70648,6 +71200,7 @@ Skips:
 `,
             tags: [],
             pragma: "##CALLED",
+            pseudoComment: "EC CALLED",
         };
     }
     getConfig() {
@@ -70724,11 +71277,32 @@ Skips:
             if (statement.getPragmas().some(t => t.getStr() === this.getMetadata().pragma)) {
                 continue;
             }
+            else if (this.suppressedbyPseudo(statement, file)) {
+                continue;
+            }
             const message = "Method \"" + i.identifier.getName() + "\" not used";
             issues.push(issue_1.Issue.atIdentifier(i.identifier, message, this.getMetadata().key, this.conf.severity));
         }
         return issues;
     }
+    suppressedbyPseudo(statement, file) {
+        if (statement === undefined) {
+            return false;
+        }
+        if (file === undefined) {
+            return false;
+        }
+        let next = false;
+        for (const s of file.getStatements()) {
+            if (next === true && s.get() instanceof _statement_1.Comment) {
+                return s.concatTokens().includes(this.getMetadata().pseudoComment + "");
+            }
+            if (s === statement) {
+                next = true;
+            }
+        }
+        return false;
+    }
     searchGlobalSubclasses(obj) {
         var _a, _b;
         if (this.wa.getLength() === 0
@@ -70839,6 +71413,7 @@ class UnusedTypes {
             extendedInformation: `Unused types are not reported if the object contains parser or syntax errors.`,
             tags: [_irule_1.RuleTag.Quickfix],
             pragma: "##NEEDED",
+            pseudoComment: "EC NEEDED",
         };
     }
     getConfig() {
@@ -70906,11 +71481,32 @@ class UnusedTypes {
             if (statement.getPragmas().some(t => t.getStr() === this.getMetadata().pragma)) {
                 continue;
             }
+            else if (this.suppressedbyPseudo(statement, file)) {
+                continue;
+            }
             const fix = this.buildFix(file, statement);
             ret.push(issue_1.Issue.atIdentifier(t, message, this.getMetadata().key, this.conf.severity, fix));
         }
         return ret;
     }
+    suppressedbyPseudo(statement, file) {
+        if (statement === undefined) {
+            return false;
+        }
+        if (file === undefined) {
+            return false;
+        }
+        let next = false;
+        for (const s of file.getStatements()) {
+            if (next === true && s.get() instanceof _statement_1.Comment) {
+                return s.concatTokens().includes(this.getMetadata().pseudoComment + "");
+            }
+            if (s === statement) {
+                next = true;
+            }
+        }
+        return false;
+    }
     ////////////////////////////
     traverse(node, obj, add) {
         if (node.getIdentifier().stype !== _scope_type_1.ScopeType.BuiltIn) {
@@ -71395,6 +71991,16 @@ class UseClassBasedExceptions extends _abap_rule_1.ABAPRule {
             shortDescription: `Use class based exceptions, checks interface and class definitions`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-class-based-exceptions`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `INTERFACE lif.
+  METHODS load_data
+    EXCEPTIONS
+      invalid_parameter.
+ENDINTERFACE.`,
+            goodExample: `INTERFACE lif.
+  METHODS load_data
+    RAISING
+      cx_something.
+ENDINTERFACE.`,
         };
     }
     getMessage() {