@abaplint/transpiler-cli 2.6.43 → 2.7.0

package/build/bundle.js

@@ -7847,6 +7847,7 @@ class StatementParser {
                 const concat = statement.concatTokens().toUpperCase();
                 if (concat.startsWith("CALL METHOD ") === false
                     && concat.startsWith("RAISE EXCEPTION TYPE ") === false
+                    && concat.startsWith("READ TABLE ") === false
                     && concat.startsWith("LOOP AT ") === false
                     && concat.startsWith("CALL FUNCTION ") === false) {
                     for (const { first, second } of this.buildSplits(statement.getTokens())) {
@@ -18622,7 +18623,7 @@ BuiltIn.methods = [
     {
         name: "BOOLC",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -18630,7 +18631,7 @@ BuiltIn.methods = [
     {
         name: "BOOLX",
         mandatory: {
-            "bool": new basic_1.StringType(),
+            "bool": new basic_1.CLikeType(),
         },
         optional: {
             "bit": new basic_1.IntegerType(),
@@ -18648,7 +18649,7 @@ BuiltIn.methods = [
     {
         name: "CHAR_OFF",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
             "add": new basic_1.IntegerType(),
         },
         optional: {
@@ -18660,23 +18661,23 @@ BuiltIn.methods = [
     {
         name: "CHARLEN",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         return: new basic_1.IntegerType(),
     },
     {
         name: "CMAX",
         mandatory: {
-            "val1": new basic_1.StringType(),
-            "val2": new basic_1.StringType(),
+            "val1": new basic_1.CLikeType(),
+            "val2": new basic_1.CLikeType(),
         },
         optional: {
-            "val3": new basic_1.StringType(),
-            "val4": new basic_1.StringType(),
-            "val5": new basic_1.StringType(),
-            "val6": new basic_1.StringType(),
-            "val7": new basic_1.StringType(),
-            "val9": new basic_1.StringType(),
+            "val3": new basic_1.CLikeType(),
+            "val4": new basic_1.CLikeType(),
+            "val5": new basic_1.CLikeType(),
+            "val6": new basic_1.CLikeType(),
+            "val7": new basic_1.CLikeType(),
+            "val9": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -18684,16 +18685,16 @@ BuiltIn.methods = [
     {
         name: "CMIN",
         mandatory: {
-            "val1": new basic_1.StringType(),
-            "val2": new basic_1.StringType(),
+            "val1": new basic_1.CLikeType(),
+            "val2": new basic_1.CLikeType(),
         },
         optional: {
-            "val3": new basic_1.StringType(),
-            "val4": new basic_1.StringType(),
-            "val5": new basic_1.StringType(),
-            "val6": new basic_1.StringType(),
-            "val7": new basic_1.StringType(),
-            "val9": new basic_1.StringType(),
+            "val3": new basic_1.CLikeType(),
+            "val4": new basic_1.CLikeType(),
+            "val5": new basic_1.CLikeType(),
+            "val6": new basic_1.CLikeType(),
+            "val7": new basic_1.CLikeType(),
+            "val9": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -18704,7 +18705,7 @@ BuiltIn.methods = [
             "table": new basic_1.TableType(new basic_1.AnyType(), { withHeader: false, keyType: basic_1.TableKeyType.default }),
         },
         optional: {
-            "sep": new basic_1.StringType(),
+            "sep": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -18712,12 +18713,12 @@ BuiltIn.methods = [
     {
         name: "CONDENSE",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "del": new basic_1.StringType(),
-            "from": new basic_1.StringType(),
-            "to": new basic_1.StringType(),
+            "del": new basic_1.CLikeType(),
+            "from": new basic_1.CLikeType(),
+            "to": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -18725,14 +18726,14 @@ BuiltIn.methods = [
     {
         name: "CONTAINS",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "start": new basic_1.StringType(),
-            "end": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "start": new basic_1.CLikeType(),
+            "end": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
@@ -18745,12 +18746,12 @@ BuiltIn.methods = [
     {
         name: "CONTAINS_ANY_NOT_OF",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "start": new basic_1.StringType(),
-            "end": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "start": new basic_1.CLikeType(),
+            "end": new basic_1.CLikeType(),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
             "occ": new basic_1.IntegerType(),
@@ -18761,12 +18762,12 @@ BuiltIn.methods = [
     {
         name: "CONTAINS_ANY_OF",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "start": new basic_1.StringType(),
-            "end": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "start": new basic_1.CLikeType(),
+            "end": new basic_1.CLikeType(),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
             "occ": new basic_1.IntegerType(),
@@ -18792,12 +18793,12 @@ BuiltIn.methods = [
     {
         name: "COUNT",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
@@ -18808,12 +18809,12 @@ BuiltIn.methods = [
     {
         name: "COUNT_ANY_NOT_OF",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
@@ -18824,12 +18825,12 @@ BuiltIn.methods = [
     {
         name: "COUNT_ANY_OF",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
@@ -18840,15 +18841,15 @@ BuiltIn.methods = [
     {
         name: "DBMAXLEN",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         return: new basic_1.IntegerType(),
     },
     {
         name: "DISTANCE",
         mandatory: {
-            "val1": new basic_1.StringType(),
-            "val2": new basic_1.StringType(),
+            "val1": new basic_1.CLikeType(),
+            "val2": new basic_1.CLikeType(),
         },
         return: new basic_1.IntegerType(),
         version: version_1.Version.v702,
@@ -18856,8 +18857,8 @@ BuiltIn.methods = [
     {
         name: "ESCAPE",
         mandatory: {
-            "val": new basic_1.StringType(),
-            "format": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
+            "format": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -18872,12 +18873,12 @@ BuiltIn.methods = [
     {
         name: "FIND",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
@@ -18889,10 +18890,10 @@ BuiltIn.methods = [
     {
         name: "FIND_ANY_NOT_OF",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
             "occ": new basic_1.IntegerType(),
@@ -18903,10 +18904,10 @@ BuiltIn.methods = [
     {
         name: "FIND_ANY_OF",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
             "occ": new basic_1.IntegerType(),
@@ -18917,12 +18918,12 @@ BuiltIn.methods = [
     {
         name: "FIND_END",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
@@ -18947,7 +18948,9 @@ BuiltIn.methods = [
     },
     {
         name: "FROM_MIXED",
-        mandatory: { "val": new basic_1.StringType() },
+        mandatory: {
+            "val": new basic_1.CLikeType()
+        },
         optional: {
             "case": new basic_1.CharacterType(1),
             "sep": new basic_1.IntegerType(),
@@ -18959,8 +18962,8 @@ BuiltIn.methods = [
     {
         name: "INSERT",
         mandatory: {
-            "val": new basic_1.StringType(),
-            "sub": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
+            "sub": new basic_1.CLikeType(),
         },
         optional: {
             "off": new basic_1.IntegerType(),
@@ -18989,7 +18992,7 @@ BuiltIn.methods = [
     {
         name: "LINE_INDEX",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         return: new basic_1.IntegerType(),
         version: version_1.Version.v740sp02,
@@ -19018,11 +19021,11 @@ BuiltIn.methods = [
     {
         name: "MATCH",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         }, optional: {
             "case": new basic_1.CharacterType(1),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "occ": new basic_1.IntegerType(),
         },
         return: new basic_1.StringType(),
@@ -19031,12 +19034,12 @@ BuiltIn.methods = [
     {
         name: "MATCHES",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
             "case": new basic_1.CharacterType(1),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
         },
@@ -19047,17 +19050,17 @@ BuiltIn.methods = [
     {
         name: "NMAX",
         mandatory: {
-            "val1": new basic_1.StringType(),
-            "val2": new basic_1.StringType(),
+            "val1": new basic_1.CLikeType(),
+            "val2": new basic_1.CLikeType(),
         },
         optional: {
-            "val3": new basic_1.StringType(),
-            "val4": new basic_1.StringType(),
-            "val5": new basic_1.StringType(),
-            "val6": new basic_1.StringType(),
-            "val7": new basic_1.StringType(),
-            "val8": new basic_1.StringType(),
-            "val9": new basic_1.StringType(),
+            "val3": new basic_1.CLikeType(),
+            "val4": new basic_1.CLikeType(),
+            "val5": new basic_1.CLikeType(),
+            "val6": new basic_1.CLikeType(),
+            "val7": new basic_1.CLikeType(),
+            "val8": new basic_1.CLikeType(),
+            "val9": new basic_1.CLikeType(),
         },
         return: new basic_1.IntegerType(),
         version: version_1.Version.v702,
@@ -19065,17 +19068,17 @@ BuiltIn.methods = [
     {
         name: "NMIN",
         mandatory: {
-            "val1": new basic_1.StringType(),
-            "val2": new basic_1.StringType(),
+            "val1": new basic_1.CLikeType(),
+            "val2": new basic_1.CLikeType(),
         },
         optional: {
-            "val3": new basic_1.StringType(),
-            "val4": new basic_1.StringType(),
-            "val5": new basic_1.StringType(),
-            "val6": new basic_1.StringType(),
-            "val7": new basic_1.StringType(),
-            "val8": new basic_1.StringType(),
-            "val9": new basic_1.StringType(),
+            "val3": new basic_1.CLikeType(),
+            "val4": new basic_1.CLikeType(),
+            "val5": new basic_1.CLikeType(),
+            "val6": new basic_1.CLikeType(),
+            "val7": new basic_1.CLikeType(),
+            "val8": new basic_1.CLikeType(),
+            "val9": new basic_1.CLikeType(),
         },
         return: new basic_1.IntegerType(),
         version: version_1.Version.v702,
@@ -19083,14 +19086,14 @@ BuiltIn.methods = [
     {
         name: "NUMOFCHAR",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         return: new basic_1.IntegerType(),
     },
     {
         name: "REPEAT",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
             "occ": new basic_1.IntegerType(),
         },
         return: new basic_1.StringType(),
@@ -19099,13 +19102,13 @@ BuiltIn.methods = [
     {
         name: "REPLACE",
         mandatory: {
-            "val": new basic_1.StringType(),
-            "with": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
+            "with": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "off": new basic_1.IntegerType(),
             "len": new basic_1.IntegerType(),
@@ -19130,7 +19133,7 @@ BuiltIn.methods = [
     {
         name: "REVERSE",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -19151,12 +19154,12 @@ BuiltIn.methods = [
     {
         name: "SEGMENT",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
             "index": new basic_1.IntegerType(),
         },
         optional: {
-            "sep": new basic_1.StringType(),
-            "space": new basic_1.StringType(),
+            "sep": new basic_1.CLikeType(),
+            "space": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -19164,10 +19167,10 @@ BuiltIn.methods = [
     {
         name: "SHIFT_LEFT",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
             "places": new basic_1.IntegerType(),
             "circular": new basic_1.IntegerType(),
         },
@@ -19177,10 +19180,10 @@ BuiltIn.methods = [
     {
         name: "SHIFT_RIGHT",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
             "places": new basic_1.IntegerType(),
             "circular": new basic_1.IntegerType(),
         },
@@ -19240,9 +19243,9 @@ BuiltIn.methods = [
             "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "len": new basic_1.IntegerType(),
             "occ": new basic_1.IntegerType(),
@@ -19256,9 +19259,9 @@ BuiltIn.methods = [
             "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "len": new basic_1.IntegerType(),
             "occ": new basic_1.IntegerType(),
@@ -19272,9 +19275,9 @@ BuiltIn.methods = [
             "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "len": new basic_1.IntegerType(),
             "occ": new basic_1.IntegerType(),
@@ -19288,9 +19291,9 @@ BuiltIn.methods = [
             "val": new basic_1.CLikeType(),
         },
         optional: {
-            "sub": new basic_1.StringType(),
-            "regex": new basic_1.StringType(),
-            "pcre": new basic_1.StringType(),
+            "sub": new basic_1.CLikeType(),
+            "regex": new basic_1.CLikeType(),
+            "pcre": new basic_1.CLikeType(),
             "case": new basic_1.CharacterType(1),
             "len": new basic_1.IntegerType(),
             "occ": new basic_1.IntegerType(),
@@ -19315,7 +19318,7 @@ BuiltIn.methods = [
     {
         name: "TO_LOWER",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -19323,7 +19326,7 @@ BuiltIn.methods = [
     {
         name: "TO_MIXED",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         optional: {
             "case": new basic_1.CharacterType(1),
@@ -19335,16 +19338,16 @@ BuiltIn.methods = [
     },
     {
         name: "TO_UPPER",
-        mandatory: { "val": new basic_1.StringType() },
+        mandatory: { "val": new basic_1.CLikeType() },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
     },
     {
         name: "TRANSLATE",
         mandatory: {
-            "val": new basic_1.StringType(),
-            "from": new basic_1.StringType(),
-            "to": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
+            "from": new basic_1.CLikeType(),
+            "to": new basic_1.CLikeType(),
         },
         return: new basic_1.StringType(),
         version: version_1.Version.v702,
@@ -19387,7 +19390,7 @@ BuiltIn.methods = [
     {
         name: "XSDBOOL",
         mandatory: {
-            "val": new basic_1.StringType(),
+            "val": new basic_1.CLikeType(),
         },
         return: new basic_1.CharacterType(1),
         version: version_1.Version.v740sp08,
@@ -19738,6 +19741,9 @@ class CurrentScope {
     getDDICReferences() {
         return this.reg.getDDICReferences();
     }
+    getMSAGReferences() {
+        return this.reg.getMSAGReferences();
+    }
     getParentObj() {
         return this.parentObj;
     }
@@ -20604,7 +20610,7 @@ class TypeUtils {
         return false;
     }
     isAssignableStrict(source, target) {
-        var _a, _b, _c, _d;
+        var _a, _b, _c, _d, _e;
         /*
             console.dir(source);
             console.dir(target);
@@ -20622,16 +20628,22 @@ class TypeUtils {
                 }
                 return false;
             }
+            else if (target instanceof basic_1.StringType) {
+                if (((_c = source.getAbstractTypeData()) === null || _c === void 0 ? void 0 : _c.derivedFromConstant) === true) {
+                    return true;
+                }
+                return false;
+            }
         }
         else if (source instanceof basic_1.HexType) {
             if (target instanceof basic_1.HexType) {
-                if (((_c = source.getAbstractTypeData()) === null || _c === void 0 ? void 0 : _c.derivedFromConstant) === true) {
+                if (((_d = source.getAbstractTypeData()) === null || _d === void 0 ? void 0 : _d.derivedFromConstant) === true) {
                     return source.getLength() <= target.getLength();
                 }
                 return source.getLength() === target.getLength();
             }
             else if (target instanceof basic_1.IntegerType) {
-                if (((_d = source.getAbstractTypeData()) === null || _d === void 0 ? void 0 : _d.derivedFromConstant) === true) {
+                if (((_e = source.getAbstractTypeData()) === null || _e === void 0 ? void 0 : _e.derivedFromConstant) === true) {
                     return true;
                 }
                 return false;
@@ -23287,9 +23299,26 @@ 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");
 class MessageSource {
     runSyntax(node, scope, filename) {
+        var _a, _b, _c, _d;
         for (const f of node.findDirectExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(f, scope, filename);
         }
+        if (node.getFirstToken().getStr().toUpperCase() === "ID") {
+            const id = (_a = node.findExpressionAfterToken("ID")) === null || _a === void 0 ? void 0 : _a.concatTokens();
+            const number = (_b = node.findDirectExpression(Expressions.MessageNumber)) === null || _b === void 0 ? void 0 : _b.concatTokens();
+            if ((id === null || id === void 0 ? void 0 : id.startsWith("'")) && number) {
+                const messageClass = id.substring(1, id.length - 1).toUpperCase();
+                scope.getMSAGReferences().addUsing(filename, node.getFirstToken(), messageClass, number);
+            }
+        }
+        else {
+            const typeAndNumber = (_c = node.findDirectExpression(Expressions.MessageTypeAndNumber)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+            const messageNumber = typeAndNumber === null || typeAndNumber === void 0 ? void 0 : typeAndNumber.substring(1);
+            const messageClass = (_d = node.findDirectExpression(Expressions.MessageClass)) === null || _d === void 0 ? void 0 : _d.concatTokens().toUpperCase();
+            if (messageNumber && messageClass) {
+                scope.getMSAGReferences().addUsing(filename, node.getFirstToken(), messageClass, messageNumber);
+            }
+        }
     }
 }
 exports.MessageSource = MessageSource;
@@ -28680,7 +28709,7 @@ const raise_with_1 = __webpack_require__(/*! ../expressions/raise_with */ "./nod
 class Raise {
     runSyntax(node, scope, filename) {
         // todo
-        var _a;
+        var _a, _b, _c;
         const classTok = (_a = node.findDirectExpression(Expressions.ClassName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
         const classNam = classTok === null || classTok === void 0 ? void 0 : classTok.getStr();
         if (classNam) {
@@ -28729,6 +28758,12 @@ class Raise {
         for (const s of node.findDirectExpressions(Expressions.MessageSource)) {
             new message_source_1.MessageSource().runSyntax(s, scope, filename);
         }
+        const id = (_b = node.findExpressionAfterToken("ID")) === null || _b === void 0 ? void 0 : _b.concatTokens();
+        const number = (_c = node.findDirectExpression(Expressions.MessageNumber)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+        if ((id === null || id === void 0 ? void 0 : id.startsWith("'")) && number) {
+            const messageClass = id.substring(1, id.length - 1).toUpperCase();
+            scope.getMSAGReferences().addUsing(filename, node.getFirstToken(), messageClass, number);
+        }
     }
 }
 exports.Raise = Raise;
@@ -29772,7 +29807,7 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Tables = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifier */ "./node_modules/@abaplint/core/build/src/abap/types/_typed_identifier.js");
-const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const unknown_type_1 = __webpack_require__(/*! ../../types/basic/unknown_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/unknown_type.js");
 class Tables {
     runSyntax(node, scope, filename) {
         var _a, _b;
@@ -29784,13 +29819,15 @@ class Tables {
         if (name.startsWith("*")) {
             name = name.substr(1);
         }
+        // lookupTableOrView will also give Unknown and Void
         const found = (_b = scope.getDDIC()) === null || _b === void 0 ? void 0 : _b.lookupTableOrView(name);
         if (found) {
             scope.getDDICReferences().addUsing(scope.getParentObj(), { object: found.object, filename: filename, token: nameToken });
             scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, filename, found.type));
             return;
         }
-        scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, filename, new basic_1.UnknownType("Tables, fallback")));
+        // this should never happen,
+        scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, filename, new unknown_type_1.UnknownType("Tables, fallback")));
     }
 }
 exports.Tables = Tables;
@@ -30996,6 +31033,7 @@ class SyntaxLogic {
         }
         this.issues = [];
         this.reg.getDDICReferences().clear(this.object);
+        this.reg.getMSAGReferences().clear(this.object);
         if (this.object instanceof objects_1.Program && this.object.isInclude()) {
             // todo, show some kind of error?
             return { issues: [], spaghetti: this.scope.pop(new position_1.Position(Number.MAX_SAFE_INTEGER, Number.MAX_SAFE_INTEGER)) };
@@ -35030,21 +35068,21 @@ exports.InterfaceDefinition = InterfaceDefinition;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Message = void 0;
 class Message {
-    constructor(num, msg) {
-        this.num = num;
-        if (this.num === undefined) {
-            this.num = "";
+    constructor(number, message) {
+        this.number = number;
+        if (this.number === undefined) {
+            this.number = "";
         }
-        this.msg = msg;
-        if (this.msg === undefined) {
-            this.msg = "";
+        this.message = message;
+        if (this.message === undefined) {
+            this.message = "";
         }
     }
     getNumber() {
-        return this.num;
+        return this.number;
     }
     getMessage() {
-        return this.msg;
+        return this.message;
     }
     getPlaceholderCount() {
         return (this.getMessage().match(/&/g) || []).length;
@@ -37359,10 +37397,11 @@ class DDICReferences {
                 }
             }
         }
-        const newName = obj.getName().toUpperCase();
-        const newType = obj.getType();
-        if ((_b = this.nameTypeIndex[newName]) === null || _b === void 0 ? void 0 : _b[newType]) {
-            this.nameTypeIndex[newName][newType] = [];
+        // remove from name + type index
+        const name = obj.getName().toUpperCase();
+        const type = obj.getType();
+        if ((_b = this.nameTypeIndex[name]) === null || _b === void 0 ? void 0 : _b[type]) {
+            this.nameTypeIndex[name][type] = [];
         }
     }
     listByFilename(filename, line) {
@@ -38435,21 +38474,27 @@ class LSPLookup {
             const hover = "Method Definition \"" + method.getName() + "\"";
             return { hover, definition: found, definitionId: method, scope: bottomScope };
         }
+        let hoverValue = "";
+        const ddicRefs = reg.getDDICReferences().listByFilename(cursor.identifier.getFilename(), cursor.identifier.getStart().getRow());
+        for (const d of ddicRefs) {
+            if (d.object && d.token && d.token.getStart().equals(cursor.identifier.getStart())) {
+                hoverValue += `DDIC: ${d.object.getType()} ${d.object.getName()}`;
+            }
+        }
         const variable = bottomScope.findVariable(cursor.token.getStr());
         if (variable !== undefined && variable.getStart().equals(cursor.token.getStart())) {
             const hover = "Variable Definition\n\n" + this.dumpType(variable);
+            if (hoverValue !== "") {
+                hoverValue = hover + "\n_________________\n" + hoverValue;
+            }
+            else {
+                hoverValue = hover;
+            }
             let location = undefined;
             if (variable.getMeta().includes("built-in" /* IdentifierMeta.BuiltIn */) === false) {
                 location = _lsp_utils_1.LSPUtils.identiferToLocation(variable);
             }
-            return { hover, definition: location, implementation: location, definitionId: variable, scope: bottomScope };
-        }
-        let hoverValue = "";
-        const ddicRefs = reg.getDDICReferences().listByFilename(cursor.identifier.getFilename(), cursor.identifier.getStart().getRow());
-        for (const d of ddicRefs) {
-            if (d.object && d.token && d.token.getStart().equals(cursor.identifier.getStart())) {
-                hoverValue += `DDIC: ${d.object.getType()} ${d.object.getName()}`;
-            }
+            return { hover: hoverValue, definition: location, implementation: location, definitionId: variable, scope: bottomScope };
         }
         const refs = this.searchReferences(bottomScope, cursor.token);
         if (refs.length > 0) {
@@ -38861,6 +38906,53 @@ exports.CodeActions = CodeActions;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/lsp/code_lens.js":
+/*!****************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/lsp/code_lens.js ***!
+  \****************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.CodeLens = void 0;
+const LServer = __webpack_require__(/*! vscode-languageserver-types */ "./node_modules/vscode-languageserver-types/lib/esm/main.js");
+const _lsp_utils_1 = __webpack_require__(/*! ./_lsp_utils */ "./node_modules/@abaplint/core/build/src/lsp/_lsp_utils.js");
+const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/syntax.js");
+const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
+class CodeLens {
+    constructor(reg) {
+        this.reg = reg;
+    }
+    list(textDocument) {
+        var _a;
+        const file = _lsp_utils_1.LSPUtils.getABAPFile(this.reg, textDocument.uri);
+        if (file === undefined) {
+            return [];
+        }
+        const obj = this.reg.findObjectForFile(file);
+        if (obj === undefined || !(obj instanceof _abap_object_1.ABAPObject)) {
+            return [];
+        }
+        new syntax_1.SyntaxLogic(this.reg, obj).run();
+        const ret = [];
+        const list = this.reg.getMSAGReferences().listByFilename(file.getFilename());
+        for (const l of list) {
+            const msag = this.reg.getObject("MSAG", l.messageClass);
+            if (msag === undefined) {
+                continue;
+            }
+            const text = (_a = msag.getByNumber(l.number)) === null || _a === void 0 ? void 0 : _a.getMessage();
+            ret.push(LServer.CodeLens.create(_lsp_utils_1.LSPUtils.tokenToRange(l.token), text));
+        }
+        return ret;
+    }
+}
+exports.CodeLens = CodeLens;
+//# sourceMappingURL=code_lens.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/lsp/definition.js":
 /*!*****************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/lsp/definition.js ***!
@@ -39558,6 +39650,7 @@ const references_1 = __webpack_require__(/*! ./references */ "./node_modules/@ab
 const implementation_1 = __webpack_require__(/*! ./implementation */ "./node_modules/@abaplint/core/build/src/lsp/implementation.js");
 const semantic_1 = __webpack_require__(/*! ./semantic */ "./node_modules/@abaplint/core/build/src/lsp/semantic.js");
 const statement_flow_1 = __webpack_require__(/*! ../abap/flow/statement_flow */ "./node_modules/@abaplint/core/build/src/abap/flow/statement_flow.js");
+const code_lens_1 = __webpack_require__(/*! ./code_lens */ "./node_modules/@abaplint/core/build/src/lsp/code_lens.js");
 // note Ranges are zero based in LSP,
 // https://github.com/microsoft/language-server-protocol/blob/main/versions/protocol-2-x.md#range
 // but 1 based in abaplint
@@ -39638,6 +39731,10 @@ class LanguageServer {
     semanticTokensRange(range) {
         return new semantic_1.SemanticHighlighting(this.reg).semanticTokensRange(range);
     }
+    // https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_codeLens
+    codeLens(textDocument) {
+        return new code_lens_1.CodeLens(this.reg).list(textDocument);
+    }
     ////////////////////////////////////////
     //  ______      _
     // |  ____|    | |
@@ -40121,6 +40218,65 @@ exports.Symbols = Symbols;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/msag_references.js":
+/*!******************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/msag_references.js ***!
+  \******************************************************************/
+/***/ ((__unused_webpack_module, exports) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.MSAGReferences = void 0;
+class MSAGReferences {
+    constructor() {
+        this.nameNumberIndex = {};
+        this.filenameIndex = {};
+    }
+    addUsing(filename, token, messageClass, number) {
+        if (this.filenameIndex[filename] === undefined) {
+            this.filenameIndex[filename] = [];
+        }
+        this.filenameIndex[filename].push({
+            token: token,
+            messageClass: messageClass,
+            number: number,
+        });
+        if (this.nameNumberIndex[messageClass] === undefined) {
+            this.nameNumberIndex[messageClass] = {};
+        }
+        if (this.nameNumberIndex[messageClass][number] === undefined) {
+            this.nameNumberIndex[messageClass][number] = [];
+        }
+        this.nameNumberIndex[messageClass][number].push({
+            filename: filename,
+            token: token,
+        });
+    }
+    clear(obj) {
+        for (const file of obj.getFiles()) {
+            const filename = file.getFilename();
+            for (const fIndex of this.filenameIndex[filename] || []) {
+                // this should be okay for performance, each message should be referenced less than 10 times typically
+                this.nameNumberIndex[fIndex.messageClass][fIndex.number] =
+                    this.nameNumberIndex[fIndex.messageClass][fIndex.number].filter(i => i.filename !== filename);
+            }
+            delete this.filenameIndex[filename];
+        }
+    }
+    listByFilename(filename) {
+        return this.filenameIndex[filename] || [];
+    }
+    listByMessage(messageClass, number) {
+        var _a;
+        return ((_a = this.nameNumberIndex[messageClass]) === null || _a === void 0 ? void 0 : _a[number]) || [];
+    }
+}
+exports.MSAGReferences = MSAGReferences;
+//# sourceMappingURL=msag_references.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js":
 /*!***********************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/_abap_object.js ***!
@@ -43634,6 +43790,7 @@ class MessageClass extends _abstract_object_1.AbstractObject {
         return msg ? msg : [];
     }
     getByNumber(num) {
+        // todo, optimize performance,
         for (const message of this.getMessages()) {
             if (message.getNumber() === num) {
                 return message;
@@ -44037,10 +44194,6 @@ 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");
 class Program extends _abap_object_1.ABAPObject {
-    constructor() {
-        super(...arguments);
-        this.isIncludeValue = undefined;
-    }
     getType() {
         return "PROG";
     }
@@ -44062,15 +44215,25 @@ class Program extends _abap_object_1.ABAPObject {
         };
     }
     setDirty() {
-        this.isIncludeValue = undefined;
+        this.parsedXML = undefined;
         super.setDirty();
     }
     isInclude() {
-        if (this.isIncludeValue === undefined) {
+        this.parseXML();
+        return this.parsedXML.isInclude;
+    }
+    isModulePool() {
+        this.parseXML();
+        return this.parsedXML.isModulePool;
+    }
+    parseXML() {
+        if (this.parsedXML === undefined) {
             const file = this.getXMLFile();
-            this.isIncludeValue = file ? file.getRaw().includes("<SUBC>I</SUBC>") : false;
+            this.parsedXML = {
+                isInclude: file ? file.getRaw().includes("<SUBC>I</SUBC>") : false,
+                isModulePool: file ? file.getRaw().includes("<SUBC>M</SUBC>") : false,
+            };
         }
-        return this.isIncludeValue;
     }
 }
 exports.Program = Program;
@@ -46589,6 +46752,7 @@ const find_global_definitions_1 = __webpack_require__(/*! ./abap/5_syntax/global
 const excludeHelper_1 = __webpack_require__(/*! ./utils/excludeHelper */ "./node_modules/@abaplint/core/build/src/utils/excludeHelper.js");
 const ddic_references_1 = __webpack_require__(/*! ./ddic_references */ "./node_modules/@abaplint/core/build/src/ddic_references.js");
 const rules_runner_1 = __webpack_require__(/*! ./rules_runner */ "./node_modules/@abaplint/core/build/src/rules_runner.js");
+const msag_references_1 = __webpack_require__(/*! ./msag_references */ "./node_modules/@abaplint/core/build/src/msag_references.js");
 // todo, this should really be an instance in case there are multiple Registry'ies
 class ParsingPerformance {
     static clear() {
@@ -46641,14 +46805,18 @@ class Registry {
         this.objectsByType = {};
         this.dependencies = {};
         this.conf = conf ? conf : config_1.Config.getDefault();
-        this.references = new ddic_references_1.DDICReferences();
+        this.ddicReferences = new ddic_references_1.DDICReferences();
+        this.msagReferences = new msag_references_1.MSAGReferences();
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.99.7";
+        return "2.100.2";
     }
     getDDICReferences() {
-        return this.references;
+        return this.ddicReferences;
+    }
+    getMSAGReferences() {
+        return this.msagReferences;
     }
     *getObjects() {
         for (const name in this.objects) {
@@ -46736,7 +46904,8 @@ class Registry {
         const obj = this.find(file.getObjectName(), file.getObjectType());
         obj.removeFile(file);
         if (obj.getFiles().length === 0) {
-            this.references.clear(obj);
+            this.ddicReferences.clear(obj);
+            this.msagReferences.clear(obj);
             this.removeObject(obj);
         }
         return this;
@@ -53942,6 +54111,88 @@ exports.Downport = Downport;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/easy_to_find_messages.js":
+/*!******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/easy_to_find_messages.js ***!
+  \******************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.EasyToFindMessages = exports.EasyToFindMessagesConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
+const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/syntax.js");
+const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
+class EasyToFindMessagesConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.EasyToFindMessagesConf = EasyToFindMessagesConf;
+class EasyToFindMessages {
+    constructor() {
+        this.conf = new EasyToFindMessagesConf();
+    }
+    getMetadata() {
+        return {
+            key: "easy_to_find_messages",
+            title: "Easy to find messages",
+            shortDescription: `Make messages easy to find`,
+            extendedInformation: `All messages must be statically referenced exactly once
+
+Only MESSAGE and RAISE statments are counted as static references
+
+Also see rule "message_exists"
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#make-messages-easy-to-find`,
+            tags: [_irule_1.RuleTag.Styleguide],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(reg) {
+        this.msagReferences = reg.getMSAGReferences();
+        // the SyntaxLogic builds the references
+        for (const obj of reg.getObjects()) {
+            if (obj instanceof _abap_object_1.ABAPObject) {
+                new syntax_1.SyntaxLogic(reg, obj).run();
+            }
+        }
+        return this;
+    }
+    run(object) {
+        const issues = [];
+        if (object.getType() === "MSAG") {
+            const msag = object;
+            for (const message of msag.getMessages()) {
+                const where = this.msagReferences.listByMessage(msag.getName().toUpperCase(), message.getNumber());
+                if (where.length === 0) {
+                    const text = `Message ${message.getNumber()} not statically referenced`;
+                    const position = new position_1.Position(1, 1);
+                    const issue = issue_1.Issue.atPosition(object.getFiles()[0], position, text, this.getMetadata().key, this.conf.severity);
+                    issues.push(issue);
+                }
+                else if (where.length >= 2) {
+                    const text = `Message ${message.getNumber()} referenced more than once`;
+                    const position = new position_1.Position(1, 1);
+                    const issue = issue_1.Issue.atPosition(object.getFiles()[0], position, text, this.getMetadata().key, this.conf.severity);
+                    issues.push(issue);
+                }
+            }
+        }
+        return issues;
+    }
+}
+exports.EasyToFindMessages = EasyToFindMessages;
+//# sourceMappingURL=easy_to_find_messages.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 ***!
@@ -56550,6 +56801,7 @@ __exportStar(__webpack_require__(/*! ./definitions_top */ "./node_modules/@abapl
 __exportStar(__webpack_require__(/*! ./description_empty */ "./node_modules/@abaplint/core/build/src/rules/description_empty.js"), exports);
 __exportStar(__webpack_require__(/*! ./double_space */ "./node_modules/@abaplint/core/build/src/rules/double_space.js"), exports);
 __exportStar(__webpack_require__(/*! ./downport */ "./node_modules/@abaplint/core/build/src/rules/downport.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_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);
@@ -57958,7 +58210,9 @@ class MainFileContents {
         if (stru === undefined) {
             return [];
         }
-        if (obj instanceof Objects.Program && obj.isInclude() === false) {
+        if (obj instanceof Objects.Program
+            && obj.isInclude() === false
+            && obj.isModulePool() === false) {
             let count = 0;
             let first = main.getStatements()[count];
             while (first !== undefined && first.get() instanceof _statement_1.Comment) {
@@ -59019,6 +59273,7 @@ class MSAGConsistency {
             key: "msag_consistency",
             title: "MSAG consistency check",
             shortDescription: `Checks the validity of messages in message classes`,
+            extendedInformation: `Message numbers must be 3 digits, and message text must not be empty`,
         };
     }
     getDescription(reason) {
@@ -62630,6 +62885,10 @@ If the target is a sorted/hashed table, no issue is reported`,
                 if (this.isTargetSortedOrHashed(s, spaghetti, file)) {
                     continue;
                 }
+                else if (s.findFirstExpression(Expressions.SQLJoin) && s.findFirstExpression(Expressions.SQLForAllEntries)) {
+                    // see https://github.com/abaplint/abaplint/issues/2957
+                    continue;
+                }
                 issues.push(issue_1.Issue.atStatement(file, s, "Add ORDER BY", this.getMetadata().key, this.conf.severity));
             }
         }
@@ -64013,7 +64272,6 @@ Activates from v750 and up`,
                 const message = "INTO/APPENDING must be last in strict SQL";
                 const issue = issue_1.Issue.atToken(file, s.getFirstToken(), message, this.getMetadata().key, this.conf.severity, fix);
                 issues.push(issue);
-                break;
             }
         }
         return issues;
@@ -71888,6 +72146,27 @@ exports.AtTranspiler = AtTranspiler;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/transpiler/build/src/statements/at_selection_screen.js":
+/*!***************************************************************************************!*\
+  !*** ./node_modules/@abaplint/transpiler/build/src/statements/at_selection_screen.js ***!
+  \***************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.AtSelectionScreenTranspiler = void 0;
+const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
+class AtSelectionScreenTranspiler {
+    transpile(_node, _traversal) {
+        return new chunk_1.Chunk(`throw new Error("AtSelectionScreen, not supported, transpiler");`);
+    }
+}
+exports.AtSelectionScreenTranspiler = AtSelectionScreenTranspiler;
+//# sourceMappingURL=at_selection_screen.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/transpiler/build/src/statements/authority_check.js":
 /*!***********************************************************************************!*\
   !*** ./node_modules/@abaplint/transpiler/build/src/statements/authority_check.js ***!
@@ -74136,6 +74415,7 @@ __exportStar(__webpack_require__(/*! ./add */ "./node_modules/@abaplint/transpil
 __exportStar(__webpack_require__(/*! ./append */ "./node_modules/@abaplint/transpiler/build/src/statements/append.js"), exports);
 __exportStar(__webpack_require__(/*! ./assert */ "./node_modules/@abaplint/transpiler/build/src/statements/assert.js"), exports);
 __exportStar(__webpack_require__(/*! ./assign */ "./node_modules/@abaplint/transpiler/build/src/statements/assign.js"), exports);
+__exportStar(__webpack_require__(/*! ./at_selection_screen */ "./node_modules/@abaplint/transpiler/build/src/statements/at_selection_screen.js"), exports);
 __exportStar(__webpack_require__(/*! ./at */ "./node_modules/@abaplint/transpiler/build/src/statements/at.js"), exports);
 __exportStar(__webpack_require__(/*! ./authority_check */ "./node_modules/@abaplint/transpiler/build/src/statements/authority_check.js"), exports);
 __exportStar(__webpack_require__(/*! ./break_id */ "./node_modules/@abaplint/transpiler/build/src/statements/break_id.js"), exports);
@@ -74876,6 +75156,11 @@ class MethodImplementationTranspiler {
         const method = this.findMethod(methodName, classDef, traversal);
         let staticMethod = "";
         methodName = methodName.replace("~", "$").toLowerCase();
+        if (methodName === "then") {
+            // todo, this should be a checked in the validation step, but no abaplint rule for it?
+            // it messes up promises when "this" is returned
+            throw new Error(`Method name "then" not allowed`);
+        }
         const superDef = traversal.findClassDefinition(classDef === null || classDef === void 0 ? void 0 : classDef.getSuperClass(), scope);
         for (const a of (superDef === null || superDef === void 0 ? void 0 : superDef.getAliases().getAll()) || []) {
             if (a.getName().toLowerCase() === methodName) {
@@ -75773,6 +76058,7 @@ class SelectTranspiler {
         var _a;
         let target = "undefined";
         if (targetOverride) {
+            // SelectLoop structure uses override
             target = targetOverride;
         }
         else if (node.findFirstExpression(abaplint.Expressions.SQLIntoTable)) {
@@ -75834,8 +76120,12 @@ class SelectTranspiler {
             select += "UP TO 1 ROWS";
         }
         let runtimeOptions = "";
+        const runtimeOptionsList = [];
         if (node.concatTokens().toUpperCase().includes(" APPENDING TABLE ")) {
-            runtimeOptions = `, {appending: true}`;
+            runtimeOptionsList.push(`appending: true`);
+        }
+        if (runtimeOptionsList.length > 0) {
+            runtimeOptions = `, {` + runtimeOptionsList.join(", ") + `}`;
         }
         let extra = "";
         const keys = this.findKeys(node, traversal);
@@ -77710,6 +78000,7 @@ exports.LoopTranspiler = LoopTranspiler;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.SelectTranspiler = void 0;
+/* eslint-disable max-len */
 const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
 const select_1 = __webpack_require__(/*! ../statements/select */ "./node_modules/@abaplint/transpiler/build/src/statements/select.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
@@ -77717,7 +78008,7 @@ const unique_identifier_1 = __webpack_require__(/*! ../unique_identifier */ "./n
 const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 class SelectTranspiler {
     transpile(node, traversal) {
-        var _a;
+        var _a, _b;
         const ret = new chunk_1.Chunk();
         const selectStatement = node.findFirstStatement(abaplint.Statements.SelectLoop);
         if (selectStatement === undefined) {
@@ -77731,11 +78022,19 @@ class SelectTranspiler {
         const loopName = unique_identifier_1.UniqueIdentifier.get();
         ret.appendString(`let ${targetName} = new abap.types.Table(abap.DDIC["${from}"].type);\n`);
         ret.appendChunk(new select_1.SelectTranspiler().transpile(selectStatement, traversal, targetName));
-        ret.appendString(`\nfor (const ${loopName} of ${targetName}.array()) {\n`);
-        if (concat.includes(" INTO CORRESPONDING FIELDS OF ")) {
+        const packageSize = (_b = node.findFirstExpression(abaplint.Expressions.SelectLoop)) === null || _b === void 0 ? void 0 : _b.findExpressionAfterToken("SIZE");
+        if (packageSize) {
+            const getSize = new expressions_1.SQLSourceTranspiler().transpile(packageSize, traversal).getCode() + ".get()";
+            ret.appendString(`if (${targetName}.array().length > ${getSize}) { throw new Error("PACKAGE SIZED loop larger than package size not supported"); };\n`);
+            ret.appendString(`abap.statements.append({source: ${targetName}, target: ${intoName}, lines: true});\n`);
+            ret.appendString(`{\n`);
+        }
+        else if (concat.includes(" INTO CORRESPONDING FIELDS OF ")) {
+            ret.appendString(`\nfor (const ${loopName} of ${targetName}.array()) {\n`);
             ret.appendString(`abap.statements.moveCorresponding(${loopName}, ${intoName});\n`);
         }
         else {
+            ret.appendString(`\nfor (const ${loopName} of ${targetName}.array()) {\n`);
             ret.appendString(`${intoName}.set(${loopName});\n`);
         }
         const body = node.findDirectStructure(abaplint.Structures.Body);
@@ -78925,13 +79224,11 @@ async function run() {
             }
         }
         ret += `// -------------------END-------------------
-    console.log(abap.console.get());
     fs.writeFileSync(__dirname + path.sep + "_output.xml", unit.xUnitXML());
   } catch (e) {
     if (meth) {
       meth.fail();
     }
-    console.log(abap.console.get());
     fs.writeFileSync(__dirname + path.sep + "_output.xml", unit.xUnitXML());
     throw e;
   }

package/package.json

@@ -1,7 +1,8 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.6.43",
+  "version": "2.7.0",
   "description": "Transpiler - Command Line Interface",
+  "funding": "https://github.com/sponsors/larshp",
   "bin": {
     "abap_transpile": "./abap_transpile"
   },
@@ -25,15 +26,15 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.6.43",
+    "@abaplint/transpiler": "^2.7.0",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
-    "@types/node": "^20.1.0",
-    "@abaplint/core": "^2.99.7",
+    "@types/node": "^20.1.4",
+    "@abaplint/core": "^2.100.2",
     "progress": "^2.0.3",
-    "webpack": "^5.82.0",
-    "webpack-cli": "^5.1.0",
+    "webpack": "^5.82.1",
+    "webpack-cli": "^5.1.1",
     "typescript": "^5.0.4"
   }
 }