@polylith/builder 0.0.27 → 0.0.28

package/App.js

@@ -156,18 +156,28 @@ export default class App {
 	 * If no manual chunk specifiers are added, then this will be used as the
 	 * default.
 	 *
-	 * @param {String} id the filename being processed.
+	 * @param {String} id this is the filename of the current file being
+	 * 	processed by rollup
 	 *
 	 * @returns {String} the chunk name if there is a match
 	 */
-	defaultManualChunkCallback(id) {
+	defaultManualChunks(id) {
 		if (id.includes('node_modules')) {
 			return 'vendor';
 		}
 	}
 
-	manualChunksArray(id) {
-		var fixId = this.fixPath(id);
+	/**
+	 * This is called when manual chunk specifications have been added as an
+	 * array.
+	 *
+	 * @param {String} id this is the filename of the current file being
+	 * 	processed by rollup
+	 * @returns {String} the name fo the chunk if there is a matching chunk
+	 * 	name specifier
+	 */
+	handleManualChunksArray(id) {
+		var fixId = App.fixPath(id);
 		var result = this.manualChunks.find(function(spec) {
 			return fixId.includes(spec.includes);
 		})
@@ -175,22 +185,38 @@ export default class App {
 		if (result) return result.name;
 	}
 
-	manualChunksCallback(id) {
-		var fixId = this.fixPath(fixId);
+	/**
+	 * This is called when the manual chunk specifiers are functions. Each
+	 * registered function is called in the reverse order to how they were
+	 * added.
+	 *
+	 * @param {String} id this is the filename of the current file being
+	 * 	processed by rollup
+	 * @returns the chunk name if any of the callbacks return one
+	 */
+	handleManualChunksCallbacks(id) {
+		var fixId = App.fixPath(id);
 		if (this.manualChunks.length === 0) {
 			return this.defaultManualChunks(fixId);
 		} else {
 			for (let idx = 0; idx < this.manualChunks.length; idx++) {
-				var result = this.manualChunks[id](fixId);
+				var result = this.manualChunks[idx](fixId);
 				if (result) return result;
 			}
 		}
 	}
 
+	/**
+	 * This is called by the builder to get the method to handle manual chunks
+	 * based on how they have been setup. The value returned from this funtion
+	 * will be the value set in the rollup options object.
+	 *
+	 * @returns {Object|Function} the value to assign to the manualChunk field.
+	 */
 	getManualChunks() {
-		if (this.manualChunksType === 'object') return this.manualChunks;
-		if (this.manualChunksType === 'array') return this.manualChunksArray.bind(this);
-		if (this.manualChunksType === 'function') return this.manualChunksCallback.bind(this);
+		if (this.manualChunkType === 'object') return this.manualChunks;
+		if (this.manualChunkType === 'array') return this.handleManualChunksArray.bind(this);
+		if (this.manualChunkType === 'function') return this.handleManualChunksCallbacks.bind(this);
 	}
 
 	/**
@@ -245,18 +271,18 @@ export default class App {
 		}
 	}
 
-	/* features will call this method if they want to add loadables
-
-	  name - name of the loadable that we be passed to the load method
-	  main - the relative path from the source folder to the entry point of the
-	    loadable.
-	  prefix - if given, the prefix on services created in this loadable.
-	    The start and ready methods will be called on these services
-	*/
+	/**
+	 *
+	 * @param {String} name unique name of the loadable that will be passed to
+	 * 	the load method
+	 * @param {String} main the relative path from the source folder to the entry
+	 * 	point of the loadable.
+	 * @param {String} [prefix] if given, the prefix on services created in
+	 * 	this loadable. When the loadable has been loaded, the start and ready
+	 *  methods will be called on all services starting with this prefix.
+	 */
 	addLoadable(name, main, prefix) {
 		var dest = path.posix.join(this.sourcePath, main);
-
-		console.log('addLoadable', {name, path: dest, prefix})
 		this.loadables.push({name, path: dest, prefix});
 	}
 
@@ -326,10 +352,13 @@ export default class App {
 						return '[name].js';
 					}.bind(this),
 					manualChunks: manualChunks,
-					treeshake: {
-						moduleSideEffects: true
-					}
 				},
+			},
+			watch : {
+				watch: {
+					buildDelay: 250,
+					exclude: 'node_modules/**',
+				}
 			}
 		};
 
@@ -338,11 +367,27 @@ export default class App {
 
 	async build() {
 		await this.buildFeatures();
-		var config = this.buildConfiguration();
+		this.config = this.buildConfiguration();
 
-		const bundle = await rollup.rollup(config.input);
-		await bundle.generate(config.output);
-		await bundle.write(config.output);
+		const bundle = await rollup.rollup(this.config.input);
+		await bundle.generate(this.config.output);
+		await bundle.write(this.config.output);
 		await bundle.close();
 	}
+
+	watch() {
+		var watchConfig  = {
+			input: this.config.input,
+			output: this.config.output,
+			watch: this.config.watch,
+		}
+
+		const watcher = rollup.watch(watchConfig);
+
+		watcher.on('event', function({result}) {
+		  if (result) {
+			  result.close();
+		  }
+		}.bind(this));
+	}
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polylith/builder",
-  "version": "0.0.27",
+  "version": "0.0.28",
   "description": "The polylith builder",
   "main": "index.js",
   "type": "module",