You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
flock-php install: report a declined permission prompt with an exit status, not text the apps match #25
The desktop apps tell a dismissed password or permission prompt apart from a real failure by matching the English error text that flock-php install prints:
cmd/flock/main.go exits with status 1 for every error, so the text is the only signal. If the message is reworded or translated, all three setup wizards quietly treat a declined prompt as a failure.
How it was found
@sineld raised this in the review of #19, on setup.html. I didn't change it there, because the same check is in the macOS and Windows apps too.
Suggested fix
Give flock-php install a dedicated exit status when the prompt is declined, for example through an error type carrying an exit code that cmd/flock/main.go passes to os.Exit.
Switch the three apps to check that status instead of the text.
Keep the message as it is, for people reading the terminal.
I’d like to work on this. Plan: a dedicated exit status (4, since 3 is already used by worktree setup and the daemon) for a declined prompt in flock install/uninstall, the three apps checking it instead of the text, and the message unchanged. OK?
Activity
lukas-frey commentedon Nov 5, 2026
Thanks a lot. This issue can be worked on if anyone wants to tackle it 👍
mendez-11 commentedon Nov 7, 2026
I’d like to work on this. Plan: a dedicated exit status (4, since 3 is already used by worktree setup and the daemon) for a declined prompt in flock install/uninstall, the three apps checking it instead of the text, and the message unchanged. OK?
lukas-frey commentedon Nov 7, 2026
Sure, go ahead :)