I understand that concern, but still think that Tinder should handle it (more) correctly. I doubt that there really is code that handles the NPE and IAE differently, but rather think it either catches both as RunTimeException or Exception, ignores both or handles both equally. But I may be wrong, of course. From the API perspective, I would argue that this is a bug fix given the Javadocs (as released).
BTW, I even think you could add “throws IllegalArgumentException” to the method, even if it does not really change requirements for the caller (IMHO).
With regard to cosmetic, consider me as a new tinder user. I do want to check whether the constructor accepted the parts for a jid, so I catch IAE and suddenly NPEs fly around. I think adding NPE to the docs makes it uglier…
Thanks for considering