cli-plugins/manager: Plugin.RunHook: improve error message

Currently, the error was a plain "exit status 1"; make the error
message more informative if we need it :)

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Author: thaJeztah

cli-plugins/manager/plugin.go

@@ -163,12 +163,16 @@ func (p *Plugin) RunHook(ctx context.Context, hookData HookPluginData) ([]byte,
 	pCmd := exec.CommandContext(ctx, p.Path, p.Name, metadata.HookSubcommandName, string(hDataBytes)) // #nosec G204 -- ignore "Subprocess launched with a potential tainted input or cmd arguments"
 	pCmd.Env = os.Environ()
 	pCmd.Env = append(pCmd.Env, metadata.ReexecEnvvar+"="+os.Args[0])
-	hookCmdOutput, err := pCmd.Output()
+
+	out, err := pCmd.Output()
 	if err != nil {
-		return nil, wrapAsPluginError(err, "failed to execute plugin hook subcommand")
+		var exitErr *exec.ExitError
+		if errors.As(err, &exitErr) {
+			return nil, wrapAsPluginError(err, "plugin hook subcommand exited unsuccessfully")
+		}
+		return nil, wrapAsPluginError(err, "failed to execute plugin hook subcommand: "+pCmd.String())
 	}
-
-	return hookCmdOutput, nil
+	return out, nil
 }
 
 // pluginNameFormat is used as part of errors for invalid plugin-names.