Allow overriding the browser log channel (#589)

* Allow overriding the browser log channel

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Fix GetConfig tests to not depend on JSON formatting

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

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

Author: SanderMuller

src/BoostServiceProvider.php

@@ -171,6 +171,10 @@ private static function buildLogMessageFromData(array $data): string
 
     protected function registerBrowserLogger(): void
     {
+        if (config('logging.channels.browser') !== null) {
+            return;
+        }
+
         config([
             'logging.channels.browser' => [
                 'driver' => 'single',

tests/Feature/BoostServiceProviderTest.php

@@ -35,6 +35,22 @@
         expect(app()->bound(Laravel\Roster\Roster::class))->toBeTrue()
             ->and(config('logging.channels.browser'))->not->toBeNull();
     });
+
+    it('does not override an existing browser log channel', function (): void {
+        Config::set('boost.enabled', true);
+        Config::set('logging.channels.browser', [
+            'driver' => 'daily',
+            'path' => storage_path('logs/custom-browser.log'),
+        ]);
+        app()->detectEnvironment(fn (): string => 'local');
+
+        $provider = new BoostServiceProvider(app());
+        $provider->register();
+        $provider->boot(app('router'));
+
+        expect(config('logging.channels.browser.driver'))->toBe('daily')
+            ->and(config('logging.channels.browser.path'))->toBe(storage_path('logs/custom-browser.log'));
+    });
 });
 
 describe('environment restrictions', function (): void {

tests/Feature/Mcp/Tools/GetConfigTest.php

@@ -17,16 +17,21 @@
 
     expect($response)->isToolResult()
         ->toolHasNoError()
-        ->toolTextContains('"key": "test.key"', '"value": "test_value"');
+        ->toolTextContains('"test.key"', '"test_value"');
+
+    expect(json_decode((string) $response->content(), true))
+        ->toBe(['key' => 'test.key', 'value' => 'test_value']);
 });
 
 test('it returns nested config value', function (): void {
     $tool = new GetConfig;
     $response = $tool->handle(new Request(['key' => 'nested.config.key']));
 
     expect($response)->isToolResult()
-        ->toolHasNoError()
-        ->toolTextContains('"key": "nested.config.key"', '"value": "nested_value"');
+        ->toolHasNoError();
+
+    expect(json_decode((string) $response->content(), true))
+        ->toBe(['key' => 'nested.config.key', 'value' => 'nested_value']);
 });
 
 test('it returns error when config key does not exist', function (): void {
@@ -43,6 +48,8 @@
     $response = $tool->handle(new Request(['key' => 'app.name']));
 
     expect($response)->isToolResult()
-        ->toolHasNoError()
-        ->toolTextContains('"key": "app.name"', '"value": "Test App"');
+        ->toolHasNoError();
+
+    expect(json_decode((string) $response->content(), true))
+        ->toBe(['key' => 'app.name', 'value' => 'Test App']);
 });