fix type errors

Also ignore node_modules/ at all levels.

Author: justinmk

packages/decorators/src/plugin/autocmd.ts

@@ -12,14 +12,15 @@ export function Autocmd(name: string, options?: AutocmdOptions) {
       pattern: '',
     };
 
+    // @ts-expect-error changing `option: keyof …` to `option: string` causes other errors.
     ['pattern', 'eval'].forEach((option: keyof AutocmdOptions) => {
       if (options && typeof options[option] !== 'undefined') {
         (opts[option] as any) = options[option];
       }
     });
 
     const nameWithPattern = `${name}${
-      options.pattern ? `:${options.pattern}` : ''
+      options?.pattern ? `:${options.pattern}` : ''
     }`;
     Object.defineProperty(f, NVIM_METHOD_NAME, {
       value: `autocmd:${nameWithPattern}`,

packages/decorators/src/plugin/command.ts

@@ -12,6 +12,7 @@ export function Command(name: string, options?: CommandOptions) {
     const f = isMethod ? cls[methodName] : cls;
     const opts: CommandOptions = {};
 
+    // @ts-expect-error changing `option: keyof …` to `option: string` causes other errors.
     ['range', 'nargs', 'complete'].forEach((option: keyof CommandOptions) => {
       if (options && typeof options[option] !== 'undefined') {
         (opts[option] as any) = options[option];

packages/decorators/src/plugin/plugin.ts

@@ -30,7 +30,7 @@ export interface PluginWrapperConstructor {
 }
 function wrapper(cls: PluginWrapperConstructor, options?: PluginOptions): any {
   return class WrapperClass extends cls implements PluginWrapperInterface {
-    public nvim: Neovim;
+    public nvim!: Neovim;
 
     constructor(plugin: NvimPlugin) {
       super(plugin.nvim, plugin);
@@ -57,7 +57,11 @@ function wrapper(cls: PluginWrapperConstructor, options?: PluginOptions): any {
                 autoCmdOpts.eval = spec.opts.eval;
               }
 
-              plugin.registerAutocmd(spec.name, [this, method], autoCmdOpts);
+              plugin.registerAutocmd(
+                spec.name,
+                [this, method],
+                autoCmdOpts as any
+              );
               break;
             case 'command':
               const cmdOpts: CommandOptions = {
@@ -94,7 +98,7 @@ function wrapper(cls: PluginWrapperConstructor, options?: PluginOptions): any {
       });
     }
 
-    setApi(nvim: Neovim) {
+    override setApi(nvim: Neovim) {
       this.nvim = nvim;
     }
   };

packages/decorators/src/plugin/types.ts

@@ -5,7 +5,7 @@ export interface FunctionOptions {
 }
 
 export interface AutocmdOptions {
-  pattern: string;
+  pattern?: string;
   eval?: string;
   sync?: boolean;
 }

packages/neovim/src/api/Base.ts

@@ -6,7 +6,7 @@ import { Logger, getLogger } from '../utils/logger';
 import { VimValue } from '../types/VimValue';
 
 export interface BaseConstructorOptions {
-  transport?: Transport;
+  transport: Transport;
   logger?: Logger;
   data?: Buffer;
   metadata?: any;
@@ -28,13 +28,13 @@ const DO_REQUEST = Symbol('DO_REQUEST');
 export class BaseApi extends EventEmitter {
   protected transport: Transport;
 
-  protected _isReady: Promise<boolean>;
+  protected _isReady: Promise<boolean> = Promise.resolve(false);
 
-  protected prefix: string;
+  protected prefix!: string;
 
   public logger: Logger;
 
-  public data: Buffer | number;
+  public data?: Buffer | number;
 
   // Node Buffer
   protected client: any;
@@ -48,7 +48,7 @@ export class BaseApi extends EventEmitter {
   }: BaseConstructorOptions) {
     super();
 
-    this.setTransport(transport);
+    this.transport = transport;
     this.data = data;
     this.logger = logger || getLogger();
     this.client = client;
@@ -58,10 +58,6 @@ export class BaseApi extends EventEmitter {
     }
   }
 
-  protected setTransport(transport: Transport): void {
-    this.transport = transport;
-  }
-
   equals(other: BaseApi): boolean {
     try {
       return String(this.data) === String(other.data);

packages/neovim/src/api/Buffer.ts

@@ -32,7 +32,7 @@ export const DETACH = Symbol('detachBuffer');
 export const ATTACH = Symbol('attachBuffer');
 
 export class Buffer extends BaseApi {
-  public prefix: string = Metadata[ExtType.Buffer].prefix;
+  public override prefix: string = Metadata[ExtType.Buffer].prefix;
 
   public get isAttached(): boolean {
     return this.client.isAttached(this);
@@ -106,9 +106,9 @@ export class Buffer extends BaseApi {
       strictIndexing: true,
     }
   ) {
-    // TODO: Error checking
-    // if (typeof start === 'undefined' || typeof end === 'undefined') {
-    // }
+    if (_start === undefined || _end === undefined) {
+      throw new Error('start and end are required');
+    }
     const indexing =
       typeof strictIndexing === 'undefined' ? true : strictIndexing;
     const lines = typeof _lines === 'string' ? [_lines] : _lines;

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

@@ -1,6 +1,5 @@
 /* eslint-env jest */
 import * as path from 'node:path';
-import { Neovim } from './Neovim';
 import * as testUtil from '../testUtil';
 
 describe('Neovim API', () => {
@@ -14,14 +13,6 @@ describe('Neovim API', () => {
     testUtil.stopNvim();
   });
 
-  it('sets transport when initialized', () => {
-    const transport = {};
-    const spy = jest.spyOn(Neovim.prototype, 'setTransport');
-    const neovim = new Neovim({ transport });
-    expect(spy).toHaveBeenCalledWith(transport);
-    expect(neovim.transport).toBe(transport);
-  });
-
   describe('Normal API calls', () => {
     it('gets a list of buffers and switches buffers', async () => {
       const buffers = await nvim.buffers;

packages/neovim/src/api/Neovim.ts

@@ -128,7 +128,7 @@ export interface OpenWindowOptions {
  * Neovim API
  */
 export class Neovim extends BaseApi {
-  protected prefix = 'nvim_';
+  protected override prefix = 'nvim_';
 
   public Buffer = Buffer;
 

packages/neovim/src/api/Tabpage.ts

@@ -4,7 +4,7 @@ import { createChainableApi } from './utils/createChainableApi';
 import { Window, AsyncWindow } from './Window';
 
 export class Tabpage extends BaseApi {
-  public prefix: string = Metadata[ExtType.Tabpage].prefix;
+  public override prefix: string = Metadata[ExtType.Tabpage].prefix;
 
   /** Returns all windows of tabpage */
   get windows(): Promise<Window[]> {
@@ -30,12 +30,12 @@ export class Tabpage extends BaseApi {
   }
 
   /** Invalid */
-  getOption(): void {
+  override getOption(): void {
     this.logger.error('Tabpage does not have `getOption`');
   }
 
   /** Invalid */
-  setOption(): void {
+  override setOption(): void {
     this.logger.error('Tabpage does not have `setOption`');
   }
 }

packages/neovim/src/api/Window.ts

@@ -5,7 +5,7 @@ import { Tabpage, AsyncTabpage } from './Tabpage';
 import { Buffer, AsyncBuffer } from './Buffer';
 
 export class Window extends BaseApi {
-  public prefix: string = Metadata[ExtType.Window].prefix;
+  public override prefix: string = Metadata[ExtType.Window].prefix;
 
   /**
    * The windowid that not change within a Vim session