- The round_start event doesn't have a userid property (and I don't think it is even called in TF2?)
- You should document what numbers correspond to what class
- Hooking player_changeclass is superfluous since you're already hooking player_spawn and forcing the class there
I'd recommend just removing everything except for the player_spawn hook and forcing the class there. Once that's done, I'd say this is approvable.
__________________