Implement last opened functionality

Fixes #4619

Author: code-asher

CHANGELOG.md

@@ -38,6 +38,8 @@ implementation (#4414).
   vscode-remote-resource endpoint still can.
 - OpenVSX has been made the default marketplace. However this means web
   extensions like Vim may be broken.
+- The last opened folder/workspace is no longer stored separately in the
+  settings file (we rely on the already-existing query object instead).
 
 ### Deprecated
 

src/node/routes/vscode.ts

@@ -4,6 +4,7 @@ import { WebsocketRequest } from "../../../typings/pluginapi"
 import { logError } from "../../common/util"
 import { isDevMode } from "../constants"
 import { toVsCodeArgs } from "../cli"
+import { settings } from "../settings"
 import { ensureAuthenticated, authenticated, redirect } from "../http"
 import { loadAMDModule, readCompilationStats } from "../util"
 import { Router as WsRouter } from "../wsRouter"
@@ -31,6 +32,32 @@ export class CodeServerRouteWrapper {
       })
     }
 
+    // Ew means the workspace was closed so clear the last folder/workspace.
+    const { query } = await settings.read()
+    if (req.query.ew) {
+      delete query.folder
+      delete query.workspace
+    }
+
+    // Redirect to the last folder/workspace if nothing else is opened.
+    if (
+      !req.query.folder &&
+      !req.query.workspace &&
+      (query.folder || query.workspace) &&
+      !req.args["ignore-last-opened"] // This flag disables this behavior.
+    ) {
+      return redirect(req, res, "", {
+        folder: query.folder,
+        workspace: query.workspace,
+      })
+    }
+
+    // Store the query parameters so we can use them on the next load.  This
+    // also allows users to create functionality around query parameters.
+    settings.write({
+      query: req.query,
+    })
+
     next()
   }