From b42aa3d428c2d40c33b56577f75a0f756f203e3d Mon Sep 17 00:00:00 2001
From: Chris Mellor <chris@mellor.pizza>
Date: Thu, 29 Oct 2026 15:28:21 +0000
Subject: [PATCH] fix: skip browser plugin install steps when it's already
 installed

---
 resources/boost/guidelines/core.blade.php | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/resources/boost/guidelines/core.blade.php b/resources/boost/guidelines/core.blade.php
index d26c219..d7497ce 100644
--- a/resources/boost/guidelines/core.blade.php
+++ b/resources/boost/guidelines/core.blade.php
@@ -21,10 +21,11 @@
 
 The one thing single quotes can't contain is a literal single quote (an apostrophe in the PHP). Only then, fall back to a file: **Write** the snippet to a `.php` file (plain body statements — no `<?php`, no `use`, fully qualified class names) and run `vendor/bin/pest --agent="$(cat /path/to/snippet.php)"`. `"$(cat …)"` passes the contents verbatim without re-parsing. The plugin resolves the test suite's `uses`/namespace itself, so the file's location does not matter (a scratch/temp path is fine — it need not live under `tests/`).
 
-### Browser checks require the browser plugin — ask before installing
+### Browser checks require the browser plugin
 
 Whenever the request can only be answered in a real browser — "does login work", "is the page responsive", "screenshot the homepage", "check the mobile layout", "does the button click through", "are there JS/console errors", or any visual/interaction check — the `visit()` browser API is needed. It comes from a **separate** package, `pestphp/pest-plugin-browser`, which is powered by Playwright.
 
+@if (! \Composer\InstalledVersions::isInstalled('pestphp/pest-plugin-browser'))
 If `visit()` is undefined (or the package is not installed), **do not install it silently — ask the user for permission first**, since it pulls in Node/Playwright dependencies and downloads browser binaries. Explain that the browser check needs it and confirm before running these commands:
 
 ```bash
@@ -33,7 +34,10 @@
 npx playwright install                                # download the browser binaries
 ```
 
-Once the user approves and it's installed, add `tests/Browser/Screenshots` to `.gitignore` so captured screenshots aren't committed. Browser assertions then run through the same `vendor/bin/pest --agent='…'` flow:
+Once the user approves and it's installed, add `tests/Browser/Screenshots` to `.gitignore` so captured screenshots aren't committed.
+
+@endif
+Browser assertions run through the same `vendor/bin/pest --agent='…'` flow:
 
 ```bash
 vendor/bin/pest --agent='visit("/login")->type("email", "test@example.com")->type("password", "password")->press("Log in")->assertPathIs("/dashboard");'