Use sequences instead of sets

src/Adapter/Filesystem/Decode.php

@@ -17,6 +17,7 @@
 use Innmind\Immutable\{
     Maybe,
     Set,
+    Sequence,
     Predicate\Instance,
 };
 
@@ -67,8 +68,7 @@ public function __invoke(Aggregate\Id $id = null): callable
                         ->map(static fn($file) => Aggregate\Property::of(
                             $file->name()->toString(),
                             Json::decode($file->content()->toString()),
-                        ))
-                        ->toSet(),
+                        )),
                 ),
             $directory
                 ->get(Name::of('entities'))
@@ -86,11 +86,9 @@ public function __invoke(Aggregate\Id $id = null): callable
                                     ->map(static fn($property) => Aggregate\Property::of(
                                         $property->name()->toString(),
                                         Json::decode($property->content()->toString()),
-                                    ))
-                                    ->toSet(),
+                                    )),
                             ),
-                        )
-                        ->toSet(),
+                        ),
                 ),
             $directory
                 ->get(Name::of('optionals'))
@@ -112,12 +110,10 @@ public function __invoke(Aggregate\Id $id = null): callable
                                             ->map(static fn($property) => Aggregate\Property::of(
                                                 $property->name()->toString(),
                                                 Json::decode($property->content()->toString()),
-                                            ))
-                                            ->toSet(),
+                                            )),
                                     ),
                             ),
-                        )
-                        ->toSet(),
+                        ),
                 ),
             $directory
                 ->get(Name::of('collections'))
@@ -132,7 +128,7 @@ public function __invoke(Aggregate\Id $id = null): callable
                                 Set::of(...Json::decode($collection->content()->toString()))->map(
                                     static fn($entity) => Aggregate\Collection\Entity::of(
                                         Reference::of($entity['reference']),
-                                        Set::of(...$entity['properties'])->map(
+                                        Sequence::of(...$entity['properties'])->map(
                                             static fn($property) => Aggregate\Property::of(
                                                 $property[0],
                                                 $property[1],
@@ -141,10 +137,9 @@ public function __invoke(Aggregate\Id $id = null): callable
                                     ),
                                 ),
                             ),
-                        )
-                        ->toSet(),
+                        ),
                 ),
-        )->map(static fn(Set $properties, Set $entities, Set $optionals, Set $collections) => Aggregate::of(
+        )->map(static fn(Sequence $properties, Sequence $entities, Sequence $optionals, Sequence $collections) => Aggregate::of(
             $id($directory),
             $properties,
             $entities,

src/Adapter/SQL/CollectionTable.php

@@ -34,8 +34,8 @@ final class CollectionTable
     /** @var Definition<T> */
     private Definition $definition;
     private Table\Name\Aliased $name;
-    /** @var Set<Column\Name\Aliased> */
-    private Set $columns;
+    /** @var Sequence<Column\Name\Aliased> */
+    private Sequence $columns;
     private Column\Name\Namespaced $id;
     private Column\Name\Namespaced $reference;
     private Select $select;
@@ -98,9 +98,9 @@ public function foreignKey(): Table\Column
     }
 
     /**
-     * @return Set<Column>
+     * @return Sequence<Column>
      */
-    public function columnsDefinition(MapType $mapType): Set
+    public function columnsDefinition(MapType $mapType): Sequence
     {
         return $this
             ->definition
@@ -117,9 +117,9 @@ public function name(): Table\Name\Aliased
     }
 
     /**
-     * @return Set<Column\Name\Aliased>
+     * @return Sequence<Column\Name\Aliased>
      */
-    public function columns(): Set
+    public function columns(): Sequence
     {
         return $this->columns;
     }

src/Adapter/SQL/Decode.php

@@ -16,6 +16,7 @@
 use Innmind\Immutable\{
     Maybe,
     Set,
+    Sequence,
     Str,
 };
 
@@ -77,8 +78,7 @@ public function __invoke(Aggregate\Id $id = null): callable
                 ->map(static fn($value) => Aggregate\Property::of(
                     Str::of($value->column()->toString())->drop(7)->toString(),
                     $value->value(),
-                ))
-                ->toSet(),
+                )),
             $this
                 ->mainTable
                 ->entities()
@@ -154,11 +154,11 @@ public static function of(
     /**
      * @psalm-pure
      *
-     * @param Set<Column\Name\Aliased> $columns
+     * @param Sequence<Column\Name\Aliased> $columns
      *
-     * @return Set<Aggregate\Property>
+     * @return Sequence<Aggregate\Property>
      */
-    private static function properties(Row $row, Set $columns): Set
+    private static function properties(Row $row, Sequence $columns): Sequence
     {
         /** @psalm-suppress MixedArgument Due to the access-layer type */
         return $columns->flatMap(
@@ -171,8 +171,7 @@ private static function properties(Row $row, Set $columns): Set
                     },
                     $value,
                 ))
-                ->toSequence()
-                ->toSet(),
+                ->toSequence(),
         );
     }
 }

src/Adapter/SQL/Encode.php

@@ -8,10 +8,7 @@
     Raw\Aggregate,
 };
 use Formal\AccessLayer\Query;
-use Innmind\Immutable\{
-    Set,
-    Sequence,
-};
+use Innmind\Immutable\Sequence;
 
 /**
  * @internal
@@ -47,12 +44,10 @@ public function __invoke(Aggregate $data): Sequence
         $optionals = $this->optionals($data);
         $collections = $this->collections($data);
 
-        return Sequence::of(
-            $main,
-            ...$entities->toList(),
-            ...$optionals->toList(),
-            ...$collections->toList(),
-        );
+        return Sequence::of($main)
+            ->append($entities)
+            ->append($optionals)
+            ->append($collections);
     }
 
     /**
@@ -73,9 +68,9 @@ public static function of(
     }
 
     /**
-     * @return Set<Query>
+     * @return Sequence<Query>
      */
-    private function entities(Aggregate $data): Set
+    private function entities(Aggregate $data): Sequence
     {
         return $data
             ->entities()
@@ -86,15 +81,14 @@ private function entities(Aggregate $data): Set
                     ->map(
                         static fn($table) => $table->insert($data->id(), $entity->properties()),
                     )
-                    ->toSequence()
-                    ->toSet(),
+                    ->toSequence(),
             );
     }
 
     /**
-     * @return Set<Query>
+     * @return Sequence<Query>
      */
-    private function optionals(Aggregate $data): Set
+    private function optionals(Aggregate $data): Sequence
     {
         return $data
             ->optionals()
@@ -107,8 +101,7 @@ private function optionals(Aggregate $data): Set
                             static fn($properties) => $table->insert($data->id(), $properties),
                         ),
                     )
-                    ->toSequence()
-                    ->toSet(),
+                    ->toSequence(),
             );
     }
 
@@ -121,9 +114,9 @@ private function main(Aggregate $data): Query
     }
 
     /**
-     * @return Set<Query>
+     * @return Sequence<Query>
      */
-    private function collections(Aggregate $data): Set
+    private function collections(Aggregate $data): Sequence
     {
         return $data
             ->collections()
@@ -137,8 +130,7 @@ private function collections(Aggregate $data): Set
                             $collection->entities(),
                         ),
                     )
-                    ->toSequence()
-                    ->toSet(),
+                    ->toSequence(),
             );
     }
 }

src/Adapter/SQL/EntityTable.php

@@ -18,7 +18,7 @@
 };
 use Innmind\Specification\Sign;
 use Innmind\Immutable\{
-    Set,
+    Sequence,
     Maybe,
 };
 
@@ -31,8 +31,8 @@ final class EntityTable
     /** @var Definition<T> */
     private Definition $definition;
     private Table\Name\Aliased $name;
-    /** @var Set<Column\Name\Aliased> */
-    private Set $columns;
+    /** @var Sequence<Column\Name\Aliased> */
+    private Sequence $columns;
 
     /**
      * @param Definition<T> $definition
@@ -77,9 +77,9 @@ public function primaryKey(): Table\Column
     }
 
     /**
-     * @return Set<Column>
+     * @return Sequence<Column>
      */
-    public function columnsDefinition(MapType $mapType): Set
+    public function columnsDefinition(MapType $mapType): Sequence
     {
         return $this
             ->definition
@@ -96,19 +96,19 @@ public function name(): Table\Name\Aliased
     }
 
     /**
-     * @return Set<Column\Name\Aliased>
+     * @return Sequence<Column\Name\Aliased>
      */
-    public function columns(): Set
+    public function columns(): Sequence
     {
         return $this->columns;
     }
 
     /**
      * @internal
      *
-     * @param Set<Property> $properties
+     * @param Sequence<Property> $properties
      */
-    public function insert(Id $id, Set $properties): Query
+    public function insert(Id $id, Sequence $properties): Query
     {
         $table = $this->name->name();
 
@@ -132,11 +132,11 @@ public function insert(Id $id, Set $properties): Query
     /**
      * @internal
      *
-     * @param Set<Property> $properties
+     * @param Sequence<Property> $properties
      *
      * @return Maybe<Query>
      */
-    public function update(Id $id, Set $properties): Maybe
+    public function update(Id $id, Sequence $properties): Maybe
     {
         return Maybe::just($properties)
             ->filter(static fn($properties) => !$properties->empty())

src/Adapter/SQL/MainTable.php

@@ -33,7 +33,7 @@
 use Innmind\Immutable\{
     Map,
     Maybe,
-    Set,
+    Sequence,
 };
 
 /**
@@ -114,7 +114,6 @@ private function __construct(Definition $definition)
                     ->toList(),
                 ...$entities
                     ->values()
-                    ->toSet()
                     ->flatMap(static fn($table) => $table->columns())
                     ->toList(),
             );
@@ -161,9 +160,9 @@ public function primaryKey(): Table\Column
     }
 
     /**
-     * @return Set<Column>
+     * @return Sequence<Column>
      */
-    public function columnsDefinition(MapType $mapType): Set
+    public function columnsDefinition(MapType $mapType): Sequence
     {
         return $this
             ->definition
@@ -212,11 +211,11 @@ public function count(Specification $specification = null): Select
     /**
      * @internal
      *
-     * @param Set<Aggregate\Property> $properties
+     * @param Sequence<Aggregate\Property> $properties
      */
     public function insert(
         Id $id,
-        Set $properties,
+        Sequence $properties,
     ): Query {
         $table = $this->name->name();
 
@@ -276,11 +275,11 @@ public function delete(): Delete
     }
 
     /**
-     * @return Set<EntityTable>
+     * @return Sequence<EntityTable>
      */
-    public function entities(): Set
+    public function entities(): Sequence
     {
-        return $this->entities->values()->toSet();
+        return $this->entities->values();
     }
 
     /**
@@ -294,11 +293,11 @@ public function entity(string $name): Maybe
     }
 
     /**
-     * @return Set<OptionalTable>
+     * @return Sequence<OptionalTable>
      */
-    public function optionals(): Set
+    public function optionals(): Sequence
     {
-        return $this->optionals->values()->toSet();
+        return $this->optionals->values();
     }
 
     /**
@@ -312,11 +311,11 @@ public function optional(string $name): Maybe
     }
 
     /**
-     * @return Set<CollectionTable>
+     * @return Sequence<CollectionTable>
      */
-    public function collections(): Set
+    public function collections(): Sequence
     {
-        return $this->collections->values()->toSet();
+        return $this->collections->values();
     }
 
     /**