microsoft / microsoft/TypeScript
Suggestion: perform excess property checks when spreading an inline object literal
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.3k
- Merge moyen
- 2 j 4 h
- PR mergées (30 j)
- 132
Description
TypeScript Version: 3.9.2
Search Terms:
Code
declare const someCondition: boolean;
type MyObject = { foo: number; bar?: number };
const a: MyObject = {
foo: 1,
bar: 2,
// ✅ Error because `invalid` is an excess property
invalid: 3,
};
const b: MyObject = {
foo: 1,
...(someCondition
? {
bar: 2,
// ❌ `invalid` is an excess property, but we don't get an error here
invalid: 3,
}
: {}),
};
In the example above, I only want to include specific properties when a condition is met. That's the only reason I'm using spread here.
I understand that TypeScript only performs excess property checks inside of object literals. Currently this does not include inline object literals which are being spread inside of another object literal.
In my experience this is a very common code pattern so it would be great if TypeScript handled this.
Expected behavior:
An error
Actual behavior:
No error
Playground Link:
Related Issues:
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 par reproduire l’exemple fourni de TypeScript 3.9.2 et comparez l’affectation directe d’un littéral d’objet avec le cas d’un spread conditionnel. Étudiez la vérification des propriétés excédentaires du compilateur pour les littéraux d’objet inline utilisés dans des spreads. Le travail est considéré comme terminé lorsque la propriété invalide présentée produit une erreur dans le cas du spread sans casser les spreads conditionnels valides.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- compilers
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- Active
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100