refactor: use semver.SemVer instances

Co-authored-by: Kristoffer K. <[email protected]>

packages/plugin-npm/sources/NpmSemverResolver.ts

@@ -84,14 +84,19 @@ export class NpmSemverResolver implements Resolver {
 
   async getSatisfying(descriptor: Descriptor, references: Array<string>, opts: ResolveOptions) {
     const range = new semver.Range(descriptor.range.slice(PROTOCOL.length));
-    const versions = references
-      .map(reference => reference.slice(PROTOCOL.length))
-      .filter(reference => semver.valid(reference));
 
-    return versions
+    return references
+      .map(reference => {
+        try {
+          return new semver.SemVer(reference.slice(PROTOCOL.length));
+        } catch {
+          return null;
+        }
+      })
+      .filter((version): version is semver.SemVer => version !== null)
       .filter(version => range.test(version))
-      .sort((a, b) => -semver.compare(a, b))
-      .map(version => structUtils.makeLocator(descriptor, `${PROTOCOL}${version}`));
+      .sort((a, b) => -a.compare(b))
+      .map(version => structUtils.makeLocator(descriptor, `${PROTOCOL}${version.raw}`));
   }
 
   async resolve(locator: Locator, opts: ResolveOptions) {