Description

Adds comprehensive validation for deposit transactions to prevent
silent failures causing 'not enough input to decode' errors.

Changes:

• Add transaction validation in depositWithConfirmation:
  • Verify transaction and params objects exist
  • Validate 'to' address format (0x prefix, 42 chars)
  • Validate data field exists and has valid hex format
  • Enforce minimum data length (10 chars)
• Improve error handling in polymarket safe utils:
  • Re-throw errors in getDeployProxyWalletTransaction instead of
    returning undefined
  • Add try-catch with proper error propagation in
    getProxyWalletAllowancesTransaction
  • Add call data length validation
• Update tests with valid mock addresses/data
• Add test coverage for all new validation error scenarios

Changelog

CHANGELOG entry: null

Related issues

Fixes: #24804

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.