fix: show failure details in compact mode

app/Branding.php

@@ -8,8 +8,11 @@ final class Branding
 {
     // Check names (sorted by display order)
     public const TESTS = '① Tests & Coverage';
+
     public const SECURITY = '② Security Audit';
+
     public const SYNTAX = '③ Pest Syntax';
+
     public const CERTIFICATION = '🏆 Sentinel Certified';
 
     // Map internal names to branded names

app/Checks/CheckInterface.php

@@ -7,5 +7,6 @@
 interface CheckInterface
 {
     public function name(): string;
+
     public function run(string $workingDirectory): CheckResult;
 }

app/Checks/PestSyntaxValidator.php

@@ -15,13 +15,13 @@ public function name(): string
 
     public function run(string $workingDirectory): CheckResult
     {
-        $testsPath = $workingDirectory . '/tests';
+        $testsPath = $workingDirectory.'/tests';
 
-        if (!is_dir($testsPath)) {
+        if (! is_dir($testsPath)) {
             return CheckResult::pass('No tests directory found');
         }
 
-        $finder = new Finder();
+        $finder = new Finder;
         $finder->files()->in($testsPath)->name('*Test.php');
 
         $violations = [];
@@ -31,7 +31,7 @@ public function run(string $workingDirectory): CheckResult
 
             // Check for test() function usage (but not in comments)
             if (preg_match('/^\s*test\s*\(/m', $contents)) {
-                $violations[] = $file->getRelativePathname() . ': Uses test() instead of describe/it blocks';
+                $violations[] = $file->getRelativePathname().': Uses test() instead of describe/it blocks';
             }
         }
 

app/Checks/SecurityScanner.php

@@ -10,7 +10,7 @@
 final class SecurityScanner implements CheckInterface
 {
     public function __construct(
-        private readonly ProcessRunner $processRunner = new SymfonyProcessRunner(),
+        private readonly ProcessRunner $processRunner = new SymfonyProcessRunner,
     ) {}
 
     public function name(): string

app/Checks/TestRunner.php

@@ -18,8 +18,8 @@ final class TestRunner implements CheckInterface
 
     public function __construct(
         private readonly int $coverageThreshold = 100,
-        private readonly PestOutputParser $parser = new PestOutputParser(),
-        private readonly ProcessRunner $processRunner = new SymfonyProcessRunner(),
+        private readonly PestOutputParser $parser = new PestOutputParser,
+        private readonly ProcessRunner $processRunner = new SymfonyProcessRunner,
     ) {}
 
     /** @internal For testing only */
@@ -40,7 +40,7 @@ public function name(): string
 
     public function run(string $workingDirectory): CheckResult
     {
-        $cloverPath = $workingDirectory . '/coverage.xml';
+        $cloverPath = $workingDirectory.'/coverage.xml';
 
         $result = $this->processRunner->run(
             ['vendor/bin/pest', '--coverage', "--min={$this->coverageThreshold}", "--coverage-clover={$cloverPath}", '--colors=never'],

app/Commands/CertifyCommand.php

@@ -61,8 +61,8 @@ public function handle(): int
 
         $checks = $this->checks ?? [
             new TestRunner($coverageThreshold),
-            new SecurityScanner(),
-            new PestSyntaxValidator(),
+            new SecurityScanner,
+            new PestSyntaxValidator,
         ];
 
         $stopOnFailure = $this->option('stop-on-failure');
@@ -100,7 +100,7 @@ public function handle(): int
         // Report Sentinel Certification (last, after all checks)
         $title = $verdict->isApproved()
             ? 'All checks passed'
-            : count($failures) . ' check(s) failed';
+            : count($failures).' check(s) failed';
 
         $checksClient->reportCheck(
             name: Branding::CERTIFICATION,
@@ -117,6 +117,15 @@ public function handle(): int
         // Output verdict
         if ($compact) {
             $this->renderCompactOutput($compactResults, $verdict);
+
+            // Still show failure details in compact mode when checks fail
+            if (! $verdict->isApproved() && ! empty($failureRows)) {
+                echo "\n";
+                table(
+                    headers: ['Check', 'Issue'],
+                    rows: $failureRows
+                );
+            }
         } elseif ($verdict->isApproved()) {
             info('');
             info('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
@@ -206,7 +215,7 @@ private function renderCompactOutput(array $results, Verdict $verdict): void
         }
 
         $status = $verdict->isApproved() ? '✓ APPROVED' : '✗ REJECTED';
-        $line = implode('  ', $parts) . "  │  {$status}";
+        $line = implode('  ', $parts)."  │  {$status}";
 
         if ($verdict->isApproved()) {
             info($line);

app/Commands/SecurityCommand.php

@@ -43,7 +43,7 @@ public function handle(): int
     {
         $token = $this->option('token') ?: getenv('GITHUB_TOKEN') ?: null;
         $checksClient = $this->checksClient ?? new ChecksClient($token);
-        $check = $this->check ?? new SecurityScanner();
+        $check = $this->check ?? new SecurityScanner;
 
         $result = $check->run(getcwd());
 

app/Commands/SyntaxCommand.php

@@ -43,7 +43,7 @@ public function handle(): int
     {
         $token = $this->option('token') ?: getenv('GITHUB_TOKEN') ?: null;
         $checksClient = $this->checksClient ?? new ChecksClient($token);
-        $check = $this->check ?? new PestSyntaxValidator();
+        $check = $this->check ?? new PestSyntaxValidator;
 
         $result = $check->run(getcwd());
 

app/GitHub/ChecksClient.php

@@ -130,6 +130,7 @@ public function reportCheck(
         if (! $this->isAvailable()) {
             $hasToken = $this->token ? 'yes' : 'no';
             echo "::warning::ChecksClient not available (token={$hasToken}, repo={$this->repo}, sha={$this->sha})\n";
+
             return false;
         }
 
@@ -152,6 +153,7 @@ public function reportCheck(
             return true;
         } catch (GuzzleException $e) {
             echo "::warning::ChecksClient error: {$e->getMessage()}\n";
+
             return false;
         }
     }

app/GitHub/CommentsClient.php

@@ -69,6 +69,7 @@ public function postOrUpdateComment(string $body, string $signature = '🏆 Syna
             return true;
         } catch (GuzzleException $e) {
             echo "::warning::Failed to post/update PR comment: {$e->getMessage()}\n";
+
             return false;
         }
     }
@@ -106,6 +107,7 @@ private function extractPRNumber(): ?int
         }
 
         $event = json_decode(file_get_contents($eventPath), true);
+
         return $event['pull_request']['number'] ?? null;
     }
 }

app/Services/CoverageReporter.php

@@ -33,7 +33,7 @@ public function parseClover(string $path): array
 
         $metrics = $xml->project->metrics ?? null;
         if ($metrics === null) {
-            throw new \RuntimeException("Invalid clover format: missing project metrics");
+            throw new \RuntimeException('Invalid clover format: missing project metrics');
         }
 
         $files = [];
@@ -127,7 +127,7 @@ private function formatMarkdown(array $current, ?array $base): string
                 $uncoveredCount = count($file['uncovered_lines']);
                 $uncoveredPreview = $uncoveredCount > 0 ? implode(', ', array_slice($file['uncovered_lines'], 0, 5)) : 'None';
                 if ($uncoveredCount > 5) {
-                    $uncoveredPreview .= "... (+".($uncoveredCount - 5)." more)";
+                    $uncoveredPreview .= '... (+'.($uncoveredCount - 5).' more)';
                 }
                 $markdown .= "| `{$fileName}` | {$coverage}% | {$uncoveredPreview} |\n";
             }

app/Services/PestOutputParser.php

@@ -36,7 +36,7 @@ private function formatFailure(string $name, string $body): string
         $location = preg_match('/at\s+(\S+:\d+)/', $body, $m) ? " ({$m[1]})" : '';
         $message = $this->extractAssertionMessage($body);
 
-        return $name . $location . $message;
+        return $name.$location.$message;
     }
 
     private function extractAssertionMessage(string $body): string

app/Stages/TechnicalGate.php

@@ -10,7 +10,7 @@
 final class TechnicalGate
 {
     /**
-     * @param array<CheckInterface> $checks
+     * @param  array<CheckInterface>  $checks
      */
     public function __construct(
         private readonly array $checks,

app/Verdict.php

@@ -7,14 +7,13 @@
 final readonly class Verdict
 {
     /**
-     * @param array<int, string> $failures
+     * @param  array<int, string>  $failures
      */
     private function __construct(
         private string $status,
         private string $reason,
         private array $failures = []
-    ) {
-    }
+    ) {}
 
     public static function approved(string $reason): self
     {

tests/Feature/RunCommandTest.php

@@ -10,69 +10,69 @@
 describe('Gate Commands', function () {
     describe('TestsCommand', function () {
         it('has the correct signature', function () {
-            $command = new TestsCommand();
+            $command = new TestsCommand;
             expect($command->getName())->toBe('tests');
         });
 
         it('has coverage option with default of 80', function () {
-            $command = new TestsCommand();
+            $command = new TestsCommand;
             $definition = $command->getDefinition();
 
             expect($definition->hasOption('coverage'))->toBeTrue();
             expect($definition->getOption('coverage')->getDefault())->toBe('80');
         });
 
         it('has token option', function () {
-            $command = new TestsCommand();
+            $command = new TestsCommand;
             expect($command->getDefinition()->hasOption('token'))->toBeTrue();
         });
     });
 
     describe('SecurityCommand', function () {
         it('has the correct signature', function () {
-            $command = new SecurityCommand();
+            $command = new SecurityCommand;
             expect($command->getName())->toBe('security');
         });
 
         it('has token option', function () {
-            $command = new SecurityCommand();
+            $command = new SecurityCommand;
             expect($command->getDefinition()->hasOption('token'))->toBeTrue();
         });
     });
 
     describe('SyntaxCommand', function () {
         it('has the correct signature', function () {
-            $command = new SyntaxCommand();
+            $command = new SyntaxCommand;
             expect($command->getName())->toBe('syntax');
         });
 
         it('has token option', function () {
-            $command = new SyntaxCommand();
+            $command = new SyntaxCommand;
             expect($command->getDefinition()->hasOption('token'))->toBeTrue();
         });
     });
 
     describe('CertifyCommand', function () {
         it('has the correct signature', function () {
-            $command = new CertifyCommand();
+            $command = new CertifyCommand;
             expect($command->getName())->toBe('certify');
         });
 
         it('has coverage option with default of 80', function () {
-            $command = new CertifyCommand();
+            $command = new CertifyCommand;
             $definition = $command->getDefinition();
 
             expect($definition->hasOption('coverage'))->toBeTrue();
             expect($definition->getOption('coverage')->getDefault())->toBe('80');
         });
 
         it('has token option', function () {
-            $command = new CertifyCommand();
+            $command = new CertifyCommand;
             expect($command->getDefinition()->hasOption('token'))->toBeTrue();
         });
 
         it('has compact option', function () {
-            $command = new CertifyCommand();
+            $command = new CertifyCommand;
             $definition = $command->getDefinition();
 
             expect($definition->hasOption('compact'))->toBeTrue();