Skip to content

feat(gui): Add the jp gui command plugin - #1013

Open
JeanMertz wants to merge 1 commit into
pr/debug-app-toolsfrom
pr/gui
Open

feat(gui): Add the jp gui command plugin#1013
JeanMertz wants to merge 1 commit into
pr/debug-app-toolsfrom
pr/gui

Conversation

@JeanMertz

Copy link
Copy Markdown
Collaborator

Opening the macOS app at the workspace you are already working in meant
finding the bundle and passing it a path by hand. jp gui does it from
wherever jp itself is pointed.

The plugin resolves nothing. InitMessage carries a workspace root the
host has already settled, so jp gui, jp -w ../other gui and
jp gui . all arrive here answered; the plugin sends Ready, launches
the app at that root, and sends Exit. The app is found by bundle
identifier rather than by path, so it works wherever it is installed.

Run directly from a terminal it prints its usage and says it is a
plugin, rather than sitting silent waiting for a protocol message on a
stdin that will never carry one.

It joins non_jp_excludes alongside the other command plugins, so the
JP-only test scope stays JP-only.

Signed-off-by: Jean Mertz git@jeanmertz.com

Opening the macOS app at the workspace you are already working in meant
finding the bundle and passing it a path by hand. `jp gui` does it from
wherever `jp` itself is pointed.

The plugin resolves nothing. `InitMessage` carries a workspace root the
host has already settled, so `jp gui`, `jp -w ../other gui` and
`jp gui .` all arrive here answered; the plugin sends `Ready`, launches
the app at that root, and sends `Exit`. The app is found by bundle
identifier rather than by path, so it works wherever it is installed.

Run directly from a terminal it prints its usage and says it is a
plugin, rather than sitting silent waiting for a protocol message on a
stdin that will never carry one.

It joins `non_jp_excludes` alongside the other command plugins, so the
JP-only test scope stays JP-only.

Signed-off-by: Jean Mertz <git@jeanmertz.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant