Move the keyframe property defaults table into @hyperframes/core
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 75/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- typescript
- 領域
- full-stack
調査の方向性
重複しているテーブルは、packages/sdk/src/engine/keyframeBackfill.ts の KEYFRAME_PROPERTY_DEFAULTS と、packages/studio/src/hooks/gsapShared.ts の PROPERTY_DEFAULTS です。両方を読んで内容が一致することを確認し、そのうえで、テーブルにないプロパティを各呼び出し元がどう扱うかを確認してください。SDK はそのプロパティをスキップし、Studio は 0 にフォールバックします。完了条件は、エクスポートされたテーブルが1つだけ @hyperframes/core に置かれ、両パッケージがそれをインポートし、KEEP IN SYNC のコメントと TODO が削除され、既存の sdk と studio のテストが通り、core の小さなテストがそのエクスポートをカバーしていることです。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- TypeScript
- スター
- 54.1k
- フォーク
- 4.9k
- 平均マージ
- 7時間 26分
- マージ済み PR（30日）
- 724
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
heygen-com/hyperframes のほかの issue
-
embedded-captions: occlusion gate records PASS when check-occlusion times out before its verdict対応中かも @xiajiahang が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
heygen-com/hyperframes#5485 ·
メンテナーはふだん 1 日以内に返信
-
`hyperframes docs examples` lists templates that don't exist対応中かも @Muhtasim-Munif-Fahim が 2 日前に担当しました。 オープンbug difficulty/easy help wanted triage/ready
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
heygen-com/hyperframes#5393 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
lint: add tests for three caption rules that have none対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンdifficulty/easy enhancement help wanted triage/ready
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
heygen-com/hyperframes#5388 ·
メンテナーはふだん 1 日以内に返信
-
docs: add open, catch-up, timeline and history to the CLI reference対応中かも @meghna810 が 2 日前に担当しました。 オープンdifficulty/easy documentation help wanted triage/ready
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
heygen-com/hyperframes#5386 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
gcp-cloud-run image pins Chrome 148, which drops `background-clip: text` glyphs inside layered children; the CLI's Chrome 152 paints them対応中かも @user-github-me が 3 日前に担当しました。 オープンbug difficulty/medium triage/ready
難易度 1/5 1時間未満 初心者へのやさしさ 84/100
heygen-com/hyperframes#5117 ·
メンテナーはふだん 1 日以内に返信
heygen-com/hyperframes の issue をすべて見る
似ている issue
-
bug help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
agent-task risk:low
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
guyellis/http-status-check#512 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
microsoft/playwright#43327 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
microsoft/TypeScript#64758 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
opensearch-project/OpenSearch-Dashboards#12937 ·
メンテナーはふだん 2 日以内に返信