fix: make sure onDidSaveTextDocument execute after `onWillSaveTextD…

…ocument`
vuejs · Apr 25, 2023 · 3ace934 · 3ace934
1 parent 86d427b
commit 3ace934
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/packages/vscode-vue/src/nodeClientMain.ts b/packages/vscode-vue/src/nodeClientMain.ts
@@ -26,12 +26,19 @@ export function activate(context: vscode.ExtensionContext) {
 		}
 	});
 
-	let start = Date.now();
+	let start: number | undefined;
 	vscode.workspace.onWillSaveTextDocument(() => {
 		start = Date.now();
 	});
 	vscode.workspace.onDidSaveTextDocument(async () => {
+
+		if (start === undefined) {
+			return;
+		}
+
 		const time = Date.now() - start;
+		start = undefined;
+
 		if (config.features.codeActions.enable && time > config.features.codeActions.savingTimeLimit) {
 			const result = await vscode.window.showInformationMessage(
 				`Saving time is too long. (${time} ms > ${config.features.codeActions.savingTimeLimit} ms), `,