Skill management

Why are we converting this to an ObjectId? Why not just let it be a string? Also, why is this named 'skill' and not 'skill_id' if we are only accepting IDs?