Gas revert (#55)

* adding revert check in receipt of fulfill, and test

* fix error msg

* lint

src/common/errors/eco-error.ts

@@ -2,7 +2,7 @@ import { Network } from 'alchemy-sdk'
 import { Logger } from '@nestjs/common'
 import * as _ from 'lodash'
 import { EcoLogMessage } from '../logging/eco-log-message'
-import { Chain } from 'viem'
+import { Chain, TransactionReceipt } from 'viem'
 
 export class EcoError extends Error {
   // Alchemy Service
@@ -42,6 +42,9 @@ export class EcoError extends Error {
   static FeasableIntentNoTransactionError = new Error('No transaction data found')
   static FulfillIntentNoTransactionError = new Error('No transaction data found')
   static FulfillIntentBatchError = new Error('Could not fulfill batch transaction')
+  static FulfillIntentRevertError(receipt: TransactionReceipt) {
+    return new Error(JSON.stringify(receipt))
+  }
 
   // ValidateIntent Service
   static ValidateIntentDescructureFailed(err?: Error) {

src/intent/fulfill-intent.service.ts

@@ -80,8 +80,11 @@ export class FulfillIntentService {
       const receipt = await kernelAccountClient.waitForTransactionReceipt({ hash: transactionHash })
 
       // set the status and receipt for the model
-      model.status = 'SOLVED'
       model.receipt = receipt as any
+      if (receipt.status === 'reverted') {
+        throw EcoError.FulfillIntentRevertError(receipt)
+      }
+      model.status = 'SOLVED'
 
       this.logger.debug(
         EcoLogMessage.fromDefault({

src/intent/tests/fulfill-intent.service.spec.ts

@@ -115,6 +115,36 @@ describe('FulfillIntentService', () => {
         await expect(() => fulfillIntentService.executeFulfillIntent(hash)).rejects.toThrow(error)
       })
 
+      it('should fail on receipt status reverted', async () => {
+        const receipt = { status: 'reverted' }
+        const error = EcoError.FulfillIntentRevertError(receipt as any)
+        utilsIntentService.getIntentProcessData = jest.fn().mockResolvedValue({ model, solver })
+        const mockGetFulfillIntentData = jest.fn()
+        fulfillIntentService['getFulfillIntentData'] = mockGetFulfillIntentData
+        fulfillIntentService['getTransactionsForTargets'] = jest.fn().mockReturnValue([])
+        jest.spyOn(ecoConfigService, 'getEth').mockReturnValue({ claimant } as any)
+        jest.spyOn(accountClientService, 'getClient').mockImplementation(async (id) => {
+          return {
+            execute: () => {
+              return '0x33'
+            },
+            waitForTransactionReceipt: () => {
+              return receipt
+            },
+          } as any
+        })
+        await expect(() => fulfillIntentService.executeFulfillIntent(hash)).rejects.toThrow(error)
+        expect(mockLogError).toHaveBeenCalledTimes(1)
+        expect((model as any).status).toBe('FAILED')
+        expect(mockLogError).toHaveBeenCalledWith({
+          msg: `fulfillIntent: Invalid transaction`,
+          error: EcoError.FulfillIntentBatchError.toString(),
+          model,
+          errorPassed: error,
+          flatExecuteData: emptyTxs,
+        })
+      })
+
       it('should log error', async () => {
         const error = new Error('stuff went bad')
         utilsIntentService.getIntentProcessData = jest.fn().mockResolvedValue({ model, solver })