fix #966 by using fixed folders

apps/vscode/command_client/command_client.py

@@ -3,11 +3,11 @@
 import time
 from dataclasses import dataclass
 from pathlib import Path
-from tempfile import gettempdir
 from typing import Any
 from uuid import uuid4
 
-from talon import Context, Module, actions, speech_system
+from talon import Context, Module, actions, app, speech_system
+from talon_init import TALON_HOME
 
 # How old a request file needs to be before we declare it stale and are willing
 # to remove it
@@ -197,20 +197,34 @@ def run_command(
     return decoded_contents["returnValue"]
 
 
+command_server_directory = None
+
+
+# NB: See https://github.com/talonhub/community/issues/966 for why we do OS-specific temp dirs
+def get_platform_specific_communication_dir_path():
+    home_dir = Path(os.path.expanduser("~"))
+    if app.platform == "linux" or app.platform == "mac":
+        return (
+            Path(home_dir) / f".talon/.comms/{actions.user.command_server_directory()}"
+        )
+    elif app.platform == "windows":
+        # subprocess.run(["attrib","+H","myfile.txt"],check=True)
+        return Path(
+            home_dir
+            / f"\\AppData\\Roaming\\talon\\{actions.user.command_server_directory()}"
+        )
+
+
 def get_communication_dir_path():
     """Returns directory that is used by command-server for communication
 
     Returns:
         Path: The path to the communication dir
     """
-    suffix = ""
-
-    # NB: We don't suffix on Windows, because the temp dir is user-specific
-    # anyways
-    if hasattr(os, "getuid"):
-        suffix = f"-{os.getuid()}"
-
-    return Path(gettempdir()) / f"{actions.user.command_server_directory()}{suffix}"
+    global command_server_directory
+    if command_server_directory is None:
+        command_server_directory = get_platform_specific_communication_dir_path()
+    return command_server_directory
 
 
 def robust_unlink(path: Path):