fix: linting warnings and initializer logic

[huaweigu]

Summary

This PR addresses the linting warnings and corrects the logic in WalletStorageInitializable.

Detail

Changeset

• Resolve all warnings (thanks to Ashutosh!)
• Correct the logic in the unused code within WalletStorageInitializable
• Eliminate redundant calls in SingleOwnerMSCA constructor
• Add a sanity check != address(0) in SingleOwnerMSCA

Checklist

• Did you add new tests and confirm all tests pass? (yarn test)
• Did you update relevant docs? (docs are found in the docs folder)
• Do your commits follow the Conventional Commits standard?
• Does your PR title also follow the Conventional Commits standard?
• If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
• Did you run lint (yarn lint) and fix any issues?
• Did you run formatter (yarn format:check) and fix any issues (yarn format:write)?

Testing

• No new tests have been added for style changes.
• New tests have been included for the fixes:
  • For the walletStorageInitializer fix, I have forked the OZ JavaScript tests and rewritten them using Foundry.
  • The remaining two changes are already covered by existing tests (we don't allow installing address(0) and testInitializeSingleOwnerMSCAWithRuntimeValidation).

Documentation

n/a