> does entering just 'bob@example.com' no longer work?

I just reverted 8740bff8820fac7c045ec55f82f7059caca85a28 locally and re-ran the tests, and it does actually work. The reason I made that change in the first place was because I didn't actually prevent the parser from throwing exceptions upon failed parses (which were already failing, but being silently ignored by SIP.js) until commit 3ff2ab927a37c2a0cde011dff5f10da57b6a76b3, so I was failing tests due to the unexpected exceptions. It hadn't yet occurred to me that not suppressing the exceptions was why the tests were failing, so I fixed the tests before preventing the exceptions.

I could revert that commit before we merge, but I think it's better (or at least not worse) to just leave the more realistic tests in place.

> I also don't really understand the commit for making debugging easier by putting new lines in, but it's really not big enough of an issue to make an argument to revert.

Similarly, I think this was another artifact of the development process. I'm definitely open to reverting it before merging, depending on what everyone else thinks.

> The addition of the {StartRule: ''} thing seems like an unfortunate side effect we'll have to live with

Yeah, I don't think there's a good way around it. If I ever end up overhauling the grammar setup to parse an entire message in one go, it might reduce the number of `SIP.Grammar.parse` calls that SIP.js has to make.

> I think the rest of this pull request looks good, awesome work

Thanks, I'm really glad I got it working!