ui5-lib-guard-router 1.6.1 → 1.6.2

package/README.md

@@ -2,7 +2,7 @@
 
 Drop-in replacement for `sap.m.routing.Router` that intercepts navigation **before** route matching, target loading, or view creation, preventing flashes of unauthorized content and polluted browser history.
 
-> Born from [SAP/openui5#3411](https://github.com/SAP/openui5/issues/3411), an open request for native navigation guard support in UI5.
+> Born from [UI5/openui5#3411](https://github.com/UI5/openui5/issues/3411), an open request for native navigation guard support in UI5.
 >
 > **Related resources**:
 >
@@ -177,7 +177,7 @@ export default class Component extends UIComponent {
 
 ## How it works
 
-The library extends [`sap.m.routing.Router`](https://sdk.openui5.org/api/sap.m.routing.Router) and intercepts navigation through two entry points:
+The library extends [`sap.m.routing.Router`](https://sdk.openui5.org/#/api/sap.m.routing.Router) and intercepts navigation through two entry points:
 
 - **`navTo()` preflight**: For programmatic navigation (`router.navTo()`), guards run _before_ any hash change occurs. If a guard blocks or redirects, the hash never changes, so no history entry is created.
 - **`parse()` fallback**: For browser-initiated navigation (back/forward buttons, URL bar entry, direct hash changes), guards run inside the `parse()` override after the browser has already changed the hash. If a guard blocks or redirects, the router restores the previous hash via `replaceHash()`.
@@ -450,10 +450,10 @@ The full object form with `enter` and `leave` keys registers both enter and leav
 
 **Module paths** use dot notation and are resolved relative to `sap.app.id`. Given `sap.app.id = "com.example.app"`, the path `"guards.authGuard"` resolves to `"com/example/app/guards/authGuard"`.
 
-To use an absolute module path, prefix it with `"module:"`:
+To use an absolute module path, prefix it with `"module:"`. Dot notation is used for consistency with relative paths; slash notation (`module:com/shared/guards/authGuard`) is also accepted:
 
 ```json
-"*": ["module:com/shared/guards/authGuard"]
+"*": ["module:com.shared.guards.authGuard"]
 ```
 
 ### Complete example
@@ -1026,10 +1026,10 @@ Or set the global log level via URL parameter (per-component filtering is only a
 
 ### Running tests
 
-The library ships three QUnit test suites that run in headless Chrome via WebdriverIO:
+The library ships nine QUnit test suites that run in headless Chrome via WebdriverIO:
 
 ```bash
-# All three suites (Router, NativeRouterCompat, UpstreamParity)
+# All suites (Router, RouterGuards, RouterAsync, RouterNavigation, RouterSettlement, RouterOptions, GuardPipeline, NativeRouterCompat, UpstreamParity)
 npm run test:qunit
 
 # Full matrix including OpenUI5 1.120 compatibility lane and E2E

package/dist/.ui5/build-manifest.json

@@ -16,7 +16,7 @@
 	},
 	"buildManifest": {
 		"manifestVersion": "0.2",
-		"timestamp": "2026-04-02T23:45:49.937Z",
+		"timestamp": "2026-05-15T15:28:23.027Z",
 		"versions": {
 			"builderVersion": "4.1.4",
 			"projectVersion": "4.0.13",
@@ -31,7 +31,7 @@
 			"includedTasks": [],
 			"excludedTasks": []
 		},
-		"version": "1.6.1",
+		"version": "1.6.2",
 		"namespace": "ui5/guard/router",
 		"tags": {
 			"/resources/ui5/guard/router/GuardPipeline-dbg.js": {

package/dist/index.d.ts

@@ -1,7 +1,7 @@
 // Generated with TypeScript 6.0.2 / OpenUI5 1.144.0 using:
 //   - typescript@6.0.2
-/// <reference path="./resources/ui5/guard/router/Router.d.ts"/>
+/// <reference path="./resources/ui5/guard/router/GuardPipeline.d.ts"/>
 /// <reference path="./resources/ui5/guard/router/library.d.ts"/>
+/// <reference path="./resources/ui5/guard/router/Router.d.ts"/>
 /// <reference path="./resources/ui5/guard/router/types.d.ts"/>
-/// <reference path="./resources/ui5/guard/router/NavigationOutcome.d.ts"/>
-/// <reference path="./resources/ui5/guard/router/GuardPipeline.d.ts"/>
+/// <reference path="./resources/ui5/guard/router/NavigationOutcome.d.ts"/>

package/dist/resources/ui5/guard/router/.library

@@ -2,7 +2,7 @@
 <library xmlns="http://www.sap.com/sap.ui.library.xsd">
 	<name>ui5.guard.router</name>
 	<vendor>Marco</vendor>
-	<version>1.6.1</version>
+	<version>1.6.2</version>
 	<copyright></copyright>
 	<title>UI5 Router extension with async navigation guards</title>
 	<documentation>Extends sap.m.routing.Router with async navigation guards, running before route matching begins.</documentation>

package/dist/resources/ui5/guard/router/Router-dbg.js

@@ -96,12 +96,18 @@ sap.ui.define(["sap/m/routing/Router", "sap/base/Log", "sap/ui/core/library", ".
   /** Parsed guard declaration from the manifest `guards` block. */
 
   /**
-   * Resolve a dot-notation module path following UI5 routing conventions.
-   * Mirrors `sap.ui.core.routing.Target._getEffectiveObjectName()`.
+   * Resolve a guard module path to a slash-separated AMD module path.
    *
-   * - Paths prefixed with `"module:"` are treated as absolute (the prefix is
-   *   stripped and dots become slashes).
-   * - All other paths are prefixed with the component namespace.
+   * Inspired by `sap.ui.core.routing.Target._getEffectiveObjectName()`,
+   * which skips namespace prepending for `"module:"`-prefixed paths. This
+   * function goes further: it also converts dots to slashes in a single
+   * step, so both dot notation (`module:some.lib.guard`) and slash
+   * notation (`module:some/lib/guard`) are accepted.
+   *
+   * - Paths prefixed with `"module:"` are treated as absolute (the prefix
+   *   is stripped, dots become slashes).
+   * - All other paths are prefixed with the component namespace, then dots
+   *   become slashes.
    */
   function resolveGuardModulePath(dotPath, componentNamespace) {
     if (dotPath.startsWith("module:")) {
@@ -1452,8 +1458,7 @@ sap.ui.define(["sap/m/routing/Router", "sap/base/Log", "sap/ui/core/library", ".
       // Sort by pattern depth (segment count) so ancestor guards run first.
       // Global ("*") descriptors return 0 here -- their relative position is irrelevant
       // because GuardPipeline evaluates globals separately from route-specific guards.
-      // oxlint-disable-next-line unicorn/no-array-sort -- expanded is a local array, mutation is intentional
-      return expanded.sort((a, b) => {
+      return expanded.toSorted((a, b) => {
         if (a.route === "*" || b.route === "*") return 0;
         const pa = patternByName.get(a.route) ?? "";
         const pb = patternByName.get(b.route) ?? "";
@@ -1468,10 +1473,8 @@ sap.ui.define(["sap/m/routing/Router", "sap/base/Log", "sap/ui/core/library", ".
       const manifest = this._manifestMeta.get(routeName);
       const runtime = this._runtimeMeta.get(routeName);
       if (!manifest && !runtime) return Router._EMPTY_META;
-      if (!runtime) return manifest ?? Router._EMPTY_META;
-      if (!manifest) return Object.freeze({
-        ...runtime
-      });
+      if (!runtime) return manifest;
+      if (!manifest) return runtime;
       return Object.freeze({
         ...manifest,
         ...runtime