Inline withHook in AccountERC7579Hooked's uninstallModule to avoid reverting

[Amxx]

Fixes H-02

PR Checklist

• Tests
• Documentation
• Changeset entry (run npx changeset add)

[changeset-bot]

🦋 Changeset detected

Latest commit: 4998f55

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
openzeppelin-solidity	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[coderabbitai]

Walkthrough

This change enhances hook module uninstallation in ERC7579 by introducing resilience against hook failures. A new withHookNoRevert modifier executes preCheck via the hook while catching failures, and only calls postCheck if preCheck succeeds. The uninstall logic is refactored to route hook uninstalling through private helpers instead of direct state mutation, allowing modules to be uninstalled even if hooks revert. Supporting changes add controllable revert behavior to the mock hook for testing, and extend test coverage to verify uninstallation succeeds when hooks fail during pre-check or post-check phases.

Suggested labels

bug, ignore-changeset, Account Abstraction, security

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding a withHookNoRevert modifier to prevent reverts during hook module uninstallation.
Description check	✅ Passed	The pull request description clearly references a specific security issue (H-02) that the PR fixes and indicates which items were completed (tests and code changes).

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}