Skip to content

feat(cli): add hotswap support for QuickSight resources via CCAPI#1457

Merged
aws-cdk-automation merged 1 commit intoaws:mainfrom
awsmadi:feature/hotswap-quicksight-ccapi
Apr 27, 2026
Merged

feat(cli): add hotswap support for QuickSight resources via CCAPI#1457
aws-cdk-automation merged 1 commit intoaws:mainfrom
awsmadi:feature/hotswap-quicksight-ccapi

Conversation

@awsmadi
Copy link
Copy Markdown
Contributor

@awsmadi awsmadi commented Apr 27, 2026

Summary

Motivation

QuickSight resources require frequent updates during development (dashboard definitions, data mappings, visualizations). Hotswap via CCAPI lets developers bypass CloudFormation and update resources directly, cutting iteration time.

This replaces #1055, which used custom per-service QuickSight SDK calls. Per feedback from @rix0rrr, the CCAPI approach is preferred since #1310 landed the generic infrastructure.

Changes

  • hotswap-deployments.ts: Add 5 QuickSight resource types to RESOURCE_DETECTORS map
  • cloud-control-hotswap-deployments.test.ts: Add QuickSight types to CCAPI sanity-check suite, plus dedicated tests for Dashboard Name and Definition changes in both FALL_BACK and HOTSWAP_ONLY modes

Test plan

  • All 50 CCAPI hotswap unit tests pass (5 new sanity checks + 4 QuickSight-specific tests)
  • Verify QuickSight Dashboard hotswap with a real CDK app using cdk deploy --hotswap

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Register five QuickSight resource types with the Cloud Control API
hotswap handler, enabling faster iterative development for dashboards,
analyses, templates, datasets, and data sources.

Leverages the generic CCAPI hotswap infrastructure from aws#1310 instead
of custom per-service SDK integrations.

Replaces aws#1055.
@awsmadi awsmadi deployed to integ-approval April 27, 2026 16:27 — with GitHub Actions Active
@github-actions github-actions Bot added the p2 label Apr 27, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team April 27, 2026 16:29
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.13%. Comparing base (37b646d) to head (c8007b5).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1457      +/-   ##
==========================================
- Coverage   88.15%   88.13%   -0.02%     
==========================================
  Files          74       74              
  Lines       10595    10595              
  Branches     1453     1454       +1     
==========================================
- Hits         9340     9338       -2     
- Misses       1227     1229       +2     
  Partials       28       28              
Flag Coverage Δ
suite.unit 88.13% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aws-cdk-automation aws-cdk-automation added this pull request to the merge queue Apr 27, 2026
Merged via the queue into aws:main with commit 0900f26 Apr 27, 2026
55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants