impel-cli 0.20.0 → 0.20.2-beta.0

package/src/selfInvocation.js

@@ -66,6 +66,7 @@ export function impelCliInvocation(args = [], options = {}) {
 
 export const IMPEL_MANAGED_MCP_ENV = brandedEnvironmentName("MANAGED_MCP");
 export const IMPEL_TASKS_MCP_SERVER_NAME = `${RUNTIME_BRAND.cli.providerId}-tasks`;
+export const IMPEL_NATIVE_AGENT_MCP_TARGET = "native-agent";
 
 export function impelMcpInvocation(args = [], options = {}) {
   return {
@@ -87,11 +88,42 @@ export function impelTasksMcpInvocation(tenantId, options = {}) {
   return impelMcpInvocation(["--target", "tasks", "--tenant", tenantId], options);
 }
 
+/**
+ * Build the fixed local composite transport used by one generated native-agent
+ * definition. All routing values are process arguments written by impel-cli;
+ * the model-facing tools consequently have no agent, scope, or tenant selector.
+ */
+export function impelNativeAgentMcpInvocation({
+  tenantId,
+  agentId,
+  scopeParam,
+  policyFingerprint,
+  recoveryOnly = false,
+}, options = {}) {
+  for (const [field, value] of Object.entries({ tenantId, agentId, scopeParam, policyFingerprint })) {
+    if (typeof value !== "string" || !value.trim()) {
+      throw new Error(`${field} is required for the managed native-agent MCP invocation`);
+    }
+  }
+  return impelMcpInvocation([
+    "--target", IMPEL_NATIVE_AGENT_MCP_TARGET,
+    "--tenant", tenantId,
+    "--agent-id", agentId,
+    "--scope-param", scopeParam,
+    "--policy-fingerprint", policyFingerprint,
+    ...(recoveryOnly ? ["--recovery-only"] : []),
+  ], options);
+}
+
 /** Recognize only the marker-bearing Tasks invocation that impel-cli owns. */
 export function isImpelTasksMcpInvocation(value) {
   if (!value || typeof value !== "object" || !Array.isArray(value.args)) return false;
   const suffix = value.args.slice(-5);
-  return value.type === "stdio"
+  // Claude Desktop persists stdio servers without the optional `type` field.
+  // Accept that vendor-normalized form so the next Impel update can still
+  // recognize, re-pin, and repair its own tenant-bound entry. Any explicit
+  // non-stdio type remains foreign and fails closed.
+  return (value.type === undefined || value.type === "stdio")
     && value.env?.[IMPEL_MANAGED_MCP_ENV] === "1"
     && suffix[0] === "mcp"
     && suffix[1] === "--target"