Duplicate error handling pattern in safe wrapper methods

Problem

The safeSend, safeSendUSDC, safeSendUSDT, etc. methods repeat the same error handling pattern.

Location

src/agent/wallet.ts - safeSend* methods

Current Pattern

async safeSendUSDC(...): Promise<SafeSendResult> {
  try {
    const result = await this.sendUSDC(...);
    return { success: true, result };
  } catch (error) {
    return { success: false, error: ... };
  }
}
// Repeated for each token type

Recommendation

Extract to a helper function or use a decorator pattern:

function wrapSafe<T>(fn: () => Promise<T>): Promise<SafeResult<T>> {
  try {
    const result = await fn();
    return { success: true, result };
  } catch (error) {
    return { success: false, error: normalizeError(error) };
  }
}

// Usage
safeSendUSDC = (...args) => wrapSafe(() => this.sendUSDC(...args));

Priority

Low - Code quality / DRY principle

Start in src/agent/wallet.ts by reading the safeSend* methods and their corresponding send methods to compare the repeated error-handling behavior. Identify whether the existing result and error shapes can share one helper without changing public behavior. Done means the duplicated pattern is removed across the safe wrapper methods while successful sends and normalized failures remain equivalent.