refactor(cli): embed templates.yaml and simplify LoadTemplatesConfig function

Author: GenerQAQ

src/client/acontext-cli/internal/config/templates.go

@@ -1,9 +1,8 @@
 package config
 
 import (
+	_ "embed"
 	"fmt"
-	"os"
-	"path/filepath"
 
 	"gopkg.in/yaml.v3"
 )
@@ -26,37 +25,19 @@ type TemplatesConfig struct {
 	Presets   map[string][]Preset                  `yaml:"presets"`
 }
 
+//go:embed templates.yaml
+var templatesYAML string
+
 var cachedConfig *TemplatesConfig
 
-// LoadTemplatesConfig loads template configuration file
+// LoadTemplatesConfig loads template configuration from embedded file
 func LoadTemplatesConfig() (*TemplatesConfig, error) {
 	if cachedConfig != nil {
 		return cachedConfig, nil
 	}
 
-	// Get config file path
-	exePath, err := os.Executable()
-	if err != nil {
-		return nil, fmt.Errorf("failed to get executable path: %w", err)
-	}
-
-	exeDir := filepath.Dir(exePath)
-	configPath := filepath.Join(exeDir, "templates", "templates.yaml")
-
-	// If not found in executable directory, try development path
-	if _, err := os.Stat(configPath); os.IsNotExist(err) {
-		// Development environment
-		cwd, _ := os.Getwd()
-		configPath = filepath.Join(cwd, "templates", "templates.yaml")
-	}
-
-	data, err := os.ReadFile(configPath)
-	if err != nil {
-		return nil, fmt.Errorf("failed to read templates config: %w", err)
-	}
-
 	var config TemplatesConfig
-	if err := yaml.Unmarshal(data, &config); err != nil {
+	if err := yaml.Unmarshal([]byte(templatesYAML), &config); err != nil {
 		return nil, fmt.Errorf("failed to parse templates config: %w", err)
 	}
 

…nt/acontext-cli/templates/templates.yaml …ntext-cli/internal/config/templates.yamlsrc/client/acontext-cli/templates/templates.yaml renamed to src/client/acontext-cli/internal/config/templates.yaml src/client/acontext-cli/templates/templates.yaml renamed to src/client/acontext-cli/internal/config/templates.yaml

@@ -19,7 +19,8 @@ func TestLoadTemplatesConfig(t *testing.T) {
 		{
 			name: "load valid config from templates directory",
 			setup: func(t *testing.T) string {
-				// This will use the actual templates.yaml file
+				// Reset cached config for clean test
+				cachedConfig = nil
 				return ""
 			},
 			wantErr: false,
@@ -28,18 +29,16 @@ func TestLoadTemplatesConfig(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
+			tt.setup(t)
 			config, err := LoadTemplatesConfig()
 			if tt.wantErr {
 				assert.Error(t, err)
 				assert.Nil(t, config)
 			} else {
-				// If config file exists, should load successfully
-				// If not, error is acceptable
-				if err == nil {
-					assert.NotNil(t, config)
-					assert.NotEmpty(t, config.Templates)
-					assert.NotEmpty(t, config.Presets)
-				}
+				require.NoError(t, err)
+				assert.NotNil(t, config)
+				assert.NotEmpty(t, config.Templates)
+				assert.NotEmpty(t, config.Presets)
 			}
 		})
 	}