microsoft / microsoft/TypeScript
Removing optional modifier in homomorphic mapped types does not work in generic contexts since 5.5.x
@ahejlsberg ya está trabajando en esto.
Desde el 9/9/2024.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.4k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 117
Descripción
🔎 Search Terms
optional modifier, required fields, generic, NonNullable, strictNullChecks, homomorphic mapped types
🕗 Version & Regression Information
- This changed between versions 5.4.5 and 5.5.2
- This changed in commit or PR e418f8d12c5f1b6c10fc3127764f34dad44d4586 (as reported by
every-ts)
⏯ Playground Link
💻 Code
const example = <T extends string>(): any => {
// Replacing with any specific sub-type of `string` makes the types correct again.
// type T = string;
type SomeOptional = { [key in T]?: 1 };
type AllRequired = { [key in keyof SomeOptional]-?: 1 };
type RequiredValues = AllRequired[keyof AllRequired];
// Complains in 5.4.x, but fine in 5.5.x
const x: RequiredValues = undefined
return x;
};
🙁 Actual behavior
Using -? leaves the fields marked as optional, and taking the values of the resulting type gives a union with undefined.
🙂 Expected behavior
Using -? makes all fields required, and taking the values of the resulting type gives a union of field types.
Additional information about the issue
Requires a --strictNullChecks flag.
Replacing keyof SomeOptional with NonNullable<keyof SomeOptional> or keyof SomeOptional as keyof SomeOptional fixes the issue. It looks like this problem is limited to homomorphic mapped types.
Using Required<{ [P in keyof SomeOptional]: 1; }> in place of { [key in keyof SomeOptional]-?: 1 } results in the same behaviour.
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.
Evaluación
Este issue todavía no se ha evaluado.