fix(router): add null checks and fix route parameter handling (#1778)

Author: xHeaven

packages/router/src/RouteBindingInitializer.php

@@ -31,6 +31,10 @@ public function initialize(ClassReflector $class, null|string|UnitEnum $tag, Con
             }
         }
 
+        if ($parameter === null) {
+            throw new RouteBindingFailed();
+        }
+
         $object = $class->callStatic('resolve', $matchedRoute->params[$parameter->getName()]);
 
         if ($object === null) {

packages/router/src/Routing/Construction/RoutingTree.php

@@ -51,7 +51,7 @@ private function isOptionalSegment(string $segment): bool
 
     private function stripOptionalMarker(string $segment): string
     {
-        return str_replace('?', '', $segment);
+        return preg_replace('/^\{\?/', '{', $segment);
     }
 
     /** @return array<string, MatchingRegex> */

packages/router/src/UriGenerator.php

@@ -178,6 +178,10 @@ public function isCurrentUri(array|string|MethodReflector $action, mixed ...$par
     {
         $action = $this->normalizeActionToUri($action);
 
+        if (! $this->container->has(MatchedRoute::class)) {
+            return false;
+        }
+
         $matchedRoute = $this->container->get(MatchedRoute::class);
         $candidateUri = $this->createUri($action, ...[...$matchedRoute->params, ...$params]);
         $currentUri = $this->createUri([$matchedRoute->route->handler->getDeclaringClass()->getName(), $matchedRoute->route->handler->getName()]);