Conductor Proxy T9: migrate built-in tools into proxy (#102) - #108
Draft
ssbn wants to merge 1 commit into
Draft
Conversation
… (T9, #102) Stop the ~19 built-in tools (browser vision + host transfer) from always bloating the tool context. They now flow through the same exposure model as any upstream, via a single-source declarative catalog. - builtin-catalog.ts: one descriptor per built-in (JSON schema + run handler), mirroring the original inline handlers exactly - conductorTools.builtinExposure config: * 'passthrough' (DEFAULT): inline registrations kept untouched -> byte-for-byte the pre-proxy behavior (zero change for current users) * 'search': built-ins skip direct registration and are handed to the proxy facade as local tools -> discoverable via search_tools, invoked via call_tool (conductor__<name>), keeping them out of the advertised set - proxy-tools: LocalTool support -- built-ins indexed under a virtual Conductor server, call_tool routes them in-process (not the supervisor), list_servers reports them - all original gates preserved (master, vision/hostTransfer group flags, Codex vision exclusion, boundSessionId routing); codex_review stays direct - +18 tests incl. JSON-schema->zod parity checks; conductor + vision suites green NOTE: default stays 'passthrough'; flipping the default to 'search' + live app verification is a follow-up decision (see #102). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Collaborator
Author
|
Converting to draft — a desktop-launch issue surfaced during verification: the interstitial loader shows but the main desktop UI never paints. Actively troubleshooting. Do not merge until fixed and a full desktop open-and-use test 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
1 participant
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.
Part of the Conductor Proxy MCP epic (#101). Resolves #102.
Stacked PR — base:
feat/96-proxy-tools.Migrate the ~19 built-in tools (vision + host transfer) into the exposure model.
builtin-catalog.ts: one declarative descriptor per built-in (JSON schema + handler)conductorTools.builtinExposure:passthrough(default, byte-for-byte current behavior) orsearch(built-ins go behind the facade, out of the advertised set)passthrough; flipping tosearch+ live app verification is a follow-up decision.🤖 Generated with Claude Code