This isn't actually compliant to the Web IDL spec, which says the prototype's @@toStringTag must be the interface's name + 'Proto'. But from @domenic in nodejs/node#10906 (comment):

Regarding toStringTag, this is actually a controversial part of the Web IDL spec. See https://www.w3.org/Bugs/Public/show_bug.cgi?id=28244. I would suggest following Chrome's version, of just putting it as a data property on the prototype, but I am somewhat biased there :).

Regardless, you probably should remove the existing prototype.toString function.

Yes, I am much happier with this than the version in the Web IDL spec, which nobody implements. But yes, we need to remove the toString function that already exists.

Should I set a getter according to the specs (add 'Prototype' for the prototype) then (which seems controversial to me as well)?

No, as I said, I am much happier with this version than the version in the Web IDL specs. The only change is that we need to remove the toString function that we're currently installing on every class.

No problem :) I personally like this version more as well.

We need to keep toString for the stringifier and impl toString method. Only the stringtag part should be removed.

Oops, sorry... Ridiculously long commit history for such a small change :)

I ended up doing this a bit differently; it turns out that whole generateToString() function is no longer needed. See 0590e5a. Thanks for the kick-start though!

No problem :) Thanks for awesome jsdom!