Vault Pruning (move PackedTokenBalance)

[EndymionJkb]

Description

After the import cleanup and warning suppression, start "pruning" the Vault by moving things out of it that don't need to be there. PackedTokenBalance is a reusable thing that can live in solidity-utils. We also have pool-utils now, and some of the base pool contracts could be moved there (albeit not easily, and we've probably coupled them a little too tightly, and designed the tests in such as way as to make them hard to move).

For ease of review, the Vault Pruning PRs will be as small and self-contained as possible.

Type of change

• [ ] Bug fix
• [ ] New feature
• [ ] Breaking change
• [ ] Dependency changes
• [X] Code refactor / cleanup
• [ ] Documentation or wording changes
• [ ] Other

Checklist:

• [X] The diff is legible and has no extraneous changes
• [ ] Complex code has been commented, including external interfaces
• [ ] Tests have 100% code coverage
• [X] The base branch is either main, or there's a description of how to merge

Issue Resolution

[openzeppelin-code[bot]]

Vault Pruning (move PackedTokenBalance)

Generated at commit: 5a930259235d720021cdf4a16b2ac75367b00fc3

🚨 Report Summary

	Severity Level	Results
Contracts	Critical High Medium Low Note Total	2 2 0 11 38 53
Dependencies	Critical High Medium Low Note Total	0 0 0 0 0 0

---

For more details view the full report in OpenZeppelin Code Inspector