@go-to-k/cdkd 0.51.9 → 0.51.10

package/dist/cli.js

@@ -13792,10 +13792,15 @@ var SNSTopicProvider = class {
    * `create()`'s behavior of only sending Tags when the template carries
    * them).
    *
-   * `DeliveryStatusLogging` is intentionally omitted: it fans out into
-   * per-protocol attributes (`{Protocol}SuccessFeedbackRoleArn`, etc.) whose
-   * round-trip back to the CFn array shape needs more thought than fits in
-   * this PR.
+   * `DeliveryStatusLogging` is reverse-mapped from per-protocol flat
+   * attributes (`{Protocol}SuccessFeedbackRoleArn` etc.) back to the CFn
+   * array shape `[{Protocol, SuccessFeedbackRoleArn?, SuccessFeedbackSampleRate?,
+   * 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).
    *
    * `Subscription` is omitted because CDK manages it via separate
    * `AWS::SNS::Subscription` resources, not as a Topic property.
@@ -13844,6 +13849,7 @@ var SNSTopicProvider = class {
         }
       }
     }
+    result["DeliveryStatusLogging"] = mapDeliveryStatusLogging(attrs);
     try {
       const tagsResp = await this.snsClient.send(
         new ListTagsForResourceCommand({ ResourceArn: physicalId })
@@ -13858,16 +13864,13 @@ var SNSTopicProvider = class {
     return result;
   }
   /**
-   * `DeliveryStatusLogging` fans out to per-protocol attributes
-   * (`{Protocol}SuccessFeedbackRoleArn` etc.) whose round-trip back to the
-   * CFn array shape is not yet implemented; `Subscription` is managed via
-   * separate `AWS::SNS::Subscription` resources rather than the Topic
-   * itself. Both are absent from `readCurrentState`, so tell the drift
-   * comparator to skip them and avoid the guaranteed false-positive that
-   * would fire on every clean run when the user did template either.
+   * Only `Subscription` remains drift-unknown — CDK manages topic
+   * subscriptions via separate `AWS::SNS::Subscription` resources, so the
+   * inline `Topic.Subscription` property is intentionally not surfaced.
+   * `DeliveryStatusLogging` is now reverse-mapped (see `readCurrentState`).
    */
   getDriftUnknownPaths() {
-    return ["DeliveryStatusLogging", "Subscription"];
+    return ["Subscription"];
   }
   /**
    * Adopt an existing SNS topic into cdkd state.
@@ -13927,6 +13930,33 @@ var SNSTopicProvider = class {
     return null;
   }
 };
+var SNS_DELIVERY_STATUS_PROTOCOLS = [
+  "Application",
+  "Firehose",
+  "HTTP",
+  "HTTPS",
+  "Lambda",
+  "SQS"
+];
+function mapDeliveryStatusLogging(attrs) {
+  const result = [];
+  for (const protocol of SNS_DELIVERY_STATUS_PROTOCOLS) {
+    const success = attrs[`${protocol}SuccessFeedbackRoleArn`];
+    const sample = attrs[`${protocol}SuccessFeedbackSampleRate`];
+    const failure = attrs[`${protocol}FailureFeedbackRoleArn`];
+    if (success === void 0 && sample === void 0 && failure === void 0)
+      continue;
+    const entry = { Protocol: protocol };
+    if (success !== void 0)
+      entry["SuccessFeedbackRoleArn"] = success;
+    if (sample !== void 0)
+      entry["SuccessFeedbackSampleRate"] = sample;
+    if (failure !== void 0)
+      entry["FailureFeedbackRoleArn"] = failure;
+    result.push(entry);
+  }
+  return result;
+}
 
 // src/provisioning/providers/sns-subscription-provider.ts
 import {
@@ -45210,7 +45240,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.51.9");
+  program.name("cdkd").description("CDK Direct - Deploy AWS CDK apps directly via SDK/Cloud Control API").version("0.51.10");
   program.addCommand(createBootstrapCommand());
   program.addCommand(createSynthCommand());
   program.addCommand(createListCommand());