Skip to content

Conversation

@Kylejeong2
Copy link
Member

why

We recently shipped updates to our MCP server that should be reflected in the documentation.

what changed

update tools list for MCP update, removing mentions of multisession, adding experimental flag + get url tool

related PR: browserbase/mcp-server-browserbase#123

test plan

n/a

@changeset-bot
Copy link

changeset-bot bot commented Oct 16, 2025

⚠️ No Changeset found

Latest commit: 5e9bcab

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Overview

Summary

Updated MCP server documentation to reflect recent changes in the mcp-server-browserbase package. Key changes include removing all multi-session functionality and documentation, updating model references to latest Claude 4.5 versions (Sonnet and Haiku), and adding the experimental get_url tool.

Major Changes:

  • Removed multi-session management tools and architecture documentation from introduction.mdx and tools.mdx
  • Updated Anthropic model names to claude-sonnet-4-5-20250929 and claude-haiku-4-5-20251001 across all docs
  • Cleaned up OpenAI model list (removed o1/o3 models, added gpt-4.1)
  • Fixed documentation link to point to correct models configuration page
  • Simplified session management section to focus on single-session workflow

Issues Found:

  • Documentation formatting inconsistency in models.mdx table (comma-separated model names)
  • Incorrect parameter documentation for browserbase_stagehand_extract in tools.mdx (using ParamField for output instead of ResponseField)

Confidence Score: 4/5

  • This PR is safe to merge after addressing minor documentation issues
  • Documentation-only changes that correctly reflect MCP server updates. Two minor issues found: table formatting inconsistency and incorrect parameter type in API documentation. These don't affect functionality but should be fixed for accuracy.
  • docs/integrations/mcp/tools.mdx has incorrect parameter documentation, docs/configuration/models.mdx has formatting inconsistency

Important Files Changed

File Analysis

Filename Score Overview
docs/configuration/models.mdx 4/5 Updated Anthropic model names to latest Claude 4.5 versions (Sonnet and Haiku), formatting inconsistent with comma-separated list
docs/integrations/mcp/tools.mdx 4/5 Removed all multi-session tools and documentation, added experimental get_url tool, has incorrect documentation for browserbase_stagehand_extract

Sequence Diagram

sequenceDiagram
    participant User
    participant Claude Desktop
    participant MCP Server
    participant Browserbase API
    participant Browser Session

    User->>Claude Desktop: Request browser automation
    Claude Desktop->>MCP Server: Call browserbase_session_create
    MCP Server->>Browserbase API: Create session with config
    Browserbase API->>Browser Session: Initialize browser
    Browser Session-->>MCP Server: Session ready
    MCP Server-->>Claude Desktop: Return session info
    
    Claude Desktop->>MCP Server: Call browserbase_stagehand_navigate
    MCP Server->>Browser Session: Navigate to URL
    Browser Session-->>MCP Server: Navigation complete
    
    Claude Desktop->>MCP Server: Call browserbase_stagehand_act
    MCP Server->>Browser Session: Perform action using LLM
    Browser Session-->>MCP Server: Action result
    
    Claude Desktop->>MCP Server: Call browserbase_stagehand_extract
    MCP Server->>Browser Session: Extract content
    Browser Session-->>MCP Server: Extracted data
    MCP Server-->>Claude Desktop: Return extracted data
    
    Claude Desktop->>MCP Server: Call browserbase_session_close
    MCP Server->>Browser Session: Close session
    Browser Session-->>Browserbase API: Cleanup
    MCP Server-->>Claude Desktop: Session closed
Loading

4 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

|----------|-------------|-----------|----------|
| **OpenAI** | `gpt-4.1`, `gpt-4.1-mini` | High accuracy, reliable | Production, complex sites |
| **Anthropic** | `claude-3-7-sonnet-latest` | Excellent reasoning | Complex automation tasks |
| **Anthropic** | `claude-sonnet-4-5-20250929, claude-haiku-4-5-20251001` | Excellent reasoning | Complex automation tasks |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style: formatting issue: model names should be separated by | or listed on separate lines for consistency with table format, not comma-separated inline

Suggested change
| **Anthropic** | `claude-sonnet-4-5-20250929, claude-haiku-4-5-20251001` | Excellent reasoning | Complex automation tasks |
| **Anthropic** | `claude-sonnet-4-5-20250929` | `claude-haiku-4-5-20251001` | Excellent reasoning | Complex automation tasks |
Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/configuration/models.mdx
Line: 51:51

Comment:
**style:** formatting issue: model names should be separated by `|` or listed on separate lines for consistency with table format, not comma-separated inline

```suggestion
| **Anthropic** | `claude-sonnet-4-5-20250929` | `claude-haiku-4-5-20251001` | Excellent reasoning | Complex automation tasks |
```

How can I resolve this? If you propose a fix, please make it concise.

Comment on lines +37 to +39
<ParamField path="instruction" type="string">
Extracted text content from the current page
</ParamField>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

syntax: instruction is an output parameter, not an input - this should be wrapped in a ResponseField not a ParamField

Suggested change
<ParamField path="instruction" type="string">
Extracted text content from the current page
</ParamField>
<ResponseField name="content" type="string">
Extracted text content from the current page
</ResponseField>
Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/integrations/mcp/tools.mdx
Line: 37:39

Comment:
**syntax:** `instruction` is an output parameter, not an input - this should be wrapped in a `ResponseField` not a `ParamField`

```suggestion
<ResponseField name="content" type="string">
  Extracted text content from the current page
</ResponseField>
```

How can I resolve this? If you propose a fix, please make it concise.

@Kylejeong2 Kylejeong2 merged commit 5da2655 into main Oct 16, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants