auto-sort and format library.json after a push

[bmos]

Parses json, sorts by manufacturer+model, and writes back to file (eliminating trailing whitespace, fixing alphabetizing, and other jank)

[bmos]

Feel free to close this if you have a different approach in mind. I'm hoping this is enough to get the automation working, since it looks like device submissions are really taking off!

[andrew-codechimp]

Thanks for your continued interest in this, I will have a look at this soon, the sorting will be useful as I'm hitting a lot of PR's that don't come sorted.

[ThomDietrich]

Little improvement suggestion: As "battery_quantity" is optional I expect some PRs to introduce a json error in which "battery_type" has a superfluous or missing comma. Could your script default to name the "battery_type" last? @andrew-codechimp wdyt?

[andrew-codechimp]

@ThomDietrich I'm not seeing too many people having an extra last comma
@bmos has done some great work on a device issue template which would make it far easier but not quite ready yet and I'm still catching up on features to look at this, GH actions are really frustrating to test.

[bmos]

One thing I have noticed is that some JSON characters are being automatically escaped that are not escaped by some folks who contribute manually.

[andrew-codechimp]

I do a trial json load before updating the library so they will at least be caught if they were going to cause a problem.

[andrew-codechimp]

Now that I've done with the large features I may get some time to look at this more.
Trying out issues/pr's/scripts on a fork is a little dangerous.
I've created a temp repo and invited you as a contributor, that way we can try as many edits as we want without danger.
The repo is up to date with your patch and the main branch here.
https://github.com/andrew-codechimp/bn-device-test

[bmos]

In merging main back into my branch here, I overwrote what I had in my library file with what you have in main in case I had messed anything up during testing.

[bmos]

It should be all ready for a merge, although I didn't add the new script folder you made:
https://github.com/andrew-codechimp/bn-device-test/tree/main/.github/scripts/library_doc

I'm happy to do it, but figured this way that commit could come from you as the author.

[andrew-codechimp]

I've just added the scripts and merged.
Thanks for all your work on this, it will hopefully make the library grow and save me a lot of time.

I'm leaving the other repo hanging about so I can test stuff in future, if you want to test any changes feel free to over there.