Skip to content

utility系のnpm依存 (clone, fast-deep-equal) を削除#12

Merged
terurou merged 14 commits intodenkiyagi-forkfrom
improvement/remove-utility-dependencies
Nov 14, 2025
Merged

utility系のnpm依存 (clone, fast-deep-equal) を削除#12
terurou merged 14 commits intodenkiyagi-forkfrom
improvement/remove-utility-dependencies

Conversation

@falworks-dy
Copy link

@falworks-dy falworks-dy commented Nov 13, 2025

変更概要

  • 事前リファクタリング
    • src/utils.js の内容を分割し、新ディレクトリー src/utils/ 内に arrays.js, decode.js として再配置。
      呼び出し側の import も更新
  • 下記のnpm依存を削除
    • cloneTTFSubset モジュールで使用)
    • fast-deep-equalCFFDict モジュールで使用)
  • 上記について、新たに作成した独自実装で置き換え
    • 独自実装は src/utils/ に配置 (clone.js, deep-equal.js)
    • それぞれ置き換え前の関数はオーバースペックであったため、
      新実装では実際の用途に合わせて単純な入力のみを受け付け、想定外の入力には例外を出すように

テスト

既存のテストはすべてAPI経由であったため、test/unit/ を新設。
今回追加分の各モジュールについてユニットテストを作成・配置

@falworks-dy falworks-dy requested a review from terurou November 13, 2025 19:51
@terurou terurou merged commit 9ef96e1 into denkiyagi-fork Nov 14, 2025
3 checks passed
@terurou terurou deleted the improvement/remove-utility-dependencies branch November 14, 2025 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants