Skip to content

chore(deps): 使われていない gulp-concat / gulp-uglify / natives を外す - #155

Merged
deguchi merged 1 commit into
masterfrom
chore/remove-unused-gulp-deps
Aug 6, 2026
Merged

chore(deps): 使われていない gulp-concat / gulp-uglify / natives を外す#155
deguchi merged 1 commit into
masterfrom
chore/remove-unused-gulp-deps

Conversation

@deguchi

@deguchi deguchi commented Aug 6, 2026

Copy link
Copy Markdown
Member

概要

未使用の devDependencies 3件を削除します。npm 未使用依存の棚卸し(knip + depcheck のクロスチェック)で検出したもので、ローカルクローンのソース・設定ファイルで実際に未使用であることを確認済みです。

  • gulp-concatgulpfile.babel.js に require/import なし
  • gulp-uglify — 同上。圧縮は browserify transform の uglifyifypackage.jsoncompile スクリプト)が担当している
  • natives — ソース・gulpfile・hooks のいずれからも参照なし。package-lock.json 内で他パッケージから間接要求されている箇所もなし

gulp-load-plugins のようなプラグイン自動ロード機構は使っていないため、暗黙の利用もありません。

動作確認

npm run copy を通しで実行し、browserify のコンパイルと gulp の sass / copy_css / copy_fonts / copy_jsons が全て正常終了することを確認しました。

@babel/register について

当初は棚卸しの検出結果に @babel/register も含まれていましたが、削除対象から除外しました。gulpfile が gulpfile.babel.js という名前のため、gulp CLI が interpret 経由で @babel/register を自動 require します(実行ログにも Loaded external module: @babel/register が出ます)。静的解析ツールでは検出できない利用形態のため、外すと gulp タスクが壊れます。

🤖 Generated with Claude Code

gulpfile.babel.js から require されておらず、gulp-load-plugins のような
自動ロード機構も使っていないため未使用。圧縮は browserify transform の
uglifyify が担当している。natives は他パッケージからの間接要求もなし。

npm run copy を通しで実行し、browserify のコンパイルと gulp の
sass / copy_css / copy_fonts / copy_jsons が正常終了することを確認した。

なお @babel/register は gulpfile が gulpfile.babel.js という名前のため
gulp CLI が interpret 経由で自動 require しており、削除すると壊れるので
対象から除外した。

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@deguchi
deguchi merged commit 961d9a1 into master Aug 6, 2026
3 checks passed
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.

1 participant