Thanks for reporting the bug @mootzville, the templates property is no longer required in a custom theme's theme.json in v0.5.1 (but since we haven't even launched v0.5.0 yet, don't worry about a thing, the fix is backwards compatible)
A couple of years ago there was also a lower adoption of html5 elements. Having one h1 tag per page is still a good Idea in general, but it is allowable to have one h1 tag per containing article tag as well. Since this theme does not seem to use tags like article and aside (that I've seen in my quick check) I agree that there should only be one h1 tag per page which contains the page title or headline. The theme structure is otherwise good, but could definitely use with some contextual revamping of quite a few divs.