fix: Persist app version over multiple 'track' calls (atuinsh#44)

Author: BinaryMuse

src/state/store/ui_state.ts

@@ -34,6 +34,7 @@ export class TabUri {
 }
 
 export interface AtuinUiState {
+  appVersion: Option<string>;
   focused: boolean;
   connectedToHubSocket: boolean;
   searchOpen: boolean;
@@ -64,6 +65,7 @@ export interface AtuinUiState {
   lightModeEditorTheme: string;
   darkModeEditorTheme: string;
 
+  setAppVersion: (version: string) => void;
   setFocused: (focused: boolean) => void;
   setConnectedToHubSocket: (online: boolean) => void;
   setSearchOpen: (open: boolean) => void;
@@ -120,6 +122,7 @@ export const persistUiKeys: (keyof AtuinUiState)[] = [
 ];
 
 export const createUiState: StateCreator<AtuinUiState> = (set, get, _store): AtuinUiState => ({
+  appVersion: None,
   focused: false,
   connectedToHubSocket: false,
   searchOpen: false,
@@ -148,6 +151,7 @@ export const createUiState: StateCreator<AtuinUiState> = (set, get, _store): Atu
   lightModeEditorTheme: "githubLight",
   darkModeEditorTheme: "githubDark",
 
+  setAppVersion: (version: string) => set(() => ({ appVersion: Some(version) })),
   setFocused: (focused: boolean) => set(() => ({ focused })),
   setConnectedToHubSocket: (online: boolean) => set(() => ({ connectedToHubSocket: online })),
   setSearchOpen: (open) => set(() => ({ searchOpen: open })),

src/tracking.ts

@@ -41,13 +41,7 @@ export const init_tracking = async () => {
   }
 
   if (track) {
-    let appVersion: string | null = null;
-
-    try {
-      appVersion = await invoke<string>("get_app_version");
-    } catch (e) {
-      console.error("Failed to get app version:", e);
-    }
+    const appVersion = await getAppVersion();
 
     Sentry.init({
       dsn: "https://ac8c00adf29c329694a0b105e1981ca3@o4507730431442944.ingest.us.sentry.io/4507741947232256",
@@ -102,28 +96,22 @@ export default async function track_event(event: string, properties: any = {}) {
   // Get current org context from store
   const store = useStore.getState();
   const selectedOrg = store.selectedOrg;
-  
+
   // Determine org context (anonymous - just personal vs org)
   const orgContext = selectedOrg ? "org" : "personal";
 
   if (AtuinEnv.isDev) {
     console.log(
-      `[dev] track_event: ${event} -> ${JSON.stringify({ 
-        ...properties, 
+      `[dev] track_event: ${event} -> ${JSON.stringify({
+        ...properties,
         platform: platformInfo,
-        org_context: orgContext
+        org_context: orgContext,
       })}`,
     );
     return;
   }
 
-  let appVersion: string | null = null;
-
-  try {
-    appVersion = await invoke<string>("get_app_version");
-  } catch (e) {
-    console.error("Failed to get app version:", e);
-  }
+  const appVersion = await getAppVersion();
 
   // Always include platform info and org context in every event
   const eventProperties = {
@@ -136,3 +124,19 @@ export default async function track_event(event: string, properties: any = {}) {
   const { default: posthog } = await import("posthog-js");
   posthog.capture(event, eventProperties);
 }
+
+async function getAppVersion() {
+  const version = useStore.getState().appVersion;
+
+  if (version.isSome()) {
+    return version.unwrap();
+  }
+
+  try {
+    const backendVersion = await invoke<string>("get_app_version");
+    if (backendVersion) {
+      useStore.getState().setAppVersion(backendVersion);
+      return backendVersion;
+    }
+  } catch (_e) {}
+}