fix(api): executeLua return type is too narrow #353

fix #346

Author: justinmk

packages/neovim/src/api/Neovim.test.ts

@@ -86,7 +86,7 @@ describe('Neovim API', () => {
 
     it('can run lua', async () => {
       expect(
-        await nvim.lua('function test(a) return a end return test(...)', 1)
+        await nvim.lua('function test(a) return a end return test(...)', [1])
       ).toBe(1);
 
       expect(

packages/neovim/src/api/Neovim.ts

@@ -448,17 +448,17 @@ export class Neovim extends BaseApi {
   }
 
   /**
-   * Executes lua, it's possible neovim client does not support this
+   * Executes Lua code.
    */
-  lua(code: string, args: VimValue[] = []): Promise<object> {
+  lua(code: string, args: VimValue[] = []): Promise<VimValue> {
     const _args = Array.isArray(args) ? args : [args];
     return this.request(`${this.prefix}execute_lua`, [code, _args]);
   }
 
   /**
    * Alias for `lua()` to be consistent with neovim API
    */
-  executeLua(code: string, args: VimValue[] = []): Promise<object> {
+  executeLua(code: string, args: VimValue[] = []): ReturnType<typeof this.lua> {
     return this.lua(code, args);
   }