@go-to-k/cdkd 0.99.3 → 0.100.1

package/dist/{deploy-engine-D44ADMVs.js → deploy-engine-CYrZgRDF.js}

@@ -3311,7 +3311,8 @@ const STATE_SCHEMA_VERSIONS_READABLE = [
 	1,
 	2,
 	3,
-	4
+	4,
+	5
 ];
 
 //#endregion
@@ -3528,7 +3529,7 @@ var S3StateBackend = class {
 		const { expectedEtag, migrateLegacy } = options;
 		const body = {
 			...state,
-			version: 4,
+			version: 5,
 			stackName,
 			region
 		};
@@ -4911,16 +4912,18 @@ var DiffCalculator = class DiffCalculator {
 				const rawDesiredProps = desiredResource.Properties || {};
 				const desiredPropsForCompare = resolveFn ? await this.resolveBestEffort(rawDesiredProps, resolveFn) : rawDesiredProps;
 				const propertyChanges = this.compareProperties(desiredResource.Type, currentResource.properties, desiredPropsForCompare);
-				if (propertyChanges.length > 0) {
+				const attributeChanges = this.compareAttributes(currentResource, desiredResource);
+				if (propertyChanges.length > 0 || attributeChanges.length > 0) {
 					changes.set(logicalId, {
 						logicalId,
 						changeType: "UPDATE",
 						resourceType: desiredResource.Type,
 						currentProperties: currentResource.properties,
 						desiredProperties: rawDesiredProps,
-						propertyChanges
+						propertyChanges,
+						...attributeChanges.length > 0 && { attributeChanges }
 					});
-					this.logger.debug(`UPDATE: ${logicalId} (${propertyChanges.length} property changes)`);
+					this.logger.debug(`UPDATE: ${logicalId} (${propertyChanges.length} property changes, ${attributeChanges.length} attribute changes)`);
 				} else {
 					changes.set(logicalId, {
 						logicalId,
@@ -4964,6 +4967,29 @@ var DiffCalculator = class DiffCalculator {
 		return resolved;
 	}
 	/**
+	* Compare CloudFormation template-level attributes (`DeletionPolicy`,
+	* `UpdateReplacePolicy`) between cdkd state and the synth template.
+	*
+	* Schema v5+ records these in `ResourceState`; state written by an older
+	* cdkd binary has the fields undefined. Treating `undefined === undefined`
+	* as "no change" means the first post-upgrade deploy of an unchanged
+	* template doesn't spuriously fire an attribute diff.
+	*/
+	compareAttributes(currentResource, desiredResource) {
+		const changes = [];
+		if (currentResource.deletionPolicy !== desiredResource.DeletionPolicy) changes.push({
+			attribute: "DeletionPolicy",
+			oldValue: currentResource.deletionPolicy,
+			newValue: desiredResource.DeletionPolicy
+		});
+		if (currentResource.updateReplacePolicy !== desiredResource.UpdateReplacePolicy) changes.push({
+			attribute: "UpdateReplacePolicy",
+			oldValue: currentResource.updateReplacePolicy,
+			newValue: desiredResource.UpdateReplacePolicy
+		});
+		return changes;
+	}
+	/**
 	* Compare properties and return list of changes
 	*
 	* Uses ReplacementRulesRegistry to determine which property changes require replacement.
@@ -5339,7 +5365,7 @@ var IntrinsicFunctionResolver = class {
 	async constructAttribute(resource, attributeName, _context) {
 		const { resourceType, physicalId } = resource;
 		const { region, accountId, partition } = await getAccountInfo(this.resolverRegion);
-		if (resourceType === "AWS::DynamoDB::Table") switch (attributeName) {
+		if (resourceType === "AWS::DynamoDB::Table" || resourceType === "AWS::DynamoDB::GlobalTable") switch (attributeName) {
 			case "Arn": return `arn:${partition}:dynamodb:${region}:${accountId}:table/${physicalId}`;
 			case "StreamArn": return;
 			default: return physicalId;
@@ -8548,7 +8574,7 @@ var DeployEngine = class {
 		try {
 			const currentStateData = await this.stateBackend.getState(stackName, this.stackRegion);
 			const currentState = currentStateData?.state ?? {
-				version: 4,
+				version: 5,
 				region: this.stackRegion,
 				stackName,
 				resources: {},
@@ -8589,7 +8615,7 @@ var DeployEngine = class {
 					await this.drainObservedCaptures(currentState.resources);
 					try {
 						const refreshedState = {
-							version: 4,
+							version: 5,
 							region: this.stackRegion,
 							stackName: currentState.stackName,
 							resources: currentState.resources,
@@ -8687,7 +8713,7 @@ var DeployEngine = class {
 			saveChain = saveChain.then(async () => {
 				try {
 					const partialState = {
-						version: 4,
+						version: 5,
 						region: this.stackRegion,
 						stackName: currentState.stackName,
 						resources: newResources,
@@ -8793,7 +8819,7 @@ var DeployEngine = class {
 		} catch (error) {
 			try {
 				const preRollbackState = {
-					version: 4,
+					version: 5,
 					region: this.stackRegion,
 					stackName: currentState.stackName,
 					resources: newResources,
@@ -8822,7 +8848,7 @@ var DeployEngine = class {
 			} else await this.performRollback(completedOperations, newResources, stackName);
 			try {
 				const postRollbackState = {
-					version: 4,
+					version: 5,
 					region: this.stackRegion,
 					stackName: currentState.stackName,
 					resources: newResources,
@@ -8837,7 +8863,7 @@ var DeployEngine = class {
 				try {
 					const freshEtag = (await this.stateBackend.getState(stackName, this.stackRegion))?.etag;
 					const postRollbackState = {
-						version: 4,
+						version: 5,
 						region: this.stackRegion,
 						stackName: currentState.stackName,
 						resources: newResources,
@@ -8856,7 +8882,7 @@ var DeployEngine = class {
 		const outputs = await this.resolveOutputs(template, newResources, stackName, parameterValues, conditions);
 		return {
 			state: {
-				version: 4,
+				version: 5,
 				region: this.stackRegion,
 				stackName: currentState.stackName,
 				resources: newResources,
@@ -9054,12 +9080,14 @@ var DeployEngine = class {
 				const { provider: createProvider, properties: createProps } = this.selectProviderWithSafetyNet(provider, resourceType, resolvedProps, logicalId);
 				const result = await this.withRetry(() => createProvider.create(logicalId, resourceType, createProps), logicalId, void 0, void 0, provider);
 				const dependencies = this.extractAllDependencies(template, logicalId);
+				const templateAttrs = this.extractTemplateAttributes(template, logicalId);
 				stateResources[logicalId] = {
 					physicalId: result.physicalId,
 					resourceType,
 					properties: resolvedProps,
 					...result.attributes && { attributes: result.attributes },
-					...dependencies && dependencies.length > 0 && { dependencies }
+					...dependencies && dependencies.length > 0 && { dependencies },
+					...templateAttrs
 				};
 				this.kickOffObservedCapture(provider, logicalId, result.physicalId, resourceType, resolvedProps);
 				if (counts) counts.created++;
@@ -9082,6 +9110,20 @@ var DeployEngine = class {
 				}, stackName);
 				const resolvedProps = await this.resolver.resolve(desiredProps, context);
 				if (JSON.stringify(resolvedProps) === JSON.stringify(currentProps)) {
+					if (change.attributeChanges && change.attributeChanges.length > 0) {
+						const attrSummary = change.attributeChanges.map((a) => `${a.attribute}: ${a.oldValue ?? "(unset)"} → ${a.newValue ?? "(unset)"}`).join(", ");
+						this.logger.info(`  ↻ ${logicalId} (${resourceType}) attribute update: ${attrSummary}`);
+						stateResources[logicalId] = {
+							...currentResource,
+							...this.extractTemplateAttributes(template, logicalId)
+						};
+						if (counts) counts.updated++;
+						if (progress) progress.current++;
+						const attrPrefix = progress ? `[${progress.current}/${progress.total}] ` : "  ";
+						renderer.removeTask(logicalId);
+						this.logger.info(`${attrPrefix}✅ ${logicalId} (${resourceType}) updated (metadata)`);
+						break;
+					}
 					this.logger.debug(`Skipping ${logicalId}: no actual changes after intrinsic function resolution`);
 					if (counts) counts.skipped++;
 					break;
@@ -9109,7 +9151,8 @@ var DeployEngine = class {
 						resourceType,
 						properties: resolvedProps,
 						...createResult.attributes && { attributes: createResult.attributes },
-						...dependencies && dependencies.length > 0 && { dependencies }
+						...dependencies && dependencies.length > 0 && { dependencies },
+						...this.extractTemplateAttributes(template, logicalId)
 					};
 					this.kickOffObservedCapture(provider, logicalId, createResult.physicalId, resourceType, resolvedProps);
 					if (counts) counts.updated++;
@@ -9149,7 +9192,8 @@ var DeployEngine = class {
 						resourceType,
 						properties: resolvedProps,
 						...result.attributes && { attributes: result.attributes },
-						...dependencies && dependencies.length > 0 && { dependencies }
+						...dependencies && dependencies.length > 0 && { dependencies },
+						...this.extractTemplateAttributes(template, logicalId)
 					};
 					this.kickOffObservedCapture(provider, logicalId, result.physicalId, resourceType, resolvedProps);
 					if (counts) counts.updated++;
@@ -9207,6 +9251,22 @@ var DeployEngine = class {
 		return deps.size > 0 ? [...deps] : void 0;
 	}
 	/**
+	* Read `DeletionPolicy` / `UpdateReplacePolicy` from the synth template
+	* so they can be persisted in `ResourceState` (schema v5+). Always returns
+	* both keys (`undefined` when the template does not carry the attribute)
+	* so that spreading into an existing `ResourceState` reliably overrides a
+	* previously-recorded value back to `undefined` — required when the user
+	* removes the attribute from their CDK code. `JSON.stringify` then omits
+	* the `undefined` keys when state is serialized to S3.
+	*/
+	extractTemplateAttributes(template, logicalId) {
+		const resource = template?.Resources?.[logicalId];
+		return {
+			deletionPolicy: resource?.DeletionPolicy,
+			updateReplacePolicy: resource?.UpdateReplacePolicy
+		};
+	}
+	/**
 	* Build a per-resource map of "must be deleted before me" dependencies for
 	* the DELETE phase, derived from state-recorded dependencies plus implicit
 	* type-based ordering rules.
@@ -9363,4 +9423,4 @@ var DeployEngine = class {
 
 //#endregion
 export { formatError as $, resolveCaptureObservedState as A, ConfigError as B, stringifyValue as C, getDefaultStateBucketName as D, Synthesizer as E, AssemblyReader as F, ProvisioningError as G, LocalInvokeBuildError as H, clearBucketRegionCache as I, RouteDiscoveryError as J, ResourceTimeoutError as K, resolveBucketRegion as L, resolveStateBucketWithDefault as M, resolveStateBucketWithDefaultAndSource as N, getLegacyStateBucketName as O, warnDeprecatedNoPrefixCliFlag as P, SynthesisError as Q, AssetError as R, AssetPublisher as S, buildDockerImage as T, LockError as U, DependencyError as V, PartialFailureError as W, StackTerminationProtectionError as X, StackHasActiveImportsError as Y, StateError as Z, DiffCalculator as _, withRetry as a, setLogger as at, LockManager as b, collectInlinePolicyNamesManagedBySiblings as c, PATTERN_B_NAME_PROPERTIES as ct, normalizeAwsTagsToCfn as d, generateResourceNameWithFallback as dt, isCdkdError as et, resolveExplicitPhysicalId as f, withSkipPrefix as ft, IntrinsicFunctionResolver as g, assertRegionMatch as h, withResourceDeadline as i, getLogger as it, resolveSkipPrefix as j, resolveApp as k, CDK_PATH_TAG as l, PATTERN_B_RESOURCE_TYPES as lt, CloudControlProvider as m, DEFAULT_RESOURCE_WARN_AFTER_MS as n, withErrorHandling as nt, IMPLICIT_DELETE_DEPENDENCIES as o, runStackBuffered as ot, ProviderRegistry as p, withStackName as pt, ResourceUpdateNotSupportedError as q, DeployEngine as r, ConsoleLogger as rt, IAMRoleProvider as s, getLiveRenderer as st, DEFAULT_RESOURCE_TIMEOUT_MS as t, normalizeAwsError as tt, matchesCdkPath as u, generateResourceName as ut, DagBuilder as v, WorkGraph as w, S3StateBackend as x, TemplateParser as y, CdkdError as z };
-//# sourceMappingURL=deploy-engine-D44ADMVs.js.map
+//# sourceMappingURL=deploy-engine-CYrZgRDF.js.map