feat: Basic natspec docs

src/LiquidDelegation.sol

@@ -13,6 +13,7 @@ interface ILiquidDelegation {
     function getPrice() external view returns(uint256);
 }
 
+/// Initial dummy implementation of Liquid Delegation contract
 contract LiquidDelegation is BaseDelegation, ILiquidDelegation {
 
     /// @custom:storage-location erc7201:zilliqa.storage.LiquidDelegation

src/LiquidDelegationV2.sol

@@ -6,6 +6,8 @@ import {ILiquidDelegation} from "src/LiquidDelegation.sol";
 import {NonRebasingLST} from "src/NonRebasingLST.sol";
 
 // the contract is supposed to be deployed with the node's signer account
+/// Liquid Delegation contract V2.
+/// A staking pool implementation in which users receive a Non-Rebasing Liquid Staking Token to represent their staked assets
 contract LiquidDelegationV2 is BaseDelegation, ILiquidDelegation {
 
     /// @custom:storage-location erc7201:zilliqa.storage.LiquidDelegation

src/NonLiquidDelegation.sol

@@ -12,6 +12,7 @@ interface INonLiquidDelegation {
     function rewards() external view returns(uint256);
 }
 
+/// Initial dummy implementation of Non-Liquid Delegation contract
 contract NonLiquidDelegation is BaseDelegation, INonLiquidDelegation {
 
     /* commented out because defining empty structs is disallowed

src/NonLiquidDelegationV2.sol

@@ -5,6 +5,8 @@ import {BaseDelegation} from "src/BaseDelegation.sol";
 import {INonLiquidDelegation} from "src/NonLiquidDelegation.sol";
 import {SafeCast} from "@openzeppelin/contracts/utils/math/SafeCast.sol";
 
+/// Non-Liquid Delegation contract V2.
+/// A staking pool implementation in which users receive no representation for their staked assets. Assets are locked up until the staking period has ended.
 contract NonLiquidDelegationV2 is BaseDelegation, INonLiquidDelegation {
     using SafeCast for int256;
 

src/NonRebasingLST.sol

@@ -4,9 +4,10 @@ pragma solidity ^0.8.26;
 import {ERC20} from "@openzeppelin/contracts/token/ERC20/ERC20.sol";
 import {Ownable} from "@openzeppelin/contracts/access/Ownable.sol";
 
+/// A simple contract implementing ERC20 and Ownable 
 contract NonRebasingLST is ERC20, Ownable {
     constructor(address initialOwner)
-        ERC20("MyToken", "MTK")
+        ERC20("NonRebasingLST", "NR-LST")
         Ownable(initialOwner)
     {}
 

src/utils/WithdrawalQueue.sol

@@ -4,6 +4,7 @@ pragma solidity ^0.8.26;
 /// Interal WithdrawalQueue error
 error WithdrawalQueueError(string msg);
 
+/// Queue implementation for withdrawals from staking contract awaiting an unbonding period
 library WithdrawalQueue {
 
     address public constant DEPOSIT_CONTRACT = address(0x5A494C4445504F53495450524F5859);
@@ -19,11 +20,11 @@ library WithdrawalQueue {
         mapping(uint256 => Item) items;
     }
 
-    function unbondingPeriod() view internal returns(uint256) {
+    function unbondingPeriod() internal view returns(uint256) {
         (bool success, bytes memory data) = DEPOSIT_CONTRACT.staticcall(
             abi.encodeWithSignature("withdrawalPeriod()")
         );
-        require(success, "unbonding period unknown");
+        require(success, WithdrawalQueueError("unbonding period unknown"));
         return abi.decode(data, (uint256));
     }
 
@@ -33,7 +34,7 @@ library WithdrawalQueue {
     }
 
     function dequeue(Fifo storage fifo) internal returns(Item memory result) {
-        require(fifo.first < fifo.last, "queue empty");
+        require(fifo.first < fifo.last, WithdrawalQueueError("queue empty"));
         result = fifo.items[fifo.first];
         delete fifo.items[fifo.first];
         fifo.first++;