Add some documentation about onParseFile and onAddModule options.

.jshintrc

@@ -35,13 +35,13 @@
 	"sub" : true,
 	"strict" : false,
 	"white" : true,
-	"smarttabs": true,
-	"expr": true,
-	"maxparams": 5,
-	"maxdepth": 5,
-	"maxstatements": 25,
-	"maxcomplexity": 10,
-	"predef": [
+	"smarttabs" : true,
+	"expr" : true,
+	"maxparams" : 5,
+	"maxdepth" : 5,
+	"maxstatements" : 25,
+	"maxcomplexity" : 10,
+	"predef" : [
 		"define",
 		"describe",
 		"it"

README.md

@@ -64,6 +64,8 @@ Only required if you want to generate the visual graphs using [Graphviz](http://
 - {Boolean} **breakOnError**. True if the parser should stop on parse errors and when modules are missing, false otherwise. Defaults to false.
 - {Boolean} **optimized**. True if the parser should read modules from a optimized file (r.js). Defaults to false.
 - {String} **requireConfig**. Path to RequireJS config used to find shim dependencies. Not used by default.
+- {Function} **onParseFile**. Function to be called when parsing a file (argument will be an object with "filename" and "src" property set).
+- {Function} **onAddModule** . Function to be called when adding a module to the module tree (argument will be an object with "id" and "dependencies" property set).
 
 ## dependency object (returned from madge)
 
@@ -223,6 +225,9 @@ minimize a global energy function, which is equivalent to statistical multi-dime
 
 # Release Notes
 
+## v0.3.0 (May 25, 2014)
+Added support for onParseFile and onAddModule options (Thanks to Brandon Selway).
+
 ## v0.2.0 (April 17, 2014)
 Added support for including shim dependencies found in RequiredJS config (specify with option -R).
 

lib/parse/base.js

@@ -17,12 +17,14 @@ var fs = require('fs'),
  * @constructor
  * @param {Array} src
  * @param {Object} opts
+ * @param {Object} parent
  */
-var Base = module.exports = function (src, opts, parent) {
-	if(opts.onParseFile){
+var Base = module.exports = function(src, opts, parent) {
+	if (opts.onParseFile) {
 		this.on('parseFile', opts.onParseFile.bind(parent));
 	}
-	if(opts.onAddModule){
+
+	if (opts.onAddModule) {
 		this.on('addModule', opts.onAddModule.bind(parent));
 	}
 

test/pluggable.js

@@ -8,15 +8,15 @@ describe('Madge', function () {
 			var fileAdd = "";
 			var idAdd = "";
 			var opts = {};
-			opts.onParseFile = function(obj){
+			opts.onParseFile = function(obj) {
 				var arr = obj.filename.split(path.sep);
 				fileAdd += arr[arr.length-1];
 			};
-			opts.onAddModule = function(obj){
+			opts.onAddModule = function(obj) {
 				idAdd += obj.id;
 			};
 			madge([__dirname + '/files/cjs/normal'], opts);
-			(fileAdd+idAdd).should.eql( "a.jsd.jsb.jsc.js" + "adsub/bsub/c" );
+			(fileAdd + idAdd).should.eql( "a.jsd.jsb.jsc.js" + "adsub/bsub/c" );
 		});
 	});
 
@@ -25,28 +25,27 @@ describe('Madge', function () {
 			var fileAdd = "";
 			var idAdd = "";
 			var opts = {};
-			opts.onParseFile = function(obj){
+			opts.onParseFile = function(obj) {
 				var arr = obj.filename.split(path.sep);
 				fileAdd += arr[arr.length-1];
 			};
-			opts.onAddModule = function(obj){
+			opts.onAddModule = function(obj) {
 				idAdd += obj.id;
 			};
 			opts.format = 'amd';
 			madge([__dirname + '/files/amd/ok'], opts);
-			(idAdd+fileAdd).should.eql( "adesub/bsub/c" + "a.jsd.jse.jsb.jsc.js" );
+			(idAdd + fileAdd).should.eql( "adesub/bsub/c" + "a.jsd.jse.jsb.jsc.js" );
 		});
 	});
 
 	describe('pluggable - scope', function () {
 		it('should add idAdd property to the returned madger', function () {
 			var opts = {};
-			opts.onAddModule = function(obj){
-				if(this.idAdd){
+			opts.onAddModule = function(obj) {
+				if (this.idAdd) {
 					this.idAdd += obj.id;
-				}
-				else{
-					this.idAdd = ""+obj.id;
+				} else {
+					this.idAdd = "" + obj.id;
 				}
 			};
 			var madger = madge([__dirname + '/files/cjs/normal'], opts);