Skip to content

chore(deps): remove barryvdh/laravel-ide-helper#376

Open
gvieira18 wants to merge 1 commit into
4.xfrom
chore/remove-ide-helper
Open

chore(deps): remove barryvdh/laravel-ide-helper#376
gvieira18 wants to merge 1 commit into
4.xfrom
chore/remove-ide-helper

Conversation

@gvieira18

@gvieira18 gvieira18 commented Jun 27, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove the barryvdh/laravel-ide-helper dev dependency (and its transitive barryvdh/reflection-docblock) from composer.json / composer.lock.
  • Drop the ide-helper composer script and its call from the setup script, Makefile, and Taskfile.yml.

Why

composer run-script ide-helper hangs indefinitely on the ide-helper:meta step. 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/sail removal. Only composer.json, composer.lock, Makefile, and Taskfile.yml change.

Test plan

  • composer validate passes; composer show barryvdh/laravel-ide-helper → not found
  • No ide-helper references remain (grep -rn ide-helper Makefile Taskfile.yml composer.json)
  • Full quality gate green: Rector, Pint, PHPStan, Pest (831 tests, 2309 assertions)

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.
@gvieira18 gvieira18 requested a review from a team June 27, 2026 16:13
@coderabbitai

coderabbitai Bot commented Jun 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 7c1251bb-dff7-485a-a421-a2430beb66a0

📥 Commits

Reviewing files that changed from the base of the PR and between 0aa31c6 and b00862e.

⛔ Files ignored due to path filters (1)
  • composer.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • Makefile
  • Taskfile.yml
  • composer.json
💤 Files with no reviewable changes (2)
  • Taskfile.yml
  • Makefile

📝 Walkthrough

Walkthrough

Removes barryvdh/laravel-ide-helper from composer.json require-dev, deletes the ide-helper script definition, and removes its invocation from the setup script. The Makefile setup target drops the corresponding composer run-script ide-helper call. The Taskfile.yml setup task replaces that call with composer run post-create-project-cmd, php artisan key:generate, and php artisan storage:link.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: removing barryvdh/laravel-ide-helper.
Description check ✅ Passed The description is directly aligned with the dependency removal and setup-script changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

2 participants