throttle convert requests to 10Hz

Author: matthewbastien

package-lock.json

@@ -1397,6 +1397,7 @@
     "@types/chai-subset": "^1.3.5",
     "@types/glob": "^7.1.6",
     "@types/lcov-parse": "^1.0.2",
+    "@types/lodash.throttle": "^4.1.9",
     "@types/mocha": "^10.0.10",
     "@types/mock-fs": "^4.13.4",
     "@types/node": "^18.19.71",
@@ -1419,6 +1420,7 @@
     "esbuild": "^0.24.2",
     "eslint": "^8.57.0",
     "eslint-config-prettier": "^10.0.1",
+    "lodash.throttle": "^4.1.1",
     "mocha": "^10.8.2",
     "mock-fs": "^5.4.1",
     "node-pty": "^1.0.0",

package.json

@@ -19,6 +19,8 @@ import { RenderNode, WebviewContent, WebviewMessage } from "./webview/WebviewMes
 import { WorkspaceContext } from "../WorkspaceContext";
 import { DocCDocumentationRequest, DocCDocumentationResponse } from "../sourcekit-lsp/extensions";
 import { LSPErrorCodes, ResponseError } from "vscode-languageclient";
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+import throttle = require("lodash.throttle");
 
 export enum PreviewEditorConstant {
     VIEW_TYPE = "swift.previewDocumentationEditor",
@@ -178,73 +180,77 @@ export class DocumentationPreviewEditor implements vscode.Disposable {
         }
     }
 
-    private async convertDocumentation(textEditor: vscode.TextEditor): Promise<void> {
-        const document = textEditor.document;
-        if (
-            document.uri.scheme !== "file" ||
-            !["markdown", "tutorial", "swift"].includes(document.languageId)
-        ) {
-            this.postMessage({
-                type: "update-content",
-                content: {
-                    type: "error",
-                    errorMessage: PreviewEditorConstant.UNSUPPORTED_EDITOR_ERROR_MESSAGE,
-                },
-            });
-            return;
-        }
+    private convertDocumentation = throttle(
+        async (textEditor: vscode.TextEditor): Promise<void> => {
+            const document = textEditor.document;
+            if (
+                document.uri.scheme !== "file" ||
+                !["markdown", "tutorial", "swift"].includes(document.languageId)
+            ) {
+                this.postMessage({
+                    type: "update-content",
+                    content: {
+                        type: "error",
+                        errorMessage: PreviewEditorConstant.UNSUPPORTED_EDITOR_ERROR_MESSAGE,
+                    },
+                });
+                return;
+            }
 
-        try {
-            const response = await this.context.languageClientManager.useLanguageClient(
-                async (client): Promise<DocCDocumentationResponse> => {
-                    return await client.sendRequest(DocCDocumentationRequest.type, {
-                        textDocument: {
-                            uri: document.uri.toString(),
-                        },
-                        position: textEditor.selection.start,
-                    });
-                }
-            );
-            this.postMessage({
-                type: "update-content",
-                content: {
-                    type: "render-node",
-                    renderNode: this.parseRenderNode(response.renderNode),
-                },
-            });
-        } catch (error) {
-            // Update the preview editor to reflect what error occurred
-            let livePreviewErrorMessage = "An internal error occurred";
-            const baseLogErrorMessage = `SourceKit-LSP request "${DocCDocumentationRequest.method}" failed: `;
-            if (error instanceof ResponseError) {
-                if (error.code === LSPErrorCodes.RequestCancelled) {
-                    // We can safely ignore cancellations
-                    return undefined;
+            try {
+                const response = await this.context.languageClientManager.useLanguageClient(
+                    async (client): Promise<DocCDocumentationResponse> => {
+                        return await client.sendRequest(DocCDocumentationRequest.type, {
+                            textDocument: {
+                                uri: document.uri.toString(),
+                            },
+                            position: textEditor.selection.start,
+                        });
+                    }
+                );
+                this.postMessage({
+                    type: "update-content",
+                    content: {
+                        type: "render-node",
+                        renderNode: this.parseRenderNode(response.renderNode),
+                    },
+                });
+            } catch (error) {
+                // Update the preview editor to reflect what error occurred
+                let livePreviewErrorMessage = "An internal error occurred";
+                const baseLogErrorMessage = `SourceKit-LSP request "${DocCDocumentationRequest.method}" failed: `;
+                if (error instanceof ResponseError) {
+                    if (error.code === LSPErrorCodes.RequestCancelled) {
+                        // We can safely ignore cancellations
+                        return undefined;
+                    }
+                    switch (error.code) {
+                        case LSPErrorCodes.RequestFailed:
+                            // RequestFailed response errors can be shown to the user
+                            livePreviewErrorMessage = error.message;
+                            break;
+                        default:
+                            // We should log additional info for other response errors
+                            this.context.outputChannel.log(
+                                baseLogErrorMessage + JSON.stringify(error.toJson(), undefined, 2)
+                            );
+                            break;
+                    }
+                } else {
+                    this.context.outputChannel.log(baseLogErrorMessage + `${error}`);
                 }
-                switch (error.code) {
-                    case LSPErrorCodes.RequestFailed:
-                        // RequestFailed response errors can be shown to the user
-                        livePreviewErrorMessage = error.message;
-                        break;
-                    default:
-                        // We should log additional info for other response errors
-                        this.context.outputChannel.log(
-                            baseLogErrorMessage + JSON.stringify(error.toJson(), undefined, 2)
-                        );
-                        break;
-                }
-            } else {
-                this.context.outputChannel.log(baseLogErrorMessage + `${error}`);
+                this.postMessage({
+                    type: "update-content",
+                    content: {
+                        type: "error",
+                        errorMessage: livePreviewErrorMessage,
+                    },
+                });
             }
-            this.postMessage({
-                type: "update-content",
-                content: {
-                    type: "error",
-                    errorMessage: livePreviewErrorMessage,
-                },
-            });
-        }
-    }
+        },
+        100 /* 10 times per second */,
+        { trailing: true }
+    );
 
     private parseRenderNode(content: string): RenderNode {
         const renderNode: RenderNode = JSON.parse(content);