Apply fixed Coderabbitai review.

Author: terabytesoftw

src/PanelView.php

@@ -117,7 +117,7 @@ public function blocks(): array
      */
     public function callout(Tone $tone, mixed ...$content): self
     {
-        return $this->append(self::paragraphBlock(array_values($content), $tone));
+        return $this->append(self::paragraphBlock($content, $tone));
     }
 
     /**
@@ -271,7 +271,7 @@ public function overview(array $values, bool $compact = false): self
      */
     public function paragraph(mixed ...$content): self
     {
-        return $this->append(self::paragraphBlock(array_values($content), null));
+        return $this->append(self::paragraphBlock($content, null));
     }
 
     /**

tests/PanelViewTest.php

@@ -6,11 +6,15 @@
 
 use InvalidArgumentException;
 use PHPForge\Debug\{ColumnStyle, PanelView, Tone};
+use PHPForge\Debug\Tests\Provider\InlineScalarProvider;
+use PHPUnit\Framework\Attributes\DataProviderExternal;
 use PHPUnit\Framework\TestCase;
 use stdClass;
 
 /**
  * Unit tests for the validated shapes {@see PanelView} exports to the host renderer.
+ *
+ * {@see InlineScalarProvider} for test case data providers.
  */
 final class PanelViewTest extends TestCase
 {
@@ -116,6 +120,72 @@ public function testMetricsAndFieldsShareOneLabelAndValueShape(): void
         );
     }
 
+    /**
+     * @param array{kind: 'text', value: string, style: 'plain'} $expected
+     */
+    #[DataProviderExternal(InlineScalarProvider::class, 'plainText')]
+    public function testScalarContentBecomesPlainInlineText(mixed $value, array $expected): void
+    {
+        self::assertSame(
+            [['kind' => 'paragraph', 'content' => [$expected], 'tone' => null]],
+            PanelView::create()->paragraph($value)->blocks(),
+            'Conversion must keep the literal and the plain style.',
+        );
+        self::assertSame(
+            [
+                [
+                    'kind' => 'overview',
+                    'fields' => [['label' => 'Field', 'value' => $expected]],
+                    'compact' => false,
+                ],
+            ],
+            PanelView::create()->overview(['Field' => $value])->blocks(),
+            'Overview fields must convert identically.',
+        );
+    }
+
+    public function testStringKeyedSpreadContentStaysAList(): void
+    {
+        $parts = ['first' => 'A', 'second' => PanelView::code('B')];
+
+        $content = [PanelView::text('A'), PanelView::code('B')];
+
+        self::assertSame(
+            [['kind' => 'paragraph', 'content' => $content, 'tone' => null]],
+            PanelView::create()->paragraph(...$parts)->blocks(),
+            'String keys must not reach the paragraph content.',
+        );
+        self::assertSame(
+            [['kind' => 'paragraph', 'content' => $content, 'tone' => Tone::WARNING]],
+            PanelView::create()->callout(Tone::WARNING, ...$parts)->blocks(),
+            'String keys must not reach the callout content.',
+        );
+    }
+
+    public function testTableKeepsEveryStyledColumnUnderItsIndex(): void
+    {
+        $view = PanelView::create()
+            ->table(
+                ['Key', 'Value', 'Hits'],
+                [['home', 'cached', 3]],
+                styles: [0 => ColumnStyle::MONOSPACE, 2 => ColumnStyle::NUMBER],
+            );
+
+        self::assertSame(
+            [
+                [
+                    'kind' => 'table',
+                    'headers' => ['Key', 'Value', 'Hits'],
+                    'rows' => [[PanelView::text('home'), PanelView::text('cached'), PanelView::text('3')]],
+                    'styles' => [0 => ColumnStyle::MONOSPACE, 2 => ColumnStyle::NUMBER],
+                    'collapsible' => false,
+                ],
+            ],
+            $view->blocks(),
+            'Every styled column must survive, not only the first.',
+        );
+    }
+
     public function testThrowInvalidArgumentExceptionForAssociativeParagraph(): void
     {
         $this->expectException(InvalidArgumentException::class);

tests/Provider/InlineScalarProvider.php

@@ -0,0 +1,26 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PHPForge\Debug\Tests\Provider;
+
+use PHPForge\Debug\Tests\PanelViewTest;
+
+/**
+ * Provides scalar and `null` inputs whose conversion to plain inline text is verified by {@see PanelViewTest}.
+ */
+final class InlineScalarProvider
+{
+    /**
+     * @return iterable<string, array{mixed, array{kind: 'text', value: string, style: 'plain'}}>
+     */
+    public static function plainText(): iterable
+    {
+        yield 'false' => [false, ['kind' => 'text', 'value' => 'false', 'style' => 'plain']];
+        yield 'float' => [1.5, ['kind' => 'text', 'value' => '1.5', 'style' => 'plain']];
+        yield 'int' => [7, ['kind' => 'text', 'value' => '7', 'style' => 'plain']];
+        yield 'null' => [null, ['kind' => 'text', 'value' => 'null', 'style' => 'plain']];
+        yield 'string' => ['<raw>', ['kind' => 'text', 'value' => '<raw>', 'style' => 'plain']];
+        yield 'true' => [true, ['kind' => 'text', 'value' => 'true', 'style' => 'plain']];
+    }
+}