Skip to content

Upgrade to Laravel 13.7#3

Draft
thekevinm wants to merge 1 commit intomasterfrom
shift/laravel-13
Draft

Upgrade to Laravel 13.7#3
thekevinm wants to merge 1 commit intomasterfrom
shift/laravel-13

Conversation

@thekevinm
Copy link
Copy Markdown

Summary

Wave 4 of the L11→L13 upgrade campaign. Composer-only + 2 minimal source patches.

Files changed:

  • composer.json — full L13 dep matrix (php ^8.3, laravel/helpers ^1.8 prod, framework ^13.7 + phpunit ^11.5.3 + testbench ^11.0 + mockery ^1.6 + collision ^8.6 dev). Bumps spatie/laravel-http-logger ^1.11.2 (first L13-supporting tag), promphp ^2.14, df-core ~1.0.4. Drops unused df-system require.
  • src/ServiceProvider.php — gate Route::pushMiddlewareToGroup('df.api', ...) with Route::hasMiddlewareGroup() to handle non-deterministic boot order.
  • src/Utility/HttpLogger/PredisAdapter.php — add no-op updateSummary() to satisfy promphp 2.x Adapter interface (added in 2.3+).

Validation

  • Stage 1: composer install --no-dev clean under L13.7.0; class_exists smoke green.
  • Stage 2: host shift-173254 worktree with standard strip-list — full sibling graph resolves.

Test plan

  • CI green
  • Live /metrics endpoint smoke once L13 release tags land

Notes

  • predis/predis kept at ^1.1 to align with host app's predis/predis: ~1.0 constraint. Predis v2/v3 bump deferred.
  • Pre-existing latent bugs in PredisAdapter::wipeStorage() (calls non-existent ensureOpenConnection(), references \Redis::OPT_PREFIX which is php-redis ext, not predis) are out of scope.

- composer.json: php ^8.3, laravel/helpers ^1.8 prod; require-dev L13.7 stack
  (laravel/framework ^13.7, phpunit ^11.5.3, testbench ^11.0, mockery ^1.6,
  collision ^8.6); spatie/laravel-http-logger ^1.11.2; promphp 2.x; tighten
  df-core ~1.0.4; drop unused df-system require
- ServiceProvider: gate Route::pushMiddlewareToGroup('df.api', ...) with
  Route::hasMiddlewareGroup() to handle non-deterministic boot order
- PredisAdapter: add no-op updateSummary() to satisfy
  promphp/prometheus_client_php 2.x Adapter interface
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