Skip to content

fix(alarms): allow Fn::Sub in alarm topic ARNs#702

Merged
zirkelc merged 1 commit intoserverless-operations:masterfrom
BANCS-Norway:fix-417-sub-alarm-topics
Mar 23, 2026
Merged

fix(alarms): allow Fn::Sub in alarm topic ARNs#702
zirkelc merged 1 commit intoserverless-operations:masterfrom
BANCS-Norway:fix-417-sub-alarm-topics

Conversation

@VirtueMe
Copy link
Copy Markdown
Collaborator

Summary

  • Extends the alarm ARN schema to accept Fn::Sub (both string and array forms) as a valid value for topics.ok, topics.alarm, and topics.insufficientData
  • Users can now reference pseudo-parameters like ${AWS::Region} and ${AWS::AccountId} without the deprecated serverless-pseudo-parameters plugin

Test plan

  • Added test verifying Fn::Sub string form passes schema validation and is passed through to AlarmActions in the CloudFormation output
  • All 15 existing alarm tests continue to pass

Closes #417

🤖 Generated with Claude Code

Extend the alarm ARN schema to accept Fn::Sub (both string and
array forms) so users can reference pseudo-parameters like
AWS::Region and AWS::AccountId without the serverless-pseudo-parameters
plugin.

Closes serverless-operations#417

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 19, 2026

Open in StackBlitz

npm i https://pkg.pr.new/serverless-operations/serverless-step-functions@702

commit: 500adea

@zirkelc zirkelc merged commit 2c136aa into serverless-operations:master Mar 23, 2026
3 checks passed
@VirtueMe VirtueMe deleted the fix-417-sub-alarm-topics branch March 23, 2026 08:04
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.

Allow !Sub/Fn::Sub in alarm topics

2 participants