neuron-technologies/neuron
3
0
Fork 0
Code Issues 44 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

feat(soul): MCP connectors — /api/connectors proxy + per-connector auto-approve #14

Closed
tim.lingo wants to merge 2 commits from feat/connectors-soul into feat/native-web-search
pull from: feat/connectors-soul
merge into: :feat/native-web-search
Conversation 1 Commits 2 Files Changed 22
+2090 -421
tim.lingo commented 2026-06-13 23:44:10 +00:00
Member
Copy Link
Copy Source

Soul side of the connectors feature. Thin-proxies the neuron-connectd bridge (127.0.0.1:7771) so the UI talks to one origin.

  • routes.el: handle_connectors + connectd_get/post (POST bodies via temp file + curl -d @file). GET /api/connectors + POST add/toggle/auto-approve/remove/secret/oauth-start.
  • chat.el: tool_auto_approved() wired into the agentic approval gate. Off by default; only an opted-in mcp__* tool skips approval. Built-ins unaffected; bridge-down is safe.
  • Regenerated dist/chat.c + dist/routes.c.

Verified LIVE on :7770: chat, recall, /api/connectors all work. Spec: docs/research/mcp-connectors-adoption-spec.md.

🤖 Generated with Claude Code

Soul side of the connectors feature. Thin-proxies the neuron-connectd bridge (127.0.0.1:7771) so the UI talks to one origin. - routes.el: handle_connectors + connectd_get/post (POST bodies via temp file + curl -d @file). GET /api/connectors + POST add/toggle/auto-approve/remove/secret/oauth-start. - chat.el: tool_auto_approved() wired into the agentic approval gate. Off by default; only an opted-in mcp__* tool skips approval. Built-ins unaffected; bridge-down is safe. - Regenerated dist/chat.c + dist/routes.c. Verified LIVE on :7770: chat, recall, /api/connectors all work. Spec: docs/research/mcp-connectors-adoption-spec.md. 🤖 Generated with Claude Code
tim.lingo added 1 commit 2026-06-13 23:44:11 +00:00
feat(soul): MCP connectors — /api/connectors proxy + per-connector auto-approve 6c57d4fe1b 
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>
tim.lingo commented 2026-06-14 20:50:02 +00:00
Author
Member
Copy Link
Copy Source

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.

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.
tim.lingo added 1 commit 2026-06-15 16:10:45 +00:00
feat(soul): add safety module, expand connectors API, memory-recall bug notes 3bb17a5296 
- safety.el/.elh: new safety module
- neuron-api.el, routes.el, soul.el, chat.el: connectors API expansion
- regenerated dist/ C artifacts
- MEMORY_RECALL_BUG.md: investigation notes

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
will.anderson closed this pull request 2026-06-15 16:38:14 +00:00
will.anderson deleted branch feat/connectors-soul 2026-06-15 16:38:18 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
BETA-CRITICAL
On the beta critical path — must resolve before ship
 blocks-public-beta
Must be fixed before any public beta
 HELD
Blocked pending an external decision (counsel / dual sign-off)
 ORTHOGONAL
Parallel workstream (testing/hardening) — safe to assign off critical path
 P0
Drop-everything severity
 POST-BETA
Important but not blocking the beta ship
 security
Security vulnerability
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
andre.almeida (Andre Almeida) tim.lingo (Tim Lingo) will.anderson (Will Anderson)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: neuron-technologies/neuron#14
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?