Less issue with psalm (#619)

composer.json

@@ -47,8 +47,7 @@
         "symfony/security-bundle": "^5.4 || ^6.2 || ^7.0",
         "symfony/twig-bundle": "^5.4 || ^6.2 || ^7.0",
         "symfony/var-dumper": "^5.4 || ^6.2 || ^7.0",
-        "vimeo/psalm": "^5.7",
-        "weirdan/doctrine-psalm-plugin": "^2.8"
+        "vimeo/psalm": "^5.7"
     },
     "conflict": {
         "doctrine/doctrine-bundle": "<2.7",

psalm-baseline.xml

@@ -6,21 +6,4 @@
             <code>Controller</code>
         </UndefinedClass>
     </file>
-    <file src="tests/BaseTest.php">
-        <ArgumentTypeCoercion>
-            <code><![CDATA[$value]]></code>
-            <code><![CDATA[$value]]></code>
-        </ArgumentTypeCoercion>
-    </file>
-    <file src="tests/CoreTest.php">
-        <InvalidArgument>
-            <code><![CDATA[ProfileAudit::class]]></code>
-            <code><![CDATA[UserAudit::class]]></code>
-        </InvalidArgument>
-    </file>
-    <file src="tests/Issue/Issue318Test.php">
-        <ArgumentTypeCoercion>
-            <code><![CDATA[$user::class]]></code>
-        </ArgumentTypeCoercion>
-    </file>
 </files>

psalm.xml

@@ -10,7 +10,6 @@
     <plugins>
         <pluginClass class="Psalm\PhpUnitPlugin\Plugin"/>
         <pluginClass class="Psalm\SymfonyPsalmPlugin\Plugin"/>
-        <pluginClass class="Weirdan\DoctrinePsalmPlugin\Plugin"/>
     </plugins>
     <issueHandlers>
         <!-- Psalm equivalent of PHPStan config `treatPhpDocTypesAsCertain: false` -->

src/Collection/AuditedCollection.php

@@ -80,7 +80,9 @@ public function __construct(
     }
 
     /**
-     * @return true
+     * @return void
+     *
+     * @phpstan-ignore-next-line https://github.com/phpstan/phpstan-doctrine/pull/560
      */
     #[\ReturnTypeWillChange]
     public function add(mixed $element)
@@ -293,9 +295,10 @@ public function exists(\Closure $p)
     /**
      * @return Collection<TKey, T>
      *
-     * @phpstan-param \Closure(T, int|string):bool $p
-     * @psalm-param \Closure(T=):bool $p
-     * @psalm-return Collection<TKey, T>
+     * @phpstan-param \Closure(T, TKey):bool $p
+     * @phpstan-return Collection<TKey, T>
+     *
+     * @psalm-suppress MoreSpecificImplementedParamType https://github.com/doctrine/collections/pull/411
      */
     #[\ReturnTypeWillChange]
     public function filter(\Closure $p)
@@ -323,9 +326,7 @@ public function forAll(\Closure $p)
      *
      * @phpstan-template U
      * @phpstan-param \Closure(T):U $func
-     * @psalm-return Collection<TKey, U>
-     *
-     * @psalm-suppress ImplementedParamTypeMismatch,InvalidArgument
+     * @phpstan-return Collection<TKey, U>
      */
     #[\ReturnTypeWillChange]
     public function map(\Closure $func)
@@ -339,7 +340,9 @@ public function map(\Closure $func)
      * @return array<Collection<TKey, T>>
      *
      * @phpstan-param \Closure(TKey, T):bool $p
-     * @psalm-return array{0: Collection<TKey, T>, 1: Collection<TKey, T>}
+     * @phpstan-return array{0: Collection<TKey, T>, 1: Collection<TKey, T>}
+     *
+     * @psalm-suppress MoreSpecificImplementedParamType https://github.com/doctrine/collections/pull/411
      */
     #[\ReturnTypeWillChange]
     public function partition(\Closure $p)
@@ -454,7 +457,6 @@ public function count()
      * @return T|null
      *
      * @phpstan-return T|null
-     * @psalm-return mixed|null
      */
     #[\ReturnTypeWillChange]
     public function findFirst(\Closure $p)

tests/Fixtures/Issue/Issue308User.php

@@ -64,11 +64,9 @@ public function addChild(self $child): void
      */
     public function getChildren(): ReadableCollection
     {
-        $activeChildren = $this->children->filter(
+        return $this->children->filter(
             static fn (self $user): bool => $user->isActive()
         );
-
-        return $activeChildren;
     }
 
     public function getParent(): ?self