Skip to content
This repository was archived by the owner on Jun 24, 2024. It is now read-only.

Commit 9a8d15e

Browse files
committed
fix: update css writer to fix refresh issues
1 parent 788a1d1 commit 9a8d15e

File tree

3 files changed

+6
-28
lines changed

3 files changed

+6
-28
lines changed

lib/middleware.js

Lines changed: 2 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ const express = require('express');
77
const JsWriter = require('@asset-pipe/js-writer');
88
const CssWriter = require('@asset-pipe/css-writer');
99
const emits = require('emits');
10-
const { Transform } = require('stream');
1110

1211
module.exports = class Middleware extends EventEmitter {
1312
constructor(jsFiles = [], cssFiles = [], options = {}) {
@@ -27,7 +26,7 @@ module.exports = class Middleware extends EventEmitter {
2726
this.emits = emits;
2827
this.writers = {
2928
js: new JsWriter(jsFiles, this.options, true),
30-
css: new CssWriter(cssFiles, this.options),
29+
css: new CssWriter(cssFiles, true),
3130
};
3231

3332
this.writers.js.plugin(watchify);
@@ -43,16 +42,6 @@ module.exports = class Middleware extends EventEmitter {
4342
bundler.pipe(devnull());
4443
});
4544

46-
this.writers.css.on('update', () => {
47-
const bundler = this.writers.css;
48-
49-
bundler.on('error', e => {
50-
this.emit('error', e);
51-
});
52-
53-
bundler.pipe(devnull());
54-
});
55-
5645
// Proxy underlaying events
5746
this.writers.js.on('error', this.emits('error'));
5847
this.writers.js.on('update', this.emits('update'));
@@ -61,10 +50,6 @@ module.exports = class Middleware extends EventEmitter {
6150
this.writers.js.on('log', this.emits('log'));
6251

6352
this.writers.css.on('error', this.emits('error'));
64-
this.writers.css.on('update', this.emits('update'));
65-
this.writers.css.on('bytes', this.emits('bytes'));
66-
this.writers.css.on('time', this.emits('time'));
67-
this.writers.css.on('log', this.emits('log'));
6853

6954
// eslint-disable-next-line new-cap
7055
this.app = express.Router({
@@ -117,14 +102,7 @@ module.exports = class Middleware extends EventEmitter {
117102
css() {
118103
return (req, res, next) => {
119104
res.type('css');
120-
const bundler = this.writers.css.pipe(
121-
new Transform({
122-
objectMode: true,
123-
transform(chunk, enc, cb) {
124-
cb(null, chunk.content);
125-
},
126-
})
127-
);
105+
const bundler = this.writers.css.bundle();
128106

129107
bundler.on('error', cleanup);
130108

package-lock.json

Lines changed: 3 additions & 3 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 & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
"access": "public"
2727
},
2828
"dependencies": {
29-
"@asset-pipe/css-writer": "^1.0.0",
29+
"@asset-pipe/css-writer": "^2.0.0",
3030
"@asset-pipe/js-writer": "^1.0.0",
3131
"dev-null": "^0.1.1",
3232
"emits": "^3.0.0",

0 commit comments

Comments
 (0)