Revamp issue handling

[happy5214]

This PR makes several changes to issue handling:

• BidsIssue has been merged into BidsHedIssue, and most of the combined class's fields have been renamed.
• A new static method has been added to IssueError as a shortcut for throwing internal errors.
• All remaining uses of plain Errors have been converted to IssueErrors.
• internalConsistencyError has been merged into internalError.

I also added a few types back where allowed by JSDoc. In particular, I found that Map and Set generics are allowed.

I attempted to create an IssueError analog for BidsHedIssue, but it wouldn't wrap issues created within the partnered schema merge, so I removed it.

[VisLab]

Looks good -- except I wonder why in issues.js on line 75 this returns a promise?

Also, can you add the additional items we discussed to the public interface. Then you can go ahead and merge.

[happy5214]

Looks good -- except I wonder why in issues.js on line 75 this returns a promise?

Also, can you add the additional items we discussed to the public interface. Then you can go ahead and merge.

It returns a Promise because all async functions return those implicitly.

I was planning to do the API changes in a separate PR once we finalized the list (which I didn't feel like we completed yesterday).