Allow custom type checking via plugin for function definitions
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Merge medio
- 1 d 18 h
- PR fusionados (30 d)
- 54
Descripción
Feature
The possibility to allow custom type checking for function definitions using plugins would be very helpful to e.g. restrict allowed return types of a function. At the moment the plugin interface doesn't seem to have hooks for function definition type checking.
Pitch
Example: I want to restrict the return types of functions to allow them only to return a specific type.
Using get_function_hook/get_method_hook in a custom plugin like this
# -*- coding: utf-8 -*-
import mypy.errorcodes
import mypy.plugin
import mypy.plugins.default
import mypy.types
import typing
RESTRICT_UNIONS: typing.Final = mypy.errorcodes.ErrorCode(
"restrict-unions",
"some description",
"custom_stuff")
class RestrictUnionsPlugin(mypy.plugin.Plugin):
def get_function_hook(self, fullname: str) -> typing.Callable[[mypy.plugin.FunctionContext], mypy.types.Type] | None:
return self.restrict_union_size
def get_method_hook(self, fullname: str) -> typing.Callable[[mypy.plugin.MethodContext], mypy.types.Type] | None:
return self.restrict_union_size
@staticmethod
def _check_union_size(ctx: mypy.plugin.FunctionContext | mypy.plugin.MethodContext, return_type) -> None:
if isinstance(return_type, mypy.types.UnionType):
if (type_count := len(ctx.default_return_type.items)) > 1:
ctx.api.fail(
f"Too many union types: {type_count} > 1, {[str(typ) for typ in ctx.default_return_type.items]}",
ctx.context, code=RESTRICT_UNIONS)
def restrict_union_size(self, ctx: mypy.plugin.FunctionContext | mypy.plugin.MethodContext) -> mypy.types.Type:
self._check_union_size(ctx, ctx.default_return_type)
return ctx.default_return_type
def plugin(version: str):
# ignore version argument if the plugin works with all mypy versions.
return RestrictUnionsPlugin
has the disadvantage that issues are raised on function call(s), not on definition.
E.g. running mypy with this custom plugin on the following piece of code
def some_method_with_union_return(some_input: str | int) -> str | int:
return some_input
def some_other_method() -> None:
some_method_with_union_return(5)
def another_method() -> None:
some_method_with_union_return("foo")
finds 2 errors
source\main.py:6: error: Too many union types: 2 > 1, ['builtins.str', 'builtins.int'] [restrict-unions]
source\main.py:10: error: Too many union types: 2 > 1, ['builtins.str', 'builtins.int'] [restrict-unions]
Found 2 errors in 1 file (checked 2 source files)
This also means in case multiple return values are ok for certain methods, each place the method with multiple return types is called needs a # type: ignore[restrict-unions]
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
Empieza leyendo las interfaces de plugins relacionadas con get_function_hook y get_method_hook, y luego sigue el recorrido hasta donde se comprueban los tipos de las definiciones de funciones, en lugar de donde se analizan las llamadas. El cambio estará completo cuando un plugin pueda informar de errores de tipo de retorno personalizados en cada definición de función, mientras que sigan siendo posibles las excepciones para los métodos permitidos; en el issue no se menciona ningún archivo de prueba específico.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- compilers
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100