Fix styling

Author: Naoray

src/Issues/Formatters/ExceptionFormatter.php

@@ -65,7 +65,7 @@ public function formatTitle(Throwable $exception, string $level): string
 
     private function formatMessage(string $message): string
     {
-        if (!str_contains($message, 'Stack trace:')) {
+        if (! str_contains($message, 'Stack trace:')) {
             return $message;
         }
 

src/Issues/Formatters/StackTraceFormatter.php

@@ -43,9 +43,6 @@ public function format(string $stackTrace, bool $collapseVendorFrames = true): s
     /**
      * Stack trace lines start with #\d. Here we pad the numbers 0-9
      * with a preceding zero to keep everything in line visually.
-     *
-     * @param string $line
-     * @return string
      */
     public function padStackTraceLine(string $line): string
     {

tests/Issues/Formatters/ExceptionFormatterTest.php

@@ -59,13 +59,14 @@
 
 test('it properly formats exception with stack trace in message', function () {
     // Create a custom exception class that mimics our problematic behavior
-    $exception = new class('Error message') extends Exception {
+    $exception = new class('Error message') extends Exception
+    {
         public function __construct()
         {
-            parent::__construct("The calculation amount [123.45] does not match the expected total [456.78]. in /path/to/app/Calculations/Calculator.php:49
+            parent::__construct('The calculation amount [123.45] does not match the expected total [456.78]. in /path/to/app/Calculations/Calculator.php:49
 Stack trace:
 #0 /path/to/app/Services/PaymentService.php(83): App\\Calculations\\Calculator->calculate()
-#1 /vendor/framework/src/Services/TransactionService.php(247): App\\Services\\PaymentService->process()");
+#1 /vendor/framework/src/Services/TransactionService.php(247): App\\Services\\PaymentService->process()');
         }
     };
 
@@ -84,10 +85,10 @@ public function __construct()
 
 test('it handles exceptions with string in context', function () {
     // Create a generic exception string
-    $exceptionString = "The calculation amount [123.45] does not match the expected total [456.78]. in /path/to/app/Calculations/Calculator.php:49
+    $exceptionString = 'The calculation amount [123.45] does not match the expected total [456.78]. in /path/to/app/Calculations/Calculator.php:49
 Stack trace:
 #0 /path/to/app/Services/PaymentService.php(83): App\\Calculations\\Calculator->calculate()
-#1 /vendor/framework/src/Services/TransactionService.php(247): App\\Services\\PaymentService->process()";
+#1 /vendor/framework/src/Services/TransactionService.php(247): App\\Services\\PaymentService->process()';
 
     // Create a record with a string in the exception context
     $record = createLogRecord('Test message', ['exception' => $exceptionString]);

tests/Issues/Formatters/StackTraceFormatterTest.php

@@ -58,7 +58,7 @@
 });
 
 test('it replaces base path in stack traces', function () {
-    $formatter = new StackTraceFormatter();
+    $formatter = new StackTraceFormatter;
     $basePath = base_path();
 
     $stackTrace = <<<TRACE
@@ -69,44 +69,44 @@
 
     // Test simplified trace (with vendor frame collapsing)
     expect($formatter->format($stackTrace, true))
-        ->toBe(<<<TRACE
-#00 /app/Services/ImageService.php(25): Spatie\\Image\\Image->loadFile()
+        ->toBe(<<<'TRACE'
+#00 /app/Services/ImageService.php(25): Spatie\Image\Image->loadFile()
 [Vendor frames]
 TRACE
         );
 
     // Test full trace (without vendor frame collapsing)
     expect($formatter->format($stackTrace, false))
-        ->toBe(<<<TRACE
-#00 /app/Services/ImageService.php(25): Spatie\\Image\\Image->loadFile()
-#01 /vendor/laravel/framework/src/Foundation/Application.php(1235): App\\Services\\ImageService->process()
-#02 /artisan(13): Illuminate\\Foundation\\Application->run()
+        ->toBe(<<<'TRACE'
+#00 /app/Services/ImageService.php(25): Spatie\Image\Image->loadFile()
+#01 /vendor/laravel/framework/src/Foundation/Application.php(1235): App\Services\ImageService->process()
+#02 /artisan(13): Illuminate\Foundation\Application->run()
 TRACE
         );
 });
 
 test('it handles empty base path correctly', function () {
-    $formatter = new StackTraceFormatter();
+    $formatter = new StackTraceFormatter;
 
-    $stackTrace = <<<TRACE
+    $stackTrace = <<<'TRACE'
 #0 /absolute/path/to/app/Services/ImageService.php(25): someMethod()
 #1 /vendor/package/src/File.php(10): otherMethod()
 TRACE;
 
     $result = $formatter->format($stackTrace, true);
 
     expect($result)
-        ->toBe(<<<TRACE
+        ->toBe(<<<'TRACE'
 #00 /absolute/path/to/app/Services/ImageService.php(25): someMethod()
 [Vendor frames]
 TRACE
         );
 });
 
 test('it preserves non-stack-trace lines', function () {
-    $formatter = new StackTraceFormatter();
+    $formatter = new StackTraceFormatter;
 
-    $stackTrace = <<<TRACE
+    $stackTrace = <<<'TRACE'
 [2024-03-21 12:00:00] Error: Something went wrong
 #0 /app/Services/Service.php(25): someMethod()
 #1 /vendor/package/src/File.php(10): otherMethod()
@@ -115,7 +115,7 @@
     $result = $formatter->format($stackTrace, true);
 
     expect($result)
-        ->toBe(<<<TRACE
+        ->toBe(<<<'TRACE'
 [2024-03-21 12:00:00] Error: Something went wrong
 #00 /app/Services/Service.php(25): someMethod()
 [Vendor frames]
@@ -124,7 +124,7 @@
 });
 
 test('it recognizes artisan lines as vendor frames', function () {
-    $formatter = new StackTraceFormatter();
+    $formatter = new StackTraceFormatter;
     $basePath = base_path();
 
     $stackTrace = <<<TRACE
@@ -135,27 +135,27 @@
 
     // Test simplified trace (with vendor frame collapsing)
     expect($formatter->format($stackTrace, true))
-        ->toBe(<<<TRACE
+        ->toBe(<<<'TRACE'
 #00 /app/Console/Commands/ImportCommand.php(25): handle()
 [Vendor frames]
 TRACE
         );
 
     // Test that artisan lines are preserved in full trace
     expect($formatter->format($stackTrace, false))
-        ->toBe(<<<TRACE
+        ->toBe(<<<'TRACE'
 #00 /app/Console/Commands/ImportCommand.php(25): handle()
-#01 /artisan(13): Illuminate\\Foundation\\Application->run()
+#01 /artisan(13): Illuminate\Foundation\Application->run()
 #02 /artisan(37): require()
 TRACE
         );
 });
 
 test('it collapses multiple artisan lines into single vendor frame', function () {
-    $formatter = new StackTraceFormatter();
+    $formatter = new StackTraceFormatter;
 
-    $stackTrace = <<<TRACE
-#0 /artisan(13): Illuminate\\Foundation\\Application->run()
+    $stackTrace = <<<'TRACE'
+#0 /artisan(13): Illuminate\Foundation\Application->run()
 #1 /vendor/laravel/framework/src/Foundation/Application.php(1235): handle()
 #2 /artisan(37): require()
 TRACE;