@abaplint/transpiler 2.12.29 → 2.12.30

package/build/src/rearranger.d.ts

@@ -1,6 +1,9 @@
 import { Nodes } from "@abaplint/core";
 export declare class Rearranger {
     run(type: string, node: Nodes.StructureNode | undefined): Nodes.StructureNode | undefined;
+    /** Rearranges list event blocks so START-OF-SELECTION runs before END-OF-SELECTION,
+     *  matching ABAP runtime behavior regardless of source order */
+    private rearrangeListEvents;
     private rebuild;
     private precedence;
     private flatten;

package/build/src/rearranger.js

@@ -3,6 +3,12 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.Rearranger = void 0;
 const core_1 = require("@abaplint/core");
 // this rearranges the AST to take precedence into account
+var EventKind;
+(function (EventKind) {
+    EventKind[EventKind["None"] = 0] = "None";
+    EventKind[EventKind["Start"] = 1] = "Start";
+    EventKind[EventKind["End"] = 2] = "End";
+})(EventKind || (EventKind = {}));
 class Rearranger {
     run(type, node) {
         if (!node) {
@@ -14,7 +20,56 @@ class Rearranger {
         }
         const flattened = this.flatten(node);
         const rebuilt = this.rebuild(flattened);
-        return rebuilt;
+        const rearranged = this.rearrangeListEvents(rebuilt);
+        return rearranged;
+    }
+    /** Rearranges list event blocks so START-OF-SELECTION runs before END-OF-SELECTION,
+     *  matching ABAP runtime behavior regardless of source order */
+    rearrangeListEvents(node) {
+        if (!(node.get() instanceof core_1.Structures.Any)) {
+            return node;
+        }
+        const children = node.getChildren();
+        // Check if there are any list event statements
+        const hasStartOfSelection = children.some(c => c instanceof core_1.Nodes.StatementNode && c.get() instanceof core_1.Statements.StartOfSelection);
+        const hasEndOfSelection = children.some(c => c instanceof core_1.Nodes.StatementNode && c.get() instanceof core_1.Statements.EndOfSelection);
+        if (!hasStartOfSelection && !hasEndOfSelection) {
+            return node;
+        }
+        // Split children into: preamble (before first event), event blocks (keyed by event type)
+        const preamble = [];
+        const startBlocks = [];
+        const endBlocks = [];
+        let currentEvent = EventKind.None;
+        for (const child of children) {
+            if (child instanceof core_1.Nodes.StatementNode && child.get() instanceof core_1.Statements.StartOfSelection) {
+                currentEvent = EventKind.Start;
+                startBlocks.push(child);
+            }
+            else if (child instanceof core_1.Nodes.StatementNode && child.get() instanceof core_1.Statements.EndOfSelection) {
+                currentEvent = EventKind.End;
+                endBlocks.push(child);
+            }
+            else {
+                switch (currentEvent) {
+                    case EventKind.None:
+                        preamble.push(child);
+                        break;
+                    case EventKind.Start:
+                        startBlocks.push(child);
+                        break;
+                    case EventKind.End:
+                        endBlocks.push(child);
+                        break;
+                    default:
+                        break;
+                }
+            }
+        }
+        // Reassemble: preamble, then START-OF-SELECTION blocks, then END-OF-SELECTION blocks
+        const newChildren = [...preamble, ...startBlocks, ...endBlocks];
+        node.setChildren(newChildren);
+        return node;
     }
     /////////////////
     rebuild(node) {

package/build/src/statements/end_of_selection.d.ts

@@ -0,0 +1,6 @@
+import * as abaplint from "@abaplint/core";
+import { Chunk } from "../chunk";
+import { IStatementTranspiler } from "./_statement_transpiler";
+export declare class EndOfSelectionTranspiler implements IStatementTranspiler {
+    transpile(_node: abaplint.Nodes.StatementNode): Chunk;
+}

package/build/src/statements/end_of_selection.js

@@ -0,0 +1,11 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.EndOfSelectionTranspiler = void 0;
+const chunk_1 = require("../chunk");
+class EndOfSelectionTranspiler {
+    transpile(_node) {
+        return new chunk_1.Chunk("");
+    }
+}
+exports.EndOfSelectionTranspiler = EndOfSelectionTranspiler;
+//# sourceMappingURL=end_of_selection.js.map

package/build/src/statements/index.d.ts

@@ -27,6 +27,7 @@ export * from "./collect";
 export * from "./commit";
 export * from "./compute";
 export * from "./concatenate";
+export * from "./end_of_selection";
 export * from "./condense";
 export * from "./constant";
 export * from "./continue";

package/build/src/statements/index.js

@@ -43,6 +43,7 @@ __exportStar(require("./collect"), exports);
 __exportStar(require("./commit"), exports);
 __exportStar(require("./compute"), exports);
 __exportStar(require("./concatenate"), exports);
+__exportStar(require("./end_of_selection"), exports);
 __exportStar(require("./condense"), exports);
 __exportStar(require("./constant"), exports);
 __exportStar(require("./continue"), exports);

package/build/src/traversal.js

@@ -667,10 +667,6 @@ this.INTERNAL_ID = abap.internalIdCounter++;\n`;
     lookupInferred(node, scope) {
         if (scope === undefined) {
             return undefined;
-            /*
-          } else if (node.concatTokens() !== "#") {
-            throw new Error("lookupInferred, unexpected, " + node.get());
-            */
         }
         return this.findInferredTypeReference(node.getFirstToken());
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler",
-  "version": "2.12.29",
+  "version": "2.12.30",
   "description": "Transpiler",
   "main": "build/src/index.js",
   "typings": "build/src/index.d.ts",
@@ -29,7 +29,7 @@
   "author": "abaplint",
   "license": "MIT",
   "dependencies": {
-    "@abaplint/core": "^2.115.25",
+    "@abaplint/core": "^2.115.26",
     "source-map": "^0.7.6"
   },
   "devDependencies": {