chore: removed cancellation from synchronous loops

Author: aryanjassal

src/client/handlers/IdentitiesAuthenticate.ts

@@ -27,7 +27,6 @@ class IdentitiesAuthenticate extends ServerHandler<
     _meta: Record<string, JSONValue>,
     ctx: ContextTimed,
   ): AsyncGenerator<ClientRPCResponseResult<AuthProcessMessage>> {
-    ctx.signal.throwIfAborted();
     const { identitiesManager }: { identitiesManager: IdentitiesManager } =
       this.container;
     const {

src/client/handlers/IdentitiesAuthenticatedGet.ts

@@ -25,7 +25,6 @@ class IdentitiesAuthenticatedGet extends ServerHandler<
     _meta: Record<string, JSONValue>,
     ctx: ContextTimed,
   ): AsyncGenerator<ClientRPCResponseResult<IdentityMessage>> {
-    ctx.signal.throwIfAborted();
     const { identitiesManager }: { identitiesManager: IdentitiesManager } =
       this.container;
     let providerId: ProviderId | undefined;

src/client/handlers/IdentitiesInfoConnectedGet.ts

@@ -28,7 +28,6 @@ class IdentitiesInfoConnectedGet extends ServerHandler<
     _meta: Record<string, JSONValue>,
     ctx: ContextTimed,
   ): AsyncGenerator<ClientRPCResponseResult<IdentityInfoMessage>> {
-    ctx.signal.throwIfAborted();
     const { identitiesManager }: { identitiesManager: IdentitiesManager } =
       this.container;
     const {

src/client/handlers/NodesGetAll.ts

@@ -22,7 +22,6 @@ class NodesGetAll extends ServerHandler<
     _meta: Record<string, JSONValue>,
     ctx: ContextTimed,
   ): AsyncGenerator<ClientRPCResponseResult<NodesGetMessage>> {
-    ctx.signal.throwIfAborted();
     const { nodeGraph } = this.container;
     for await (const [index, bucket] of nodeGraph.getBuckets()) {
       for (const [id, nodeContact] of bucket) {
@@ -32,7 +31,7 @@ class NodesGetAll extends ServerHandler<
         yield {
           bucketIndex: index,
           nodeIdEncoded: encodedId,
-          nodeContact,
+          nodeContact: nodeContact,
         };
       }
     }

src/client/handlers/NotificationsInboxRead.ts

@@ -26,7 +26,6 @@ class NotificationsInboxRead extends ServerHandler<
     _meta: Record<string, JSONValue>,
     ctx: ContextTimed,
   ): AsyncGenerator<ClientRPCResponseResult<NotificationInboxMessage>> {
-    ctx.signal.throwIfAborted();
     const {
       db,
       notificationsManager,

src/client/handlers/NotificationsOutboxRead.ts

@@ -26,7 +26,6 @@ class NotificationsOutboxRead extends ServerHandler<
     _meta: Record<string, JSONValue>,
     ctx: ContextTimed,
   ): AsyncGenerator<ClientRPCResponseResult<NotificationOutboxMessage>> {
-    ctx.signal.throwIfAborted();
     const {
       db,
       notificationsManager,

src/client/handlers/VaultsList.ts

@@ -26,7 +26,6 @@ class VaultsList extends ServerHandler<
   ): AsyncGenerator<ClientRPCResponseResult<VaultListMessage>> {
     const { db, vaultManager }: { db: DB; vaultManager: VaultManager } =
       this.container;
-    ctx.signal.throwIfAborted();
     const vaults = await db.withTransactionF((tran) =>
       vaultManager.listVaults(ctx, tran),
     );

src/client/handlers/VaultsLog.ts

@@ -26,7 +26,6 @@ class VaultsLog extends ServerHandler<
     _meta: Record<string, JSONValue> | undefined,
     ctx: ContextTimed,
   ): AsyncGenerator<ClientRPCResponseResult<LogEntryMessage>> {
-    ctx.signal.throwIfAborted();
     const { db, vaultManager }: { db: DB; vaultManager: VaultManager } =
       this.container;
     const log = await db.withTransactionF(async (tran) => {

src/client/handlers/VaultsPermissionGet.ts

@@ -32,7 +32,6 @@ class VaultsPermissionGet extends ServerHandler<
     _meta: Record<string, JSONValue> | undefined,
     ctx: ContextTimed,
   ): AsyncGenerator<ClientRPCResponseResult<VaultPermissionMessage>> {
-    ctx.signal.throwIfAborted();
     const {
       db,
       vaultManager,

src/client/handlers/VaultsScan.ts

@@ -26,7 +26,6 @@ class VaultsScan extends ServerHandler<
     _meta: Record<string, JSONValue> | undefined,
     ctx: ContextTimed,
   ): AsyncGenerator<ClientRPCResponseResult<VaultsScanMessage>> {
-    ctx.signal.throwIfAborted();
     const { vaultManager }: { vaultManager: VaultManager } = this.container;
     const { nodeId }: { nodeId: NodeId } = validateSync(
       (keyPath, value) => {