fix: parse player name in connect/disconnect events (#108)

* fix: parse player name in connect/disconnect events

Connect and disconnect events have the format [frameNum, "type", "playerName"]
where the third element is a string. The parser was incorrectly treating this
as a numeric SourceID, causing the player name to be lost when converting to
protobuf/flatbuffers format.

This fix adds special handling for connected/disconnected event types to store
the player name in the Message field, which the frontend correctly reads.

* test: add edge case for connect event without player name

Author: fank

internal/storage/parser_test.go

@@ -544,6 +544,54 @@ func TestParserV1_parseEvent_EdgeCases(t *testing.T) {
 			t.Errorf("Type = %q, want %q", evt.Type, "test")
 		}
 	})
+
+	t.Run("connected event", func(t *testing.T) {
+		evt := p.parseEvent([]interface{}{2.0, "connected", "Cal"})
+		if evt == nil {
+			t.Fatal("expected non-nil event")
+		}
+		if evt.FrameNum != 2 {
+			t.Errorf("FrameNum = %d, want %d", evt.FrameNum, 2)
+		}
+		if evt.Type != "connected" {
+			t.Errorf("Type = %q, want %q", evt.Type, "connected")
+		}
+		if evt.Message != "Cal" {
+			t.Errorf("Message = %q, want %q", evt.Message, "Cal")
+		}
+		if evt.SourceID != 0 {
+			t.Errorf("SourceID = %d, want %d (should not be set)", evt.SourceID, 0)
+		}
+	})
+
+	t.Run("disconnected event", func(t *testing.T) {
+		evt := p.parseEvent([]interface{}{3.0, "disconnected", "Wraith"})
+		if evt == nil {
+			t.Fatal("expected non-nil event")
+		}
+		if evt.FrameNum != 3 {
+			t.Errorf("FrameNum = %d, want %d", evt.FrameNum, 3)
+		}
+		if evt.Type != "disconnected" {
+			t.Errorf("Type = %q, want %q", evt.Type, "disconnected")
+		}
+		if evt.Message != "Wraith" {
+			t.Errorf("Message = %q, want %q", evt.Message, "Wraith")
+		}
+	})
+
+	t.Run("connected event without player name", func(t *testing.T) {
+		evt := p.parseEvent([]interface{}{5.0, "connected"})
+		if evt == nil {
+			t.Fatal("expected non-nil event")
+		}
+		if evt.Type != "connected" {
+			t.Errorf("Type = %q, want %q", evt.Type, "connected")
+		}
+		if evt.Message != "" {
+			t.Errorf("Message = %q, want empty string", evt.Message)
+		}
+	})
 }
 
 func TestParserV1_parseMarker_EdgeCases(t *testing.T) {

internal/storage/parser_v1.go

@@ -130,6 +130,14 @@ func (p *ParserV1) parseEvent(evtArr []interface{}) *Event {
 		Type:     toString(evtArr[1]),
 	}
 
+	// Connect/disconnect events: [frameNum, "type", "playerName"]
+	if event.Type == "connected" || event.Type == "disconnected" {
+		if len(evtArr) > 2 {
+			event.Message = toString(evtArr[2])
+		}
+		return event
+	}
+
 	// Parse additional fields based on event type
 	// Common format: [frameNum, "type", sourceId, targetId, ...]
 	if len(evtArr) > 2 {