Skip to content

Tighten fetch skill safety docs#42

Merged
Kylejeong2 merged 3 commits intomainfrom
shrey/fetch-skill-followup
Mar 15, 2026
Merged

Tighten fetch skill safety docs#42
Kylejeong2 merged 3 commits intomainfrom
shrey/fetch-skill-followup

Conversation

@shrey150
Copy link
Copy Markdown
Contributor

@shrey150 shrey150 commented Mar 12, 2026

Summary

  • add a top-level safety note for untrusted response.content
  • replace the risky redirect and internal TLS examples with public test URLs
  • tighten the fetch skill description and allowInsecureSsl guidance

Testing

  • not run (docs-only changes)

Note

Low Risk
Docs-only changes that clarify safe handling of fetched content and restrict guidance around insecure TLS; no runtime or API behavior changes.

Overview
Adds explicit safety guidance across the Fetch skill docs to treat returned content/response.content as untrusted and not follow embedded instructions.

Removes/rewrites examples that encouraged riskier usage (redirect-following and internal/self-signed TLS scenarios), and tightens allowInsecureSsl documentation to limit it to trusted test/staging or public test hosts (and explicitly warn against private-network/metadata endpoints). Also refreshes the Fetch skill description and best-practices list to emphasize safer usage.

Written by Cursor Bugbot for commit 9ced10e. This will update automatically on new commits. Configure here.

Copy link
Copy Markdown
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.

Left a comment about adding stuff to the browse CLI description, otherwise looks good

Comment thread skills/fetch/SKILL.md
---
name: fetch
description: "Fetch web pages and return their content, headers, and metadata using the Browserbase Fetch API. Use when the user wants to retrieve page content without a full browser session — ideal for scraping static pages, checking HTTP responses, or getting page source. Supports proxies, redirect control, and insecure SSL bypass."
description: "Use this skill when the user wants to retrieve a URL without a full browser session: fetch HTML or JSON from static pages, inspect status codes or headers, follow redirects, or get page source for simple scraping. Prefer it over a browser when JavaScript rendering and page interaction are not needed. Supports proxies and redirect control."
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we add a line to use a browser instead of fetch when JS rendering is required / use it as a fallback in the description of the browse CLI

@Kylejeong2 Kylejeong2 merged commit afc8d9c into main Mar 15, 2026
1 check passed
shrey150 added a commit that referenced this pull request Mar 24, 2026
* Tighten fetch skill safety docs

* Remove fetch redirect example

* Move fetch TLS bypass docs to reference
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.

2 participants