chore(deps): remove barryvdh/laravel-ide-helper#376
Conversation
Its ide-helper:meta step hangs on an O(N^2) translation scanner, amplified by Filament's large multi-locale tree. Laravel Boost already covers the IDE autocomplete it provided.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
💤 Files with no reviewable changes (2)
📝 WalkthroughWalkthroughRemoves 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
barryvdh/laravel-ide-helperdev dependency (and its transitivebarryvdh/reflection-docblock) fromcomposer.json/composer.lock.ide-helpercomposer script and its call from thesetupscript,Makefile, andTaskfile.yml.Why
composer run-script ide-helperhangs indefinitely on theide-helper:metastep. The cause is an O(keys × lines) translation scanner inside the package (php-templates/translations.php), amplified by Filament's large multi-locale tree (~3.3k lang files across ~64 locales) — it pins a CPU core and never returns. It can't be configured away.Scope
Intentionally limited to removing the IDE Helper — no dependency bumps, no
laravel/sailremoval. Onlycomposer.json,composer.lock,Makefile, andTaskfile.ymlchange.Test plan
composer validatepasses;composer show barryvdh/laravel-ide-helper→ not foundide-helperreferences remain (grep -rn ide-helper Makefile Taskfile.yml composer.json)