Fix @JSON Blade directive parsing with nested structures

src/Illuminate/View/Compilers/Concerns/CompilesJson.php

@@ -19,12 +19,90 @@ trait CompilesJson
      */
     protected function compileJson($expression)
     {
-        $parts = explode(',', $this->stripParentheses($expression));
+        $parts = $this->parseArguments($this->stripParentheses($expression));
 
         $options = isset($parts[1]) ? trim($parts[1]) : $this->encodingOptions;
 
         $depth = isset($parts[2]) ? trim($parts[2]) : 512;
 
-        return "<?php echo json_encode($parts[0], $options, $depth) ?>";
+        // Ensure we have at least one argument, default to null if empty
+        $data = $parts[0] ?? 'null';
+
+        return "<?php echo json_encode($data, $options, $depth) ?>";
+    }
+
+    /**
+     * Parse arguments from an expression, respecting nested structures.
+     *
+     * This method properly handles commas inside arrays, closures, function calls,
+     * and other nested structures by using PHP's tokenizer.
+     *
+     * @param  string  $expression
+     * @return array
+     */
+    protected function parseArguments($expression)
+    {
+        if ('' === trim($expression)) {
+            return [];
+        }
+
+        $tokens = @token_get_all('<?php '.$expression);
+
+        if (false === $tokens) {
+            // Fallback to simple explode if tokenization fails
+            return array_map('trim', explode(',', $expression));
+        }
+
+        $parts = [];
+        $current = '';
+        $depth = 0;
+
+        foreach ($tokens as $index => $token) {
+            // Skip the initial <?php token
+            if (0 === $index && is_array($token) && T_OPEN_TAG === $token[0]) {
+                continue;
+            }
+
+            if (is_array($token)) {
+                [$id, $text] = $token;
+
+                // Handle strings - preserve them completely
+                if (in_array($id, [T_CONSTANT_ENCAPSED_STRING, T_ENCAPSED_AND_WHITESPACE])) {
+                    $current .= $text;
+                    continue;
+                }
+
+                // Handle whitespace at top level when $current is empty (can be ignored)
+                if (T_WHITESPACE === $id && 0 === $depth && '' === $current) {
+                    continue;
+                }
+
+                $current .= $text;
+            } else {
+                $char = $token;
+
+                // Track nesting depth for parentheses, brackets, and braces
+                if ('(' === $char || '[' === $char || '{' === $char) {
+                    $depth++;
+                    $current .= $char;
+                } elseif (')' === $char || ']' === $char || '}' === $char) {
+                    $depth--;
+                    $current .= $char;
+                } elseif (',' === $char && 0 === $depth) {
+                    // Only split on commas at the top level
+                    $parts[] = trim($current);
+                    $current = '';
+                } else {
+                    $current .= $char;
+                }
+            }
+        }
+
+        // Add the last part
+        if ('' !== $current) {
+            $parts[] = trim($current);
+        }
+
+        return $parts;
     }
 }

tests/View/Blade/BladeJsonTest.php

@@ -19,4 +19,53 @@ public function testEncodingOptionsCanBeOverwritten()
 
         $this->assertEquals($expected, $this->compiler->compileString($string));
     }
+
+    public function testJsonWithComplexArrayContainingNestedStructures()
+    {
+        $string = '@json([\'items\' => collect([1, 2, 3])->map(fn($x) => [\'id\' => $x, \'name\' => "test"]), \'translation\' => \'%:booking.benefits%\'])';
+        $expected = '<?php echo json_encode([\'items\' => collect([1, 2, 3])->map(fn($x) => [\'id\' => $x, \'name\' => "test"]), \'translation\' => \'%:booking.benefits%\'], 15, 512) ?>';
+
+        $this->assertEquals($expected, $this->compiler->compileString($string));
+    }
+
+    public function testJsonWithArrayContainingMultipleCommas()
+    {
+        $string = '@json([\'a\' => 1, \'b\' => 2, \'c\' => 3], JSON_PRETTY_PRINT)';
+        $expected = '<?php echo json_encode([\'a\' => 1, \'b\' => 2, \'c\' => 3], JSON_PRETTY_PRINT, 512) ?>';
+
+        $this->assertEquals($expected, $this->compiler->compileString($string));
+    }
+
+    public function testJsonWithClosureContainingCommas()
+    {
+        $string = '@json($items->map(fn($item) => [\'icon\' => $item->icon, \'title\' => (string)$item->title, \'description\' => (string)$item->description]))';
+        $expected = '<?php echo json_encode($items->map(fn($item) => [\'icon\' => $item->icon, \'title\' => (string)$item->title, \'description\' => (string)$item->description]), 15, 512) ?>';
+
+        $this->assertEquals($expected, $this->compiler->compileString($string));
+    }
+
+    public function testJsonWithAllThreeArguments()
+    {
+        $string = '@json($data, JSON_PRETTY_PRINT, 256)';
+        $expected = '<?php echo json_encode($data, JSON_PRETTY_PRINT, 256) ?>';
+
+        $this->assertEquals($expected, $this->compiler->compileString($string));
+    }
+
+    public function testJsonWithEmptyExpressionDefaultsToNull()
+    {
+        $string = '@json()';
+        $expected = '<?php echo json_encode(null, 15, 512) ?>';
+
+        $this->assertEquals($expected, $this->compiler->compileString($string));
+    }
+
+    public function testJsonWithIssue56331ExactCase()
+    {
+        // This is the exact case from GitHub issue #56331
+        $string = '@json([\'items\' => $helpers[\'benefit\'][\'getAll\']()->map(fn($item) => [\'icon\' => $item->icon, \'title\' => (string)$item->title, \'description\' => (string)$item->description]), \'translation\' => \'%:booking.benefits%\'])';
+        $expected = '<?php echo json_encode([\'items\' => $helpers[\'benefit\'][\'getAll\']()->map(fn($item) => [\'icon\' => $item->icon, \'title\' => (string)$item->title, \'description\' => (string)$item->description]), \'translation\' => \'%:booking.benefits%\'], 15, 512) ?>';
+
+        $this->assertEquals($expected, $this->compiler->compileString($string));
+    }
 }