Fix swagger ui in dev mode and package naming

Signed-off-by: ppadti <ppadti@redhat.com>

Author: ppadti

clients/ui/bff/internal/api/app.go

@@ -281,8 +281,10 @@ func (app *App) Routes() http.Handler {
 	apiRouter.POST(CheckNamespaceRegistryAccessPath, app.CheckNamespaceRegistryAccessHandler)
 	apiRouter.GET(ModelRegistryListPath, app.AttachNamespace(app.RequireListServiceAccessInNamespace(app.GetAllModelRegistriesHandler)))
 
-	// Swagger UI (interactive API docs)
-	apiRouter.GET(SwaggerPath+"/*filepath", app.GetSwaggerHandler)
+	// Swagger UI (interactive API docs) — only in dev mode
+	if app.config.DevMode {
+		apiRouter.GET(SwaggerPath+"/*filepath", app.GetSwaggerHandler)
+	}
 
 	// Enable these routes in all cases except Kubeflow integration mode
 	// (Kubeflow integration mode is when DeploymentMode is kubeflow)

clients/ui/bff/internal/api/healthcheck__handler_test.go

@@ -9,7 +9,7 @@ import (
 
 	"github.com/kubeflow/model-registry/ui/bff/internal/config"
 	"github.com/kubeflow/model-registry/ui/bff/internal/mocks"
-	"github.com/kubeflow/model-registry/ui/bff/internal/models/health_check"
+	"github.com/kubeflow/model-registry/ui/bff/internal/models/healthcheck"
 	"github.com/kubeflow/model-registry/ui/bff/internal/repositories"
 	"github.com/stretchr/testify/assert"
 )
@@ -35,15 +35,15 @@ func TestHealthCheckHandler(t *testing.T) {
 	body, err := io.ReadAll(rs.Body)
 	assert.NoError(t, err)
 
-	var healthCheckRes health_check.HealthCheckModel
+	var healthCheckRes healthcheck.HealthCheckModel
 	err = json.Unmarshal(body, &healthCheckRes)
 	assert.NoError(t, err)
 
 	assert.Equal(t, http.StatusOK, rr.Code)
 
-	expected := health_check.HealthCheckModel{
+	expected := healthcheck.HealthCheckModel{
 		Status: "available",
-		SystemInfo: health_check.SystemInfo{
+		SystemInfo: healthcheck.SystemInfo{
 			Version: Version,
 		},
 	}

clients/ui/bff/internal/api/healthcheck_handler.go

@@ -6,7 +6,7 @@ import (
 	"github.com/julienschmidt/httprouter"
 
 	// imported for swag documentation
-	_ "github.com/kubeflow/model-registry/ui/bff/internal/models/health_check"
+	_ "github.com/kubeflow/model-registry/ui/bff/internal/models/healthcheck"
 )
 
 // HealthcheckHandler returns the health status of the application.
@@ -16,7 +16,7 @@ import (
 //	@Tags			healthcheck
 //	@ID				getHealthcheck
 //	@Produce		application/json
-//	@Success		200	{object}	health_check.HealthCheckModel	"Successful healthcheck response"
+//	@Success		200	{object}	healthcheck.HealthCheckModel	"Successful healthcheck response"
 //	@Failure		401	{object}	ErrorEnvelope					"Unauthorized. Authentication is required."
 //	@Failure		403	{object}	ErrorEnvelope					"Forbidden. User does not have permission to access the resource."
 //	@Failure		404	{object}	ErrorEnvelope					"Not Found. Resource does not exist."

clients/ui/bff/internal/api/swagger_handler.go

@@ -1,5 +1,5 @@
 /*
-Copyright 2024.
+Copyright 2026.
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -24,7 +24,7 @@ import (
 )
 
 // GetSwaggerHandler serves the Swagger UI for interactive API documentation.
-func (app *App) GetSwaggerHandler(w http.ResponseWriter, r *http.Request, _ httprouter.Params) { //nolint:unused // receiver required for route signature
+func (app *App) GetSwaggerHandler(w http.ResponseWriter, r *http.Request, _ httprouter.Params) {
 	httpSwagger.Handler(
 		httpSwagger.URL(SwaggerDocPath),
 		httpSwagger.DeepLinking(true),

…rnal/models/health_check/health_check.go …ternal/models/healthcheck/healthcheck.goclients/ui/bff/internal/models/health_check/health_check.go renamed to clients/ui/bff/internal/models/healthcheck/healthcheck.go clients/ui/bff/internal/models/health_check/health_check.go renamed to clients/ui/bff/internal/models/healthcheck/healthcheck.go

@@ -1,4 +1,4 @@
-package health_check
+package healthcheck
 
 type SystemInfo struct {
 	Version string `json:"version"`

clients/ui/bff/internal/repositories/health_check.go

@@ -1,17 +1,17 @@
 package repositories
 
-import "github.com/kubeflow/model-registry/ui/bff/internal/models/health_check"
+import "github.com/kubeflow/model-registry/ui/bff/internal/models/healthcheck"
 
 type HealthCheckRepository struct{}
 
 func NewHealthCheckRepository() *HealthCheckRepository {
 	return &HealthCheckRepository{}
 }
 
-func (r *HealthCheckRepository) HealthCheck(version string) (health_check.HealthCheckModel, error) {
-	return health_check.HealthCheckModel{
+func (r *HealthCheckRepository) HealthCheck(version string) (healthcheck.HealthCheckModel, error) {
+	return healthcheck.HealthCheckModel{
 		Status: "available",
-		SystemInfo: health_check.SystemInfo{
+		SystemInfo: healthcheck.SystemInfo{
 			Version: version,
 		},
 	}, nil

clients/ui/bff/openapi/docs.go

@@ -32,7 +32,7 @@
                     "200": {
                         "description": "Successful healthcheck response",
                         "schema": {
-                            "$ref": "#/definitions/health_check.HealthCheckModel"
+                            "$ref": "#/definitions/healthcheck.HealthCheckModel"
                         }
                     },
                     "401": {
@@ -81,7 +81,7 @@
                 }
             }
         },
-        "health_check.HealthCheckModel": {
+        "healthcheck.HealthCheckModel": {
             "type": "object",
             "required": [
                 "status",
@@ -92,11 +92,11 @@
                     "type": "string"
                 },
                 "system_info": {
-                    "$ref": "#/definitions/health_check.SystemInfo"
+                    "$ref": "#/definitions/healthcheck.SystemInfo"
                 }
             }
         },
-        "health_check.SystemInfo": {
+        "healthcheck.SystemInfo": {
             "type": "object",
             "required": [
                 "version"

clients/ui/bff/openapi/swagger.json

@@ -7,17 +7,17 @@ definitions:
     required:
     - error
     type: object
-  health_check.HealthCheckModel:
+  healthcheck.HealthCheckModel:
     properties:
       status:
         type: string
       system_info:
-        $ref: '#/definitions/health_check.SystemInfo'
+        $ref: '#/definitions/healthcheck.SystemInfo'
     required:
     - status
     - system_info
     type: object
-  health_check.SystemInfo:
+  healthcheck.SystemInfo:
     properties:
       version:
         type: string
@@ -54,7 +54,7 @@ paths:
         "200":
           description: Successful healthcheck response
           schema:
-            $ref: '#/definitions/health_check.HealthCheckModel'
+            $ref: '#/definitions/healthcheck.HealthCheckModel'
         "401":
           description: Unauthorized. Authentication is required.
           schema: