launch-editor icon indicating copy to clipboard operation
launch-editor copied to clipboard

Published 20 hours ago • verified publisher icon yyx990803

fix: strict exact match of macOS process

Open chika3742 opened this issue 1 year ago • 1 comments

This PR fixes a problem in which the search is not strictly an exact match in the part that searches for processes with an exact match.

In case running processes includes /Users/foo/Applications/IntelliJ IDEA.app/Contents/MacOS/idea, the original code will match this as well, since COMMON_EDITORS_OSX contains /Applications/IntelliJ IDEA.app/Contents/MacOS/idea. This prevents finding editor installation not in /Applications, such as /Users/foo/Applications/**.

I addressed this by doing an exact match search on a \n separated list of output string.

This fixes #52.

chika3742 avatar Jan 08 '24 12:01 chika3742

Hey, I can confirm I tested that fix locally and it works. @sodatea would it be possible to merge that?

greg2451 avatar Apr 15 '24 12:04 greg2451