Migration smithy Part 4

tools/code-generation/smithy/cpp-codegen/smithy-cpp-codegen/src/main/java/com/amazonaws/util/awsclientsmithygenerator/generators/ShapeUtil.java

@@ -64,54 +64,6 @@ public class ShapeUtil {
         "cloudfront", "2020_05_31"
     );
 
-    /**
-     * Hardcoded shape rename collision resolutions from C2J.
-     * These shapes had name collisions with operation result wrappers in C2J
-     * and were given specific alternative names.
-     * Map: service-name -> Map of original-shape-name -> resolved-name
-     */
-    private static final Map<String, Map<String, String>> HARDCODED_COLLISION_RESOLUTIONS = Map.of(
-        "s3", Map.of("CopyObjectResult", "CopyObjectResultDetails"),
-        "accessanalyzer", Map.of("GeneratedPolicyResult", "GeneratedPolicyResults"),
-        "cloudsearchdomain", Map.of("SearchResult", "SearchResultDetails")
-    );
-
-    /**
-     * S3 shapes that exist in C2J but not in Smithy.
-     * These must be synthetically injected into the model before generation.
-     */
-    public static final Map<String, Set<String>> C2J_ONLY_SHAPES = Map.of(
-        "s3", Set.of(
-            "CopyObjectResultDetails", "SelectObjectContentEventStreamUnmarshallerError",
-            "CloudFunctionConfiguration", "QueueConfigurationDeprecated",
-            "TopicConfigurationDeprecated", "NotificationConfigurationDeprecated",
-            "RequestPaymentConfiguration", "PutObjectLockConfigurationRequestAlias",
-            "GetObjectLockConfigurationResultAlias", "ObjectLockConfigurationAlias",
-            "ObjectLockRuleAlias", "DefaultRetentionAlias", "ObjectLockRetentionAlias"
-        )
-    );
-
-    /**
-     * Returns the hardcoded collision resolution for a shape, if one exists.
-     */
-    public static Optional<String> getHardcodedResolution(String smithyServiceName, String shapeName) {
-        Map<String, String> serviceResolutions = HARDCODED_COLLISION_RESOLUTIONS.get(smithyServiceName);
-        if (serviceResolutions == null) return Optional.empty();
-        return Optional.ofNullable(serviceResolutions.get(shapeName));
-    }
-
-    /**
-     * Returns the C++ class name for a shape, applying collision renames and numeric prefix rules.
-     */
-    public static String getShapeCppName(String shapeName, String smithyServiceName) {
-        Optional<String> resolved = getHardcodedResolution(smithyServiceName, shapeName);
-        if (resolved.isPresent()) return resolved.get();
-        if (!shapeName.isEmpty() && Character.isDigit(shapeName.charAt(0))) {
-            return "The" + shapeName;
-        }
-        return shapeName;
-    }
-
     /**
      * C2J/Smithy model mismatches: tokens that are integers in C2J but strings in Smithy.
      *

tools/code-generation/smithy/cpp-codegen/smithy-cpp-codegen/src/main/java/com/amazonaws/util/awsclientsmithygenerator/generators/model/CppTypeMapper.java

@@ -275,24 +275,10 @@ public static List<String> getIncludesForShape(Shape structureShape, Model model
                 }
             } else {
                 addMemberInclude(includes, target, selfId, model, projectName);
-                // For list/map, also include the element/key/value types
-                if (target.isListShape()) {
-                    ListShape list = target.asListShape().get();
-                    addMemberInclude(includes, model.expectShape(list.getMember().getTarget()),
-                        selfId, model, projectName);
-                }
-                if (target.isMapShape()) {
-                    MapShape map = target.asMapShape().get();
-                    addMemberInclude(includes, model.expectShape(map.getKey().getTarget()),
-                        selfId, model, projectName);
-                    addMemberInclude(includes, model.expectShape(map.getValue().getTarget()),
-                        selfId, model, projectName);
-                }
-                // A @sparse list/map wraps its element/value in Aws::Crt::Optional, declared in
-                // <aws/crt/Optional.h>. Matches C2J's generated SparseNullsOperationRequest.h.
-                if ((target.isListShape() || target.isMapShape()) && target.hasTrait(SparseTrait.class)) {
-                    includes.add("<aws/crt/Optional.h>");
-                }
+                // For list/map, recursively include every nested element/key/value type so leaf
+                // struct/enum headers reach the surface even through nested containers (e.g.
+                // apigateway Deployment.apiSummary: Map<String, Map<String, MethodSnapshot>>).
+                addContainerIncludes(includes, target, selfId, model, projectName);
             }
             // @idempotencyToken members are brace-initialized with
             // Aws::Utils::UUID::PseudoRandomUUID(), which requires UUID.h. Matches C2J
@@ -316,6 +302,39 @@ private static void addMemberInclude(Set<String> includes, Shape shape, ShapeId
         }
     }
 
+    /**
+     * Recursively adds member-type includes for every nested element/key/value of a list or map
+     * shape. Recursion only descends through further list/map shapes and stops at
+     * structures/enums/scalars, so it is bounded by the container-nesting depth (no infinite
+     * recursion). {@code addMemberInclude} remains a no-op for container/scalar shapes without
+     * their own header. This lets a member typed, e.g., {@code Map<String, Map<String, Leaf>>}
+     * reach {@code Leaf}'s header, matching C2J's recursive unwrap.
+     *
+     * <p>The {@code @sparse}-&gt;{@code <aws/crt/Optional.h>} handling fires at each nested
+     * container level that is sparse, matching C2J's generated headers.
+     */
+    private static void addContainerIncludes(Set<String> includes, Shape target, ShapeId selfId,
+                                             Model model, String projectName) {
+        if (target.isListShape()) {
+            Shape elem = model.expectShape(target.asListShape().get().getMember().getTarget());
+            addMemberInclude(includes, elem, selfId, model, projectName);
+            addContainerIncludes(includes, elem, selfId, model, projectName);
+        } else if (target.isMapShape()) {
+            MapShape map = target.asMapShape().get();
+            Shape key = model.expectShape(map.getKey().getTarget());
+            Shape value = model.expectShape(map.getValue().getTarget());
+            addMemberInclude(includes, key, selfId, model, projectName);
+            addMemberInclude(includes, value, selfId, model, projectName);
+            addContainerIncludes(includes, key, selfId, model, projectName);
+            addContainerIncludes(includes, value, selfId, model, projectName);
+        }
+        // A @sparse list/map wraps its element/value in Aws::Crt::Optional, declared in
+        // <aws/crt/Optional.h>. Matches C2J's generated SparseNullsOperationRequest.h.
+        if ((target.isListShape() || target.isMapShape()) && target.hasTrait(SparseTrait.class)) {
+            includes.add("<aws/crt/Optional.h>");
+        }
+    }
+
     /**
      * Returns the sorted C++ class names of every direct member whose target forms a reference
      * cycle with {@code structureShape} (see {@link #isRecursiveStructMember}). These are stored

tools/code-generation/smithy/cpp-codegen/smithy-cpp-codegen/src/main/java/com/amazonaws/util/awsclientsmithygenerator/generators/model/EnumRenderer.java

@@ -11,6 +11,7 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Map;
 import java.util.Set;
 import java.util.stream.Collectors;
 
@@ -50,6 +51,20 @@ public static void renderHeader(CppWriter writer, Shape enumShape, String servic
         writer.write("#include <aws/$1L/$2L_EXPORTS.h>",
             projectName, serviceName);
         writer.write("");
+
+        // Windows defines some enum values as preprocessor macros (e.g. EC2's `interface` via
+        // <combaseapi.h>). Undefine them so the generated enum constant compiles, matching C2J's
+        // ModelEnumHeader.vm predefined-symbol guard.
+        List<String> windowsMacros = predefinedWindowsSymbols(serviceName, values);
+        if (!windowsMacros.isEmpty()) {
+            for (String macro : windowsMacros) {
+                writer.write("#if defined(_WIN32) && defined($L)", macro);
+                writer.write("#undef $L", macro);
+                writer.write("#endif");
+            }
+            writer.write("");
+        }
+
         writer.write("namespace Aws {");
         writer.write("namespace $L {", serviceName);
         writer.write("namespace Model {");
@@ -225,6 +240,34 @@ private static List<String> getEnumWireValues(Shape enumShape) {
         "STATIC", "T_CHAR", "DOMAIN", "OVERFLOW", "WINDOWS"
     );
 
+    /**
+     * Per-service enum constant names that collide with a Windows preprocessor macro and must be
+     * {@code #undef}'d in the enum header. Keyed by C++ service namespace, mirroring C2J
+     * PlatformAndKeywordSanitizer.PREDEFINED_SYMBOLS_MAPPING.
+     */
+    private static final Map<String, Set<String>> PREDEFINED_WINDOWS_SYMBOLS = Map.of(
+        "DynamoDB", Set.of("IN"),
+        "EC2", Set.of("interface"),
+        "S3Crt", Set.of("IGNORE")
+    );
+
+    /**
+     * Returns, in enum-declaration order, the sanitized enum constant names of {@code values} that
+     * collide with a Windows macro for {@code serviceNamespace} (see
+     * {@link #PREDEFINED_WINDOWS_SYMBOLS}). Empty when the service has no such symbols.
+     *
+     * @param serviceNamespace the C++ service namespace (e.g., "EC2")
+     * @param values           the sanitized enum constant names in declaration order
+     * @return the subset needing a {@code #undef} guard, preserving declaration order
+     */
+    static List<String> predefinedWindowsSymbols(String serviceNamespace, List<String> values) {
+        Set<String> symbols = PREDEFINED_WINDOWS_SYMBOLS.get(serviceNamespace);
+        if (symbols == null) {
+            return List.of();
+        }
+        return values.stream().filter(symbols::contains).collect(Collectors.toList());
+    }
+
     /**
      * Sanitizes an enum wire value into a valid C++ identifier, matching C2J
      * PlatformAndKeywordSanitizer.fixEnumValue() behavior.