feat(x): status message field

Signed-off-by: Johannes Großmann <grossmann.johannes@t-online.de>

Author: joe0BAB

client/client.go

@@ -251,12 +251,13 @@ func (c client) ListPlugins(ctx context.Context) ([]PluginInfo, error) {
 			continue
 		}
 		info := PluginInfo{
-			Name:         name,
-			Version:      version,
-			Disabled:     item.GetDisabled(),
-			External:     item.GetExternal(),
-			Configurable: item.GetConfigurable(),
-			RunStatus:    item.GetRunStatus(),
+			Name:          name,
+			Version:       version,
+			Disabled:      item.GetDisabled(),
+			External:      item.GetExternal(),
+			Configurable:  item.GetConfigurable(),
+			RunStatus:     item.GetRunStatus(),
+			StatusMessage: item.GetStatusMessage(),
 		}
 		if sp := item.GetSecretsProvider(); sp != nil {
 			pattern, err := secrets.ParsePattern(sp.GetPattern())
@@ -297,6 +298,7 @@ type PluginInfo struct {
 	External        bool
 	Configurable    bool
 	RunStatus       pluginsv1.RunStatus
+	StatusMessage   string
 	SecretsProvider *SecretsProviderMetadata
 }
 

client/client_test.go

@@ -78,12 +78,13 @@ func (m mockPluginsList) ListPlugins(_ context.Context, _ *connect.Request[plugi
 			version = plugin.Version.String()
 		}
 		b := pluginsv1.Plugin_builder{
-			Name:         proto.String(name),
-			Version:      proto.String(version),
-			Disabled:     proto.Bool(plugin.Disabled),
-			External:     proto.Bool(plugin.External),
-			Configurable: proto.Bool(plugin.Configurable),
-			RunStatus:    plugin.RunStatus.Enum(),
+			Name:          proto.String(name),
+			Version:       proto.String(version),
+			Disabled:      proto.Bool(plugin.Disabled),
+			External:      proto.Bool(plugin.External),
+			Configurable:  proto.Bool(plugin.Configurable),
+			RunStatus:     plugin.RunStatus.Enum(),
+			StatusMessage: proto.String(plugin.StatusMessage),
 		}
 		if plugin.SecretsProvider != nil {
 			b.SecretsProvider = pluginsv1.SecretsProvider_builder{
@@ -165,6 +166,7 @@ func Test_ListPlugins(t *testing.T) {
 				SecretsProvider: &SecretsProviderMetadata{Pattern: secrets.MustParsePattern("**")},
 				External:        true,
 				RunStatus:       pluginsv1.RunStatus_RUN_STATUS_CRASHED,
+				StatusMessage:   "exit status 1: connection refused",
 			},
 		}
 		socket := mockListPluginsEngine(t, plugins)

x/api/plugins/v1/api.pb.go

@@ -50,9 +50,13 @@ message Plugin {
   bool configurable = 5;
   // Run status
   RunStatus run_status = 6;
+  // Human-readable detail for the current run_status. Populated for
+  // terminal/error statuses (CRASHED, RETRIES_EXCEEDED); typically empty
+  // otherwise. Not machine-parseable — intended for display only.
+  string status_message = 7;
   // Type-specific fields
   oneof metadata {
-    SecretsProvider secrets_provider = 7;
+    SecretsProvider secrets_provider = 8;
   }
 }