Add: node_modulesをキャッシュするように

[sevenc-nanashi]

内容

node_modulesをキャッシュして、テストを高速化します。

関連 Issue

（なし）

スクリーンショット・動画など

（なし）

その他

（なし）

[Hiroshiba]

PRありがとうございます！

ちょっと調べてたんですが、ci = clean installなんですね。
そしてnode_modulesを消さずにnpm iでlockからインストールする方法は、なんかなさそうな雰囲気を感じました･･･。
npm/cli#564 (comment)

やるとしたら

1. npm iする
2. lockが変わってしまったらlockを戻してnpm ciする

とか･･･？

[sevenc-nanashi]

そんな感じの実装にしてみましたー。

[Hiroshiba]

良さそう！
なのですが、ちょっと探してみた感じ、actions/setup-nodeのcache引数が求めてる挙動をしそうな気がしました。
https://github.com/actions/setup-node/tree/3e8819f8f297df8b29fc3e383683a29be2bf129a#caching-global-packages-data
今のままでも良さそうですが、どうしましょう･･･。

[sevenc-nanashi]

Note: The action does not cache node_modulesとのことです

[Hiroshiba]

たしかにnode_modulesはキャッシュしてくれなそうですが、たぶんnpm ciの高速化にはなるかもと思いました。
lockが異なっていてもキャッシュが働くのであれば、パッケージがちょっと変わっただけの時とかはglobalキャッシュのが早いかも？

でもまあ目標は達成しているのでこのままでも問題なさそう。

[Hiroshiba]

LGTM！

あ、buildの方もnpm ciしてるとこあるかもです。よかったらそちらも…！

[sevenc-nanashi]

どうせbuildは長いので余り変わらないと思いますし、変に変えてビルドが落ちるのが一番怖いので置いておきます…

[Hiroshiba]

大丈夫だと思うのでマージします！

[Hiroshiba]

と思ったのですがやっぱりActions周りなので別の方からもレビューあると嬉しいかもです。
@aoirint さん or @y-chan さん、レビューお願いできると･･･！

[y-chan]

なかなかややこしいことしてるなぁと思いましたが、まあ問題ないと思うのでLGTMです

P.S. 意図をコメントで書いてもいいかもしれない

[Hiroshiba]

今思ったら、lockが同じときnpm ciしなおす感じだと、再インストールしなくても良いような被ってるライブラリも再インストールしちゃいますね。
となると、setup-nodeでcache npmを指定すればそこの課題を解決できるので、実は最高なのではという気がしてきました。
https://github.com/actions/setup-node#caching-global-packages-data

[sevenc-nanashi]

graph LR
  action_start(開始)
  action_check(package-lock.jsonにあうキャッシュの存在確認)
  action_cache(キャッシュの復元)
  action_npm_ci(npm ci)
  action_other(...)
  action_to_cache(キャッシュの保存)

  action_start --> action_check -- 存在する --> action_cache --> action_other
  action_check -- 存在しない --> action_npm_ci --> action_other
  action_other --> action_to_cache

Loading

自分の理解です

[Hiroshiba]

@y-chan さんもOKとのことなので、マージで大丈夫だという気持ちです！
マージボタン押そうとしたのですが、 @sevenc-nanashi さん的に大丈夫でしょうか 👀
（なにか検討中だったら悲しいことになりそうなので確認した次第です）

[y-chan]

となると、setup-nodeでcache npmを指定すればそこの課題を解決できるので、実は最高なのでは

今現在も~/.npmはキャッシュされているので、それを適用しても挙動としては変わらないはず...?

voicevox/.github/workflows/test.yml

Lines 26 to 31 in 9869e23

	- uses: actions/cache@v3
	with:
	path: ~/.npm
	key: ${{ env.cache-version }}-node-${{ hashFiles('**/package-lock.json') }}
	restore-keys: |
	${{ env.cache-version }}-node-

もう特に問題ないと思います...!

[Hiroshiba]

マージします！