fix: Add empty segments to streaming response (#473)

Mike Zorn · web-flow · commit 9f0f4073edcd · 2025-01-29T09:52:01.000-08:00
Add empty segments to streaming response

This is needed for python SDK support
diff --git a/internal/dev_server/sdk/server_flags.go b/internal/dev_server/sdk/server_flags.go
@@ -36,7 +36,8 @@ type ServerFlag struct {
 type ServerFlags map[string]ServerFlag
 
 type data struct {
-	Flags ServerFlags `json:"flags"`
+	Flags    ServerFlags `json:"flags"`
+	Segments struct{}    `json:"segments"` // We need to send an empty object over the wire for compatibility with some SDKs
 }
 type ServerAllPayload struct {
 	Path string `json:"path"`
@@ -46,7 +47,7 @@ type ServerAllPayload struct {
 func ServerAllPayloadFromFlagsState(state model.FlagsState) ServerAllPayload {
 	return ServerAllPayload{
 		Path: "",
-		Data: data{ServerFlagsFromFlagsState(state)},
+		Data: data{Flags: ServerFlagsFromFlagsState(state)},
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,8 @@ type ServerFlag struct {`
`36`	`36`	`type ServerFlags map[string]ServerFlag`
`37`	`37`
`38`	`38`	`type data struct {`
`39`		- Flags ServerFlags `json:"flags"`
	`39`	+ Flags ServerFlags `json:"flags"`
	`40`	+ Segments struct{} `json:"segments"` // We need to send an empty object over the wire for compatibility with some SDKs
`40`	`41`	`}`
`41`	`42`	`type ServerAllPayload struct {`
`42`	`43`	Path string `json:"path"`
`@@ -46,7 +47,7 @@ type ServerAllPayload struct {`
`46`	`47`	`func ServerAllPayloadFromFlagsState(state model.FlagsState) ServerAllPayload {`
`47`	`48`	`return ServerAllPayload{`
`48`	`49`	`Path: "",`
`49`		`- Data: data{ServerFlagsFromFlagsState(state)},`
	`50`	`+ Data: data{Flags: ServerFlagsFromFlagsState(state)},`
`50`	`51`	`}`
`51`	`52`	`}`
`52`	`53`