@fjall/components-infrastructure 38.0.0 → 39.0.0

package/dist/lib/utils/standardTagsAspect.d.ts

@@ -5,12 +5,19 @@ import { type IConstruct } from "constructs";
  *
  * This aspect handles tags that depend on resource-specific properties:
  * - fjall:operations:pool on VPCs with IPAM configuration
- * - fjall:disasterRecovery:tier on S3 buckets with backup configuration
  *
  * Standard Fjall tags (costAllocation:environment, costAllocation:service, etc.)
  * are applied via Tags.of(app).add() in the App class, which properly merges
  * with existing tags from L2 constructs.
  *
+ * The disaster-recovery tier tag is NOT applied here. It is stamped at
+ * construction by the five constructs that own a backup-eligible resource
+ * (`tagBackupTier`), because an aspect can only reach the resource by walking
+ * up the scope chain — which cannot tell an Aurora cluster from its member
+ * instances, or a primary from its read replica — and because `Tags.of(...)`
+ * called inside `visit()` is an aspect adding an aspect, dropped silently when
+ * `aspectStabilization` is off.
+ *
  * This aspect uses Tags.of(node).add() to properly merge with existing tags
  */
 export declare class StandardTagsAspect implements IAspect {
@@ -21,21 +28,4 @@ export declare class StandardTagsAspect implements IAspect {
      * Tag format: fjall:operations:pool = "{accountId}-{region}"
      */
     private addIpamPoolTag;
-    /**
-     * Add the disaster recovery tier tag to S3 buckets from the bucket's
-     * effective tier — the per-bucket `backupVaultTier` override, else the
-     * app-level tier the factory threaded in. CDK child tags override parent
-     * tags, so an override wins over the app-level tag set via
-     * `App.getApp({ backup })`; an inherited app tier writes the value that tag
-     * already carries. The same resolution decides the bucket's versioning
-     * default (`S3Bucket`).
-     *
-     * Tag format: fjall:disasterRecovery:tier = "{tier}"
-     *
-     * Tier mapping:
-     * - standard -> "default" (matches existing backup plans)
-     * - resilient -> "resilient"
-     * - enterprise -> "enterprise"
-     */
-    private addBackupTierTag;
 }

package/dist/lib/utils/standardTagsAspect.js

@@ -1,17 +1,23 @@
 import { CfnResource, Stack, Tags } from "aws-cdk-lib";
-import { BACKUP_TIER_TAG_KEY, BACKUP_TIER_TAG_MAP } from "./backupTierMapping.js";
 import { formatIpamPairTagValue, IPAM_OPERATIONS_POOL_TAG_KEY } from "@fjall/util/aws";
 /**
  * Aspect to apply special Fjall tags to specific resource types.
  *
  * This aspect handles tags that depend on resource-specific properties:
  * - fjall:operations:pool on VPCs with IPAM configuration
- * - fjall:disasterRecovery:tier on S3 buckets with backup configuration
  *
  * Standard Fjall tags (costAllocation:environment, costAllocation:service, etc.)
  * are applied via Tags.of(app).add() in the App class, which properly merges
  * with existing tags from L2 constructs.
  *
+ * The disaster-recovery tier tag is NOT applied here. It is stamped at
+ * construction by the five constructs that own a backup-eligible resource
+ * (`tagBackupTier`), because an aspect can only reach the resource by walking
+ * up the scope chain — which cannot tell an Aurora cluster from its member
+ * instances, or a primary from its read replica — and because `Tags.of(...)`
+ * called inside `visit()` is an aspect adding an aspect, dropped silently when
+ * `aspectStabilization` is off.
+ *
  * This aspect uses Tags.of(node).add() to properly merge with existing tags
  */
 export class StandardTagsAspect {
@@ -31,9 +37,6 @@ export class StandardTagsAspect {
         if (resource.cfnResourceType === "AWS::EC2::VPC") {
             this.addIpamPoolTag(resource);
         }
-        if (resource.cfnResourceType === "AWS::S3::Bucket") {
-            this.addBackupTierTag(resource);
-        }
     }
     /**
      * Add IPAM pool tag to VPCs that use IPAM for IP address allocation.
@@ -53,33 +56,4 @@ export class StandardTagsAspect {
             Tags.of(vpc).add(IPAM_OPERATIONS_POOL_TAG_KEY, formatIpamPairTagValue(accountId, region));
         }
     }
-    /**
-     * Add the disaster recovery tier tag to S3 buckets from the bucket's
-     * effective tier — the per-bucket `backupVaultTier` override, else the
-     * app-level tier the factory threaded in. CDK child tags override parent
-     * tags, so an override wins over the app-level tag set via
-     * `App.getApp({ backup })`; an inherited app tier writes the value that tag
-     * already carries. The same resolution decides the bucket's versioning
-     * default (`S3Bucket`).
-     *
-     * Tag format: fjall:disasterRecovery:tier = "{tier}"
-     *
-     * Tier mapping:
-     * - standard -> "default" (matches existing backup plans)
-     * - resilient -> "resilient"
-     * - enterprise -> "enterprise"
-     */
-    addBackupTierTag(resource) {
-        let construct = resource;
-        while (construct) {
-            const backupConstruct = construct;
-            if (backupConstruct.effectiveBackupTier) {
-                const tier = backupConstruct.effectiveBackupTier;
-                const tagValue = BACKUP_TIER_TAG_MAP[tier];
-                Tags.of(resource).add(BACKUP_TIER_TAG_KEY, tagValue);
-                break;
-            }
-            construct = construct.node.scope;
-        }
-    }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fjall/components-infrastructure",
-  "version": "38.0.0",
+  "version": "39.0.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/fjall-tech/fjall.git",
@@ -76,12 +76,12 @@
     "reflect-metadata": "^0.2.2",
     "tsx": "^4.23.1",
     "typescript": "^6.0.3",
-    "vitest": "^4.1.10"
+    "vitest": "^4.1.11"
   },
   "dependencies": {
     "@aws-sdk/client-organizations": "^3.1098.0",
-    "@fjall/generator": "^38.0.0",
-    "@fjall/util": "^38.0.0",
+    "@fjall/generator": "^39.0.0",
+    "@fjall/util": "^39.0.0",
     "constructs": "^10.7.2",
     "zod": "^4.4.3"
   },