fix: do not retry incorrect clarity values (#143)

* fix: do not retry incorrect clarity values

* fix: tests

Author: rafaelcr

src/token-processor/stacks-node/stacks-node-rpc-client.ts

@@ -128,7 +128,7 @@ export class StacksNodeRpcClient {
     if (unwrappedClarityValue.type_id === ClarityTypeID.UInt) {
       return unwrappedClarityValue;
     }
-    throw new RetryableJobError(
+    throw new Error(
       `Unexpected Clarity type '${unwrappedClarityValue.type_id}' while unwrapping uint`
     );
   }
@@ -143,7 +143,7 @@ export class StacksNodeRpcClient {
     } else if (unwrappedClarityValue.type_id === ClarityTypeID.OptionalNone) {
       return undefined;
     }
-    throw new RetryableJobError(
+    throw new Error(
       `Unexpected Clarity type '${unwrappedClarityValue.type_id}' while unwrapping string`
     );
   }

tests/stacks-node-rpc-client.test.ts

@@ -113,7 +113,7 @@ describe('StacksNodeRpcClient', () => {
     }
   });
 
-  test('clarity value parse errors get retried', async () => {
+  test('clarity value parse errors are not retried', async () => {
     const mockResponse = {
       okay: true,
       result: cvToHex(uintCV(5)), // `get-token-uri` will fail because this is a `uint`
@@ -129,9 +129,7 @@ describe('StacksNodeRpcClient', () => {
       .reply(200, mockResponse);
     setGlobalDispatcher(agent);
 
-    await expect(client.readStringFromContract('get-token-uri', [])).rejects.toThrow(
-      RetryableJobError
-    );
+    await expect(client.readStringFromContract('get-token-uri', [])).rejects.toThrow(Error);
   });
 
   test('incorrect none uri strings are parsed as undefined', async () => {