fix: Git hook bug

--bug=1

Author: studyzy

.codei18n/mappings.json

@@ -35,7 +35,6 @@ go.work.sum
 bin/
 
 # Build artifacts
-codei18n
 codei18n.exe
 
 # macOS

.gitignore

@@ -10,21 +10,21 @@ import (
 	"github.com/studyzy/codei18n/internal/log"
 )
 
-// NewFromConfig 根据配置创建合适的翻译引擎实现。
+// NewFromConfig creates an appropriate translation engine implementation based on the configuration.
 //
-// 约定：
-//   - provider 为 "openai" 或 "llm" 时，使用基于 OpenAI 兼容协议的 LLM 翻译
-//   - provider 为 "ollama" 时，使用本地 Ollama 服务
-//   - provider 为 "mock" 时，使用测试用的 MockTranslator
-//   - provider 为 "google" 或 "deepl" 时，视为已废弃并返回错误
-//   - 若 provider 为空，则默认视为 "openai"
+// Conventions:
+//   - When provider is "openai" or "llm", use LLM translation based on OpenAI-compatible protocol
+//   - When provider is "ollama", use the local Ollama service
+//   - When provider is "mock", use the MockTranslator for testing
+//   - When provider is "google" or "deepl", it is considered deprecated and an error is returned
+//   - If provider is empty, it defaults to "openai"
 func NewFromConfig(cfg *config.Config) (core.Translator, error) {
 	provider := strings.ToLower(strings.TrimSpace(cfg.TranslationProvider))
 	if provider == "" {
 		provider = "openai"
 	}
 
-	// 兼容文档中可能出现的 llm-api 命名
+	// Handle compatibility with the llm-api naming that may appear in the documentation
 	if provider == "llm-api" {
 		provider = "openai"
 	}
@@ -42,7 +42,7 @@ func NewFromConfig(cfg *config.Config) (core.Translator, error) {
 
 		baseURL := os.Getenv("OPENAI_BASE_URL")
 		if cfg.TranslationConfig != nil {
-			// 不同大小写和命名的兼容
+			// Compatibility for different cases and naming conventions
 			if val, ok := cfg.TranslationConfig["baseUrl"]; ok && val != "" {
 				baseURL = val
 			} else if val, ok := cfg.TranslationConfig["BaseUrl"]; ok && val != "" {
@@ -63,7 +63,7 @@ func NewFromConfig(cfg *config.Config) (core.Translator, error) {
 			}
 		}
 
-		// DeepSeek 模型的默认 BaseURL 自动检测
+		// DeepSeek model default BaseURL auto-detection
 		if baseURL == "" && (model == "deepseek-chat" || model == "deepseek-coder") {
 			baseURL = "https://api.deepseek.com"
 			log.Info("自动检测到 DeepSeek 模型，设置 BaseURL 为 %s", baseURL)

adapters/translator/factory.go

@@ -6,7 +6,7 @@ import (
 	"github.com/studyzy/codei18n/core/config"
 )
 
-// 验证历史 provider（google/deepl）会被明确拒绝，并提示迁移路径。
+// Verification history providers (google/deepl) will be explicitly rejected with migration path instructions.
 func TestNewFromConfig_RejectsLegacyProviders(t *testing.T) {
 	cases := []string{"google", "deepl"}
 	for _, provider := range cases {
@@ -21,7 +21,7 @@ func TestNewFromConfig_RejectsLegacyProviders(t *testing.T) {
 	}
 }
 
-// 验证未知 provider 会被拒绝。
+// Verify that unknown providers are rejected.
 func TestNewFromConfig_UnknownProvider(t *testing.T) {
 	cfg := &config.Config{TranslationProvider: "unknown-provider"}
 	tr, err := NewFromConfig(cfg)

adapters/translator/factory_test.go

@@ -7,7 +7,7 @@ import (
 	"time"
 )
 
-// 单元测试：验证 NewOllamaTranslator 默认值行为（不依赖本地服务）。
+// Unit test: Verify the default value behavior of NewOllamaTranslator (without relying on local service).
 func TestNewOllamaTranslator_Defaults(t *testing.T) {
 	tr := NewOllamaTranslator("", "")
 	if tr.endpoint == "" {
@@ -21,16 +21,16 @@ func TestNewOllamaTranslator_Defaults(t *testing.T) {
 	}
 }
 
-// 集成测试（可选）：连接本地 Ollama 服务并尝试真实翻译。
+// Integration test (optional): Connect to local Ollama service and attempt real translation.
 //
-// 启用方式：
+// How to enable:
 //
 //	CODEI18N_OLLAMA_TEST=1 go test ./adapters/translator -run TestOllamaTranslator_Translate_Integration -v
 //
-// 可选环境变量：
+// Optional environment variables:
 //
-//	CODEI18N_OLLAMA_ENDPOINT  默认为 http://localhost:11434
-//	CODEI18N_OLLAMA_MODEL     默认为 qwen3:4b
+//	CODEI18N_OLLAMA_ENDPOINT  Defaults to http://localhost:11434
+//	CODEI18N_OLLAMA_MODEL     Defaults to qwen3:4b
 func TestOllamaTranslator_Translate_Integration(t *testing.T) {
 	if os.Getenv("CODEI18N_OLLAMA_TEST") == "" {
 		t.Skip("跳过 Ollama 集成测试：未设置 CODEI18N_OLLAMA_TEST")

adapters/translator/ollama_test.go

@@ -140,6 +140,9 @@ func processFile(file string, adapter core.LanguageAdapter, store *mapping.Store
 		newText     string
 	}
 	var replacements []replacement
+	
+	// Track comments that couldn't find translation
+	missingTranslations := make(map[string]bool)
 
 	lines := strings.Split(string(src), "\n")
 
@@ -214,6 +217,9 @@ func processFile(file string, adapter core.LanguageAdapter, store *mapping.Store
 								// Ensure the new ID has bilingual data
 								store.Set(newID, cfg.SourceLanguage, enText)
 								store.Set(newID, cfg.LocalLanguage, zhText)
+								// Delete the old ID (based on Chinese text) to keep mappings clean
+								store.Delete(id)
+								log.Info("Deleted old mapping ID: %s", id)
 							}
 
 							break
@@ -222,10 +228,11 @@ func processFile(file string, adapter core.LanguageAdapter, store *mapping.Store
 				}
 			}
 
-			if !found {
-				log.Warn("未找到注释的英文翻译: '%s'", normalizedCurrent)
-				// Return early to count this as missing
-			}
+		if !found {
+			log.Warn("未找到注释的英文翻译: '%s'", normalizedCurrent)
+			// Mark this comment as missing translation
+			missingTranslations[c.ID] = true
+		}
 		} else {
 			// Apply Mode (EN -> ZH)
 			// c.SourceText is EN
@@ -303,28 +310,8 @@ func processFile(file string, adapter core.LanguageAdapter, store *mapping.Store
 		return 0
 	}
 
-	// Count missing translations
-	// Only count as missing if:
-	// 1. The comment ID exists in mappings
-	// 2. But the target language translation is missing
-	missingCount := 0
-	for _, c := range comments {
-		if c.ID == "" {
-			c.ID = utils.GenerateCommentID(c)
-		}
-		
-		// Check if this comment ID exists in mappings
-		if transMap, exists := store.GetMapping().Comments[c.ID]; exists {
-			// ID exists, check if target language translation exists
-			if targetVal, hasTarget := transMap[convertTo]; !hasTarget || targetVal == "" {
-				// Target translation is missing
-				missingCount++
-				log.Warn("注释 ID %s 缺少 %s 翻译", c.ID, convertTo)
-			}
-		}
-		// If ID doesn't exist in mappings at all, it's a new comment
-		// We don't count it as "missing translation" because it hasn't been scanned yet
-	}
+	// Count missing translations based on what we tracked during conversion
+	missingCount := len(missingTranslations)
 
 	if err := os.WriteFile(file, []byte(newSrc), 0644); err != nil {
 		log.Error("写入文件 %s 失败: %v", file, err)

cmd/codei18n/convert.go

@@ -25,7 +25,7 @@ func init() {
 
 	initCmd.Flags().StringVar(&initSourceLang, "source-lang", "en", "源码语言")
 	initCmd.Flags().StringVar(&initTargetLang, "target-lang", "zh-CN", "本地目标语言")
-	// 默认使用基于 LLM 的远程翻译（OpenAI 兼容协议）
+	// Default to using LLM-based remote translation (OpenAI-compatible protocol)
 	initCmd.Flags().StringVar(&initProvider, "provider", "openai", "翻译提供商 (openai/llm/ollama)")
 }
 

cmd/codei18n/init.go

@@ -57,15 +57,15 @@ func runTranslate() {
 		cfg.BatchSize = translateBatchSize
 	}
 
-	// 如果通过命令行指定了模型，则覆盖配置中的 model
+	// If a model is specified via the command line, it overrides the model in the configuration
 	if translateModel != "" {
 		if cfg.TranslationConfig == nil {
 			cfg.TranslationConfig = make(map[string]string)
 		}
 		cfg.TranslationConfig["model"] = translateModel
 	}
 
-	// 2. Init Translator（通过统一工厂创建）
+	// 2. Init Translator (Created via unified factory)
 	trans, err := translator.NewFromConfig(cfg)
 	if err != nil {
 		log.Fatal("初始化翻译引擎失败: %v", err)

cmd/codei18n/translate.go

@@ -24,7 +24,7 @@ func DefaultConfig() *Config {
 		SourceLanguage:  "en",
 		LocalLanguage:   "zh-CN",
 		ExcludePatterns: []string{".git/**", "vendor/**", ".codei18n/**"},
-		// 默认使用基于 LLM 的远程翻译（OpenAI 兼容协议）
+		// Default to using LLM-based remote translation (OpenAI-compatible protocol)
 		TranslationProvider: "openai",
 		TranslationConfig:   make(map[string]string),
 		BatchSize:           10,

core/config/config.go

@@ -104,6 +104,14 @@ func (s *Store) Set(id, lang, text string) {
 	s.mapping.Comments[id][lang] = cleanText
 }
 
+// Delete removes a comment mapping by ID
+func (s *Store) Delete(id string) {
+	s.mu.Lock()
+	defer s.mu.Unlock()
+
+	delete(s.mapping.Comments, id)
+}
+
 // GetMapping returns the underlying mapping object (read-only copy recommended for complex ops)
 // For now returning pointer for simplicity in MVP
 func (s *Store) GetMapping() *domain.Mapping {