Replace DevTools semver usages with a simpler inlined method

Author: markerikson

packages/react-devtools-shared/src/backend/renderer.js

@@ -7,7 +7,6 @@
  * @flow
  */
 
-import {gt, gte} from 'semver';
 import {
   ComponentFilterDisplayName,
   ComponentFilterElementType,
@@ -39,6 +38,7 @@ import {
   utfEncodeString,
 } from 'react-devtools-shared/src/utils';
 import {sessionStorageGetItem} from 'react-devtools-shared/src/storage';
+import {gt, gte} from 'react-devtools-shared/src/backend/utils';
 import {
   cleanForBridge,
   copyToClipboard,

packages/react-devtools-shared/src/backend/utils.js

@@ -275,3 +275,26 @@ export function isSynchronousXHRSupported(): boolean {
     window.document.featurePolicy.allowsFeature('sync-xhr')
   );
 }
+
+// https://www.npmjs.com/package/semver-compare
+export function semvercmp(a: string = '', b: string = ''): number {
+  const pa = a.split('.');
+  const pb = b.split('.');
+  for (let i = 0; i < 3; i++) {
+    const na = +pa[i];
+    const nb = +pb[i];
+    if (na > nb) return 1;
+    if (nb > na) return -1;
+    if (!isNaN(na) && isNaN(nb)) return 1;
+    if (isNaN(na) && !isNaN(nb)) return -1;
+  }
+  return 0;
+}
+
+export function gt(a: string = '', b: string = ''): boolean {
+  return semvercmp(a, b) === 1;
+}
+
+export function gte(a: string = '', b: string = ''): boolean {
+  return semvercmp(a, b) > -1;
+}

packages/react-devtools-shell/src/e2e-regression/app-legacy.js

@@ -4,9 +4,10 @@
 
 import * as React from 'react';
 import * as ReactDOM from 'react-dom';
-import {gte} from 'semver';
 import ListApp from '../e2e-apps/ListApp';
 import ListAppLegacy from '../e2e-apps/ListAppLegacy';
+import {gte} from 'react-devtools-shared/src/backend/utils';
+
 const version = process.env.E2E_APP_REACT_VERSION;
 
 function mountApp(App: () => React$Node) {