Skip to content

Reduce Psalm analysis cost for activities#400

Open
rmcdaniel wants to merge 8 commits into
v2from
master
Open

Reduce Psalm analysis cost for activities#400
rmcdaniel wants to merge 8 commits into
v2from
master

Conversation

@rmcdaniel

Copy link
Copy Markdown
Member

Addresses #399.

This refactor reduces Psalm's analysis cost for Laravel Durable Activities by:

  • Replacing the dependency on Illuminate\Routing\ResolvesRouteDependencies with a local resolver implementation.
  • Extracting webhook and heartbeat helpers into an optional ActivityWithWebhookSupport trait so the base Activity class stays lean.

Validation:

  • composer ecs
  • composer stan

phpunit activity coverage was attempted, but the local container is missing the SQLite PDO driver required by the testbench database setup.

rmcdaniel and others added 8 commits April 9, 2026 19:38
Rebrand the Composer package name from laravel-workflow/laravel-workflow
to durable-workflow/workflow. The `replace` clause keeps the old name
resolvable so existing consumers continue to work during the transition.
PHP namespace (Workflow\) is unchanged.

Co-authored-by: Durable Workflow <support@durable-workflow.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Added a new sponsor link for 'Translate a Book' to the README.
Updated shield links in README to reflect package name change from 'laravel-workflow' to 'durable-workflow'.
Signal webhook route closures captured the ReflectionMethod instance,
so php artisan route:cache failed with "Serialization of
'ReflectionMethod' is not allowed" for any app with a #[Webhook]
signal method. Capture the method name instead.
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