status: unexport SubServerStatus & rename to subServer

Author: ViktorT-11

status/manager.go

@@ -9,36 +9,36 @@ import (
 	"github.com/lightninglabs/lightning-terminal/litrpc"
 )
 
-type SubServerOption func(status *SubServerStatus)
+type SubServerOption func(status *subServer)
 
 // WithIsReadyOverride is a functional option that can be used to set a call
 // back function that is used to check if a system is ready _iff_ the system
 // running status is not yet true. The call-back will be passed the request URI
 // along with any manual status that has been set for the subsystem.
 func WithIsReadyOverride(fn func(string, string) (bool, bool)) SubServerOption {
-	return func(status *SubServerStatus) {
+	return func(status *subServer) {
 		status.isReadyOverride = fn
 	}
 }
 
-// SubServerStatus represents the status of a sub-server.
-type SubServerStatus struct {
-	// Disabled is true if the sub-server is available in the LiT bundle but
+// subServer represents the status of a sub-server.
+type subServer struct {
+	// disabled is true if the sub-server is available in the LiT bundle but
 	// has explicitly been disabled by the user.
-	Disabled bool
+	disabled bool
 
-	// Running is true if the sub-server is enabled and has successfully
+	// running is true if the sub-server is enabled and has successfully
 	// been started.
-	Running bool
+	running bool
 
 	// manualStatus will be a non-empty string that details the current
 	// status of the sub-server. This status can be set to a unique status
 	// that only exists for the specific sub-server, and will be displayed
 	// to the user with the litrpc.SubServerStatus.
 	manualStatus string
 
-	// Err will be a non-empty string if the sub-server failed to start.
-	Err string
+	// err will be a non-empty string if the sub-server failed to start.
+	err string
 
 	// isReadyOverride is a call back that, when set and only if `running`
 	// is not yet true, will be used to determine if a system is ready for
@@ -50,10 +50,10 @@ type SubServerStatus struct {
 	isReadyOverride func(string, string) (bool, bool)
 }
 
-// newSubServerStatus constructs a new SubServerStatus.
-func newSubServerStatus(disabled bool) *SubServerStatus {
-	return &SubServerStatus{
-		Disabled: disabled,
+// newSubServer constructs a new subServer.
+func newSubServer(disabled bool) *subServer {
+	return &subServer{
+		disabled: disabled,
 	}
 }
 
@@ -63,14 +63,14 @@ func newSubServerStatus(disabled bool) *SubServerStatus {
 type Manager struct {
 	litrpc.UnimplementedStatusServer
 
-	subServers map[string]*SubServerStatus
+	subServers map[string]*subServer
 	mu         sync.RWMutex
 }
 
 // NewStatusManager constructs a new Manager.
 func NewStatusManager() *Manager {
 	return &Manager{
-		subServers: make(map[string]*SubServerStatus),
+		subServers: make(map[string]*subServer),
 	}
 }
 
@@ -88,7 +88,7 @@ func (s *Manager) IsSystemReady(name, req string) (bool, bool, error) {
 			"name %s has not yet been registered")
 	}
 
-	if server.Disabled {
+	if server.disabled {
 		return false, true, nil
 	}
 
@@ -100,8 +100,8 @@ func (s *Manager) IsSystemReady(name, req string) (bool, bool, error) {
 		)
 	}
 
-	if !handled || server.Running {
-		return server.Running, false, nil
+	if !handled || server.running {
+		return server.running, false, nil
 	}
 
 	return isReady, false, nil
@@ -120,9 +120,9 @@ func (s *Manager) SubServerStatus(_ context.Context,
 	resp := make(map[string]*litrpc.SubServerStatus, len(s.subServers))
 	for server, status := range s.subServers {
 		resp[server] = &litrpc.SubServerStatus{
-			Disabled: status.Disabled,
-			Running:  status.Running,
-			Error:    status.Err,
+			Disabled: status.disabled,
+			Running:  status.running,
+			Error:    status.err,
 			Status:   status.manualStatus,
 		}
 	}
@@ -155,7 +155,7 @@ func (s *Manager) RegisterAndEnableSubServer(name string,
 func (s *Manager) registerSubServerUnsafe(name string, disabled bool,
 	opts ...SubServerOption) {
 
-	ss := newSubServerStatus(disabled)
+	ss := newSubServer(disabled)
 	ss.manualStatus = "Registered"
 
 	for _, o := range opts {
@@ -194,7 +194,7 @@ func (s *Manager) SetEnabled(name string) {
 		return
 	}
 
-	ss.Disabled = false
+	ss.disabled = false
 }
 
 // SetRunning can be used to set the status of a sub-server as Running
@@ -213,8 +213,8 @@ func (s *Manager) SetRunning(name string) {
 		return
 	}
 
-	ss.Running = true
-	ss.Err = ""
+	ss.running = true
+	ss.err = ""
 	ss.manualStatus = "Running"
 }
 
@@ -234,8 +234,8 @@ func (s *Manager) SetStopped(name string) {
 		return
 	}
 
-	ss.Running = false
-	ss.Err = ""
+	ss.running = false
+	ss.err = ""
 	ss.manualStatus = "Stopped"
 }
 
@@ -260,7 +260,7 @@ func (s *Manager) SetErrored(name string, errStr string,
 	err := fmt.Sprintf(errStr, params...)
 	log.Errorf("could not start the %s sub-server: %s", name, err)
 
-	ss.Running = false
-	ss.Err = err
+	ss.running = false
+	ss.err = err
 	ss.manualStatus = "Errored"
 }