modelcontextprotocol / modelcontextprotocol/php-sdk
[Server] Handler type uses bare Closure, hard to decorate RegistryInterface under strict PHPStan
Personne n'a encore pris cette issue.
- Langage dominant
- PHP
- Étoiles
- 1.6k
- Forks
- 173
- Merge moyen
- 2 j 49 min
- PR mergées (30 j)
- 23
Description
The Handler phpstan-type on ElementReference uses a bare \Closure. Implementing RegistryInterface with checkMissingCallableSignature enabled is then impossible: bare Closure fails the signature rule, and adding one breaks contravariance against the interface. \Closure(mixed...): mixed on the alias fixes both. Happy to send the one-liner.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Trouvez l’alias de type phpstan-type Handler sur ElementReference et examinez comment RegistryInterface l’utilise. Remplacez le type Closure nu par la signature compatible décrite dans l’issue, puis exécutez les vérifications PHPStan du projet pour confirmer que checkMissingCallableSignature passe sans erreur de contravariance.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- php
- Domaine
- backend
- Type d'issue
- Bug
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Activité
- Active
- Clarté
- Plutôt claire
- Accessibilité débutants
- 78/100