From 1b35a674ea2de9d95d46e096e81e077bd248f01d Mon Sep 17 00:00:00 2001 From: Itxaka Date: Thu, 28 Oct 2021 11:42:56 +0200 Subject: [PATCH] Print plugin success messages + print plugin location on load (#267) * report plugin state if succeed We havbe a state field in the plugin response that its not being used for anything. This patch makes luet print the state reported from the plugin if its not empty as a way for plugins to report data on success to users. If the field is empty it will be ignored. Signed-off-by: Itxaka * Print plugin path This patch adds the plugin location to the printed plugin list for a more rich view of the loaded plugins Signed-off-by: Itxaka --- cmd/root.go | 2 +- pkg/api/core/bus/events.go | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/cmd/root.go b/cmd/root.go index a02a8d3c..da2788bc 100644 --- a/cmd/root.go +++ b/cmd/root.go @@ -103,7 +103,7 @@ To build a package, from a tree definition: if len(bus.Manager.Plugins) != 0 { util.DefaultContext.Info(":lollipop:Enabled plugins:") for _, p := range bus.Manager.Plugins { - util.DefaultContext.Info("\t:arrow_right:", p.Name) + util.DefaultContext.Info(fmt.Sprintf("\t:arrow_right: %s (at %s)", p.Name, p.Executable)) } } }, diff --git a/pkg/api/core/bus/events.go b/pkg/api/core/bus/events.go index 7ed92f92..4fb55b96 100644 --- a/pkg/api/core/bus/events.go +++ b/pkg/api/core/bus/events.go @@ -99,6 +99,12 @@ func (b *Bus) Initialize(ctx *types.Context, plugin ...string) { if r.Errored() { err := fmt.Sprintf("Plugin %s at %s had an error: %s", p.Name, p.Executable, r.Error) ctx.Fatal(err) + } else { + if r.State != "" { + message := fmt.Sprintf(":lollipop: Plugin %s at %s succeded, state reported:", p.Name, p.Executable) + ctx.Success(message) + ctx.Info(r.State) + } } }) }