Merge pull request #5353 from dotty-staging/fix/5324

Fix #5324: Ask before switching workspaces

Author: Duhemm

vscode-dotty/src/extension.ts

@@ -76,8 +76,9 @@ export function activate(context: ExtensionContext) {
   } else if (!fs.existsSync(disableDottyIDEFile)) {
 
     if (!vscode.workspace.workspaceFolders) {
-      if (vscode.window.activeTextEditor) {
-        setWorkspaceAndReload(vscode.window.activeTextEditor.document)
+      const editor = vscode.window.activeTextEditor
+      if (editor && editor.document.uri.fsPath && editor.document.uri.fsPath.length > 0) {
+        setWorkspaceAndReload(editor.document)
       }
     } else {
       let configuredProject: Thenable<void> = Promise.resolve()
@@ -116,7 +117,16 @@ export function activate(context: ExtensionContext) {
 function setWorkspaceAndReload(document: vscode.TextDocument) {
   const documentPath = path.parse(document.uri.fsPath).dir
   const workspaceRoot = findWorkspaceRoot(documentPath) || documentPath
-  vscode.workspace.updateWorkspaceFolders(0, null, { uri: vscode.Uri.file(workspaceRoot) })
+
+  vscode.window.showInformationMessage(
+    `It looks like '${workspaceRoot}' is the root of your Scala workspace. ` +
+    'Would you like to open it?',
+    'Yes', 'No'
+  ).then((value: String | undefined) => {
+    if (value === 'Yes') {
+      vscode.workspace.updateWorkspaceFolders(0, null, { uri: vscode.Uri.file(workspaceRoot) })
+    }
+  })
 }
 
 /**