Package updates#592
Conversation
WalkthroughDie Änderungen aktualisieren Laravel-IDE-Metadaten und generierte Hilfsmethoden sowie mehrere Frontend-Entwicklungsabhängigkeiten. Ergänzt werden Image- und Konfigurationshinweise, Laravel-Fake-Wrapper und aktualisierte Paketversionen. ChangesLaravel-Tooling-Aktualisierung
Estimated code review effort: 3 (Moderate) | ~20 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 PHPStan (2.2.5)PHPStan was skipped because the config uses disallowed 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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
_ide_helper.php (1)
12056-12068: 📐 Maintainability & Code Quality | 🔵 Trivial | 🏗️ Heavy liftTypdeklarationen für neue PHP-Methoden ergänzen oder die generierte Datei ausdrücklich ausnehmen.
Die neuen Wrapper deklarieren weder Parametertypen noch Rückgabetypen, obwohl die Repository-Guidelines dies für PHP-Methoden verlangen. Da
_ide_helper.phpgeneriert wird, bitte den Generator bzw. das Template anpassen und anschließend neu generieren — nicht nur diese Datei manuell patchen. Dabei die tatsächlichen Laravel-13.20.0-Signaturen verifizieren.Also applies to: 14006-14030, 15871-15880, 21195-21206, 21388-21398
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@_ide_helper.php` around lines 12056 - 12068, Update the generator or template that produces the wrapper methods including assertQueuedTimes and the additionally affected methods, adding parameter and return type declarations matching the verified Laravel 13.20.0 signatures. Regenerate _ide_helper.php from that source instead of editing the generated file manually; if generated files are intentionally exempt, document that repository policy explicitly.Source: Coding guidelines
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@_ide_helper.php`:
- Around line 12056-12068: Update the generator or template that produces the
wrapper methods including assertQueuedTimes and the additionally affected
methods, adding parameter and return type declarations matching the verified
Laravel 13.20.0 signatures. Regenerate _ide_helper.php from that source instead
of editing the generated file manually; if generated files are intentionally
exempt, document that repository policy explicitly.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 90aa1c65-15af-411d-8ff4-8f5c1d5e3151
⛔ Files ignored due to path filters (2)
composer.lockis excluded by!**/*.lockpnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (3)
.phpstorm.meta.php_ide_helper.phppackage.json
Summary by CodeRabbit
Verbesserungen
Wartung