Inconsistent behavior when validating type parameter substitutions
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 35.9k
- Métriques de merge des PR
- Métriques de PR en attente
Description
Bug report
Bug description:
There are currently three different cases (two of them being closely related) where type parameters can be substituted/parameterized with concrete types:
-
On a user-defined generic class:
class MyGeneric[T1, T2]: ... alias = MyGeneric[int, str] -
On an already parameterized alias (following the previous example):
temp_alias = MyGeneric[int, T2] alias = temp_alias[str] -
On a PEP 695 type alias:
type MyAlias[T1, T2] = dict[T1, T2] gen_alias = MyAlias[int, str]
All of these three cases have slightly different behavior. The one that seems to be the most accurate is the second one. As alias is a _GenericAlias instance, parameterizing it will call _GenericAlias.__getitem__. There is a bunch of logic in there, and it seems that both __typing_prepare_subst__ and then __typing_subst__ (which is only doing type check assertions) are being called.
However, the first case is not making the calls to __typing_subst__, meaning the following would unexpectedly work:
class A[T, **P]: ...
A[int, str]
# ok at runtime, should fail as `P` should be substituted with a valid parameter expression
# (another ParamSpec, the ellipsis, a list/tuple of types or a Concatenate form).
If you do the same on a _GenericAlias instance (matches the second case), an error is raised:
alias = A[T, P]
alias[int, str]
# TypeError: Expected a list of types, an ellipsis, ParamSpec, or Concatenate. Got <class 'str'>
This leads us to the first point: should we apply the same logic between these two cases? To avoid breaking changes, we might have to consider forward references:
class A[T, **P]: ...
A[int, 'ForwardParamSpec']
ForwardParamSpec = ParamSpec('ForwardParamSpec')
So perhaps we can exclude ForwardRef from the type check in ParamSpec.__typing_subst__ (note that it already doesn't work for the second case).
I'll also note that having __typing_subst__ not called is not the only difference. For instance, the second case also calls _unpack_args() on the passed args, while the first case doesn't [^1]
Onto the last case (PEP 695 type aliases), currently no validation is performed whatsoever:
type MyAlias[T1, T2] = dict[T1, T2]
MyAlias[int] # no error
So the second point is: should we apply the same logic as in case 1/2? Again, I don't know if applying the same logic here on type aliases is going to introduce any breaking changes concerns?
[^1]: Here is an example of how this manifests: https://gist.github.com/Viicos/db10da58914809a87c0a82c1b2f19162
CPython versions tested on:
3.14
Operating systems tested on:
Linux
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
Commencez dans Lib/typing.py avec _GenericAlias.getitem, ParamSpec.typing_subst et _unpack_args(), puis comparez ces chemins avec la substitution directe des classes génériques et des alias de type PEP 695. Utilisez les exemples du rapport pour déterminer si la validation doit être cohérente, y compris pour les références anticipées, et vérifiez le comportement qui en résulte dans les trois cas.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- compilers
- Type d'issue
- Bug
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100