fix: remove "stream" dependency in browser bundles, #396

rollup.config.ts

@@ -14,6 +14,16 @@ const treeshake = {
   propertyReadSideEffects: false
 }
 const input = './src/liquid.ts'
+const replaceFS = {
+  include: './src/liquid-options.ts',
+  delimiters: ['', ''],
+  './fs/node': './fs/browser'
+}
+const replaceStream = {
+  include: './src/render/render.ts',
+  delimiters: ['', ''],
+  '../emitters/streamed-emitter': '../emitters/streamed-emitter-browser'
+}
 
 const nodeCjs = {
   output: [{
@@ -65,11 +75,8 @@ const browserEsm = {
   }],
   external: ['path', 'fs'],
   plugins: [
-    replace({
+    replace(replaceFS),
-      include: './src/liquid-options.ts',
+    replace(replaceStream),
-      delimiters: ['', ''],
-      './fs/node': './fs/browser'
-    }),
     typescript({
       tsconfigOverride: {
         include: [ 'src' ],
@@ -94,11 +101,8 @@ const browserUmd = {
     banner
   }],
   plugins: [
-    replace({
+    replace(replaceFS),
-      include: './src/liquid-options.ts',
+    replace(replaceStream),
-      delimiters: ['', ''],
-      './fs/node': './fs/browser'
-    }),
     typescript({
       tsconfigOverride: {
         include: [ 'src' ],
@@ -122,11 +126,8 @@ const browserMin = {
     sourcemap
   }],
   plugins: [
-    replace({
+    replace(replaceFS),
-      include: './src/liquid-options.ts',
+    replace(replaceStream),
-      delimiters: ['', ''],
-      './fs/node': './fs/browser'
-    }),
     typescript({
       tsconfigOverride: {
         include: [ 'src' ],

src/emitters/streamed-emitter-browser.ts

@@ -0,0 +1,12 @@
+import { Emitter } from './emitter'
+
+export class StreamedEmitter implements Emitter {
+  public buffer = '';
+  public stream: NodeJS.ReadableStream = null as any
+  constructor () {
+    throw new Error('streaming not supported in browser')
+  }
+  public write: (html: any) => void
+  public error: (err: Error) => void
+  public end: () => void
+}