refactor: minor tweaks

Author: jaybuidl

contracts/src/snapshot-proxy/KlerosCoreSnapshotProxy.sol renamed to contracts/src/arbitration/view/KlerosCoreSnapshotProxy.sol

@@ -8,7 +8,7 @@
 
 pragma solidity 0.8.24;
 
-import {ISortitionModule} from "../arbitration/interfaces/ISortitionModule.sol";
+import {ISortitionModule} from "../interfaces/ISortitionModule.sol";
 
 interface IKlerosCore {
     function sortitionModule() external view returns (ISortitionModule);
@@ -17,25 +17,41 @@ interface IKlerosCore {
 /// @title KlerosCoreSnapshotProxy
 /// Proxy contract for V2 that exposes staked PNK with balanceOf() function for Snapshot voting.
 contract KlerosCoreSnapshotProxy {
+    // ************************************* //
+    // *         State Modifiers           * //
+    // ************************************* //
+
     IKlerosCore public core;
     address public governor;
-    string public name = "Staked Pinakion";
-    string public symbol = "stPNK";
-    uint8 public immutable decimals = 18;
+    string public constant name = "Staked Pinakion";
+    string public constant symbol = "stPNK";
+    uint8 public constant decimals = 18;
+
+    // ************************************* //
+    // *         Modifiers                 * //
+    // ************************************* //
 
     modifier onlyByGovernor() {
         require(governor == msg.sender, "Access not allowed: Governor only.");
         _;
     }
 
+    // ************************************* //
+    // *         Constructor               * //
+    // ************************************* //
+
     /// @dev Constructor
-    /// @param _governor The govenor of the contract.
+    /// @param _governor The governor of the contract.
     /// @param _core KlerosCore to read the balance from.
     constructor(address _governor, IKlerosCore _core) {
         governor = _governor;
         core = _core;
     }
 
+    // ************************************* //
+    // *             Governance            * //
+    // ************************************* //
+
     /// @dev Changes the `governor` storage variable.
     /// @param _governor The new value for the `governor` storage variable.
     function changeGovernor(address _governor) external onlyByGovernor {
@@ -48,6 +64,10 @@ contract KlerosCoreSnapshotProxy {
         core = _core;
     }
 
+    // ************************************* //
+    // *           Public Views            * //
+    // ************************************* //
+
     /// @dev Returns the amount of PNK staked in KlerosV2 for a particular address.
     /// Note: Proxy doesn't need to differentiate between courts so we pass 0 as courtID.
     /// @param _account The address to query.