@bsv/wallet-toolbox 1.2.33 → 1.2.36

package/src/sdk/WalletStorage.interfaces.ts

@@ -15,26 +15,34 @@ import {
   TXIDHexString
 } from '@bsv/sdk'
 import {
-  sdk,
-  TableSettings,
-  TableOutputBasket,
-  TableCertificateX,
-  TableSyncState,
-  TableUser,
   TableCertificate,
-  TableOutput,
-  TableProvenTxReq,
-  TableProvenTx,
   TableCertificateField,
+  TableCertificateX,
   TableCommission,
-  TableOutputTagMap,
+  TableMonitorEvent,
+  TableOutput,
+  TableOutputBasket,
   TableOutputTag,
+  TableOutputTagMap,
+  TableProvenTx,
+  TableProvenTxReq,
+  TableSettings,
+  TableSyncState,
   TableTransaction,
-  TableTxLabelMap,
   TableTxLabel,
-  TableMonitorEvent
-} from '../index.client'
-import { PostReqsToNetworkDetailsStatus } from '../storage/methods/attemptToPostReqsToNetwork'
+  TableTxLabelMap,
+  TableUser
+} from '../storage/schema/tables'
+import { WalletServices } from './WalletServices.interfaces'
+import {
+  ValidCreateActionArgs,
+  ValidCreateActionOutput,
+  ValidListActionsArgs,
+  ValidListCertificatesArgs,
+  ValidListOutputsArgs
+} from './validationHelpers'
+import { Chain, Paged, ProvenTxReqStatus, TransactionStatus } from './types'
+import { WalletError } from './WalletError'
 
 /**
  * This is the `WalletStorage` interface implemented by a class such as `WalletStorageManager`,
@@ -54,26 +62,26 @@ export interface WalletStorage {
   migrate(storageName: string, storageIdentityKey: string): Promise<string>
   destroy(): Promise<void>
 
-  setServices(v: sdk.WalletServices): void
-  getServices(): sdk.WalletServices
+  setServices(v: WalletServices): void
+  getServices(): WalletServices
   getSettings(): TableSettings
 
-  getAuth(): Promise<sdk.AuthId>
+  getAuth(): Promise<AuthId>
 
   findOrInsertUser(identityKey: string): Promise<{ user: TableUser; isNew: boolean }>
 
   abortAction(args: AbortActionArgs): Promise<AbortActionResult>
-  createAction(args: sdk.ValidCreateActionArgs): Promise<sdk.StorageCreateActionResult>
-  processAction(args: sdk.StorageProcessActionArgs): Promise<sdk.StorageProcessActionResults>
+  createAction(args: ValidCreateActionArgs): Promise<StorageCreateActionResult>
+  processAction(args: StorageProcessActionArgs): Promise<StorageProcessActionResults>
   internalizeAction(args: InternalizeActionArgs): Promise<InternalizeActionResult>
 
-  findCertificates(args: sdk.FindCertificatesArgs): Promise<TableCertificateX[]>
-  findOutputBaskets(args: sdk.FindOutputBasketsArgs): Promise<TableOutputBasket[]>
-  findOutputs(args: sdk.FindOutputsArgs): Promise<TableOutput[]>
-  findProvenTxReqs(args: sdk.FindProvenTxReqsArgs): Promise<TableProvenTxReq[]>
+  findCertificates(args: FindCertificatesArgs): Promise<TableCertificateX[]>
+  findOutputBaskets(args: FindOutputBasketsArgs): Promise<TableOutputBasket[]>
+  findOutputs(args: FindOutputsArgs): Promise<TableOutput[]>
+  findProvenTxReqs(args: FindProvenTxReqsArgs): Promise<TableProvenTxReq[]>
 
   listActions(args: ListActionsArgs): Promise<ListActionsResult>
-  listCertificates(args: sdk.ValidListCertificatesArgs): Promise<ListCertificatesResult>
+  listCertificates(args: ValidListCertificatesArgs): Promise<ListCertificatesResult>
   listOutputs(args: ListOutputsArgs): Promise<ListOutputsResult>
 
   insertCertificate(certificate: TableCertificateX): Promise<number>
@@ -91,12 +99,12 @@ export interface WalletStorageProvider extends WalletStorageSync {
    * @returns true if this object's interface can be extended to the full `StorageProvider` interface
    */
   isStorageProvider(): boolean
-  setServices(v: sdk.WalletServices): void
+  setServices(v: WalletServices): void
 }
 
 export interface WalletStorageSync extends WalletStorageWriter {
   findOrInsertSyncStateAuth(
-    auth: sdk.AuthId,
+    auth: AuthId,
     storageIdentityKey: string,
     storageName: string
   ): Promise<{ syncState: TableSyncState; isNew: boolean }>
@@ -106,10 +114,10 @@ export interface WalletStorageSync extends WalletStorageWriter {
    * @param auth
    * @param newActiveStorageIdentityKey
    */
-  setActive(auth: sdk.AuthId, newActiveStorageIdentityKey: string): Promise<number>
+  setActive(auth: AuthId, newActiveStorageIdentityKey: string): Promise<number>
 
-  getSyncChunk(args: sdk.RequestSyncChunkArgs): Promise<sdk.SyncChunk>
-  processSyncChunk(args: sdk.RequestSyncChunkArgs, chunk: sdk.SyncChunk): Promise<sdk.ProcessSyncChunkResult>
+  getSyncChunk(args: RequestSyncChunkArgs): Promise<SyncChunk>
+  processSyncChunk(args: RequestSyncChunkArgs, chunk: SyncChunk): Promise<ProcessSyncChunkResult>
 }
 
 /**
@@ -127,31 +135,31 @@ export interface WalletStorageWriter extends WalletStorageReader {
 
   findOrInsertUser(identityKey: string): Promise<{ user: TableUser; isNew: boolean }>
 
-  abortAction(auth: sdk.AuthId, args: AbortActionArgs): Promise<AbortActionResult>
-  createAction(auth: sdk.AuthId, args: sdk.ValidCreateActionArgs): Promise<sdk.StorageCreateActionResult>
-  processAction(auth: sdk.AuthId, args: sdk.StorageProcessActionArgs): Promise<sdk.StorageProcessActionResults>
-  internalizeAction(auth: sdk.AuthId, args: InternalizeActionArgs): Promise<InternalizeActionResult>
+  abortAction(auth: AuthId, args: AbortActionArgs): Promise<AbortActionResult>
+  createAction(auth: AuthId, args: ValidCreateActionArgs): Promise<StorageCreateActionResult>
+  processAction(auth: AuthId, args: StorageProcessActionArgs): Promise<StorageProcessActionResults>
+  internalizeAction(auth: AuthId, args: InternalizeActionArgs): Promise<InternalizeActionResult>
 
-  insertCertificateAuth(auth: sdk.AuthId, certificate: TableCertificateX): Promise<number>
+  insertCertificateAuth(auth: AuthId, certificate: TableCertificateX): Promise<number>
 
-  relinquishCertificate(auth: sdk.AuthId, args: RelinquishCertificateArgs): Promise<number>
-  relinquishOutput(auth: sdk.AuthId, args: RelinquishOutputArgs): Promise<number>
+  relinquishCertificate(auth: AuthId, args: RelinquishCertificateArgs): Promise<number>
+  relinquishOutput(auth: AuthId, args: RelinquishOutputArgs): Promise<number>
 }
 
 export interface WalletStorageReader {
   isAvailable(): boolean
 
-  getServices(): sdk.WalletServices
+  getServices(): WalletServices
   getSettings(): TableSettings
 
-  findCertificatesAuth(auth: sdk.AuthId, args: sdk.FindCertificatesArgs): Promise<TableCertificateX[]>
-  findOutputBasketsAuth(auth: sdk.AuthId, args: sdk.FindOutputBasketsArgs): Promise<TableOutputBasket[]>
-  findOutputsAuth(auth: sdk.AuthId, args: sdk.FindOutputsArgs): Promise<TableOutput[]>
-  findProvenTxReqs(args: sdk.FindProvenTxReqsArgs): Promise<TableProvenTxReq[]>
+  findCertificatesAuth(auth: AuthId, args: FindCertificatesArgs): Promise<TableCertificateX[]>
+  findOutputBasketsAuth(auth: AuthId, args: FindOutputBasketsArgs): Promise<TableOutputBasket[]>
+  findOutputsAuth(auth: AuthId, args: FindOutputsArgs): Promise<TableOutput[]>
+  findProvenTxReqs(args: FindProvenTxReqsArgs): Promise<TableProvenTxReq[]>
 
-  listActions(auth: sdk.AuthId, vargs: sdk.ValidListActionsArgs): Promise<ListActionsResult>
-  listCertificates(auth: sdk.AuthId, vargs: sdk.ValidListCertificatesArgs): Promise<ListCertificatesResult>
-  listOutputs(auth: sdk.AuthId, vargs: sdk.ValidListOutputsArgs): Promise<ListOutputsResult>
+  listActions(auth: AuthId, vargs: ValidListActionsArgs): Promise<ListActionsResult>
+  listCertificates(auth: AuthId, vargs: ValidListCertificatesArgs): Promise<ListCertificatesResult>
+  listOutputs(auth: AuthId, vargs: ValidListOutputsArgs): Promise<ListOutputsResult>
 }
 
 export interface AuthId {
@@ -162,8 +170,8 @@ export interface AuthId {
 
 export interface FindSincePagedArgs {
   since?: Date
-  paged?: sdk.Paged
-  trx?: sdk.TrxToken
+  paged?: Paged
+  trx?: TrxToken
 }
 
 export interface FindForUserSincePagedArgs extends FindSincePagedArgs {
@@ -188,7 +196,7 @@ export interface FindOutputBasketsArgs extends FindSincePagedArgs {
 export interface FindOutputsArgs extends FindSincePagedArgs {
   partial: Partial<TableOutput>
   noScript?: boolean
-  txStatus?: sdk.TransactionStatus[]
+  txStatus?: TransactionStatus[]
 }
 
 export type StorageProvidedBy = 'you' | 'storage' | 'you-and-storage'
@@ -212,7 +220,7 @@ export interface StorageCreateTransactionSdkInput {
   senderIdentityKey?: string
 }
 
-export interface StorageCreateTransactionSdkOutput extends sdk.ValidCreateActionOutput {
+export interface StorageCreateTransactionSdkOutput extends ValidCreateActionOutput {
   vout: number
   providedBy: StorageProvidedBy
   purpose?: string
@@ -345,50 +353,50 @@ export interface StorageGetBeefOptions {
 }
 
 export interface StorageSyncReaderOptions {
-  chain: sdk.Chain
+  chain: Chain
 }
 
-export interface FindCertificateFieldsArgs extends sdk.FindSincePagedArgs {
+export interface FindCertificateFieldsArgs extends FindSincePagedArgs {
   partial: Partial<TableCertificateField>
 }
 
-export interface FindCommissionsArgs extends sdk.FindSincePagedArgs {
+export interface FindCommissionsArgs extends FindSincePagedArgs {
   partial: Partial<TableCommission>
 }
-export interface FindOutputTagMapsArgs extends sdk.FindSincePagedArgs {
+export interface FindOutputTagMapsArgs extends FindSincePagedArgs {
   partial: Partial<TableOutputTagMap>
   tagIds?: number[]
 }
-export interface FindOutputTagsArgs extends sdk.FindSincePagedArgs {
+export interface FindOutputTagsArgs extends FindSincePagedArgs {
   partial: Partial<TableOutputTag>
 }
-export interface FindProvenTxReqsArgs extends sdk.FindSincePagedArgs {
+export interface FindProvenTxReqsArgs extends FindSincePagedArgs {
   partial: Partial<TableProvenTxReq>
-  status?: sdk.ProvenTxReqStatus[]
+  status?: ProvenTxReqStatus[]
   txids?: string[]
 }
-export interface FindProvenTxsArgs extends sdk.FindSincePagedArgs {
+export interface FindProvenTxsArgs extends FindSincePagedArgs {
   partial: Partial<TableProvenTx>
 }
-export interface FindSyncStatesArgs extends sdk.FindSincePagedArgs {
+export interface FindSyncStatesArgs extends FindSincePagedArgs {
   partial: Partial<TableSyncState>
 }
-export interface FindTransactionsArgs extends sdk.FindSincePagedArgs {
+export interface FindTransactionsArgs extends FindSincePagedArgs {
   partial: Partial<TableTransaction>
-  status?: sdk.TransactionStatus[]
+  status?: TransactionStatus[]
   noRawTx?: boolean
 }
-export interface FindTxLabelMapsArgs extends sdk.FindSincePagedArgs {
+export interface FindTxLabelMapsArgs extends FindSincePagedArgs {
   partial: Partial<TableTxLabelMap>
   labelIds?: number[]
 }
-export interface FindTxLabelsArgs extends sdk.FindSincePagedArgs {
+export interface FindTxLabelsArgs extends FindSincePagedArgs {
   partial: Partial<TableTxLabel>
 }
-export interface FindUsersArgs extends sdk.FindSincePagedArgs {
+export interface FindUsersArgs extends FindSincePagedArgs {
   partial: Partial<TableUser>
 }
-export interface FindMonitorEventsArgs extends sdk.FindSincePagedArgs {
+export interface FindMonitorEventsArgs extends FindSincePagedArgs {
   partial: Partial<TableMonitorEvent>
 }
 /**
@@ -401,7 +409,7 @@ export interface UpdateProvenTxReqWithNewProvenTxArgs {
   provenTxReqId: number
   txid: string
   attempts: number
-  status: sdk.ProvenTxReqStatus
+  status: ProvenTxReqStatus
   history: string
   height: number
   index: number
@@ -411,7 +419,7 @@ export interface UpdateProvenTxReqWithNewProvenTxArgs {
 }
 
 export interface UpdateProvenTxReqWithNewProvenTxResult {
-  status: sdk.ProvenTxReqStatus
+  status: ProvenTxReqStatus
   history: string
   provenTxId: number
   log?: string
@@ -513,5 +521,5 @@ export interface ProcessSyncChunkResult {
   maxUpdated_at: Date | undefined
   updates: number
   inserts: number
-  error?: sdk.WalletError
+  error?: WalletError
 }

package/src/sdk/types.ts

@@ -196,3 +196,19 @@ export const specOpFailedActions = '97d4eb1e49215e3374cc2c1939a7c43a55e95c7427bf
 export function isListActionsSpecOp(label: string): boolean {
   return [specOpNoSendActions, specOpFailedActions].indexOf(label) >= 0
 }
+
+/**
+ * `createAction` special operation label name value.
+ *
+ * Causes WERR_REVIEW_ACTIONS throw with dummy properties.
+ *
+ */
+export const specOpThrowReviewActions = 'a496e747fc3ad5fabdd4ae8f91184e71f87539bd3d962aa2548942faaaf0047a'
+
+/**
+ * @param label Action / Transaction label name value.
+ * @returns true iff the `label` name is a reserved `createAction` special operation identifier.
+ */
+export function isCreateActionSpecOp(label: string): boolean {
+  return [specOpThrowReviewActions].indexOf(label) >= 0
+}

package/src/storage/StorageKnex.ts

@@ -1,4 +1,4 @@
-import { ListActionsArgs, ListActionsResult, ListOutputsArgs, ListOutputsResult } from '@bsv/sdk'
+import { ListActionsResult, ListOutputsResult } from '@bsv/sdk'
 import { sdk, verifyOne, verifyOneOrNone, verifyTruthy } from '../index.all'
 import {
   KnexMigrations,

package/src/storage/WalletStorageManager.ts

@@ -3,29 +3,25 @@ import {
   AbortActionResult,
   InternalizeActionArgs,
   InternalizeActionResult,
-  ListActionsArgs,
   ListActionsResult,
   ListCertificatesResult,
-  ListOutputsArgs,
   ListOutputsResult,
   RelinquishCertificateArgs,
   RelinquishOutputArgs
 } from '@bsv/sdk'
+import { EntitySyncState } from '../storage/schema/entities'
+import * as sdk from '../sdk'
 import {
-  EntitySyncState,
-  sdk,
-  StorageProvider,
-  StorageSyncReader,
   TableCertificate,
   TableCertificateX,
   TableOutput,
   TableOutputBasket,
   TableProvenTxReq,
   TableSettings,
-  TableUser,
-  wait
-} from '../index.client'
-import { WERR_INVALID_PARAMETER } from '../sdk'
+  TableUser
+} from '../storage/schema/tables'
+import { wait } from '../utility/utilityHelpers'
+import { StorageProvider } from './StorageProvider'
 
 class ManagedStorage {
   isAvailable: boolean

package/src/storage/methods/attemptToPostReqsToNetwork.ts

@@ -1,8 +1,9 @@
 import { Beef, Transaction } from '@bsv/sdk'
 import { StorageProvider } from '../StorageProvider'
 import { EntityProvenTxReq } from '../schema/entities'
-import { sdk, wait } from '../../index.client'
+import * as sdk from '../../sdk'
 import { ReqHistoryNote } from '../../sdk'
+import { wait } from '../../utility/utilityHelpers'
 
 /**
  * Attempt to post one or more `ProvenTxReq` with status 'unsent'

package/test/Wallet/local/localWallet.man.test.ts

@@ -2,12 +2,10 @@ import { EntitySyncState, sdk } from '../../../src'
 import { _tu } from '../../utils/TestUtilsWalletStorage'
 import { specOpInvalidChange } from '../../../src/sdk'
 import {
-  burnOneSatTestOutput,
   createOneSatTestOutput,
   createSetup,
   LocalWalletTestOptions,
-  recoverOneSatTestOutputs
-} from './localWalletMethods'
+} from '../../utils/localWalletMethods'
 
 const chain: sdk.Chain = 'test'
 

package/test/Wallet/local/localWallet2.man.test.ts

@@ -1,4 +1,5 @@
-import { EntitySyncState, sdk, Services, Setup, StorageKnex, TableOutput, TableUser } from '../../../src'
+import { Beef, WalletOutput } from '@bsv/sdk'
+import { sdk, Services, Setup, StorageKnex, TableUser } from '../../../src'
 import { _tu, TuEnv } from '../../utils/TestUtilsWalletStorage'
 import { specOpInvalidChange, ValidListOutputsArgs, WERR_REVIEW_ACTIONS } from '../../../src/sdk'
 import {
@@ -8,11 +9,9 @@ import {
   doubleSpendOldChange,
   LocalWalletTestOptions,
   recoverOneSatTestOutputs
-} from './localWalletMethods'
-import { abort } from 'process'
+} from '../../utils/localWalletMethods'
 
 import * as dotenv from 'dotenv'
-import { WalletOutput } from '@bsv/sdk'
 dotenv.config()
 
 const chain: sdk.Chain = 'main'
@@ -144,6 +143,71 @@ describe('localWallet2 tests', () => {
     }
     await storage.destroy()
   })
+
+  test('6 review and unfail false doubleSpends', async () => {
+    const { env, storage, services } = await createMainReviewSetup()
+    let offset = 1100
+    const limit = 100
+    let allUnfails: number[] = []
+    for (;;) {
+      let log = ''
+      const unfails: number[] = []
+      const reqs = await storage.findProvenTxReqs({ partial: { status: 'doubleSpend' }, paged: { limit, offset } })
+      for (const req of reqs) {
+        const gsr = await services.getStatusForTxids([req.txid])
+        if (gsr.results[0].status !== 'unknown') {
+          log += `unfail ${req.provenTxReqId} ${req.txid}\n`
+          unfails.push(req.provenTxReqId)
+        }
+      }
+      console.log(`OFFSET: ${offset} ${unfails.length} unfails\n${log}`)
+      allUnfails = allUnfails.concat(unfails)
+      if (reqs.length < limit) break
+      offset += reqs.length
+    }
+    debugger
+    for (const id of allUnfails) {
+      await storage.updateProvenTxReq(id, { status: 'unfail' })
+    }
+    await storage.destroy()
+  })
+
+  test('7 review and unfail false invalids', async () => {
+    const { env, storage, services } = await createMainReviewSetup()
+    let offset = 400
+    const limit = 100
+    let allUnfails: number[] = []
+    for (;;) {
+      let log = ''
+      const unfails: number[] = []
+      const reqs = await storage.findProvenTxReqs({ partial: { status: 'invalid' }, paged: { limit, offset } })
+      for (const req of reqs) {
+        if (!req.txid || !req.rawTx) continue
+        const gsr = await services.getStatusForTxids([req.txid])
+        if (gsr.results[0].status !== 'unknown') {
+          log += `unfail ${req.provenTxReqId} ${req.txid}\n`
+          unfails.push(req.provenTxReqId)
+        }
+      }
+      console.log(`OFFSET: ${offset} ${unfails.length} unfails\n${log}`)
+      allUnfails = allUnfails.concat(unfails)
+      if (reqs.length < limit) break
+      offset += reqs.length
+    }
+    debugger
+    for (const id of allUnfails) {
+      await storage.updateProvenTxReq(id, { status: 'unfail' })
+    }
+    await storage.destroy()
+  })
+
+  test('8 jackie Beef', async () => { 
+    const setup = await createSetup(chain, options)
+    const beef = Beef.fromBinary(beefJackie)
+    console.log(beef.toLogString())
+    const ok = beef.verify(await setup.services.getChainTracker())
+    await setup.wallet.destroy()
+  })
 })
 
 async function createMainReviewSetup(): Promise<{
@@ -167,3 +231,6 @@ async function createMainReviewSetup(): Promise<{
   await storage.makeAvailable()
   return { env, storage, services }
 }
+
+  const beefJackie = [
+    1, 1, 1, 1, 196, 222, 98, 76, 119, 112, 138, 49, 125, 79, 3, 8, 17, 96, 88, 134, 18, 94, 233, 6, 43, 58, 55, 200, 53, 21, 225, 58, 243, 130, 114, 64, 2, 0, 190, 239, 0, 1, 0, 1, 0, 0, 0, 1, 157, 193, 59, 124, 10, 214, 21, 108, 182, 51, 203, 122, 124, 52, 230, 65, 248, 166, 3, 136, 224, 45, 213, 116, 91, 81, 101, 168, 142, 252, 196, 20, 110, 0, 0, 0, 107, 72, 48, 69, 2, 33, 0, 144, 86, 132, 240, 56, 253, 101, 20, 254, 1, 184, 144, 98, 236, 225, 242, 239, 88, 99, 196, 58, 33, 141, 79, 234, 140, 7, 22, 254, 140, 65, 83, 2, 32, 113, 198, 86, 176, 19, 16, 165, 168, 5, 227, 70, 44, 5, 22, 144, 179, 172, 170, 13, 148, 3, 236, 35, 2, 74, 238, 235, 84, 148, 192, 102, 138, 65, 33, 3, 15, 101, 106, 207, 42, 192, 187, 51, 59, 128, 27, 240, 244, 240, 4, 224, 230, 41, 166, 89, 216, 46, 7, 24, 242, 180, 20, 90, 12, 57, 59, 144, 255, 255, 255, 255, 2, 136, 19, 0, 0, 0, 0, 0, 0, 25, 118, 169, 20, 240, 178, 178, 204, 51, 126, 211, 251, 43, 177, 154, 94, 189, 29, 53, 41, 220, 136, 142, 80, 136, 172, 208, 140, 0, 0, 0, 0, 0, 0, 25, 118, 169, 20, 217, 48, 110, 108, 236, 100, 116, 90, 181, 114, 45, 176, 198, 216, 150, 134, 16, 251, 10, 177, 136, 172, 0, 0, 0, 0]

package/test/Wallet/specOps/specOps.man.test.ts

@@ -1,5 +1,11 @@
+import { WalletClient } from '@bsv/sdk'
 import { sdk, verifyOne } from '../../../src'
-import { specOpInvalidChange, specOpSetWalletChangeParams, specOpWalletBalance } from '../../../src/sdk'
+import {
+  specOpInvalidChange,
+  specOpSetWalletChangeParams,
+  specOpThrowReviewActions,
+  specOpWalletBalance
+} from '../../../src/sdk'
 import { _tu, TestWalletNoSetup } from '../../utils/TestUtilsWalletStorage'
 
 describe('specOps tests', () => {
@@ -156,6 +162,40 @@ describe('specOps tests', () => {
 
     await setup.wallet.destroy()
   })
+
+  test('5 Wallet specOpThrowReviewActions', async () => {
+    const setup = await createSetup('test')
+
+    try {
+      const r = await setup.wallet.createAction({
+        labels: [specOpThrowReviewActions],
+        description: 'must throw'
+      })
+      expect(true).toBe(false)
+    } catch (eu: unknown) {
+      const e = sdk.WalletError.fromUnknown(eu) as sdk.WERR_REVIEW_ACTIONS
+      expect(e.code).toBe('WERR_REVIEW_ACTIONS')
+      expect(e.reviewActionResults).toBeTruthy()
+    }
+
+    await setup.wallet.destroy()
+  })
+
+  test('6 WalletClient specOpThrowReviewActions', async () => {
+    const wallet = new WalletClient(undefined, '6.specOps.man.test')
+
+    try {
+      const r = await wallet.createAction({
+        labels: [specOpThrowReviewActions],
+        description: 'must throw'
+      })
+      expect(true).toBe(false)
+    } catch (eu: unknown) {
+      const e = sdk.WalletError.fromUnknown(eu) as sdk.WERR_REVIEW_ACTIONS
+      expect(e.code).toBe('WERR_REVIEW_ACTIONS')
+      expect(e.reviewActionResults).toBeTruthy()
+    }
+  })
 })
 
 async function createSetup(chain: sdk.Chain): Promise<TestWalletNoSetup> {