Check that request is current when iterating through matched routes. Ignore duplicate URL navigation.

packages/application/src/router.ts

@@ -219,9 +219,14 @@ export class Router implements IRouter {
     const { base } = this;
     const { history } = window;
     const { hard, silent } = options;
+    const old = document.location.href;
     const url =
       path && path.indexOf(base) === 0 ? path : URLExt.join(base, path);
 
+    if (url === old) {
+      return hard ? this.reload() : undefined;
+    }
+
     if (silent) {
       history.replaceState({}, '', url);
     } else {
@@ -292,29 +297,28 @@ export class Router implements IRouter {
 
     // Process each enqueued command sequentially and short-circuit if a promise
     // resolves with the `stop` token.
-    (function next() {
+    const next = async () => {
       if (!queue.length) {
         routed.emit(current);
-        done.resolve(void 0);
+        done.resolve(undefined);
         return;
       }
 
       const { command } = queue.pop();
 
-      commands
-        .execute(command, current)
-        .then(result => {
-          if (result === stop) {
-            queue.length = 0;
-            console.log(`Routing ${request} was short-circuited by ${command}`);
-          }
-          next();
-        })
-        .catch(reason => {
-          console.warn(`Routing ${request} to ${command} failed`, reason);
-          next();
-        });
-    })();
+      try {
+        const request = this.current.request;
+        const result = await commands.execute(command, current);
+        if (result === stop) {
+          queue.length = 0;
+          console.log(`Routing ${request} was short-circuited by ${command}`);
+        }
+      } catch (reason) {
+        console.warn(`Routing ${request} to ${command} failed`, reason);
+      }
+      void next();
+    };
+    void next();
 
     return done.promise;
   }