chore: support packed lists for proto2

Author: kafkiansky

src/Compiler/FieldOptions.php

@@ -32,9 +32,7 @@ public function __construct(
         public bool $debugRedact = false,
         #[Reflection\Field(17, new Reflection\EnumT(FieldOptions\OptionRetention::class))]
         public ?FieldOptions\OptionRetention $retention = null,
-        #[Reflection\Field(19, new Reflection\ListT(
-            new Reflection\ObjectT(FieldOptions\OptionTargetType::class),
-        ))]
+        #[Reflection\Field(19, new Reflection\ListT(new Reflection\EnumT(FieldOptions\OptionTargetType::class), false))]
         public array $targets = [],
         #[Reflection\Field(20, new Reflection\ListT(
             new Reflection\ObjectT(FieldOptions\EditionDefault::class),

src/Compiler/FileDescriptorProto.php

@@ -28,9 +28,9 @@ public function __construct(
         public ?string $package = null,
         #[Reflection\Field(3, new Reflection\ListT(Reflection\StringT::T))]
         public array $dependencies = [],
-        #[Reflection\Field(10, new Reflection\ListT(Reflection\Int32T::T))]
+        #[Reflection\Field(10, new Reflection\ListT(Reflection\Int32T::T, false))]
         public array $publicDependencies = [],
-        #[Reflection\Field(11, new Reflection\ListT(Reflection\Int32T::T))]
+        #[Reflection\Field(11, new Reflection\ListT(Reflection\Int32T::T, false))]
         public array $weakDependencies = [],
         #[Reflection\Field(15, new Reflection\ListT(Reflection\StringT::T))]
         public array $optionDependencies = [],

src/Reflection/Internal/Visitor/RecursionBreakTypeVisitor.php

@@ -32,7 +32,7 @@ public function __construct(
     public function list(ListT $type): mixed
     {
         /** @phpstan-ignore argument.type */
-        return Protobuf\listT($type->element->accept($this));
+        return Protobuf\listT($type->element->accept($this), $type->packed);
     }
 
     #[\Override]

src/Reflection/Internal/Visitor/ToProtobufTypeTypeVisitor.php

@@ -134,7 +134,7 @@ public function list(ListT $type): Type
         /** @var Type<mixed, 'repeatable'> $element */
         $element = $type->element->accept($this);
 
-        return Protobuf\listT($element);
+        return Protobuf\listT($element, $type->packed);
     }
 
     #[\Override]

src/Reflection/ListT.php

@@ -16,6 +16,7 @@
      */
     public function __construct(
         public Type $element,
+        public ?bool $packed = null,
     ) {}
 
     #[\Override]

src/Type/ListT.php

@@ -17,6 +17,7 @@
      */
     public function __construct(
         public Type $element,
+        public ?bool $packed = null,
     ) {}
 
     #[\Override]

src/Type/Visitor/TypeDeserializerVisitor.php

@@ -146,7 +146,7 @@ public function list(ListT $type): DeserializeValue
             /** @phpstan-ignore argument.type */
             $type->element->accept($this),
             $this->tag,
-            $type->element->accept(new IsPacked()),
+            $type->packed ?? $type->element->accept(new IsPacked()),
         );
     }
 

src/Type/Visitor/TypeSerializerVisitor.php

@@ -155,7 +155,7 @@ public function list(ListT $type): SerializeValue
                     ->accept($this)
                     ->without(SerializeTag::class),
                 $this->tag,
-                $type->element->accept(new IsPacked()),
+                $type->packed ?? $type->element->accept(new IsPacked()),
             ),
         );
     }

src/Value.php

@@ -155,10 +155,11 @@ public static function message(Message $message): self
     public static function listOf(
         Type $type,
         array $values,
+        ?bool $packed = null,
     ): self {
         return new self(
             $values,
-            listT($type),
+            listT($type, $packed),
         );
     }
 

src/constructors.php

@@ -247,8 +247,9 @@ function recursionT(\Closure $continuation): Type\RecursionT
 function listOf(
     Type $type,
     array $values,
+    ?bool $packed = null,
 ): Value {
-    return Value::listOf($type, $values);
+    return Value::listOf($type, $values, $packed);
 }
 
 /**
@@ -257,9 +258,9 @@ function listOf(
  * @param Type<T, 'repeatable', *, *> $element
  * @return Type\ListT<T>
  */
-function listT(Type $element): Type\ListT
+function listT(Type $element, ?bool $packed = null): Type\ListT
 {
-    return new Type\ListT($element);
+    return new Type\ListT($element, $packed);
 }
 
 /**