fix: add policyholder check to approvals and disapprovals and move errors to interfaces

[AustinGreen]

Motivation:

Addressed a few smaller bug fixes and refactors that will improve our code quality.

• Closes chore: try removing the ignored_error_codes specified in foundry.toml #58
• Closes refactor: Move all errors to their respective interfaces #132
• Closes chore: Check if there are sufficient validations for all the non-protected external functions in VertexCore #134
• Closes fix: FOUNDRY_PROFILE=test in github workflow file #136

Modifications:

Moved all errors to their interfaces to clean up the contracts, added a policyholder check for submitting approvals and disapprovals, fixed the name of the test profile in ci.yml, and removed ignored_error_codes in foundry.toml.

Result:

Code will be safer and more consistent.

[github-actions]

Coverage after merging austin/issue-followups into main will be

59.65%

Coverage Report

File	Stmts	Branches	Funcs	Lines	Uncovered Lines
src
VertexAccount.sol	100%	100%	100%	100%
VertexCore.sol	89.78%	87.04%	78.95%	92.92%	101, 103, 180, 187–188, 188, 188–189, 202, 209–210, 210, 210–211, 297
VertexFactory.sol	100%	100%	100%	100%
VertexLens.sol	76.47%	100%	71.43%	80%	93, 97
VertexPolicy.sol	46.63%	26.47%	57.14%	50.93%	104, 106–107, 107, 107–108, 110–111, 118, 120–122, 140, 140, 140–142, 144–146, 149–151, 155–158, 158, 158–159, 162–164, 166, 166, 166–167, 167, 167–168, 177, 184–185, 185–186, 186, 186–187, 205, 205, 205, 207–209, 212–213, 217–218, 241, 250, 265, 279, 287, 48, 63, 67–68, 68, 68–69, 80–81, 86–87, 95–98
VertexStrategy.sol	76.32%	50%	83.33%	78.57%	104, 114, 76–78, 82–84
src/lib
Checkpoints.sol	0%	0%	0%	0%	105, 105, 105–106, 108, 110–111, 113, 113, 113–115, 115, 115–116, 118, 122, 124, 124, 124–126, 136, 146, 153–154, 171–172, 172, 172–173, 175–176, 184, 197, 199, 199, 199, 201, 204, 204, 204, 207, 207, 207–210, 212, 214, 216–217, 234–236, 236, 236–237, 239, 242, 258–260, 260, 260–261, 263, 266, 276, 285, 294, 294, 294–295, 308, 315–322, 331, 333, 333, 333–335, 337, 337, 337–339, 341, 341, 341–343, 345, 345, 345–347, 349, 349, 349, 35, 35, 35, 350–351, 353, 353, 353–355, 357, 357, 357–359, 36, 361, 361, 361–362, 365, 372, 38, 380, 380, 380–381, 389, 389, 389, 39, 390, 40, 53, 53, 53–54, 56–57, 59, 59, 59–61, 71, 71, 71–72, 74, 76–77, 79, 79, 79–81, 81, 81–82, 84, 88, 90
ERC721NonTransferableMinimalProxy.sol	92.75%	95.45%	81.82%	94.44%	117, 189, 44

[0xrajath]

LGTM. Left a few comments