fix: improve schema name generation with mutex for thread safety

Author: sattvikc

pkg/spec/main.go

@@ -60,11 +60,14 @@ func New() *Spec {
 				},
 			),
 			openapi3gen.CreateTypeNameGenerator(func(t reflect.Type) string {
+				var name string = ""
 				if t.Name() == "" {
-					return ""
+					name = generateSchemaName()
+				} else {
+					name = t.Name()
 				}
 
-				name := t.PkgPath() + "_" + t.Name()
+				name = t.PkgPath() + "_" + name
 
 				name = strings.ReplaceAll(name, "github.com/go-simpl/simplapi/example/", "")
 				name = strings.ReplaceAll(name, "/", "_")

pkg/spec/utils.go

@@ -1,8 +1,10 @@
 package spec
 
 import (
+	"fmt"
 	"mime/multipart"
 	"reflect"
+	"sync"
 
 	"github.com/getkin/kin-openapi/openapi3"
 )
@@ -33,3 +35,13 @@ func getSchemaForType(t reflect.Type) *openapi3.Schema {
 
 	return openapi3.NewStringSchema()
 }
+
+var schemaNameCount int = 0
+var schemaNameMutex sync.Mutex
+
+func generateSchemaName() string {
+	schemaNameMutex.Lock()
+	defer schemaNameMutex.Unlock()
+	schemaNameCount++
+	return fmt.Sprintf("Schema%d", schemaNameCount)
+}