Skip to content

Commit d44be8f

Browse files
realitykingrealityking
committed
Replace ejs templates with a simple js file
1 parent 37f048a commit d44be8f

File tree

5 files changed

+79
-161
lines changed

5 files changed

+79
-161
lines changed

‎package-lock.json‎

Lines changed: 8 additions & 81 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package.json‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,15 +30,13 @@
3030
"files": [
3131
"public",
3232
"lib",
33-
"src",
34-
"views"
33+
"src"
3534
],
3635
"dependencies": {
3736
"acorn": "^8.0.4",
3837
"acorn-walk": "^8.0.0",
3938
"chalk": "^4.1.0",
4039
"commander": "^6.2.0",
41-
"ejs": "^3.1.5",
4240
"express": "^4.17.1",
4341
"filesize": "^6.1.0",
4442
"gzip-size": "^6.0.0",

‎views/viewer.ejs‎ renamed to ‎src/template.js‎

Lines changed: 46 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,61 @@
1-
<!DOCTYPE html>
1+
/* eslint-disable max-len */
2+
const path = require('path');
3+
const fs = require('fs');
4+
5+
const _ = require('lodash');
6+
7+
const projectRoot = path.resolve(__dirname, '..');
8+
const assetsRoot = path.join(projectRoot, 'public');
9+
10+
exports.renderViewer = renderViewer;
11+
12+
/**
13+
* Escapes `<` characters in JSON to safely use it in `<script>` tag.
14+
*/
15+
function escapeJson(json) {
16+
return JSON.stringify(json).replace(/</gu, '\\u003c');
17+
}
18+
19+
function getAssetContent(filename) {
20+
const assetPath = path.join(assetsRoot, filename);
21+
22+
if (!assetPath.startsWith(assetsRoot)) {
23+
throw new Error(`"${filename}" is outside of the assets root`);
24+
}
25+
26+
return fs.readFileSync(assetPath, 'utf8');
27+
}
28+
29+
function getScript(filename, mode) {
30+
if (mode === 'static') {
31+
return `<!-- ${_.escape(filename)} -->
32+
<script>${getAssetContent(filename)}</script>`;
33+
} else {
34+
return `<script src="${_.escape(filename)}"></script>`;
35+
}
36+
}
37+
38+
function renderViewer({title, enableWebSocket, chartData, defaultSizes, mode} = {}) {
39+
return `<!DOCTYPE html>
240
<html>
341
<head>
442
<meta charset="UTF-8"/>
543
<meta name="viewport" content="width=device-width, initial-scale=1"/>
6-
<title><%= title %></title>
44+
<title>${_.escape(title)}</title>
745
<link rel="shortcut icon" href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEAAAABACAMAAACdt4HsAAABrVBMVEUAAAD///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////+O1foceMD///+J0/qK1Pr7/v8Xdr/9///W8P4UdL7L7P0Scr2r4Pyj3vwad8D5/f/2/f+55f3E6f34+/2H0/ojfMKpzOd0rNgQcb3F3O/j9f7c8v6g3Pz0/P/w+v/q+P7n9v6T1/uQ1vuE0vqLut/y+v+Z2fvt+f+15Pzv9fuc2/vR7v2V2Pvd6/bg9P7I6/285/2y4/yp3/zp8vk8i8kqgMT7/P31+fyv4vxGkcz6/P6/6P3j7vfS5PNnpNUxhcbO7f7F6v3O4vHK3/DA2u631Ouy0eqXweKJud5wqthfoNMMbLvY8f73+v2dxeR8sNtTmdDx9/zX6PSjyeaCtd1YnNGX2PuQveCGt95Nls42h8dLlM3F4vBtAAAAM3RSTlMAAyOx0/sKBvik8opWGBMOAe3l1snDm2E9LSb06eHcu5JpHbarfHZCN9CBb08zzkdNS0kYaptYAAAFV0lEQVRYw92X51/aYBDHHS2O2qqttVbrqNq9m+TJIAYIShBkWwqIiCgoWvfeq7Z2/s29hyQNyUcR7LveGwVyXy6XH8/9rqxglLfUPLxVduUor3h0rfp2TYvpivk37929TkG037hffoX0+peVtZQc1589rigVUdXS/ABSAyEmGIO/1XfvldSK8vs3OqB6u3m0nxmIrvgB0dj7rr7Y9IbuF68hnfFaiHA/sxqm0wciIG43P60qKv9WXWc1RXGh/mFESFABTSBi0sNAKzqet17eCtOb3kZIDwxEEU0oAIJGYxNBDhBND29e0rtXXbcpuPmED9IhEAAQ/AXEaF8EPmnrrKsv0LvWR3fg5sWDNAFZOgAgaKvZDogHNU9MFwnnYROkc56RD5CjAbQX9Ow4g7upCsvYu55aSI/Nj0H1akgKQEUM94dwK65hYRmFU9MIcH/fqJYOZYcnuJSU/waKDgTOEVaVKhwrTRP5XzgSpAITYzom7UvkhFX5VutmxeNnWDjjswTKTyfgluNDGbUpWissXhF3s7mlSml+czWkg3D0l1nNjGNjz3myOQOa1KM/jOS6ebdbAVTCi4gljHSFrviza7tOgRWcS0MOUX9zdNgag5w7rRqA44Lzw0hr1WqES36dFliSJFlh2rXIae3FFcDDgKdxrUIDePr8jGcSClV1u7A9xeN0ModY/pHMxmR1EzRh8TJiwqsHmKW0l4FCEZI+jHio+JdPPE9qwQtTRxku2D8sIeRL2LnxWSllANCQGOIiqVHAz2ye2JR0DcH+HoxDkaADLjgxjKQ+AwCX/g0+DNgdG0ukYCONAe+dbc2IAc6fwt1ARoDSezNHxV2Cmzwv3O6lDMV55edBGwGK9n1+x2F8EDfAGCxug8MhpsMEcTEAWf3rx2vZhe/LAmtIn/6apE6PN0ULKgywD9mmdxbmFl3OvD5AS5fW5zLbv/YHmcsBTjf/afDz3MaZTVCfAP9z6/Bw6ycv8EUBWJIn9zYcoAWWlW9+OzO3vkTy8H+RANLmdrpOuYWdZYEXpo+TlCJrW5EARb7fF+bWdqf3hhyZI1nWJQHgznErZhbjoEsWqi8dQNoE294aldzFurwSABL2XXMf9+H1VQGke9exw5P/AnA5Pv5ngMul7LOvO922iwACu8WkCwLCafvM4CeWPxfA8lNHcWZSoi8EwMAIciKX2Z4SWCMAa3snCZ/G4EA8D6CMLNFsGQhkkz/gQNEBbPCbWsxGUpYVu3z8IyNAknwJkfPMEhLyrdi5RTyUVACkw4GSFRNWJNEW+fgPGwHD8/JxnRuLabN4CGNRkAE23na2+VmEAUmrYymSGjMAYqH84YUIyzgzs3XC7gNgH36Vcc4zKY9o9fgPBXUAiHHwVboBHGLiX6Zcjp1f2wu4tvzZKo0ecPnDtQYDQvJXaBeNzce45Fp28ZQLrEZVuFqgBwOalArKXnW1UzlnSusQKJqKYNuz4tOnI6sZG4zanpemv+7ySU2jbA9h6uhcgpfy6G2PahirDZ6zvq6zDduMVFTKvzw8wgyEdelwY9in3XkEPs3osJuwRQ4qTkfzifndg9Gfc4pdsu82+tTnHZTBa2EAMrqr2t43pguc8tNm7JQVQ2S0ukj2d22dhXYP0/veWtwKrCkNoNimAN5+Xr/oLrxswKbVJjteWrX7eR63o4j9q0GxnaBdWgGA5VStpanIjQmEhV0/nVt5VOFUvix6awJhPcAaTEShgrG+iGyvb5a0Ndb1YGHFPEwoqAinoaykaID1o1pdPNu7XsnCKQ3R+hwWIIhGvORcJUBYXe3Xa3vq/mF/N9V13ugufMkfXn+KHsRD0B8AAAAASUVORK5CYII=" type="image/x-icon" />
846
947
<script>
10-
window.enableWebSocket = <%- escapeJson(enableWebSocket) %>;
48+
window.enableWebSocket = ${escapeJson(enableWebSocket)};
1149
</script>
12-
<%- include('script', { filename: 'viewer.js' }) %>
50+
${getScript('viewer.js', mode)}
1351
</head>
1452
1553
<body>
1654
<div id="app"></div>
1755
<script>
18-
window.chartData = <%- escapeJson(chartData) %>;
19-
window.defaultSizes = <%- escapeJson(defaultSizes) %>;
56+
window.chartData = ${escapeJson(chartData)};
57+
window.defaultSizes = ${escapeJson(defaultSizes)};
2058
</script>
2159
</body>
22-
</html>
60+
</html>`;
61+
}

‎src/viewer.js‎

Lines changed: 24 additions & 62 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,14 @@ const http = require('http');
55
const WebSocket = require('ws');
66
const _ = require('lodash');
77
const express = require('express');
8-
const ejs = require('ejs');
98
const {bold} = require('chalk');
109

1110
const Logger = require('./Logger');
1211
const analyzer = require('./analyzer');
1312
const {open} = require('./utils');
13+
const {renderViewer} = require('./template');
1414

1515
const projectRoot = path.resolve(__dirname, '..');
16-
const assetsRoot = path.join(projectRoot, 'public');
1716

1817
function resolveTitle(reportTitle) {
1918
if (typeof reportTitle === 'function') {
@@ -50,24 +49,18 @@ async function startServer(bundleStats, opts) {
5049
if (!chartData) return;
5150

5251
const app = express();
53-
54-
// Explicitly using our `ejs` dependency to render templates
55-
// Fixes #17
56-
app.engine('ejs', require('ejs').renderFile);
57-
app.set('view engine', 'ejs');
58-
app.set('views', `${projectRoot}/views`);
5952
app.use(express.static(`${projectRoot}/public`));
6053

61-
app.use('/', (req, res) => {
62-
res.render('viewer', {
54+
app.get('/', (req, res) => {
55+
res.writeHead(200, {'Content-Type': 'text/html'});
56+
const html = renderViewer({
6357
mode: 'server',
6458
title: resolveTitle(reportTitle),
65-
get chartData() { return chartData },
59+
chartData,
6660
defaultSizes,
67-
enableWebSocket: true,
68-
// Helpers
69-
escapeJson
61+
enableWebSocket: true
7062
});
63+
return res.end(html);
7164
});
7265

7366
const server = http.createServer(app);
@@ -140,42 +133,28 @@ async function generateReport(bundleStats, opts) {
140133
if (!chartData) return;
141134

142135
await new Promise((resolve, reject) => {
143-
ejs.renderFile(
144-
`${projectRoot}/views/viewer.ejs`,
145-
{
136+
try {
137+
const reportHtml = renderViewer({
146138
mode: 'static',
147139
title: resolveTitle(reportTitle),
148140
chartData,
149141
defaultSizes,
150-
enableWebSocket: false,
151-
// Helpers
152-
assetContent: getAssetContent,
153-
escapeJson
154-
},
155-
(err, reportHtml) => {
156-
try {
157-
if (err) {
158-
logger.error(err);
159-
reject(err);
160-
return;
161-
}
162-
163-
const reportFilepath = path.resolve(bundleDir || process.cwd(), reportFilename);
164-
165-
fs.mkdirSync(path.dirname(reportFilepath), {recursive: true});
166-
fs.writeFileSync(reportFilepath, reportHtml);
167-
168-
logger.info(`${bold('Webpack Bundle Analyzer')} saved report to ${bold(reportFilepath)}`);
169-
170-
if (openBrowser) {
171-
open(`file://${reportFilepath}`, logger);
172-
}
173-
resolve();
174-
} catch (e) {
175-
reject(e);
176-
}
142+
enableWebSocket: false
143+
});
144+
const reportFilepath = path.resolve(bundleDir || process.cwd(), reportFilename);
145+
146+
fs.mkdirSync(path.dirname(reportFilepath), {recursive: true});
147+
fs.writeFileSync(reportFilepath, reportHtml);
148+
149+
logger.info(`${bold('Webpack Bundle Analyzer')} saved report to ${bold(reportFilepath)}`);
150+
151+
if (openBrowser) {
152+
open(`file://${reportFilepath}`, logger);
177153
}
178-
);
154+
resolve();
155+
} catch (e) {
156+
reject(e);
157+
}
179158
});
180159
}
181160

@@ -192,23 +171,6 @@ async function generateJSONReport(bundleStats, opts) {
192171
logger.info(`${bold('Webpack Bundle Analyzer')} saved JSON report to ${bold(reportFilename)}`);
193172
}
194173

195-
function getAssetContent(filename) {
196-
const assetPath = path.join(assetsRoot, filename);
197-
198-
if (!assetPath.startsWith(assetsRoot)) {
199-
throw new Error(`"${filename}" is outside of the assets root`);
200-
}
201-
202-
return fs.readFileSync(assetPath, 'utf8');
203-
}
204-
205-
/**
206-
* Escapes `<` characters in JSON to safely use it in `<script>` tag.
207-
*/
208-
function escapeJson(json) {
209-
return JSON.stringify(json).replace(/</gu, '\\u003c');
210-
}
211-
212174
function getChartData(analyzerOpts, ...args) {
213175
let chartData;
214176
const {logger} = analyzerOpts;

‎views/script.ejs‎

Lines changed: 0 additions & 8 deletions
This file was deleted.

0 commit comments

Comments
 (0)