Skip to content

fix: bump Python to 3.10, drop archived selenium-wire, note webhook example needs Python 3.12#134

Open
MagMueller wants to merge 2 commits intomainfrom
MagMueller/docs-audit-fixes
Open

fix: bump Python to 3.10, drop archived selenium-wire, note webhook example needs Python 3.12#134
MagMueller wants to merge 2 commits intomainfrom
MagMueller/docs-audit-fixes

Conversation

@MagMueller
Copy link
Copy Markdown
Contributor

@MagMueller MagMueller commented Apr 11, 2026

Summary

Three small fixes surfaced by a 13-agent doc audit:

  • browser-use-python requires-python: >=3.9>=3.10. The generated models use PEP 604 str | None union syntax, which crashes on Python 3.9 with TypeError: Unable to evaluate type annotation 'str | None'. Stock macOS ships Python 3.9, so the current metadata is broken for any new user on a fresh machine.
  • Remove archived selenium-wire link. github.com/wkeeling/selenium-wire has been read-only since Jan 2024. The Playwright CDP bridge workaround is kept.
  • Webhook example: note that Python 3.12+ is required. The verify_webhook snippet uses an f-string with nested quotes (f"...{json.dumps(payload, separators=(',', ':'), ...)}") which is a SyntaxError on Python < 3.12. Added a <Note> callout pointing at the fix instead of rewriting the snippet.

Model-name fixes (hyphens → dots) were already landed in #116, so no changes needed there.

Also regenerated llms.txt / llms-full.txt.

Test plan

  • uv sync in browser-use-python/ on Python 3.10
  • Verify Mintlify docs preview renders the new Webhooks <Note>
  • Confirm Playwright page still reads cleanly after selenium-wire removal

Summary by cubic

Bump browser-use-python to Python 3.10+ and clean up docs by removing the archived selenium-wire. Also removed an incorrect Python 3.12 note from the webhook example and reverted accidental LLM index regeneration.

  • Bug Fixes

    • Set requires-python to >=3.10 and pyright.pythonVersion to 3.10; simplified uv.lock by dropping 3.9 markers.
    • Updated Selenium guidance to use Playwright’s CDP bridge; removed selenium-wire.
    • Removed incorrect Webhooks Python 3.12 warning; example works on 3.9+. Reverted llms.txt / llms-full.txt regen.
  • Migration

    • Use Python 3.10+ for browser-use-python.

Written for commit ed01472. Summary will update on new commits.

…xample needs Python 3.12

- browser-use-python: bump requires-python from 3.9 to 3.10 (generated models use
  PEP 604 `str | None` union syntax which breaks on 3.9, contradicting metadata)
- docs(playwright): remove archived selenium-wire reference
  (github.com/wkeeling/selenium-wire has been read-only since Jan 2024)
- docs(webhooks): note that the Python verify_webhook example requires Python 3.12+
  because of f-string nested quotes in `separators=(',', ':')`
- regenerate llms.txt / llms-full.txt

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mintlify
Copy link
Copy Markdown

mintlify bot commented Apr 11, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
browser-use 🟢 Ready View Preview Apr 11, 2026, 2:21 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Copy link
Copy Markdown

@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.

2 issues found across 9 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="docs/cloud/llms-full.txt">

<violation number="1" location="docs/cloud/llms-full.txt:1882">
P2: The Hermes in-page link `#agent-self-registration` is ambiguous because this file now has multiple `## Agent Self-Registration` headings, so the link can jump to the wrong section.</violation>
</file>

<file name="docs/cloud/guides/webhooks.mdx">

<violation number="1" location="docs/cloud/guides/webhooks.mdx:40">
P2: The new note incorrectly claims this f-string requires Python 3.12+, but the shown syntax is valid on earlier versions (e.g., Python 3.11), so the docs now state a false version constraint.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review, or fix all with cubic.

Comment thread docs/cloud/llms-full.txt Outdated
Comment thread docs/cloud/guides/webhooks.mdx Outdated
…ote, revert llms.txt regen

- webhooks.mdx: the f-string uses different outer/inner quote chars (`"` vs `'`),
  so it works on Python 3.9+ — the 3.12+ claim was wrong. Remove the Note.
- Revert llms.txt / llms-full.txt regeneration. Regenerating surfaced a
  pre-existing duplicate `## Agent Self-Registration` heading in two tutorial
  MDX files (hermes-agent and claude-code), which made the `#agent-self-registration`
  anchor ambiguous. That's out of scope for this PR; the docs team can regenerate
  llms.txt in a follow-up.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@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.

3 issues found across 6 files (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="docs/open-source/llms-full.txt">

<violation number="1">
P2: This new generic “Other models” section removes provider-specific setup details that are still present in the source docs.</violation>

<violation number="2">
P2: The Cloud API command table was changed to unsupported/undocumented `cloud connect` flags, diverging from the source CLI docs.</violation>

<violation number="3">
P2: The generated tab command table is out of sync with the canonical CLI docs and now documents different commands.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review, or fix all with cubic.

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