fix: change all permission hooks to require UO context

src/account/AccountStorage.sol

@@ -38,8 +38,6 @@ struct ValidationData {
     bool isDefault;
     // Whether or not this validation is a signature validator.
     bool isSignatureValidation;
-    // How many execution hooks require the UO context.
-    uint8 requireUOHookCount;
     // The pre validation hooks for this function selector.
     EnumerableSet.Bytes32Set preValidationHooks;
     // Permission hooks for this validation function.

src/account/PluginManager2.sol

@@ -60,6 +60,7 @@ abstract contract PluginManager2 {
 
             for (uint256 i = 0; i < permissionFunctions.length; ++i) {
                 ExecutionHook memory permissionFunction = permissionFunctions[i];
+                permissionFunction.requireUOContext = true;
 
                 if (
                     !_storage.validationData[validationFunction].permissionHooks.add(toSetValue(permissionFunction))

src/account/UpgradeableModularAccount.sol

@@ -223,7 +223,7 @@ contract UpgradeableModularAccount is
         // Revert if the provided `authorization` less than 21 bytes long, rather than right-padding.
         FunctionReference runtimeValidationFunction = FunctionReference.wrap(bytes21(authorization[:21]));
 
-        if (getAccountStorage().validationData[runtimeValidationFunction].requireUOHookCount > 0) {
+        if (getAccountStorage().validationData[runtimeValidationFunction].permissionHooks.length() > 0) {
             revert RequireUserOperationContext();
         }
 
@@ -401,12 +401,12 @@ contract UpgradeableModularAccount is
 
         _checkIfValidationApplies(selector, userOpValidationFunction, isDefaultValidation);
 
-        // Check if there are exec hooks associated with the validator that require UO context, and revert if the
-        // call isn't to `executeUserOp`
-        // This check must be here because if context isn't passed, we wouldn't be able to get the exec hooks
-        // associated with the validator
+        // Check if there are permission hooks associated with the validator, and revert if the call isn't to
+        // `executeUserOp`
+        // This check must be here because if context isn't passed, we can't tell in execution which hooks should
+        // have ran
         if (
-            getAccountStorage().validationData[userOpValidationFunction].requireUOHookCount > 0
+            getAccountStorage().validationData[userOpValidationFunction].permissionHooks.length() > 0
                 && bytes4(userOp.callData[:4]) != this.executeUserOp.selector
         ) {
             revert RequireUserOperationContext();