Refactor: defaulted members must be optional

During usage of the library, it was found that members with default values
must be optional, otherwise deserialization fails if the member is missing
in reflect-cpp.

Author: pingelit

code-gen/src/poly_scribe_code_gen/cpp_gen.py

@@ -90,7 +90,7 @@ def _transform_types(parsed_idl: ParsedIDL) -> ParsedIDL:
     for struct_data in parsed_idl["structs"].values():
         for member_data in struct_data["members"].values():
             member_data["type"] = _transformer(member_data["type"], parsed_idl["inheritance_data"])
-            if not member_data["required"] and member_data["default"] is None:
+            if not member_data["required"]:
                 member_data["type"] = f"std::optional<{member_data['type']}>"
 
             if "std::string" in member_data["type"] and member_data["default"]:

code-gen/src/poly_scribe_code_gen/py_gen.py

@@ -170,7 +170,7 @@ def _transform_types(parsed_idl: ParsedIDL) -> ParsedIDL:
                     type_str = type_str[1:-1]
                 member_data["default"] = f"{type_str}()"
 
-            if not member_data["required"] and member_data["default"] is None:
+            if not member_data["required"]:
                 member_data["type"] = f"Optional[{member_data['type']}]"
 
             if "str" in member_data["type"] and member_data["default"] is not None: