Skip to content

Conversation

@nathancahill
Copy link

Sketched out the structure that I was referring to in #21.

Adds two new packages: @openworkflow/dashboard for the dashboard SPA and @openworkflow/cli for the ow CLI.

Adds a method called .serve() to the OW client which dynamically imports the observability functionality, which in turn depends on @openworkflow/dashboard for the HTTP server. If the dashboard package isn't installed it prompts for installation.

The CLI package includes @openworkflow/dashboard as a dependency so calling ow serve or ow dev (most common approach) will work out of the box.

Test it out with: tsx packages/cli/bin/cli.ts from the root of the project. openworkflow.config.ts config included for demonstration but not for merging.

Some more notes in the README.md of the two packages.

cc/ @mariusflorescu

@nathancahill
Copy link
Author

@mariusflorescu Continuing the discussion here:

The CLI should have dashboard as a dependency so the user does not need to install the dashboard

See above:

The CLI package includes @openworkflow/dashboard as a dependency so calling ow serve or ow dev (most common approach) will work out of the box.

The error is only if calling the "internal" method without deps

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.

2 participants