add tabs only in testing

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"

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);
           }
         },