🤖 fix: resolve symlinks in runtime.resolvePath

Change-Id: Ib3ce2b2401e3ce8e0ec0e41579d26be3f79e62a9
Signed-off-by: Thomas Kosiewski <tk@coder.com>

Author: ThomasK33

src/node/runtime/LocalRuntime.ts

@@ -294,12 +294,18 @@ export class LocalRuntime implements Runtime {
     }
   }
 
-  resolvePath(filePath: string): Promise<string> {
+  async resolvePath(filePath: string): Promise<string> {
     // Expand tilde to actual home directory path
     const expanded = expandTilde(filePath);
+    const absolute = path.resolve(expanded);
 
-    // Resolve to absolute path (handles relative paths like "./foo")
-    return Promise.resolve(path.resolve(expanded));
+    try {
+      // Try to resolve symlinks (canonical path)
+      return await fsPromises.realpath(absolute);
+    } catch {
+      // If file doesn't exist or other error, return absolute path
+      return absolute;
+    }
   }
 
   normalizePath(targetPath: string, basePath: string): string {

src/node/runtime/SSHRuntime.ts

@@ -392,7 +392,8 @@ export class SSHRuntime implements Runtime {
     // Use shell to expand tildes on remote system
     // Bash will expand ~ automatically when we echo the unquoted variable
     // This works with BusyBox (doesn't require GNU coreutils)
-    const command = `bash -c 'p=${shescape.quote(filePath)}; echo $p'`;
+    // We use readlink -f to resolve symlinks if possible, falling back to just echo
+    const command = `bash -c 'p=${shescape.quote(filePath)}; readlink -f "$p" 2>/dev/null || echo "$p"'`;
     // Use 10 second timeout for path resolution to allow for slower SSH connections
     return this.execSSHCommand(command, 10000);
   }