Skip to content

fix: Give bundled subagents access to the semble MCP tools - #291

Closed
konopka wants to merge 1 commit into
MinishLab:mainfrom
konopka:fix/subagent-mcp-tool-access
Closed

konopka wants to merge 1 commit into
MinishLab:mainfrom
konopka:fix/subagent-mcp-tool-access

Conversation

@konopka

@konopka konopka commented Oct 8, 2026

Copy link
Copy Markdown

Fixes #290.

The bundled semble-search subagent definitions gated their tools to shell/read only, so none of the shipped subagents could call the semble MCP tools registered by the same semble install — every search fell back to the ~5 s-per-call CLI instead of the in-memory MCP path (~18 ms median on the benchmark repo).

Changes, per tier:

All rewritten bodies are MCP-first with a CLI fallback section. Templates keep the literal uvx --from "semble[mcp]" semble line, which the installer rewrites into the version-pinned spec at install time.

Also updates the claude.md gate test (which pinned the old Bash-only behavior) and adds parametrized gate/body/pin tests per tier.

Verified end-to-end (2026-10-08):

  • Opencode: fresh session — subagent called semble_search via MCP on the first search, no CLI fallback
  • Claude Code: fresh session on the branch build — "Searched for 1 pattern, called semble 2 times" (MCP calls, no bash)

The bundled semble-search subagent definitions gated their tools to shell/read only, so the in-memory MCP tools (tens of ms per search) were never reachable and every search fell back to the ~5s-per-call CLI.

- Tier A: claude, opencode, kilo - allow the confirmed MCP tool names alongside the original shell/read entries
- Tier B: gemini, qwen, copilot, kiro, zcode, commandcode, reasonix - same, with each harness's documented naming convention
- Tier C: cursor, codex, pi, grok - body-only MCP-first with graceful CLI fallback (no tool gate)
- Bodies are MCP-first with a CLI fallback section keeping the pinned uvx line; antigravity left unchanged (naming unconfirmed, known gap)
- Tests: parametrized gate/body/pin assertions per tier; updated the claude.md gate test that pinned the old Bash-only behavior
@Pringled

Pringled commented Oct 8, 2026

Copy link
Copy Markdown
Member

Doesn't follow our guidelines.

@Pringled Pringled closed this Oct 8, 2026
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.

Shipped subagent definitions are CLI-only for every client: subagents can't use the MCP server semble install registers

2 participants