🤖 refactor: check SSH type once in openTerminal method

Store config.type === 'ssh' check in isSSH constant at the start
of openTerminal() to avoid multiple type checks throughout.

Generated with `cmux`

Author: ethanndickson

src/services/ipcMain.ts

@@ -1322,7 +1322,7 @@ export class IpcMain {
       if (terminal === "ghostty") {
         const cmd = "open";
         let args: string[];
-        if (config.type === "ssh" && sshArgs) {
+        if (isSSH && sshArgs) {
           // Ghostty: Use --command flag to run SSH
           // Build the full SSH command as a single string
           const sshCommand = ["ssh", ...sshArgs].join(" ");
@@ -1340,9 +1340,9 @@ export class IpcMain {
         child.unref();
       } else {
         // Terminal.app
-        const cmd = config.type === "ssh" ? "osascript" : "open";
+        const cmd = isSSH ? "osascript" : "open";
         let args: string[];
-        if (config.type === "ssh" && sshArgs) {
+        if (isSSH && sshArgs) {
           // Terminal.app: Use osascript with proper AppleScript structure
           // Properly escape single quotes in args before wrapping in quotes
           const sshCommand = `ssh ${sshArgs
@@ -1374,7 +1374,7 @@ export class IpcMain {
       // Windows
       const cmd = "cmd";
       let args: string[];
-      if (config.type === "ssh" && sshArgs) {
+      if (isSSH && sshArgs) {
         // Windows - use cmd to start ssh
         args = ["/c", "start", "cmd", "/K", "ssh", ...sshArgs];
       } else {
@@ -1392,7 +1392,7 @@ export class IpcMain {
       // Linux - try terminal emulators in order of preference
       let terminals: Array<{ cmd: string; args: string[]; cwd?: string }>;
 
-      if (config.type === "ssh" && sshArgs) {
+      if (isSSH && sshArgs) {
         // x-terminal-emulator is checked first as it respects user's system-wide preference
         terminals = [
           { cmd: "x-terminal-emulator", args: ["-e", "ssh", ...sshArgs] },