@go-to-k/cdkd 0.50.6 → 0.50.8

package/dist/cli.js

@@ -8705,10 +8705,10 @@ var IAMRoleProvider = class {
       const updateParams = {
         RoleName: physicalId
       };
-      if (properties["Description"]) {
+      if (properties["Description"] !== void 0) {
         updateParams.Description = properties["Description"];
       }
-      if (properties["MaxSessionDuration"]) {
+      if (properties["MaxSessionDuration"] !== void 0) {
         updateParams.MaxSessionDuration = properties["MaxSessionDuration"];
       }
       await this.iamClient.send(new UpdateRoleCommand(updateParams));
@@ -12641,6 +12641,14 @@ import {
 } from "@aws-sdk/client-sqs";
 import { GetCallerIdentityCommand as GetCallerIdentityCommand4 } from "@aws-sdk/client-sts";
 init_aws_clients();
+function serializeRedrivePolicy(value) {
+  if (value === null || value === void 0)
+    return "";
+  if (typeof value === "object" && Object.keys(value).length === 0) {
+    return "";
+  }
+  return JSON.stringify(value);
+}
 var CDK_TO_SQS_ATTRIBUTES = {
   VisibilityTimeout: "VisibilityTimeout",
   MaximumMessageSize: "MaximumMessageSize",
@@ -12699,7 +12707,7 @@ var SQSQueueProvider = class {
         if (properties[cdkKey] !== void 0) {
           const value = properties[cdkKey];
           if (cdkKey === "RedrivePolicy" && typeof value === "object") {
-            attributes[sqsKey] = JSON.stringify(value);
+            attributes[sqsKey] = serializeRedrivePolicy(value);
           } else {
             attributes[sqsKey] = String(value);
           }
@@ -12757,7 +12765,7 @@ var SQSQueueProvider = class {
         if (properties[cdkKey] !== void 0) {
           const value = properties[cdkKey];
           if (cdkKey === "RedrivePolicy" && typeof value === "object") {
-            attributes[sqsKey] = JSON.stringify(value);
+            attributes[sqsKey] = serializeRedrivePolicy(value);
           } else {
             attributes[sqsKey] = String(value);
           }
@@ -15469,18 +15477,36 @@ var LambdaUrlProvider = class {
   /**
    * Update a Lambda Function URL
    */
-  async update(logicalId, physicalId, _resourceType, properties, _previousProperties) {
+  async update(logicalId, physicalId, _resourceType, properties, previousProperties) {
     this.logger.debug(`Updating Lambda URL ${logicalId}: ${physicalId}`);
+    const handled = this.handledProperties.get("AWS::Lambda::Url") ?? /* @__PURE__ */ new Set();
+    let changed = false;
+    for (const key of handled) {
+      if (JSON.stringify(properties[key] ?? null) !== JSON.stringify(previousProperties[key] ?? null)) {
+        changed = true;
+        break;
+      }
+    }
+    if (!changed) {
+      return {
+        physicalId,
+        wasReplaced: false,
+        attributes: {}
+      };
+    }
     const authType = properties["AuthType"] || "NONE";
     const cors = properties["Cors"];
     const updateParams = {
       FunctionName: physicalId,
       AuthType: authType
     };
-    if (properties["InvokeMode"])
+    if (properties["InvokeMode"] !== void 0)
       updateParams.InvokeMode = properties["InvokeMode"];
     if (cors) {
-      updateParams.Cors = this.buildCorsConfig(cors);
+      const builtCors = this.buildCorsConfig(cors);
+      if (Object.keys(builtCors).length > 0) {
+        updateParams.Cors = builtCors;
+      }
     }
     const response = await this.lambdaClient.send(new UpdateFunctionUrlConfigCommand(updateParams));
     return {
@@ -15630,19 +15656,36 @@ var LambdaUrlProvider = class {
     return null;
   }
   /**
-   * Build CORS configuration from CDK properties
+   * Build CORS configuration from CDK properties.
+   *
+   * Empty arrays from `readCurrentState`'s always-emit placeholder
+   * (`AllowOrigins: []`, `AllowMethods: []`, `AllowHeaders: []`,
+   * `ExposeHeaders: []`) are intentionally dropped here — emitting them
+   * to AWS would configure CORS with empty allowlists instead of
+   * leaving CORS unset. The caller (`update()` / `create()`) treats an
+   * empty `Cors` object as "no CORS configured" and omits it from the
+   * SDK input. `MaxAge` uses `!== undefined` so the valid AWS input
+   * `MaxAge: 0` (= "do not cache preflight responses") is preserved.
    */
   buildCorsConfig(cors) {
     const config = {};
-    if (cors["AllowOrigins"])
-      config.AllowOrigins = cors["AllowOrigins"];
-    if (cors["AllowMethods"])
-      config.AllowMethods = cors["AllowMethods"];
-    if (cors["AllowHeaders"])
-      config.AllowHeaders = cors["AllowHeaders"];
-    if (cors["ExposeHeaders"])
-      config.ExposeHeaders = cors["ExposeHeaders"];
-    if (cors["MaxAge"])
+    const allowOrigins = cors["AllowOrigins"];
+    if (Array.isArray(allowOrigins) && allowOrigins.length > 0) {
+      config.AllowOrigins = allowOrigins;
+    }
+    const allowMethods = cors["AllowMethods"];
+    if (Array.isArray(allowMethods) && allowMethods.length > 0) {
+      config.AllowMethods = allowMethods;
+    }
+    const allowHeaders = cors["AllowHeaders"];
+    if (Array.isArray(allowHeaders) && allowHeaders.length > 0) {
+      config.AllowHeaders = allowHeaders;
+    }
+    const exposeHeaders = cors["ExposeHeaders"];
+    if (Array.isArray(exposeHeaders) && exposeHeaders.length > 0) {
+      config.ExposeHeaders = exposeHeaders;
+    }
+    if (cors["MaxAge"] !== void 0)
       config.MaxAge = cors["MaxAge"];
     if (cors["AllowCredentials"] !== void 0)
       config.AllowCredentials = cors["AllowCredentials"];
@@ -15661,6 +15704,43 @@ import {
   ResourceNotFoundException as ResourceNotFoundException4
 } from "@aws-sdk/client-lambda";
 init_aws_clients();
+function classifyEventSource(resp) {
+  if (resp.SelfManagedEventSource !== void 0)
+    return "kafka";
+  if (resp.SelfManagedKafkaEventSourceConfig !== void 0)
+    return "kafka";
+  if (resp.AmazonManagedKafkaEventSourceConfig !== void 0)
+    return "kafka";
+  if (resp.DocumentDBEventSourceConfig !== void 0)
+    return "documentdb";
+  const arn = resp.EventSourceArn;
+  if (!arn)
+    return "unknown";
+  if (arn.startsWith("arn:aws:sqs:") || arn.startsWith("arn:aws-cn:sqs:"))
+    return "sqs";
+  if (arn.startsWith("arn:aws:kinesis:") || arn.startsWith("arn:aws-cn:kinesis:"))
+    return "kinesis";
+  if (arn.startsWith("arn:aws:dynamodb:") || arn.startsWith("arn:aws-cn:dynamodb:"))
+    return "dynamodb";
+  if (arn.startsWith("arn:aws:kafka:") || arn.startsWith("arn:aws-cn:kafka:"))
+    return "kafka";
+  if (arn.startsWith("arn:aws:mq:") || arn.startsWith("arn:aws-cn:mq:"))
+    return "mq";
+  if (arn.startsWith("arn:aws:rds:") || arn.startsWith("arn:aws-cn:rds:")) {
+    return "documentdb";
+  }
+  return "unknown";
+}
+var KINDS_WITH_FUNCTION_RESPONSE_TYPES = /* @__PURE__ */ new Set([
+  "sqs",
+  "kinesis",
+  "dynamodb"
+]);
+var KINDS_WITH_SOURCE_ACCESS_CONFIGURATIONS = /* @__PURE__ */ new Set([
+  "kafka",
+  "mq",
+  "documentdb"
+]);
 var LambdaEventSourceMappingProvider = class {
   lambdaClient;
   logger = getLogger().child("LambdaEventSourceMappingProvider");
@@ -15787,33 +15867,33 @@ var LambdaEventSourceMappingProvider = class {
       UUID: physicalId,
       FunctionName: properties["FunctionName"]
     };
-    if (properties["BatchSize"])
+    if (properties["BatchSize"] !== void 0)
       updateParams.BatchSize = properties["BatchSize"];
     if (properties["Enabled"] !== void 0)
       updateParams.Enabled = properties["Enabled"];
-    if (properties["MaximumBatchingWindowInSeconds"])
+    if (properties["MaximumBatchingWindowInSeconds"] !== void 0)
       updateParams.MaximumBatchingWindowInSeconds = properties["MaximumBatchingWindowInSeconds"];
     if (properties["MaximumRetryAttempts"] !== void 0)
       updateParams.MaximumRetryAttempts = properties["MaximumRetryAttempts"];
     if (properties["BisectBatchOnFunctionError"] !== void 0)
       updateParams.BisectBatchOnFunctionError = properties["BisectBatchOnFunctionError"];
-    if (properties["MaximumRecordAgeInSeconds"])
+    if (properties["MaximumRecordAgeInSeconds"] !== void 0)
       updateParams.MaximumRecordAgeInSeconds = properties["MaximumRecordAgeInSeconds"];
-    if (properties["ParallelizationFactor"])
+    if (properties["ParallelizationFactor"] !== void 0)
       updateParams.ParallelizationFactor = properties["ParallelizationFactor"];
-    if (properties["FilterCriteria"])
+    if (properties["FilterCriteria"] !== void 0)
       updateParams.FilterCriteria = properties["FilterCriteria"];
-    if (properties["DestinationConfig"])
+    if (properties["DestinationConfig"] !== void 0)
       updateParams.DestinationConfig = properties["DestinationConfig"];
-    if (properties["TumblingWindowInSeconds"])
+    if (properties["TumblingWindowInSeconds"] !== void 0)
       updateParams.TumblingWindowInSeconds = properties["TumblingWindowInSeconds"];
-    if (properties["FunctionResponseTypes"])
+    if (properties["FunctionResponseTypes"] !== void 0)
       updateParams.FunctionResponseTypes = properties["FunctionResponseTypes"];
-    if (properties["SourceAccessConfigurations"])
+    if (properties["SourceAccessConfigurations"] !== void 0)
       updateParams.SourceAccessConfigurations = properties["SourceAccessConfigurations"];
-    if (properties["ScalingConfig"])
+    if (properties["ScalingConfig"] !== void 0)
       updateParams.ScalingConfig = properties["ScalingConfig"];
-    if (properties["DocumentDBEventSourceConfig"])
+    if (properties["DocumentDBEventSourceConfig"] !== void 0)
       updateParams.DocumentDBEventSourceConfig = properties["DocumentDBEventSourceConfig"];
     const updateResp = await this.lambdaClient.send(
       new UpdateEventSourceMappingCommand(updateParams)
@@ -15987,8 +16067,17 @@ var LambdaEventSourceMappingProvider = class {
     if (resp.TumblingWindowInSeconds !== void 0) {
       result["TumblingWindowInSeconds"] = resp.TumblingWindowInSeconds;
     }
-    result["FunctionResponseTypes"] = resp.FunctionResponseTypes ? [...resp.FunctionResponseTypes] : [];
-    result["SourceAccessConfigurations"] = resp.SourceAccessConfigurations ?? [];
+    const kind = classifyEventSource(resp);
+    if (KINDS_WITH_FUNCTION_RESPONSE_TYPES.has(kind)) {
+      result["FunctionResponseTypes"] = resp.FunctionResponseTypes ? [...resp.FunctionResponseTypes] : [];
+    } else if (resp.FunctionResponseTypes !== void 0) {
+      result["FunctionResponseTypes"] = [...resp.FunctionResponseTypes];
+    }
+    if (KINDS_WITH_SOURCE_ACCESS_CONFIGURATIONS.has(kind)) {
+      result["SourceAccessConfigurations"] = resp.SourceAccessConfigurations ?? [];
+    } else if (resp.SourceAccessConfigurations !== void 0) {
+      result["SourceAccessConfigurations"] = resp.SourceAccessConfigurations;
+    }
     if (resp.SelfManagedEventSource !== void 0) {
       result["SelfManagedEventSource"] = resp.SelfManagedEventSource;
     }
@@ -16116,19 +16205,35 @@ var LambdaLayerVersionProvider = class {
     }
   }
   /**
-   * Update a Lambda layer version
+   * Update a Lambda layer version.
    *
-   * Lambda layer versions are immutable. An update publishes a new version.
-   * The new LayerVersionArn becomes the physical ID.
+   * Lambda layer versions are immutable on AWS — there is no API to mutate
+   * `Content` / `CompatibleRuntimes` / `CompatibleArchitectures` /
+   * `Description` / `LicenseInfo` of an existing version. The only path to
+   * a "new value" is publishing a new version (new LayerVersionArn).
+   *
+   * Why this rejects with `ResourceUpdateNotSupportedError` instead of
+   * silently publishing a new version:
+   *
+   *   - `cdkd drift --revert` calls `update(observed, observed)` to push
+   *     state values back into AWS. For an immutable resource that cannot
+   *     have its in-place value changed, the only AWS-side effect of an
+   *     "update" is leaking a duplicate version of the same content,
+   *     which is never what `--revert` should do.
+   *   - On the deploy path, content / runtime / arch changes flow
+   *     through CDK's hash-based logical naming, which produces a fresh
+   *     logical ID and a CREATE+DELETE in cdkd's diff — `update()` is
+   *     not the path taken in practice. Users who hit this on a non-CDK
+   *     template should re-deploy with `--replace`.
    */
-  async update(logicalId, physicalId, resourceType, properties, _previousProperties) {
-    this.logger.debug(`Updating Lambda layer version ${logicalId}: ${physicalId}`);
-    const createResult = await this.create(logicalId, resourceType, properties);
-    return {
-      physicalId: createResult.physicalId,
-      wasReplaced: true,
-      attributes: createResult.attributes ?? {}
-    };
+  async update(logicalId, _physicalId, resourceType, _properties, _previousProperties) {
+    return Promise.reject(
+      new ResourceUpdateNotSupportedError(
+        resourceType,
+        logicalId,
+        "Lambda layer versions are immutable on AWS; re-deploy with cdkd deploy --replace, or change the resource definition to publish a new version"
+      )
+    );
   }
   /**
    * Delete a Lambda layer version
@@ -39824,6 +39929,19 @@ async function runAccept(reports, stateBackend, stateConfig, awsClients, options
     }
   }
 }
+function buildRevertNewProperties(drifts, desiredProperties, awsProperties) {
+  const result = { ...awsProperties };
+  for (const d of drifts) {
+    const topLevelKey = d.path.split(".", 1)[0];
+    if (!topLevelKey)
+      continue;
+    if (topLevelKey in desiredProperties) {
+      result[topLevelKey] = desiredProperties[topLevelKey];
+    } else {
+    }
+  }
+  return result;
+}
 async function runRevert(reports, providerRegistry, stateConfig, awsClients, options) {
   const logger = getLogger();
   printRevertPlan(reports);
@@ -39866,13 +39984,18 @@ async function runRevert(reports, providerRegistry, stateConfig, awsClients, opt
         }
         const provider = providerRegistry.getProvider(outcome.resourceType);
         const desiredProperties = stateResource.observedProperties ?? stateResource.properties ?? {};
+        const newProperties = buildRevertNewProperties(
+          outcome.changes,
+          desiredProperties,
+          outcome.awsProperties
+        );
         try {
           await withRetry(
             () => provider.update(
               outcome.logicalId,
               stateResource.physicalId,
               outcome.resourceType,
-              desiredProperties,
+              newProperties,
               outcome.awsProperties
             ),
             outcome.logicalId,
@@ -43505,7 +43628,7 @@ function reorderArgs(argv) {
 }
 async function main() {
   const program = new Command14();
-  program.name("cdkd").description("CDK Direct - Deploy AWS CDK apps directly via SDK/Cloud Control API").version("0.50.6");
+  program.name("cdkd").description("CDK Direct - Deploy AWS CDK apps directly via SDK/Cloud Control API").version("0.50.8");
   program.addCommand(createBootstrapCommand());
   program.addCommand(createSynthCommand());
   program.addCommand(createListCommand());