fix(drywrite): throw on bad drywrite and continue if successful

src/common.ts

@@ -15,7 +15,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 import { ArconnectSigner, ArweaveSigner, DataItem } from 'arbundles';
-import { InteractionResult, Transaction } from 'warp-contracts';
+import { Transaction } from 'warp-contracts';
 
 import {
   ANTRecord,
@@ -98,7 +98,7 @@ export interface WriteContract {
     inputs,
     evaluationOptions,
   }: EvaluationParameters<WriteParameters<Input>>): Promise<
-    Transaction | DataItem | InteractionResult<unknown, Input>
+    Transaction | DataItem
   >;
 }
 

src/common/contracts/warp-contract.ts

@@ -18,7 +18,6 @@ import { DataItem, Signer } from 'arbundles';
 import Arweave from 'arweave';
 import {
   Contract,
-  InteractionResult,
   LoggerFactory,
   Transaction,
   Warp,
@@ -169,7 +168,7 @@ export class WarpContract<T>
     inputs,
     dryWrite = true,
   }: EvaluationParameters<WriteParameters<Input>>): Promise<
-    Transaction | DataItem | InteractionResult<unknown, Input>
+    Transaction | DataItem
   > {
     try {
       this.log.debug(`Write interaction: ${functionName}`, {
@@ -179,11 +178,15 @@ export class WarpContract<T>
       await this.ensureContractInit();
 
       if (dryWrite) {
-        // responsibility of the caller to handle the dry write result for control flows
-        return (await this.contract.dryWrite<Input>({
+        const { errorMessage, type } = await this.contract.dryWrite<Input>({
           function: functionName,
           ...inputs,
-        })) as InteractionResult<unknown, Input>;
+        });
+        if (type !== 'ok') {
+          throw new Error(
+            `Failed to dry write contract interaction ${functionName}: ${errorMessage}`,
+          );
+        }
       }
 
       const writeResult = await this.contract.writeInteraction<Input>({