Added each method to the Rows class and use the latest version of phi… (#13)

* Added each method to the Rows class and use the latest version of phive tools.

* Changed trust-gpg-key for psalm

* Changed trust-gpg-key for psalm for mutation also

* Changed trust-gpg-key

* Fixed psalm errors

* Use psalm-param instead of psalm-suppress

Author: DawidSajdak

.github/workflows/mutation.yml

@@ -62,7 +62,7 @@ jobs:
         run: "composer install --no-interaction --no-progress --no-suggest"
 
       - name: "Install tools"
-        run: "phive install --trust-gpg-keys E82B2FB314E9906E,C5095986493B4AA0,CF1A108D0E7AE720,8A03EA3B385DBAA1 --force-accept-unsigned"
+        run: "phive install --trust-gpg-keys E82B2FB314E9906E,C5095986493B4AA0,CF1A108D0E7AE720,12CE0F1D262429A5 --force-accept-unsigned"
 
       - name: "Mutation Tests"
         run: "composer test:mutation"

.github/workflows/static-analyze.yml

@@ -62,7 +62,7 @@ jobs:
         run: "composer install --no-interaction --no-progress --no-suggest"
 
       - name: "Install tools"
-        run: "phive install --trust-gpg-keys E82B2FB314E9906E,C5095986493B4AA0,CF1A108D0E7AE720,8A03EA3B385DBAA1 --force-accept-unsigned"
+        run: "phive install --trust-gpg-keys E82B2FB314E9906E,C5095986493B4AA0,CF1A108D0E7AE720,12CE0F1D262429A5 --force-accept-unsigned"
 
       - name: "Static Analyze"
         run: "composer static:analyze"

.phive/phars.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phive xmlns="https://phar.io/phive">
-  <phar name="php-cs-fixer" version="^2.16.4" installed="2.16.4" location="./tools/php-cs-fixer" copy="true"/>
-  <phar name="infection" version="^0.20.1" installed="0.20.1" location="./tools/infection" copy="true"/>
-  <phar name="phpstan" version="^0.12.59" installed="0.12.59" location="./tools/phpstan" copy="true"/>
-  <phar name="psalm" version="^3.12.1" installed="3.14.2" location="./tools/psalm" copy="true"/>
-</phive>
+  <phar name="php-cs-fixer" version="^2.18.3" installed="2.18.3" location="./tools/php-cs-fixer" copy="true"/>
+  <phar name="infection" version="^0.21.4" installed="0.21.4" location="./tools/infection" copy="true"/>
+  <phar name="phpstan" version="^0.12.81" installed="0.12.81" location="./tools/phpstan" copy="true"/>
+  <phar name="psalm" version="^4.5.2" installed="4.5.2" location="./tools/psalm" copy="true"/>
+</phive>

src/Flow/ETL/ETL.php

@@ -10,26 +10,17 @@ final class ETL
 
     private Transformers $transformers;
 
-    /**
-     * @psalm-pure
-     */
     private function __construct(Extractor $extractor, Transformers $transformers)
     {
         $this->extractor = $extractor;
         $this->transformers = $transformers;
     }
 
-    /**
-     * @psalm-pure
-     */
     public static function extract(Extractor $extractor) : self
     {
         return new self($extractor, new Transformers());
     }
 
-    /**
-     * @psalm-pure
-     */
     public function transform(Transformer ...$transformer) : self
     {
         return new self($this->extractor, $this->transformers->add(...$transformer));

src/Flow/ETL/Row/Entries.php

@@ -96,6 +96,7 @@ public function count() : int
     }
 
     /**
+     * @psalm-suppress ImpureFunctionCall
      * @psalm-suppress MixedReturnTypeCoercion
      * @template ReturnType
      *
@@ -109,6 +110,9 @@ public function map(callable $callable) : array
     }
 
     /**
+     * @psalm-suppress ImpureFunctionCall
+     * @psalm-suppress MixedArgumentTypeCoercion
+     *
      * @param callable(Entry) : bool $callable
      */
     public function filter(callable $callable) : self

src/Flow/ETL/Row/Entry/CollectionEntry.php

@@ -87,6 +87,9 @@ public function value() : array
         return $this->mapEntries(fn (Entries $entries) : array => $entries->toArray());
     }
 
+    /**
+     * @psalm-suppress InvalidArgument
+     */
     public function is(string $name) : bool
     {
         return $this->key === \mb_strtolower($name);
@@ -98,7 +101,7 @@ public function rename(string $name) : Entry
     }
 
     /**
-     * @psalm-param callable(Entries) : bool $filter
+     * @psalm-param pure-callable(Entries) : bool $filter
      */
     public function filterEntries(callable $filter) : self
     {
@@ -119,6 +122,7 @@ public function isEqual(Entry $entry) : bool
     }
 
     /**
+     * @psalm-suppress ImpureFunctionCall
      * @phpstan-ignore-next-line
      */
     private function mapEntries(callable $callable) : array

src/Flow/ETL/Rows.php

@@ -28,6 +28,7 @@ public function __construct(Row ...$rows)
     }
 
     /**
+     * @psalm-suppress MixedArgument
      * @pslam-param callable (Row) : string|int|float $groupBy  Returns group identifier for the given Row
      */
     public function groupTo(string $collectionEntryName, callable $groupBy) : self
@@ -86,6 +87,9 @@ public function empty() : bool
     }
 
     /**
+     * @psalm-param pure-callable(Row) : bool $callable
+     * @psalm-suppress MixedArgumentTypeCoercion
+     *
      * @param callable(Row) : bool $callable
      */
     public function filter(callable $callable) : self
@@ -94,6 +98,7 @@ public function filter(callable $callable) : self
     }
 
     /**
+     * @psalm-param pure-callable(Row) : Row $callable
      * @psalm-suppress MixedArgument Psalm doesn't understand that array_map will return array<int, Row>
      *
      * @param callable(Row) : Row $callable
@@ -104,6 +109,19 @@ public function map(callable $callable) : self
     }
 
     /**
+     * @psalm-param pure-callable(Row) : Row $callable
+     *
+     * @param callable(Row) : Row $callable
+     */
+    public function each(callable $callable) : void
+    {
+        \array_map($callable, $this->rows);
+    }
+
+    /**
+     * @psalm-param pure-callable(mixed, Row) : mixed $callable
+     * @psalm-suppress MixedAssignment
+     *
      * @param callable(mixed, Row) : mixed $callable
      * @param null|mixed $input
      *