Correct parse token enqueuing during cast/braced disambiguation

Fixes #60

Author: thekid

ChangeLog.md

@@ -3,6 +3,10 @@ XP AST ChangeLog
 
 ## ?.?.? / ????-??-??
 
+## 12.1.1 / 2026-05-16
+
+* Fixed issue #60: IIFE parse error - @thekid
+
 ## 12.1.0 / 2026-04-23
 
 * Merged PR #59: Implement logical AND and OR assignment operators `&&=`

src/main/php/lang/ast/syntax/PHP.class.php

@@ -246,8 +246,9 @@ public function __construct() {
 
     // This is ambiguous:
     //
-    // - An expression surrounded by parentheses `($a ?? $b)->invoke()`;
-    // - A cast `(int)$a` or `(int)($a / 2)`.
+    // - An expression surrounded by parentheses `($a ?? $b)->invoke()`
+    // - An invocation of a braced expression like `($expr)()`
+    // - A cast `(int)$a` or `(int)($a / 2)`
     //
     // Resolve by looking ahead after the closing ")"
     $this->prefix('(', 0, function($parse, $token) {
@@ -553,7 +554,7 @@ public function __construct() {
       if (':' === $parse->token->value) {
         $node= new Label($token->value, $token->line);
       } else {
-        $parse->queue[]= $parse->token;
+        array_unshift($parse->queue, $parse->token);
         $parse->token= $token;
         $node= $this->expression($parse, 0);
       }
@@ -705,7 +706,7 @@ public function __construct() {
               continue;
             }
 
-            $parse->queue[]= $parse->token;
+            array_unshift($parse->queue, $parse->token);
             $parse->token= $const;
           }
 
@@ -1080,8 +1081,8 @@ public function __construct() {
         if ('=' === $parse->token->value) {
           $name= $first->value;
         } else {
-          $parse->queue[]= $first;
-          $parse->queue[]= $parse->token;
+          array_unshift($parse->queue, $parse->token);
+          array_unshift($parse->queue, $first);
           $parse->token= $first;
 
           $type= $this->type($parse, false);

src/test/php/lang/ast/unittest/parse/ClosuresTest.class.php

@@ -1,11 +1,21 @@
 <?php namespace lang\ast\unittest\parse;
 
-use lang\ast\nodes\{BinaryExpression, ClosureExpression, Literal, Parameter, ReturnStatement, Signature, Variable};
+use lang\ast\nodes\{
+  BinaryExpression,
+  Braced,
+  ClosureExpression,
+  InvokeExpression,
+  Literal,
+  Parameter,
+  ReturnStatement,
+  Signature,
+  Variable
+};
 use lang\ast\{FunctionType, Type};
 use test\{Assert, Before, Test};
 
 class ClosuresTest extends ParseTest {
-  private $returns;
+  private $returns, $invoke;
 
   #[Before]
   public function returns() {
@@ -20,6 +30,15 @@ public function returns() {
     );
   }
 
+  #[Before]
+  public function invoke() {
+    $this->invoke= new InvokeExpression(
+      new Literal('var_dump', self::LINE),
+      [new Literal('true', self::LINE)],
+      self::LINE
+    );
+  }
+
   #[Test]
   public function with_body() {
     $this->assertParsed(
@@ -64,24 +83,40 @@ public function with_use_by_reference() {
   #[Test]
   public function with_return_type() {
     $this->assertParsed(
-      [new ClosureExpression(new Signature([], new Type('int'), false, self::LINE), null, [$this->returns], false, self::LINE)],
-      'function(): int { return $a + 1; };'
+      [new ClosureExpression(new Signature([], new Type('int'), false, self::LINE), null, [$this->invoke], false, self::LINE)],
+      'function(): int { var_dump(true); };'
     );
   }
 
   #[Test]
   public function with_nullable_return_type() {
     $this->assertParsed(
-      [new ClosureExpression(new Signature([], new Type('?int'), false, self::LINE), null, [$this->returns], false, self::LINE)],
-      'function(): ?int { return $a + 1; };'
+      [new ClosureExpression(new Signature([], new Type('?int'), false, self::LINE), null, [$this->invoke], false, self::LINE)],
+      'function(): ?int { var_dump(true); };'
     );
   }
 
   #[Test]
   public function static_function() {
     $this->assertParsed(
-      [new ClosureExpression(new Signature([], null, false, self::LINE), null, [$this->returns], true, self::LINE)],
-      'static function() { return $a + 1; };'
+      [new ClosureExpression(new Signature([], null, false, self::LINE), null, [$this->invoke], true, self::LINE)],
+      'static function() { var_dump(true); };'
+    );
+  }
+
+  /** @see https://github.com/xp-framework/ast/issues/60 */
+  #[Test]
+  public function iife_with_statement() {
+    $this->assertParsed(
+      [new InvokeExpression(
+        new Braced(
+          new ClosureExpression(new Signature([], null, false, self::LINE), null, [$this->invoke], false, self::LINE),
+          self::LINE
+        ),
+        [],
+        self::LINE
+      )],
+      '(function() { var_dump(true); })();'
     );
   }
 }