diff --git a/src/AcDream.Core.Net/Messages/PlayerDescriptionParser.cs b/src/AcDream.Core.Net/Messages/PlayerDescriptionParser.cs index ac88579..2e040b0 100644 --- a/src/AcDream.Core.Net/Messages/PlayerDescriptionParser.cs +++ b/src/AcDream.Core.Net/Messages/PlayerDescriptionParser.cs @@ -378,6 +378,21 @@ public static class PlayerDescriptionParser list.Add(ReadU32(payload, ref pos)); hotbarSpells.Add(list); } + + if (optionFlags.HasFlag(CharacterOptionDataFlag.DesiredComps)) + { + // holtburger events.rs:558-574 — u16 count + u16 padding (4-byte header). + if (payload.Length - pos < 4) throw new FormatException("truncated desired_comps header"); + ushort count = ReadU16(payload, ref pos); + ReadU16(payload, ref pos); // padding/buckets — discarded + if (count > 10_000) throw new FormatException("unreasonable desired_comps count"); + for (int i = 0; i < count; i++) + { + uint id = ReadU32(payload, ref pos); + uint amt = ReadU32(payload, ref pos); + desiredComps.Add((id, amt)); + } + } } } catch (FormatException ex) diff --git a/tests/AcDream.Core.Net.Tests/PlayerDescriptionParserTests.cs b/tests/AcDream.Core.Net.Tests/PlayerDescriptionParserTests.cs index 68ff345..418c586 100644 --- a/tests/AcDream.Core.Net.Tests/PlayerDescriptionParserTests.cs +++ b/tests/AcDream.Core.Net.Tests/PlayerDescriptionParserTests.cs @@ -533,4 +533,37 @@ public sealed class PlayerDescriptionParserTests Assert.Equal(PlayerDescriptionParser.CharacterOptionDataFlag.None, parsed.Value.OptionFlags); Assert.Equal(0u, parsed.Value.Options1); } + + [Fact] + public void TryParse_TrailerDesiredComps_ReadsIdAmtPairs() + { + var sb = new MemoryStream(); + using var writer = new BinaryWriter(sb); + writer.Write(0u); // propertyFlags + writer.Write(0x52u); // weenieType + writer.Write(0x201u); // ATTRIBUTE | ENCHANTMENT + writer.Write(1u); // has_health + writer.Write(0u); // empty attribute_flags + writer.Write(0u); // empty enchantment mask + + // option_flags = DESIRED_COMPS (0x08); no SPELL_LISTS8 so legacy hotbar list (count=0). + writer.Write(0x08u); + writer.Write(0u); // options1 + + // Legacy hotbar list: count=0 + writer.Write(0u); + + // DESIRED_COMPS: u16 count=2, u16 padding, then 2 (id,amt) pairs of 8 bytes each. + writer.Write((ushort)2); + writer.Write((ushort)0); + writer.Write(0xAAu); writer.Write(50u); + writer.Write(0xBBu); writer.Write(75u); + + var parsed = PlayerDescriptionParser.TryParse(sb.ToArray()); + + Assert.NotNull(parsed); + Assert.Equal(2, parsed!.Value.DesiredComps.Count); + Assert.Equal((0xAAu, 50u), parsed.Value.DesiredComps[0]); + Assert.Equal((0xBBu, 75u), parsed.Value.DesiredComps[1]); + } }