Skip to content

fix(prism): avoid requiring full laravel framework#1025

Open
MiddleSokilAI wants to merge 1 commit into
prism-php:mainfrom
MiddleSokilAI:sokil/remove-runtime-laravel-framework
Open

fix(prism): avoid requiring full laravel framework#1025
MiddleSokilAI wants to merge 1 commit into
prism-php:mainfrom
MiddleSokilAI:sokil/remove-runtime-laravel-framework

Conversation

@MiddleSokilAI

Copy link
Copy Markdown

Problem

prism-php/prism currently requires the full laravel/framework package at runtime. Packages that only need Prism through laravel/ai then install the complete Laravel framework even when the host application already provides the required Illuminate components.

Change Summary

  • Replace the broad laravel/framework runtime requirement with the concrete Illuminate components used by src/.
  • Keep Laravel 11, 12, and 13 compatibility ranges aligned with the previous framework constraint.
  • Leave dev/test dependencies unchanged.

Validation

  • Parsed composer.json successfully with PowerShell ConvertFrom-Json.
  • Ran git diff --check successfully.
  • Reviewed src/ Illuminate namespace usage against the listed component packages.

Risk

Low-to-medium. This changes dependency metadata only, but consumers relying on Prism to implicitly install unrelated Laravel framework classes may need to require those packages directly.

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