Router::toUri: update hard-coded uri heuristic, no longer ignores reflection exceptions

Author: wryk

packages/router/src/GenericRouter.php

@@ -131,27 +131,20 @@ private function getCallable(MatchedRoute $matchedRoute): HttpMiddlewareCallable
 
     public function toUri(array|string $action, ...$params): string
     {
-        try {
-            if (is_array($action)) {
-                $controllerClass = $action[0];
-                $reflection = new ClassReflector($controllerClass);
-                $controllerMethod = $reflection->getMethod($action[1]);
-            } else {
-                $controllerClass = $action;
-                $reflection = new ClassReflector($controllerClass);
-                $controllerMethod = $reflection->getMethod('__invoke');
-            }
+        if (is_string($action) && str_starts_with($action, '/')) {
+            $uri = $action;
+        } else {
+            [$controllerClass, $controllerMethod] = is_array($action) ? $action : [$action, '__invoke'];
 
-            /** @var Route|null $routeAttribute */
-            $routeAttribute = $controllerMethod->getAttribute(Route::class);
+            $routeAttribute = new ClassReflector($controllerClass)
+                ->getMethod($controllerMethod)
+                ->getAttribute(Route::class);
 
-            $uri = $routeAttribute->uri;
-        } catch (ReflectionException) {
-            if (is_array($action)) {
-                throw new InvalidRouteException($action[0], $action[1]);
+            if ($routeAttribute === null) {
+                throw new InvalidRouteException($controllerClass, $controllerMethod);
             }
 
-            $uri = $action;
+            $uri = $routeAttribute->uri;
         }
 
         $uri = str($uri);