Merge pull request #66 from browserbase/kj/context-fix

fix context id being set to empty string

Author: Kylejeong2

browserbase/package.json

@@ -1,11 +1,12 @@
 {
-  "name": "@browserbasehq/mcp-server-browserbase",
-  "version": "1.0.0",
+  "name": "@browserbasehq/mcp",
+  "version": "1.0.2",
   "description": "MCP server for browser automation using browserbase",
   "license": "MIT",
   "author": "Browserbase, Inc. (https://browserbase.com)",
   "homepage": "https://browserbase.com",
   "type": "module",
+  "main": "./cli.js",
   "engines": {
     "node": ">=18"
   },
@@ -28,7 +29,7 @@
     "./package.json": "./package.json",
     ".": {
       "types": "./index.d.ts",
-      "default": "./index.js"
+      "default": "./cli.js"
     }
   },
   "dependencies": {
@@ -49,5 +50,8 @@
   },
   "bin": {
     "mcp-server-browserbase": "cli.js"
+  },
+  "publishConfig": {
+    "access": "public"
   }
 }

browserbase/src/config.ts

@@ -44,10 +44,6 @@ const defaultConfig: Config = {
   browserbaseApiKey: process.env.BROWSERBASE_API_KEY,
   browserbaseProjectId: process.env.BROWSERBASE_PROJECT_ID,
   proxies: false,
-  context: {
-    contextId: undefined,
-    persist: true,
-  },
   server: {
     port: undefined,
     host: undefined,

browserbase/src/sessionManager.ts

@@ -96,10 +96,10 @@ export async function createNewBrowserSession(
         width: config.viewPort?.browserWidth ?? 1024,
         height: config.viewPort?.browserHeight ?? 768,
       },
-      context: {
-        id: config.context?.contextId ?? "",
+      context: config.context?.contextId ? {
+        id: config.context?.contextId,
         persist: config.context?.persist ?? true,
-      }
+      } : undefined,
     }
   };