Is the providers array and the stuff with remainingDOIs still relevant now there's only one provider?

Is the providers array and the stuff with remainingDOIs still relevant now there's only one provider?

You're completely right, I think we can scrap all of that (need to make sure that we're still failing gracefully on DOIs that either aren't DOIs or aren't covered by the search translator)

You want me to take a stab at that?

I couldn't find anything in the existing code that dealt with graceful handling of non-resolvable DOIs. The remaining DOIs were passed to the next provider, but if DOIs went unresolved at the end of that chain, completeDOIs was just called.

Not that I'd propose changing too much at the same time, but are promises available in browser? So much of this could at a later change be cleaned up with promises.

if DOIs went unresolved at the end of that chain, completeDOIs was just called

That's graceful enough for given purposes

are promises available in browser

I believe so, yes, but would want @dstillman to confirm. Obviously only four Zotero version 5+

It's not something I intend to introduce right now, but I can see the outline of how it would look.

I was a little nervous around removing the IIFE as sometimes async requires it to keep scope, but I've looked at the code and it seems safe. Tests pass.

This is great, thanks -- and yes, it does feel quite zippy