Skip to content

Commit 5ccf1f2

Browse files
gnoviawanclaude
andcommitted
fix: use shell.path instead of shell.name when spawning terminal
When users select "PowerShell 7" from the shell dropdown, the terminal was spawning with Windows PowerShell 5 instead. This happened because the frontend passed shell.name ("pwsh") to the spawn function, but the backend's resolve_shell_path() couldn't find pwsh.exe in PATH and fell back to powershell.exe. By passing shell.path (the already-detected absolute path), we bypass PATH resolution entirely and use the correct shell executable. Fixes #59 Co-Authored-By: Claude <noreply@anthropic.com>
1 parent bde8122 commit 5ccf1f2

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/renderer/layouts/WorkspaceLayout.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -761,7 +761,7 @@ export default function WorkspaceLayout(): React.JSX.Element {
761761
handleCreateTerminalInPane(paneId)
762762
}}
763763
onNewTerminalWithShell={(paneId, shell) => {
764-
handleCreateTerminalInPane(paneId, shell.name)
764+
handleCreateTerminalInPane(paneId, shell.path)
765765
}}
766766
onCloseTerminal={handleCloseTerminal}
767767
onRenameTerminal={renameTerminal}

0 commit comments

Comments
 (0)