Regression: macOS Shortcuts "New Terminal" broken since 1.3.0 #11520
Unanswered
XanderHess
asked this question in
Issue Triage
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Issue Description
When using the "New Terminal" App Intent in Shortcuts on macOS, anything but a simple command will fail running.
There are two issues here, the first appears to be due to the change in commit 471098d:
If the Command input has been used at any point in the element's lifetime and has been cleared again, it still puts quotes around an empty string and tries to run
''; exitthis fails withzsh: permission denied:in ZSH andbash: : command not foundin Bash.The other issue appears to be due to the change in 1ff0dd8:
The escaping of special characters is overzealous in this context and wil cause trying to run a command with parameters, for example
git status, to fail because the space between the actual command and the parameter(s) will have been escaped.Expected Behavior
If the Command input in the "New Terminal" intent is empty, the user's login shell should be launched and run until exited manually.
Specifying a command should allow for adding parameters to said command.
Actual Behavior
Inputting text into the Command input and then clearing it again will cause
''; exitto be run, resulting in an error.Specifying a command with parameters, e.g.
git statuswill cause an error because the space inbetween gets escaped and effectively running'git\ status'; exit, resulting in an error.Reproduction Steps
ls -alor input anything into Command input and clear the field againGhostty Logs
No response
Ghostty Version
OS Version Information
macOS Sequoia 15.7.4
(Linux only) Display Server
None
(Linux only) Desktop Environment/Window Manager
No response
Minimal Ghostty Configuration
Additional Relevant Configuration
No response
I acknowledge that:
```) on separate lines.Beta Was this translation helpful? Give feedback.
All reactions