refactor: implement interface segregation for ExtManager

Addresses interface pollution by separating concerns between general extension
management and AI-specific plugin management using Interface Segregation Principle.

Changes:
- Split ExtManager into three focused interfaces:
  * ExtManager: handles general extension start/stop/download operations
  * AIPluginManager: manages AI plugin registration, discovery, health checks
  * CompositeManager: combines both interfaces for full functionality
- Updated constructor functions to return CompositeManager type
- Fixed test isolation issues by using NewStoreExtManagerInstance in tests
- All Go server tests and frontend tests (34/34) passing

This refactoring improves code maintainability, follows SOLID principles,
and maintains backward compatibility while enabling independent development
of AI and general extension features.

Author: KariHall619

.gitignore

@@ -29,3 +29,5 @@ e2e/server.srl
 e2e/server.key
 e2e/server.csr
 e2e/server.crt
+
+atest

cmd/server.go

@@ -438,7 +438,7 @@ func postRequestProxy(proxy string) func(w http.ResponseWriter, r *http.Request,
 	}
 }
 
-func startPlugins(storeExtMgr server.ExtManager, stores *server.Stores) (err error) {
+func startPlugins(storeExtMgr server.CompositeManager, stores *server.Stores) (err error) {
 	for _, store := range stores.Data {
 		if store.Disabled || store.Kind == nil {
 			continue

pkg/server/remote_server.go

@@ -76,7 +76,7 @@ type server struct {
 	loader             testing.Writer
 	storeWriterFactory testing.StoreWriterFactory
 	configDir          string
-	storeExtMgr        ExtManager
+	storeExtMgr        CompositeManager
 
 	secretServer SecretServiceServer
 
@@ -119,7 +119,7 @@ func (f *fakeSecretServer) UpdateSecret(ctx context.Context, in *Secret) (reply
 }
 
 // NewRemoteServer creates a remote server instance
-func NewRemoteServer(loader testing.Writer, storeWriterFactory testing.StoreWriterFactory, secretServer SecretServiceServer, storeExtMgr ExtManager, configDir string, grpcMaxRecvMsgSize int) RunnerServer {
+func NewRemoteServer(loader testing.Writer, storeWriterFactory testing.StoreWriterFactory, secretServer SecretServiceServer, storeExtMgr CompositeManager, configDir string, grpcMaxRecvMsgSize int) RunnerServer {
 	if secretServer == nil {
 		secretServer = &fakeSecretServer{}
 	}

pkg/server/store_ext_manager.go

@@ -61,18 +61,28 @@ type AIPluginHealth struct {
 	Metrics      map[string]string `json:"metrics,omitempty"`
 }
 
+// ExtManager handles general extension management (start, stop, download)
 type ExtManager interface {
 	Start(name, socket string) (err error)
 	StopAll() (err error)
 	WithDownloader(downloader.PlatformAwareOCIDownloader)
-	// AI Plugin Management
+}
+
+// AIPluginManager handles AI-specific plugin management
+type AIPluginManager interface {
 	DiscoverAIPlugins() ([]AIPluginInfo, error)
 	CheckAIPluginHealth(name string) (*AIPluginHealth, error)
 	GetAllAIPluginHealth() (map[string]*AIPluginHealth, error)
 	RegisterAIPlugin(info AIPluginInfo) error
 	UnregisterAIPlugin(name string) error
 }
 
+// CompositeManager combines both extension and AI plugin management
+type CompositeManager interface {
+	ExtManager
+	AIPluginManager
+}
+
 type storeExtManager struct {
 	execer               fakeruntime.Execer
 	ociDownloader        downloader.PlatformAwareOCIDownloader
@@ -93,7 +103,7 @@ type storeExtManager struct {
 
 var ss *storeExtManager
 
-func NewStoreExtManager(execer fakeruntime.Execer) ExtManager {
+func NewStoreExtManager(execer fakeruntime.Execer) CompositeManager {
 	if ss == nil {
 		ctx, cancel := context.WithCancel(context.Background())
 		ss = &storeExtManager{
@@ -117,7 +127,7 @@ func NewStoreExtManager(execer fakeruntime.Execer) ExtManager {
 	return ss
 }
 
-func NewStoreExtManagerInstance(execer fakeruntime.Execer) ExtManager {
+func NewStoreExtManagerInstance(execer fakeruntime.Execer) CompositeManager {
 	ctx, cancel := context.WithCancel(context.Background())
 	ss = &storeExtManager{
 		processChan: make(chan fakeruntime.Process),

pkg/server/store_ext_manager_test.go

@@ -26,7 +26,7 @@ import (
 
 func TestStoreExtManager(t *testing.T) {
 	t.Run("not found", func(t *testing.T) {
-		mgr := NewStoreExtManager(&fakeruntime.FakeExecer{
+		mgr := NewStoreExtManagerInstance(&fakeruntime.FakeExecer{
 			ExpectLookPathError: errors.New("not found"),
 		})
 		err := mgr.Start("fake", "")