Skip to content

Commit 9ec25ef

Browse files
Regenerate all codegen including session events, fix Go DataContent
The codegen check requires all generated files to match. Regenerate session events alongside RPC types. The Go session events now have DataContent (union type) for Content field — update all Go consumers to use .Content.String instead of .Content directly. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 8d1af43 commit 9ec25ef

43 files changed

Lines changed: 177 additions & 136 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

go/README.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,8 @@ func main() {
5858
done := make(chan bool)
5959
session.On(func(event copilot.SessionEvent) {
6060
if event.Type == "assistant.message" {
61-
if event.Data.Content != nil {
62-
fmt.Println(*event.Data.Content)
61+
if event.Data.Content != nil && event.Data.Content.String != nil {
62+
fmt.Println(*event.Data.Content.String)
6363
}
6464
}
6565
if event.Type == "session.idle" {
@@ -417,14 +417,14 @@ func main() {
417417
} else if event.Type == "assistant.message" {
418418
// Final message - complete content
419419
fmt.Println("\n--- Final message ---")
420-
if event.Data.Content != nil {
421-
fmt.Println(*event.Data.Content)
420+
if event.Data.Content != nil && event.Data.Content.String != nil {
421+
fmt.Println(*event.Data.Content.String)
422422
}
423423
} else if event.Type == "assistant.reasoning" {
424424
// Final reasoning content (if model supports reasoning)
425425
fmt.Println("--- Reasoning ---")
426-
if event.Data.Content != nil {
427-
fmt.Println(*event.Data.Content)
426+
if event.Data.Content != nil && event.Data.Content.String != nil {
427+
fmt.Println(*event.Data.Content.String)
428428
}
429429
}
430430
if event.Type == "session.idle" {

go/generated_session_events.go

Lines changed: 82 additions & 41 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

go/internal/e2e/compaction_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ func TestCompaction(t *testing.T) {
8484
if err != nil {
8585
t.Fatalf("Failed to send verification message: %v", err)
8686
}
87-
if answer.Data.Content == nil || !strings.Contains(strings.ToLower(*answer.Data.Content), "dragon") {
87+
if answer.Data.Content == nil || !strings.Contains(strings.ToLower(*answer.Data.Content.String), "dragon") {
8888
t.Errorf("Expected answer to contain 'dragon', got %v", answer.Data.Content)
8989
}
9090
})

go/internal/e2e/mcp_and_agents_test.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ func TestMCPServers(t *testing.T) {
5151
t.Fatalf("Failed to get final message: %v", err)
5252
}
5353

54-
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "4") {
54+
if message.Data.Content == nil || !strings.Contains(*message.Data.Content.String, "4") {
5555
t.Errorf("Expected message to contain '4', got: %v", message.Data.Content)
5656
}
5757

@@ -100,7 +100,7 @@ func TestMCPServers(t *testing.T) {
100100
t.Fatalf("Failed to send message: %v", err)
101101
}
102102

103-
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "6") {
103+
if message.Data.Content == nil || !strings.Contains(*message.Data.Content.String, "6") {
104104
t.Errorf("Expected message to contain '6', got: %v", message.Data.Content)
105105
}
106106

@@ -146,7 +146,7 @@ func TestMCPServers(t *testing.T) {
146146
t.Fatalf("Failed to send message: %v", err)
147147
}
148148

149-
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "hunter2") {
149+
if message.Data.Content == nil || !strings.Contains(*message.Data.Content.String, "hunter2") {
150150
t.Errorf("Expected message to contain 'hunter2', got: %v", message.Data.Content)
151151
}
152152

@@ -231,7 +231,7 @@ func TestCustomAgents(t *testing.T) {
231231
t.Fatalf("Failed to get final message: %v", err)
232232
}
233233

234-
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "10") {
234+
if message.Data.Content == nil || !strings.Contains(*message.Data.Content.String, "10") {
235235
t.Errorf("Expected message to contain '10', got: %v", message.Data.Content)
236236
}
237237

@@ -280,7 +280,7 @@ func TestCustomAgents(t *testing.T) {
280280
t.Fatalf("Failed to send message: %v", err)
281281
}
282282

283-
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "12") {
283+
if message.Data.Content == nil || !strings.Contains(*message.Data.Content.String, "12") {
284284
t.Errorf("Expected message to contain '12', got: %v", message.Data.Content)
285285
}
286286

@@ -441,7 +441,7 @@ func TestCombinedConfiguration(t *testing.T) {
441441
t.Fatalf("Failed to get final message: %v", err)
442442
}
443443

444-
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "14") {
444+
if message.Data.Content == nil || !strings.Contains(*message.Data.Content.String, "14") {
445445
t.Errorf("Expected message to contain '14', got: %v", message.Data.Content)
446446
}
447447

0 commit comments

Comments
 (0)