Cleanup, document, natspec for contract

[0xAlcibiades]

No description provided.

[codecov-commenter]

Codecov Report

Merging #150 (97c9b87) into audit-fixes (a78ca48) will decrease coverage by 0.12%.
The diff coverage is 100.00%.

@@               Coverage Diff               @@
##           audit-fixes     #150      +/-   ##
===============================================
- Coverage        88.94%   88.82%   -0.13%     
===============================================
  Files                2        2              
  Lines              371      367       -4     
  Branches            56       56              
===============================================
- Hits               330      326       -4     
  Misses              31       31              
  Partials            10       10              

Impacted Files	Coverage Δ
src/TokenURIGenerator.sol	71.09% <ø> (ø)
src/OptionSettlementEngine.sol	98.32% <100.00%> (-0.03%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[neodaoist]

Additional — personally I would remove subheaders in Events, doesn't add that much imo.

If keeping them, need to move OptionsWritten to top section

Subheaders in Errors adds more value, but needs more categories, they're wrong/incomplete now.

[0xAlcibiades]

@neodaoist we can do subheaders and reorder in a follow on PR.