@go-to-k/cdkd 0.57.0 → 0.57.1

package/dist/cli.js

@@ -13565,7 +13565,7 @@ var SNSTopicProvider = class {
       if (properties["DeliveryStatusLogging"]) {
         const loggingConfigs = properties["DeliveryStatusLogging"];
         for (const config of loggingConfigs) {
-          const protocol = config["Protocol"];
+          const protocol = normalizeDeliveryStatusProtocolOrThrow(config["Protocol"], logicalId);
           if (config["SuccessFeedbackRoleArn"]) {
             await this.snsClient.send(
               new SetTopicAttributesCommand({
@@ -13658,7 +13658,7 @@ var SNSTopicProvider = class {
     if (JSON.stringify(properties["DeliveryStatusLogging"]) !== JSON.stringify(previousProperties["DeliveryStatusLogging"])) {
       const loggingConfigs = properties["DeliveryStatusLogging"] || [];
       for (const config of loggingConfigs) {
-        const protocol = config["Protocol"];
+        const protocol = normalizeDeliveryStatusProtocolOrThrow(config["Protocol"], logicalId);
         if (config["SuccessFeedbackRoleArn"]) {
           await this.snsClient.send(
             new SetTopicAttributesCommand({
@@ -13798,16 +13798,22 @@ var SNSTopicProvider = class {
    * FailureFeedbackRoleArn?}]`. Walks the known protocol prefix list
    * (`HTTP` / `HTTPS` / `SQS` / `Lambda` / `Firehose` / `Application`); a
    * protocol is included in the result iff at least one of its three
-   * sub-attributes is set on the topic. Entries are sorted by `Protocol`
-   * for stable positional compare (AWS does not preserve template order
-   * across `GetTopicAttributes` calls).
+   * sub-attributes is set on the topic. Entries are sorted by canonical
+   * PascalCase `Protocol` for stable positional compare (AWS does not
+   * preserve template order across `GetTopicAttributes` calls).
+   *
+   * The emitted `Protocol` value preserves state's case when known
+   * (CDK templates emit lowercase `'lambda'` / `'sqs'` / ...; AWS's
+   * attribute prefix is PascalCase). Without case preservation the
+   * comparator would fire false drift on every clean run for any
+   * lowercase-`Protocol` template.
    *
    * `Subscription` is omitted because CDK manages it via separate
    * `AWS::SNS::Subscription` resources, not as a Topic property.
    *
    * Returns `undefined` when the topic is gone (`NotFoundException`).
    */
-  async readCurrentState(physicalId, _logicalId, _resourceType) {
+  async readCurrentState(physicalId, _logicalId, _resourceType, properties) {
     let attrs;
     try {
       const resp = await this.snsClient.send(
@@ -13849,7 +13855,10 @@ var SNSTopicProvider = class {
         }
       }
     }
-    result["DeliveryStatusLogging"] = mapDeliveryStatusLogging(attrs);
+    result["DeliveryStatusLogging"] = mapDeliveryStatusLogging(
+      attrs,
+      stateProtocolCaseMap(properties?.["DeliveryStatusLogging"])
+    );
     try {
       const tagsResp = await this.snsClient.send(
         new ListTagsForResourceCommand({ ResourceArn: physicalId })
@@ -13938,7 +13947,55 @@ var SNS_DELIVERY_STATUS_PROTOCOLS = [
   "Lambda",
   "SQS"
 ];
-function mapDeliveryStatusLogging(attrs) {
+function normalizeDeliveryStatusProtocol(input) {
+  if (typeof input !== "string")
+    return void 0;
+  const lower = input.toLowerCase();
+  switch (lower) {
+    case "application":
+      return "Application";
+    case "firehose":
+      return "Firehose";
+    case "http":
+      return "HTTP";
+    case "https":
+      return "HTTPS";
+    case "lambda":
+      return "Lambda";
+    case "sqs":
+      return "SQS";
+    default:
+      return void 0;
+  }
+}
+function normalizeDeliveryStatusProtocolOrThrow(input, logicalId) {
+  const normalized = normalizeDeliveryStatusProtocol(input);
+  if (normalized === void 0) {
+    throw new Error(
+      `SNS topic ${logicalId}: unsupported DeliveryStatusLogging protocol ${JSON.stringify(input)}. Expected one of ${SNS_DELIVERY_STATUS_PROTOCOLS.join(", ")} (case-insensitive).`
+    );
+  }
+  return normalized;
+}
+function stateProtocolCaseMap(stateLogging) {
+  const map = /* @__PURE__ */ new Map();
+  if (!Array.isArray(stateLogging))
+    return map;
+  for (const entry of stateLogging) {
+    if (!entry || typeof entry !== "object")
+      continue;
+    const raw = entry["Protocol"];
+    if (typeof raw !== "string")
+      continue;
+    const normalized = normalizeDeliveryStatusProtocol(raw);
+    if (!normalized)
+      continue;
+    if (!map.has(normalized))
+      map.set(normalized, raw);
+  }
+  return map;
+}
+function mapDeliveryStatusLogging(attrs, stateCaseMap = /* @__PURE__ */ new Map()) {
   const result = [];
   for (const protocol of SNS_DELIVERY_STATUS_PROTOCOLS) {
     const success = attrs[`${protocol}SuccessFeedbackRoleArn`];
@@ -13946,7 +14003,9 @@ function mapDeliveryStatusLogging(attrs) {
     const failure = attrs[`${protocol}FailureFeedbackRoleArn`];
     if (success === void 0 && sample === void 0 && failure === void 0)
       continue;
-    const entry = { Protocol: protocol };
+    const entry = {
+      Protocol: stateCaseMap.get(protocol) ?? protocol
+    };
     if (success !== void 0)
       entry["SuccessFeedbackRoleArn"] = success;
     if (sample !== void 0)
@@ -27595,7 +27654,18 @@ var ECSProvider = class {
     let resp;
     try {
       resp = await this.getClient().send(
-        new DescribeClustersCommand({ clusters: [physicalId], include: ["TAGS"] })
+        // AWS DescribeClusters omits `settings` / `configuration` from the
+        // response unless they are explicitly requested via `include`. Without
+        // SETTINGS / CONFIGURATIONS the readCurrentState round-trip silently
+        // surfaces empty `ClusterSettings: []` even when the cluster has
+        // containerInsights enabled — a console-side toggle then can't be
+        // detected as drift because both the deploy-time observedProperties
+        // baseline AND the drift-time AWS read would identically miss the
+        // field. Discovered by the drift-revert integ test (PR #201).
+        new DescribeClustersCommand({
+          clusters: [physicalId],
+          include: ["TAGS", "SETTINGS", "CONFIGURATIONS"]
+        })
       );
     } catch {
       return void 0;
@@ -32342,7 +32412,10 @@ var ServiceDiscoveryProvider = class {
   providerRegion = process.env["AWS_REGION"];
   logger = getLogger().child("ServiceDiscoveryProvider");
   handledProperties = /* @__PURE__ */ new Map([
-    ["AWS::ServiceDiscovery::PrivateDnsNamespace", /* @__PURE__ */ new Set(["Name", "Vpc", "Description", "Tags"])],
+    [
+      "AWS::ServiceDiscovery::PrivateDnsNamespace",
+      /* @__PURE__ */ new Set(["Name", "Vpc", "Description", "Tags", "Properties"])
+    ],
     [
       "AWS::ServiceDiscovery::Service",
       /* @__PURE__ */ new Set([
@@ -32444,13 +32517,18 @@ var ServiceDiscoveryProvider = class {
         logicalId
       );
     }
+    const propsBag = properties["Properties"];
+    const dnsProps = propsBag?.["DnsProperties"];
+    const soa = dnsProps?.["SOA"];
+    const inputProperties = soa?.TTL !== void 0 ? { DnsProperties: { SOA: { TTL: Number(soa.TTL) } } } : void 0;
     try {
       const response = await client.send(
         new CreatePrivateDnsNamespaceCommand({
           Name: name,
           Vpc: vpc,
           ...description && { Description: description },
-          ...tags && tags.length > 0 && { Tags: tags }
+          ...tags && tags.length > 0 && { Tags: tags },
+          ...inputProperties && { Properties: inputProperties }
         })
       );
       const operationId = response.OperationId;
@@ -32840,6 +32918,12 @@ var ServiceDiscoveryProvider = class {
     if (ns.Name !== void 0)
       result["Name"] = ns.Name;
     result["Description"] = ns.Description ?? "";
+    const soa = ns.Properties?.DnsProperties?.SOA;
+    if (soa?.TTL !== void 0) {
+      result["Properties"] = { DnsProperties: { SOA: { TTL: soa.TTL } } };
+    } else {
+      result["Properties"] = {};
+    }
     if (ns.Arn)
       await this.attachTags(result, ns.Arn);
     return result;
@@ -46250,7 +46334,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.57.0");
+  program.name("cdkd").description("CDK Direct - Deploy AWS CDK apps directly via SDK/Cloud Control API").version("0.57.1");
   program.addCommand(createBootstrapCommand());
   program.addCommand(createSynthCommand());
   program.addCommand(createListCommand());