chore(integrity): fixed new flow issues

src/fetchers/tarball-fetcher.js

@@ -265,7 +265,7 @@ export default class TarballFetcher extends BaseFetcher {
 
   _findIntegrity(): ?Object {
     if (this.remote.integrity) {
-      return this.remote.integrity;
+      return ssri.parse(this.remote.integrity);
     }
     if (this.hash) {
       return ssri.fromHex(this.hash, 'sha1');

src/resolvers/registries/npm-resolver.js

@@ -181,6 +181,10 @@ export default class NpmResolver extends RegistryResolver {
   async resolve(): Promise<Manifest> {
     // lockfile
     const shrunk = this.request.getLocked('tarball');
+    if (shrunk && shrunk._remote && shrunk._remote.integrity) {
+      // if the integrity field does not exist, it needs to be created
+      return shrunk;
+    }
     if (shrunk) {
       if (this.config.packBuiltPackages && shrunk.prebuiltVariants && shrunk._remote) {
         const prebuiltVariants = shrunk.prebuiltVariants;
@@ -195,10 +199,6 @@ export default class NpmResolver extends RegistryResolver {
         }
       }
     }
-    if (shrunk && shrunk._remote && shrunk._remote.integrity) {
-      // if the integrity field does not exist, it needs to be created
-      return shrunk;
-    }
 
     const desiredVersion = shrunk && shrunk.version ? shrunk.version : null;
     const info: ?Manifest = await this.resolveRequest(desiredVersion);
@@ -207,7 +207,7 @@ export default class NpmResolver extends RegistryResolver {
     }
 
     const {deprecated, dist} = info;
-    if (shrunk) {
+    if (shrunk && shrunk._remote) {
       shrunk._remote.integrity =
         dist && dist.integrity
           ? ssri.parse(dist.integrity)

src/types.js

@@ -40,7 +40,7 @@ export type PackageRemote = {
   reference: string,
   resolved?: ?string,
   hash: ?string,
-  integrity?: Object,
+  integrity?: ?string,
   packageName?: string,
 };