Refactored the merging code

Author: sdangol

packages/event-handler/src/rest/RouteHandlerRegistry.ts

@@ -205,25 +205,22 @@ class RouteHandlerRegistry {
     routeHandlerRegistry: RouteHandlerRegistry,
     options?: { prefix: Path }
   ): void {
-    for (const route of routeHandlerRegistry.#staticRoutes.values()) {
-      this.register(
-        new Route(
-          route.method as HttpMethod,
-          options?.prefix
-            ? route.path === '/'
-              ? options.prefix
-              : `${options.prefix}${route.path}`
-            : route.path,
-          route.handler,
-          route.middleware
-        )
-      );
-    }
-    for (const route of routeHandlerRegistry.#dynamicRoutes) {
+    const routes = [
+      ...routeHandlerRegistry.#staticRoutes.values(),
+      ...routeHandlerRegistry.#dynamicRoutes,
+    ];
+    for (const route of routes) {
+      let path = route.path;
+      if (options?.prefix) {
+        path =
+          route.path === '/'
+            ? options.prefix
+            : `${options.prefix}${route.path}`;
+      }
       this.register(
         new Route(
           route.method as HttpMethod,
-          options?.prefix ? `${options.prefix}${route.path}` : route.path,
+          path,
           route.handler,
           route.middleware
         )

packages/event-handler/src/rest/Router.ts

@@ -562,11 +562,11 @@ class Router {
    * const todosRouter = new Router();
    *
    * todosRouter.get('/todos', async () => {
-   *   // List Todos
+   *   // List API
    * });
    *
    * todosRouter.get('/todos/{todoId}', async () => {
-   *   // Get Todo
+   *   // Get API
    * });
    *
    * const app = new Router();