feat(ctb): Naming improvements

Standardizing on the term owner rather than signer except in and around calls to _getNSigners.

packages/contracts-bedrock/src/Safe/LivenessGuard.sol

@@ -21,9 +21,9 @@ import { EnumerableSet } from "@openzeppelin/contracts/utils/structs/EnumerableS
 contract LivenessGuard is ISemver, BaseGuard {
     using EnumerableSet for EnumerableSet.AddressSet;
 
-    /// @notice Emitted when a signer is recorded.
-    /// @param signer The signer's address.
-    event SignerRecorded(bytes32 indexed txHash, address signer);
+    /// @notice Emitted when an owner is recorded.
+    /// @param owner The owner's address.
+    event OwnerRecorded(bytes32 indexed txHash, address owner);
 
     /// @notice Semantic version.
     /// @custom:semver 1.0.0
@@ -95,7 +95,7 @@ contract LivenessGuard is ISemver, BaseGuard {
 
         for (uint256 i = 0; i < signers.length; i++) {
             lastLive[signers[i]] = block.timestamp;
-            emit SignerRecorded(txHash, signers[i]);
+            emit OwnerRecorded(txHash, signers[i]);
         }
     }
 
@@ -135,7 +135,7 @@ contract LivenessGuard is ISemver, BaseGuard {
         require(SAFE.isOwner(msg.sender), "LivenessGuard: only Safe owners may demontstrate liveness");
         lastLive[msg.sender] = block.timestamp;
 
-        emit SignerRecorded(0x0, msg.sender);
+        emit OwnerRecorded(0x0, msg.sender);
     }
 
     /// @notice Getter function for the Safe contract instance

packages/contracts-bedrock/src/Safe/LivenessModule.sol

@@ -147,11 +147,11 @@ contract LivenessModule is ISemver {
         );
 
         // Check that the guard has not been changed.
-        _verifyGuard();
+        _requireCorrectGuard();
     }
 
     /// @notice Reverts if the guard address does not match the expected value.
-    function _verifyGuard() internal view {
+    function _requireCorrectGuard() internal view {
         require(
             address(LIVENESS_GUARD) == address(uint160(uint256(bytes32(SAFE.getStorageAt(GUARD_STORAGE_SLOT, 1))))),
             "LivenessModule: guard has been changed"

packages/contracts-bedrock/test/LivenessGuard.t.sol

@@ -21,7 +21,7 @@ import { LivenessGuard } from "src/Safe/LivenessGuard.sol";
 contract LivenessGuard_TestInit is Test, SafeTestTools {
     using SafeTestLib for SafeInstance;
 
-    event SignerRecorded(bytes32 indexed txHash, address signer);
+    event OwnerRecorded(bytes32 indexed txHash, address signer);
 
     LivenessGuard livenessGuard;
     SafeInstance safeInstance;
@@ -72,7 +72,7 @@ contract LivenessGuard_CheckTx_Test is LivenessGuard_TestInit {
         for (uint256 i; i < signers.length; i++) {
             // Don't check topic1 so that we can avoid the ugly txHash calculation.
             vm.expectEmit(false, true, true, true, address(livenessGuard));
-            emit SignerRecorded(0x0, signers[i]);
+            emit OwnerRecorded(0x0, signers[i]);
         }
         vm.expectCall(address(safeInstance.safe), abi.encodeWithSignature("nonce()"));
         vm.expectCall(address(safeInstance.safe), abi.encodeCall(OwnerManager.getThreshold, ()));
@@ -99,7 +99,7 @@ contract LivenessGuard_ShowLiveness_Test is LivenessGuard_TestInit {
         address caller = safeInstance.owners[0];
 
         vm.expectEmit(address(livenessGuard));
-        emit SignerRecorded(0x0, caller);
+        emit OwnerRecorded(0x0, caller);
 
         vm.prank(caller);
         livenessGuard.showLiveness();