clementine-agent 1.18.198 → 1.18.199
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/gateway/router.js +28 -4
- package/package.json +1 -1
package/dist/gateway/router.js
CHANGED
|
@@ -2431,14 +2431,38 @@ export class Gateway {
|
|
|
2431
2431
|
});
|
|
2432
2432
|
// Wire Composio + external MCP only for real chat. Builder
|
|
2433
2433
|
// skips entirely — builder turns never call tools.
|
|
2434
|
+
//
|
|
2435
|
+
// 1.18.199 — `fullSurface: true` (NOT per-turn scopeText routing).
|
|
2436
|
+
//
|
|
2437
|
+
// The previous per-message routing was a regression: each chat
|
|
2438
|
+
// turn called `routeToolSurface(originalText)` and only loaded
|
|
2439
|
+
// MCP servers that matched THAT message's text. When the owner
|
|
2440
|
+
// sent a one-word reply like "sure" mid-conversation, zero MCPs
|
|
2441
|
+
// matched → zero servers registered → Ross said "DataForSEO
|
|
2442
|
+
// isn't in scope for this session context" mid-task. Verified
|
|
2443
|
+
// live on 2026-05-12 at 13:35:33.
|
|
2444
|
+
//
|
|
2445
|
+
// The right structural fix: chat always loads the agent's
|
|
2446
|
+
// full profile-allowed MCP surface. The profile's
|
|
2447
|
+
// `allowedComposioToolkits` / `allowedMcpServers` still bound
|
|
2448
|
+
// what each agent can see (Ross gets his scope, Sasha gets
|
|
2449
|
+
// hers). Within those bounds, every server registers every
|
|
2450
|
+
// turn — tools don't vanish mid-conversation. The SDK's
|
|
2451
|
+
// `ENABLE_TOOL_SEARCH=auto:5` (execution-policy.ts) still
|
|
2452
|
+
// defers individual tool-schema loading until needed, so the
|
|
2453
|
+
// per-turn token cost stays bounded even with many servers
|
|
2454
|
+
// registered.
|
|
2455
|
+
//
|
|
2456
|
+
// The skill-resolver still runs above this point — it
|
|
2457
|
+
// contributes the SKILL PROMPT BLOCK to system context. It
|
|
2458
|
+
// simply no longer affects MCP server registration. That
|
|
2459
|
+
// experiment (1.18.170 `skillHintedMcpServers`) caused more
|
|
2460
|
+
// problems than it solved.
|
|
2434
2461
|
const chatMcp = isBuilderSession
|
|
2435
2462
|
? null
|
|
2436
2463
|
: await buildExtraMcpForRunAgent({
|
|
2437
|
-
|
|
2464
|
+
fullSurface: true,
|
|
2438
2465
|
profile: resolvedProfile,
|
|
2439
|
-
...(resolvedSkills && resolvedSkills.hintedMcpServers.length > 0
|
|
2440
|
-
? { skillHintedMcpServers: resolvedSkills.hintedMcpServers }
|
|
2441
|
-
: {}),
|
|
2442
2466
|
});
|
|
2443
2467
|
// Vault context (SOUL.md / MEMORY.md / AGENTS.md + optional
|
|
2444
2468
|
// profile body) — real chat only. Builder gets just its own
|