Skip to content

Commit d3cd6a3

Browse files
patnikoCopilot
andcommitted
Revert ToolName addition to Go PermissionRequest — use Extra map instead
The ToolName field doesn't exist on PermissionRequest in other SDKs. The scenario test now reads toolName from the Extra map to stay consistent without modifying SDK types. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 39f0ec4 commit d3cd6a3

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

go/types.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,6 @@ type SystemMessageConfig struct {
103103
type PermissionRequest struct {
104104
Kind string `json:"kind"`
105105
ToolCallID string `json:"toolCallId,omitempty"`
106-
ToolName string `json:"toolName,omitempty"`
107106
Extra map[string]any `json:"-"` // Additional fields vary by kind
108107
}
109108

test/scenarios/callbacks/permissions/go/main.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ func main() {
3030
Model: "claude-haiku-4.5",
3131
OnPermissionRequest: func(req copilot.PermissionRequest, inv copilot.PermissionInvocation) (copilot.PermissionRequestResult, error) {
3232
permissionLogMu.Lock()
33-
permissionLog = append(permissionLog, fmt.Sprintf("approved:%s", req.ToolName))
33+
toolName, _ := req.Extra["toolName"].(string)
34+
permissionLog = append(permissionLog, fmt.Sprintf("approved:%s", toolName))
3435
permissionLogMu.Unlock()
3536
return copilot.PermissionRequestResult{Kind: "approved"}, nil
3637
},

0 commit comments

Comments
 (0)