Merge pull request #60 from laravel/fix-boost-inject-with-livewire

fix #49: disable boost inject if HTML isn't expected

Author: ashleyhindle

src/Middleware/InjectBoost.php

@@ -17,7 +17,7 @@ public function handle(Request $request, Closure $next): Response
         /** @var \Symfony\Component\HttpFoundation\Response $response */
         $response = $next($request);
 
-        if ($this->shouldInject($response->getContent())) {
+        if ($this->shouldInject($response)) {
             $originalView = $response->original ?? null;
             $injectedContent = $this->injectScript($response->getContent());
             $response->setContent($injectedContent);
@@ -30,8 +30,13 @@ public function handle(Request $request, Closure $next): Response
         return $response;
     }
 
-    private function shouldInject(string $content): bool
+    private function shouldInject(Response $response): bool
     {
+        if (str_contains($response->headers->get('content-type', ''), 'html') === false) {
+            return false;
+        }
+
+        $content = $response->getContent();
         // Check if it's HTML
         if (! str_contains($content, '<html') && ! str_contains($content, '<head')) {
             return false;

tests/Feature/Mcp/Tools/BrowserLogsTest.php

@@ -208,7 +208,7 @@
 HTML;
 
     $request = Request::create('/');
-    $response = new \Illuminate\Http\Response($html);
+    $response = new \Illuminate\Http\Response($html, 200, ['Content-Type' => 'text/html']);
 
     $result = $middleware->handle($request, function ($req) use ($response) {
         return $response;
@@ -277,7 +277,7 @@
 HTML;
 
     $request = Request::create('/');
-    $response = new \Illuminate\Http\Response($html);
+    $response = new \Illuminate\Http\Response($html, 200, ['Content-Type' => 'text/html']);
 
     $result = $middleware->handle($request, function ($req) use ($response) {
         return $response;