From 33a822e3a10c9d3776ce1c43ed89c8776a579d96 Mon Sep 17 00:00:00 2001
From: u9g <git@u9g.dev>
Date: Sun, 6 Sep 2026 16:06:55 -0400
Subject: [PATCH] pc/protocol: cookie_response value is optional on 1.21.8

ServerboundCookieResponsePacket writes the payload with writeNullable on 1.21.8 as on every
other version; the schema declared it as a plain ByteArray, so a stored cookie could not be
echoed and a null one deserialized as an empty array.
---
 data/pc/1.21.8/proto.yml     | 2 +-
 data/pc/1.21.8/protocol.json | 5 ++++-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/data/pc/1.21.8/proto.yml b/data/pc/1.21.8/proto.yml
index b8759aa2c..5906f926d 100644
--- a/data/pc/1.21.8/proto.yml
+++ b/data/pc/1.21.8/proto.yml
@@ -1273,7 +1273,7 @@
    # MC: ServerboundCookieResponsePacket
    packet_common_cookie_response:
       key: string
-      value: ByteArray
+      value?: ByteArray
    # MC: ServerboundSelectKnownPacks
    # MC: ClientboundSelectKnownPacks
    packet_common_select_known_packs:
diff --git a/data/pc/1.21.8/protocol.json b/data/pc/1.21.8/protocol.json
index b80ba438b..8bfa742c0 100644
--- a/data/pc/1.21.8/protocol.json
+++ b/data/pc/1.21.8/protocol.json
@@ -3403,7 +3403,10 @@
         },
         {
           "name": "value",
-          "type": "ByteArray"
+          "type": [
+            "option",
+            "ByteArray"
+          ]
         }
       ]
     ],