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

Route and matched path are now pure context values — read them via
\$http->getResource('route') / 'matchedPath', or inject them into a
hook/action. Internal call sites that wrote them go through the
context container directly (or via setContext()).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: loks0n

src/Http/Http.php

@@ -457,38 +457,6 @@ public static function getRoutes(): array
         return Router::getRoutes();
     }
 
-    /**
-     * Get the current route
-     */
-    public function getRoute(): ?Route
-    {
-        $container = $this->server->getContainer();
-
-        return $container->has('route') ? $container->get('route') : null;
-    }
-
-    private function setRoute(?Route $route): void
-    {
-        $this->server->getContainer()->set('route', fn() => $route);
-    }
-
-    /**
-     * Read the prepared-path string the current request matched under.
-     * Falls back to '' so callers (e.g. tests invoking execute() directly)
-     * keep getting the legacy "first registered path-params" behavior.
-     */
-    private function getMatchedPath(): string
-    {
-        $container = $this->server->getContainer();
-
-        return $container->has('matchedPath') ? $container->get('matchedPath') : '';
-    }
-
-    private function setMatchedPath(string $path): void
-    {
-        $this->server->getContainer()->set('matchedPath', fn() => $path);
-    }
-
     /**
      * Add Route
      *
@@ -601,8 +569,10 @@ public function start(): void
      */
     public function match(Request $request, bool $fresh = true): ?Route
     {
-        if (!$fresh) {
-            $cached = $this->getRoute();
+        $context = $this->server->getContainer();
+
+        if (!$fresh && $context->has('route')) {
+            $cached = $context->get('route');
             if (null !== $cached) {
                 return $cached;
             }
@@ -615,14 +585,14 @@ public function match(Request $request, bool $fresh = true): ?Route
 
         $matched = Router::match($method, $url);
         if (null === $matched) {
-            $this->setRoute(null);
-            $this->setMatchedPath('');
+            $context->set('route', fn() => null);
+            $context->set('matchedPath', fn() => '');
             return null;
         }
 
         [$route, $matchedPath] = $matched;
-        $this->setRoute($route);
-        $this->setMatchedPath($matchedPath);
+        $context->set('route', fn() => $route);
+        $context->set('matchedPath', fn() => $matchedPath);
 
         return $route;
     }
@@ -635,7 +605,9 @@ public function execute(Route $route, Request $request, Response $response): sta
         $arguments = [];
         $groups = $route->getGroups();
 
-        $preparedPath = Router::preparePath($this->getMatchedPath());
+        $context = $this->server->getContainer();
+        $matchedPath = $context->has('matchedPath') ? $context->get('matchedPath') : '';
+        $preparedPath = Router::preparePath($matchedPath);
         $pathValues = $route->getPathValues($request, $preparedPath[0]);
 
         try {
@@ -766,10 +738,12 @@ public function run(Request $request, Response $response): static
         $result = $this->runInternal($request, $response);
 
         $requestDuration = microtime(true) - $start;
+        $context = $this->server->getContainer();
+        $route = $context->has('route') ? $context->get('route') : null;
         $attributes = [
             'url.scheme' => $request->getProtocol(),
             'http.request.method' => $request->getMethod(),
-            'http.route' => $this->getRoute()?->getPath(),
+            'http.route' => $route?->getPath(),
             'http.response.status_code' => $response->getStatusCode(),
         ];
         $this->requestDuration->record($requestDuration, $attributes);
@@ -883,8 +857,7 @@ private function runInternal(Request $request, Response $response): static
             $path = \is_string($path) ? ($path === '' ? '/' : $path) : '/';
             $route->path($path);
 
-            $this->setRoute($route);
-            $this->setContext('route', fn() => $route, []);
+            $this->setContext('route', fn() => $route);
         }
         if (null !== $route) {
             return $this->execute($route, $request, $response);

tests/HttpTest.php

@@ -413,7 +413,7 @@ public function testCanMatchRoute(string $method, string $path, ?string $url = n
         $_SERVER['REQUEST_URI'] = $url;
 
         $this->assertSame($expected, $this->http->match(new Request()));
-        $this->assertSame($expected, $this->http->getRoute());
+        $this->assertSame($expected, $this->http->getResource("route"));
     }
 
     public function testNoMismatchRoute(): void
@@ -442,7 +442,7 @@ public function testNoMismatchRoute(): void
             $route = $this->http->match(new Request(), fresh: true);
 
             $this->assertNull($route);
-            $this->assertNull($this->http->getRoute());
+            $this->assertNull($this->http->getResource("route"));
         }
     }
 
@@ -457,20 +457,20 @@ public function testCanMatchFreshRoute(): void
             $_SERVER['REQUEST_URI'] = '/path1';
             $matched = $this->http->match(new Request());
             $this->assertSame($route1, $matched);
-            $this->assertSame($route1, $this->http->getRoute());
+            $this->assertSame($route1, $this->http->getResource("route"));
 
             // Second request match returns cached route
             $_SERVER['REQUEST_METHOD'] = 'HEAD';
             $_SERVER['REQUEST_URI'] = '/path2';
             $request2 = new Request();
             $matched = $this->http->match($request2, fresh: false);
             $this->assertSame($route1, $matched);
-            $this->assertSame($route1, $this->http->getRoute());
+            $this->assertSame($route1, $this->http->getResource("route"));
 
             // Fresh match returns new route
             $matched = $this->http->match($request2, fresh: true);
             $this->assertSame($route2, $matched);
-            $this->assertSame($route2, $this->http->getRoute());
+            $this->assertSame($route2, $this->http->getResource("route"));
         } catch (\Exception $e) {
             $this->fail($e->getMessage());
         }
@@ -484,7 +484,7 @@ public function testCanMatchRootRouteWhenUriHasNoPath(): void
         $_SERVER['REQUEST_URI'] = 'https://example.com?x=1';
 
         $this->assertSame($route, $this->http->match(new Request()));
-        $this->assertSame($route, $this->http->getRoute());
+        $this->assertSame($route, $this->http->getResource("route"));
     }
 
     public function testCanRunRequest(): void
@@ -523,8 +523,8 @@ public function testWildcardRoute(): void
         $_SERVER['REQUEST_URI'] = '/unknown_path';
 
         Http::init()
-            ->action(function () {
-                $route = $this->http->getRoute();
+            ->inject('route')
+            ->action(function ($route) {
                 $this->container->set('myRoute', fn() => $route);
             });