chore: remove psalm (#1242)

.gitattributes

@@ -16,6 +16,4 @@ phpbench.json export-ignore
 phpstan.neon.dist export-ignore
 phpstan-baseline.neon export-ignore
 phpunit.xml.dist export-ignore
-psalm.xml.dist export-ignore
-psalm-baseline.xml export-ignore
 renovate.json export-ignore

.github/workflows/static-analysis.yml

@@ -34,29 +34,3 @@ jobs:
 
       - name: Run a static analysis with phpstan/phpstan
         run: vendor/bin/phpstan --error-format=checkstyle | cs2pr
-
-  static-analysis-psalm:
-    name: Static Analysis with Psalm
-    runs-on: ubuntu-20.04
-
-    strategy:
-      matrix:
-        php-version:
-          - 8.1
-
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v3
-
-      - name: Install PHP
-        uses: shivammathur/setup-php@v2
-        with:
-          coverage: none
-          php-version: ${{ matrix.php-version }}
-          tools: cs2pr
-
-      - name: Install dependencies with Composer
-        uses: ramsey/composer-install@v2
-
-      - name: Run a static analysis with vimeo/psalm
-        run: vendor/bin/psalm --output-format=github --shepherd

.gitignore

@@ -3,6 +3,5 @@
 composer.lock
 composer.phar
 phpstan.neon
-psalm.xml
 vendor
 .idea

README.md

@@ -3,7 +3,6 @@
 [![CI](https://github.com/webonyx/graphql-php/workflows/CI/badge.svg)](https://github.com/webonyx/graphql-php/actions?query=workflow:CI+branch:master)
 [![Coverage Status](https://codecov.io/gh/webonyx/graphql-php/branch/master/graph/badge.svg)](https://codecov.io/gh/webonyx/graphql-php/branch/master)
 [![Latest Stable Version](https://poser.pugx.org/webonyx/graphql-php/version)](https://packagist.org/packages/webonyx/graphql-php)
-[![Shepherd Type](https://shepherd.dev/github/webonyx/graphql-php/coverage.svg)](ttps://shepherd.dev/github/webonyx/graphql-php)
 
 This is a PHP implementation of the GraphQL [specification](https://github.com/graphql/graphql-spec)
 based on the [reference implementation in JavaScript](https://github.com/graphql/graphql-js).

composer.json

@@ -25,13 +25,11 @@
     "phpstan/phpstan-phpunit": "1.2.2",
     "phpstan/phpstan-strict-rules": "1.4.4",
     "phpunit/phpunit": "^9.5",
-    "psalm/plugin-phpunit": "^0.17",
     "psr/http-message": "^1",
     "react/promise": "^2",
     "symfony/polyfill-php81": "^1.23",
     "symfony/var-exporter": "^5 || ^6",
-    "thecodingmachine/safe": "^1.3",
-    "vimeo/psalm": "^4.19"
+    "thecodingmachine/safe": "^1.3"
   },
   "suggest": {
     "psr/http-message": "To use standard GraphQL server",
@@ -59,12 +57,11 @@
     "sort-packages": true
   },
   "scripts": {
-    "baseline": "phpstan --generate-baseline && psalm --set-baseline",
     "bench": "phpbench run",
     "check": "composer fix && composer stan && composer test",
     "docs": "php generate-class-reference.php",
     "fix": "php-cs-fixer fix",
-    "stan": "phpstan && psalm",
+    "stan": "phpstan",
     "test": "phpunit"
   }
 }

docs/class-reference.md

@@ -1264,17 +1264,8 @@ const CLASS_MAP = [
 Implements the "Evaluating requests" section of the GraphQL specification.
 
 @phpstan-type FieldResolver callable(mixed, array<string, mixed>, mixed, ResolveInfo): mixed
-
-@see https://github.com/vimeo/psalm/issues/6928
-
-@psalm-type FieldResolver callable(mixed, array, mixed, ResolveInfo): mixed
-
 @phpstan-type ImplementationFactory callable(PromiseAdapter, Schema, DocumentNode, mixed, mixed, array<mixed>, ?string, callable): ExecutorImplementation
 
-@see https://github.com/vimeo/psalm/issues/6928, https://github.com/vimeo/psalm/issues/7527
-
-@psalm-type ImplementationFactory callable(PromiseAdapter, Schema, DocumentNode, mixed, mixed, array, ?string, callable): ExecutorImplementation
-
 ### GraphQL\Executor\Executor Methods
 
 ```php
@@ -1356,10 +1347,6 @@ extensions?: array<string, mixed>
 @phpstan-type ErrorFormatter callable(Throwable): SerializableError
 @phpstan-type ErrorsHandler callable(array<Error> $errors, ErrorFormatter $formatter): SerializableErrors
 
-@see https://github.com/vimeo/psalm/issues/6928
-
-@psalm-type ErrorsHandler callable(Error[], ErrorFormatter): SerializableErrors
-
 ### GraphQL\Executor\ExecutionResult Props
 
 ```php
@@ -1676,10 +1663,6 @@ Also, it is possible to override warning handler (which is **trigger_error()** b
 
 @phpstan-type WarningHandler callable(string $errorMessage, int $warningId, ?int $messageLevel): void
 
-@see https://github.com/vimeo/psalm/issues/7527
-
-@psalm-type WarningHandler callable(string, int, int|null): void
-
 ### GraphQL\Error\Warning Constants
 
 ```php

src/Error/Warning.php

@@ -11,10 +11,6 @@
  * Also, it is possible to override warning handler (which is **trigger_error()** by default).
  *
  * @phpstan-type WarningHandler callable(string $errorMessage, int $warningId, ?int $messageLevel): void
- *
- * @see https://github.com/vimeo/psalm/issues/7527
- *
- * @psalm-type WarningHandler callable(string, int, int|null): void
  */
 final class Warning
 {

src/Executor/ExecutionResult.php

@@ -34,10 +34,6 @@
  * }
  * @phpstan-type ErrorFormatter callable(Throwable): SerializableError
  * @phpstan-type ErrorsHandler callable(array<Error> $errors, ErrorFormatter $formatter): SerializableErrors
- *
- * @see https://github.com/vimeo/psalm/issues/6928
- *
- * @psalm-type ErrorsHandler callable(Error[], ErrorFormatter): SerializableErrors
  */
 class ExecutionResult implements \JsonSerializable
 {

src/Executor/Executor.php

@@ -14,16 +14,7 @@
  * Implements the "Evaluating requests" section of the GraphQL specification.
  *
  * @phpstan-type FieldResolver callable(mixed, array<string, mixed>, mixed, ResolveInfo): mixed
- *
- * @see https://github.com/vimeo/psalm/issues/6928
- *
- * @psalm-type FieldResolver callable(mixed, array, mixed, ResolveInfo): mixed
- *
  * @phpstan-type ImplementationFactory callable(PromiseAdapter, Schema, DocumentNode, mixed, mixed, array<mixed>, ?string, callable): ExecutorImplementation
- *
- * @see https://github.com/vimeo/psalm/issues/6928, https://github.com/vimeo/psalm/issues/7527
- *
- * @psalm-type ImplementationFactory callable(PromiseAdapter, Schema, DocumentNode, mixed, mixed, array, ?string, callable): ExecutorImplementation
  */
 class Executor
 {