Let use encoding option and return stdout and stderr buffers.

Author: ygu

__tests__/execa_spec.re

@@ -21,6 +21,23 @@ let () =
           expect(Execa.commandSync("ls __tests__/execa_spec.re", ()))
           |> toEqual(res)
         );
+
+        let options = Execa.options(~shell=false, ~encoding=Js.Nullable.null, ());
+        let res = Execa.child_process_result_buffer(
+          ~command="ls __tests__/execa_spec.re",
+          ~exitCode=0,
+          ~stdout=Node.Buffer.fromString("__tests__/execa_spec.re"),
+          ~stderr=Node.Buffer.fromString(""),
+          ~isCanceled=false,
+          ~killed=false,
+          ~timedOut=false,
+          ~failed=false
+        );
+
+        test("#commandSyncBuffer works with options", () =>
+          expect(Execa.commandSyncBuffer("ls __tests__/execa_spec.re", ~options, ()))
+          |> toEqual(res)
+        );
       }
     ),
   );

src/Execa.re

@@ -10,6 +10,18 @@ type child_process_result = {
   failed: bool,
 };
 
+[@bs.deriving abstract]
+type child_process_result_buffer = {
+  command: string,
+  exitCode: int,
+  stdout: Node.Buffer.t,
+  stderr: Node.Buffer.t,
+  isCanceled: bool,
+  killed: bool,
+  timedOut: bool,
+  failed: bool,
+};
+
 [@bs.deriving abstract]
 type options = {
   shell: bool,
@@ -23,6 +35,9 @@ type options = {
   stdio: string,
   [@bs.optional]
   env: Js.Dict.t(string),
+  [@bs.optional]
+  encoding: Js.Nullable.t(string)
 };
 
 [@bs.module "execa"] external commandSync: string => ~options:options=? => unit => child_process_result = "commandSync";
+[@bs.module "execa"] external commandSyncBuffer: string => ~options:options=? => unit => child_process_result_buffer = "commandSync";