feat(ui-vue3): improve func empty description

Author: Similarityoung

pkg/console/service/service.go

@@ -349,37 +349,20 @@ func buildServiceMethodSummaries(metadataList []*meshresource.ServiceProviderMet
 
 func buildServiceMethodCandidates(metadataList []*meshresource.ServiceProviderMetadataResource) []*serviceMethodCandidate {
 	candidateByKey := make(map[string]*serviceMethodCandidate)
-	structuredMethodNames := make(map[string]struct{})
-	fallbackMethodNames := make(map[string]struct{})
 
 	for _, metadata := range metadataList {
+		if metadata == nil || metadata.Spec == nil {
+			continue
+		}
 		for _, method := range metadata.Spec.Methods {
 			candidate, ok := newStructuredServiceMethodCandidate(method)
 			if !ok {
 				continue
 			}
 			candidateByKey[serviceMethodKey(candidate.detail.MethodName, candidate.signature)] = candidate
-			structuredMethodNames[candidate.detail.MethodName] = struct{}{}
-		}
-
-		if len(metadata.Spec.Methods) > 0 {
-			continue
-		}
-		for _, methodName := range methodNamesFromMetadataParameters(metadata.Spec.Parameters) {
-			fallbackMethodNames[methodName] = struct{}{}
 		}
 	}
 
-	// Legacy metadata may only expose flat method-name parameters without structured method definitions.
-	// Keep those names as a fallback for method listing, but never let them shadow authoritative
-	// structured metadata for the same method name.
-	for methodName := range fallbackMethodNames {
-		if _, ok := structuredMethodNames[methodName]; ok {
-			continue
-		}
-		candidateByKey[serviceMethodKey(methodName, "")] = newFallbackServiceMethodCandidate(methodName)
-	}
-
 	candidates := make([]*serviceMethodCandidate, 0, len(candidateByKey))
 	for _, candidate := range candidateByKey {
 		candidates = append(candidates, candidate)
@@ -429,12 +412,6 @@ func resolveStructuredServiceMethodCandidate(candidates []*serviceMethodCandidat
 		)
 	}
 	if candidate == nil {
-		if hasFallbackOnlyMethodCandidate(candidates, req.MethodName) {
-			return nil, bizerror.New(
-				bizerror.InvalidArgument,
-				fmt.Sprintf("structured metadata not found for method %s of service %s", req.MethodName, req.ServiceName),
-			)
-		}
 		return nil, bizerror.New(
 			bizerror.NotFoundError,
 			fmt.Sprintf("method %s not found for service %s", req.MethodName, req.ServiceName),
@@ -449,15 +426,6 @@ func resolveStructuredServiceMethodCandidate(candidates []*serviceMethodCandidat
 	return candidate, nil
 }
 
-func hasFallbackOnlyMethodCandidate(candidates []*serviceMethodCandidate, methodName string) bool {
-	for _, candidate := range candidates {
-		if candidate.detail.MethodName == methodName && candidate.method == nil {
-			return true
-		}
-	}
-	return false
-}
-
 func newStructuredServiceMethodCandidate(method *meshproto.Method) (*serviceMethodCandidate, bool) {
 	if method == nil {
 		return nil, false
@@ -473,38 +441,6 @@ func newStructuredServiceMethodCandidate(method *meshproto.Method) (*serviceMeth
 	}, true
 }
 
-func newFallbackServiceMethodCandidate(methodName string) *serviceMethodCandidate {
-	return &serviceMethodCandidate{
-		detail: &model.ServiceMethodDetailResp{
-			MethodName:     methodName,
-			Signature:      "",
-			ParameterTypes: []string{},
-			Parameters:     []model.ServiceMethodParameter{},
-			Types:          []model.ServiceMethodTypeResp{},
-		},
-		signature: "",
-	}
-}
-
-func methodNamesFromMetadataParameters(parameters map[string]string) []string {
-	if parameters == nil {
-		return nil
-	}
-	methodsRaw := strings.TrimSpace(parameters["methods"])
-	if strutil.IsBlank(methodsRaw) {
-		return nil
-	}
-	methodNames := make([]string, 0)
-	for _, methodName := range strings.Split(methodsRaw, ",") {
-		methodName = strings.TrimSpace(methodName)
-		if strutil.IsBlank(methodName) {
-			continue
-		}
-		methodNames = append(methodNames, methodName)
-	}
-	return methodNames
-}
-
 func serviceMethodKey(methodName, signature string) string {
 	return methodName + "\x00" + signature
 }

ui-vue3/.env.mock

@@ -1 +1,16 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 VITE_MOCK_ENABLED=true

ui-vue3/src/views/resources/services/tabs/debug.vue

@@ -19,7 +19,7 @@
     <a-card :bordered="false" :body-style="{ padding: '24px' }">
       <div class="tabs-title">方法列表</div>
       <a-spin :spinning="loadingMethods">
-        <a-empty v-if="!loadingMethods && methodList.length === 0" description="暂无方法" />
+        <a-empty v-if="!loadingMethods && methodList.length === 0" :description="emptyDescription" />
         <a-tabs
           v-else
           v-model:activeKey="activeKey"
@@ -175,6 +175,10 @@ import {
 } from '@/api/service/service'
 import { useMeshStore } from '@/stores/mesh'
 
+defineOptions({
+  name: 'ServiceDebugTab'
+})
+
 interface MethodSummary {
   methodName: string
   parameterTypes: string[]
@@ -233,6 +237,16 @@ const timeout = ref(3000)
 const attachmentsModalOpen = ref(false)
 const attachmentsList = ref<{ key: string; value: string }[]>([])
 
+const isMeshSelected = computed(() => Boolean(meshStore.mesh))
+const emptyDescription = computed(() => {
+  if (!serviceName.value) {
+    return '暂无方法'
+  }
+  if (!isMeshSelected.value) {
+    return '请先选择 mesh'
+  }
+  return '暂无可调试方法'
+})
 const attachmentCount = computed(() => attachmentsList.value.filter((a) => a.key).length)
 const providerInstanceOptions = computed(() =>
   providerInstances.value.map((instance) => ({
@@ -392,7 +406,7 @@ function syncSelectedInstance() {
 }
 
 async function loadProviderInstances() {
-  if (!serviceName.value) {
+  if (!serviceName.value || !isMeshSelected.value) {
     providerInstances.value = []
     instanceName.value = ''
     return
@@ -412,7 +426,7 @@ async function loadProviderInstances() {
 }
 
 async function loadMethods() {
-  if (!serviceName.value) {
+  if (!serviceName.value || !isMeshSelected.value) {
     methodList.value = []
     currentMethodDetail.value = null
     requestValue.value = '[]'
@@ -547,7 +561,14 @@ function removeAttachment(idx: number) {
 
 async function loadPageData() {
   responseValue.value = ''
-  await Promise.all([loadProviderInstances(), loadMethods()])
+  if (!isMeshSelected.value) {
+    return
+  }
+  try {
+    await Promise.all([loadProviderInstances(), loadMethods()])
+  } catch (error) {
+    console.error('load debug page data failed', error)
+  }
 }
 
 watch(
@@ -557,6 +578,8 @@ watch(
     providerInstances.value = []
     currentMethodDetail.value = null
     instanceName.value = ''
+    requestValue.value = '[]'
+    responseValue.value = ''
     void loadPageData()
   },
   { immediate: true }