Skip to content

fix(install): keep model: in opencode agents, strip only tools array #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
0p9b merged 1 commit into from
Jul 15, 2026
Merged

fix(install): keep model: in opencode agents, strip only tools array #9

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 12 additions & 13 deletions bin/eap-install.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -513,26 +513,25 @@ function copyDirRecursive(src, dest) {
}
}

// Strip Claude-only agent frontmatter (`tools:` as a YAML array, `model:`) when
// writing eapcrew agents into a non-Claude host like opencode. opencode rejects
// the YAML-array `tools:` form — one such file makes the WHOLE config invalid,
// so no MCP/skills/agents load at all — and it cannot resolve Anthropic model
// aliases such as `haiku`. Omitting both lets opencode fall back to its defaults
// while the agent prompt body still self-restricts. Mirrors TLDR's
// bin/lib/opencode-agent.js. Claude keeps the array + model (it honors both).
function stripClaudeAgentFrontmatter(content) {
// Strip Claude's YAML-array `tools:` frontmatter when writing eapcrew agents
// into opencode. opencode requires `tools` as an object map (or the field
// omitted) and rejects the array form — one such file makes the WHOLE opencode
// config invalid, so no MCP/skills/agents load at all. Omitting it lets opencode
// use its defaults while the agent prompt body still self-restricts. `model:` is
// a valid opencode field and is left intact (matches TLDR's bin/lib/opencode-
// agent.js, whose tests deliberately preserve it). Claude keeps the array as-is.
function stripOpencodeAgentTools(content) {
const FENCE = '---\n';
if (typeof content !== 'string' || !content.startsWith(FENCE)) return content;
const fmEnd = content.indexOf('\n---', FENCE.length);
if (fmEnd < 0) return content;
const fm = content.slice(FENCE.length, fmEnd);
const rest = content.slice(fmEnd);
const DROP = /^(tools|model)[ \t]*:/;
const out = [];
let dropping = false;
for (const line of fm.split('\n')) {
if (dropping) { if (/^[ \t]/.test(line)) continue; dropping = false; }
if (DROP.test(line)) { dropping = true; continue; }
if (/^tools[ \t]*:/.test(line)) { dropping = true; continue; }
out.push(line);
}
return FENCE + out.join('\n') + rest;
Expand Down Expand Up @@ -870,9 +869,9 @@ function installNativeAssets(ctx, prov) {
const dest = resolveSentinelPath(n.agents);
if (opts.dryRun) note(` agents: copy eapcrew into ${dest}/`);
else {
// Native hosts that take eapcrew subagents (opencode) need the Claude-only
// `tools:`/`model:` frontmatter stripped or their config load breaks.
const err = installAgentsInto(dest, opts, stripClaudeAgentFrontmatter);
// opencode rejects Claude's YAML-array `tools:`; strip it (a single bad
// file invalidates the whole opencode config). `model:` is left intact.
const err = installAgentsInto(dest, opts, stripOpencodeAgentTools);
if (err) warn(` agents failed (${dest}): ${err}`);
else ok(` agents → ${dest}`);
}
Expand Down