fix(host): Add Buffer to global scope in sandbox (fixes #141) (#143)

Author: rhysd

packages/integration-tests/src/factory.test.ts

@@ -38,6 +38,9 @@ describe('Plugin Factory (used by host)', () => {
     expect(
       await pluginObj.handleRequest('Global', 'function', ['loaded'])
     ).toEqual(true);
+    expect(
+      await pluginObj.handleRequest('Global', 'function', ['Buffer'])
+    ).not.toEqual(undefined);
     expect(
       await pluginObj.handleRequest('Global', 'function', ['process'])
     ).not.toContain(['chdir', 'exit']);

packages/neovim/src/host/factory.ts

@@ -99,6 +99,7 @@ export interface Sandbox {
   module: NodeModule;
   require: (p: string) => any;
   console: { [key in keyof Console]?: Function };
+  Buffer: typeof Buffer;
 }
 
 function createSandbox(filename: string): Sandbox {
@@ -151,6 +152,9 @@ function createSandbox(filename: string): Sandbox {
   sandbox.process.stdout = devNull;
   sandbox.process.stderr = devNull;
 
+  // Buffer is no longer an owned property of 'global' variable in Node.js v12 (#141)
+  sandbox.Buffer = global.Buffer;
+
   return sandbox;
 }