Make ?module use latest dependency version when many are given

[dakami]

Attempting to acquire https://unpkg.com/[email protected]/es/index.js?module yields:

import React, { Component } from "https://unpkg.com/react@^15.x.x || ^16.x.x?module";

[mjackson]

Ah, good catch. This is because we pull the version straight from the peerDependencies and dependencies in the package.json file. react-table version 6.8.6 has:

In this case, since it's an ||, we should probably just default to using the latest version of all the options listed.

[natevw]

Hmm, this might be a separate issue but right now I'm not even seeing the peerDependencies value get used?

If I fetch https://unpkg.com/[email protected]/hooks/dist/hooks.mjs?module, the preact import gets rewritten to https://unpkg.com/preact@latest?module even though https://unpkg.com/[email protected]/hooks/package.json specifies:

"peerDependencies": {
    "preact": "^10.0.0-alpha.0"
 }

Since preact@latest redirects to [email protected], the re-written import won't be compatible.