feat(skills): remove interaction-skills; ship only BROWSER.md + cloud-browser.md#52
Merged
Merged
Conversation
…-browser.md
Evals show the agent rarely reads the 16 interaction-skills/*.md files inherited from the Python harness, and is performing well without them. Pulling them out of the embedded skills tree so we can be data-driven about what gets shipped: reintroduce only the recipes evals show are needed, one at a time, with before/after evidence. Files moved to <workspace>/reference/interaction-skills/ for reference. In-repo updates: trimmed three references in BROWSER.md (Locations bullet, the model paragraph, the UI-mechanics paragraph, and the iframe/shadow-dom troubleshooting bullet which now inlines the CDP one-liner); dropped the trailing UI-mechanics paragraph in browser-execute.txt; updated README.md, src/index.ts comment, browser-execute.ts substitution comment, and skills.test.ts assertions. Also normalized skills.test.ts assertions to be cross-platform (the original used path.join which mismatches the forward slash used in {{SKILLS_DIR}}/cloud-browser.md after substitution on Windows). bun test test/skills.test.ts test/browser-execute.test.ts: 4 pass / 6 skip (chrome smoke) / 0 fail. Both packages typecheck clean.
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.
Summary
Evals show the agent rarely reads the 16
interaction-skills/*.mdfilesinherited from the Python harness era, and is performing well without them.
Pulling them out of the embedded skills tree so we can be data-driven about
what ships: reintroduce only the recipes evals show are needed, one at a
time, with before/after evidence.
What's left in
skills/BROWSER.md— the agent's prompt forbrowser_execute(snippet model,three connection methods, workspace pattern,
sessionAPI surface,guardrails, troubleshooting).
cloud-browser.md— Way 3 cloud-browser provisioning recipe.What moved out
packages/bcode-browser/skills/interaction-skills/(16 files) is moved to<workspace>/reference/interaction-skills/on the maintainer's machine,with a
PROVENANCE.mdnoting origin and reason. Not committed anywhere —kept as a private reference for picking what to restore.
In-repo edits
skills/BROWSER.md— trimmed three references:one-liner (
Page.getFrameTree/element.shadowRoot.querySelector)instead of pointing at deleted skill files.
packages/opencode/src/tool/browser-execute.txt— drops the trailing"For UI-mechanic recipes …" paragraph.
packages/opencode/src/tool/browser-execute.ts— comment updated to sayBROWSER.md / cloud-browser.mdinstead of the oldinteraction-skillsreference.
packages/bcode-browser/src/index.ts— comment forskills/updated.packages/bcode-browser/README.md—skills/row updated; one-sentencenote about the archival decision.
packages/bcode-browser/test/skills.test.ts:interaction-skillssubstring assertion.cloud-browser.mdassertion to be cross-platform(the original used
path.joinwhich doesn't match the forward slashbaked into
{{SKILLS_DIR}}/cloud-browser.mdafter substitution onWindows). Drive-by fix; same intent.
Out of scope
UPSTREAM.md§3 andAGENTS.md"Vendored harness" still reference aninteraction-skills/path under the (already-retired) Python harness.That whole section is stale post Phase H and unrelated to this change —
will be cleaned up separately.
Verification
bun test test/skills.test.ts test/browser-execute.test.ts— 4 pass /6 skip (chrome smoke gated on
BCODE_SMOKE_CHROME=1) / 0 fail.bun typecheckclean inpackages/bcode-browserandpackages/opencode.22 files changed, 11 insertions(+), 5 deletions(-) for code; 16 deletions
under
skills/interaction-skills/for the archived files.