diff --git a/lib/forge-std b/lib/forge-std
index 37a37ab7..2f112697 160000
--- a/lib/forge-std
+++ b/lib/forge-std
@@ -1 +1 @@
-Subproject commit 37a37ab73364d6644bfe11edf88a07880f99bd56
+Subproject commit 2f112697506eab12d433a65fdc31a639548fe365
diff --git a/lib/p256-verifier b/lib/p256-verifier
index fcaa173c..29475ae3 160000
--- a/lib/p256-verifier
+++ b/lib/p256-verifier
@@ -1 +1 @@
-Subproject commit fcaa173c36e6a0fd55370f74f949998356869b2c
+Subproject commit 29475ae300ec95d98d5c7cc34c094846f0aa2dcd
diff --git a/lib/solady b/lib/solady
index cde0a5fb..cb801a60 160000
--- a/lib/solady
+++ b/lib/solady
@@ -1 +1 @@
-Subproject commit cde0a5fb594da8655ba6bfcdc2e40a7c870c0cc0
+Subproject commit cb801a60f8319a148697b09d19b748d04e3d65c4
diff --git a/src/validator/WeightedECDSAValidator.sol b/src/validator/WeightedECDSAValidator.sol
index faa4f3bc..d77b8807 100644
--- a/src/validator/WeightedECDSAValidator.sol
+++ b/src/validator/WeightedECDSAValidator.sol
@@ -29,8 +29,8 @@ enum ProposalStatus {
 struct ProposalStorage {
     ProposalStatus status;
     ValidAfter validAfter;
-    uint24 weightApproved;
 }
+// TODO add validUntil
 
 enum VoteStatus {
     NA,
@@ -49,7 +49,7 @@ contract WeightedECDSAValidator is EIP712, IKernelValidator {
         voteStatus;
 
     function _domainNameAndVersion() internal pure override returns (string memory, string memory) {
-        return ("WeightedECDSAValidator", "1");
+        return ("WeightedECDSAValidator", "0.0.1");
     }
 
     function enable(bytes calldata _data) external payable override {
@@ -59,6 +59,7 @@ contract WeightedECDSAValidator is EIP712, IKernelValidator {
         require(weightedStorage[msg.sender].totalWeight == 0, "Already enabled");
         weightedStorage[msg.sender].firstGuardian = msg.sender;
         for (uint256 i = 0; i < _guardians.length; i++) {
+            require(_guardians[i] != msg.sender, "Guardian cannot be self");
             require(_guardians[i] != address(0), "Guardian cannot be 0");
             require(_weights[i] != 0, "Weight cannot be 0");
             require(guardian[_guardians[i]][msg.sender].weight == 0, "Guardian already enabled");
@@ -82,15 +83,44 @@ contract WeightedECDSAValidator is EIP712, IKernelValidator {
         delete weightedStorage[msg.sender];
     }
 
+    function renew(address[] calldata _guardians, uint24[] calldata _weights, uint24 _threshold, uint48 _delay)
+        external
+        payable
+    {
+        require(weightedStorage[msg.sender].totalWeight != 0, "Not enabled");
+        address currentGuardian = weightedStorage[msg.sender].firstGuardian;
+        while (currentGuardian != msg.sender) {
+            address nextGuardian = guardian[currentGuardian][msg.sender].nextGuardian;
+            delete guardian[currentGuardian][msg.sender];
+            currentGuardian = nextGuardian;
+        }
+        delete weightedStorage[msg.sender];
+        require(_guardians.length == _weights.length, "Length mismatch");
+        weightedStorage[msg.sender].firstGuardian = msg.sender;
+        for (uint256 i = 0; i < _guardians.length; i++) {
+            require(_guardians[i] != msg.sender, "Guardian cannot be self");
+            require(_guardians[i] != address(0), "Guardian cannot be 0");
+            require(_weights[i] != 0, "Weight cannot be 0");
+            require(guardian[_guardians[i]][msg.sender].weight == 0, "Guardian already enabled");
+            guardian[_guardians[i]][msg.sender] =
+                GuardianStorage({weight: _weights[i], nextGuardian: weightedStorage[msg.sender].firstGuardian});
+            weightedStorage[msg.sender].firstGuardian = _guardians[i];
+            weightedStorage[msg.sender].totalWeight += _weights[i];
+        }
+        weightedStorage[msg.sender].delay = _delay;
+        weightedStorage[msg.sender].threshold = _threshold;
+    }
+
     function approve(bytes32 _callDataAndNonceHash, address _kernel) external {
         require(guardian[msg.sender][_kernel].weight != 0, "Guardian not enabled");
+        require(weightedStorage[_kernel].threshold != 0, "Kernel not enabled");
         ProposalStorage storage proposal = proposalStatus[_callDataAndNonceHash][_kernel];
         require(proposal.status == ProposalStatus.Ongoing, "Proposal not ongoing");
         VoteStorage storage vote = voteStatus[_callDataAndNonceHash][msg.sender][_kernel];
         require(vote.status == VoteStatus.NA, "Already voted");
         vote.status = VoteStatus.Approved;
-        proposal.weightApproved += guardian[msg.sender][_kernel].weight;
-        if (proposal.weightApproved >= weightedStorage[_kernel].threshold) {
+        (, bool isApproved) = getApproval(_kernel, _callDataAndNonceHash);
+        if (isApproved) {
             proposal.status = ProposalStatus.Approved;
             proposal.validAfter = ValidAfter.wrap(uint48(block.timestamp + weightedStorage[_kernel].delay));
         }
@@ -98,6 +128,7 @@ contract WeightedECDSAValidator is EIP712, IKernelValidator {
 
     function approveWithSig(bytes32 _callDataAndNonceHash, address _kernel, bytes calldata sigs) external {
         uint256 sigCount = sigs.length / 65;
+        require(weightedStorage[_kernel].threshold != 0, "Kernel not enabled");
         ProposalStorage storage proposal = proposalStatus[_callDataAndNonceHash][_kernel];
         require(proposal.status == ProposalStatus.Ongoing, "Proposal not ongoing");
         for (uint256 i = 0; i < sigCount; i++) {
@@ -110,9 +141,10 @@ contract WeightedECDSAValidator is EIP712, IKernelValidator {
             VoteStorage storage vote = voteStatus[_callDataAndNonceHash][signer][_kernel];
             require(vote.status == VoteStatus.NA, "Already voted");
             vote.status = VoteStatus.Approved;
-            proposal.weightApproved += guardian[signer][_kernel].weight;
         }
-        if (proposal.weightApproved >= weightedStorage[_kernel].threshold) {
+
+        (, bool isApproved) = getApproval(_kernel, _callDataAndNonceHash);
+        if (isApproved) {
             proposal.status = ProposalStatus.Approved;
             proposal.validAfter = ValidAfter.wrap(uint48(block.timestamp + weightedStorage[_kernel].delay));
         }
@@ -132,10 +164,15 @@ contract WeightedECDSAValidator is EIP712, IKernelValidator {
         payable
         returns (ValidationData validationData)
     {
-        bytes32 callDataAndNonceHash = keccak256(abi.encode(userOp.callData, userOp.nonce));
+        bytes32 callDataAndNonceHash = keccak256(abi.encode(userOp.sender, userOp.callData, userOp.nonce));
         ProposalStorage storage proposal = proposalStatus[callDataAndNonceHash][msg.sender];
         WeightedECDSAValidatorStorage storage strg = weightedStorage[msg.sender];
-        if (proposal.status == ProposalStatus.Ongoing) {
+        if (strg.threshold == 0) {
+            return SIG_VALIDATION_FAILED;
+        }
+        (uint256 totalWeight, bool passed) = getApproval(msg.sender, callDataAndNonceHash);
+        uint256 threshold = strg.threshold;
+        if (proposal.status == ProposalStatus.Ongoing && !passed) {
             if (strg.delay != 0) {
                 // if delay > 0, only allow proposal to be approved before execution
                 return SIG_VALIDATION_FAILED;
@@ -143,10 +180,9 @@ contract WeightedECDSAValidator is EIP712, IKernelValidator {
             bytes calldata sig = userOp.signature;
             // parse sig with 65 bytes
             uint256 sigCount = sig.length / 65;
-            uint256 totalWeight = proposal.weightApproved;
             address signer;
             VoteStorage storage vote;
-            for (uint256 i = 0; i < sigCount; i++) {
+            for (uint256 i = 0; i < sigCount && !passed; i++) {
                 // last sig is for userOpHash verification
                 signer = ECDSA.recover(
                     _hashTypedData(
@@ -160,14 +196,17 @@ contract WeightedECDSAValidator is EIP712, IKernelValidator {
                 } // skip if already voted
                 vote.status = VoteStatus.Approved;
                 totalWeight += guardian[signer][msg.sender].weight;
+                if (totalWeight >= threshold) {
+                    passed = true;
+                }
             }
-            if (totalWeight >= strg.threshold) {
+            if (passed) {
                 validationData = packValidationData(ValidAfter.wrap(0), ValidUntil.wrap(0));
                 proposal.status = ProposalStatus.Executed;
             } else {
                 validationData = SIG_VALIDATION_FAILED;
             }
-        } else if (proposal.status == ProposalStatus.Approved) {
+        } else if (proposal.status == ProposalStatus.Approved || passed) {
             validationData = packValidationData(proposal.validAfter, ValidUntil.wrap(0));
             proposal.status = ProposalStatus.Executed;
         } else {
@@ -175,6 +214,20 @@ contract WeightedECDSAValidator is EIP712, IKernelValidator {
         }
     }
 
+    function getApproval(address kernel, bytes32 hash) public view returns (uint256 approvals, bool passed) {
+        WeightedECDSAValidatorStorage storage strg = weightedStorage[kernel];
+        for (
+            address currentGuardian = strg.firstGuardian;
+            currentGuardian != address(0);
+            currentGuardian = guardian[currentGuardian][kernel].nextGuardian
+        ) {
+            if (voteStatus[hash][currentGuardian][kernel].status == VoteStatus.Approved) {
+                approvals += guardian[currentGuardian][kernel].weight;
+            }
+        }
+        passed = approvals >= strg.threshold;
+    }
+
     function validCaller(address, bytes calldata) external pure override returns (bool) {
         return false;
     }