Just an observation - it seems odd (especially since all of these particular XML types are defined by this repo/project) that this XML generation would be done manually (i.e. with writelines) vs. using serialization (which was used everywhere else)