microsoft / microsoft/TypeScript
Certain conditional types allow unsound assignments
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.4k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 117
Descripción
🔎 Search Terms
"conditional type" generic assignment
🕗 Version & Regression Information
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about conditional types
⏯ Playground Link
💻 Code
type Box<T> = { value: T }
type UnboxInner<T> = T extends Box<infer R> ? UnboxInner<R> : { unboxed: T }
type Unbox<T> = UnboxInner<T>['unboxed']
declare function unbox<T>(x: T): Unbox<T>
function example<T>(x: T) {
let a = unbox(x)
a = 'this should be an error'
}
🙁 Actual behavior
In example we can assign arbitrary values to a, even though it's typed as Unbox<T>.
🙂 Expected behavior
Attempting to assign invalid values to a (just about anything, since its type depends on a type parameter) should cause a type error.
Additional information about the issue
These issues may possibly be related:
- #52021
- #35533
cc @webstrand
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
Comienza con la reproducción vinculada en TypeScript Playground y compara el comportamiento con los issues relacionados #52021 y #35533. Rastrea la comprobación de asignaciones genéricas de tipos condicionales en el compilador de TypeScript; se considera terminado cuando se rechaza la asignación mostrada y una prueba de regresión cubre el comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- compilers
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100