From 1138cc4f37c6ef078ba933b842a9f332aa24f56f Mon Sep 17 00:00:00 2001 From: avneetbansal-aws Date: Fri, 2 Oct 2026 13:15:07 -0400 Subject: [PATCH] fix(deploy-on-aws): dedupe duplicate `iot` key in aws4-shapes.json The shape registry had two top-level `iot` entries. Lenient parsers keep the last one, so the first block (fillColor #1A9C37) was silently shadowed by a second block that used #3F8624 (the Storage color) and a shorter shape list. This dropped ~45 valid iot_* shape names from the set built by validate_drawio.py and broke the strict check-json pre-commit hook. Merge the two entries into a single `iot` category: keep fillColor #1A9C37 (matching the authoritative IoT mapping in fix_icon_colors.py) and take the union of both shape lists so no valid shape name is lost. Fixes #216 Signed-off-by: avneetbansal-aws --- .../scripts/lib/aws4-shapes.json | 27 +++---------------- 1 file changed, 4 insertions(+), 23 deletions(-) diff --git a/plugins/deploy-on-aws/scripts/lib/aws4-shapes.json b/plugins/deploy-on-aws/scripts/lib/aws4-shapes.json index 10fe63f6..a38020dc 100644 --- a/plugins/deploy-on-aws/scripts/lib/aws4-shapes.json +++ b/plugins/deploy-on-aws/scripts/lib/aws4-shapes.json @@ -543,7 +543,10 @@ "iot_expresslink", "iot_fleetwise", "iot_roborunner", - "iot_twinmaker" + "iot_twinmaker", + "greengrass", + "internet_of_things", + "iot_edukit" ] }, "media_services": { @@ -571,28 +574,6 @@ "thinkbox_xmesh" ] }, - "iot": { - "fillColor": "#3F8624", - "shapes": [ - "freertos", - "greengrass", - "internet_of_things", - "iot_1click", - "iot_analytics", - "iot_button", - "iot_core", - "iot_device_defender", - "iot_device_management", - "iot_edukit", - "iot_events", - "iot_expresslink", - "iot_fleetwise", - "iot_roborunner", - "iot_sitewise", - "iot_things_graph", - "iot_twinmaker" - ] - }, "games": { "fillColor": "#ED7100", "shapes": [