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>

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() {

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")
+}