From fa97dcd10e08d936c5a78c8fa117bb881bf3157b Mon Sep 17 00:00:00 2001
From: Jordan Partridge <jordan@partridge.rocks>
Date: Fri, 13 Mar 2026 01:42:18 +0000
Subject: [PATCH] fix: display full error messages instead of truncated table
 output

Replace Laravel Prompts table() with line-by-line output so long error
messages are no longer truncated. Closes #63.
---
 app/Commands/CertifyCommand.php            | 10 ++--
 tests/Unit/Commands/CertifyCommandTest.php | 55 ++++++++++++++++++++++
 2 files changed, 60 insertions(+), 5 deletions(-)

diff --git a/app/Commands/CertifyCommand.php b/app/Commands/CertifyCommand.php
index f50d6bb..aad9e10 100644
--- a/app/Commands/CertifyCommand.php
+++ b/app/Commands/CertifyCommand.php
@@ -18,7 +18,6 @@
 use function Laravel\Prompts\error;
 use function Laravel\Prompts\info;
 use function Laravel\Prompts\spin;
-use function Laravel\Prompts\table;
 
 final class CertifyCommand extends Command
 {
@@ -146,10 +145,11 @@ public function handle(): int
             error('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
 
             if (! empty($failureRows)) {
-                table(
-                    headers: ['Check', 'Issue'],
-                    rows: $failureRows
-                );
+                $this->newLine();
+                foreach ($failureRows as $row) {
+                    $this->line("  <fg=red>✗</> <fg=white;options=bold>{$row[0]}</>: {$row[1]}");
+                }
+                $this->newLine();
             }
         }
 
diff --git a/tests/Unit/Commands/CertifyCommandTest.php b/tests/Unit/Commands/CertifyCommandTest.php
index 10f2818..bf7549a 100644
--- a/tests/Unit/Commands/CertifyCommandTest.php
+++ b/tests/Unit/Commands/CertifyCommandTest.php
@@ -322,6 +322,61 @@
                 ->assertFailed();
         });
 
+        it('displays full error messages without truncation', function () {
+            $longDetail = 'Error in src/Services/PaymentGateway.php: Method processTransaction() has cyclomatic complexity of 25 which exceeds the configured maximum of 10. Consider refactoring this method into smaller, more focused methods to improve maintainability and testability.';
+
+            $failingCheck = Mockery::mock(CheckInterface::class);
+            $failingCheck->shouldReceive('name')->andReturn('Tests');
+            $failingCheck->shouldReceive('run')
+                ->once()
+                ->andReturn(CheckResult::fail('1 test failed', [$longDetail]));
+
+            $mock = new MockHandler([
+                new Response(201),
+                new Response(201),
+            ]);
+            $httpClient = new Client(['handler' => HandlerStack::create($mock)]);
+            $checksClient = new ChecksClient('token', $httpClient, 'owner/repo', 'sha123');
+
+            ($this->createCommand)([$failingCheck], $checksClient);
+
+            $this->artisan('certify')
+                ->expectsOutputToContain($longDetail)
+                ->assertFailed();
+        });
+
+        it('displays multiple long error details from different checks', function () {
+            $detail1 = 'FAIL Tests\\Unit\\OrderServiceTest > it calculates the total price correctly including tax, shipping, and discount adjustments for international orders';
+            $detail2 = 'CVE-2024-9999: Critical vulnerability found in vendor/acme/library v2.3.1 - Remote code execution via deserialization of untrusted data in JsonParser::decode()';
+
+            $failingCheck1 = Mockery::mock(CheckInterface::class);
+            $failingCheck1->shouldReceive('name')->andReturn('Tests');
+            $failingCheck1->shouldReceive('run')
+                ->once()
+                ->andReturn(CheckResult::fail('Tests failed', [$detail1]));
+
+            $failingCheck2 = Mockery::mock(CheckInterface::class);
+            $failingCheck2->shouldReceive('name')->andReturn('Security');
+            $failingCheck2->shouldReceive('run')
+                ->once()
+                ->andReturn(CheckResult::fail('Security failed', [$detail2]));
+
+            $mock = new MockHandler([
+                new Response(201),
+                new Response(201),
+                new Response(201),
+            ]);
+            $httpClient = new Client(['handler' => HandlerStack::create($mock)]);
+            $checksClient = new ChecksClient('token', $httpClient, 'owner/repo', 'sha123');
+
+            ($this->createCommand)([$failingCheck1, $failingCheck2], $checksClient);
+
+            $this->artisan('certify')
+                ->expectsOutputToContain($detail1)
+                ->expectsOutputToContain($detail2)
+                ->assertFailed();
+        });
+
         it('shortens check names in compact output', function () {
             // This test covers the shortName() method by using the actual check names
             $testsCheck = Mockery::mock(CheckInterface::class);