chore(catalog): allow empty sources list (kubeflow#1957)

pboyd · web-flow · commit 55c50d0d8cf4 · 2025-12-05T17:49:36.000Z
Always return items from the sources endpoint, even if it's an empty
list.

Signed-off-by: Paul Boyd &lt;paul@pboyd.io&gt;
diff --git a/api/openapi/catalog.yaml b/api/openapi/catalog.yaml
@@ -606,6 +606,8 @@ components:
               type: array
               items:
                 $ref: "#/components/schemas/CatalogSource"
+          required:
+            - items
         - $ref: "#/components/schemas/BaseResourceList"
     Error:
       description: Error code and message.
diff --git a/api/openapi/src/catalog.yaml b/api/openapi/src/catalog.yaml
@@ -489,6 +489,8 @@ components:
               type: array
               items:
                 $ref: "#/components/schemas/CatalogSource"
+          required:
+            - items
         - $ref: "#/components/schemas/BaseResourceList"
     FilterOption:
       type: object
diff --git a/catalog/internal/server/openapi/type_asserts.go b/catalog/internal/server/openapi/type_asserts.go
@@ -262,6 +262,7 @@ func AssertCatalogSourceListRequired(obj model.CatalogSourceList) error {
 		"nextPageToken": obj.NextPageToken,
 		"pageSize":      obj.PageSize,
 		"size":          obj.Size,
+		"items":         obj.Items,
 	}
 	for name, el := range elements {
 		if isZero := IsZeroValue(el); isZero {
diff --git a/catalog/pkg/openapi/model_catalog_source_list.go b/catalog/pkg/openapi/model_catalog_source_list.go

Original file line number	Diff line number	Diff line change
`@@ -262,6 +262,7 @@ func AssertCatalogSourceListRequired(obj model.CatalogSourceList) error {`
`262`	`262`	`"nextPageToken": obj.NextPageToken,`
`263`	`263`	`"pageSize": obj.PageSize,`
`264`	`264`	`"size": obj.Size,`
	`265`	`+ "items": obj.Items,`
`265`	`266`	`}`
`266`	`267`	`for name, el := range elements {`
`267`	`268`	`if isZero := IsZeroValue(el); isZero {`