@abaplint/transpiler-cli 2.10.10 → 2.10.12

package/build/bundle.js

@@ -6428,7 +6428,7 @@ class SelectLoop extends combi_1.Expression {
         const packTab = (0, combi_1.seq)(pack, _1.SQLIntoTable);
         const into = (0, combi_1.altPrio)(sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList);
         const perm = (0, combi_1.per)(_1.SQLFrom, where, sql_up_to_1.SQLUpTo, sql_order_by_1.SQLOrderBy, sql_having_1.SQLHaving, _1.SQLClient, bypass, _1.SQLGroupBy, _1.SQLForAllEntries, (0, combi_1.alt)(tab, into, packTab));
-        const strict = (0, combi_1.seq)(_1.SQLFrom, (0, combi_1.ver)(version_1.Version.v750, _1.SQLFields), where, into, sql_up_to_1.SQLUpTo);
+        const strict = (0, combi_1.seq)(_1.SQLFrom, (0, combi_1.ver)(version_1.Version.v750, _1.SQLFields), (0, combi_1.optPrio)((0, combi_1.seq)(where, into, sql_up_to_1.SQLUpTo)));
         const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), sql_field_list_loop_1.SQLFieldListLoop, perm), strict), (0, combi_1.optPrio)(sql_hints_1.SQLHints));
         return ret;
     }
@@ -34140,36 +34140,18 @@ exports.FlowGraph = FlowGraph;
 
 /***/ }),
 
-/***/ "./node_modules/@abaplint/core/build/src/abap/flow/statement_flow.js":
-/*!***************************************************************************!*\
-  !*** ./node_modules/@abaplint/core/build/src/abap/flow/statement_flow.js ***!
-  \***************************************************************************/
+/***/ "./node_modules/@abaplint/core/build/src/abap/flow/selection_events.js":
+/*!*****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/flow/selection_events.js ***!
+  \*****************************************************************************/
 /***/ ((__unused_webpack_module, exports, __webpack_require__) => {
 
 "use strict";
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.StatementFlow = void 0;
-const nodes_1 = __webpack_require__(/*! ../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
-const Structures = __webpack_require__(/*! ../3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
+exports.DECLARATION_STUFF = exports.SELECTION_EVENTS = void 0;
 const Statements = __webpack_require__(/*! ../2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
-const Expressions = __webpack_require__(/*! ../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
-const flow_graph_1 = __webpack_require__(/*! ./flow_graph */ "./node_modules/@abaplint/core/build/src/abap/flow/flow_graph.js");
-// Levels: top, FORM, METHOD, FUNCTION-MODULE, (MODULE, AT, END-OF-*, GET, START-OF-SELECTION, TOP-OF-PAGE)
-//
-// Loop branching: LOOP, DO, WHILE, SELECT(loop), WITH, PROVIDE
-//
-// Branching: IF, CASE, CASE TYPE OF, TRY, ON, CATCH SYSTEM-EXCEPTIONS, AT
-//
-// Conditional exits: CHECK, ASSERT
-//
-// Exits: RETURN, EXIT, RAISE(not RESUMABLE), MESSAGE(type E and A?), CONTINUE, REJECT, RESUME, STOP
-//
-// Not handled? INCLUDE + malplaced macro calls
-/////////////////////////////////////
-// TODO: handling static exceptions(only static), refactor some logic from UncaughtException to common file
-// TODO: RAISE
-const FLOW_EVENTS = [
+exports.SELECTION_EVENTS = [
     Statements.StartOfSelection,
     Statements.AtSelectionScreen,
     Statements.AtLineSelection,
@@ -34177,17 +34159,51 @@ const FLOW_EVENTS = [
     Statements.EndOfSelection,
     Statements.Initialization,
     Statements.TopOfPage,
+    Statements.LoadOfProgram,
     Statements.EndOfPage,
 ];
+exports.DECLARATION_STUFF = [
+    Statements.Data,
+    Statements.DataBegin,
+    Statements.Constant,
+    Statements.Tables,
+    Statements.Include, // this is not super correct, but anyhow
+    Statements.Parameter,
+    Statements.SelectionScreen,
+    Statements.ConstantBegin,
+];
+//# sourceMappingURL=selection_events.js.map
+
+/***/ }),
+
+/***/ "./node_modules/@abaplint/core/build/src/abap/flow/statement_flow.js":
+/*!***************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/flow/statement_flow.js ***!
+  \***************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.StatementFlow = void 0;
+const nodes_1 = __webpack_require__(/*! ../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
+const Structures = __webpack_require__(/*! ../3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
+const Statements = __webpack_require__(/*! ../2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const Expressions = __webpack_require__(/*! ../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const flow_graph_1 = __webpack_require__(/*! ./flow_graph */ "./node_modules/@abaplint/core/build/src/abap/flow/flow_graph.js");
+const objects_1 = __webpack_require__(/*! ../../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const selection_events_1 = __webpack_require__(/*! ./selection_events */ "./node_modules/@abaplint/core/build/src/abap/flow/selection_events.js");
 class StatementFlow {
     constructor() {
         this.counter = 0;
     }
-    build(stru) {
+    build(stru, obj) {
         var _a, _b, _c, _d;
         const ret = [];
         let name = "";
-        const structures = stru.findAllStructuresMulti([Structures.Form, Structures.ClassImplementation, Structures.FunctionModule]);
+        const structures = stru.findAllStructuresMulti([
+            Structures.Form, Structures.ClassImplementation, Structures.FunctionModule, Structures.Module
+        ]);
         for (const s of structures) {
             if (s.get() instanceof Structures.Form) {
                 name = "FORM " + ((_a = s.findFirstExpression(Expressions.FormName)) === null || _a === void 0 ? void 0 : _a.concatTokens());
@@ -34205,40 +34221,46 @@ class StatementFlow {
                 name = "FUNCTION " + ((_d = s.findFirstExpression(Expressions.Field)) === null || _d === void 0 ? void 0 : _d.concatTokens());
                 ret.push(this.run(s, name));
             }
+            else if (s.get() instanceof Structures.Module) {
+                name = s.getFirstStatement().concatTokens().toUpperCase();
+                ret.push(this.run(s, name));
+            }
             else {
                 throw new Error("StatementFlow, unknown structure");
             }
         }
-        // find the top level events
-        let inFlow = false;
-        let collected = [];
-        for (const s of stru.getChildren() || []) {
-            if (FLOW_EVENTS.some(f => s.get() instanceof f)) {
-                if (inFlow === true) {
-                    ret.push(this.runEvent(collected, name));
+        if (obj instanceof objects_1.Program) {
+            // find the top level events
+            let inEvent = false;
+            let collected = [];
+            for (const s of stru.getChildren() || []) {
+                if (selection_events_1.SELECTION_EVENTS.some(f => s.get() instanceof f)) {
+                    if (inEvent === true) {
+                        ret.push(this.runEvent(collected, name));
+                    }
+                    collected = [];
+                    inEvent = true;
+                    name = s.concatTokens();
+                }
+                else if (s.get() instanceof Structures.Normal) {
+                    collected.push(s);
+                }
+                else {
+                    if (inEvent === true) {
+                        ret.push(this.runEvent(collected, name));
+                        inEvent = false;
+                    }
+                    collected = [];
                 }
-                collected = [];
-                inFlow = true;
-                name = s.concatTokens();
             }
-            else if (s.get() instanceof Structures.Normal) {
-                collected.push(s);
+            if (inEvent === true) {
+                ret.push(this.runEvent(collected, name));
             }
-            else {
-                if (inFlow === true) {
-                    ret.push(this.runEvent(collected, name));
-                    inFlow = false;
-                }
-                collected = [];
+            else if (collected.length > 0) {
+                // implicit START-OF-SELECTION
+                ret.push(this.runEvent(collected, "START-OF-SELECTION."));
             }
         }
-        if (inFlow === true) {
-            ret.push(this.runEvent(collected, name));
-        }
-        else if (collected.length > 0) {
-            // implicit START-OF-SELECTION
-            ret.push(this.runEvent(collected, "START-OF-SELECTION."));
-        }
         return ret.map(f => f.reduce());
     }
     ////////////////////
@@ -34385,6 +34407,7 @@ class StatementFlow {
             graph.addEdge(loopName, graph.getEnd(), flow_graph_1.FLOW_EDGE_TYPE.false);
         }
         else if (type instanceof Structures.Data
+            || type instanceof Structures.Constants
             || type instanceof Structures.Types) {
             // these doesnt affect control flow, so just take the first statement
             const statement = n.getFirstStatement();
@@ -41397,8 +41420,10 @@ const _typed_identifier_1 = __webpack_require__(/*! ../abap/types/_typed_identif
 const _reference_1 = __webpack_require__(/*! ../abap/5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const _builtin_1 = __webpack_require__(/*! ../abap/5_syntax/_builtin */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_builtin.js");
 const _scope_type_1 = __webpack_require__(/*! ../abap/5_syntax/_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 const types_1 = __webpack_require__(/*! ../abap/types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
+const include_graph_1 = __webpack_require__(/*! ../utils/include_graph */ "./node_modules/@abaplint/core/build/src/utils/include_graph.js");
 class LSPLookup {
     static lookup(cursor, reg, obj) {
         var _a, _b;
@@ -41411,7 +41436,17 @@ class LSPLookup {
         if (fm) {
             return { hover: "Function Module " + fm };
         }
-        const bottomScope = new syntax_1.SyntaxLogic(reg, obj).run().spaghetti.lookupPosition(cursor.identifier.getStart(), cursor.identifier.getFilename());
+        let main = obj;
+        if (obj instanceof objects_1.Program && obj.isInclude()) {
+            // todo: this is slow
+            const ig = new include_graph_1.IncludeGraph(reg);
+            const mains = ig.listMainForInclude(cursor.identifier.getFilename());
+            if (mains.length === 1) {
+                // yea, well, or it has to be a popup
+                main = reg.findObjectForFile(reg.getFileByName(mains[0])) || obj;
+            }
+        }
+        const bottomScope = new syntax_1.SyntaxLogic(reg, main).run().spaghetti.lookupPosition(cursor.identifier.getStart(), cursor.identifier.getFilename());
         if (bottomScope === undefined) {
             return undefined;
         }
@@ -42945,11 +42980,15 @@ class LanguageServer {
         if (file === undefined) {
             return "file not found";
         }
+        const obj = this.reg.findObjectForFile(file);
+        if (obj === undefined) {
+            return "obj not found";
+        }
         const stru = file.getStructure();
         if (stru === undefined) {
             return "empty structure";
         }
-        const graphs = new statement_flow_1.StatementFlow().build(stru);
+        const graphs = new statement_flow_1.StatementFlow().build(stru, obj);
         const wiz = graphs.map(g => g.toDigraph());
         return JSON.stringify(wiz);
     }
@@ -43814,6 +43853,45 @@ exports.AbstractObject = AbstractObject;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/_dynpros.js":
+/*!*******************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/_dynpros.js ***!
+  \*******************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.parseDynpros = parseDynpros;
+const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
+function parseDynpros(parsed) {
+    var _a, _b, _c, _d;
+    const dynpros = [];
+    const xmlDynpros = (_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.DYNPROS;
+    if (xmlDynpros !== undefined) {
+        for (const d of (0, xml_utils_1.xmlToArray)(xmlDynpros.item)) {
+            const fields = [];
+            for (const f of (0, xml_utils_1.xmlToArray)((_d = d.FIELDS) === null || _d === void 0 ? void 0 : _d.RPY_DYFATC)) {
+                fields.push({
+                    name: f.NAME,
+                    type: f.TYPE,
+                    length: f.LENGTH,
+                });
+            }
+            dynpros.push({
+                number: d.HEADER.SCREEN,
+                description: d.HEADER.DESCRIPT,
+                nextScreen: d.HEADER.NEXTSCREEN,
+                fields: fields,
+            });
+        }
+    }
+    return dynpros;
+}
+//# sourceMappingURL=_dynpros.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/_unknown_object.js":
 /*!**************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/_unknown_object.js ***!
@@ -45505,6 +45583,7 @@ class DataElement extends _abstract_object_1.AbstractObject {
     }
     getDescription() {
         var _a;
+        this.parse();
         return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.description;
     }
     getAllowedNaming() {
@@ -45522,6 +45601,11 @@ class DataElement extends _abstract_object_1.AbstractObject {
         this.parse();
         return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.domname;
     }
+    getTexts() {
+        var _a;
+        this.parse();
+        return (_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.texts;
+    }
     parseType(reg) {
         const references = [];
         let lookup = undefined;
@@ -45580,6 +45664,12 @@ class DataElement extends _abstract_object_1.AbstractObject {
             datatype: dd04v === null || dd04v === void 0 ? void 0 : dd04v.DATATYPE,
             leng: dd04v === null || dd04v === void 0 ? void 0 : dd04v.LENG,
             decimals: dd04v === null || dd04v === void 0 ? void 0 : dd04v.DECIMALS,
+            texts: {
+                short: dd04v === null || dd04v === void 0 ? void 0 : dd04v.SCRTEXT_S,
+                medium: dd04v === null || dd04v === void 0 ? void 0 : dd04v.SCRTEXT_M,
+                long: dd04v === null || dd04v === void 0 ? void 0 : dd04v.SCRTEXT_L,
+                heading: dd04v === null || dd04v === void 0 ? void 0 : dd04v.REPTEXT,
+            },
         };
         const end = Date.now();
         return { updated: true, runtime: end - start };
@@ -46217,18 +46307,23 @@ const _abap_object_1 = __webpack_require__(/*! ./_abap_object */ "./node_modules
 const types_1 = __webpack_require__(/*! ../abap/types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
 const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
 const fast_xml_parser_1 = __webpack_require__(/*! fast-xml-parser */ "./node_modules/fast-xml-parser/src/fxp.js");
+const _dynpros_1 = __webpack_require__(/*! ./_dynpros */ "./node_modules/@abaplint/core/build/src/objects/_dynpros.js");
 class FunctionGroup extends _abap_object_1.ABAPObject {
     constructor() {
         super(...arguments);
         this.includes = undefined;
         this.modules = undefined;
+        this.description = undefined;
+        this.dynpros = undefined;
     }
     getType() {
         return "FUGR";
     }
     getDescription() {
-        // todo
-        return undefined;
+        if (this.description === undefined) {
+            this.parseXML();
+        }
+        return this.description;
     }
     setDirty() {
         super.setDirty();
@@ -46241,6 +46336,12 @@ class FunctionGroup extends _abap_object_1.ABAPObject {
             allowNamespace: true,
         };
     }
+    getDynpros() {
+        if (this.dynpros === undefined) {
+            this.parseXML();
+        }
+        return this.dynpros || [];
+    }
     getSequencedFiles() {
         const main = this.getMainABAPFile();
         if (main === undefined) {
@@ -46342,22 +46443,24 @@ class FunctionGroup extends _abap_object_1.ABAPObject {
     }
     /////////////////////////////////
     parseXML() {
-        var _a, _b;
+        var _a, _b, _c;
         this.includes = [];
         this.modules = [];
         const parsed = this.parseRaw2();
         if (parsed === undefined) {
             return;
         }
+        this.description = (_a = parsed.abapGit["asx:abap"]["asx:values"]) === null || _a === void 0 ? void 0 : _a.AREAT;
+        this.dynpros = (0, _dynpros_1.parseDynpros)(parsed);
         // INCLUDES
-        const includes = (_a = parsed.abapGit["asx:abap"]["asx:values"]) === null || _a === void 0 ? void 0 : _a.INCLUDES;
+        const includes = (_b = parsed.abapGit["asx:abap"]["asx:values"]) === null || _b === void 0 ? void 0 : _b.INCLUDES;
         if (includes !== undefined) {
             for (const i of (0, xml_utils_1.xmlToArray)(includes.SOBJ_NAME)) {
                 this.includes.push(i);
             }
         }
         // FUNCTION MODULES
-        const functions = (_b = parsed.abapGit["asx:abap"]["asx:values"]) === null || _b === void 0 ? void 0 : _b.FUNCTIONS;
+        const functions = (_c = parsed.abapGit["asx:abap"]["asx:values"]) === null || _c === void 0 ? void 0 : _c.FUNCTIONS;
         for (const module of (0, xml_utils_1.xmlToArray)(functions === null || functions === void 0 ? void 0 : functions.item)) {
             this.modules.push(new types_1.FunctionModuleDefinition(module));
         }
@@ -48716,7 +48819,7 @@ exports.PersonalizationObject = PersonalizationObject;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Program = void 0;
 const _abap_object_1 = __webpack_require__(/*! ./_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
-const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
+const _dynpros_1 = __webpack_require__(/*! ./_dynpros */ "./node_modules/@abaplint/core/build/src/objects/_dynpros.js");
 class Program extends _abap_object_1.ABAPObject {
     getType() {
         return "PROG";
@@ -48752,11 +48855,10 @@ class Program extends _abap_object_1.ABAPObject {
     }
     getDynpros() {
         this.parseXML();
-        return this.parsedXML.dynpros;
+        return this.parsedXML.dynpros || [];
     }
     ////////////////////////////
     parseXML() {
-        var _a, _b, _c, _d;
         if (this.parsedXML !== undefined) {
             return;
         }
@@ -48770,24 +48872,7 @@ class Program extends _abap_object_1.ABAPObject {
             };
             return;
         }
-        const dynpros = [];
-        const xmlDynpros = (_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.DYNPROS;
-        if (xmlDynpros !== undefined) {
-            for (const d of (0, xml_utils_1.xmlToArray)(xmlDynpros.item)) {
-                const fields = [];
-                for (const f of (0, xml_utils_1.xmlToArray)((_d = d.FIELDS) === null || _d === void 0 ? void 0 : _d.RPY_DYFATC)) {
-                    fields.push({
-                        name: f.NAME,
-                        type: f.TYPE,
-                        length: f.LENGTH,
-                    });
-                }
-                dynpros.push({
-                    number: d.HEADER.SCREEN,
-                    fields: fields,
-                });
-            }
-        }
+        const dynpros = (0, _dynpros_1.parseDynpros)(parsed);
         this.parsedXML = {
             isInclude: file ? file.getRaw().includes("<SUBC>I</SUBC>") : false,
             isModulePool: file ? file.getRaw().includes("<SUBC>M</SUBC>") : false,
@@ -51564,7 +51649,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.112.22";
+        return "2.113.6";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -52202,6 +52287,111 @@ exports.Abapdoc = Abapdoc;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/add_test_attributes.js":
+/*!****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/add_test_attributes.js ***!
+  \****************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.AddTestAttributes = exports.AddTestAttributesConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+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 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 _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+class AddTestAttributesConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.AddTestAttributesConf = AddTestAttributesConf;
+class AddTestAttributes extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new AddTestAttributesConf();
+    }
+    getMetadata() {
+        return {
+            key: "add_test_attributes",
+            title: "Add test attributes for tests classes with test methods",
+            shortDescription: `Add test attributes DURATION and RISK LEVEL for tests classes with test methods`,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `CLASS ltcl_test1 DEFINITION FINAL FOR TESTING.
+  PUBLIC SECTION.
+  PROTECTED SECTION.
+  PRIVATE SECTION.
+    METHODS test FOR TESTING RAISING cx_static_check.
+ENDCLASS.
+
+CLASS ltcl_test1 IMPLEMENTATION.
+  METHOD test.
+  ENDMETHOD.
+ENDCLASS.`,
+            goodExample: `CLASS ltcl_test2 DEFINITION FINAL FOR TESTING DURATION SHORT RISK LEVEL HARMLESS.
+  PUBLIC SECTION.
+  PROTECTED SECTION.
+  PRIVATE SECTION.
+    METHODS test FOR TESTING RAISING cx_static_check.
+ENDCLASS.
+
+CLASS ltcl_test2 IMPLEMENTATION.
+  METHOD test.
+  ENDMETHOD.
+ENDCLASS.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        const stru = file.getStructure();
+        if (stru === undefined) {
+            return [];
+        }
+        for (const classStructure of stru.findAllStructures(Structures.ClassDefinition)) {
+            const cdef = classStructure.findFirstStatement(Statements.ClassDefinition);
+            if (cdef === undefined) {
+                continue;
+            }
+            const cdefConcat = cdef === null || cdef === void 0 ? void 0 : cdef.concatTokens().toUpperCase();
+            if ((cdefConcat === null || cdefConcat === void 0 ? void 0 : cdefConcat.includes(" FOR TESTING")) === false) {
+                continue;
+            }
+            const hasDuration = cdefConcat === null || cdefConcat === void 0 ? void 0 : cdefConcat.includes(" DURATION ");
+            const hasRiskLevel = cdefConcat === null || cdefConcat === void 0 ? void 0 : cdefConcat.includes(" RISK LEVEL ");
+            if (hasDuration === true && hasRiskLevel === true) {
+                continue;
+            }
+            let hasTestMethod = false;
+            for (const mdef of classStructure.findAllStatements(Statements.MethodDef)) {
+                const concat = mdef.concatTokens().toUpperCase();
+                if (concat.includes(" FOR TESTING")) {
+                    hasTestMethod = true;
+                }
+            }
+            if (hasTestMethod === false) {
+                continue;
+            }
+            if (hasDuration === false) {
+                issues.push(issue_1.Issue.atStatement(file, cdef, "Add DURATION", this.getMetadata().key, this.getConfig().severity));
+            }
+            if (hasRiskLevel === false) {
+                issues.push(issue_1.Issue.atStatement(file, cdef, "Add RISK LEVEL", this.getMetadata().key, this.getConfig().severity));
+            }
+        }
+        return issues;
+    }
+}
+exports.AddTestAttributes = AddTestAttributes;
+//# sourceMappingURL=add_test_attributes.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/align_parameters.js":
 /*!*************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/align_parameters.js ***!
@@ -59625,6 +59815,100 @@ exports.EasyToFindMessages = EasyToFindMessages;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/empty_event.js":
+/*!********************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/empty_event.js ***!
+  \********************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.EmptyEvent = exports.EmptyEventConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+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 Structures = __webpack_require__(/*! ../abap/3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const selection_events_1 = __webpack_require__(/*! ../abap/flow/selection_events */ "./node_modules/@abaplint/core/build/src/abap/flow/selection_events.js");
+class EmptyEventConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.EmptyEventConf = EmptyEventConf;
+class EmptyEvent extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new EmptyEventConf();
+    }
+    getMetadata() {
+        return {
+            key: "empty_event",
+            title: "Empty selection screen or list processing event block",
+            shortDescription: `Empty selection screen or list processing event block`,
+            extendedInformation: ``,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `
+INITIALIZATION.
+  WRITE 'hello'.
+END-OF-SELECTION.`,
+            goodExample: `
+START-OF-SELECTION.
+  PERFORM sdf.
+  COMMIT WORK.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, obj) {
+        const issues = [];
+        if (!(obj instanceof objects_1.Program) || obj.isInclude()) {
+            return issues;
+        }
+        const stru = file.getStructure();
+        if (stru === undefined) {
+            return [];
+        }
+        let currentEvent = undefined;
+        let children = [];
+        for (const s of stru.getChildren() || []) {
+            if (selection_events_1.SELECTION_EVENTS.some(f => s.get() instanceof f)) {
+                if (currentEvent !== undefined && children.length === 0) {
+                    issues.push(issue_1.Issue.atStatement(file, currentEvent, "Empty event", this.getMetadata().key, this.getConfig().severity));
+                }
+                children = [];
+                currentEvent = s;
+            }
+            else if (s.get() instanceof Structures.Normal) {
+                const stru = s;
+                // ignore declaration stuff
+                if (selection_events_1.DECLARATION_STUFF.some(d => { var _a; return ((_a = stru.getFirstStatement()) === null || _a === void 0 ? void 0 : _a.get()) instanceof d; })) {
+                    continue;
+                }
+                children.push(s);
+            }
+            else {
+                if (currentEvent !== undefined && children.length === 0) {
+                    issues.push(issue_1.Issue.atStatement(file, currentEvent, "Empty event", this.getMetadata().key, this.getConfig().severity));
+                }
+                children = [];
+                currentEvent = undefined;
+            }
+        }
+        if (currentEvent !== undefined && children.length === 0) {
+            issues.push(issue_1.Issue.atStatement(file, currentEvent, "Empty event", this.getMetadata().key, this.getConfig().severity));
+        }
+        return issues;
+    }
+}
+exports.EmptyEvent = EmptyEvent;
+//# sourceMappingURL=empty_event.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/empty_line_in_statement.js":
 /*!********************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/empty_line_in_statement.js ***!
@@ -61991,6 +62275,109 @@ exports.ImplementMethods = ImplementMethods;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/implicit_start_of_selection.js":
+/*!************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/implicit_start_of_selection.js ***!
+  \************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.ImplicitStartOfSelection = exports.ImplicitStartOfSelectionConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+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 Structures = __webpack_require__(/*! ../abap/3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const nodes_1 = __webpack_require__(/*! ../abap/nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
+const selection_events_1 = __webpack_require__(/*! ../abap/flow/selection_events */ "./node_modules/@abaplint/core/build/src/abap/flow/selection_events.js");
+class ImplicitStartOfSelectionConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.ImplicitStartOfSelectionConf = ImplicitStartOfSelectionConf;
+class ImplicitStartOfSelection extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new ImplicitStartOfSelectionConf();
+    }
+    getMetadata() {
+        return {
+            key: "implicit_start_of_selection",
+            title: "Implicit START-OF-SELECTION",
+            shortDescription: `Add explicit selection screen event handling`,
+            extendedInformation: `Only runs for executable programs
+
+https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapstart-of-selection.htm`,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `REPORT zfoo.
+WRITE 'hello'.`,
+            goodExample: `
+START-OF-SELECTION.
+  WRITE 'hello'.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, obj) {
+        const issues = [];
+        if (!(obj instanceof objects_1.Program) || obj.isInclude()) {
+            return issues;
+        }
+        const stru = file.getStructure();
+        if (stru === undefined) {
+            return [];
+        }
+        let inEvent = false;
+        let collected = [];
+        for (const s of stru.getChildren() || []) {
+            if (selection_events_1.SELECTION_EVENTS.some(f => s.get() instanceof f)) {
+                if (inEvent === false && collected.length > 0) {
+                    // implicit START-OF-SELECTION
+                    let first = collected[0];
+                    if (first instanceof nodes_1.StructureNode) {
+                        first = first.getFirstStatement();
+                    }
+                    issues.push(issue_1.Issue.atStatement(file, first, "Implicit START-OF-SELECTION", this.getMetadata().key, this.getConfig().severity));
+                }
+                collected = [];
+                inEvent = true;
+            }
+            else if (s.get() instanceof Structures.Normal) {
+                const stru = s;
+                // ignore declaration stuff
+                if (selection_events_1.DECLARATION_STUFF.some(d => { var _a; return ((_a = stru.getFirstStatement()) === null || _a === void 0 ? void 0 : _a.get()) instanceof d; })) {
+                    continue;
+                }
+                collected.push(s);
+            }
+            else {
+                if (inEvent === true) {
+                    inEvent = false;
+                }
+                collected = [];
+            }
+        }
+        if (inEvent === false && collected.length > 0) {
+            // implicit START-OF-SELECTION
+            let first = collected[0];
+            if (first instanceof nodes_1.StructureNode) {
+                first = first.getFirstStatement();
+            }
+            issues.push(issue_1.Issue.atStatement(file, first, "Implicit START-OF-SELECTION", this.getMetadata().key, this.getConfig().severity));
+        }
+        return issues;
+    }
+}
+exports.ImplicitStartOfSelection = ImplicitStartOfSelection;
+//# sourceMappingURL=implicit_start_of_selection.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/in_statement_indentation.js":
 /*!*********************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/in_statement_indentation.js ***!
@@ -62294,6 +62681,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
 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__(/*! ./add_test_attributes */ "./node_modules/@abaplint/core/build/src/rules/add_test_attributes.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);
@@ -62335,6 +62723,7 @@ __exportStar(__webpack_require__(/*! ./double_space */ "./node_modules/@abaplint
 __exportStar(__webpack_require__(/*! ./downport */ "./node_modules/@abaplint/core/build/src/rules/downport.js"), exports);
 __exportStar(__webpack_require__(/*! ./dynpro_checks */ "./node_modules/@abaplint/core/build/src/rules/dynpro_checks.js"), exports);
 __exportStar(__webpack_require__(/*! ./easy_to_find_messages */ "./node_modules/@abaplint/core/build/src/rules/easy_to_find_messages.js"), exports);
+__exportStar(__webpack_require__(/*! ./empty_event */ "./node_modules/@abaplint/core/build/src/rules/empty_event.js"), exports);
 __exportStar(__webpack_require__(/*! ./empty_line_in_statement */ "./node_modules/@abaplint/core/build/src/rules/empty_line_in_statement.js"), exports);
 __exportStar(__webpack_require__(/*! ./empty_statement */ "./node_modules/@abaplint/core/build/src/rules/empty_statement.js"), exports);
 __exportStar(__webpack_require__(/*! ./empty_structure */ "./node_modules/@abaplint/core/build/src/rules/empty_structure.js"), exports);
@@ -62356,6 +62745,7 @@ __exportStar(__webpack_require__(/*! ./identical_descriptions */ "./node_modules
 __exportStar(__webpack_require__(/*! ./identical_form_names */ "./node_modules/@abaplint/core/build/src/rules/identical_form_names.js"), exports);
 __exportStar(__webpack_require__(/*! ./if_in_if */ "./node_modules/@abaplint/core/build/src/rules/if_in_if.js"), exports);
 __exportStar(__webpack_require__(/*! ./implement_methods */ "./node_modules/@abaplint/core/build/src/rules/implement_methods.js"), exports);
+__exportStar(__webpack_require__(/*! ./implicit_start_of_selection */ "./node_modules/@abaplint/core/build/src/rules/implicit_start_of_selection.js"), exports);
 __exportStar(__webpack_require__(/*! ./in_statement_indentation */ "./node_modules/@abaplint/core/build/src/rules/in_statement_indentation.js"), exports);
 __exportStar(__webpack_require__(/*! ./indentation */ "./node_modules/@abaplint/core/build/src/rules/indentation.js"), exports);
 __exportStar(__webpack_require__(/*! ./inline_data_old_versions */ "./node_modules/@abaplint/core/build/src/rules/inline_data_old_versions.js"), exports);
@@ -76160,7 +76550,7 @@ class FieldChainTranspiler {
             if (c.get() instanceof core_1.Expressions.SourceField
                 || c.get() instanceof core_1.Expressions.Field) {
                 const name = traversal.prefixAndName(c.getFirstToken(), filename).replace("~", "$");
-                ret.append(traversal_1.Traversal.escapeNamespace(name), c, traversal);
+                ret.append(traversal_1.Traversal.prefixVariable(traversal_1.Traversal.escapeNamespace(name)), c, traversal);
             }
             else if (c instanceof core_1.Nodes.ExpressionNode && c.get() instanceof core_1.Expressions.SourceFieldSymbol) {
                 ret.appendChunk(new field_symbol_1.FieldSymbolTranspiler().transpile(c, traversal));
@@ -78098,7 +78488,7 @@ class TargetTranspiler {
             const next = children[i + 1];
             if (c.get() instanceof core_1.Expressions.TargetField) {
                 const prefix = traversal.prefixAndName(c.getFirstToken()).replace("~", "$");
-                ret.append(traversal_1.Traversal.escapeNamespace(prefix), c, traversal);
+                ret.append(traversal_1.Traversal.prefixVariable(traversal_1.Traversal.escapeNamespace(prefix)), c, traversal);
             }
             else if (c.get() instanceof core_1.Expressions.ClassName) {
                 const name = traversal.lookupClassOrInterface(c.getFirstToken().getStr(), c.getFirstToken());
@@ -78961,10 +79351,10 @@ exports.Transpiler = Transpiler;
 "use strict";
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.Keywords = exports.defaultKeywords = void 0;
+exports.Keywords = exports.DEFAULT_KEYWORDS = void 0;
 const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
 // https://www.w3schools.com/js/js_reserved.asp
-exports.defaultKeywords = [
+exports.DEFAULT_KEYWORDS = [
     "abstract", "arguments", "await",
     "break", "byte", "catch",
     "char", "class", "const", "continue",
@@ -78992,10 +79382,13 @@ class Keywords {
             this.keywords = keywords;
         }
         else {
-            this.keywords = exports.defaultKeywords;
+            this.keywords = exports.DEFAULT_KEYWORDS;
         }
     }
     handle(reg) {
+        if (this.keywords.length === 0) {
+            return;
+        }
         reg.parse();
         for (const o of reg.getObjects()) {
             if (!(o instanceof abaplint.ABAPObject)) {
@@ -80676,6 +81069,7 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.DataTranspiler = void 0;
 const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
 const transpile_types_1 = __webpack_require__(/*! ../transpile_types */ "./node_modules/@abaplint/transpiler/build/src/transpile_types.js");
+const traversal_1 = __webpack_require__(/*! ../traversal */ "./node_modules/@abaplint/transpiler/build/src/traversal.js");
 const constant_1 = __webpack_require__(/*! ../expressions/constant */ "./node_modules/@abaplint/transpiler/build/src/expressions/constant.js");
 const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
@@ -80702,7 +81096,7 @@ class DataTranspiler {
         }
         const ret = new chunk_1.Chunk()
             .appendString("let ")
-            .append(found.getName().toLowerCase(), token, traversal)
+            .append(traversal_1.Traversal.prefixVariable(traversal_1.Traversal.escapeNamespace(found.getName().toLowerCase())), token, traversal)
             .appendString(" = " + new transpile_types_1.TranspileTypes().toType(found.getType()))
             .append(";", node.getLastToken(), traversal)
             .append(value, node.getLastToken(), traversal);
@@ -81601,6 +81995,7 @@ exports.FormTranspiler = void 0;
 const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
 const unique_identifier_1 = __webpack_require__(/*! ../unique_identifier */ "./node_modules/@abaplint/transpiler/build/src/unique_identifier.js");
+const traversal_1 = __webpack_require__(/*! ../traversal */ "./node_modules/@abaplint/transpiler/build/src/traversal.js");
 class FormTranspiler {
     transpile(node, traversal) {
         unique_identifier_1.UniqueIdentifier.resetIndexBackup();
@@ -81610,13 +82005,13 @@ class FormTranspiler {
         const ret = new chunk_1.Chunk("async function " + name + "(INPUT) {");
         const params = [];
         for (const p of def?.getChangingParameters() || []) {
-            params.push(`let ${p.getName()} = INPUT.${p.getName()};`);
+            params.push(`let ${traversal_1.Traversal.prefixVariable(p.getName())} = INPUT.${p.getName()};`);
         }
         for (const p of def?.getTablesParameters() || []) {
-            params.push(`let ${p.getName()} = INPUT.${p.getName()};`);
+            params.push(`let ${traversal_1.Traversal.prefixVariable(p.getName())} = INPUT.${p.getName()};`);
         }
         for (const p of def?.getUsingParameters() || []) {
-            params.push(`let ${p.getName()} = INPUT.${p.getName()};`);
+            params.push(`let ${traversal_1.Traversal.prefixVariable(p.getName())} = INPUT.${p.getName()};`);
         }
         if (params.length > 0) {
             ret.appendString("\n" + params.join("\n"));
@@ -82679,6 +83074,7 @@ class MethodImplementationTranspiler {
         for (const n in vars) {
             const identifier = vars[n];
             const varName = n.toLowerCase();
+            const varPrefixed = traversal_1.Traversal.prefixVariable(varName);
             if (identifier.getMeta().includes("importing" /* abaplint.IdentifierMeta.MethodImporting */)
                 || identifier.getMeta().includes("changing" /* abaplint.IdentifierMeta.MethodChanging */)
                 || identifier.getMeta().includes("event_parameter" /* abaplint.IdentifierMeta.EventParameter */)
@@ -82691,31 +83087,31 @@ class MethodImplementationTranspiler {
                 const passByValue = identifier.getMeta().includes("pass_by_value" /* abaplint.IdentifierMeta.PassByValue */);
                 const type = identifier.getType();
                 if (identifier.getMeta().includes("exporting" /* abaplint.IdentifierMeta.MethodExporting */)) {
-                    after += `let ${varName} = ${unique}?.${varName} || ${new transpile_types_1.TranspileTypes().toType(identifier.getType())};\n`;
+                    after += `let ${varPrefixed} = ${unique}?.${varName} || ${new transpile_types_1.TranspileTypes().toType(identifier.getType())};\n`;
                 }
                 else if (identifier.getMeta().includes("importing" /* abaplint.IdentifierMeta.MethodImporting */)
                     && parameterDefault === undefined
                     && passByValue === false
                     && isOptional === false
                     && type.isGeneric() === false) {
-                    after += `let ${varName} = ${unique}?.${varName};\n`;
+                    after += `let ${varPrefixed} = ${unique}?.${varName};\n`;
                     if (identifier.getType().getQualifiedName() !== undefined && identifier.getType().getQualifiedName() !== "") {
-                        after += `if (${varName}?.getQualifiedName === undefined || ${varName}.getQualifiedName() !== "${identifier.getType().getQualifiedName()?.toUpperCase()}") { ${varName} = undefined; }\n`;
+                        after += `if (${varPrefixed}?.getQualifiedName === undefined || ${varPrefixed}.getQualifiedName() !== "${identifier.getType().getQualifiedName()?.toUpperCase()}") { ${varPrefixed} = undefined; }\n`;
                     }
-                    after += `if (${varName} === undefined) { ${varName} = ${new transpile_types_1.TranspileTypes().toType(identifier.getType())}.set(${unique}.${varName}); }\n`;
+                    after += `if (${varPrefixed} === undefined) { ${varPrefixed} = ${new transpile_types_1.TranspileTypes().toType(identifier.getType())}.set(${unique}.${varName}); }\n`;
                 }
                 else if (identifier.getMeta().includes("importing" /* abaplint.IdentifierMeta.MethodImporting */)
                     && type.isGeneric() === true) {
                     if (isOptional === true) {
-                        after += `let ${varName} = ${unique}?.${varName} || ${new transpile_types_1.TranspileTypes().toType(identifier.getType())};\n`;
+                        after += `let ${varPrefixed} = ${unique}?.${varName} || ${new transpile_types_1.TranspileTypes().toType(identifier.getType())};\n`;
                     }
                     else {
-                        after += `let ${varName} = ${unique}?.${varName};\n`;
+                        after += `let ${varPrefixed} = ${unique}?.${varName};\n`;
                     }
                     if (type instanceof abaplint.BasicTypes.NumericGenericType) {
-                        after += `if (${varName}.constructor.name === "Character") {
-  ${varName} = ${new transpile_types_1.TranspileTypes().toType(identifier.getType())};
-  ${varName}.set(${unique}?.${varName});
+                        after += `if (${varPrefixed}.constructor.name === "Character") {
+  ${varPrefixed} = ${new transpile_types_1.TranspileTypes().toType(identifier.getType())};
+  ${varPrefixed}.set(${unique}?.${varName});
 }\n`;
                     }
                 }
@@ -82723,19 +83119,19 @@ class MethodImplementationTranspiler {
                     && type.isGeneric() === false) {
                     after += new transpile_types_1.TranspileTypes().declare(identifier) + "\n";
                     // note: it might be nessesary to do a type conversion, eg char is passed to xstring parameter
-                    after += "if (" + unique + " && " + unique + "." + varName + ") {" + varName + ".set(" + unique + "." + varName + ");}\n";
+                    after += "if (" + unique + " && " + unique + "." + varName + ") {" + varPrefixed + ".set(" + unique + "." + varName + ");}\n";
                 }
                 else {
                     after += new transpile_types_1.TranspileTypes().declare(identifier) + "\n";
-                    after += "if (" + unique + " && " + unique + "." + varName + ") {" + varName + " = " + unique + "." + varName + ";}\n";
+                    after += "if (" + unique + " && " + unique + "." + varName + ") {" + varPrefixed + " = " + unique + "." + varName + ";}\n";
                 }
                 if (parameterDefault) {
                     const val = this.buildDefault(parameterDefault, traversal, methodDef?.getFilename());
                     if (passByValue === true || identifier.getMeta().includes("changing" /* abaplint.IdentifierMeta.MethodChanging */)) {
-                        after += "if (" + unique + " === undefined || " + unique + "." + varName + " === undefined) {" + varName + ".set(" + val + ");}\n";
+                        after += "if (" + unique + " === undefined || " + unique + "." + varName + " === undefined) {" + varPrefixed + ".set(" + val + ");}\n";
                     }
                     else {
-                        after += "if (" + unique + " === undefined || " + unique + "." + varName + " === undefined) {" + varName + " = " + val + ";}\n";
+                        after += "if (" + unique + " === undefined || " + unique + "." + varName + " === undefined) {" + varPrefixed + " = " + val + ";}\n";
                     }
                 }
             }
@@ -86356,6 +86752,7 @@ const StructureTranspilers = __webpack_require__(/*! ./structures */ "./node_mod
 const transpile_types_1 = __webpack_require__(/*! ./transpile_types */ "./node_modules/@abaplint/transpiler/build/src/transpile_types.js");
 const chunk_1 = __webpack_require__(/*! ./chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
 const expressions_1 = __webpack_require__(/*! ./expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
+const keywords_1 = __webpack_require__(/*! ./keywords */ "./node_modules/@abaplint/transpiler/build/src/keywords.js");
 class Traversal {
     constructor(spaghetti, file, obj, reg, options) {
         this.scopeCache = undefined;
@@ -86368,6 +86765,13 @@ class Traversal {
     static escapeNamespace(name) {
         return name?.replace(/\//g, "$");
     }
+    static prefixVariable(name) {
+        // TODO: performace, make this a hash lookup?,
+        if (keywords_1.DEFAULT_KEYWORDS.some(k => k === name)) {
+            return "$" + name;
+        }
+        return name + "";
+    }
     getCurrentObject() {
         return this.obj;
     }
@@ -87578,7 +87982,7 @@ class Validation {
         }
         exports.config.rules.forbidden_identifier.check = ["^unique\\d+$"];
         if (this.options?.keywords === undefined) {
-            for (const d of keywords_1.defaultKeywords) {
+            for (const d of keywords_1.DEFAULT_KEYWORDS) {
                 const add = "^" + d + "$";
                 exports.config.rules["forbidden_identifier"]["check"].push(add);
             }
@@ -87932,6 +88336,35 @@ module.exports = {
 
 /***/ }),
 
+/***/ "./node_modules/fast-xml-parser/src/ignoreAttributes.js":
+/*!**************************************************************!*\
+  !*** ./node_modules/fast-xml-parser/src/ignoreAttributes.js ***!
+  \**************************************************************/
+/***/ ((module) => {
+
+function getIgnoreAttributesFn(ignoreAttributes) {
+    if (typeof ignoreAttributes === 'function') {
+        return ignoreAttributes
+    }
+    if (Array.isArray(ignoreAttributes)) {
+        return (attrName) => {
+            for (const pattern of ignoreAttributes) {
+                if (typeof pattern === 'string' && attrName === pattern) {
+                    return true
+                }
+                if (pattern instanceof RegExp && pattern.test(attrName)) {
+                    return true
+                }
+            }
+        }
+    }
+    return () => false
+}
+
+module.exports = getIgnoreAttributesFn
+
+/***/ }),
+
 /***/ "./node_modules/fast-xml-parser/src/util.js":
 /*!**************************************************!*\
   !*** ./node_modules/fast-xml-parser/src/util.js ***!
@@ -88461,6 +88894,7 @@ function getPositionFromMatch(match) {
 
 //parse Empty Node as self closing node
 const buildFromOrderedJs = __webpack_require__(/*! ./orderedJs2Xml */ "./node_modules/fast-xml-parser/src/xmlbuilder/orderedJs2Xml.js");
+const getIgnoreAttributesFn = __webpack_require__(/*! ../ignoreAttributes */ "./node_modules/fast-xml-parser/src/ignoreAttributes.js")
 
 const defaultOptions = {
   attributeNamePrefix: '@_',
@@ -88498,11 +88932,12 @@ const defaultOptions = {
 
 function Builder(options) {
   this.options = Object.assign({}, defaultOptions, options);
-  if (this.options.ignoreAttributes || this.options.attributesGroupName) {
+  if (this.options.ignoreAttributes === true || this.options.attributesGroupName) {
     this.isAttribute = function(/*a*/) {
       return false;
     };
   } else {
+    this.ignoreAttributesFn = getIgnoreAttributesFn(this.options.ignoreAttributes)
     this.attrPrefixLen = this.options.attributeNamePrefix.length;
     this.isAttribute = isAttribute;
   }
@@ -88531,13 +88966,14 @@ Builder.prototype.build = function(jObj) {
         [this.options.arrayNodeName] : jObj
       }
     }
-    return this.j2x(jObj, 0).val;
+    return this.j2x(jObj, 0, []).val;
   }
 };
 
-Builder.prototype.j2x = function(jObj, level) {
+Builder.prototype.j2x = function(jObj, level, ajPath) {
   let attrStr = '';
   let val = '';
+  const jPath = ajPath.join('.')
   for (let key in jObj) {
     if(!Object.prototype.hasOwnProperty.call(jObj, key)) continue;
     if (typeof jObj[key] === 'undefined') {
@@ -88560,9 +88996,9 @@ Builder.prototype.j2x = function(jObj, level) {
     } else if (typeof jObj[key] !== 'object') {
       //premitive type
       const attr = this.isAttribute(key);
-      if (attr) {
+      if (attr && !this.ignoreAttributesFn(attr, jPath)) {
         attrStr += this.buildAttrPairStr(attr, '' + jObj[key]);
-      }else {
+      } else if (!attr) {
         //tag value
         if (key === this.options.textNodeName) {
           let newval = this.options.tagValueProcessor(key, '' + jObj[key]);
@@ -88586,13 +89022,13 @@ Builder.prototype.j2x = function(jObj, level) {
           // val += this.indentate(level) + '<' + key + '/' + this.tagEndChar;
         } else if (typeof item === 'object') {
           if(this.options.oneListGroup){
-            const result = this.j2x(item, level + 1);
+            const result = this.j2x(item, level + 1, ajPath.concat(key));
             listTagVal += result.val;
             if (this.options.attributesGroupName && item.hasOwnProperty(this.options.attributesGroupName)) {
               listTagAttr += result.attrStr
             }
           }else{
-            listTagVal += this.processTextOrObjNode(item, key, level)
+            listTagVal += this.processTextOrObjNode(item, key, level, ajPath)
           }
         } else {
           if (this.options.oneListGroup) {
@@ -88617,7 +89053,7 @@ Builder.prototype.j2x = function(jObj, level) {
           attrStr += this.buildAttrPairStr(Ks[j], '' + jObj[key][Ks[j]]);
         }
       } else {
-        val += this.processTextOrObjNode(jObj[key], key, level)
+        val += this.processTextOrObjNode(jObj[key], key, level, ajPath)
       }
     }
   }
@@ -88632,8 +89068,8 @@ Builder.prototype.buildAttrPairStr = function(attrName, val){
   } else return ' ' + attrName + '="' + val + '"';
 }
 
-function processTextOrObjNode (object, key, level) {
-  const result = this.j2x(object, level + 1);
+function processTextOrObjNode (object, key, level, ajPath) {
+  const result = this.j2x(object, level + 1, ajPath.concat(key));
   if (object[this.options.textNodeName] !== undefined && Object.keys(object).length === 1) {
     return this.buildTextValNode(object[this.options.textNodeName], key, result.attrStr, level);
   } else {
@@ -89121,6 +89557,7 @@ const util = __webpack_require__(/*! ../util */ "./node_modules/fast-xml-parser/
 const xmlNode = __webpack_require__(/*! ./xmlNode */ "./node_modules/fast-xml-parser/src/xmlparser/xmlNode.js");
 const readDocType = __webpack_require__(/*! ./DocTypeReader */ "./node_modules/fast-xml-parser/src/xmlparser/DocTypeReader.js");
 const toNumber = __webpack_require__(/*! strnum */ "./node_modules/strnum/strnum.js");
+const getIgnoreAttributesFn = __webpack_require__(/*! ../ignoreAttributes */ "./node_modules/fast-xml-parser/src/ignoreAttributes.js")
 
 // const regx =
 //   '<((!\\[CDATA\\[([\\s\\S]*?)(]]>))|((NAME:)?(NAME))([^>]*)>|((\\/)(NAME)\\s*>))([^<]*)'
@@ -89169,6 +89606,7 @@ class OrderedObjParser{
     this.readStopNodeData = readStopNodeData;
     this.saveTextToParentTag = saveTextToParentTag;
     this.addChild = addChild;
+    this.ignoreAttributesFn = getIgnoreAttributesFn(this.options.ignoreAttributes)
   }
 
 }
@@ -89241,7 +89679,7 @@ function resolveNameSpace(tagname) {
 const attrsRegx = new RegExp('([^\\s=]+)\\s*(=\\s*([\'"])([\\s\\S]*?)\\3)?', 'gm');
 
 function buildAttributesMap(attrStr, jPath, tagName) {
-  if (!this.options.ignoreAttributes && typeof attrStr === 'string') {
+  if (this.options.ignoreAttributes !== true && typeof attrStr === 'string') {
     // attrStr = attrStr.replace(/\r?\n/g, ' ');
     //attrStr = attrStr || attrStr.trim();
 
@@ -89250,6 +89688,9 @@ function buildAttributesMap(attrStr, jPath, tagName) {
     const attrs = {};
     for (let i = 0; i < len; i++) {
       const attrName = this.resolveNameSpace(matches[i][1]);
+      if (this.ignoreAttributesFn(attrName, jPath)) {
+        continue
+      }
       let oldVal = matches[i][4];
       let aName = this.options.attributeNamePrefix + attrName;
       if (attrName.length) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.10.10",
+  "version": "2.10.12",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -26,12 +26,12 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.10.10",
+    "@abaplint/transpiler": "^2.10.12",
     "@types/glob": "^8.1.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.7",
-    "@types/node": "^22.5.1",
-    "@abaplint/core": "^2.112.22",
+    "@types/node": "^22.5.4",
+    "@abaplint/core": "^2.113.6",
     "progress": "^2.0.3",
     "webpack": "^5.94.0",
     "webpack-cli": "^5.1.4",