Skip fallback receipt wait

[TuDo1403]

Summary

• Return the direct EOA fallback transaction hash immediately after submission.
• Avoid creating an HTTP receipt waiter from a fallback chain with no RPC URLs.
• Add coverage for the fallback path that previously surfaced the post-submit RPC URL error.

Validation

• pnpm --filter @openzeppelin/role-manager-app exec vitest run src/core/ecosystems/evm/tests/safeSignAndBroadcast.test.ts src/core/ecosystems/evm/tests/EvmAccessManagerService.test.ts
• pnpm --filter @openzeppelin/role-manager-app exec eslint src/core/ecosystems/evm/safeSignAndBroadcast.ts src/core/ecosystems/evm/tests/safeSignAndBroadcast.test.ts

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
role-manager	Ready	Preview, Comment	Apr 13, 2026 9:21am
role-manager-a74d	Ready	Preview, Comment	Apr 13, 2026 9:21am
role-manager-b9b9	Ready	Preview, Comment	Apr 13, 2026 9:21am
role-manager-rotd	Ready	Preview, Comment	Apr 13, 2026 9:21am

[gemini-code-assist]

Code Review

This pull request introduces unit tests for the wrapSignAndBroadcastForSafe function and refactors the sendViaDirectEoa implementation. The refactor removes the logic for waiting for transaction receipts and simplifies the viem imports. A review comment suggests improving test robustness by resetting the window.ethereum global in the beforeEach block to prevent potential state leakage between tests.

[gemini-code-assist]

The beforeEach block resets window.parent but does not reset window.ethereum. While afterEach handles the cleanup, it is generally safer to ensure a clean state for every test by resetting all global mocks and properties in beforeEach as well, to prevent potential state leakage between tests if an afterEach fails or during parallel execution.

  beforeEach(() => {
    vi.clearAllMocks();
    Object.defineProperty(window, 'parent', {
      configurable: true,
      value: originalParent,
    });
    Object.defineProperty(window, 'ethereum', {
      configurable: true,
      value: originalEthereum,
    });
  });