Little documentation tweaks
MoveInstance uses String reference to Move
Move contestDescription is a string, not an int
MovePropertyFactory added

Following pattern of ItemAttributeFactory and AbilityEffectFactory
Item updates

Documentation clean up
passive and active hold effect fields removed
attribute API cleaned up