TagsProvider$TagAppender constructor includes an additional and essentially unused mod id argument, making it impossible to extend the class in the common part of multi-loader projects

[Fuzss]

The vanilla constructor for TagsProvider$TagAppender only taking a single TagBuilder is missing, only one requiring an additional mod id in form of a String exists. This makes it impossible to extend TagsProvider$TagAppender (e.g. for adding more convenient overloads) in the common part of a multi-loader project.

The added mod id argument is passed along a couple of times, but never actually ends up being used for anything. According to the Javadoc for the methods in ITagBuilderExtension where the argument eventually ends up being ignored, this was supposed to be used for logging purposes to indicate a source mod (which never happens).
Also both the field and corresponding getter being named mod id is misleading, since e.g. for vanilla data providers not minecraft, but vanilla is passed in.

Would be great if the additional parameter could either be removed to restore the vanilla constructor, or if at least the vanilla constructor could be added back as an overload passing on an empty string.