update to http server, docker build working

Author: Kylejeong2

browserbase/Dockerfile

@@ -1,38 +1,46 @@
-# Basic Node.js Dockerfile
-FROM node:20-slim
+# Build stage
+FROM node:20-slim AS builder
 
 WORKDIR /app
 
-# Install dependencies
-COPY browserbase/package.json browserbase/package-lock.json ./
-RUN npm ci --only=production
+# Copy package files first for better layer caching
+COPY package.json package-lock.json ./
 
-# Install dependencies
-RUN npm install
+# Install all dependencies including dev dependencies for building
+# --ignore-scripts prevents the prepare script from running prematurely
+RUN npm ci --ignore-scripts
 
-# Copy the rest of the application source code
-COPY browserbase/ .
+# Copy all source files
+COPY . .
 
-# Build the TypeScript source code
-RUN npm run build
+# Build the TypeScript source code manually instead of using the npm script
+RUN npx tsc && npx shx chmod +x dist/*.js 2>/dev/null || echo "No executable JS files found"
 
-# Create the final image from a smaller Node.js runtime
+# Production stage
 FROM node:18-alpine
 
 # Set the working directory
 WORKDIR /app
 
-# Copy built files from the builder stage
+# Copy package files
+COPY package.json package-lock.json ./
+
+# Install only production dependencies
+# --ignore-scripts prevents the prepare script from running
+RUN npm ci --omit=dev --ignore-scripts
+
+# Copy built files from builder stage
 COPY --from=builder /app/dist /app/dist
-COPY --from=builder /app/package.json /app/package.json
-COPY --from=builder /app/package-lock.json /app/package-lock.json
+COPY --from=builder /app/cli.js /app/cli.js
+COPY --from=builder /app/index.js /app/index.js
+COPY --from=builder /app/index.d.ts /app/index.d.ts
+COPY --from=builder /app/config.d.ts /app/config.d.ts
 
-# Define environment variables (can be overridden)
+# Define environment variables (will be provided by Smithery)
 ENV NODE_ENV=production
-ENV BROWSERBASE_API_KEY=<YOUR_BROWSERBASE_API_KEY>
-ENV BROWSERBASE_PROJECT_ID=<YOUR_BROWSERBASE_PROJECT_ID>
 
-# Expose a default port (useful if deploying as a service) 8931 is standard for sse mcp
+# Expose a default port (useful if deploying as a service)
 EXPOSE 8931
 
-CMD [ "node", "cli.js" ] 
+# Command to run the application
+CMD [ "node", "cli.js", "--port", "8931" ] 

browserbase/README.md

@@ -35,7 +35,28 @@ to provide browser automation tools.
 
 ## Local Dev
 
-To run locally we can self-host over SSE. 
+To run locally we can use STDIO or self-host over SSE. 
+
+### STDIO: 
+
+To your MCP Config JSON file add the following: 
+
+```json
+{
+  "mcpServers": {
+    "playwright": {
+      "command" : "node",
+      "args" : ["/path/to/mcp-server-browserbase/browserbase/cli.js"],
+      "env": {
+        "BROWSERBASE_API_KEY": "",
+        "BROWSERBASE_PROJECT_ID": ""
+      }
+    }
+  }
+}
+```
+
+### SSE: 
 
 ```bash
    node cli.js --port 8931

browserbase/smithery.yaml

@@ -1,24 +1,32 @@
 # Smithery configuration file: https://smithery.ai/docs/config#smitheryyaml
 
-startCommand:
-  type: stdio
-  configSchema:
-    # JSON Schema defining the configuration options for the MCP.
-    type: object
-    required:
-      - browserbaseApiKey
-      - browserbaseProjectId
-    properties:
-      browserbaseApiKey:
-        type: string
-        description: The API key for Browserbase.
-      browserbaseProjectId:
-        type: string
-        description: The project ID for Browserbase.
-  commandFunction:
-    |-
-    config => ({command: 'node', args: ['cli.js'], env: {BROWSERBASE_API_KEY: config.browserbaseApiKey, BROWSERBASE_PROJECT_ID: config.browserbaseProjectId}})
+# startCommand:
+#   # Using stdio type which is the most common for MCPs
+#   type: stdio
+#   configSchema:
+#     # JSON Schema defining the configuration options for the MCP.
+#     type: object
+#     required:
+#       - browserbaseApiKey
+#       - browserbaseProjectId
+#     properties:
+#       browserbaseApiKey:
+#         type: string
+#         description: The API key for Browserbase.
+#       browserbaseProjectId:
+#         type: string
+#         description: The project ID for Browserbase.
+#   commandFunction: |
+#     config => ({
+#       command: 'node',
+
+#       env: {
+#         BROWSERBASE_API_KEY: config.browserbaseApiKey,
+#         BROWSERBASE_PROJECT_ID: config.browserbaseProjectId
+#       }
+#     })
 
+startCommand:
   type: http
   configSchema:
     type: object
@@ -36,6 +44,12 @@ startCommand:
       browserbaseProjectId:
         type: string
         description: The project ID for Browserbase.
-  commandFunction:
-    |-
-    config => ({command: 'node', args: ['cli.js --port', config.port], env: {BROWSERBASE_API_KEY: config.browserbaseApiKey, BROWSERBASE_PROJECT_ID: config.browserbaseProjectId}})
+  commandFunction: |
+    config => ({
+      command: 'node',
+      args: ['cli.js', '--port', config.port.toString()],
+      env: {
+        BROWSERBASE_API_KEY: config.browserbaseApiKey,
+        BROWSERBASE_PROJECT_ID: config.browserbaseProjectId
+      }
+    })