Don’t pass package.json and options over IPC (#1304)

* Don’t pass package.json over IPC
* Add backward compatibility getter for asset.package

Author: devongovett

src/Asset.js

@@ -5,6 +5,8 @@ const objectHash = require('./utils/objectHash');
 const md5 = require('./utils/md5');
 const isURL = require('./utils/is-url');
 const config = require('./utils/config');
+const syncPromise = require('./utils/syncPromise');
+const logger = require('./Logger');
 
 let ASSET_ID = 1;
 
@@ -15,12 +17,11 @@ let ASSET_ID = 1;
  * for subclasses to implement.
  */
 class Asset {
-  constructor(name, pkg, options) {
+  constructor(name, options) {
     this.id = ASSET_ID++;
     this.name = name;
     this.basename = path.basename(this.name);
     this.relativeName = path.relative(options.rootDir, this.name);
-    this.package = pkg || {};
     this.options = options;
     this.encoding = 'utf8';
     this.type = path.extname(this.name).slice(1);
@@ -98,13 +99,35 @@ class Asset {
     );
 
     parsed.pathname = this.options.parser
-      .getAsset(resolved, this.package, this.options)
+      .getAsset(resolved, this.options)
       .generateBundleName();
 
     return URL.format(parsed);
   }
 
+  get package() {
+    logger.warn(
+      '`asset.package` is deprecated. Please use `await asset.getPackage()` instead.'
+    );
+    return syncPromise(this.getPackage());
+  }
+
+  async getPackage() {
+    if (!this._package) {
+      this._package = await this.getConfig(['package.json']);
+    }
+
+    return this._package;
+  }
+
   async getConfig(filenames, opts = {}) {
+    if (opts.packageKey) {
+      let pkg = await this.getPackage();
+      if (pkg && pkg[opts.packageKey]) {
+        return pkg[opts.packageKey];
+      }
+    }
+
     // Resolve the config file
     let conf = await config.resolve(opts.path || this.name, filenames);
     if (conf) {

src/Bundler.js

@@ -357,12 +357,16 @@ class Bundler extends EventEmitter {
   }
 
   async resolveAsset(name, parent) {
-    let {path, pkg} = await this.resolver.resolve(name, parent);
+    let {path} = await this.resolver.resolve(name, parent);
+    return this.getLoadedAsset(path);
+  }
+
+  getLoadedAsset(path) {
     if (this.loadedAssets.has(path)) {
       return this.loadedAssets.get(path);
     }
 
-    let asset = this.parser.getAsset(path, pkg, this.options);
+    let asset = this.parser.getAsset(path, this.options);
     this.loadedAssets.set(path, asset);
 
     this.watch(path, asset);
@@ -490,7 +494,7 @@ class Bundler extends EventEmitter {
     let startTime = Date.now();
     let processed = this.cache && (await this.cache.read(asset.name));
     if (!processed || asset.shouldInvalidate(processed.cacheData)) {
-      processed = await this.farm.run(asset.name, asset.package, this.options);
+      processed = await this.farm.run(asset.name);
       if (this.cache) {
         this.cache.write(asset.name, processed);
       }

src/Parser.js

@@ -75,10 +75,10 @@ class Parser {
     return parser;
   }
 
-  getAsset(filename, pkg, options = {}) {
+  getAsset(filename, options = {}) {
     let Asset = this.findParser(filename);
     options.parser = this;
-    return new Asset(filename, pkg, options);
+    return new Asset(filename, options);
   }
 }
 

src/Pipeline.js

@@ -11,8 +11,13 @@ class Pipeline {
     this.parser = new Parser(options);
   }
 
-  async process(path, pkg, options) {
-    let asset = this.parser.getAsset(path, pkg, options);
+  async process(path, isWarmUp) {
+    let options = this.options;
+    if (isWarmUp) {
+      options = Object.assign({isWarmUp}, options);
+    }
+
+    let asset = this.parser.getAsset(path, options);
     let generated = await this.processAsset(asset);
     let generatedMap = {};
     for (let rendition of generated) {
@@ -51,7 +56,7 @@ class Pipeline {
       );
       if (!(asset instanceof AssetType)) {
         let opts = Object.assign({rendition}, asset.options);
-        let subAsset = new AssetType(asset.name, asset.package, opts);
+        let subAsset = new AssetType(asset.name, opts);
         subAsset.contents = value;
         subAsset.dependencies = asset.dependencies;
 

src/assets/CSSAsset.js

@@ -9,8 +9,8 @@ const IMPORT_RE = /@import/;
 const PROTOCOL_RE = /^[a-z]+:/;
 
 class CSSAsset extends Asset {
-  constructor(name, pkg, options) {
-    super(name, pkg, options);
+  constructor(name, options) {
+    super(name, options);
     this.type = 'css';
   }
 

src/assets/CoffeeScriptAsset.js

@@ -2,8 +2,8 @@ const Asset = require('../Asset');
 const localRequire = require('../utils/localRequire');
 
 class CoffeeScriptAsset extends Asset {
-  constructor(name, pkg, options) {
-    super(name, pkg, options);
+  constructor(name, options) {
+    super(name, options);
     this.type = 'js';
   }
 

src/assets/GLSLAsset.js

@@ -5,8 +5,8 @@ const promisify = require('../utils/promisify');
 const Resolver = require('../Resolver');
 
 class GLSLAsset extends Asset {
-  constructor(name, pkg, options) {
-    super(name, pkg, options);
+  constructor(name, options) {
+    super(name, options);
     this.type = 'js';
   }
 

src/assets/GlobAsset.js

@@ -4,8 +4,8 @@ const micromatch = require('micromatch');
 const path = require('path');
 
 class GlobAsset extends Asset {
-  constructor(name, pkg, options) {
-    super(name, pkg, options);
+  constructor(name, options) {
+    super(name, options);
     this.type = null; // allows this asset to be included in any type bundle
   }
 

src/assets/GraphqlAsset.js

@@ -2,8 +2,8 @@ const Asset = require('../Asset');
 const localRequire = require('../utils/localRequire');
 
 class GraphqlAsset extends Asset {
-  constructor(name, pkg, options) {
-    super(name, pkg, options);
+  constructor(name, options) {
+    super(name, options);
     this.type = 'js';
   }
 

src/assets/HTMLAsset.js

@@ -73,8 +73,8 @@ const OPTIONS = {
 };
 
 class HTMLAsset extends Asset {
-  constructor(name, pkg, options) {
-    super(name, pkg, options);
+  constructor(name, options) {
+    super(name, options);
     this.type = 'html';
     this.isAstDirty = false;
   }