Duplicate keys cause earlier TypedDict values to be skipped during type checking

Ouverte
#21,841 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
74/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Calme
Stack technique
python
Domaine
compilers

Piste de recherche

Commencez dans mypy/checkexpr.py autour des lignes 839-842 et reproduisez le problème avec example.py à l’aide de la commande mypy fournie. Suivez la manière dont les clés TypedDict en double remplacent les expressions précédentes ; le travail est terminé lorsque l’expression de chaque valeur est vérifiée du point de vue des types et que le cas parse(123) signale l’erreur arg-type attendue.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

bug topic-typed-dict

Bug Report

When a dictionary literal used to construct a TypedDict contains duplicate
keys, mypy only type-checks the final value for that key. Earlier value
expressions are skipped entirely.

This can hide real errors because Python still evaluates every value expression
in the dictionary literal, including values that are later overwritten by a
duplicate key.

To Reproduce

from typing import TypedDict


class Payload(TypedDict):
    value: int


def parse(value: str) -> int:
    return len(value)


payload = Payload({"value": parse(123), "value": 0})
$ mypy --show-error-codes --no-incremental example.py
Success: no issues found in 1 source file

Running the program demonstrates that the skipped expression is evaluated:

TypeError: object of type 'int' has no len()

Expected Behavior

Mypy should type-check every value expression in the dictionary literal, even
if a later duplicate key determines the final value of the TypedDict field.
In this example, it should report an error similar to:

error: Argument 1 to "parse" has incompatible type "int"; expected "str"  [arg-type]

The same expression is correctly rejected when the target is a regular
dict[str, int]. The issue also occurs with a contextually typed literal:

payload: Payload = {"value": parse(123), "value": 0}

Actual Behavior

Mypy reports no issues because only the last value associated with the
duplicate key is checked.

Additional Context

The current implementation replaces the previously collected expression
without type-checking it and has a TODO to type-check all duplicate-key values:

https://github.com/python/mypy/blob/6eb141f03a8b43c274f15cc4978d617a71d8f37e/mypy/checkexpr.py#L839-L842

PR #15425 also mentioned repeated TypedDict keys as a known limitation, but I
could not find an existing issue that tracks it:

https://github.com/python/mypy/pull/15425

Environment Used to Reproduce

  • Mypy versions used:
    • mypy 2.3.0 (compiled: yes)
    • mypy 2.4.0+dev.6eb141f03a8b43c274f15cc4978d617a71d8f37e (compiled: no)
  • Mypy command-line flags: --show-error-codes --no-incremental
  • Mypy configuration options: none
  • Python version used: 3.12.8
Langage dominant
Python
Étoiles
20.6k
Forks
3.3k
Merge moyen
1 j 18 h
PR mergées (30 j)
54

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de python/mypy

Toutes les issues de python/mypy

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.