From 8cd2aad719c7ed2ee1367e3244e9580ae6622870 Mon Sep 17 00:00:00 2001
From: "Tommy D. Rossi" <beats.by.morse@gmail.com>
Date: Mon, 24 Nov 2025 13:49:16 +0100
Subject: [PATCH] add tabs only in testing

---
 extension/manifest.json   |  9 ++-------
 extension/vite.config.mts | 10 ++++++----
 2 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/extension/manifest.json b/extension/manifest.json
index e96ac0a..5555728 100644
--- a/extension/manifest.json
+++ b/extension/manifest.json
@@ -3,13 +3,8 @@
   "name": "Playwriter MCP",
   "version": "0.0.48",
   "description": "Playwright MCP. 90% less context window. 10x more capable (full playwright API)",
-  "permissions": [
-    "debugger",
-    "tabs"
-  ],
-  "host_permissions": [
-    "<all_urls>"
-  ],
+  "permissions": ["debugger"],
+  "host_permissions": ["<all_urls>"],
   "background": {
     "service_worker": "lib/background.mjs",
     "type": "module"
diff --git a/extension/vite.config.mts b/extension/vite.config.mts
index 1f1152c..c3ef963 100644
--- a/extension/vite.config.mts
+++ b/extension/vite.config.mts
@@ -25,12 +25,14 @@ export default defineConfig({
           dest: '.',
           transform: (content) => {
             const manifest = JSON.parse(content);
-            
+
             // Only include tabs permission during testing
-            if (!process.env.TESTING) {
-              manifest.permissions = manifest.permissions.filter((p: string) => p !== 'tabs');
+            if (process.env.TESTING) {
+              if (!manifest.permissions.includes('tabs')) {
+                manifest.permissions.push('tabs');
+              }
             }
-            
+
             return JSON.stringify(manifest, null, 2);
           }
         },