Default to_s

codegen/smithy-ruby-codegen/CHANGELOG.md

@@ -1,6 +1,7 @@
 Unreleased Changes
 ------------------
 
+* Issue - Add `to_s` to `Structure`.
 * Issue - Remove `Struct` from generated Types, Config, and other places to allow for better RBS typing as well as less reserved words.
 * Feature - Add support for stringArray Endpoint parameters + operationContextParams binding.
 * Issue - Update reserved member names for new Struct methods.

codegen/smithy-ruby-codegen/src/main/java/software/amazon/smithy/ruby/codegen/generators/types/StructureGenerator.java

@@ -191,25 +191,29 @@ private void renderToSMethod(RubyCodeWriter writer) {
 
             writer
                     .openBlock("\ndef to_s")
-                    .write("\"#<$L \"\\", fullQualifiedShapeName)
+                    .write("'#<$L ' \\", fullQualifiedShapeName)
                     .indent();
 
             while (iterator.hasNext()) {
                 MemberShape memberShape = iterator.next();
                 String key = symbolProvider.toMemberName(memberShape);
                 String value = "#{" + key + " || 'nil'}";
+                boolean memberLiteral = false;
 
                 if (memberShape.isBlobShape() || memberShape.isStringShape()) {
                     // Strings are wrapped in quotes
                     value = "\"" + value + "\"";
+                    memberLiteral = true;
                 } else if (memberShape.getMemberTrait(model, SensitiveTrait.class).isPresent()) {
-                    value = "\\\"[SENSITIVE]\\\"";
+                    value = "[SENSITIVE]";
+                    memberLiteral = true;
                 }
 
+                String quote = memberLiteral ? "'" : "\"";
                 if (iterator.hasNext()) {
-                    writer.write("\"$L=$L, \"\\", key, value);
+                    writer.write("$3L$1L=$2L, $3L \\", key, value, quote);
                 } else {
-                    writer.write("\"$L=$L>\"", key, value);
+                    writer.write("$3L$1L=$2L>$3L", key, value, quote);
                 }
             }
             writer

codegen/smithy-ruby-codegen/src/main/java/software/amazon/smithy/ruby/codegen/generators/types/UnionGenerator.java

@@ -112,19 +112,17 @@ public void render() {
     }
 
     private void renderUnionToSMethod(RubyCodeWriter writer, MemberShape memberShape) {
-        String fullQualifiedShapeName = settings.getModule() + "::Types::"
-                + symbolProvider.toMemberName(memberShape);
-
-        writer.write("")
-                .openBlock("def to_s");
-
-        if (memberShape.getMemberTrait(model, SensitiveTrait.class).isPresent()) {
-            writer.write("\"#<$L [SENSITIVE]>\"", fullQualifiedShapeName);
-        } else {
-            writer.write("\"#<$L #{__getobj__ || 'nil'}>\"", fullQualifiedShapeName);
+        if (!memberShape.getMemberTrait(model, SensitiveTrait.class).isPresent()) {
+            return;
         }
 
-        writer.closeBlock("end");
+        String fullQualifiedShapeName = settings.getModule() + "::Types::"
+                + symbolProvider.toMemberName(memberShape);
+        writer
+                .write("")
+                .openBlock("def to_s")
+                .write("\"#<$L [SENSITIVE]>\"", fullQualifiedShapeName)
+                .closeBlock("end");
     }
 
     private class UnionToHValueRbsVisitor extends ShapeVisitor.Default<String> {