fix: issues in SSR mode

Author: satazor

jest.config.js

@@ -14,11 +14,6 @@ module.exports = compose(
             './jest.setup.js',
         ];
 
-        config.moduleNameMapper = {
-            'NextScrollBehavior\\.node$': '<rootDir>/src/scroll-behavior/NextScrollBehavior.browser.js',
-            ...config.moduleNameMapper,
-        };
-
         return config;
     },
 );

package.json

@@ -8,10 +8,6 @@
     "lib",
     "es"
   ],
-  "browser": {
-    "lib/scroll-behavior/NextScrollBehavior.node.js": "./lib/scroll-behavior/NextScrollBehavior.browser.js",
-    "es/scroll-behavior/NextScrollBehavior.node.js": "./es/scroll-behavior/NextScrollBehavior.browser.js"
-  },
   "homepage": "https://github.com/moxystudio/next-router-scroll#readme",
   "author": "Afonso Reis <[email protected]>",
   "license": "MIT",

src/RouterScrollProvider.js

@@ -29,9 +29,9 @@ const useScrollBehavior = (shouldUpdateScroll) => {
 
     shouldUpdateScrollRef.current = shouldUpdateScroll;
 
-    if (!scrollBehaviorRef.current && typeof window !== 'undefined') {
+    if (!scrollBehaviorRef.current) {
         scrollBehaviorRef.current = new NextScrollBehavior(
-            (...args) => shouldUpdateScrollRef.current?.(...args),
+            (...args) => shouldUpdateScrollRef.current(...args),
         );
     }
 
@@ -50,9 +50,9 @@ const ScrollBehaviorProvider = ({ disableNextLinkScroll, shouldUpdateScroll, chi
 
     // Create facade to use as the provider value.
     const providerValue = useMemo(() => ({
-        updateScroll: scrollBehavior ? scrollBehavior.updateScroll.bind(scrollBehavior) : () => {},
-        registerElement: scrollBehavior ? scrollBehavior.registerElement.bind(scrollBehavior) : () => {},
-        unregisterElement: scrollBehavior ? scrollBehavior.unregisterElement.bind(scrollBehavior) : () => {},
+        updateScroll: scrollBehavior.updateScroll.bind(scrollBehavior),
+        registerElement: scrollBehavior.registerElement.bind(scrollBehavior),
+        unregisterElement: scrollBehavior.unregisterElement.bind(scrollBehavior),
     }), [scrollBehavior]);
 
     return (

src/scroll-behavior/NextScrollBehavior.node.js

@@ -1 +1,5 @@
-export default class NextScrollBehaviorMock {}
+export default class NextScrollBehaviorMock {
+    updateScroll() {}
+    registerElement() {}
+    unregisterElement() {}
+}

src/scroll-behavior/index.js

@@ -1 +1,5 @@
-export { default } from './NextScrollBehavior.node';
+/* eslint-disable import/no-commonjs */
+
+module.exports = typeof window === 'undefined' ?
+    require('./NextScrollBehavior.node') :
+    require('./NextScrollBehavior.browser');