Skip to content

Fix validator false positives and runtime lifecycle messages - #366

Merged
satyakigh merged 4 commits into
mainfrom
fp
Sep 3, 2026
Merged

Fix validator false positives and runtime lifecycle messages#366
satyakigh merged 4 commits into
mainfrom
fp

Conversation

@satyakigh

@satyakigh satyakigh commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

This change fixes two confirmed validator false positives, adds service-grounded handling for undocumented parameter types, and corrects runtime lifecycle message grammar.

Changes

Remove confirmed false positives

  • Deduplicate IAM identity-policy findings when multiple scenarios render the same authored root PolicyDocument type defect.
    • Distinct conditional branch defects remain separate.
  • Stop treating the format of AWS::SNS::Topic.KmsMasterKeyId as a Fatal schema constraint.
    • CloudFormation successfully created an SNS topic using the previously rejected identifier, proving that SNS accepts it as an opaque runtime value.

Handle undocumented parameter types accurately

Add W2002 for undocumented AWS-shaped parameter types that CloudFormation accepts without validating their values, including:

  • AWS::SSM::Parameter::Value<AWS::Service::Type>
  • AWS::SSM::Parameter::Value<List<AWS::Service::Type>>
  • List<AWS::Service::Type>

Correct runtime lifecycle grammar

Runtime lifecycle messages now use tense appropriate to the embedded lifecycle band:

  • Deprecated: creation and updates will be disabled
  • Create-blocked: creation was disabled, updates will be disabled
  • End-of-life: creation was disabled, updates were disabled

@satyakigh satyakigh changed the title bug fixes Fix validator false positives and runtime lifecycle messages Sep 3, 2026
@satyakigh
satyakigh merged commit f9376db into main Sep 3, 2026
3 checks passed
@satyakigh
satyakigh deleted the fp branch September 3, 2026 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant