diff --git a/src/contract/HandlerBasedContract.ts b/src/contract/HandlerBasedContract.ts index b525cbb9..36df7fb7 100644 --- a/src/contract/HandlerBasedContract.ts +++ b/src/contract/HandlerBasedContract.ts @@ -380,7 +380,7 @@ export class HandlerBasedContract implements Contract { const handlerResult = await this.callContract(input, 'write', undefined, undefined, tags, transfer, strict, vrf); if (strict && handlerResult.type !== 'ok') { - throw Error(`Cannot create interaction: ${handlerResult.errorMessage}`); + throw Error('Cannot create interaction: ' + JSON.stringify(handlerResult.error || handlerResult.errorMessage)); } const callStack: ContractCallRecord = this.getCallStack(); const innerWrites = this._innerWritesEvaluator.eval(callStack); diff --git a/src/core/modules/impl/HandlerExecutorFactory.ts b/src/core/modules/impl/HandlerExecutorFactory.ts index d45328a4..fb874894 100644 --- a/src/core/modules/impl/HandlerExecutorFactory.ts +++ b/src/core/modules/impl/HandlerExecutorFactory.ts @@ -23,9 +23,9 @@ import { Buffer } from 'warp-isomorphic'; // eslint-disable-next-line const BigNumber = require('bignumber.js'); -export class ContractError extends Error { - constructor(message, readonly subtype?: string) { - super(message); +export class ContractError extends Error { + constructor(readonly error: T, readonly subtype?: string) { + super(error.toString()); this.name = 'ContractError'; } } @@ -277,6 +277,7 @@ export type HandlerResult = { export type InteractionResult = HandlerResult & { type: InteractionResultType; errorMessage?: string; + error?: unknown; originalValidity?: Record; originalErrorMessages?: Record; }; diff --git a/src/core/modules/impl/handler/AbstractContractHandler.ts b/src/core/modules/impl/handler/AbstractContractHandler.ts index 92d29522..d32c11ff 100644 --- a/src/core/modules/impl/handler/AbstractContractHandler.ts +++ b/src/core/modules/impl/handler/AbstractContractHandler.ts @@ -88,7 +88,7 @@ export abstract class AbstractContractHandler implements HandlerApi extends AbstractContractHandler { @@ -44,32 +44,21 @@ export class WasmHandlerApi extends AbstractContractHandler { }; } catch (e) { await this.swGlobal.kv.rollback(); - // note: as exceptions handling in WASM is currently somewhat non-existent - // https://www.assemblyscript.org/status.html#exceptions - // and since we have to somehow differentiate different types of exceptions - // - each exception message has to have a proper prefix added. - - // exceptions with prefix [RE:] ("Runtime Exceptions") should break the execution immediately - // - eg: [RE:OOG] - [RuntimeException: OutOfGas] - - // exception with prefix [CE:] ("Contract Exceptions") should be logged, but should not break - // the state evaluation - as they are considered as contracts' business exception (eg. validation errors) - // - eg: [CE:ITT] - [ContractException: InvalidTokenTransfer] const result = { errorMessage: e.message, state: currentResult.state, result: null }; - if (e.message.startsWith('[RE:')) { - this.logger.fatal(e); + if (e instanceof ContractError) { return { ...result, - type: 'exception' + error: e.error, + type: 'error' }; } else { return { ...result, - type: 'error' + type: 'exception' }; } } finally { @@ -95,32 +84,15 @@ export class WasmHandlerApi extends AbstractContractHandler { const handleResult = action.interactionType === 'write' ? await this.wasmExports.warpContractWrite(action.input) : await this.wasmExports.warpContractView(action.input); - if (Object.prototype.hasOwnProperty.call(handleResult, 'WriteResponse')) { - return handleResult.WriteResponse; + if (!handleResult) { + return; } - if (Object.prototype.hasOwnProperty.call(handleResult, 'ViewResponse')) { - return handleResult.ViewResponse; - } - { - this.logger.error('Error from rust', handleResult); - let errorKey; - let errorArgs = ''; - if (typeof handleResult.Err === 'string' || handleResult.Err instanceof String) { - errorKey = handleResult.Err; - } else if ('kind' in handleResult.Err) { - errorKey = handleResult.Err.kind; - errorArgs = 'data' in handleResult.Err ? ' ' + handleResult.Err.data : ''; - } else { - errorKey = Object.keys(handleResult.Err)[0]; - errorArgs = ' ' + handleResult.Err[errorKey]; - } - - if (errorKey == 'RuntimeError') { - throw new Error(`[RE:RE]${errorArgs}`); - } else { - throw new Error(`[CE:${errorKey}${errorArgs}]`); - } + if (handleResult.type === 'ok') { + return handleResult.result; } + this.logger.error('Error from rust', handleResult); + if (handleResult.type === 'error') throw new ContractError(handleResult.error); + throw new Error(handleResult.errorMessage); } default: { throw new Error(`Support for ${this.contractDefinition.srcWasmLang} not implemented yet.`);