Draft
Conversation
Bump dependency matrix for Laravel 13.7: - Tighten php requirement to ^8.3 - Add dreamfactory/df-core ~1.0.4 (was implicit, now explicit) - Add laravel/helpers ^1.8 for array_get/camel_case/array_except polyfills - Move test deps to require-dev: laravel/framework ^13.7, phpunit/phpunit ^11.5.3, orchestra/testbench ^11.0, mockery/mockery ^1.6, nunomaduro/collision ^8.6 No source changes required — package owns no Connection/Builder/Grammar extensions, no DispatchesJobs trait, no CorsService imports, no getDates overrides, no setupBeforeClass typos. ServiceProvider, controllers, models, and the daemon client all use stable Laravel public APIs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bumps
df-mcp-serverto Laravel 13.7 / PHPUnit 11. Composer-only change — no source patches were required.composer.json
php:>=8.3->^8.3dreamfactory/df-core: ~1.0.4(previously implicit via host app, now explicit since the package extendsBaseRestService,BaseServiceConfigModel,Controllerfrom df-core)laravel/helpers: ^1.8to polyfillarray_get/camel_case/array_except(defensive — none currently used in src/, but aligned with the rest of the upgrade campaign so consumers are insulated against future siblings)require-devand pin to L13 stack:laravel/framework: ^13.7phpunit/phpunit: ^11.5.3orchestra/testbench: ^11.0mockery/mockery: ^1.6nunomaduro/collision: ^8.6Why no source changes
Surveyed for all 45 wave invariants. Clean across the board:
DispatchesJobstrait, nodispatchNowcallsCorsServiceimports (Fruitcake still works in L13.7 anyway)Illuminate\Database\Connection|Builder|Grammarextensions (the OAuth + stream layer is plain Eloquent + Guzzle)getDates()overrides onHasAttributesuserssetupBeforeClass/setUpcasing bugs (no tests in this package)withTablePrefix()/compileTableExists/getDoctrineDrivercallersConnectionInterfacetest stubsphpunit.xmlschema bump needed (no test config exists)ServiceProvider already uses
prependMiddlewarewith apushMiddlewarefallback — both are still in L13'sIlluminate\Contracts\Http\Kernel.Validation
Stage 1 (isolated install) on
dreamfactory_web_1:composer validate --strictcleancomposer installresolves cleanly with path-repos for df-core (1.0.99), df-system (0.6.99) onshift/laravel-13Stage 2 (host-app integration) against
dreamfactory/dreamfactory@shift-173254:shift/laravel-13)package:discoversucceeds (df-mcp-server's ServiceProvider auto-registers)Out of scope
array_get/camel_case/array_excepttoArr::get/Str::camel/Arr::except— origin/main HEAD already cleaned the one prior site; polyfill stays in for defense in depth.daemon/dist/utils/utils.js) is a separate Node.js artifact and is unaffected by Laravel.Test plan
shift/laravel-13dreamfactory/dreamfactory@shift-173254) lands and exposes a realcomposer installsmokedf-mcp-serverfrom any remaining wave strip-lists once this PR is merged