Move the keyframe property defaults table into @hyperframes/core
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 75/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- full-stack
Línea de trabajo
La tabla duplicada es KEYFRAME_PROPERTY_DEFAULTS en packages/sdk/src/engine/keyframeBackfill.ts y PROPERTY_DEFAULTS en packages/studio/src/hooks/gsapShared.ts. Lee ambas para confirmar que coinciden y luego revisa cómo maneja cada llamador una propiedad que falta en la tabla: el SDK la omite, Studio recurre a 0. Está terminado cuando una tabla exportada vive en @hyperframes/core, ambos paquetes la importan, el comentario KEEP IN SYNC y el TODO desaparecen, las pruebas existentes de sdk y studio pasan y una pequeña prueba en core cubre la exportación.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
The same table of keyframe property defaults is written out twice:
packages/sdk/src/engine/keyframeBackfill.ts(KEYFRAME_PROPERTY_DEFAULTS)packages/studio/src/hooks/gsapShared.ts(PROPERTY_DEFAULTS)
The SDK copy carries a comment saying the two must stay in sync, or keyframes written by the SDK drift from ones written by Studio, plus a TODO to move the table into @hyperframes/core. Today the two tables are identical, but nothing enforces that.
Proposed solution
Move the table into @hyperframes/core (both packages already depend on it), export it, and import it from both places. Keep each caller's current behaviour, including how it treats a property that isn't in the table: the SDK skips it, while Studio's callers fall back to 0.
Acceptance criteria
- One definition of the table, in
@hyperframes/core, imported by the SDK and Studio. - The "KEEP IN SYNC" comment and the TODO are removed.
- Existing tests in
packages/sdkandpackages/studiopass, and there's a small test in core covering the exported table.
- Lenguaje dominante
- TypeScript
- Estrellas
- 54.1k
- Forks
- 4.9k
- Merge medio
- 8 h 39 min
- PR fusionados (30 d)
- 736
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de heygen-com/hyperframes
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
heygen-com/hyperframes#5538 ·
Los mantenedores suelen responder en 1 día
-
`hyperframes docs examples` lists templates that don't existPosiblemente ocupada @Muhtasim-Munif-Fahim la tomó hace 2 días. Abiertobug difficulty/easy help wanted triage/ready
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
heygen-com/hyperframes#5393 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
lint: add tests for three caption rules that have nonePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertodifficulty/easy enhancement help wanted triage/ready
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
heygen-com/hyperframes#5388 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
gcp-cloud-run image pins Chrome 148, which drops `background-clip: text` glyphs inside layered children; the CLI's Chrome 152 paints themPosiblemente ocupada @user-github-me la tomó hace 4 días. Abiertobug difficulty/medium triage/ready
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
heygen-com/hyperframes#5117 ·
Los mantenedores suelen responder en 1 día
-
Docs: clarify that "Enable auto-update" is only available in the Claude Code terminal (CLI) /plugin UIPosiblemente ocupada @rumi7911 la tomó hace 7 días. Abiertodifficulty/easy documentation triage/ready
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
heygen-com/hyperframes#5027 ·
Los mantenedores suelen responder en 1 día
Todos los issues de heygen-com/hyperframes
Issues similares
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 70/100
siyuan-note/siyuan#20517 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
NaturalIntelligence/fast-xml-parser#890 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
zsviczian/obsidian-excalidraw-plugin#3000 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
pnpm/get.pnpm.io#68 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
nuxt-modules/better-auth#578 · 2 comentarios ·
Los mantenedores suelen responder en 1 día