Unify Template type definitions across frontend and backend
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 52/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- python, typescript
- Domaine
- full-stack
Piste de recherche
Compare the type definitions in apps/agent/src/templates.py, apps/app/src/components/template-library/index.tsx, and apps/app/src/components/template-library/seed-templates.ts, starting with the backend UITemplate schema and the missing seed-template fields. Unify the frontend definitions and align them with the backend fields, then verify that chart seed templates no longer need casts and the three definitions cannot drift independently.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
There are three separate Template type definitions that can drift out of sync:
UITemplateinapps/agent/src/templates.py(backend)Templateinterface inapps/app/src/components/template-library/index.tsx(frontend)SeedTemplateinapps/app/src/components/template-library/seed-templates.ts(frontend)
SeedTemplate is missing component_type and component_data fields, so seed templates for charts would need to be cast. Any field added to one type but not the others causes silent mismatches.
Suggested fix
- Unify
TemplateandSeedTemplateinto a single shared frontend type - Ensure the frontend type matches the backend
UITemplateschema (all optional fields present) - Consider a shared schema definition (e.g. Zod on frontend, matching TypedDict on backend)
Files
apps/agent/src/templates.pyapps/app/src/components/template-library/index.tsxapps/app/src/components/template-library/seed-templates.ts
From PR #20 review
- Langage dominant
- TypeScript
- Étoiles
- 1.6k
- Forks
- 205
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire 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.
Autres issues de CopilotKit/OpenGenerativeUI
-
Script dedup key truncation may cause collisionsPeut-être pris @saschabuehrle l’a pris il y a 195 jours. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
CopilotKit/OpenGenerativeUI#87 · 2 commentaires ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
-
Edit existing HTML instead of regenerating from scratchPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
Toutes les issues de CopilotKit/OpenGenerativeUI
Issues similaires
-
fix(data-lake): wizard source step still previews the local slug, not the server-disambiguated oneOuvertedata-lake
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
enhancement good first issue priority: low size: XS
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
Empty label or headline exports the editor hint ("LABEL" / "Headline goes here") into the PNGOuverte
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
-
Spray wall wizard: Done button on the hold review step sits under the navigation header (iOS)Ouvertebug ios mobile priority:P1
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
streamplace/streamplace#1351 ·
Les mainteneurs répondent en général sous 2 jours