use assert instead of new error

local-first-web · Dec 28, 2024 · c2a7c1b · c2a7c1b
1 parent 608cb85
commit c2a7c1b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/packages/auth-provider-automerge-repo/src/AuthProvider.ts b/packages/auth-provider-automerge-repo/src/AuthProvider.ts
@@ -211,9 +211,7 @@ export class AuthProvider extends EventEmitter<AuthProviderEvents> {
    * Creates a team and registers it with all of our sync servers.
    */
   public async createTeam(teamName: string) {
-    if (!this.#user) {
-      throw new Error('Cannot create team as user is missing on AuthProvider')
-    }
+    assert(this.#user, 'Cannot create team as user is missing on AuthProvider')
 
     const team = Auth.createTeam(teamName, {
       device: this.#device,
@@ -665,9 +663,7 @@ export class AuthProvider extends EventEmitter<AuthProviderEvents> {
 
     const savedShares = unpack(serializedState) as SerializedState
 
-    if (!this.#user) {
-      throw new Error('Cannot load state as user is missing on AuthProvider')
-    }
+    assert(this.#user, 'Cannot load state as user is missing on AuthProvider')
 
     await Promise.all(
       Object.values(savedShares).map(async share => {

diff --git a/packages/auth/src/connection/getDeviceUserFromGraph.ts b/packages/auth/src/connection/getDeviceUserFromGraph.ts
@@ -43,9 +43,7 @@ export const getDeviceUserFromGraph = ({
   const userKeyring = select.keyring(state, { type: USER, name: userId }, starterKeys)
   const keys = getLatestGeneration(userKeyring)
 
-  if (!keys) {
-    throw new Error('Failed to get device user from graph as there are no keys for user keyring')
-  }
+  assert(keys, 'Failed to get device user from graph as there are no keys for user keyring')
 
   const user: UserWithSecrets = { userName, userId, keys }