AA-488 replace by fee reputation (#235)

eth-infinitism · Jan 1, 2025 · d49018d · d49018d
1 parent 3579626
commit d49018d
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/packages/bundler/src/modules/MempoolManager.ts b/packages/bundler/src/modules/MempoolManager.ts
@@ -80,8 +80,9 @@ export class MempoolManager {
     )
     const packedNonce = getPackedNonce(entry.userOp)
     const index = this._findBySenderNonce(userOp.sender, packedNonce)
+    let oldEntry: MempoolEntry | undefined
     if (index !== -1) {
-      const oldEntry = this.mempool[index]
+      oldEntry = this.mempool[index]
       this.checkReplaceUserOp(oldEntry, entry)
       debug('replace userOp', userOp.sender, packedNonce)
       this.mempool[index] = entry
@@ -100,19 +101,22 @@ export class MempoolManager {
       }
       this.mempool.push(entry)
     }
+    if (oldEntry != null) {
+      this.updateSeenStatus(oldEntry.aggregator, oldEntry.userOp, validationResult.senderInfo, -1)
+    }
     this.updateSeenStatus(validationResult.aggregatorInfo?.addr, userOp, validationResult.senderInfo)
   }
 
-  private updateSeenStatus (aggregator: string | undefined, userOp: OperationBase, senderInfo: StakeInfo): void {
+  private updateSeenStatus (aggregator: string | undefined, userOp: OperationBase, senderInfo: StakeInfo, val = 1): void {
     try {
       this.reputationManager.checkStake('account', senderInfo)
       this.reputationManager.updateSeenStatus(userOp.sender)
     } catch (e: any) {
       if (!(e instanceof RpcError)) throw e
     }
-    this.reputationManager.updateSeenStatus(aggregator)
-    this.reputationManager.updateSeenStatus(userOp.paymaster)
-    this.reputationManager.updateSeenStatus(userOp.factory)
+    this.reputationManager.updateSeenStatus(aggregator, val)
+    this.reputationManager.updateSeenStatus(userOp.paymaster, val)
+    this.reputationManager.updateSeenStatus(userOp.factory, val)
   }
 
   private checkReputation (