@abaplint/transpiler-cli 2.13.91 → 2.13.93

package/build/bundle.js

@@ -35333,6 +35333,11 @@ class Source {
                             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                             return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                         }
+                        if ((foundType === null || foundType === void 0 ? void 0 : foundType.isGeneric()) && bodyType !== undefined) {
+                            this.addIfInferred(node, input, bodyType);
+                            this.traverseRemainingChildren(children, input);
+                            return bodyType;
+                        }
                         this.addIfInferred(node, input, foundType);
                         this.traverseRemainingChildren(children, input);
                         return foundType;
@@ -71237,7 +71242,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.58";
+        return "2.120.59";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -85675,6 +85680,7 @@ __exportStar(__webpack_require__(/*! ./types_naming */ "./node_modules/@abaplint
 __exportStar(__webpack_require__(/*! ./uncaught_exception */ "./node_modules/@abaplint/core/build/src/rules/uncaught_exception.js"), exports);
 __exportStar(__webpack_require__(/*! ./unknown_types */ "./node_modules/@abaplint/core/build/src/rules/unknown_types.js"), exports);
 __exportStar(__webpack_require__(/*! ./unnecessary_chaining */ "./node_modules/@abaplint/core/build/src/rules/unnecessary_chaining.js"), exports);
+__exportStar(__webpack_require__(/*! ./unnecessary_loop_guard */ "./node_modules/@abaplint/core/build/src/rules/unnecessary_loop_guard.js"), exports);
 __exportStar(__webpack_require__(/*! ./unnecessary_pragma */ "./node_modules/@abaplint/core/build/src/rules/unnecessary_pragma.js"), exports);
 __exportStar(__webpack_require__(/*! ./unnecessary_return */ "./node_modules/@abaplint/core/build/src/rules/unnecessary_return.js"), exports);
 __exportStar(__webpack_require__(/*! ./unreachable_code */ "./node_modules/@abaplint/core/build/src/rules/unreachable_code.js"), exports);
@@ -85687,6 +85693,7 @@ __exportStar(__webpack_require__(/*! ./unused_variables */ "./node_modules/@abap
 __exportStar(__webpack_require__(/*! ./use_bool_expression */ "./node_modules/@abaplint/core/build/src/rules/use_bool_expression.js"), exports);
 __exportStar(__webpack_require__(/*! ./use_class_based_exceptions */ "./node_modules/@abaplint/core/build/src/rules/use_class_based_exceptions.js"), exports);
 __exportStar(__webpack_require__(/*! ./use_line_exists */ "./node_modules/@abaplint/core/build/src/rules/use_line_exists.js"), exports);
+__exportStar(__webpack_require__(/*! ./use_message_class */ "./node_modules/@abaplint/core/build/src/rules/use_message_class.js"), exports);
 __exportStar(__webpack_require__(/*! ./use_new */ "./node_modules/@abaplint/core/build/src/rules/use_new.js"), exports);
 __exportStar(__webpack_require__(/*! ./when_others_last */ "./node_modules/@abaplint/core/build/src/rules/when_others_last.js"), exports);
 __exportStar(__webpack_require__(/*! ./whitespace_end */ "./node_modules/@abaplint/core/build/src/rules/whitespace_end.js"), exports);
@@ -99423,6 +99430,151 @@ exports.UnnecessaryChaining = UnnecessaryChaining;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/unnecessary_loop_guard.js"
+/*!*******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/unnecessary_loop_guard.js ***!
+  \*******************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.UnnecessaryLoopGuard = exports.UnnecessaryLoopGuardConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js"));
+const Structures = __importStar(__webpack_require__(/*! ../abap/3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.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 _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+class UnnecessaryLoopGuardConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.UnnecessaryLoopGuardConf = UnnecessaryLoopGuardConf;
+class UnnecessaryLoopGuard extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new UnnecessaryLoopGuardConf();
+    }
+    getMetadata() {
+        return {
+            key: "unnecessary_loop_guard",
+            title: "Unnecessary loop guard",
+            shortDescription: `Detects unnecessary IS [NOT] INITIAL check before LOOP AT`,
+            extendedInformation: `LOOP AT iterates zero times on an empty table, so an IS [NOT] INITIAL guard is redundant.`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
+            badExample: `IF lt_data IS NOT INITIAL.
+  LOOP AT lt_data INTO DATA(ls_item).
+    WRITE ls_item-name.
+  ENDLOOP.
+ENDIF.`,
+            goodExample: `LOOP AT lt_data INTO DATA(ls_item).
+  WRITE ls_item-name.
+ENDLOOP.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        const stru = file.getStructure();
+        if (stru === undefined) {
+            return issues;
+        }
+        for (const ifStru of stru.findAllStructuresRecursive(Structures.If)) {
+            if (ifStru.findDirectStructures(Structures.ElseIf).length > 0
+                || ifStru.findDirectStructures(Structures.Else).length > 0) {
+                continue;
+            }
+            const ifStatement = ifStru.findDirectStatement(Statements.If);
+            if (ifStatement === undefined) {
+                continue;
+            }
+            const cond = ifStatement.findDirectExpression(Expressions.Cond);
+            if (cond === undefined) {
+                continue;
+            }
+            const compareNodes = cond.findDirectExpressions(Expressions.Compare);
+            if (compareNodes.length !== 1) {
+                continue;
+            }
+            const m = compareNodes[0].concatTokens().toUpperCase().match(/^(.+?)\s+IS\s+(?:NOT\s+)?INITIAL\s*$/);
+            if (!m) {
+                continue;
+            }
+            const tableName = m[1];
+            const bodyStructures = ifStru.findDirectStructures(Structures.Body);
+            if (bodyStructures.length !== 1) {
+                continue;
+            }
+            const normalStructures = bodyStructures[0].findDirectStructures(Structures.Normal);
+            if (normalStructures.length !== 1) {
+                continue;
+            }
+            const loopStructures = normalStructures[0].findDirectStructures(Structures.Loop);
+            if (loopStructures.length !== 1) {
+                continue;
+            }
+            const loopStatement = loopStructures[0].findDirectStatement(Statements.Loop);
+            if (loopStatement === undefined) {
+                continue;
+            }
+            const loopSource = loopStatement.findFirstExpression(Expressions.LoopSource);
+            if (loopSource === undefined) {
+                continue;
+            }
+            if (loopSource.concatTokens().toUpperCase() !== tableName) {
+                continue;
+            }
+            const endifStatement = ifStru.findDirectStatement(Statements.EndIf);
+            const fix = edit_helper_1.EditHelper.merge(edit_helper_1.EditHelper.deleteStatement(file, ifStatement), edit_helper_1.EditHelper.deleteStatement(file, endifStatement));
+            issues.push(issue_1.Issue.atStatement(file, ifStatement, "Unnecessary IS [NOT] INITIAL check before LOOP AT", this.getMetadata().key, this.conf.severity, fix));
+        }
+        return issues;
+    }
+}
+exports.UnnecessaryLoopGuard = UnnecessaryLoopGuard;
+//# sourceMappingURL=unnecessary_loop_guard.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/unnecessary_pragma.js"
 /*!***************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/unnecessary_pragma.js ***!
@@ -101497,6 +101649,103 @@ exports.UseLineExists = UseLineExists;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/use_message_class.js"
+/*!**************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/use_message_class.js ***!
+  \**************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.UseMessageClass = exports.UseMessageClassConf = 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 Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.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");
+class UseMessageClassConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.UseMessageClassConf = UseMessageClassConf;
+class UseMessageClass extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new UseMessageClassConf();
+    }
+    getMetadata() {
+        return {
+            key: "use_message_class",
+            title: "Use message class",
+            shortDescription: `Define messages in SE91 instead of using inline text or variables`,
+            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `MESSAGE 'Something went wrong' TYPE 'E'.`,
+            goodExample: `MESSAGE e001(bc_msg).`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, _obj) {
+        const structure = file.getStructure();
+        if (structure === undefined) {
+            return [];
+        }
+        const issues = [];
+        const key = this.getMetadata().key;
+        for (const statement of structure.findAllStatements(Statements.Message)) {
+            // The parser populates MessageSource only for the class-based forms (e001(bc_msg) or ID/TYPE/NUMBER).
+            // The text form (MESSAGE <source> TYPE <source>) uses MessageSourceSource directly with no MessageSource child.
+            // Absence of MessageSource is therefore a reliable discriminator between the two forms.
+            if (statement.findDirectExpression(Expressions.MessageSource) !== undefined) {
+                continue;
+            }
+            issues.push(issue_1.Issue.atStatement(file, statement, "Use a message class (SE91) instead of inline text or variable", key, this.conf.severity));
+        }
+        return issues;
+    }
+}
+exports.UseMessageClass = UseMessageClass;
+//# sourceMappingURL=use_message_class.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/use_new.js"
 /*!****************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/use_new.js ***!
@@ -104938,6 +105187,7 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.AttributeNameTranspiler = void 0;
 const abaplint = __importStar(__webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js"));
+const traversal_1 = __webpack_require__(/*! ../traversal */ "./node_modules/@abaplint/transpiler/build/src/traversal.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
 class AttributeNameTranspiler {
     transpile(node, traversal) {
@@ -104948,6 +105198,16 @@ class AttributeNameTranspiler {
             && ref.getVisibility() === abaplint.Visibility.Private) {
             concat = "#" + concat;
         }
+        // an attribute declared in an interface lives on the object as "<intf>$<name>",
+        // also when it is reached via an interface reference without the "intf~" alias,
+        // eg. "obj->method( )->value" where method returns "REF TO lif_intf"
+        const interfaceName = traversal.isInterfaceAttribute(node.getFirstToken());
+        if (interfaceName && concat.startsWith(interfaceName) === false) {
+            concat = traversal_1.Traversal.escapeNamespace(interfaceName) + "$" + traversal_1.Traversal.escapeNamespace(concat).replace("~", "$");
+        }
+        else {
+            concat = traversal_1.Traversal.escapeNamespace(concat).replace("~", "$");
+        }
         return new chunk_1.Chunk().append(concat, node, traversal);
     }
 }
@@ -111645,6 +111905,7 @@ exports.Transpiler = exports.Chunk = exports.UnknownTypesEnum = exports.config =
 const abaplint = __importStar(__webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js"));
 const validation_1 = __webpack_require__(/*! ./validation */ "./node_modules/@abaplint/transpiler/build/src/validation.js");
 Object.defineProperty(exports, "config", ({ enumerable: true, get: function () { return validation_1.config; } }));
+const unique_identifier_1 = __webpack_require__(/*! ./unique_identifier */ "./node_modules/@abaplint/transpiler/build/src/unique_identifier.js");
 const unit_test_1 = __webpack_require__(/*! ./unit_test */ "./node_modules/@abaplint/transpiler/build/src/unit_test.js");
 const types_1 = __webpack_require__(/*! ./types */ "./node_modules/@abaplint/transpiler/build/src/types.js");
 Object.defineProperty(exports, "UnknownTypesEnum", ({ enumerable: true, get: function () { return types_1.UnknownTypesEnum; } }));
@@ -111703,6 +111964,12 @@ class Transpiler {
         progress?.set(reg.getObjectCount().total, "Building");
         for (const obj of reg.getObjects()) {
             await progress?.tick("Building, " + obj.getName());
+            // the temporary names ("unique1", ..., and the DO/WHILE sy-index backups
+            // "indexBackup1", ...) are local to the module an object becomes, so each
+            // object numbers its own: its output then depends on the registry alone,
+            // not on the objects built before it or on an earlier run in the same process
+            unique_identifier_1.UniqueIdentifier.reset();
+            unique_identifier_1.UniqueIdentifier.resetIndexBackup();
             if (obj instanceof abaplint.Objects.TypePool) {
                 output.objects.push(...new handle_type_pool_1.HandleTypePool().runObject(obj, reg));
             }
@@ -115118,10 +115385,12 @@ class DeleteInternalTranspiler {
             extra.push("where: async " + traversal.traverse(componentCompare).getCode());
         }
         // todo, this is not completely correct, fields might have the name ADJACENT
-        // comparisons should be on table key unless other is specified, but we're unaware
         if (node.findDirectTokenByText("ADJACENT")) {
             extra.push("adjacent: true");
-            if (node.findDirectTokenByText("COMPARING") && !concat.includes("COMPARING ALL FIELDS")) {
+            if (concat.includes("COMPARING ALL FIELDS")) {
+                extra.push("allFields: true");
+            }
+            else if (node.findDirectTokenByText("COMPARING")) {
                 const comparing = node.findAllExpressions(abaplint.Expressions.SimpleFieldChain2);
                 if (comparing) {
                     const compareFields = comparing.map(i => "'" + i.getFirstToken().getStr() + "'").join(",");
@@ -117336,6 +117605,9 @@ class InsertDatabaseTranspiler {
             const tvalues = traversal.traverse(fromTable);
             options.push(`"table": ` + tvalues.getCode());
         }
+        if (node.findDirectTokenByText("ACCEPTING") !== undefined) {
+            options.push(`"acceptingDuplicateKeys": true`);
+        }
         const connection = node.findDirectExpression(abaplint.Expressions.DatabaseConnection);
         if (connection) {
             const con = findConnection(connection, traversal);
@@ -119514,10 +119786,22 @@ class ReadTableTranspiler {
                 const s = traversal.traverse(source).getCode();
                 let field = "";
                 let key = left.concatTokens().toLowerCase();
+                // the component name is a JS object key, computed at runtime for a dynamic variable
+                let keyExpression = "";
                 if (left.get() instanceof abaplint.Expressions.Dynamic
                     && left instanceof abaplint.Nodes.ExpressionNode) {
-                    key = key.substring(2, key.length - 2);
-                    field = "i." + key;
+                    const chain = left.findDirectExpression(abaplint.Expressions.FieldChain);
+                    if (chain) {
+                        // WITH TABLE KEY (variable) = value
+                        const name = new expressions_1.FieldChainTranspiler(true).transpile(chain, traversal).getCode();
+                        keyExpression = name + ".trimEnd().toLowerCase()";
+                        field = "i[" + keyExpression + "]";
+                    }
+                    else {
+                        // WITH TABLE KEY ('LITERAL') = value
+                        key = key.substring(2, key.length - 2);
+                        field = "i." + key;
+                    }
                 }
                 else if (left.get() instanceof abaplint.Expressions.ComponentChainSimple
                     && left instanceof abaplint.Nodes.ExpressionNode) {
@@ -119529,17 +119813,18 @@ class ReadTableTranspiler {
                 if (field === "i.table_line") {
                     usesTableLine = true;
                 }
+                const simpleKey = keyExpression !== "" ? "[" + keyExpression + "]" : JSON.stringify(key);
                 if (s.includes("await")) {
                     const id = unique_identifier_1.UniqueIdentifier.get();
                     prefix += "const " + id + " = " + s + ";\n";
                     withKey.push("abap.compare.eq(" + field + ", " + id + ")");
                     withKeyValue.push(`{key: (i) => {return ${field}}, value: ${id}}`);
-                    withKeySimple.push(`${JSON.stringify(key)}: ${id}`);
+                    withKeySimple.push(`${simpleKey}: ${id}`);
                 }
                 else {
                     withKey.push("abap.compare.eq(" + field + ", " + s + ")");
                     withKeyValue.push(`{key: (i) => {return ${field}}, value: ${s}}`);
-                    withKeySimple.push(`${JSON.stringify(key)}: ${s}`);
+                    withKeySimple.push(`${simpleKey}: ${s}`);
                 }
             }
             extra.push("withKey: (i) => {return " + withKey.join(" && ") + ";}");
@@ -120273,7 +120558,7 @@ class SelectTranspiler {
   }
   if (!(${target} instanceof abap.types.HashedTable) && ${target}.getOptions()?.primaryKey?.type !== "SORTED") {
     abap.statements.sort(${target}, {by: ${by}.map(k => { return {component: k}; })});
-    await abap.statements.deleteInternal(${target}, {adjacent: true, by: ${by}});
+    await abap.statements.deleteInternal(${target}, {adjacent: true, allFields: true});
   }
   abap.builtin.sy.get().dbcnt.set(${target}.getArrayLength());
 }`;
@@ -124459,6 +124744,7 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Traversal = void 0;
+exports.referencesAt = referencesAt;
 const abaplint = __importStar(__webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js"));
 const StatementTranspilers = __importStar(__webpack_require__(/*! ./statements */ "./node_modules/@abaplint/transpiler/build/src/statements/index.js"));
 const ExpressionTranspilers = __importStar(__webpack_require__(/*! ./expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js"));
@@ -124467,6 +124753,43 @@ const transpile_types_1 = __webpack_require__(/*! ./transpile_types */ "./node_m
 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");
+/** the references of one scope, by where they start: built once per scope,
+ *  so finding the reference at a token no longer walks every reference of
+ *  the scope for every token, which made a long method quadratic */
+const REFERENCE_INDEX = new WeakMap();
+/** the references of a scope whose start equals the token's start, in the
+ *  scope's own order -- the same set and order the linear scan found:
+ *  Position.equals compares row and column, a VirtualPosition compares its
+ *  virtual row and column and only with another VirtualPosition */
+function referencesAt(scope, token) {
+    const references = scope.getData().references;
+    let index = REFERENCE_INDEX.get(references);
+    if (index === undefined) {
+        index = { plain: new Map(), virtual: new Map() };
+        references.forEach((r, n) => {
+            const start = r.position.getStart();
+            const key = start instanceof abaplint.VirtualPosition
+                ? start.vrow + ":" + start.vcol
+                : start.getRow() + ":" + start.getCol();
+            const map = start instanceof abaplint.VirtualPosition ? index.virtual : index.plain;
+            const list = map.get(key);
+            if (list === undefined) {
+                map.set(key, [n]);
+            }
+            else {
+                list.push(n);
+            }
+        });
+        REFERENCE_INDEX.set(references, index);
+    }
+    const start = token.getStart();
+    const found = [...(index.plain.get(start.getRow() + ":" + start.getCol()) ?? [])];
+    if (start instanceof abaplint.VirtualPosition) {
+        found.push(...(index.virtual.get(start.vrow + ":" + start.vcol) ?? []));
+        found.sort((a, b) => a - b);
+    }
+    return found.map(n => references[n]);
+}
 class Traversal {
     spaghetti;
     file;
@@ -124820,9 +125143,8 @@ class Traversal {
         if (scope === undefined) {
             return false;
         }
-        for (const r of scope.getData().references) {
-            if (r.referenceType === abaplint.ReferenceType.BuiltinMethodReference
-                && r.position.getStart().equals(token.getStart())) {
+        for (const r of referencesAt(scope, token)) {
+            if (r.referenceType === abaplint.ReferenceType.BuiltinMethodReference) {
                 return true;
             }
         }
@@ -124842,9 +125164,8 @@ class Traversal {
         if (scope === undefined) {
             return undefined;
         }
-        for (const r of scope.getData().references) {
+        for (const r of referencesAt(scope, token)) {
             if (r.referenceType === abaplint.ReferenceType.MethodReference
-                && r.position.getStart().equals(token.getStart())
                 && r.resolved instanceof abaplint.Types.MethodDefinition) {
                 let name = r.resolved.getName();
                 if (r.extra?.ooName && r.extra?.ooType === "INTF") {
@@ -124858,8 +125179,7 @@ class Traversal {
                 }
                 return candidate;
             }
-            else if (r.referenceType === abaplint.ReferenceType.BuiltinMethodReference
-                && r.position.getStart().equals(token.getStart())) {
+            else if (r.referenceType === abaplint.ReferenceType.BuiltinMethodReference) {
                 const def = r.resolved;
                 const name = def.getName();
                 return { def, name };
@@ -125007,10 +125327,9 @@ class Traversal {
         if (scope === undefined) {
             return undefined;
         }
-        for (const r of scope.getData().references) {
-            if ((r.referenceType === abaplint.ReferenceType.DataReadReference
-                || r.referenceType === abaplint.ReferenceType.DataWriteReference)
-                && r.position.getStart().equals(token.getStart())) {
+        for (const r of referencesAt(scope, token)) {
+            if (r.referenceType === abaplint.ReferenceType.DataReadReference
+                || r.referenceType === abaplint.ReferenceType.DataWriteReference) {
                 return r.resolved;
             }
         }
@@ -125021,9 +125340,8 @@ class Traversal {
         if (scope === undefined) {
             return undefined;
         }
-        for (const r of scope.getData().references) {
-            if (r.referenceType === abaplint.ReferenceType.InferredType
-                && r.position.getStart().equals(token.getStart())) {
+        for (const r of referencesAt(scope, token)) {
+            if (r.referenceType === abaplint.ReferenceType.InferredType) {
                 if (r.resolved instanceof abaplint.TypedIdentifier) {
                     return r.resolved.getType();
                 }
@@ -125036,9 +125354,8 @@ class Traversal {
         if (scope === undefined) {
             return undefined;
         }
-        for (const r of scope.getData().references) {
-            if (r.referenceType === abaplint.ReferenceType.TypeReference
-                && r.position.getStart().equals(token.getStart())) {
+        for (const r of referencesAt(scope, token)) {
+            if (r.referenceType === abaplint.ReferenceType.TypeReference) {
                 if (r.resolved instanceof abaplint.TypedIdentifier) {
                     return r.resolved.getType();
                 }

package/build/types.d.ts

@@ -12,6 +12,13 @@ export interface ITranspilerConfig {
     exclude_filter?: string[];
     /** @minLength 1 */
     output_folder: string;
+    /** experimental */
+    converter?: {
+        /** @uniqueItems true */
+        input_folder: string | string[];
+        /** @minLength 1 */
+        output_folder: string;
+    };
     libs?: {
         url?: string;
         /** relative to the current working directory */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.13.91",
+  "version": "2.13.93",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,8 +27,8 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.120.58",
-    "@abaplint/transpiler": "^2.13.91",
+    "@abaplint/core": "^2.120.59",
+    "@abaplint/transpiler": "^2.13.93",
     "@types/glob": "^9.0.0",
     "@types/node": "^26.6.2",
     "@types/progress": "^2.0.7",