Quantcast
Channel: Node.jsタグが付けられた新着記事 - Qiita
Viewing all articles
Browse latest Browse all 8829

レガシーサービスにおけるパフォーマンス改善 - 使われていないCSSを削除してくれるCLIを作った話

$
0
0

リクルートテクノロジーズ でフロントエンドエンジニアをしている@SW20_Toshiです。
本記事はRecruit Engineers Advent Calendar 2019 - Adventar 20日目の記事です。

皆様はウェブのパフォーマンスを気にしていますか?
おそらく大抵の方はSQLのチューニングやロジックの改良などをした経験があるのではないでしょうか?
今回は、レガシーサービスにおけるパフォーマンス改善として、Puppeteerを使って不要なCSSを削除する取り組みを行いました。
ツールはOSSとして公開しているので使ってみてください!

サイボウズ株式会社のこちらの記事には大変お世話になりました!
ありがとうございました!

きっかけ

10年近くABテストや機能追加を繰り返してきたWebサービス....
スクリーンショット 2019-12-19 14.39.50.png

1画面に大量のCSSファイルが読み込まれていて、カバレッジ関しては目も当てられない酷さ

パフォーマンス・開発者観点で、次のような問題があげられます。

パフォーマンス観点

  • カバレッジが低いファイルが複数あり、描画速度が遅くなる
    • CSSの解決のされ方についてはこの記事がわかりやすいです
  • minifyもconcatもできていない

開発者観点

  • CSSが3万行以上あるため、影響範囲がわからない。
    • 案件に入る前に、リファクタリングが必要なことも...

やったこと

css-optimizationというツールを開発し、結果としてCSSを180KBの削減に成功しました
スクリーンショット 2019-12-19 14.40.35.png
light houseのパフォーマンススコアも大幅に改善
before
スクリーンショット 2019-12-19 11.54.51.png
after
スクリーンショット 2019-12-19 11.54.42.png

作ったツールについて

そのページの使われているCSSのみを抽出するcss-optimizationというCLIを開発しました。
任意のURLと操作をyamlに記述するだけで、使われているCSSのみを取得することができます。

name:demo# CSSを最適化したいページのURLurl:'https://hogehoge/fuga/'# userAgentを指定userAgent:'bot'steps:# 良しなにDOM操作をして、モーダルとかを表示-action:type:hoverselector:'#js-mylist>div>ul>li:nth-child(2)>div>div>ul'-action:type:clickselector:'#js-mylist-myHistory'-action:type:waitduration:500# 意図した操作が行われているか、スクショをとる-action:type:screenshotname:'demo'

前提条件(ログインなど)を定義することも可能です:v:
ツールを開発しリリースへ向けて作業していく上で、躓いた点などを説明していきたいと思います。

Puppeteerとは

puppeteerとは、GUIを操作することなく、プログラムからAPIでChromeを制御できる ライブラリ です。

Puppeteerでカバレッジを取る方法

// カバレッジ収集を開始awaitpage.coverage.startCSSCoverage();// ブラウザを操作しカバレッジを収集したいページを表示するawaitpage.goto('ここにURLが入ります');// カバレッジ収集を終了constcoverage=awaitpage.coverage.stopCSSCoverage();

coverageの中には、各CSSファイルの内容と、何文字目から何文字目から使われているかが返り値として入っています。
これをゴニョゴニョして最適化されたものだけが取り出されると思いきや、楽にはいきませんでした。
イメージとしてはこんな感じ

constcoverage=[{url:"ファイルのURL",text:"ファイルの中身",ranges:[{start:"開始位置",end:"終了位置"},{start:"開始位置",end:"終了位置"},]},]constoptimizedCSSMap=cssCoverage.map(entry=>{const{url,ranges,text}=entryreturn{fileName:convertUrl(url),coverage:ranges.map(range=>{returncode+text.slice(range.start,range.end)+'\n'}).join(''),}})

ここで上がってきた課題

  • 動的に表示される要素が、使われていないと判定されてしまう

これは現状レンダリングしているものを「使用している」と判断しているためです。
JSによって動作するモーダルやポップアップ、メニューバーがある場合は表示させないと「使用している」と判断されません。
→つまりは、stopCSSCoverageまでにpuppeteerでDOM操作をしなければならない

  • media queryfont-faceなどの@ルールが使われていないとみなされる
  • コメントを残したい
  • 最適化されたCSSを、読み込ませて回帰テストを楽にしたい

どうやって解決したか

動的に表示される要素を考慮したカバレッジを出力する

↓のように、stopCSSCoverageの前にDOMを操作する処理を書いても良いのですが
他のサービスに横展開するときにハードルが高くなってしまったり、各画面に合わせてソースコードを書き換えるのはしんどいです

// カバレッジ収集を開始awaitpage.coverage.startCSSCoverage();// ブラウザを操作しカバレッジを収集したいページを表示するawaitpage.goto('ここにURLが入ります');// ここに、動的に操作する処理を書くawaitpage.hover('hoge')awaitpage.click('fuga')// カバレッジ収集を終了constcoverage=awaitpage.coverage.stopCSSCoverage();

そこで、pupperiumのyamlでpuppeteerを操作する機能を流用しました。

steps:# 良しなにDOM操作をして、モーダルとかを表示-action:type:clickselector:'#hoge'-action:type:waitduration:500-action:type:screenshotname:'demo'

media queryやfont-face、コメントを残すために

PostCSSには、ASTを簡単に操作するためのAPIが用意されています。
ASTはJavaScriptのオブジェクトで簡単に取り扱うことができるため、AtRuleCommentノードの探索と削除を行います。

constisUnneededNode=(node,coverage)=>{// Root, Comment, Declarationは削除しないif(['root','comment','decl'].includes(node.type)){returnfalse;}// AtRuleは削除しないif(node.type==='atrule'){returnfalse;}};// ASTの探索root.walk(node=>{// 削除対象か?if(isUnneededNode(node)){// 削除対象ならASTから削除するnode.remove();}});

ノードの一覧

  • Rootノード: ASTの1番上のノード(Rootノードは親ノードがない)
  • Ruleノード: 1つのルールセット
  • AtRuleノード: 1つの@ルール
  • Declarationノード: 1プロパティ宣言
  • Comment: 1つのコメント

回帰テストを楽にする

最適化されたCSSを代わりに読み込ませるためには、setRequestInterceptionでリクエストに対するインターセプトを有効し、request.continueで上書きをすることができます。

// リクエストに対するインターセプトを有効にするpage.setRequestInterception(true)// リクエストを監視するpage.on('request',request=>{if(scrapingUrl===request.url()){// overridesが上書きする内容request.continue(overrides).catch(err=>console.error(err))}else{request.abort().catch(err=>console.error(err))}})

CSSを上書きした後に、同じシナリオで画像比較
わかりやすいように、比較結果をHTMLとして吐き出してくれるCLIも作りました。

68747470733a2f2f71696974612d696d6167652d73746f72652e73332e61702d6e6f727468656173742d312e616d617a6f6e6177732e636f6d2f302f3231383230312f66346565333336652d306239372d306132372d656366632d3138383163643635393233302e706e67.png

最後に

使われていないCSSを削除して、concatするだけでここまでパフォーマンスが改善するとは思いませんでした。
機会があれば、普段のパフォーマンスモニタリングやパフォーマンスバジェットなどもお話しできればと思います。


Viewing all articles
Browse latest Browse all 8829

Trending Articles