langgenius/dify

improve dict check in py

Fermée

#22 651 ouverte le 18 juil. 2025

Voir sur GitHubGuide contributeur
 (15 commentaires) (2 réactions) (2 personnes assignées)TypeScript (22 210 forks)batch import
good first issueno-issue-activity💪 enhancement

Métriques du dépôt

Stars
 (141 412 étoiles)
Métriques de merge PR
 (Merge moyen 1j 21h) (579 PRs mergées en 30 j)

Description

Self Checks

  • I have read the Contributing Guide and Language Policy.
  • I have searched for existing issues search for existing issues, including closed ones.
  • I confirm that I am using English to submit this report, otherwise it will be closed.
  • Please do not modify this template :) and fill in all the required fields.

1. Is this request related to a challenge you're experiencing? Tell me about your story.

now a lot of type is bare dict

rg ": dict\)" --glob "*.py" | wc -l
165

use pydantic can reduce many dup pattern like

if not field:
    raise ValueError...

example #24837

2. Additional context or comments

https://lexi-lambda.github.io/blog/2019/11/05/parse-don-t-validate/

3. Can you help us with this feature?

  • I am interested in contributing to this feature.

Guide contributeur

Direction de recherche
Identifiez toutes les occurrences de types dict nus dans les fichiers Python en utilisant la commande grep fournie. Étudiez les modèles Pydantic existants (par exemple, de l'issue #24837) pour comprendre le modèle, puis remplacez les dict nus par des modèles Pydantic appropriés en assurant une validation correcte. Coordonnez vous avec les deux assignés pour éviter les doublons et aligner la portée.
Stack technique
python
Domaine
backenddeveloper experience
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Bloquée
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonPydantic basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
20