[markshust]

Summary

Worker and FailedJobHandler now log through marko/log's LoggerInterface instead of calling error_log(). No error_log() calls are left in packages/queue/src.

Level	Line
info	[marko/queue] Worker stopping: ... (once per run, including the timeout path inside the alarm handler)
warning	A job sets a timeout but ext-pcntl is not loaded
error	Pop failure (worker keeps polling), job exceeded its timeout, timed-out job could not be released or failed, failed() hook threw

Tests pass a FakeLogger and assert on its entries, including the levels. The ini_set('error_log', ...) capture helpers in WorkerTest, WorkerLimitsTest, FailedJobHandlerTest and UniqueJobTest are gone, so the worker suite prints nothing extra.

Decisions

• Hard-require marko/log, no queue-owned logging contract. marko/log is small (core, clock, config), owns LoggerInterface and binds it itself, and FakeLogger already exists for tests. A second queue-only logging contract would duplicate it.
• Messages keep the [marko/queue] prefix. The log channel is app-wide (log.channel), so the prefix is what tells you which package a line came from. The message text is unchanged.
• The logger is a required constructor argument on both classes. Worker takes it after $clock and before the defaulted $processControl. FailedJobHandler takes it last. A default would mean a silent fallback.
• Integration fixtures (INTEGRATION_MODULES, QUEUE_SESSION_MODULES) now link log + log-file, and each fixture has a config/log.php that writes logs into its temporary project instead of the cwd.

Breaking changes

• Apps using marko/queue need a marko/log handler package. Resolving Worker (and SyncQueue/DatabaseQueue, which use FailedJobHandler) resolves LoggerInterface. With marko/log installed but no handler, that throws NoDriverException, which names the packages to install. Upgrade: composer require marko/log-file, or marko/log-stream with LOG_HANDLERS=stream to keep worker lines on STDERR as before.
• Code that builds these classes by hand: new Worker($queue, $failedJobHandler, $config, $jobRunner, $retryPolicy, $clock, $logger) and new FailedJobHandler($failedJobs, $envelope, $jobRunner, $eventDispatcher, $clock, $logger). Code that relies on the container needs no change.

Both are covered in a new "Upgrading: worker logs go through LoggerInterface" note on the queue docs page.

Docs

queue.md has a new Worker Logs section: where logs go (log.handlers: storage/logs with log-file, STDERR/STDOUT with log-stream), the level of each line, and how to test with FakeLogger. The install section, the pop-failure, pcntl-warning, stop-reason and failed() notes, the FailedJobHandler API reference and the testing example were updated to match. The testing example was also missing the $clock argument, which is now added.

Tests

• composer ci passes locally (13680 passed, phpcs, php-cs-fixer, phpstan, docs:lint).
• The integration-services group passes locally against Postgres and Redis for tests/Integration, queue-database and queue-redis, including queue:work in both fixture apps.

Fixes #608

🤖 Generated with Claude Code

https://claude.ai/code/session_017KaHtaJjfe2D38CdEdnmTb