feat: rename SkillAllowedToolCodes to ToolWhitelist and ToolCodes to ExposedToolCodes for consistency across responses

This commit is contained in:
mlogclub
2026-04-12 10:17:27 +08:00
parent 44b20813fe
commit dce9c5eb25
4 changed files with 31 additions and 31 deletions
+4 -4
View File
@@ -140,8 +140,8 @@ func buildSkillDebugRunResponse(req request.SkillDebugRunRequest, summary *Summa
resp.ReplyText = summary.ReplyText resp.ReplyText = summary.ReplyText
resp.PlanReason = summary.PlanReason resp.PlanReason = summary.PlanReason
resp.SkillRouteTrace = summary.SkillRouteTrace resp.SkillRouteTrace = summary.SkillRouteTrace
resp.SkillAllowedToolCodes = append([]string(nil), summary.SkillAllowedToolCodes...) resp.ToolWhitelist = append([]string(nil), summary.SkillAllowedToolCodes...)
resp.ToolCodes = append([]string(nil), summary.ToolCodes...) resp.ExposedToolCodes = append([]string(nil), summary.ToolCodes...)
resp.InvokedToolCodes = append([]string(nil), summary.InvokedToolCodes...) resp.InvokedToolCodes = append([]string(nil), summary.InvokedToolCodes...)
resp.ToolSearchTrace = extractToolSearchTrace(summary) resp.ToolSearchTrace = extractToolSearchTrace(summary)
resp.GraphToolTrace = extractGraphToolTrace(summary) resp.GraphToolTrace = extractGraphToolTrace(summary)
@@ -167,8 +167,8 @@ func buildSkillDebugResumeResponse(req request.SkillDebugResumeRequest, summary
resp.ReplyText = summary.ReplyText resp.ReplyText = summary.ReplyText
resp.PlanReason = summary.PlanReason resp.PlanReason = summary.PlanReason
resp.SkillRouteTrace = summary.SkillRouteTrace resp.SkillRouteTrace = summary.SkillRouteTrace
resp.SkillAllowedToolCodes = append([]string(nil), summary.SkillAllowedToolCodes...) resp.ToolWhitelist = append([]string(nil), summary.SkillAllowedToolCodes...)
resp.ToolCodes = append([]string(nil), summary.ToolCodes...) resp.ExposedToolCodes = append([]string(nil), summary.ToolCodes...)
resp.InvokedToolCodes = append([]string(nil), summary.InvokedToolCodes...) resp.InvokedToolCodes = append([]string(nil), summary.InvokedToolCodes...)
resp.ToolSearchTrace = extractToolSearchTrace(summary) resp.ToolSearchTrace = extractToolSearchTrace(summary)
resp.GraphToolTrace = extractGraphToolTrace(summary) resp.GraphToolTrace = extractGraphToolTrace(summary)
+18 -18
View File
@@ -21,24 +21,24 @@ type SkillDefinitionResponse struct {
} }
type SkillDebugRunResponse struct { type SkillDebugRunResponse struct {
SkillCode string `json:"skillCode"` SkillCode string `json:"skillCode"`
SkillName string `json:"skillName"` SkillName string `json:"skillName"`
ReplyText string `json:"replyText"` ReplyText string `json:"replyText"`
PlanReason string `json:"planReason"` PlanReason string `json:"planReason"`
SkillRouteTrace string `json:"skillRouteTrace"` SkillRouteTrace string `json:"skillRouteTrace"`
SkillAllowedToolCodes []string `json:"skillAllowedToolCodes"` ToolWhitelist []string `json:"toolWhitelist"`
ToolCodes []string `json:"toolCodes"` ExposedToolCodes []string `json:"exposedToolCodes"`
InvokedToolCodes []string `json:"invokedToolCodes"` InvokedToolCodes []string `json:"invokedToolCodes"`
ToolSearchTrace string `json:"toolSearchTrace"` ToolSearchTrace string `json:"toolSearchTrace"`
GraphToolTrace string `json:"graphToolTrace"` GraphToolTrace string `json:"graphToolTrace"`
GraphToolCode string `json:"graphToolCode"` GraphToolCode string `json:"graphToolCode"`
InterruptType string `json:"interruptType"` InterruptType string `json:"interruptType"`
CheckPointID string `json:"checkPointId"` CheckPointID string `json:"checkPointId"`
Interrupted bool `json:"interrupted"` Interrupted bool `json:"interrupted"`
TraceData string `json:"traceData"` TraceData string `json:"traceData"`
ErrorMessage string `json:"errorMessage"` ErrorMessage string `json:"errorMessage"`
ConversationID int64 `json:"conversationId"` ConversationID int64 `json:"conversationId"`
AIAgentID int64 `json:"aiAgentId"` AIAgentID int64 `json:"aiAgentId"`
} }
type AgentRunLogResponse struct { type AgentRunLogResponse struct {
@@ -403,10 +403,10 @@ function DebugDialogBody({
</CardHeader> </CardHeader>
<CardContent className="space-y-3 text-sm"> <CardContent className="space-y-3 text-sm">
<div className="rounded-lg bg-muted/50 p-3"> <div className="rounded-lg bg-muted/50 p-3">
<div className="text-xs text-muted-foreground">Skill Allowed Tools</div> <div className="text-xs text-muted-foreground"></div>
<div className="mt-2 flex flex-wrap gap-2"> <div className="mt-2 flex flex-wrap gap-2">
{(result?.skillAllowedToolCodes ?? []).length > 0 ? ( {(result?.toolWhitelist ?? []).length > 0 ? (
result?.skillAllowedToolCodes.map((toolCode) => ( result?.toolWhitelist.map((toolCode) => (
<Badge key={toolCode} variant="outline"> <Badge key={toolCode} variant="outline">
{toolCode} {toolCode}
</Badge> </Badge>
@@ -417,10 +417,10 @@ function DebugDialogBody({
</div> </div>
</div> </div>
<div className="rounded-lg bg-muted/50 p-3"> <div className="rounded-lg bg-muted/50 p-3">
<div className="text-xs text-muted-foreground">Candidate Tools</div> <div className="text-xs text-muted-foreground"></div>
<div className="mt-2 flex flex-wrap gap-2"> <div className="mt-2 flex flex-wrap gap-2">
{(result?.toolCodes ?? []).length > 0 ? ( {(result?.exposedToolCodes ?? []).length > 0 ? (
result?.toolCodes.map((toolCode) => ( result?.exposedToolCodes.map((toolCode) => (
<Badge key={toolCode} variant="outline"> <Badge key={toolCode} variant="outline">
{toolCode} {toolCode}
</Badge> </Badge>
@@ -431,7 +431,7 @@ function DebugDialogBody({
</div> </div>
</div> </div>
<div className="rounded-lg bg-muted/50 p-3"> <div className="rounded-lg bg-muted/50 p-3">
<div className="text-xs text-muted-foreground">Invoked Tools</div> <div className="text-xs text-muted-foreground"></div>
<div className="mt-2 flex flex-wrap gap-2"> <div className="mt-2 flex flex-wrap gap-2">
{(result?.invokedToolCodes ?? []).length > 0 ? ( {(result?.invokedToolCodes ?? []).length > 0 ? (
result?.invokedToolCodes.map((toolCode) => ( result?.invokedToolCodes.map((toolCode) => (
+2 -2
View File
@@ -321,8 +321,8 @@ export type SkillDebugRunResult = {
replyText: string replyText: string
planReason: string planReason: string
skillRouteTrace: string skillRouteTrace: string
skillAllowedToolCodes: string[] toolWhitelist: string[]
toolCodes: string[] exposedToolCodes: string[]
invokedToolCodes: string[] invokedToolCodes: string[]
toolSearchTrace: string toolSearchTrace: string
graphToolTrace: string graphToolTrace: string