refactor(Dockerfile, executor.ts): enhance Dockerfile and browser initialization logic

* Updated Dockerfile to use Node.js lts instead of lts-alpine, improving compatibility.
* Added installation of Playwright browser dependencies and browsers to the Docker image.
* Modified browser initialization in executor.ts to run headless in Docker environments, enhancing automation capabilities.

Author: imprvhub

Dockerfile

@@ -1,30 +1,51 @@
 # Generated by https://smithery.ai. See: https://smithery.ai/docs/config#dockerfile
-# Build stage
-FROM node:lts-alpine AS build
+# Builder stage
+FROM node:lts AS builder
 WORKDIR /app
 
-# Copy dependency manifests and TypeScript config
-COPY package.json package-lock.json tsconfig.json ./
-
-# Copy TypeScript source files and public assets
+# Copy project files
+COPY package.json tsconfig.json ./
 COPY src ./src
-COPY public ./public
 
-# Install dependencies and build
-RUN npm install
-RUN npm run build
+# Install dependencies and build TypeScript
+RUN npm install && \
+    npm run build
 
-# Runtime stage
-FROM node:lts-alpine AS runtime
+# Final image
+FROM node:lts
 WORKDIR /app
 
-# Copy built artifacts and production modules
-COPY --from=build /app/build ./build
-COPY --from=build /app/node_modules ./node_modules
-COPY --from=build /app/public ./public
+COPY --from=builder /app/dist ./dist
+COPY --from=builder /app/node_modules ./node_modules
+COPY package.json ./
+
+# Install Playwright browser dependencies
+RUN apt-get update && apt-get install -y --no-install-recommends \
+    libglib2.0-0 \
+    libnss3 \
+    libnspr4 \
+    libatk1.0-0 \
+    libatk-bridge2.0-0 \
+    libcups2 \
+    libdrm2 \
+    libdbus-1-3 \
+    libxcb1 \
+    libxkbcommon0 \
+    libx11-6 \
+    libxcomposite1 \
+    libxdamage1 \
+    libxext6 \
+    libxfixes3 \
+    libxrandr2 \
+    libgbm1 \
+    libpango-1.0-0 \
+    libcairo2 \
+    libasound2 \
+    libatspi2.0-0 \
+    && rm -rf /var/lib/apt/lists/*
 
-# Expose port for MCP server if needed
-EXPOSE 8888
+# Install Playwright browsers
+RUN npx playwright install chromium firefox webkit --with-deps
 
-# Default command to start the MCP server
-CMD ["node", "build/index.js"]
+# Expose stdio (no ports) and run the MCP server
+CMD ["node", "dist/index.js"]

SECURITY.md

@@ -6,8 +6,8 @@ Only the latest version of the MCP Browser Agent is actively maintained and rece
 
 | Version | Supported          |
 | ------- | ------------------ |
-| 0.7.x   | :white_check_mark: |
-| < 0.7.0 | :x:                |
+| 0.8.x   | :white_check_mark: |
+| < 0.8.0 | :x:                |
 
 ## Reporting a Vulnerability
 

docker-compose.yml

@@ -0,0 +1,13 @@
+version: '3'
+services:
+  mcp-browser-agent:
+    build:
+      context: .
+      dockerfile: Dockerfile
+    volumes:
+      - /tmp:/tmp  # For file sharing if needed
+    environment:
+      - MCP_BROWSER_TYPE=chromium  # Use chromium by default
+      - MCP_VIEWPORT_WIDTH=1280
+      - MCP_VIEWPORT_HEIGHT=800
+      - MCP_DEVICE_SCALE_FACTOR=1.25

package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-browser-agent",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "author":{
     "name": "Iván Luna",
     "email": "[email protected]",

smithery.yaml

@@ -1,22 +1,26 @@
 name: mcp-browser-agent
-displayName: Status Observer MCP
+displayName: Browser Agent MCP
+description: A Model Context Protocol (MCP) integration that provides Claude Desktop with autonomous browser automation capabilities. 
 visibility: public
 type: mcp
 author:
   name: Iván Luna
   url: https://github.com/imprvhub
 repository: https://github.com/imprvhub/mcp-browser-agent
 keywords:
-  - status
-  - monitoring
-  - platforms
-  - operational
+  - browser
+  - agent
+  - orchestration
+  - automation
 files:
   - README.md
   - package.json
   - tsconfig.json
   - Dockerfile
   - src/index.ts
+  - src/tools.ts
+  - src/handlers.ts
+  - src/executor.ts
 startCommand:
   type: stdio
   configSchema:

src/executor.ts

@@ -110,9 +110,12 @@ async function initBrowser(): Promise<Page> {
         break;
     }
 
+    // Determine if we're running in a Docker container
+    const isDocker = fs.existsSync('/.dockerenv') || fs.existsSync('/proc/1/cgroup') && fs.readFileSync('/proc/1/cgroup', 'utf8').includes('docker');
+    
     browser = await browserInstance.launch({ 
-      headless: false,
-      channel: config.browserType === 'chrome' ? 'chrome' : undefined
+      headless: isDocker ? true : false,
+      channel: config.browserType === 'chrome' && !isDocker ? 'chrome' : undefined
     });
 
     const context = await browser.newContext({

src/tests/basic.test.ts

@@ -0,0 +1,40 @@
+import { test, expect, describe, jest } from '@jest/globals';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const rootDir = path.resolve(__dirname, '..');
+
+describe('MCP Browser Agent - Basic Tests', () => {
+  test('Package should be properly configured', async () => {
+    const packageJsonPath = path.join(rootDir, 'package.json');
+    const packageJsonContent = await fs.promises.readFile(packageJsonPath, 'utf8');
+    const packageJson = JSON.parse(packageJsonContent);
+    
+    expect(packageJson.name).toBe('mcp-browser-agent');
+    expect(packageJson.type).toBe('module');
+    expect(typeof packageJson.scripts.test).toBe('string');
+  });
+  
+  test('Project should have required files', async () => {
+    expect(fs.existsSync(path.join(rootDir, 'src/index.ts'))).toBeTruthy();
+    expect(fs.existsSync(path.join(rootDir, 'src/executor.ts'))).toBeTruthy();
+    expect(fs.existsSync(path.join(rootDir, 'src/tools.ts'))).toBeTruthy();
+    expect(fs.existsSync(path.join(rootDir, 'src/handlers.ts'))).toBeTruthy();
+  });
+  
+  test('Tools module should exist', async () => {
+    const toolsPath = path.join(rootDir, 'src/tools.ts');
+    expect(fs.existsSync(toolsPath)).toBeTruthy();
+
+    const toolsContent = await fs.promises.readFile(toolsPath, 'utf8');
+    expect(toolsContent).toContain('BROWSER_TOOLS');
+    expect(toolsContent).toContain('browser_navigate');
+    expect(toolsContent).toContain('browser_screenshot');
+    expect(toolsContent).toContain('API_TOOLS');
+    expect(toolsContent).toContain('api_get');
+    expect(toolsContent).toContain('api_post');
+  });
+});

src/tests/browser-error.test.ts

@@ -0,0 +1,52 @@
+import { test, expect, describe, jest, beforeAll } from '@jest/globals';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const rootDir = path.resolve(__dirname, '..');
+const processEvents: Record<string, Array<() => void>> = {
+  'SIGINT': [],
+  'SIGTERM': []
+};
+
+let cleanupFn: boolean | undefined;
+
+beforeAll(() => {
+  const executorPath = path.join(rootDir, 'src/executor.ts');
+  const executorContent = fs.readFileSync(executorPath, 'utf8');
+  if (executorContent.includes('process.on(\'SIGINT\'')) {
+    processEvents['SIGINT'] = [() => {}];
+  }
+  
+  if (executorContent.includes('process.on(\'SIGTERM\'')) {
+    processEvents['SIGTERM'] = [() => {}];
+  }
+  
+  if (executorContent.includes('cleanupBrowser')) {
+    cleanupFn = true;
+  }
+});
+
+describe('Browser Error Handling Tests', () => {
+  test('Executor should contain process cleanup handlers', async () => {
+    const executorPath = path.join(rootDir, 'src/executor.ts');
+    const executorContent = fs.readFileSync(executorPath, 'utf8');
+    expect(executorContent.includes('process.on(\'SIGINT\'')).toBeTruthy();
+    expect(executorContent.includes('process.on(\'SIGTERM\'')).toBeTruthy();
+    expect(executorContent.includes('cleanupBrowser')).toBeTruthy();
+  });
+
+  test('README should contain browser process cleanup documentation', () => {
+    const readmePath = path.join(rootDir, 'README.md');
+    const readmeContent = fs.readFileSync(readmePath, 'utf8');
+    expect(readmeContent).toContain('Browser process not closing properly');
+    expect(readmeContent).toContain('Windows');
+    expect(readmeContent).toContain('macOS');
+    expect(readmeContent).toContain('Linux');
+    expect(readmeContent).toContain('Playwright');
+    expect(readmeContent).toContain('issues');
+    expect(readmeContent).toContain('github.com/microsoft/playwright/issues');
+  });
+});

src/tests/docker-test.js

@@ -0,0 +1,32 @@
+const { chromium } = require('playwright');
+
+async function test() {
+  console.log('Starting browser test in Docker environment...');
+
+  const isDocker = require('fs').existsSync('/.dockerenv') || 
+    (require('fs').existsSync('/proc/1/cgroup') && 
+     require('fs').readFileSync('/proc/1/cgroup', 'utf8').includes('docker'));
+  
+  console.log(`Running in Docker environment: ${isDocker}`);
+  
+  try {
+    const browser = await chromium.launch({ headless: true });
+    console.log('Browser launched successfully');
+    const context = await browser.newContext();
+    const page = await context.newPage();
+    console.log('Browser page created successfully');
+    await page.goto('https://example.com');
+    console.log('Navigation successful');
+    await page.screenshot({ path: '/tmp/example.png' });
+    console.log('Screenshot saved to /tmp/example.png');
+    const title = await page.title();
+    console.log(`Page title: ${title}`);
+    await browser.close();
+    console.log('Browser closed successfully');
+    console.log('Test completed successfully!');
+  } catch (error) {
+    console.error('Test failed:', error);
+  }
+}
+
+test();

tsconfig.json

@@ -8,9 +8,10 @@
     "strict": true,
     "skipLibCheck": true,
     "isolatedModules": true,
+    "rootDir": "src",
     "outDir": "dist",
     "declaration": true
   },
-  "include": ["src/**/*", "tests/**/*"],
+  "include": ["src/**/*", "src/tests/**/*"],
   "exclude": ["node_modules", "dist"]
 }