Fix optional struct fields (#1889)

Now optional struct fields can be encoded and decoded properly

I also made it so that gofmt/goimports runs on generate so this is the
last time the diff should be this annoying!
<!-- ELLIPSIS_HIDDEN -->


----

> [!IMPORTANT]
> Fixes encoding/decoding of optional struct fields and adds `gofmt` to
Go code generation.
> 
>   - **Behavior**:
> - Fixes encoding and decoding of optional struct fields in
`generate_types.rs`.
> - Updates `Decode` functions in `types.go` to handle nil values for
optional fields.
>   - **Code Generation**:
> - Adds `gofmt` execution to `generators.baml` and `primary.yml` to
format Go code on generation.
>   - **Misc**:
> - Updates `generators.baml` in multiple `inlinedbaml` files to include
`gofmt` command.
> 
> <sup>This description was created by </sup>[<img alt="Ellipsis"
src="https://img.shields.io/badge/Ellipsis-blue?color=175173">](https://www.ellipsis.dev?ref=BoundaryML%2Fbaml&utm_source=github&utm_medium=referral)<sup>
for 2c08b82. You can
[customize](https://app.ellipsis.dev/BoundaryML/settings/summaries) this
summary. It will automatically update as commits are pushed.</sup>


<!-- ELLIPSIS_HIDDEN -->

Author: tberman

.github/workflows/primary.yml

@@ -55,6 +55,9 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - uses: jdx/mise-action@v2
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.21.x"
       - uses: actions/setup-node@v4
         with:
           node-version: 20
@@ -133,6 +136,9 @@ jobs:
       - uses: actions/checkout@v4
       - uses: jdx/mise-action@v2
       - uses: astral-sh/setup-uv@v5
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.21.x"
       - uses: dtolnay/rust-toolchain@stable
         with:
           toolchain: stable

engine/language_client_codegen/src/go/generate_types.rs

@@ -54,7 +54,20 @@ pub(crate) fn cast_value(container_variable_name: &str, field_type: &GoType) ->
 }
 
 fn render_value_coercion(container_variable_name: &str, field_type: &GoType) -> String {
-    if field_type.is_class {
+    if field_type.is_pointer {
+        let inner_type = field_type.underlying_type.as_ref().unwrap();
+        return format!(
+            "func () {} {{
+    val := baml.Decode({})
+    if val == nil {{
+        return nil
+    }}
+    castVal := val.({})
+    return &castVal
+}}()",
+            field_type.name, container_variable_name, inner_type.name,
+        );
+    } else if field_type.is_class {
         return format!(
             "*baml.Decode({}).(*{})",
             container_variable_name,

engine/language_client_codegen/src/go/mod.rs

@@ -287,14 +287,18 @@ class Foo {
         .unwrap()
     }
 
-    #[test]
-    fn generate_streaming_go() {
-        let ir = mk_ir();
-        let generator_args = mk_gen();
-        let res = generate(&ir, &generator_args).unwrap();
-        let partial_types = res
-            .get(&PathBuf::from("stream_types/stream_types.go"))
-            .unwrap();
-        eprintln!("{}", partial_types);
-    }
+    // Commenting out this test because streaming is not yet supported.
+    //
+    // TODO: re-enable this test once streaming is supported.
+    //
+    // #[test]
+    // fn generate_streaming_go() {
+    //     let ir = mk_ir();
+    //     let generator_args = mk_gen();
+    //     let res = generate(&ir, &generator_args).unwrap();
+    //     let partial_types = res
+    //         .get(&PathBuf::from("stream_types/stream_types.go"))
+    //         .unwrap();
+    //     eprintln!("{}", partial_types);
+    // }
 }

engine/language_client_codegen/src/go/templates/partial_types.go.j2

@@ -64,7 +64,6 @@ type StreamState[T any] struct {
 }
 
 
-
 {% for class in partial_classes %}
 {%- if let Some(docstring) = class.docstring %}
 {{ docstring }}

integ-tests/baml_src/generators.baml

@@ -43,6 +43,7 @@ generator lang_go {
   output_dir "../go"
   version "0.86.1"
   client_package_name "example.com/integ-tests"
+  on_generate "gofmt -w ."
 }