helix-cli icon indicating copy to clipboard operation
helix-cli copied to clipboard

fix: Give a warning when using Safari for CLI login

Open andreituicu opened this issue 11 months ago • 1 comments

Related Issues

#2498

Temporary Fix

Until Safari is supported, the suggestion from @shsteimer was to give a warning to developers, so they don't have to debug themselves.

Not sure how I feel about such warnings. On one hand I understand how they could be useful, but at the same time they require the developer to upgrade the CLI to see it.

andreituicu avatar Mar 03 '25 17:03 andreituicu

The message itself I think is fine.

I get the hesitation on such warnings, and open to the idea that the a temp fix like this isn't worth the trouble. Relatively small fraction of projects use site auth. Relatively small fraction of users use safari on desktop. It's entirely plausible this fix has an audience of one person (me). Given that, I'd be supportive of holding off and seeing if we get more reports of the problem before we do anything.

shsteimer avatar Mar 03 '25 18:03 shsteimer