feat(soul): MCP connectors — /api/connectors proxy + per-connector auto-approve #14
Reference in New Issue
Block a user
Delete Branch "feat/connectors-soul"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Soul side of the connectors feature. Thin-proxies the neuron-connectd bridge (127.0.0.1:7771) so the UI talks to one origin.
Verified LIVE on :7770: chat, recall, /api/connectors all work. Spec: docs/research/mcp-connectors-adoption-spec.md.
🤖 Generated with Claude Code
Adds the soul side of the connectors feature (spec: docs/research/ mcp-connectors-adoption-spec.md). The soul thin-proxies the neuron-connectd bridge on 127.0.0.1:7771 so the UI talks to one origin and never reaches the bridge directly. routes.el: - handle_connectors + connectd_get/connectd_post helpers (POST bodies go via a temp file + curl -d @file, so model/UI input can't reach the shell). - GET /api/connectors and POST /api/connectors/{add,toggle,auto-approve, remove,secret,oauth/start} registered in both GET and POST routers. chat.el: - tool_auto_approved(): an mcp__* tool skips the approval card only when its server is explicitly opted in (off by default; built-in tools unaffected; bridge down -> false). Wired into the agentic approval gate so an auto-approved connector tool flows straight to execution. Regenerated dist/chat.c and dist/routes.c. Verified live on :7770: real chat, recall, and /api/connectors all work after promotion. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>Handoff (auto) — prereq #16.
WHAT: soul-side connectors surface — chat.el (+528 -177), routes.el (+89 -4); pairs with the neuron-connectd bridge.
REAL SOURCE: 617 lines.
RISK: medium — modifies the agentic chat path + adds routes. Review dispatch/approval changes.
VERIFY: soul builds; a connector tool round-trips through /api/connectors; approval gating intact.
ORDER: after #1 and #3.
Pull request closed