random_safe_spawn_coord infinite loop

The range parameter is used in the repeat loop as one of the exit clauses, but this will only trigger if range is 3 or higher. Ranges of 1 or 2 can result in the loop never ending and the game freezing.

on_post_command

Be ware of using this with checks against time, there's no guarantee it will be called (due to various speed/time slicing shenanigans) within a given time frame so it's entirely possible for things relying on_post_command being called at a certain time or within a time period to work inconsistently.

attaching entities to elevators like locks

Lock entities attached to elevators should be attached using elevator:equip("entity") not elevator:attach("entity") otherwise the elevator won't have the child entities graphics blueprint correctly applied on enter level (though it will weirdly apply if you save and reload the level)