@abaplint/transpiler-cli 2.8.24 → 2.8.26

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;
@@ -1920,7 +1921,7 @@ class Star {
         return this.sta.getUsing();
     }
     run(r) {
-        const result = r;
+        let result = r;
         try {
             let res = r;
             let input = [];
@@ -1930,7 +1931,13 @@ class Star {
                 if (res.length === 0) {
                     break;
                 }
-                result.push(...res);
+                if (res.length > 1000) {
+                    // avoid stack overflow
+                    result = result.concat(res);
+                }
+                else {
+                    result.push(...res);
+                }
             }
         }
         catch (err) {
@@ -2063,7 +2070,7 @@ class Sequence {
         return this.list.reduce((a, c) => { return a.concat(c.getUsing()); }, []);
     }
     run(r) {
-        const result = [];
+        let result = [];
         for (const input of r) {
             let temp = [input];
             let match = true;
@@ -2075,7 +2082,13 @@ class Sequence {
                 }
             }
             if (match === true) {
-                result.push(...temp);
+                if (temp.length > 1000) {
+                    // avoid stack overflow
+                    result = result.concat(temp);
+                }
+                else {
+                    result.push(...temp);
+                }
             }
         }
         return result;
@@ -7831,9 +7844,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 +10838,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);
     }
 }
@@ -16900,7 +16912,7 @@ class Sequence {
     }
     run(statements, parent) {
         let inn = statements;
-        const out = [];
+        let out = [];
         for (const i of this.list) {
             const match = i.run(inn, parent);
             if (match.error) {
@@ -16912,7 +16924,14 @@ class Sequence {
                     errorMatched: out.length,
                 };
             }
-            out.push(...match.matched);
+            if (match.matched.length < 100) {
+                out.push(...match.matched);
+            }
+            else {
+                // avoid using the spread operator, it might trigger "Maximum call stack size exceeded"
+                // when the number of matched elements is very large
+                out = out.concat(match.matched);
+            }
             inn = match.unmatched;
         }
         return {
@@ -17044,7 +17063,7 @@ class Star {
     }
     run(statements, parent) {
         let inn = statements;
-        const out = [];
+        let out = [];
         while (true) {
             if (inn.length === 0) {
                 return {
@@ -17076,7 +17095,14 @@ class Star {
                     };
                 }
             }
-            out.push(...match.matched);
+            if (match.matched.length < 100) {
+                out.push(...match.matched);
+            }
+            else {
+                // avoid using the spread operator, it might trigger "Maximum call stack size exceeded"
+                // when the number of matched elements is very large
+                out = out.concat(match.matched);
+            }
             inn = match.unmatched;
         }
     }
@@ -21617,6 +21643,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 +22885,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 +22912,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 +25470,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 +25481,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 +25639,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 +26094,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 +26131,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 +26444,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 +26576,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 +28878,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 +32718,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 +32894,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 +40261,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 +40363,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 +40399,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 +40412,7 @@ class Issue {
             key,
             start,
             end,
-            fix,
+            defaultFix: fix,
             severity,
         });
     }
@@ -40338,11 +40426,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 +40438,9 @@ class Issue {
             key,
             start,
             end,
-            fix,
+            defaultFix: fix,
             severity,
+            alternativeFixes,
         });
     }
     static atToken(file, token, message, key, severity, fix) {
@@ -40363,7 +40452,7 @@ class Issue {
             start: token.getStart(),
             end: token.getEnd(),
             severity,
-            fix,
+            defaultFix: fix,
         });
     }
     static atIdentifier(identifier, message, key, severity, fix) {
@@ -40375,14 +40464,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 +40498,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 +40537,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 +41039,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 +41084,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 +44879,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 +45054,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 +45067,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 +49056,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 +49575,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;
@@ -49557,6 +49733,10 @@ const Types = __webpack_require__(/*! ../abap/types/basic */ "./node_modules/@ab
 const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
 const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
 const ddic_1 = __webpack_require__(/*! ../ddic */ "./node_modules/@abaplint/core/build/src/ddic.js");
+var ViewClass;
+(function (ViewClass) {
+    ViewClass["ExternalView"] = "X";
+})(ViewClass || (ViewClass = {}));
 class View extends _abstract_object_1.AbstractObject {
     getType() {
         return "VIEW";
@@ -49600,6 +49780,16 @@ class View extends _abstract_object_1.AbstractObject {
                 // ignore, this is a special case of old style .INCLUDE
                 continue;
             }
+            else if (this.parsedData.header.VIEWCLASS === ViewClass.ExternalView) {
+                components.push({
+                    name: field.VIEWFIELD,
+                    type: new Types.VoidType("ExternalView")
+                });
+                continue;
+            }
+            else if (field.TABNAME === this.getName()) {
+                throw new Error("Unexpected self reference in view " + this.getName() + ", " + field.FIELDNAME + " " + field.FIELDNAME);
+            }
             const lookup = ddic.lookupTableOrView(field.TABNAME);
             let found = lookup.type;
             if (lookup.object) {
@@ -49635,13 +49825,23 @@ class View extends _abstract_object_1.AbstractObject {
     }
     ///////////////
     parseXML() {
-        var _a, _b;
-        this.parsedData = { fields: [], join: [] };
+        var _a, _b, _c;
+        this.parsedData = {
+            header: {
+                VIEWCLASS: "",
+            },
+            fields: [],
+            join: [],
+        };
         const parsed = super.parseRaw2();
         if (parsed === undefined || parsed.abapGit === undefined) {
             return;
         }
-        const fields = (_a = parsed.abapGit["asx:abap"]["asx:values"]) === null || _a === void 0 ? void 0 : _a.DD27P_TABLE;
+        const header = (_a = parsed.abapGit["asx:abap"]["asx:values"]) === null || _a === void 0 ? void 0 : _a.DD25V;
+        this.parsedData.header = {
+            VIEWCLASS: (header === null || header === void 0 ? void 0 : header.VIEWCLASS) || "",
+        };
+        const fields = (_b = parsed.abapGit["asx:abap"]["asx:values"]) === null || _b === void 0 ? void 0 : _b.DD27P_TABLE;
         for (const field of (0, xml_utils_1.xmlToArray)(fields === null || fields === void 0 ? void 0 : fields.DD27P)) {
             this.parsedData.fields.push({
                 VIEWFIELD: field.VIEWFIELD,
@@ -49649,7 +49849,7 @@ class View extends _abstract_object_1.AbstractObject {
                 FIELDNAME: field.FIELDNAME,
             });
         }
-        const join = (_b = parsed.abapGit["asx:abap"]["asx:values"]) === null || _b === void 0 ? void 0 : _b.DD28J_TABLE;
+        const join = (_c = parsed.abapGit["asx:abap"]["asx:values"]) === null || _c === void 0 ? void 0 : _c.DD28J_TABLE;
         for (const j of (0, xml_utils_1.xmlToArray)(join === null || join === void 0 ? void 0 : join.DD28J)) {
             this.parsedData.join.push({
                 LTAB: j.LTAB,
@@ -50383,7 +50583,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.106.5";
+        return "2.108.12";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -50705,6 +50905,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 +51603,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 +52401,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 +52883,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 +53412,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);
@@ -52977,7 +53448,7 @@ If sy-dbcnt is checked after database statements, it is considered okay.
 If IS ASSIGNED is checked after assigning, it is considered okay.
 
 FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
-            tags: [_irule_1.RuleTag.SingleFile],
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             pseudoComment: "EC CI_SUBRC",
             pragma: "##SUBRC_OK",
         };
@@ -52988,6 +53459,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 +53479,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 +53497,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 +53507,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 +54274,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 +54586,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 +56057,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 +56415,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 +58765,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 +58817,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 +58851,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 +58876,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 +58918,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 +61179,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 +61308,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 +62011,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 +62100,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 +62112,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 +62677,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 +63656,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 +63854,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 +64692,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 +64837,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 +67157,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 +67328,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 +68045,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() {
@@ -67675,7 +68275,9 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.SelectSingleFullKey = exports.SelectSingleFullKeyConf = void 0;
 const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.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 __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 class SelectSingleFullKeyConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -67696,7 +68298,7 @@ class SelectSingleFullKey {
 
 If the statement contains a JOIN it is not checked`,
             pseudoComment: "EC CI_NOORDER",
-            tags: [],
+            tags: [_irule_1.RuleTag.Quickfix],
         };
     }
     initialize(reg) {
@@ -67717,6 +68319,12 @@ If the statement contains a JOIN it is not checked`,
     setConfig(conf) {
         this.conf = conf;
     }
+    buildFix(file, statement) {
+        return {
+            description: `Add "#EC CI_NOORDER`,
+            edit: edit_helper_1.EditHelper.insertAt(file, statement.getLastToken().getStart(), ` "#EC CI_NOORDER`),
+        };
+    }
     run(obj) {
         var _a, _b;
         if (!(obj instanceof __1.ABAPObject)) {
@@ -67778,7 +68386,8 @@ If the statement contains a JOIN it is not checked`,
                     }
                 }
                 if (set.size > 0) {
-                    issues.push(issue_1.Issue.atStatement(file, s, message, this.getMetadata().key, this.getConfig().severity));
+                    const fix = this.buildFix(file, s);
+                    issues.push(issue_1.Issue.atStatement(file, s, message, this.getMetadata().key, this.getConfig().severity, undefined, [fix]));
                 }
             }
         }
@@ -69464,6 +70073,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() {
@@ -70241,7 +70852,13 @@ const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 class UnnecessaryReturnConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** Allow empty METHODs + FORMs + FUNCTION-MODULEs */
+        this.allowEmpty = false;
+    }
 }
 exports.UnnecessaryReturnConf = UnnecessaryReturnConf;
 class UnnecessaryReturn extends _abap_rule_1.ABAPRule {
@@ -70285,15 +70902,29 @@ ENDFORM.`,
         }
         const message = "Unnecessary RETURN";
         const statements = file.getStatements();
+        let statementCounter = 0;
         for (let i = 0; i < statements.length; i++) {
             const node = statements[i];
-            if (!(node.get() instanceof Statements.EndMethod
-                || node.get() instanceof Statements.EndForm
-                || node.get() instanceof Statements.EndFunction)) {
+            const nodeType = node.get();
+            if ((nodeType instanceof Statements.MethodImplementation
+                || nodeType instanceof Statements.Form
+                || nodeType instanceof Statements.FunctionModule)) {
+                statementCounter = 0;
+                continue;
+            }
+            if (!(nodeType instanceof _statement_1.Comment)) {
+                statementCounter++;
+            }
+            if (!(nodeType instanceof Statements.EndMethod
+                || nodeType instanceof Statements.EndForm
+                || nodeType instanceof Statements.EndFunction)) {
                 continue;
             }
             const prev = statements[i - 1];
             if (prev && prev.get() instanceof Statements.Return) {
+                if (this.conf.allowEmpty === true && statementCounter === 2) {
+                    continue;
+                }
                 const fix = edit_helper_1.EditHelper.deleteStatement(file, prev);
                 issues.push(issue_1.Issue.atStatement(file, prev, message, this.getMetadata().key, this.getConfig().severity, fix));
             }
@@ -70648,6 +71279,7 @@ Skips:
 `,
             tags: [],
             pragma: "##CALLED",
+            pseudoComment: "EC CALLED",
         };
     }
     getConfig() {
@@ -70724,11 +71356,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 +71492,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 +71560,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 +72070,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() {
@@ -76945,12 +77630,12 @@ class Transpiler {
         progress?.set(reg.getObjectCount(false), "Building");
         for (const obj of reg.getObjects()) {
             await progress?.tick("Building, " + obj.getName());
-            if (obj instanceof abaplint.ABAPObject && !(obj instanceof abaplint.Objects.TypePool)) {
-                output.objects.push(...new handle_abap_1.HandleABAP(this.options).runObject(obj, reg));
-            }
-            else if (obj instanceof abaplint.Objects.TypePool) {
+            if (obj instanceof abaplint.Objects.TypePool) {
                 output.objects.push(...new handle_type_pool_1.HandleTypePool().runObject(obj, reg));
             }
+            else if (obj instanceof abaplint.ABAPObject) {
+                output.objects.push(...new handle_abap_1.HandleABAP(this.options).runObject(obj, reg));
+            }
             else if (obj instanceof abaplint.Objects.Oauth2Profile) {
                 output.objects.push(...new handle_oa2p_1.HandleOA2P().runObject(obj, reg));
             }
@@ -85227,28 +85912,29 @@ async function run() {
         for (const st of this.getSortedTests(reg)) {
             ret += `// --------------------------------------------\n`;
             ret += `    clas = unit.addObject("${st.obj.getName()}");\n`;
+            const lc = st.localClass.toLowerCase();
             ret += `    {
-        const {${st.localClass}} = await import("./${this.escapeNamespace(st.obj.getName().toLowerCase())}.${st.obj.getType().toLowerCase()}.testclasses.mjs");
-        locl = clas.addTestClass("${st.localClass}");
-        if (${st.localClass}.class_setup) await ${st.localClass}.class_setup();\n`;
+        const {${lc}} = await import("./${this.escapeNamespace(st.obj.getName().toLowerCase())}.${st.obj.getType().toLowerCase()}.testclasses.mjs");
+        locl = clas.addTestClass("${lc}");
+        if (${lc}.class_setup) await ${lc}.class_setup();\n`;
             for (const m of st.methods) {
-                const skipThis = (skip || []).some(a => a.object === st.obj.getName() && a.class === st.localClass && a.method === m);
+                const skipThis = (skip || []).some(a => a.object === st.obj.getName() && a.class === lc && a.method === m);
                 if (skipThis) {
-                    ret += `  console.log('${st.obj.getName()}: running ${st.localClass}->${m}, skipped');\n`;
+                    ret += `  console.log('${st.obj.getName()}: running ${lc}->${m}, skipped');\n`;
                     ret += `  meth = locl.addMethod("${m}");\n`;
                     ret += `  meth.skip();\n`;
                     continue;
                 }
-                ret += `      {\n        const test = await (new ${st.localClass}()).constructor_();\n`;
+                ret += `      {\n        const test = await (new ${lc}()).constructor_();\n`;
                 ret += `        if (test.setup) await test.setup();\n`;
-                ret += `        console.log("${st.obj.getName()}: running ${st.localClass}->${m}");\n`;
+                ret += `        console.log("${st.obj.getName()}: running ${lc}->${m}");\n`;
                 ret += `        meth = locl.addMethod("${m}");\n`;
                 ret += `        await test.${m}();\n`;
                 ret += `        meth.pass();\n`;
                 ret += `        if (test.teardown) await test.teardown();\n`;
                 ret += `      }\n`;
             }
-            ret += `      if (${st.localClass}.class_teardown) await ${st.localClass}.class_teardown();\n`;
+            ret += `      if (${lc}.class_teardown) await ${lc}.class_teardown();\n`;
             ret += `    }\n`;
         }
         /*
@@ -86070,6 +86756,8 @@ exports.validate = function (xmlData, options) {
             return getErrorObject('InvalidTag', "Closing tag '"+tagName+"' doesn't have proper closing.", getLineNumberForPosition(xmlData, i));
           } else if (attrStr.trim().length > 0) {
             return getErrorObject('InvalidTag', "Closing tag '"+tagName+"' can't have attributes or invalid starting.", getLineNumberForPosition(xmlData, tagStartPos));
+          } else if (tags.length === 0) {
+            return getErrorObject('InvalidTag', "Closing tag '"+tagName+"' has not been opened.", getLineNumberForPosition(xmlData, tagStartPos));
           } else {
             const otg = tags.pop();
             if (tagName !== otg.tagName) {
@@ -87357,10 +88045,18 @@ const parseXml = function(xmlData) {
           let tagContent = "";
           //self-closing tag
           if(tagExp.length > 0 && tagExp.lastIndexOf("/") === tagExp.length - 1){
+            if(tagName[tagName.length - 1] === "/"){ //remove trailing '/'
+              tagName = tagName.substr(0, tagName.length - 1);
+              jPath = jPath.substr(0, jPath.length - 1);
+              tagExp = tagName;
+            }else{
+              tagExp = tagExp.substr(0, tagExp.length - 1);
+            }
             i = result.closeIndex;
           }
           //unpaired tag
           else if(this.options.unpairedTags.indexOf(tagName) !== -1){
+            
             i = result.closeIndex;
           }
           //normal tag