electron: restore previous workspace (#9995)

The commit fixes logic when determining whether to open a default
window or existing workspace on electron startup. The update now
correctly restores the workspace which would previously never restore
properly.

Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>

Author: vince-fugnitto

packages/core/src/browser/window/window-service.ts

@@ -26,9 +26,9 @@ export interface NewWindowOptions {
 export const WindowService = Symbol('WindowService');
 
 /**
- *  The window hash value that is used to spawn a new default window.
+ * The window hash value that is used to spawn a new default window.
  */
-export const DEFAULT_WINDOW_HASH: string = '#!empty';
+export const DEFAULT_WINDOW_HASH: string = '!empty';
 
 export interface WindowService {
 

packages/core/src/electron-main/electron-main-application.ts

@@ -290,7 +290,9 @@ export class ElectronMainApplication {
     }
 
     protected async handleMainCommand(params: ElectronMainExecutionParams, options: ElectronMainCommandOptions): Promise<void> {
-        if (options.file === undefined) {
+        if (params.secondInstance === false) {
+            await this.openWindowWithWorkspace(''); // restore previous workspace.
+        } else if (options.file === undefined) {
             await this.openDefaultWindow();
         } else {
             let workspacePath: string | undefined;

packages/workspace/src/browser/workspace-service.ts

@@ -130,7 +130,8 @@ export class WorkspaceService implements FrontendApplicationContribution {
     protected async doGetDefaultWorkspaceUri(): Promise<string | undefined> {
 
         // If an empty window is explicitly requested do not restore a previous workspace.
-        if (window.location.hash === DEFAULT_WINDOW_HASH) {
+        // Note: `window.location.hash` includes leading "#" if non-empty.
+        if (window.location.hash === `#${DEFAULT_WINDOW_HASH}`) {
             window.location.hash = '';
             return undefined;
         }
@@ -213,6 +214,7 @@ export class WorkspaceService implements FrontendApplicationContribution {
             this.setURLFragment('');
         }
         this.updateTitle();
+        await this.server.setMostRecentlyUsedWorkspace(this._workspace ? this._workspace.resource.toString() : '');
         await this.updateWorkspace();
     }
 

packages/workspace/src/node/default-workspace-server.ts

@@ -92,20 +92,9 @@ export class DefaultWorkspaceServer implements WorkspaceServer {
 
     async setMostRecentlyUsedWorkspace(uri: string): Promise<void> {
         this.root = new Deferred();
-        const listUri: string[] = [];
-        const oldListUri = await this.getRecentWorkspaces();
-        listUri.push(uri);
-        if (oldListUri) {
-            oldListUri.forEach(element => {
-                if (element !== uri && element.length > 0) {
-                    listUri.push(element);
-                }
-            });
-        }
         this.root.resolve(uri);
-        this.writeToUserHome({
-            recentRoots: listUri
-        });
+        const recentRoots = Array.from(new Set([uri, ...await this.getRecentWorkspaces()]));
+        this.writeToUserHome({ recentRoots });
     }
 
     async getRecentWorkspaces(): Promise<string[]> {