Skip to content

[docs]: correct Python dict syntax for cdp_url in browser config#1684

Open
shrey150 wants to merge 1 commit intobrowserbase:mainfrom
shrey150:fix/python-cdp-url-dict-syntax
Open

[docs]: correct Python dict syntax for cdp_url in browser config#1684
shrey150 wants to merge 1 commit intobrowserbase:mainfrom
shrey150:fix/python-cdp-url-dict-syntax

Conversation

@shrey150
Copy link
Contributor

@shrey150 shrey150 commented Feb 13, 2026

Summary

  • Fixes invalid Python syntax in the "Connecting to your local browser" docs example
  • The cdp_url was written using keyword argument syntax (cdp_url="...") inside a dictionary literal {}, which is a syntax error in Python
  • Changed to proper dictionary syntax ("cdp_url": "...")

Fixes #1288

Test plan

  • Verify the corrected code example is valid Python syntax

🤖 Generated with Claude Code


Summary by cubic

Fixes invalid Python dictionary syntax in the local browser configuration docs by changing cdp_url="..." to "cdp_url": "...". This removes the syntax error so the example runs when copy-pasted.

Written for commit c6bf368. Summary will update on new commits. Review in cubic

The CDP URL example used keyword argument syntax (`cdp_url="..."`) inside
a dictionary literal, which is invalid Python. Changed to proper dict
syntax (`"cdp_url": "..."`).

Fixes browserbase#1288

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@changeset-bot
Copy link

changeset-bot bot commented Feb 13, 2026

⚠️ No Changeset found

Latest commit: c6bf368

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.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 13, 2026

Greptile Overview

Greptile Summary

This PR fixes a Python syntax error in the browser configuration documentation. The original code used keyword argument syntax (cdp_url="...") inside a dictionary literal, which is invalid Python. The fix changes it to proper dictionary syntax ("cdp_url": "..."), making the code example valid and copy-pasteable.

  • Corrected Python dictionary syntax in local_browser_launch_options parameter example
  • The TypeScript example was already correct and remains unchanged
  • No other similar syntax errors found in the documentation

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change is a trivial documentation fix that corrects invalid Python syntax. It's a single-line change in a code example that fixes a clear syntax error without affecting any production code or functionality.
  • No files require special attention

Important Files Changed

Filename Overview
packages/docs/v2/configuration/browser.mdx Fixed invalid Python dictionary syntax from cdp_url="..." to "cdp_url": "..." in local browser connection example

Last reviewed commit: c6bf368

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.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@shrey150 shrey150 changed the title fix: correct Python dict syntax for cdp_url in browser config docs [docs]: correct Python dict syntax for cdp_url in browser config Feb 13, 2026
@shrey150 shrey150 requested a review from a team February 13, 2026 06:45
Copy link
Member

@Kylejeong2 Kylejeong2 left a comment

Choose a reason for hiding this comment

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

lgtm, are there other instances in the docs where this is wrong in python? we should have an LLM run through the docs here to verify + also this should be updated in the migration script (if there)

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.

Browser Configuration Python Code in Document Website's Official Docs Contains Errors

2 participants