[refactor] Use more explicit pathing that does not rely on reading the file system in lib/transports.js. Fixes #731.

Author: indexzero

lib/winston/transports.js

@@ -6,29 +6,24 @@
  *
  */
 
-var fs = require('fs'),
-    path = require('path'),
-    common = require('./common');
-
-var transports = exports;
+var path = require('path');
 
 //
 // Setup all transports as lazy-loaded getters.
 //
-fs.readdirSync(path.join(__dirname, 'transports')).forEach(function (file) {
-  var transport = file.replace('.js', ''),
-      name  = common.capitalize(transport);
-
-  if (transport === 'transport') {
-    return;
-  }
-  else if (~transport.indexOf('-')) {
-    name = transport.split('-').map(function (part) {
-      return common.capitalize(part);
-    }).join('');
-  }
+Object.defineProperties(
+  exports,
+  ['Console', 'File', 'Http', 'Memory']
+    .reduce(function (acc, name) {
+      acc[name] = {
+        configurable: true,
+        enumerable: true,
+        get: function () {
+          var fullpath = path.join(__dirname, 'transports', name);
+          return exports[name] = require(fullpath)[name];
+        }
+      };
 
-  transports.__defineGetter__(name, function () {
-    return require('./transports/' + transport)[name];
-  });
-});
+      return acc;
+    }, {})
+);