Skip to content

fix(docs): correct browser CDP API usage in documentation - #208

Open
warjiang wants to merge 1 commit into
agent-infra:mainfrom
warjiang:fix/browser-cdp-doc-errors
Open

fix(docs): correct browser CDP API usage in documentation#208
warjiang wants to merge 1 commit into
agent-infra:mainfrom
warjiang:fix/browser-cdp-doc-errors

Conversation

@warjiang

Copy link
Copy Markdown

fix some outdated document

  • get_browser_info() -> get_info()
  • browser_info.cdp_url -> browser_info.data.cdp_url
  • take_screenshot() -> screenshot()

- Fix wrong method name: get_browser_info() -> get_info()
- Fix missing .data accessor: browser_info.cdp_url -> browser_info.data.cdp_url
- Fix wrong method name: take_screenshot() -> screenshot()

The get_info() method returns ResponseBrowserInfoResult which wraps the
actual BrowserInfoResult in a .data field. Documentation was incorrectly
accessing fields directly and using non-existent method names.

Affected files:
- website/docs/en/blog/announcing-0.mdx
- website/docs/zh/blog/announcing-0.mdx
- website/docs/en/guide/basic/browser.mdx
- website/docs/zh/guide/basic/browser.mdx
- website/docs/en/guide/advanced/browser-downloads.md
- website/docs/zh/guide/advanced/browser-downloads.md
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.

1 participant