fix: do not dump file contents to string; closes #74

src/pandoc.ts

@@ -3,7 +3,7 @@ import childProcess from 'child_process'
 import { pandocDataDir, pandocPath } from './boot'
 import * as Pandoc from './pandoc-types'
 import * as rpng from './rpng'
-import { create, dump, load, VFile, write } from './vfile'
+import { create, load, VFile, write } from './vfile'
 
 export { InputFormat, OutputFormat } from './pandoc-types'
 
@@ -26,7 +26,7 @@ export async function parse(
   ensureFile: boolean = false
 ): Promise<stencila.Node> {
   const args = [`--from=${from}`, `--to=json`].concat(options)
-  let content = dump(file)
+  let content = file.contents
   if (!content || ensureFile) {
     if (ensureFile && !file.path) throw new Error('Must supply a file')
     args.push(`${file.path}`)
@@ -95,13 +95,14 @@ let unparsePromises: Promise<any>[] = []
 /**
  * Run the Pandoc binary
  */
-function run(input: string, args: string[]): Promise<string> {
+function run(input: string | Buffer, args: string[]): Promise<string> {
   args.push(`--data-dir=${pandocDataDir}`)
   if (process.env.DEBUG) {
     console.log(`Running ${pandocPath} with args:\n  ${args.join('\n  ')}`)
   }
   return new Promise((resolve, reject) => {
     const child = childProcess.spawn(pandocPath, args)
+
     let stdout = ''
     let stderr = ''
     child.stdout.on('data', data => {
@@ -121,8 +122,13 @@ function run(input: string, args: string[]): Promise<string> {
     child.on('error', err => {
       reject(err)
     })
-    child.stdin.write(input)
-    child.stdin.end()
+
+    if (input.length) {
+      child.stdin.write(input, err => {
+        if (err) return reject(err)
+        child.stdin.end()
+      })
+    }
   })
 }