@tinkoff/router 0.1.75 → 0.1.78

package/lib/index.browser.js

@@ -221,17 +221,20 @@ class AbstractRouter {
         await this.runHooks('afterUpdateCurrent', navigation);
     }
     async navigate(navigateOptions) {
-        return this.internalNavigate(typeof navigateOptions === 'string' ? { url: navigateOptions } : navigateOptions, {});
+        return this.internalNavigate(makeNavigateOptions(navigateOptions), {});
     }
-    async internalNavigate(navigateOptions, { history }) {
+    async internalNavigate(navigateOptions, { history, redirect }) {
         var _a;
         const { url, replace, params, navigateState, code } = navigateOptions;
-        const prevNavigation = (_a = this.currentNavigation) !== null && _a !== void 0 ? _a : this.lastNavigation;
+        const prevNavigation = redirect
+            ? this.lastNavigation
+            : (_a = this.currentNavigation) !== null && _a !== void 0 ? _a : this.lastNavigation;
         if (!url && !prevNavigation) {
             throw new Error('Navigate url should be specified and cannot be omitted for first navigation');
         }
         const resolvedUrl = this.resolveUrl(navigateOptions);
         const { to: from, url: fromUrl } = prevNavigation !== null && prevNavigation !== void 0 ? prevNavigation : {};
+        const redirectFrom = redirect ? this.currentNavigation.to : undefined;
         let navigation = {
             type: 'navigate',
             from,
@@ -241,6 +244,8 @@ class AbstractRouter {
             history,
             navigateState,
             code,
+            redirect,
+            redirectFrom,
         };
         await this.runHooks('beforeResolve', navigation);
         const to = this.resolveRoute({ url: resolvedUrl, params, navigateState }, { wildcard: true });
@@ -366,7 +371,9 @@ class AbstractRouter {
                 pathname = makePath(currentRoute.path, { ...currentRoute.params, ...params });
             }
         }
-        pathname = this.normalizePathname(pathname);
+        if (isSameHost(resultUrl)) {
+            pathname = this.normalizePathname(pathname);
+        }
         return convertRawUrl(rawAssignUrl(resultUrl, {
             pathname,
             search: url ? resultUrl.search : '',
@@ -556,7 +563,7 @@ const wrapHistory = ({ onNavigate }) => {
     const history = (delta) => {
         go(delta);
     };
-    const browserNavigate = (replace) => {
+    const browserNavigate = (replace = false) => {
         return (navigateState, title, url) => {
             onNavigate({ url, replace, navigateState });
         };
@@ -770,8 +777,11 @@ class ClientRouter extends AbstractRouter {
         // in case we didn't find any matched route just force hard page navigation
         const prevUrl = (_b = (_a = navigation.fromUrl) === null || _a === void 0 ? void 0 : _a.href) !== null && _b !== void 0 ? _b : window.location.href;
         const nextUrl = navigation.url.href;
-        // prevent redirect cycle on the same page
-        if (prevUrl !== nextUrl) {
+        const isNoSpaNavigation = navigation.from && !navigation.to;
+        // prevent redirect cycle on the same page,
+        // except cases when we run no-spa navigations,
+        // because we need hard reload in this cases
+        if (isNoSpaNavigation ? true : prevUrl !== nextUrl) {
             if (navigation.replace) {
                 window.location.replace(nextUrl);
             }
@@ -787,7 +797,12 @@ class ClientRouter extends AbstractRouter {
     }
     async redirect(navigation, target) {
         await super.redirect(navigation, target);
-        return this.navigate({ ...target, replace: target.replace || navigation.replace });
+        return this.internalNavigate({
+            ...target,
+            replace: target.replace || navigation.replace,
+        }, {
+            redirect: true,
+        });
     }
 }
 

package/lib/index.es.js

@@ -221,17 +221,20 @@ class AbstractRouter {
         await this.runHooks('afterUpdateCurrent', navigation);
     }
     async navigate(navigateOptions) {
-        return this.internalNavigate(typeof navigateOptions === 'string' ? { url: navigateOptions } : navigateOptions, {});
+        return this.internalNavigate(makeNavigateOptions(navigateOptions), {});
     }
-    async internalNavigate(navigateOptions, { history }) {
+    async internalNavigate(navigateOptions, { history, redirect }) {
         var _a;
         const { url, replace, params, navigateState, code } = navigateOptions;
-        const prevNavigation = (_a = this.currentNavigation) !== null && _a !== void 0 ? _a : this.lastNavigation;
+        const prevNavigation = redirect
+            ? this.lastNavigation
+            : (_a = this.currentNavigation) !== null && _a !== void 0 ? _a : this.lastNavigation;
         if (!url && !prevNavigation) {
             throw new Error('Navigate url should be specified and cannot be omitted for first navigation');
         }
         const resolvedUrl = this.resolveUrl(navigateOptions);
         const { to: from, url: fromUrl } = prevNavigation !== null && prevNavigation !== void 0 ? prevNavigation : {};
+        const redirectFrom = redirect ? this.currentNavigation.to : undefined;
         let navigation = {
             type: 'navigate',
             from,
@@ -241,6 +244,8 @@ class AbstractRouter {
             history,
             navigateState,
             code,
+            redirect,
+            redirectFrom,
         };
         await this.runHooks('beforeResolve', navigation);
         const to = this.resolveRoute({ url: resolvedUrl, params, navigateState }, { wildcard: true });
@@ -366,7 +371,9 @@ class AbstractRouter {
                 pathname = makePath(currentRoute.path, { ...currentRoute.params, ...params });
             }
         }
-        pathname = this.normalizePathname(pathname);
+        if (isSameHost(resultUrl)) {
+            pathname = this.normalizePathname(pathname);
+        }
         return convertRawUrl(rawAssignUrl(resultUrl, {
             pathname,
             search: url ? resultUrl.search : '',
@@ -786,7 +793,7 @@ const wrapHistory = ({ onNavigate }) => {
     const history = (delta) => {
         go(delta);
     };
-    const browserNavigate = (replace) => {
+    const browserNavigate = (replace = false) => {
         return (navigateState, title, url) => {
             onNavigate({ url, replace, navigateState });
         };
@@ -1000,8 +1007,11 @@ class ClientRouter extends AbstractRouter {
         // in case we didn't find any matched route just force hard page navigation
         const prevUrl = (_b = (_a = navigation.fromUrl) === null || _a === void 0 ? void 0 : _a.href) !== null && _b !== void 0 ? _b : window.location.href;
         const nextUrl = navigation.url.href;
-        // prevent redirect cycle on the same page
-        if (prevUrl !== nextUrl) {
+        const isNoSpaNavigation = navigation.from && !navigation.to;
+        // prevent redirect cycle on the same page,
+        // except cases when we run no-spa navigations,
+        // because we need hard reload in this cases
+        if (isNoSpaNavigation ? true : prevUrl !== nextUrl) {
             if (navigation.replace) {
                 window.location.replace(nextUrl);
             }
@@ -1017,7 +1027,12 @@ class ClientRouter extends AbstractRouter {
     }
     async redirect(navigation, target) {
         await super.redirect(navigation, target);
-        return this.navigate({ ...target, replace: target.replace || navigation.replace });
+        return this.internalNavigate({
+            ...target,
+            replace: target.replace || navigation.replace,
+        }, {
+            redirect: true,
+        });
     }
 }
 

package/lib/index.js

@@ -236,17 +236,20 @@ class AbstractRouter {
         await this.runHooks('afterUpdateCurrent', navigation);
     }
     async navigate(navigateOptions) {
-        return this.internalNavigate(typeof navigateOptions === 'string' ? { url: navigateOptions } : navigateOptions, {});
+        return this.internalNavigate(makeNavigateOptions(navigateOptions), {});
     }
-    async internalNavigate(navigateOptions, { history }) {
+    async internalNavigate(navigateOptions, { history, redirect }) {
         var _a;
         const { url, replace, params, navigateState, code } = navigateOptions;
-        const prevNavigation = (_a = this.currentNavigation) !== null && _a !== void 0 ? _a : this.lastNavigation;
+        const prevNavigation = redirect
+            ? this.lastNavigation
+            : (_a = this.currentNavigation) !== null && _a !== void 0 ? _a : this.lastNavigation;
         if (!url && !prevNavigation) {
             throw new Error('Navigate url should be specified and cannot be omitted for first navigation');
         }
         const resolvedUrl = this.resolveUrl(navigateOptions);
         const { to: from, url: fromUrl } = prevNavigation !== null && prevNavigation !== void 0 ? prevNavigation : {};
+        const redirectFrom = redirect ? this.currentNavigation.to : undefined;
         let navigation = {
             type: 'navigate',
             from,
@@ -256,6 +259,8 @@ class AbstractRouter {
             history,
             navigateState,
             code,
+            redirect,
+            redirectFrom,
         };
         await this.runHooks('beforeResolve', navigation);
         const to = this.resolveRoute({ url: resolvedUrl, params, navigateState }, { wildcard: true });
@@ -381,7 +386,9 @@ class AbstractRouter {
                 pathname = makePath(currentRoute.path, { ...currentRoute.params, ...params });
             }
         }
-        pathname = this.normalizePathname(pathname);
+        if (isSameHost(resultUrl)) {
+            pathname = this.normalizePathname(pathname);
+        }
         return url.convertRawUrl(url.rawAssignUrl(resultUrl, {
             pathname,
             search: url$1 ? resultUrl.search : '',
@@ -801,7 +808,7 @@ const wrapHistory = ({ onNavigate }) => {
     const history = (delta) => {
         go(delta);
     };
-    const browserNavigate = (replace) => {
+    const browserNavigate = (replace = false) => {
         return (navigateState, title, url) => {
             onNavigate({ url, replace, navigateState });
         };
@@ -1015,8 +1022,11 @@ class ClientRouter extends AbstractRouter {
         // in case we didn't find any matched route just force hard page navigation
         const prevUrl = (_b = (_a = navigation.fromUrl) === null || _a === void 0 ? void 0 : _a.href) !== null && _b !== void 0 ? _b : window.location.href;
         const nextUrl = navigation.url.href;
-        // prevent redirect cycle on the same page
-        if (prevUrl !== nextUrl) {
+        const isNoSpaNavigation = navigation.from && !navigation.to;
+        // prevent redirect cycle on the same page,
+        // except cases when we run no-spa navigations,
+        // because we need hard reload in this cases
+        if (isNoSpaNavigation ? true : prevUrl !== nextUrl) {
             if (navigation.replace) {
                 window.location.replace(nextUrl);
             }
@@ -1032,7 +1042,12 @@ class ClientRouter extends AbstractRouter {
     }
     async redirect(navigation, target) {
         await super.redirect(navigation, target);
-        return this.navigate({ ...target, replace: target.replace || navigation.replace });
+        return this.internalNavigate({
+            ...target,
+            replace: target.replace || navigation.replace,
+        }, {
+            redirect: true,
+        });
     }
 }
 

package/lib/router/abstract.d.ts

@@ -20,6 +20,7 @@ export interface Options {
 }
 interface InternalOptions {
     history?: boolean;
+    redirect?: boolean;
 }
 export declare abstract class AbstractRouter {
     protected started: boolean;
@@ -45,7 +46,7 @@ export declare abstract class AbstractRouter {
     protected internalUpdateCurrentRoute(updateRouteOptions: UpdateCurrentRouteOptions, { history }: InternalOptions): Promise<void>;
     protected runUpdateCurrentRoute(navigation: Navigation): Promise<void>;
     navigate(navigateOptions: NavigateOptions | string): Promise<void>;
-    protected internalNavigate(navigateOptions: NavigateOptions, { history }: InternalOptions): Promise<void>;
+    protected internalNavigate(navigateOptions: NavigateOptions, { history, redirect }: InternalOptions): Promise<void>;
     protected runNavigate(navigation: Navigation): Promise<void>;
     protected run(navigation: Navigation): Promise<void>;
     resolve(resolveOptions: NavigateOptions | string, options?: Parameters<AbstractRouter['resolveRoute']>[1]): import("../types").NavigationRoute;

package/lib/types.d.ts

@@ -40,6 +40,8 @@ export interface Navigation {
     history?: boolean;
     cancelled?: boolean;
     code?: number;
+    redirect?: boolean;
+    redirectFrom?: NavigationRoute;
 }
 export declare type NavigationGuard = (navigation: Navigation) => Promise<void | NavigateOptions | string | boolean>;
 export declare type NavigationHook = (navigation: Navigation) => Promise<void>;

package/lib/utils.d.ts

@@ -3,6 +3,6 @@ import type { NavigateOptions, NavigationHook, NavigationSyncHook, NavigationGua
 export declare const isFilePath: (pathname: string) => boolean;
 export declare const normalizeTrailingSlash: (pathname: string, trailingSlash?: boolean) => string;
 export declare const normalizeManySlashes: (hrefOrPath: string) => string;
-export declare const isSameHost: import("@tinkoff/utils/typings/types").Func<true> | ((url: Url) => boolean);
+export declare const isSameHost: import("@tinkoff/utils/typings/types").Func<true> | ((url: Url | URL) => boolean);
 export declare const makeNavigateOptions: (options: string | NavigateOptions) => NavigateOptions;
 export declare const registerHook: <T extends NavigationHook | NavigationSyncHook | NavigationGuard>(hooksSet: Set<T>, hook: T) => () => void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tinkoff/router",
-  "version": "0.1.75",
+  "version": "0.1.78",
   "description": "router",
   "main": "lib/index.js",
   "typings": "lib/index.d.ts",