Rework basic output unlock conditions + features

[DaughterOfMars]

Description of change

This PR is a test to see if it's feasible to use static typing for unlock conditions and features instead of arrays of enums. So far I have only changed the BasicOutput.

Downsides

1. In order to get a UnlockConditions or Features from an output, you must now clone. This is usually not necessary since you can just use the specific getter most of the time, but it's useful for certain things.
2. There is a lot of repeated code in Unpack/TryFromDto/try_from_dtos that really should be shared somehow. Unfortunately, this is not very straightforward.