making miniprofiler faster. v0.2.1

Author: goenning

README.md

@@ -28,20 +28,20 @@ var express = require('express')
   , app = express();
 
 app.set('view engine', 'pug');
-app.use(miniprofiler.profile());
+app.use(miniprofiler.express());
 
 app.get('/', function(req, res) {
   req.miniprofiler.step('Step 1', function() {
     req.miniprofiler.step('Step 2', function() {
-      res.render('home');
+      res.render('index');
     });
   });
 });
 
 app.listen(8080);
 ```
 
-`home.pug`
+`index.pug`
 
 ```javascript
 doctype html

lib/miniprofiler.js

@@ -10,17 +10,12 @@
  *  Guilherme Oenning, 2016 @goenning
  */
 var _ = require('underscore');
-var fs = require('fs');
 var os = require('os');
-var path = require('path');
 var qs = require('querystring');
 var url = require('url');
 var uuid = require('node-uuid');
 var debug = require('debug')('miniprofiler');
-
-_.templateSettings = {
-	interpolate: /\{(.+?)\}/g
-};
+var ui = require('./ui.js');
 
 var	ignoredPaths = [];
 var trivialDurationThresholdMilliseconds = 2.5;
@@ -66,7 +61,6 @@ var storage = function(id, json) {
 	return null;
 };
 
-var includesDir = path.join(path.dirname(module.filename), '../ui');
 var resourcePath = '/mini-profiler-resources/';
 var version = '';
 
@@ -116,7 +110,7 @@ function handleRequest(f, req, res) {
 }
 
 function assets(req, res, lastPathSegment, done) {
-	fs.readFile(path.join(includesDir, lastPathSegment), 'utf-8', function(err, data) {
+	ui.readFile(lastPathSegment, function(err, data) {
 		if (err) {
 			debug(err);
       done({
@@ -163,7 +157,7 @@ function results(req, res, lastPathSegment, done) {
     done({
       type: 'text/html; charset=utf-8',
       status: 200,
-      body: includes.share({
+      body: ui.share({
         name: json.Name,
         duration: json.DurationMilliseconds,
         path: resourcePath,
@@ -185,13 +179,8 @@ function results(req, res, lastPathSegment, done) {
 	});
 }
 
-var includes = {
-	partial: _.template(fs.readFileSync(path.join(includesDir, 'include.partial.html')).toString()),
-	share: _.template(fs.readFileSync(path.join(includesDir, 'share.html')).toString())
-};
-
 function include(id) {
-	return includes.partial({
+	return ui.partial({
 		path: resourcePath,
 		position: popupRenderPosition,
 		showChildren: popupShowTimeWithChildren,

lib/ui.js

@@ -0,0 +1,34 @@
+'use strict';
+var fs = require('fs');
+var path = require('path');
+var _ = require('underscore');
+
+_.templateSettings = {
+	interpolate: /\{(.+?)\}/g
+};
+
+var fileStore = { };
+var includesDir = path.join(path.dirname(module.filename), '../ui');
+
+var readFile = function(name, callback) {
+  if (fileStore[name]) {
+    callback(null, fileStore[name]);
+  } else {
+    fs.readFile(path.join(includesDir, name), 'utf-8', callback);
+  }
+};
+
+var templates = {
+	partial: _.template(fs.readFileSync(path.join(includesDir, 'include.partial.html')).toString()),
+	share: _.template(fs.readFileSync(path.join(includesDir, 'share.html')).toString())
+};
+
+var partial = function(options) {
+  return templates.partial(options);
+};
+
+var share = function(options) {
+  return templates.share(options);
+};
+
+module.exports = { readFile, partial, share };

package.json

@@ -1,6 +1,6 @@
 {
   "name": "miniprofiler",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "A simple but effective mini-profiler.",
   "main": "lib/miniprofiler.js",
   "scripts": {