Skip to content

configurable debouncing delays #327

New issue
New issue
Open
Open
configurable debouncing delays#327
Labels
enhancement
@strager

Description

@strager
strager
opened on Dec 8, 2021

TypeScript's LSP server has an artificial 200 ms debounce/delay:

typescript-language-server/src/lsp-server.ts

Line 316 in bb1c105

readonly doRequestDiagnosticsDebounced = debounce(() => this.doRequestDiagnostics(), 200);

and an artificial 50 ms debounce/delay:

typescript-language-server/src/diagnostic-queue.ts

Lines 30 to 33 in bb1c105

protected readonly firePublishDiagnostics = debounce(() => {
const diagnostics = this.getDiagnostics();
this.publishDiagnostics({ uri: this.uri, diagnostics });
}, 50);

I think this hard-coded delay makes TypeScript's LSP server look bad when compared to other LSP servers: https://quick-lint-js.com/benchmarks/

I think the delay should be user-configurable (e.g. disable-able or a custom delay).

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancement

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions