Narrow skill description to match When Not to Use - #555
Open
illmethinks0 wants to merge 1 commit into
Open
Conversation
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.
✅ Skill review passedReviewed 1 file(s) — no findings. |
Kastan97
reviewed
Jul 28, 2026
Kastan97
left a comment
There was a problem hiding this comment.
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
The
SKILL.mdfrontmatter description still reads:0.1.7 added a When Not to Use section that says the opposite for the simple cases - prefer
curlfor a plain HTTP fetch of public content. The description was never updated to match.git log -L 3,3:SKILL.mdshows it unchanged apart from the browser-use -> browser-harness rename.Why it matters
In Claude Code and Codex the frontmatter
descriptionis 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.mddescription to match guidance: usebrowser-harnessfor real-browser interactions (clicking, typing, navigation, logged-in, JS-rendered or bot-protected), not plain HTTP fetches. Prefercurlfor simple public fetches to avoid unnecessary browser launches.Written for commit 06cbbc7. Summary will update on new commits.