fix(service): resolve rebase duplicates and drop versionGroups detail

Author: mochengqian

pkg/console/handler/service.go

@@ -269,25 +269,6 @@ func GetServiceGraph(ctx consolectx.Context) gin.HandlerFunc {
 	}
 }
 
-// GetServiceDetail returns service detail information
-func GetServiceDetail(ctx consolectx.Context) gin.HandlerFunc {
-	return func(c *gin.Context) {
-		req := &model.ServiceDetailReq{}
-		if err := c.ShouldBindQuery(req); err != nil {
-			c.JSON(http.StatusBadRequest, model.NewErrorResp(err.Error()))
-			return
-		}
-
-		resp, err := service.GetServiceDetail(ctx, req)
-		if err != nil {
-			util.HandleServiceError(c, err)
-			return
-		}
-
-		c.JSON(http.StatusOK, model.NewSuccessResp(resp))
-	}
-}
-
 // GetServiceInterfaces returns service interfaces information
 func GetServiceInterfaces(ctx consolectx.Context) gin.HandlerFunc {
 	return func(c *gin.Context) {

pkg/console/model/service.go

@@ -174,25 +174,6 @@ func ParseServiceKey(serviceKey string) (serviceName, version, group string, err
 	return serviceName, version, group, nil
 }
 
-type ServiceDetailReq struct {
-	ServiceName string `form:"serviceName" json:"serviceName" binding:"required"`
-	Version     string `form:"version" json:"version"`
-	Group       string `form:"group" json:"group"`
-	Mesh        string `form:"mesh" json:"mesh" binding:"required"`
-}
-
-type VersionGroup struct {
-	Version string `json:"version"`
-	Group   string `json:"group"`
-}
-
-type ServiceDetailResp struct {
-	VersionGroups []*VersionGroup `json:"versionGroups"`
-	AvgRT         string          `json:"avgRT"`
-	AvgQPS        string          `json:"avgQPS"`
-	RequestTotal  string          `json:"requestTotal"`
-}
-
 type ServiceInterfacesReq struct {
 	ServiceName string `form:"serviceName" json:"serviceName" binding:"required"`
 	Mesh        string `form:"mesh" json:"mesh" binding:"required"`

pkg/console/service/service.go

@@ -18,7 +18,6 @@
 package service
 
 import (
-	"encoding/json"
 	"strconv"
 	"strings"
 
@@ -673,65 +672,6 @@ func SearchServiceAsCrossLinkedList(ctx consolectx.Context, req *model.ServiceGr
 	}, nil
 }
 
-// GetServiceDetail get service detail information including version groups and metrics
-func GetServiceDetail(ctx consolectx.Context, req *model.ServiceDetailReq) (*model.ServiceDetailResp, error) {
-	// Query all service provider metadata resources for the given service name
-	indexes := map[string]string{
-		index.ByServiceProviderServiceName: req.ServiceName,
-	}
-	if strutil.IsNotBlank(req.Mesh) {
-		indexes[index.ByMeshIndex] = req.Mesh
-	}
-
-	serviceResources, err := manager.ListByIndexes[*meshresource.ServiceProviderMetadataResource](
-		ctx.ResourceManager(),
-		meshresource.ServiceProviderMetadataKind,
-		indexes,
-	)
-	byteJsonStr, err := json.Marshal(serviceResources)
-	logger.Infof("service resources for service %s: %s", req.ServiceName, string(byteJsonStr))
-	if err != nil {
-		logger.Errorf("get service provider metadata for %s failed, cause: %v", req.ServiceName, err)
-		return nil, bizerror.New(bizerror.InternalError, "get service provider failed")
-	}
-
-	if len(serviceResources) == 0 {
-		logger.Warnf("service %s not found", req.ServiceName)
-		return nil, bizerror.New(bizerror.NotFoundError, "service not found")
-	}
-
-	// Collect unique version and group combinations
-	versionGroupMap := make(map[string]*model.VersionGroup)
-	for _, res := range serviceResources {
-		if res.Spec == nil {
-			continue
-		}
-		key := res.Spec.Version + "|" + res.Spec.Group
-		if _, exists := versionGroupMap[key]; !exists {
-			versionGroupMap[key] = &model.VersionGroup{
-				Version: res.Spec.Version,
-				Group:   res.Spec.Group,
-			}
-		}
-	}
-
-	versionGroups := make([]*model.VersionGroup, 0)
-	for _, vg := range versionGroupMap {
-		versionGroups = append(versionGroups, vg)
-	}
-
-	// Return service detail response with mock metrics data
-	// In production, metrics would come from Prometheus or a metrics store
-	resp := &model.ServiceDetailResp{
-		VersionGroups: versionGroups,
-		AvgRT:         "96ms",
-		AvgQPS:        "12.5",
-		RequestTotal:  "1386",
-	}
-
-	return resp, nil
-}
-
 // GetServiceInterfaces get service interfaces
 func GetServiceInterfaces(ctx consolectx.Context, req *model.ServiceInterfacesReq) (*model.ServiceInterfacesResp, error) {
 	// Query all service provider metadata resources for the given service name