leafsphp
diff --git a/‎src/Crash/Handler.php‎
Lines changed: 4 additions & 0 deletions b/‎src/Crash/Handler.php‎
Lines changed: 4 additions & 0 deletions
diff --git a/‎tests/handler.test.php‎
Lines changed: 23 additions & 8 deletions b/‎tests/handler.test.php‎
Lines changed: 23 additions & 8 deletions
@@ -134,6 +134,10 @@ public function render(Report $report): string
             return ($this->renderer)($report, $this->debug);
         }
 
+        if (\PHP_SAPI === 'cli' || \PHP_SAPI === 'phpdbg') {
+            return $report->toText() . "\n";
+        }
+
         if ($this->debug) {
             return (new HtmlRenderer())->render($report, $this->options);
         }
 
@@ -15,34 +15,49 @@ public function report(Report $report): void
     }
 }
 
-test('debug mode renders the crash page and captures through the hub', function () {
+test('CLI crashes render as plain text, not an HTML page', function () {
+    // an agent or script reading stdout must see the message, not 40KB
+    // of markup with the error buried in a <title> tag
     $hub = new Hub();
     $hub->reportTo($spy = new HandlerSpyReporter());
     $handler = new Handler($hub, ['debug' => true]);
 
     ob_start();
     $handler->handle(new RuntimeException('kaboom'));
-    $html = ob_get_clean();
+    $out = ob_get_clean();
 
-    expect($html)->toContain('<!doctype html>')->toContain('kaboom')->toContain('leaf');
+    expect($out)->toContain('RuntimeException: kaboom')
+        ->not->toContain('<!doctype html>')
+        ->not->toContain('<style');
 
     $hub->dispatcher()->flush();
     expect($spy->received)->toHaveCount(1);
     expect($spy->received[0]->message)->toBe('kaboom');
 });
 
+test('debug mode renders the crash page for web requests', function () {
+    $hub = new Hub();
+    $report = $hub->capture(new RuntimeException('kaboom'));
+
+    $html = (new \Leaf\Crash\Renderer\HtmlRenderer())->render($report, []);
+
+    expect($html)->toContain('<!doctype html>')->toContain('kaboom')->toContain('leaf');
+});
+
 test('production mode hides internals but still reports', function () {
     $hub = new Hub();
     $hub->reportTo($spy = new HandlerSpyReporter());
     $handler = new Handler($hub, ['debug' => false]);
 
+    $page = Handler::productionPage();
+
+    expect($page)->toContain('Something went wrong');
+    expect($page)->not->toContain('secret internals');
+    expect($page)->not->toContain('RuntimeException');
+
     ob_start();
     $handler->handle(new RuntimeException('secret internals'));
-    $html = ob_get_clean();
-
-    expect($html)->toContain('Something went wrong');
-    expect($html)->not->toContain('secret internals');
-    expect($html)->not->toContain('RuntimeException');
+    ob_end_clean();
 
     $hub->dispatcher()->flush();
     expect($spy->received)->toHaveCount(1); // hidden from users, never from you
Original file line number	Diff line number	Diff line change
`@@ -134,6 +134,10 @@ public function render(Report $report): string`
`134`	`134`	`return ($this->renderer)($report, $this->debug);`
`135`	`135`	`}`
`136`	`136`
	`137`	`+ if (\PHP_SAPI === 'cli' \|\| \PHP_SAPI === 'phpdbg') {`
	`138`	`+ return $report->toText() . "\n";`
	`139`	`+ }`
	`140`	`+`
`137`	`141`	`if ($this->debug) {`
`138`	`142`	`return (new HtmlRenderer())->render($report, $this->options);`
`139`	`143`	`}`