From 3322ae6e4a044cd84d1637078aa85cb033971e2c Mon Sep 17 00:00:00 2001 From: "blink-so[bot]" <211532188+blink-so[bot]@users.noreply.github.com> Date: Thu, 17 Jul 2025 23:43:10 +0000 Subject: [PATCH 1/3] Add confirmation prompt to workspace update action Adds a warning dialog with explicit consent before updating workspace. Includes warning text about workspace restart and potential data loss. Co-authored-by: kylecarbs <7122116+kylecarbs@users.noreply.github.com> --- src/commands.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/commands.ts b/src/commands.ts index d6734376..c2f3eb67 100644 --- a/src/commands.ts +++ b/src/commands.ts @@ -654,14 +654,15 @@ export class Commands { if (!this.workspace || !this.workspaceRestClient) { return; } - const action = await this.vscodeProposed.window.showInformationMessage( + const action = await this.vscodeProposed.window.showWarningMessage( "Update Workspace", { useCustom: true, modal: true, - detail: `Update ${this.workspace.owner_name}/${this.workspace.name} to the latest version?`, + detail: `Update ${this.workspace.owner_name}/${this.workspace.name} to the latest version?\n\nUpdating will restart your workspace and stop any running processes that may result in loss of unsaved data.`, }, "Update", + "Cancel", ); if (action === "Update") { await this.workspaceRestClient.updateWorkspaceVersion(this.workspace); From dfc10c18c03a1f2de455815379812e0eff4d955b Mon Sep 17 00:00:00 2001 From: "blink-so[bot]" <211532188+blink-so[bot]@users.noreply.github.com> Date: Fri, 18 Jul 2025 05:58:05 +0000 Subject: [PATCH 2/3] Update warning text to use 'unsaved work' instead of 'unsaved data' Co-authored-by: kylecarbs <7122116+kylecarbs@users.noreply.github.com> --- src/commands.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/commands.ts b/src/commands.ts index c2f3eb67..e2f9c2a9 100644 --- a/src/commands.ts +++ b/src/commands.ts @@ -659,7 +659,7 @@ export class Commands { { useCustom: true, modal: true, - detail: `Update ${this.workspace.owner_name}/${this.workspace.name} to the latest version?\n\nUpdating will restart your workspace and stop any running processes that may result in loss of unsaved data.`, + detail: `Update ${this.workspace.owner_name}/${this.workspace.name} to the latest version?\n\nUpdating will restart your workspace and stop any running processes that may result in loss of unsaved work.`, }, "Update", "Cancel", From 9432181e6c369ef68423b5b000030162310a64f3 Mon Sep 17 00:00:00 2001 From: "blink-so[bot]" <211532188+blink-so[bot]@users.noreply.github.com> Date: Tue, 22 Jul 2025 16:50:13 +0000 Subject: [PATCH 3/3] Update warning text to match coder/coder wording Use 'which stops' instead of 'and stop' to clarify that the restart causes the process stopping and potential data loss. Co-authored-by: kylecarbs <7122116+kylecarbs@users.noreply.github.com> --- src/commands.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/commands.ts b/src/commands.ts index e2f9c2a9..677265bf 100644 --- a/src/commands.ts +++ b/src/commands.ts @@ -659,7 +659,7 @@ export class Commands { { useCustom: true, modal: true, - detail: `Update ${this.workspace.owner_name}/${this.workspace.name} to the latest version?\n\nUpdating will restart your workspace and stop any running processes that may result in loss of unsaved work.`, + detail: `Update ${this.workspace.owner_name}/${this.workspace.name} to the latest version?\n\nUpdating will restart your workspace which stops any running processes and may result in the loss of unsaved work.`, }, "Update", "Cancel",