fix: remove redundant LookPath call in detectOpenSSHInfo

Signed-off-by: Bartek Mucha <muchzill4@gmail.com>

Author: muchzill4

pkg/sshutil/sshutil.go

@@ -374,14 +374,12 @@ func detectOpenSSHInfo(sshExe SSHExe) openSSHInfo {
 		exe    sshExecutable
 		stderr bytes.Buffer
 	)
-	// TODO: Fix this function to properly handle complex SSH commands like "kitten ssh"
-	// The current LookPath, os.Stat, and caching logic doesn't work well for multi-word commands
-	path, err := exec.LookPath(sshExe.Exe)
-	if err != nil {
-		logrus.Warnf("failed to find ssh executable: %v", err)
-	} else {
-		st, _ := os.Stat(path)
-		exe = sshExecutable{Path: path, Size: st.Size(), ModTime: st.ModTime()}
+	// Note: For SSH wrappers like "kitten ssh", os.Stat will check the wrapper
+	// executable (kitten) instead of the underlying ssh binary. This means
+	// cache invalidation won't work properly - ssh upgrades won't be detected
+	// since kitten's size/mtime won't change. This is probably acceptable.
+	if st, err := os.Stat(sshExe.Exe); err == nil {
+		exe = sshExecutable{Path: sshExe.Exe, Size: st.Size(), ModTime: st.ModTime()}
 		openSSHInfosRW.RLock()
 		info := openSSHInfos[exe]
 		openSSHInfosRW.RUnlock()