Yes, good. Since you are not using Blender 2.80, you will need to do it this way. As I have already spent some time looking at this, I integrated it in TOB (I do not really publish this in recent memory), adding a checkbox to include it.
I refactored it into its own small class. Added a to_json_file(), if the changes to main repo cause the info to not be ignored. I am not making load code changes.