Skip to content

docs: update browser use documentation to clarify model support requi…#131

Merged
hannesrudolph merged 1 commit into
mainfrom
browser-use-tweak
Apr 14, 2025
Merged

docs: update browser use documentation to clarify model support requi…#131
hannesrudolph merged 1 commit into
mainfrom
browser-use-tweak

Conversation

@hannesrudolph

@hannesrudolph hannesrudolph commented Apr 14, 2025

Copy link
Copy Markdown
Contributor

…rements


Important

Update browser-use.mdx to emphasize model support requirements with a caution box and additional note.

  • Documentation Updates:
    • Changes info box to caution box for model support requirements in browser-use.mdx.
    • Adds note emphasizing Claude Sonnet 3.5 or 3.7 model requirement in the usage section of browser-use.mdx.

This description was created by Ellipsis for 132666e. It will automatically update as commits are pushed.

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 14, 2025
@vercel

vercel Bot commented Apr 14, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
roo-code-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 14, 2025 4:50pm

@ellipsis-dev ellipsis-dev Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 132666e in 1 minute and 33 seconds

More details
  • Looked at 24 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 6 drafted comments based on config settings.
1. docs/features/browser-use.mdx:25
  • Draft comment:
    Good update changing the admonition from 'info' to 'caution' for clearer emphasis. Confirm that the additional
    with margin is necessary for spacing in all display contexts.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50%
    None
2. docs/features/browser-use.mdx:43
  • Draft comment:
    The 'Important: Browser Use requires Claude Sonnet 3.5 or 3.7 model.' note duplicates the message in the caution block. Ensure this redundancy is intentional for clarity.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
3. docs/features/browser-use.mdx:23
  • Draft comment:
    Consider using markdown-native spacing (e.g. margin utilities) instead of an inline
    for consistency with our style guidelines.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
4. docs/features/browser-use.mdx:25
  • Draft comment:
    The admonition type has been changed from 'info' to 'caution'. Ensure this aligns with the intended severity for model requirements.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
5. docs/features/browser-use.mdx:43
  • Draft comment:
    The model requirement is repeated here. Confirm if duplicating the 'Important: ...' message is intentional or if it could be consolidated for clarity.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
6. docs/features/browser-use.mdx:25
  • Draft comment:
    Typographical Error: The caution admonition starts with "::::caution Model Support Required". In MDX, admonitions typically start with three colons (:::caution). Please remove the extra colon to maintain consistency.
  • Reason this comment was not posted:
    Comment looked like it was already resolved.

Workflow ID: wflow_mbZFpZFFlVJJXcyl


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@dosubot dosubot Bot added the documentation Improvements or additions to documentation label Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant