Fix #3294 - handle invalid semver strings in crates/:crate_id/versions

* Add a tooltip message and a new icon to components/version-list/row
* Handle null returned from `semverParse` in models/version
* Add `loose` option to `semverParse` (https://github.com/npm/node-semver#functions)

Author: plaets

app/components/version-list/row.hbs

@@ -12,6 +12,8 @@
     <div local-class="release-track">
       {{#if @version.yanked}}
         {{svg-jar "trash"}}
+      {{else if @version.invalidSemver}}
+        ?
       {{else if @version.isFirst}}
         {{svg-jar "star"}}
       {{else}}

app/components/version-list/row.js

@@ -13,6 +13,9 @@ export default class VersionRow extends Component {
     if (version.yanked) {
       return 'This version was yanked';
     }
+    if (version.invalidSemver) {
+      return `Failed to parse version ${version.num}`;
+    }
     if (version.isFirst) {
       return 'This is the first version that was released';
     }

app/models/version.js

@@ -44,29 +44,41 @@ export default class Version extends Model {
     return oldestVersion === this;
   }
 
-  @cached get semver() {
-    return semverParse(this.num);
+  get semver() {
+    return semverParse(this.num, { loose: true });
+  }
+
+  get invalidSemver() {
+    return this.semver === null;
   }
 
   get isPrerelease() {
+    if (this.invalidSemver) {
+      return false;
+    }
+
     return this.semver.prerelease.length !== 0;
   }
 
   get releaseTrack() {
+    if (this.invalidSemver) {
+      return null;
+    }
+
     let { semver } = this;
     return `${semver.major}.${semver.major === 0 ? semver.minor : 'x'}`;
   }
 
   @cached get isHighestOfReleaseTrack() {
-    if (this.isPrerelease) {
+    if (this.isPrerelease || this.invalidSemver) {
       return false;
     }
 
     let { crate, semver, releaseTrack } = this;
     let { versions } = crate;
     // find all other non-prerelease versions on the same release track
     let sameTrackVersions = versions.filter(
-      it => it !== this && !it.yanked && !it.isPrerelease && it.releaseTrack === releaseTrack,
+      it => it !== this && !it.yanked && !it.isPrerelease && !it.invalidSemver && it.releaseTrack === releaseTrack,
     );
     // check if we're the "highest"
     return sameTrackVersions.every(it => it.semver.compare(semver) === -1);