Repository navigation
feat(cloud-assembly-api): expose property mutation sites - #2082
proton0210 wants to merge 1 commit into
Conversation
|
Automated review A maintainer will still review this — treat the notes below as a starting point. 📌 Commit This PR surfaces property-mutation traces on The change is purely additive, with no modification to existing exports or behavior. The accumulation logic uses a null-prototype map to neutralize prototype-pollution from user-controlled property names, preserves traces in metadata order, and the destructured entry shape matches the schema's 🔴 0 blocking · 🟡 0 recommended · ⚪ 0 optional Generated automatically. React 👍 or 👎 to tell us whether this review helped, so we can improve these reviews. |
Expose
aws:cdk:propertyAssignmentmetadata asAssemblyIndexNode.propertyMutationStacks, grouped by CloudFormation property name. Preserve every recorded trace and make it available through both construct-path and template-location lookups, including nested stacks and assemblies.Fixes #2064.
This PR targets
explorer_2_assembly_loadingand depends on #2055, which introducesAssemblyIndex. The diff contains only the mutation-site follow-up.Validation:
assembly with tree metadata having no file property specifiedalso fails on base commit e61caab: it expectsInvalid assembly manifest, while the base throwsInvalid TreeCloudArtifact. Missing "file" property.git diff --checkpass.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license