Skip to content

Fix docs R2: auth header, legacy property names#100

Open
MagMueller wants to merge 2 commits intomainfrom
magnus/citrine-function
Open

Fix docs R2: auth header, legacy property names#100
MagMueller wants to merge 2 commits intomainfrom
magnus/citrine-function

Conversation

@MagMueller
Copy link
Copy Markdown
Contributor

@MagMueller MagMueller commented Mar 30, 2026

Summary

  • Fix auth header in agent/quickstart.mdx curl example: Authorization: BearerX-Browser-Use-API-Key
  • Fix legacy/agent.mdx: upload.presignedUrlupload.url, method PUTPOST; output.presignedUrloutput.downloadUrl
  • Fix legacy/public-share.mdx: share.urlshare.shareUrl (both Python and TS)

All Authorization: Bearer instances now removed from MDX sources (only remain in generated llms-full.txt which will regenerate).

Still needs input

  • profile.sh URL is 404 — need correct URL or removal
  • browser-use-cli GitHub repo is 404 — need correct repo URL

Test plan

  • Verify curl examples work with X-Browser-Use-API-Key
  • Verify legacy file upload/download/share snippets compile

🤖 Generated with Claude Code


Summary by cubic

Fix docs to use the X-Browser-Use-API-Key header, align legacy examples with the current API, and remove a dead CLI repo link. Curl and code samples now work as written; CLI docs clarify that profile is built-in.

  • Bug Fixes
    • In docs/cloud/agent/quickstart.mdx, replace Authorization: Bearer with X-Browser-Use-API-Key.
    • In docs/cloud/legacy/agent.mdx, change upload.presignedUrlupload.url with POST, and output.presignedUrloutput.downloadUrl.
    • In docs/cloud/legacy/public-share.mdx, change share.urlshare.shareUrl (TS) and share.share_url (Python).
    • In docs/open-source/browser-use-cli.mdx, remove the dead repo link and state the profile subcommand syncs cookies and is part of the CLI.

Written for commit 37ed738. Summary will update on new commits.

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.

1 issue found across 3 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/legacy/agent.mdx">

<violation number="1" location="docs/cloud/legacy/agent.mdx:77">
P2: The snippet now uses POST, but the section explicitly says presigned uploads should use PUT. This makes the example inconsistent and likely incorrect for presigned URLs.</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.

await fetch(upload.presignedUrl, {
method: "PUT",
await fetch(upload.url, {
method: "POST",
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot Mar 30, 2026

Choose a reason for hiding this comment

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

P2: The snippet now uses POST, but the section explicitly says presigned uploads should use PUT. This makes the example inconsistent and likely incorrect for presigned URLs.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At docs/cloud/legacy/agent.mdx, line 77:

<comment>The snippet now uses POST, but the section explicitly says presigned uploads should use PUT. This makes the example inconsistent and likely incorrect for presigned URLs.</comment>

<file context>
@@ -73,8 +73,8 @@ const upload = await client.files.sessionUrl(session.id, {
-await fetch(upload.presignedUrl, {
-  method: "PUT",
+await fetch(upload.url, {
+  method: "POST",
   body: readFileSync("input.pdf"),
   headers: { "Content-Type": "application/pdf" },
</file context>
Suggested change
method: "POST",
method: "PUT",
Fix with Cubic

MagMueller and others added 2 commits March 30, 2026 15:17
- agent/quickstart.mdx: Authorization: Bearer → X-Browser-Use-API-Key
- legacy/agent.mdx: upload.presignedUrl → upload.url, method PUT → POST
- legacy/agent.mdx: output.presignedUrl → output.downloadUrl
- legacy/public-share.mdx: share.url → share.shareUrl (both Python and TS)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Remove link to non-existent github.com/browser-use/browser-use-cli repo.
The profile subcommand is part of the CLI itself.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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