fix: browser history

4 files changed, 32 insertions, 21 deletions

diff --git a/src/App.web.tsx b/src/App.web.tsx
index 4f9fd3e53..1c6efba8f 100644
--- a/src/App.web.tsx
+++ b/src/App.web.tsx
@@ -1,5 +1,6 @@
 import React, {useState, useEffect} from 'react'
 import {SafeAreaProvider} from 'react-native-safe-area-context'
+import {getInitialURL} from 'platform/urls'
 import * as view from './view/index'
 import {RootStoreModel, setupState, RootStoreProvider} from './state'
 import {WebShell} from './view/shell/web'
@@ -13,7 +14,14 @@ function App() {
   // init
   useEffect(() => {
     view.setup()
-    setupState().then(setRootStore)
+    setupState().then(store => {
+      setRootStore(store)
+      getInitialURL().then(url => {
+        if (url) {
+          store.nav.handleLink(url)
+        }
+      })
+    })
   }, [])
 
   // show nothing prior to init
diff --git a/src/platform/urls.tsx b/src/platform/urls.tsx
index 0f24dd678..f544262d7 100644
--- a/src/platform/urls.tsx
+++ b/src/platform/urls.tsx
@@ -1,31 +1,20 @@
 import {Linking} from 'react-native'
-import {isIOS, isAndroid, isNative, isWeb} from './detection'
+import {createBrowserHistory, createMemoryHistory} from 'history'
+import {isNative, isWeb} from './detection'
 
-export function makeAppUrl(path = '') {
-  if (isIOS) {
-    return `bskyapp://${path}`
-  } else if (isAndroid) {
-    return `bsky://app${path}`
-  } else {
-    // @ts-ignore window exists -prf
-    return `${window.location.origin}${path}`
-  }
-}
-
-export function extractHashFragment(url: string): string {
-  return url.split('#')[1] || ''
-}
-
-export async function getInitialURL(): Promise<string> {
+export async function getInitialURL(): Promise<string | undefined> {
   if (isNative) {
     const url = await Linking.getInitialURL()
     if (url) {
       return url
     }
-    return makeAppUrl()
+    return undefined
   } else {
     // @ts-ignore window exists -prf
-    return window.location.toString()
+    if (window.location.pathname !== '/') {
+      return window.location.pathname
+    }
+    return undefined
   }
 }
 
@@ -35,3 +24,11 @@ export function clearHash() {
     window.location.hash = ''
   }
 }
+
+export function getHistory() {
+  if (isWeb) {
+    return createBrowserHistory()
+  } else {
+    return createMemoryHistory()
+  }
+}
diff --git a/src/state/models/navigation.ts b/src/state/models/navigation.ts
index feb03b367..9cbc678d0 100644
--- a/src/state/models/navigation.ts
+++ b/src/state/models/navigation.ts
@@ -2,6 +2,7 @@ import {RootStoreModel} from './root-store'
 import {makeAutoObservable} from 'mobx'
 import {TABS_ENABLED} from 'lib/build-flags'
 import {segmentClient} from 'lib/analytics'
+import {getHistory} from 'platform/urls'
 
 let __id = 0
 function genId() {
@@ -41,6 +42,7 @@ export type HistoryPtr = string // `{tabId}-{historyId}`
 export class NavigationTabModel {
   id = genId()
   history: HistoryItem[]
+  browserHistory: any
   index = 0
   isNewTab = false
 
@@ -48,6 +50,7 @@ export class NavigationTabModel {
     this.history = [
       {url: TabPurposeMainPath[fixedTabPurpose], ts: Date.now(), id: genId()},
     ]
+    this.browserHistory = getHistory()
     makeAutoObservable(this, {
       serialize: false,
       hydrate: false,
@@ -122,6 +125,7 @@ export class NavigationTabModel {
         this.history.push({url: fixedUrl, ts: Date.now(), id: genId()})
       }
       this.history.push({url, title, ts: Date.now(), id: genId()})
+      this.browserHistory.push(url)
       this.index = this.history.length - 1
     }
   }
@@ -142,6 +146,7 @@ export class NavigationTabModel {
   goBack() {
     if (this.canGoBack) {
       this.index--
+      this.browserHistory.back()
     }
   }
 
@@ -155,12 +160,14 @@ export class NavigationTabModel {
   goForward() {
     if (this.canGoForward) {
       this.index++
+      this.browserHistory.forward()
     }
   }
 
   goToIndex(index: number) {
     if (index >= 0 && index <= this.history.length - 1) {
       this.index = index
+      this.browserHistory.go(index)
     }
   }
 
diff --git a/src/state/models/root-store.ts b/src/state/models/root-store.ts
index 11d496351..229c7b3c0 100644
--- a/src/state/models/root-store.ts
+++ b/src/state/models/root-store.ts
@@ -120,7 +120,6 @@ export class RootStoreModel {
   async handleSessionChange(agent: AtpAgent) {
     this.log.debug('RootStoreModel:handleSessionChange')
     this.agent = agent
-    this.nav.clear()
     this.me.clear()
     await this.me.load()
   }