SpellStorage doesn't inherit from std::vector anymore

[vincent-grosbois]

This is cleaner because now it's impossible to directly modify the underlying std::vector But it required several refactors in different places, so not sure it's worth it overall