Skip to content

New: [AEA-6520] - SnsAlarm#708

Merged
tstephen-nhs merged 8 commits intomainfrom
aea-0000-sns-alarm
Apr 27, 2026
Merged

New: [AEA-6520] - SnsAlarm#708
tstephen-nhs merged 8 commits intomainfrom
aea-0000-sns-alarm

Conversation

@tstephen-nhs
Copy link
Copy Markdown
Contributor

Summary

  • ✨ New Feature

Details

Share SnsAlarm construct from PfP for use in PSU

@tstephen-nhs tstephen-nhs marked this pull request as ready for review April 24, 2026 14:45
Copilot AI review requested due to automatic review settings April 24, 2026 14:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a reusable CDK construct (SnsAlarm) to create a CloudWatch Alarm and publish alarm state changes to an SNS topic, along with Vitest coverage to validate defaulting/overrides.

Changes:

  • Introduce SnsAlarm construct (and supporting types/helpers) to standardise alarm creation and SNS wiring.
  • Add tests verifying “sane defaults” and override behaviour for alarm/metric configuration.
  • Update package-lock.json due to dependency resolution changes.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 5 comments.

File Description
packages/cdkConstructs/src/constructs/SnsAlarm.ts New construct for creating a CloudWatch alarm and attaching SNS actions, with defaults for common fields.
packages/cdkConstructs/tests/constructs/SnsAlarm.test.ts New Vitest suite validating alarm synthesis and override behaviour.
package-lock.json Lockfile updates (root package name + transitive dependency bumps).

Comment thread packages/cdkConstructs/src/constructs/SnsAlarm.ts Outdated
Comment thread packages/cdkConstructs/src/constructs/SnsAlarm.ts
Comment thread packages/cdkConstructs/src/constructs/SnsAlarm.ts
Comment thread packages/cdkConstructs/tests/constructs/SnsAlarm.test.ts
Comment thread packages/cdkConstructs/src/constructs/SnsAlarm.ts
@tstephen-nhs tstephen-nhs enabled auto-merge (squash) April 24, 2026 16:19
@sonarqubecloud
Copy link
Copy Markdown

@tstephen-nhs tstephen-nhs merged commit d0e50de into main Apr 27, 2026
11 checks passed
@tstephen-nhs tstephen-nhs deleted the aea-0000-sns-alarm branch April 27, 2026 14:20
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.

3 participants