fix(formatters): strip duplicate keys from Extra Data section (#41)

Closes #32

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: Naoray

src/Issues/Formatters/ExtraFormatter.php

@@ -2,10 +2,30 @@
 
 namespace Naoray\LaravelGithubMonolog\Issues\Formatters;
 
+use Illuminate\Support\Arr;
+
 class ExtraFormatter
 {
     public function format(array $extra): string
     {
+        // Exclude keys that have dedicated sections to avoid duplicate output
+        $extra = Arr::except($extra, [
+            'exception',
+            'environment',
+            'request',
+            'route',
+            'route_summary',
+            'user',
+            'queries',
+            'job',
+            'command',
+            'outgoing_requests',
+            'session',
+            'livewire',
+            'livewire_originating_page',
+            'inertia',
+        ]);
+
         if (empty($extra)) {
             return '';
         }

tests/Issues/Formatters/ExtraFormatterTest.php

@@ -38,3 +38,52 @@
         ->toContain('"level": 400')
         ->toContain('"datetime": "2024-01-01 12:00:00"');
 });
+
+it('excludes keys that have dedicated sections', function () {
+    $extra = [
+        'exception' => ['class' => 'RuntimeException'],
+        'environment' => ['app_env' => 'testing'],
+        'request' => ['url' => 'https://example.com'],
+        'route' => ['name' => 'test.route'],
+        'route_summary' => '/dashboard',
+        'user' => ['id' => 123],
+        'queries' => [],
+        'job' => ['name' => 'TestJob'],
+        'command' => ['name' => 'test:command'],
+        'outgoing_requests' => [],
+        'session' => ['data' => []],
+        'livewire' => ['component' => 'App\\Livewire\\Counter'],
+        'livewire_originating_page' => '/dashboard',
+        'inertia' => ['page' => '/dashboard'],
+        'custom_key' => 'custom_value',
+    ];
+
+    $result = $this->formatter->format($extra);
+
+    expect($result)
+        ->not->toContain('exception')
+        ->not->toContain('environment')
+        ->not->toContain('request')
+        ->not->toContain('"route"')
+        ->not->toContain('route_summary')
+        ->not->toContain('user')
+        ->not->toContain('queries')
+        ->not->toContain('job')
+        ->not->toContain('command')
+        ->not->toContain('outgoing_requests')
+        ->not->toContain('session')
+        ->not->toContain('livewire')
+        ->not->toContain('inertia')
+        ->toContain('custom_key')
+        ->toContain('custom_value');
+});
+
+it('returns empty string when all keys are excluded', function () {
+    $extra = [
+        'environment' => ['app_env' => 'testing'],
+        'user' => ['id' => 123],
+        'route' => ['name' => 'home'],
+    ];
+
+    expect($this->formatter->format($extra))->toBe('');
+});