`not isinstance(x, cls)` inside classmethod does not narrow type
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
Bug Report
Inside a @classmethod, not isinstance(x, cls) does not appear to narrow the type, while isinstance(x, cls) does.
To Reproduce
This is what I wanted to write:
class Foo:
def __init__(self, x: float) -> None:
...
@classmethod
def from_foo_or_float_cls(cls, x: Self | float) -> Self:
if isinstance(x, cls):
return x
return cls(x) # error: Argument 1 to "Foo" has incompatible type "Self | float"; expected "float" [arg-type]
Further reduced:
class Foo:
@classmethod
def foo_1(cls, x: Self | float) -> None:
assert isinstance(x, cls)
reveal_type(x) # expected: "Self"; was: "Self`0"; ✅
@classmethod
def foo_2(cls, x: Self | float) -> None:
assert not isinstance(x, cls)
reveal_type(x) # expected: "float"; was: "Self`0" | builtins.float"; ❌
Expected Behavior
not isinstance(x, cls) should narrow the type union to remove Self.
Actual Behavior
Self is still part of the type union.
Your Environment
- Mypy version used: 1.19.1
- Mypy command-line flags: none
- Mypy configuration options from
mypy.ini(and other config files): none - Python version used: 3.10
Same behavior can be observers on mypy master with Python 3.14 (tested on the playground).
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Reproduce los dos ejemplos reducidos de classmethod con mypy 1.19.1 o con el master actual y, después, inspecciona la ruta de estrechamiento de tipos de isinstance para Self. Añade cobertura de regresión que muestre que assert not isinstance(x, cls) estrecha Self | float a float, preservando al mismo tiempo el comportamiento existente de estrechamiento positivo.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- compilers
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 55/100