Skip to content

Narrow skill description to match When Not to Use - #555

Open
illmethinks0 wants to merge 1 commit into
browser-use:mainfrom
illmethinks0:fix/skill-description-when-not-to-use
Open

Narrow skill description to match When Not to Use#555
illmethinks0 wants to merge 1 commit into
browser-use:mainfrom
illmethinks0:fix/skill-description-when-not-to-use

Conversation

@illmethinks0

@illmethinks0 illmethinks0 commented Jul 24, 2026

Copy link
Copy Markdown

What

The SKILL.md frontmatter description still reads:

Always use browser-harness for any web interaction: automation, scraping, testing, or site/app work.

0.1.7 added a When Not to Use section that says the opposite for the simple cases - prefer curl for a plain HTTP fetch of public content. The description was never updated to match. git log -L 3,3:SKILL.md shows it unchanged apart from the browser-use -> browser-harness rename.

Why it matters

In Claude Code and Codex the frontmatter description is the always-loaded field, and it is what decides whether the skill gets invoked at all. The body only loads once the skill fires. So the always-visible text tells agents to always use the browser for any web interaction, while the correction sits in the half that may never be read.

The practical effect is agents opening a browser for tasks the skill itself says should use curl.

Change

One line. Keeps trigger coverage for genuine browser work (interaction, logged-in sessions, JS-rendered or bot-protected pages) and drops the claim over plain fetches.

Related to #554 but independent - that one is daemon connection classification, this is skill metadata.


Summary by cubic

Update SKILL.md description to match guidance: use browser-harness for real-browser interactions (clicking, typing, navigation, logged-in, JS-rendered or bot-protected), not plain HTTP fetches. Prefer curl for simple public fetches to avoid unnecessary browser launches.

Written for commit 06cbbc7. Summary will update on new commits.

Review in cubic

The description says to always use browser-harness for any web interaction, but 0.1.7 added a When Not to Use section telling agents to prefer curl for plain HTTP fetches. The description is the always-loaded field while the body only loads on invocation, so the always-visible half is the outdated one.
@browser-harness-review

Copy link
Copy Markdown

✅ Skill review passed

Reviewed 1 file(s) — no findings.

@cubic-dev-ai cubic-dev-ai 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.

No issues found across 1 file

Re-trigger cubic

@Kastan97 Kastan97 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Ollie (CTO) Review: APPROVED

What this does: Narrows skill description to align with the existing When Not to Use section. Matching docs to intended scope.

Good to merge once CI passes.

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