Merge pull request #2 from CASParser/release-please--branches--main--changes--next

release: 0.0.3

Author: sameerkumar18

.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.0.2"
+  ".": "0.0.3"
 }

.stats.yml

@@ -1,4 +1,4 @@
 configured_endpoints: 5
 openapi_spec_url: https://storage.googleapis.com/stainless-sdk-openapi-specs/cas-parser%2Fcas-parser-b7fdba3d3f97c7debc22c7ca30b828bce81bcd64648df8c94029b27a3321ebb9.yml
 openapi_spec_hash: 03f1315f1d32ada42445ca920f047dff
-config_hash: 0e1291f316b20497ad29b59a231a8680
+config_hash: cb5d75abef6264b5d86448caf7295afa

CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## 0.0.3 (2025-09-06)
+
+Full Changelog: [v0.0.2...v0.0.3](https://github.com/CASParser/cas-parser-go/compare/v0.0.2...v0.0.3)
+
+### Bug Fixes
+
+* close body before retrying ([891bbc6](https://github.com/CASParser/cas-parser-go/commit/891bbc6d9b7b2840269c438f9242274fbab9fe97))
+* **internal:** unmarshal correctly when there are multiple discriminators ([08c4f8a](https://github.com/CASParser/cas-parser-go/commit/08c4f8ad7fa898837bef8f1a8898f3e8b56329d9))
+
+
+### Chores
+
+* **internal:** codegen related update ([86015d2](https://github.com/CASParser/cas-parser-go/commit/86015d220838b6d6d72472ba134308e8c851100b))
+
 ## 0.0.2 (2025-08-18)
 
 Full Changelog: [v0.0.1...v0.0.2](https://github.com/CASParser/cas-parser-go/compare/v0.0.1...v0.0.2)

README.md

@@ -1,7 +1,11 @@
 # Cas Parser Go API Library
 
+<!-- x-release-please-start-version -->
+
 <a href="https://pkg.go.dev/github.com/CASParser/cas-parser-go"><img src="https://pkg.go.dev/badge/github.com/CASParser/cas-parser-go.svg" alt="Go Reference"></a>
 
+<!-- x-release-please-end -->
+
 The Cas Parser Go library provides convenient access to the [Cas Parser REST API](https://docs.casparser.in/reference)
 from applications written in Go.
 
@@ -24,7 +28,7 @@ Or to pin the version:
 <!-- x-release-please-start-version -->
 
 ```sh
-go get -u 'github.com/CASParser/[email protected].2'
+go get -u 'github.com/CASParser/[email protected].3'
 ```
 
 <!-- x-release-please-end -->

internal/apijson/decodeparam_test.go

@@ -351,6 +351,36 @@ func init() {
 	})
 }
 
+type FooVariant struct {
+	Type  string `json:"type,required"`
+	Value string `json:"value,required"`
+}
+
+type BarVariant struct {
+	Type   string `json:"type,required"`
+	Enable bool   `json:"enable,required"`
+}
+
+type MultiDiscriminatorUnion struct {
+	OfFoo *FooVariant `json:",inline"`
+	OfBar *BarVariant `json:",inline"`
+
+	paramUnion
+}
+
+func init() {
+	apijson.RegisterDiscriminatedUnion[MultiDiscriminatorUnion]("type", map[string]reflect.Type{
+		"foo":        reflect.TypeOf(FooVariant{}),
+		"foo_v2":     reflect.TypeOf(FooVariant{}),
+		"bar":        reflect.TypeOf(BarVariant{}),
+		"bar_legacy": reflect.TypeOf(BarVariant{}),
+	})
+}
+
+func (m *MultiDiscriminatorUnion) UnmarshalJSON(data []byte) error {
+	return apijson.UnmarshalRoot(data, m)
+}
+
 func (d *DiscriminatedUnion) UnmarshalJSON(data []byte) error {
 	return apijson.UnmarshalRoot(data, d)
 }
@@ -408,3 +438,61 @@ func TestDiscriminatedUnion(t *testing.T) {
 		})
 	}
 }
+
+func TestMultiDiscriminatorUnion(t *testing.T) {
+	tests := map[string]struct {
+		raw        string
+		target     MultiDiscriminatorUnion
+		shouldFail bool
+	}{
+		"foo_variant": {
+			raw: `{"type":"foo","value":"test"}`,
+			target: MultiDiscriminatorUnion{OfFoo: &FooVariant{
+				Type:  "foo",
+				Value: "test",
+			}},
+		},
+		"foo_v2_variant": {
+			raw: `{"type":"foo_v2","value":"test_v2"}`,
+			target: MultiDiscriminatorUnion{OfFoo: &FooVariant{
+				Type:  "foo_v2",
+				Value: "test_v2",
+			}},
+		},
+		"bar_variant": {
+			raw: `{"type":"bar","enable":true}`,
+			target: MultiDiscriminatorUnion{OfBar: &BarVariant{
+				Type:   "bar",
+				Enable: true,
+			}},
+		},
+		"bar_legacy_variant": {
+			raw: `{"type":"bar_legacy","enable":false}`,
+			target: MultiDiscriminatorUnion{OfBar: &BarVariant{
+				Type:   "bar_legacy",
+				Enable: false,
+			}},
+		},
+		"invalid_type": {
+			raw:        `{"type":"unknown","value":"test"}`,
+			target:     MultiDiscriminatorUnion{},
+			shouldFail: true,
+		},
+	}
+
+	for name, test := range tests {
+		t.Run(name, func(t *testing.T) {
+			var dst MultiDiscriminatorUnion
+			err := json.Unmarshal([]byte(test.raw), &dst)
+			if err != nil && !test.shouldFail {
+				t.Fatalf("failed unmarshal with err: %v", err)
+			}
+			if err == nil && test.shouldFail {
+				t.Fatalf("expected unmarshal to fail but it succeeded")
+			}
+			if !reflect.DeepEqual(dst, test.target) {
+				t.Fatalf("failed equality, got %#v but expected %#v", dst, test.target)
+			}
+		})
+	}
+}

internal/apijson/union.go

@@ -39,12 +39,10 @@ func RegisterDiscriminatedUnion[T any](key string, mappings map[string]reflect.T
 
 func (d *decoderBuilder) newStructUnionDecoder(t reflect.Type) decoderFunc {
 	type variantDecoder struct {
-		decoder            decoderFunc
-		field              reflect.StructField
-		discriminatorValue any
+		decoder decoderFunc
+		field   reflect.StructField
 	}
-
-	variants := []variantDecoder{}
+	decoders := []variantDecoder{}
 	for i := 0; i < t.NumField(); i++ {
 		field := t.Field(i)
 
@@ -53,18 +51,26 @@ func (d *decoderBuilder) newStructUnionDecoder(t reflect.Type) decoderFunc {
 		}
 
 		decoder := d.typeDecoder(field.Type)
-		variants = append(variants, variantDecoder{
+		decoders = append(decoders, variantDecoder{
 			decoder: decoder,
 			field:   field,
 		})
 	}
 
+	type discriminatedDecoder struct {
+		variantDecoder
+		discriminator any
+	}
+	discriminatedDecoders := []discriminatedDecoder{}
 	unionEntry, discriminated := unionRegistry[t]
-	for _, unionVariant := range unionEntry.variants {
-		for i := 0; i < len(variants); i++ {
-			variant := &variants[i]
-			if variant.field.Type.Elem() == unionVariant.Type {
-				variant.discriminatorValue = unionVariant.DiscriminatorValue
+	for _, variant := range unionEntry.variants {
+		// For each union variant, find a matching decoder and save it
+		for _, decoder := range decoders {
+			if decoder.field.Type.Elem() == variant.Type {
+				discriminatedDecoders = append(discriminatedDecoders, discriminatedDecoder{
+					decoder,
+					variant.DiscriminatorValue,
+				})
 				break
 			}
 		}
@@ -73,10 +79,10 @@ func (d *decoderBuilder) newStructUnionDecoder(t reflect.Type) decoderFunc {
 	return func(n gjson.Result, v reflect.Value, state *decoderState) error {
 		if discriminated && n.Type == gjson.JSON && len(unionEntry.discriminatorKey) != 0 {
 			discriminator := n.Get(unionEntry.discriminatorKey).Value()
-			for _, variant := range variants {
-				if discriminator == variant.discriminatorValue {
-					inner := v.FieldByIndex(variant.field.Index)
-					return variant.decoder(n, inner, state)
+			for _, decoder := range discriminatedDecoders {
+				if discriminator == decoder.discriminator {
+					inner := v.FieldByIndex(decoder.field.Index)
+					return decoder.decoder(n, inner, state)
 				}
 			}
 			return errors.New("apijson: was not able to find discriminated union variant")
@@ -85,15 +91,15 @@ func (d *decoderBuilder) newStructUnionDecoder(t reflect.Type) decoderFunc {
 		// Set bestExactness to worse than loose
 		bestExactness := loose - 1
 		bestVariant := -1
-		for i, variant := range variants {
+		for i, decoder := range decoders {
 			// Pointers are used to discern JSON object variants from value variants
-			if n.Type != gjson.JSON && variant.field.Type.Kind() == reflect.Ptr {
+			if n.Type != gjson.JSON && decoder.field.Type.Kind() == reflect.Ptr {
 				continue
 			}
 
 			sub := decoderState{strict: state.strict, exactness: exact}
-			inner := v.FieldByIndex(variant.field.Index)
-			err := variant.decoder(n, inner, &sub)
+			inner := v.FieldByIndex(decoder.field.Index)
+			err := decoder.decoder(n, inner, &sub)
 			if err != nil {
 				continue
 			}
@@ -116,11 +122,11 @@ func (d *decoderBuilder) newStructUnionDecoder(t reflect.Type) decoderFunc {
 			return errors.New("apijson: was not able to coerce type as union strictly")
 		}
 
-		for i := 0; i < len(variants); i++ {
+		for i := 0; i < len(decoders); i++ {
 			if i == bestVariant {
 				continue
 			}
-			v.FieldByIndex(variants[i].field.Index).SetZero()
+			v.FieldByIndex(decoders[i].field.Index).SetZero()
 		}
 
 		return nil

internal/requestconfig/requestconfig.go

@@ -461,6 +461,11 @@ func (cfg *RequestConfig) Execute() (err error) {
 			break
 		}
 
+		// Close the response body before retrying to prevent connection leaks
+		if res != nil && res.Body != nil {
+			res.Body.Close()
+		}
+
 		time.Sleep(retryDelay(res, retryCount))
 	}
 

internal/version.go

@@ -2,4 +2,4 @@
 
 package internal
 
-const PackageVersion = "0.0.2" // x-release-please-version
+const PackageVersion = "0.0.3" // x-release-please-version