@codemieai/cdk 0.1.626 → 0.1.628

package/README.md

@@ -321,6 +321,11 @@ npm run deploy    # Deploy (updates state file with current checksums)
 - This happens when assistant doesn't exist in backup (e.g., deleted or from another project)
 - Manually replace with correct assistant name or ID in workflow YAML file
 
+**All MCP Assistants Show as Changed After an Upgrade?**
+- After upgrading to the release containing EPMCDME-14873, MCP server configuration is normalized before hashing, so every assistant with `mcp_servers` gets a new `configChecksum` once
+- `preview` and `deploy` list each of them under `⚠️  UPDATE:` as `- assistant:<name>` with `Prompt or configuration changed` and no per-field diff
+- Expected, not a configuration error - `deploy` stores the new `configChecksum` in `.codemie/state.json` and the next run reports them as unchanged
+
 **Resume From Specific Point?**
 - Transaction file tracks what's completed: `transaction.json`
 - Contains: `completed: ['ast-1', 'ast-2']` and `failed: [{id: 'ast-3', error: '...'}]`
@@ -792,6 +797,12 @@ Changes to prompts and configuration are detected via SHA256 checksums, ensuring
 - On deployment: Compare stored checksums with current checksums
 - Only update if either checksum differs
 
+**One-time checksum change for assistants with MCP servers**
+
+After upgrading to the release that contains EPMCDME-14873, MCP server configuration is normalized before it is hashed: keys with no value (`null`) and an empty top-level `mcp_connect_url` no longer contribute to the hash, and the remaining keys are hashed in a fixed order at every depth, so the order the platform or your YAML uses for MCP fields never affects the checksum. Every assistant that has `mcp_servers` therefore gets a new `configChecksum` once - and only once.
+
+The first `npm run preview` (or `codemie preview`) and `npm run deploy` after the upgrade list each such assistant under `⚠️  UPDATE:` as `- assistant:<name>` followed by `Prompt or configuration changed`, with no per-field diff. This is expected and is not a configuration error - nothing in your `codemie.yaml` changed. `deploy` writes the new `configChecksum` back to `.codemie/state.json`, and the second run reports the same assistants as unchanged.
+
 ### Orphaned Resource Cleanup
 
 Resources removed from `codemie.yaml` are NOT automatically deleted by default. To remove them, you must explicitly use the `--prune` flag.

package/dist/cli/index.js

@@ -243,6 +243,34 @@ var init_logger = __esm({
     logger = Logger.getInstance();
   }
 });
+
+// src/lib/mcpSanitizer.ts
+function isPlainObject(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function sanitizeObject(obj, isTopLevel) {
+  const result = {};
+  for (const [key, value] of Object.entries(obj)) {
+    if (value === null || value === void 0) {
+      continue;
+    }
+    if (isTopLevel && key === "mcp_connect_url" && value === "") {
+      continue;
+    }
+    result[key] = isPlainObject(value) ? sanitizeObject(value, false) : value;
+  }
+  return result;
+}
+function sanitizeMcpServer(server) {
+  if (!isPlainObject(server)) {
+    return server;
+  }
+  return sanitizeObject(server, true);
+}
+var init_mcpSanitizer = __esm({
+  "src/lib/mcpSanitizer.ts"() {
+  }
+});
 function normalizeIntegrationSettings(settings) {
   if (!settings || typeof settings !== "object") {
     return settings;
@@ -277,15 +305,34 @@ function normalizeToolkits(toolkits) {
     settings: normalizeIntegrationSettings(settings)
   }));
 }
+function isPlainObject2(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function sortKeysDeep(value) {
+  if (!isPlainObject2(value)) {
+    return value;
+  }
+  const sorted = {};
+  for (const key of Object.keys(value).sort()) {
+    sorted[key] = sortKeysDeep(value[key]);
+  }
+  return sorted;
+}
 function normalizeMcpServers(mcpServers) {
   if (!mcpServers) {
     return [];
   }
-  return mcpServers.map((mcp) => ({
-    ...mcp,
-    settings: normalizeIntegrationSettings(mcp.settings),
-    mcp_connect_auth_token: normalizeIntegrationSettings(mcp.mcp_connect_auth_token)
-  }));
+  return mcpServers.map((mcp) => {
+    const sanitized = sanitizeMcpServer(mcp);
+    if (!isPlainObject2(sanitized)) {
+      return sanitized;
+    }
+    return sortKeysDeep({
+      ...sanitized,
+      settings: normalizeIntegrationSettings(sanitized.settings),
+      mcp_connect_auth_token: normalizeIntegrationSettings(sanitized.mcp_connect_auth_token)
+    });
+  });
 }
 function calculateChecksum(content) {
   if (typeof content !== "string") {
@@ -324,6 +371,7 @@ var DEFAULT_LLM_MODEL;
 var init_checksumUtils = __esm({
   "src/lib/checksumUtils.ts"() {
     init_logger();
+    init_mcpSanitizer();
     DEFAULT_LLM_MODEL = "gpt-4";
   }
 });
@@ -432,10 +480,11 @@ function isValidMcpServer(server) {
   const hasConfigUrl = s.config && typeof s.config === "object" && typeof s.config.url === "string" && s.config.url.length > 0;
   const hasUrl = hasTopLevelUrl || hasConfigUrl;
   const hasSettings = s.settings != null;
+  const hasMcpConfigId = typeof s.mcp_config_id === "string" && s.mcp_config_id.length > 0;
   if (hasCommand && hasUrl) {
     return false;
   }
-  if (!hasCommand && !hasUrl && !hasSettings) {
+  if (!hasCommand && !hasUrl && !hasSettings && !hasMcpConfigId) {
     return false;
   }
   if (hasCommand && !validateMcpCommandArgs(s, hasTopLevelCommand)) {
@@ -457,7 +506,7 @@ function convertMcpServers(servers) {
   if (validServers.length === 0) {
     return void 0;
   }
-  return validServers;
+  return validServers.map((server) => sanitizeMcpServer(server));
 }
 function assistantResponseToResource(assistant) {
   const safeName = assistant.name || assistant.id || "assistant";
@@ -645,11 +694,12 @@ function iacToolkitToSdk(toolkit) {
   };
 }
 function iacMcpServerToSdk(mcp) {
+  const sanitized = sanitizeMcpServer(mcp);
   return {
-    ...mcp,
-    enabled: mcp.enabled ?? true,
-    settings: isResolvedIntegration(mcp.settings) ? mcp.settings : void 0,
-    mcp_connect_auth_token: isResolvedIntegration(mcp.mcp_connect_auth_token) ? mcp.mcp_connect_auth_token : void 0
+    ...sanitized,
+    enabled: sanitized.enabled ?? true,
+    settings: isResolvedIntegration(sanitized.settings) ? sanitized.settings : void 0,
+    mcp_connect_auth_token: isResolvedIntegration(sanitized.mcp_connect_auth_token) ? sanitized.mcp_connect_auth_token : void 0
   };
 }
 function assistantResourceToCreateParams(assistant, projectName, promptContent) {
@@ -685,6 +735,7 @@ var init_converters = __esm({
   "src/lib/converters.ts"() {
     init_checksumUtils();
     init_logger();
+    init_mcpSanitizer();
     init_typeGuards();
     CHUNK_SUMMARY_ALIASES = /* @__PURE__ */ new Set(["chunk_summary", "chunk-summary"]);
     FILE_SUMMARY_ALIASES = /* @__PURE__ */ new Set(["file_summary", "file-summary", "file-summay"]);
@@ -808,17 +859,7 @@ function transformTool(tool, integrationIdToAlias, integrationSpecPaths) {
   return result;
 }
 function transformMcpServer(mcp, integrationIdToAlias) {
-  const result = {
-    name: mcp.name,
-    description: mcp.description,
-    enabled: mcp.enabled,
-    command: mcp.command,
-    arguments: mcp.arguments,
-    config: mcp.config,
-    mcp_connect_url: mcp.mcp_connect_url,
-    mcp_connect_auth_token: mcp.mcp_connect_auth_token,
-    tools_tokens_size_limit: mcp.tools_tokens_size_limit
-  };
+  const result = { ...mcp };
   if (mcp.settings && isResolvedIntegration(mcp.settings)) {
     const alias = mcp.settings.alias || integrationIdToAlias.get(mcp.settings.id);
     if (alias) {
@@ -840,7 +881,7 @@ function transformMcpServer(mcp, integrationIdToAlias) {
       );
     }
   }
-  return result;
+  return sanitizeMcpServer(result);
 }
 function transformToolkits(toolkits, integrationIdToAlias, integrationSpecPaths) {
   return toolkits?.map(({ toolkit, tools, label, settings_config, is_external, settings }) => ({
@@ -974,6 +1015,7 @@ var init_backupTransformers = __esm({
     init_converters();
     init_fileUtils();
     init_logger();
+    init_mcpSanitizer();
     init_skillUtils();
     init_typeGuards();
   }
@@ -981,7 +1023,7 @@ var init_backupTransformers = __esm({
 
 // package.json
 var package_default = {
-  version: "0.1.626"};
+  version: "0.1.628"};
 var appConfigSchema = z.object({
   rootDir: z.string(),
   codemieConfig: z.string(),

package/dist/index.js

@@ -181,6 +181,34 @@ var init_logger = __esm({
     logger = Logger.getInstance();
   }
 });
+
+// src/lib/mcpSanitizer.ts
+function isPlainObject(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function sanitizeObject(obj, isTopLevel) {
+  const result = {};
+  for (const [key, value] of Object.entries(obj)) {
+    if (value === null || value === void 0) {
+      continue;
+    }
+    if (isTopLevel && key === "mcp_connect_url" && value === "") {
+      continue;
+    }
+    result[key] = isPlainObject(value) ? sanitizeObject(value, false) : value;
+  }
+  return result;
+}
+function sanitizeMcpServer(server) {
+  if (!isPlainObject(server)) {
+    return server;
+  }
+  return sanitizeObject(server, true);
+}
+var init_mcpSanitizer = __esm({
+  "src/lib/mcpSanitizer.ts"() {
+  }
+});
 function normalizeIntegrationSettings(settings) {
   if (!settings || typeof settings !== "object") {
     return settings;
@@ -215,15 +243,34 @@ function normalizeToolkits(toolkits) {
     settings: normalizeIntegrationSettings(settings)
   }));
 }
+function isPlainObject2(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function sortKeysDeep(value) {
+  if (!isPlainObject2(value)) {
+    return value;
+  }
+  const sorted = {};
+  for (const key of Object.keys(value).sort()) {
+    sorted[key] = sortKeysDeep(value[key]);
+  }
+  return sorted;
+}
 function normalizeMcpServers(mcpServers) {
   if (!mcpServers) {
     return [];
   }
-  return mcpServers.map((mcp) => ({
-    ...mcp,
-    settings: normalizeIntegrationSettings(mcp.settings),
-    mcp_connect_auth_token: normalizeIntegrationSettings(mcp.mcp_connect_auth_token)
-  }));
+  return mcpServers.map((mcp) => {
+    const sanitized = sanitizeMcpServer(mcp);
+    if (!isPlainObject2(sanitized)) {
+      return sanitized;
+    }
+    return sortKeysDeep({
+      ...sanitized,
+      settings: normalizeIntegrationSettings(sanitized.settings),
+      mcp_connect_auth_token: normalizeIntegrationSettings(sanitized.mcp_connect_auth_token)
+    });
+  });
 }
 function calculateChecksum(content) {
   if (typeof content !== "string") {
@@ -262,6 +309,7 @@ var DEFAULT_LLM_MODEL;
 var init_checksumUtils = __esm({
   "src/lib/checksumUtils.ts"() {
     init_logger();
+    init_mcpSanitizer();
     DEFAULT_LLM_MODEL = "gpt-4";
   }
 });
@@ -370,10 +418,11 @@ function isValidMcpServer(server) {
   const hasConfigUrl = s.config && typeof s.config === "object" && typeof s.config.url === "string" && s.config.url.length > 0;
   const hasUrl = hasTopLevelUrl || hasConfigUrl;
   const hasSettings = s.settings != null;
+  const hasMcpConfigId = typeof s.mcp_config_id === "string" && s.mcp_config_id.length > 0;
   if (hasCommand && hasUrl) {
     return false;
   }
-  if (!hasCommand && !hasUrl && !hasSettings) {
+  if (!hasCommand && !hasUrl && !hasSettings && !hasMcpConfigId) {
     return false;
   }
   if (hasCommand && !validateMcpCommandArgs(s, hasTopLevelCommand)) {
@@ -395,7 +444,7 @@ function convertMcpServers(servers) {
   if (validServers.length === 0) {
     return void 0;
   }
-  return validServers;
+  return validServers.map((server) => sanitizeMcpServer(server));
 }
 function assistantResponseToResource(assistant) {
   const safeName = assistant.name || assistant.id || "assistant";
@@ -583,11 +632,12 @@ function iacToolkitToSdk(toolkit) {
   };
 }
 function iacMcpServerToSdk(mcp) {
+  const sanitized = sanitizeMcpServer(mcp);
   return {
-    ...mcp,
-    enabled: mcp.enabled ?? true,
-    settings: isResolvedIntegration(mcp.settings) ? mcp.settings : void 0,
-    mcp_connect_auth_token: isResolvedIntegration(mcp.mcp_connect_auth_token) ? mcp.mcp_connect_auth_token : void 0
+    ...sanitized,
+    enabled: sanitized.enabled ?? true,
+    settings: isResolvedIntegration(sanitized.settings) ? sanitized.settings : void 0,
+    mcp_connect_auth_token: isResolvedIntegration(sanitized.mcp_connect_auth_token) ? sanitized.mcp_connect_auth_token : void 0
   };
 }
 function assistantResourceToCreateParams(assistant, projectName, promptContent) {
@@ -623,6 +673,7 @@ var init_converters = __esm({
   "src/lib/converters.ts"() {
     init_checksumUtils();
     init_logger();
+    init_mcpSanitizer();
     init_typeGuards();
     CHUNK_SUMMARY_ALIASES = /* @__PURE__ */ new Set(["chunk_summary", "chunk-summary"]);
     FILE_SUMMARY_ALIASES = /* @__PURE__ */ new Set(["file_summary", "file-summary", "file-summay"]);
@@ -746,17 +797,7 @@ function transformTool(tool, integrationIdToAlias, integrationSpecPaths) {
   return result;
 }
 function transformMcpServer(mcp, integrationIdToAlias) {
-  const result = {
-    name: mcp.name,
-    description: mcp.description,
-    enabled: mcp.enabled,
-    command: mcp.command,
-    arguments: mcp.arguments,
-    config: mcp.config,
-    mcp_connect_url: mcp.mcp_connect_url,
-    mcp_connect_auth_token: mcp.mcp_connect_auth_token,
-    tools_tokens_size_limit: mcp.tools_tokens_size_limit
-  };
+  const result = { ...mcp };
   if (mcp.settings && isResolvedIntegration(mcp.settings)) {
     const alias = mcp.settings.alias || integrationIdToAlias.get(mcp.settings.id);
     if (alias) {
@@ -778,7 +819,7 @@ function transformMcpServer(mcp, integrationIdToAlias) {
       );
     }
   }
-  return result;
+  return sanitizeMcpServer(result);
 }
 function transformToolkits(toolkits, integrationIdToAlias, integrationSpecPaths) {
   return toolkits?.map(({ toolkit, tools, label, settings_config, is_external, settings }) => ({
@@ -912,6 +953,7 @@ var init_backupTransformers = __esm({
     init_converters();
     init_fileUtils();
     init_logger();
+    init_mcpSanitizer();
     init_skillUtils();
     init_typeGuards();
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codemieai/cdk",
-  "version": "0.1.626",
+  "version": "0.1.628",
   "type": "module",
   "description": "Infrastructure as Code solution for managing Codemie AI assistants, datasources, and workflows through declarative YAML configuration",
   "main": "./dist/index.js",
@@ -41,7 +41,7 @@
   },
   "dependencies": {
     "@commander-js/extra-typings": "^14.0.0",
-    "codemie-sdk": "0.1.440",
+    "codemie-sdk": "0.1.626",
     "dotenv": "^17.2.3",
     "ora": "^8.2.0",
     "p-limit": "^7.2.0",