Fix integration test: check run_ids_or_urls (plural) in audit error envelope (#28519)

Copilot · web-flow · commit bd91ffc7c1c5 · 2026-04-25T16:34:02.000-07:00
diff --git a/pkg/cli/mcp_server_error_codes_test.go b/pkg/cli/mcp_server_error_codes_test.go
@@ -194,8 +194,8 @@ func TestMCPServer_ErrorCodes_InternalError(t *testing.T) {
 		if _, hasError := envelope["error"]; !hasError {
 			t.Errorf("Expected 'error' field in audit JSON response, got: %s", textContent.Text)
 		}
-		if _, hasRunID := envelope["run_id_or_url"]; !hasRunID {
-			t.Errorf("Expected 'run_id_or_url' field in audit JSON response, got: %s", textContent.Text)
+		if _, hasRunID := envelope["run_ids_or_urls"]; !hasRunID {
+			t.Errorf("Expected 'run_ids_or_urls' field in audit JSON response, got: %s", textContent.Text)
 		}
 		if _, hasSuggestions := envelope["suggestions"]; !hasSuggestions {
 			t.Errorf("Expected 'suggestions' field in audit JSON response, got: %s", textContent.Text)

Original file line number	Diff line number	Diff line change
`@@ -194,8 +194,8 @@ func TestMCPServer_ErrorCodes_InternalError(t *testing.T) {`
`194`	`194`	`if _, hasError := envelope["error"]; !hasError {`
`195`	`195`	`t.Errorf("Expected 'error' field in audit JSON response, got: %s", textContent.Text)`
`196`	`196`	`}`
`197`		`- if _, hasRunID := envelope["run_id_or_url"]; !hasRunID {`
`198`		`- t.Errorf("Expected 'run_id_or_url' field in audit JSON response, got: %s", textContent.Text)`
	`197`	`+ if _, hasRunID := envelope["run_ids_or_urls"]; !hasRunID {`
	`198`	`+ t.Errorf("Expected 'run_ids_or_urls' field in audit JSON response, got: %s", textContent.Text)`
`199`	`199`	`}`
`200`	`200`	`if _, hasSuggestions := envelope["suggestions"]; !hasSuggestions {`
`201`	`201`	`t.Errorf("Expected 'suggestions' field in audit JSON response, got: %s", textContent.Text)`