fix: correct configDir parameter for GetPluginsByCategory method

- Add configDir field to storeExtManager struct
- Update NewStoreExtManager and NewStoreExtManagerInstance to accept configDir parameter
- Fix cmd/server.go to pass o.configDir when creating storeExtManager
- Update all test calls to use proper configDir (../testing/testdata)
- Add AI plugin test data to extension.yaml for proper testing
- GetPluginsByCategory now uses correct configuration directory instead of hardcoded empty string
- All tests now pass with proper plugin discovery functionality

Author: KariHall619

cmd/server.go

@@ -261,7 +261,7 @@ func (o *serverOption) runE(cmd *cobra.Command, args []string) (err error) {
 	extDownloader := downloader.NewStoreDownloader()
 	extDownloader.WithRegistry(o.extensionRegistry)
 	extDownloader.WithTimeout(o.downloadTimeout)
-	storeExtMgr := server.NewStoreExtManager(o.execer)
+	storeExtMgr := server.NewStoreExtManager(o.execer, o.configDir)
 	storeExtMgr.WithDownloader(extDownloader)
 	remoteServer := server.NewRemoteServer(loader, remote.NewGRPCloaderFromStore(), secretServer, storeExtMgr, o.configDir, o.grpcMaxRecvMsgSize)
 	if stores, storeErr := remoteServer.GetStores(ctx, nil); storeErr == nil {

pkg/server/store_ext_manager.go

@@ -73,18 +73,20 @@ type storeExtManager struct {
 	processChan          chan fakeruntime.Process
 	stopSingal           chan struct{}
 	lock                 *sync.RWMutex
+	configDir            string
 }
 
 var ss *storeExtManager
 
-func NewStoreExtManager(execer fakeruntime.Execer) ExtManager {
+func NewStoreExtManager(execer fakeruntime.Execer, configDir string) ExtManager {
 	if ss == nil {
 		ss = &storeExtManager{
 			processChan: make(chan fakeruntime.Process),
 			stopSingal:  make(chan struct{}, 1),
 			lock:        &sync.RWMutex{},
 		}
 		ss.execer = execer
+		ss.configDir = configDir
 		ss.socketPrefix = "unix://"
 		ss.extStatusMap = map[string]bool{}
 		ss.processCollect()
@@ -93,13 +95,14 @@ func NewStoreExtManager(execer fakeruntime.Execer) ExtManager {
 	return ss
 }
 
-func NewStoreExtManagerInstance(execer fakeruntime.Execer) ExtManager {
+func NewStoreExtManagerInstance(execer fakeruntime.Execer, configDir string) ExtManager {
 	ss = &storeExtManager{
 		processChan: make(chan fakeruntime.Process),
 		stopSingal:  make(chan struct{}, 1),
 		lock:        &sync.RWMutex{},
 	}
 	ss.execer = execer
+	ss.configDir = configDir
 	ss.socketPrefix = "unix://"
 	ss.extStatusMap = map[string]bool{}
 	ss.processCollect()
@@ -271,12 +274,10 @@ func (n *nonDownloader) GetTargetFile() string {
 
 // GetPluginsByCategory returns plugins filtered by category (e.g., "ai")
 func (s *storeExtManager) GetPluginsByCategory(category string) ([]testing.StoreKind, error) {
-	storeFactory := testing.NewStoreFactory("") // Use default config directory
+	storeFactory := testing.NewStoreFactory(s.configDir)
 	allStoreKinds, err := storeFactory.GetStoreKinds()
 	if err != nil {
-		// In test environment or when extension.yaml doesn't exist, return empty list gracefully
-		serverLogger.Info("failed to get store kinds, returning empty list", "error", err.Error())
-		return []testing.StoreKind{}, nil
+		return nil, fmt.Errorf("failed to get store kinds: %w", err)
 	}
 
 	var filteredKinds []testing.StoreKind

pkg/server/store_ext_manager_test.go

@@ -28,15 +28,15 @@ func TestStoreExtManager(t *testing.T) {
 	t.Run("not found", func(t *testing.T) {
 		mgr := NewStoreExtManagerInstance(&fakeruntime.FakeExecer{
 			ExpectLookPathError: errors.New("not found"),
-		})
+		}, "../testing/testdata")
 		err := mgr.Start("fake", "")
 		assert.Error(t, err)
 	})
 
 	t.Run("exist executable file", func(t *testing.T) {
 		mgr := NewStoreExtManagerInstance(&fakeruntime.FakeExecer{
 			ExpectLookPath: "/usr/local/bin/go",
-		})
+		}, "../testing/testdata")
 		err := mgr.Start("go", "")
 		assert.NoError(t, err, err)
 
@@ -53,7 +53,7 @@ func TestStoreExtManager(t *testing.T) {
 func TestUnifiedPluginManagement(t *testing.T) {
 	mgr := NewStoreExtManagerInstance(&fakeruntime.FakeExecer{
 		ExpectLookPath: "/usr/local/bin/go",
-	})
+	}, "../testing/testdata")
 
 	t.Run("discover AI plugins via category filter", func(t *testing.T) {
 		// Test AI plugin discovery using the new unified method

pkg/testing/testdata/data/core/extension.yaml

@@ -3,3 +3,37 @@ items:
     dependencies:
       - name: atest-store-database
     link: https://github.com/LinuxSuRen/atest-ext-store-database
+    categories:
+      - database
+  - name: openai-gpt4
+    dependencies:
+      - name: openai-gpt4
+    link: https://github.com/LinuxSuRen/atest-ext-ai-openai
+    categories:
+      - ai
+    params:
+      - key: version
+        defaultValue: "1.0.0"
+        description: "Plugin version"
+      - key: capabilities
+        defaultValue: "sql-generation,code-analysis"
+        description: "AI capabilities"
+  - name: claude-ai
+    dependencies:
+      - name: claude-ai
+    link: https://github.com/LinuxSuRen/atest-ext-ai-claude
+    categories:
+      - ai
+    params:
+      - key: version
+        defaultValue: "1.0.0"
+        description: "Plugin version"
+      - key: capabilities
+        defaultValue: "natural-language,text-analysis"
+        description: "AI capabilities"
+  - name: web-scraper
+    dependencies:
+      - name: web-scraper
+    link: https://github.com/LinuxSuRen/atest-ext-web-scraper
+    categories:
+      - web