Make the previously loaded artifacts a list rather than an iterator
That's needed as we check its contents once per version referenced in the list of upstream releases.
Test Plan
Running the loader twice on the same module now only loads each artifact once
Migrated from D413 (view on Phabricator)