| 1 | package openai |
| 2 | |
| 3 | import ( |
| 4 | "bytes" |
| 5 | "encoding/json" |
| 6 | "testing" |
| 7 | |
| 8 | "reasonix/internal/provider" |
| 9 | ) |
| 10 | |
| 11 | // Message ids are local transcript identity. The chat wire struct copies |
| 12 | // fields explicitly, so a tagged history must serialize byte-for-byte like an |
| 13 | // untagged one; anything else would break the prompt-cache prefix. |
| 14 | func TestMessageIDNeverReachesChatWire(t *testing.T) { |
| 15 | plain := []provider.Message{ |
| 16 | {Role: provider.RoleSystem, Content: "sys"}, |
| 17 | {Role: provider.RoleUser, Content: "list the files"}, |
| 18 | {Role: provider.RoleAssistant, ReasoningContent: "think", ToolCalls: []provider.ToolCall{{ID: "call_1", Name: "ls", Arguments: `{"path":"."}`}}}, |
| 19 | {Role: provider.RoleTool, Content: "main.go", ToolCallID: "call_1", Name: "ls"}, |
| 20 | {Role: provider.RoleAssistant, Content: "done"}, |
| 21 | } |
| 22 | tagged := append([]provider.Message(nil), plain...) |
| 23 | for i := range tagged { |
| 24 | tagged[i].ID = "01ARZ3NDEKTSV4RRFFQ69G5FA" + string(rune('A'+i)) |
| 25 | } |
| 26 | c := &client{model: "deepseek-v4"} |
| 27 | want, err := json.Marshal(c.buildRequest(provider.Request{Messages: plain})) |
| 28 | if err != nil { |
| 29 | t.Fatal(err) |
| 30 | } |
| 31 | got, err := json.Marshal(c.buildRequest(provider.Request{Messages: tagged})) |
| 32 | if err != nil { |
| 33 | t.Fatal(err) |
| 34 | } |
| 35 | if !bytes.Equal(want, got) { |
| 36 | t.Fatalf("message ids changed chat request bytes\nplain=%s\ntagged=%s", want, got) |
| 37 | } |
| 38 | if bytes.Contains(got, []byte("01ARZ3NDEKTSV4RRFFQ69G5FA")) { |
| 39 | t.Fatalf("message id leaked into request: %s", got) |
| 40 | } |
| 41 | } |
| 42 |