Fix concurrency races on shared Http/Route singletons#251

Closed

loks0n wants to merge 21 commits into

mainfrom

fix/concurrency-shared-state

Commits on Apr 27, 2026

Fix: Eliminate shared-state races on the Http/Route/Router singletons

loks0n
and
claude
committed
Chore: Mark dead Route::matchedPath methods and unused Http::$requestContainer as @deprecated

loks0n
and
claude
committed

Commits on Apr 28, 2026

Remove dead Route::matchedPath methods and unused Http::$requestContainer

loks0n
and
claude
committed
Use plain 'route'/'matchedPath' keys in the request container

loks0n
and
claude
committed
Make Http::setRoute private

loks0n
and
claude
committed
Rename request-scoped helpers to use 'context' terminology

loks0n
and
claude
committed
Remove Http::getRoute / setRoute / get-set MatchedPath

loks0n
and
claude
committed
Format: single-quote 'route' in HttpTest

loks0n
and
claude
committed
Split Adapter::getContainer() into getContainer + getContext

loks0n
and
claude
committed
Inject route's resolved arguments into shutdown/error hooks

loks0n
and
claude
committed
Bundle route + matchedPath + arguments into immutable RouteMatch

loks0n
and
claude
committed
Rename RouteMatch::\$matchedPath to RouteMatch::\$path

loks0n
and
claude
committed
Drop RouteMatch::withArguments(), use the constructor directly

loks0n
and
claude
committed
Seed context 'match' to null at the top of runInternal

loks0n
and
claude
committed
Seed RouteMatch::\$arguments with path values at match-time

loks0n
and
claude
committed
Make RouteMatch::\$arguments mutable, drop the replace-on-update dance

loks0n
and
claude
committed
Revert: path-value seeding into RouteMatch::\$arguments at match-time

loks0n
and
claude
committed
Collapse Adapter API to a single getContext()

loks0n
and
claude
committed
Fix RouteMatch docblock: \$arguments is single-write, not two-pass

loks0n
and
claude
committed
Trim verbose comments and docblocks

loks0n
and
claude
committed
Populate \$match->arguments incrementally so error hooks see partials

loks0n
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix concurrency races on shared Http/Route singletons#251

Fix concurrency races on shared Http/Route singletons#251
loks0n wants to merge 21 commits into
mainfrom
fix/concurrency-shared-state

Commits on Apr 27, 2026

Fix: Eliminate shared-state races on the Http/Route/Router singletons

Chore: Mark dead Route::matchedPath methods and unused Http::$requestContainer as @deprecated

Commits on Apr 28, 2026

Remove dead Route::matchedPath methods and unused Http::$requestContainer

Use plain 'route'/'matchedPath' keys in the request container

Make Http::setRoute private

Rename request-scoped helpers to use 'context' terminology

Remove Http::getRoute / setRoute / get-set MatchedPath

Format: single-quote 'route' in HttpTest

Split Adapter::getContainer() into getContainer + getContext

Inject route's resolved arguments into shutdown/error hooks

Bundle route + matchedPath + arguments into immutable RouteMatch

Rename RouteMatch::\$matchedPath to RouteMatch::\$path

Drop RouteMatch::withArguments(), use the constructor directly

Seed context 'match' to null at the top of runInternal

Seed RouteMatch::\$arguments with path values at match-time

Make RouteMatch::\$arguments mutable, drop the replace-on-update dance

Revert: path-value seeding into RouteMatch::\$arguments at match-time

Collapse Adapter API to a single getContext()

Fix RouteMatch docblock: \$arguments is single-write, not two-pass

Trim verbose comments and docblocks

Populate \$match->arguments incrementally so error hooks see partials