fix: handle swagger:type array by falling through to underlying type resolution

When swagger:type is set to a value not handled by swaggerSchemaForType
(e.g., "array"), the function returns an error. Previously this error was
silently ignored and the function returned nil, losing all type info.

Now, when swaggerSchemaForType returns an error, the code falls through
to buildFromType with the underlying type, producing the correct schema.
For example, swagger:type array on type StringSlice []string now produces
{type: "array", items: {type: "string"}} with the field's description
preserved, instead of a $ref that drops the description.

* fixes #10

Signed-off-by: Kevin Doan <kevin.doan@ory.sh>
Signed-off-by: KT-Doan <kevin.doan@ory.sh>

Author: KT-Doan

application_test.go

@@ -51,7 +51,7 @@ func TestMain(m *testing.M) {
 func TestApplication_LoadCode(t *testing.T) {
 	sctx := loadClassificationPkgsCtx(t)
 	require.NotNil(t, sctx)
-	require.Len(t, sctx.app.Models, 39)
+	require.Len(t, sctx.app.Models, 40)
 	require.Len(t, sctx.app.Meta, 1)
 	require.Len(t, sctx.app.Routes, 7)
 	require.Empty(t, sctx.app.Operations)

fixtures/goparsing/classification/models/nomodel.go

@@ -799,3 +799,13 @@ type NamedMapOfStoreOrderSlices GenericMap[string, GenericSlice[StoreOrder]]
 //
 // End of models related to named types with type arguments
 //
+
+// SomeStringSlice is a named slice type with swagger:type array.
+// swagger:type array
+type SomeStringSlice []string
+
+// swagger:model namedWithArrayType
+type NamedWithArrayType struct {
+	// Tags for this item.
+	Tags SomeStringSlice `json:"tags"`
+}

schema.go

@@ -429,10 +429,15 @@ func (s *schemaBuilder) buildNamedType(titpe *types.Named, tgt swaggerTypable) e
 		cmt = new(ast.CommentGroup)
 	}
 
-	if typeName, ok := typeName(cmt); ok {
-		_ = swaggerSchemaForType(typeName, tgt)
-
-		return nil
+	if tn, ok := typeName(cmt); ok {
+		if err := swaggerSchemaForType(tn, tgt); err == nil {
+			return nil
+		}
+		// For unsupported swagger:type values (e.g., "array"), fall through
+		// to underlying type resolution so the full schema (including items
+		// for slices) is properly built. Build directly from the underlying
+		// type to bypass the named-type $ref creation.
+		return s.buildFromType(titpe.Underlying(), tgt)
 	}
 
 	if s.decl.Spec.Assign.IsValid() {
@@ -559,9 +564,12 @@ func (s *schemaBuilder) buildNamedStruct(tio *types.TypeName, cmt *ast.CommentGr
 		return nil
 	}
 
-	if typeName, ok := typeName(cmt); ok {
-		_ = swaggerSchemaForType(typeName, tgt)
-		return nil
+	if tn, ok := typeName(cmt); ok {
+		if err := swaggerSchemaForType(tn, tgt); err == nil {
+			return nil
+		}
+		// For unsupported swagger:type values, fall through to makeRef
+		// rather than silently returning an empty schema.
 	}
 
 	return s.makeRef(decl, tgt)
@@ -583,6 +591,14 @@ func (s *schemaBuilder) buildNamedArray(tio *types.TypeName, cmt *ast.CommentGro
 		tgt.Items().Typed("string", sfnm)
 		return nil
 	}
+	// When swagger:type is set to an unsupported value (e.g., "array"),
+	// skip the $ref and inline the array schema with proper items type.
+	if tn, ok := typeName(cmt); ok {
+		if err := swaggerSchemaForType(tn, tgt); err != nil {
+			return s.buildFromType(elem, tgt.Items())
+		}
+		return nil
+	}
 	if decl, ok := s.ctx.FindModel(tio.Pkg().Path(), tio.Name()); ok {
 		return s.makeRef(decl, tgt)
 	}
@@ -600,6 +616,16 @@ func (s *schemaBuilder) buildNamedSlice(tio *types.TypeName, cmt *ast.CommentGro
 		tgt.Items().Typed("string", sfnm)
 		return nil
 	}
+	// When swagger:type is set to an unsupported value (e.g., "array"),
+	// skip the $ref and inline the slice schema with proper items type.
+	// This preserves the field's description that would be lost with $ref.
+	if tn, ok := typeName(cmt); ok {
+		if err := swaggerSchemaForType(tn, tgt); err != nil {
+			// Unsupported type name (e.g., "array") — build inline from element type.
+			return s.buildFromType(elem, tgt.Items())
+		}
+		return nil
+	}
 	if decl, ok := s.ctx.FindModel(tio.Pkg().Path(), tio.Name()); ok {
 		return s.makeRef(decl, tgt)
 	}

schema_test.go

@@ -2582,3 +2582,20 @@ func TestSetEnumDoesNotPanic(t *testing.T) {
 
 	require.NoError(t, err)
 }
+
+func TestSwaggerTypeNamedArray(t *testing.T) {
+	sctx := loadClassificationPkgsCtx(t)
+	decl := getClassificationModel(sctx, "NamedWithArrayType")
+	require.NotNil(t, decl)
+	prs := &schemaBuilder{
+		ctx:  sctx,
+		decl: decl,
+	}
+	models := make(map[string]spec.Schema)
+	require.NoError(t, prs.Build(models))
+	schema := models["namedWithArrayType"]
+
+	// swagger:type array on a named []string type should produce
+	// an inlined array with string items, not a $ref.
+	assertArrayProperty(t, &schema, "string", "tags", "", "Tags")
+}