Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Move the keyframe property defaults table into @hyperframes/core

Cerrado Apto para principiantes
#5,387 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

@meghna810 ya está trabajando en esto.

Desde el 10/10/2026.

  • #5438 de @meghna810 — abierto

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

difficulty/easy enhancement help wanted triage/ready

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/sdk and packages/studio pass, 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

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de heygen-com/hyperframes

Todos los issues de heygen-com/hyperframes

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.