Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Build Compression #116

Open
tatsukoni-pra opened this issue May 14, 2022 · 2 comments
Open

Fix Build Compression #116

tatsukoni-pra opened this issue May 14, 2022 · 2 comments
Assignees
Labels
v1 Required for first release

Comments

@tatsukoni-pra
Copy link
Collaborator

tatsukoni-pra commented May 14, 2022

ビルド方式を、圧縮, 難読化しない方式に変更する。
最初に見てた記事 ↓
https://r17n.page/2019/09/28/js-minify-packages/

@tatsukoni-pra tatsukoni-pra added the v1 Required for first release label May 14, 2022
@tatsukoni-pra tatsukoni-pra self-assigned this May 14, 2022
@tatsukoni-pra
Copy link
Collaborator Author

tatsukoni-pra commented May 15, 2022

調査メモ

  • コードの難読化がダメっぽい。
  • 一方でコードの圧縮(minification)については、以下の圧縮は許可されているみたい
     - 空白、改行、コードコメント、およびブロック区切り文字の削除
     - 変数名と関数名の短縮
     - JavaScriptファイルの数を折りたたむ

参考

@tatsukoni-pra
Copy link
Collaborator Author

tatsukoni-pra commented May 15, 2022

圧縮について

圧縮要件

上でも記載しているように

  • 空白、改行、コードコメント、およびブロック区切り文字の削除
  • 変数名と関数名の短縮
  • JavaScriptファイルの数を折りたたむ

は許可されている。
また、「Google developersで推奨される圧縮方法」があり、UglifyJsが推奨されている。

現状のビルド方式

一方現状のビルドは、内部ではwebpackのmode: productionでビルドされているのだと思うが、こちらについても内部ではUglifyJsPluginによる圧縮が実施されている。
なので、現状のビルド方式のままでもChrome拡張機能ポリシーが要求するレベルのminifyになっている気がする。

参考情報

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v1 Required for first release
Projects
None yet
Development

No branches or pull requests

1 participant