refactor: move initial default browser check to be included in redux …

…pipeline

src/main/main.ts

@@ -2,22 +2,13 @@ import type { AnyAction } from '@reduxjs/toolkit'
 import electron, { app } from 'electron'
 
 import { Channel } from '../shared/state/channels'
-import { database } from './database'
 import { readiedApp } from './state/actions'
 import { dispatch } from './state/store'
 import { urlOpener } from './state/thunk.url-opener'
 
 // Attempt to fix this bug: https://github.com/electron/electron/issues/20944
 app.commandLine.appendArgument('--enable-features=Metal')
 
-if (database.get('firstRun')) {
-  // Prompt to set as default browser
-  app.setAsDefaultProtocolClient('http')
-  app.setAsDefaultProtocolClient('https')
-}
-
-database.set('firstRun', false)
-
 app.on('ready', () => dispatch(readiedApp()))
 
 // App doesn't always close on ctrl-c in console, this fixes that

src/main/state/middleware.action-hub.ts

@@ -78,6 +78,13 @@ export const actionHubMiddleware =
   (action) => {
     const previousState = getState()
 
+    // Initial request to prompt to become default browser
+    // Check must happen before reducer run, before firstRun set to false
+    if (readiedApp.match(action) && previousState.storage.firstRun !== false) {
+      app.setAsDefaultProtocolClient('http')
+      app.setAsDefaultProtocolClient('https')
+    }
+
     // Move to next middleware
     const result = next(action)
 

src/shared/state/reducer.storage.ts

@@ -4,6 +4,7 @@ import type { AppId } from '../../config/apps'
 import {
   changedPickerWindowBounds,
   installedAppsRetrieved,
+  readiedApp,
   syncReducers,
 } from '../../main/state/actions'
 import {
@@ -31,6 +32,10 @@ export const defaultStorage: Storage = {
 
 export const storage = createReducer<Storage>(defaultStorage, (builder) =>
   builder
+    .addCase(readiedApp, (state) => {
+      state.firstRun = false
+    })
+
     .addCase(syncReducers, (_, action) => action.payload.storage)
 
     .addCase(installedAppsRetrieved, (state, action) => {