The automaton npc should be public and not internal.
In my mod, I need to move the automaton away from the spawn point upon world creation. Right now, I can just hardcode the npc ID (688) to get the npc. For tiles in magic storage, weak references work perfectly well and are flexible in case of changes. However, because the automaton is internal I have to hardcode it's ID, which is prone to failure.
To my knowledge, there's no real reason why this should be internal. Mods might want to access the NPC for a plethora of reasons, and I don't see why Magic Storage should impede them and force them to hardcode it's ID.
The automaton npc should be public and not internal.
In my mod, I need to move the automaton away from the spawn point upon world creation. Right now, I can just hardcode the npc ID (688) to get the npc. For tiles in magic storage, weak references work perfectly well and are flexible in case of changes. However, because the automaton is internal I have to hardcode it's ID, which is prone to failure.
To my knowledge, there's no real reason why this should be internal. Mods might want to access the NPC for a plethora of reasons, and I don't see why Magic Storage should impede them and force them to hardcode it's ID.