From c146457de423570474da590e6b50274959092f55 Mon Sep 17 00:00:00 2001
From: Asher <ash@coder.com>
Date: Mon, 2 Mar 2020 11:52:39 -0600
Subject: [PATCH] Add recents section

---
 src/browser/pages/home.html | 10 +++++++++
 src/common/api.ts           |  1 +
 src/node/app/api.ts         | 44 +++++++++++++++++++++----------------
 src/node/app/app.ts         | 18 ++++++++++++++-
 4 files changed, 53 insertions(+), 20 deletions(-)

diff --git a/src/browser/pages/home.html b/src/browser/pages/home.html
index 139963adc..d2bc92f7f 100644
--- a/src/browser/pages/home.html
+++ b/src/browser/pages/home.html
@@ -30,6 +30,16 @@
         </div>
       </div>
 
+      <div class="card-box">
+        <div class="header">
+          <h2 class="main">Recent</h2>
+          <div class="sub">Choose a recent directory or workspace to launch below.</div>
+        </div>
+        <div class="content">
+          {{APP_LIST:RECENT_PROJECTS}}
+        </div>
+      </div>
+
       <div class="card-box">
         <div class="header">
           <h2 class="main">Editors</h2>
diff --git a/src/common/api.ts b/src/common/api.ts
index 6f6784f98..a338bb3c9 100644
--- a/src/common/api.ts
+++ b/src/common/api.ts
@@ -34,6 +34,7 @@ export interface SessionResponse {
 
 export interface RecentResponse {
   readonly paths: string[]
+  readonly workspaces: string[]
 }
 
 export interface RunningResponse {
diff --git a/src/node/app/api.ts b/src/node/app/api.ts
index b5f2e9381..f3a866804 100644
--- a/src/node/app/api.ts
+++ b/src/node/app/api.ts
@@ -28,7 +28,7 @@ interface ServerSession {
 }
 
 interface VsRecents {
-  [key: string]: (string | object)[]
+  [key: string]: (string | { configURIPath: string })[]
 }
 
 type VsSettings = [string, string][]
@@ -322,28 +322,34 @@ export class ApiHttpProvider extends HttpProvider {
         throw new Error("settings appear malformed")
       }
 
-      const paths: { [key: string]: Promise<string> } = {}
+      const pathPromises: { [key: string]: Promise<string> } = {}
+      const workspacePromises: { [key: string]: Promise<string> } = {}
       Object.values(JSON.parse(setting[1]) as VsRecents).forEach((recents) => {
-        recents
-          .filter((recent) => typeof recent === "string")
-          .forEach((recent) => {
-            try {
-              const pathname = url.parse(recent as string).pathname
-              if (pathname && !paths[pathname]) {
-                paths[pathname] = new Promise<string>((resolve) => {
-                  fs.stat(pathname)
-                    .then(() => resolve(pathname))
-                    .catch(() => resolve())
-                })
-              }
-            } catch (error) {
-              logger.debug("invalid path", field("path", recent))
+        recents.forEach((recent) => {
+          try {
+            const target = typeof recent === "string" ? pathPromises : workspacePromises
+            const pathname = url.parse(typeof recent === "string" ? recent : recent.configURIPath).pathname
+            if (pathname && !target[pathname]) {
+              target[pathname] = new Promise<string>((resolve) => {
+                fs.stat(pathname)
+                  .then(() => resolve(pathname))
+                  .catch(() => resolve())
+              })
             }
-          })
+          } catch (error) {
+            logger.debug("invalid path", field("path", recent))
+          }
+        })
       })
 
+      const [paths, workspaces] = await Promise.all([
+        Promise.all(Object.values(pathPromises)),
+        Promise.all(Object.values(workspacePromises)),
+      ])
+
       return {
-        paths: await Promise.all(Object.values(paths)),
+        paths: paths.filter((p) => !!p),
+        workspaces: workspaces.filter((p) => !!p),
       }
     } catch (error) {
       if (error.code !== "ENOENT") {
@@ -351,7 +357,7 @@ export class ApiHttpProvider extends HttpProvider {
       }
     }
 
-    return { paths: [] }
+    return { paths: [], workspaces: [] }
   }
 
   /**
diff --git a/src/node/app/app.ts b/src/node/app/app.ts
index cd2111312..8f75cc2b8 100644
--- a/src/node/app/app.ts
+++ b/src/node/app/app.ts
@@ -1,6 +1,6 @@
 import * as http from "http"
 import * as querystring from "querystring"
-import { Application } from "../../common/api"
+import { Application, RecentResponse } from "../../common/api"
 import { HttpCode, HttpError } from "../../common/http"
 import { HttpProvider, HttpProviderOptions, HttpResponse, Route } from "../http"
 import { ApiHttpProvider } from "./api"
@@ -86,6 +86,7 @@ export class MainHttpProvider extends HttpProvider {
     response.content = response.content
       .replace(/{{UPDATE:NAME}}/, await this.getUpdate())
       .replace(/{{APP_LIST:RUNNING}}/, this.getAppRows(running.applications))
+      .replace(/{{APP_LIST:RECENT_PROJECTS}}/, this.getRecentProjectRows(await this.api.recent()))
       .replace(
         /{{APP_LIST:EDITORS}}/,
         this.getAppRows(apps.filter((app) => app.categories && app.categories.includes("Editor"))),
@@ -107,6 +108,21 @@ export class MainHttpProvider extends HttpProvider {
     return undefined
   }
 
+  private getRecentProjectRows(recents: RecentResponse): string {
+    return recents.paths.length > 0 || recents.workspaces.length > 0
+      ? recents.paths.map((recent) => this.getRecentProjectRow(recent)).join("\n") +
+          recents.workspaces.map((recent) => this.getRecentProjectRow(recent, true)).join("\n")
+      : `<div class="none">No recent directories or workspaces.</div>`
+  }
+
+  private getRecentProjectRow(recent: string, workspace?: boolean): string {
+    return `<div class="block-row">
+      <a class="item -row -link" href="./vscode/?${workspace ? "workspace" : "folder"}=${recent}">
+        <div class="name">${recent}${workspace ? " (workspace)" : ""}</div>
+      </a>
+    </div>`
+  }
+
   private getAppRows(apps: ReadonlyArray<Application>): string {
     return apps.length > 0
       ? apps.map((app) => this.getAppRow(app)).join("\n")