diff --git a/src/L1Deployer.sol b/src/L1Deployer.sol
index 6f24595..55a6bb7 100644
--- a/src/L1Deployer.sol
+++ b/src/L1Deployer.sol
@@ -367,7 +367,7 @@ contract L1Deployer is DeployerBase {
      */
     function getEscrowManager(
         uint32 _rollupID
-    ) public view virtual returns (address) {
+    ) external view virtual returns (address) {
         return _chainConfig[_rollupID].escrowManager;
     }
 
diff --git a/src/RoleManager.sol b/src/RoleManager.sol
index 2dc34ab..e1a8779 100644
--- a/src/RoleManager.sol
+++ b/src/RoleManager.sol
@@ -579,7 +579,7 @@ contract RoleManager is Positions {
      * @param _asset The underlying asset used.
      * @return The default vault for the specified `_asset`.
      */
-    function getVault(address _asset) public view virtual returns (address) {
+    function getVault(address _asset) external view virtual returns (address) {
         return getVault(_asset, ORIGIN_NETWORK_ID);
     }
 
@@ -611,7 +611,7 @@ contract RoleManager is Positions {
      */
     function isVaultsRoleManager(
         address _vault
-    ) public view virtual returns (bool) {
+    ) external view virtual returns (bool) {
         return vaultConfig[_vault].asset != address(0);
     }