Move socket check after window flags

These flags mean the user explicitly wants to open in an existing
instance so if the socket is down it should error rather than try to
spawn code-server normally.

Author: code-asher

src/node/cli.ts

@@ -762,11 +762,6 @@ export const shouldOpenInExistingInstance = async (
   const paths = getResolvedPathsFromArgs(args)
   const client = new EditorSessionManagerClient(sessionSocket)
 
-  // If we can't connect to the socket then there's no existing instance.
-  if (!(await client.canConnect())) {
-    return undefined
-  }
-
   // If these flags are set then assume the user is trying to open in an
   // existing instance since these flags have no effect otherwise.
   const openInFlagCount = ["reuse-window", "new-window"].reduce((prev, cur) => {
@@ -783,6 +778,9 @@ export const shouldOpenInExistingInstance = async (
   // 2. That a file or directory was passed.
   // 3. That the socket is active.
   if (Object.keys(args).length === 1 && typeof args._ !== "undefined" && args._.length > 0) {
+    if (!(await client.canConnect())) {
+      return undefined
+    }
     const socketPath = await client.getConnectedSocketPath(paths[0])
     if (socketPath) {
       logger.debug("Found existing code-server socket")