From 1fdbb01ceb8a6642e9d1c32a6ba90033e44d9687 Mon Sep 17 00:00:00 2001 From: Tom Butcher Date: Sun, 2 Aug 2026 22:25:14 +0100 Subject: [PATCH] Refactor Windows instance directory retrieval in deeplink handling - Updated the `getInstanceDir` function to use `os.homedir()` as a fallback for the local app data path, ensuring compatibility across different Windows environments. - Improved code clarity by consolidating the logic for determining the local application data directory. --- src/desktop/deeplink-ipc.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/desktop/deeplink-ipc.js b/src/desktop/deeplink-ipc.js index c6b4e2a..3ef2b54 100644 --- a/src/desktop/deeplink-ipc.js +++ b/src/desktop/deeplink-ipc.js @@ -6,6 +6,7 @@ import { writeFileSync } from 'node:fs' import net from 'node:net' +import os from 'node:os' import { join } from 'node:path' export const PROTOCOL_PREFIX = 'farmcontrol://' @@ -18,10 +19,9 @@ const PIPE_RETRY_DELAY_MS = 100 export function getInstanceDir() { if (process.platform === 'win32') { - const localAppData = process.env.LOCALAPPDATA || process.env.APPDATA - if (localAppData) { - return join(localAppData, 'com.tombutcher.farmcontrol', 'instance') - } + const localAppData = + process.env.LOCALAPPDATA || join(os.homedir(), 'AppData', 'Local') + return join(localAppData, 'com.tombutcher.farmcontrol', 'instance') } return join(process.env.TMPDIR || '/tmp', 'com.tombutcher.farmcontrol', 'instance')