M14, M15, M19 — declare and check what the bundle actually uses

[gplanchat]

First batch of the bundle's major findings. Three of them, one theme: the bundle accepts what it cannot honour, and nothing says so before the incident.

M15 — the configured cache pool was silently discarded

$cacheRef = null !== $cacheId && $container->hasDefinition($cacheId) ? new Reference($cacheId) : null;

hasDefinition() answers false for an alias — and Psr\Cache\CacheItemPoolInterface is one — as it does for a definition placed by an extension that runs after this one. The operator wrote activity_contracts.cache: my.pool, nothing was wired, and nothing reported it.

The reference is now placed unconditionally. A non-existent pool becomes a compile-time error, which the container knows how to name — this is the upstream convention: pass a Reference and let CheckExceptionOnInvalidReferenceBehaviorPass decide.

And its second half, which the report did not separate

The pool defaults to null. So the cache warmer warmed nothing, and above all: every activity call redid the reflection on its contract. These metadata derive from attributes, hence from code — they cannot change while the process lives.

A per-instance memo now serves them, and short-circuits the pool when there is one: on Redis, the previous code made one network round trip per activity call for immutable data. The test measures it against a spy pool — two resolutions, a single getItem.

M19 — a missing lock.factory did not say what to configure

Contrary to what the report implied, this was not silent: the container already failed. But on a "non-existent service" naming lock.factory and leaving you to search, when what you need to know is which configuration section would have placed it, and why that section is not optional here.

RequireLockFactoryPass says so, and names both ways out: framework.lock, or your own factory in durable.dbal.lock_factory. With the reason: without a lock, two workers replay the same journal at the same time.

In a compiler pass rather than in the extension, for the same reason as the pool above: when extensions load, the one that places lock.factory may not have run, and an existence test would answer false for a correctly configured application.

M14 — two bridges hard-wired, declared nowhere

The extension imports sixteen classes from durable-bridge-temporal and seven from durable-bridge-dbal. A bare composer require gplanchat/durable-bundle gave you a container that compiles and a "class not found" fatal on the first call. Both enter suggest.

One existing test changed, and why

DurableMiddlewareReachesTheBusTest built a container with the DBAL lock and without a lock factory. The new pass refuses it — rightly: an application in that state would not boot either. The test's container therefore places lock.factory, exactly as it already placed messenger.bus.default to stand in for what FrameworkExtension would have done.

It is the only existing test touched, and it is touched because it described an impossible container.

Verification

	main	this branch
unit suite	1073 tests, 4 errors	1081 tests, 4 errors
PHPStan	2 errors	2 errors

Pre-existing and identical errors (illuminate/cache missing on this machine). No new diagnostic.

What this batch does not do

• M20 — reworking the 819-line procedural extension into config/services.php + AbstractBundle. That is a change in its own right, not a fix.
• M17 (#[AsWorkflow] not autoconfigured, issue [bundle] #[AsWorkflow] is not autoconfigured: WorkflowPass finds nothing unless the class is tagged by hand #255) — the obvious fix breaks compilation, because a workflow is instantiated by reflection with a WorkflowEnvironment that is not a service. It deserves its own pass.
• The cache warmer itself. It stays, and stays debatable: it writes into a runtime pool with a one-hour TTL instead of the build directory, and the activity_contracts.contracts list re-enumerates by hand what the tags already carry. With the memo added here, its value shrinks further. Removing it is a public API break: to be decided, not slipped into a fix.