refactor(i18n): Agrupa claves de traducción de servicios AI (#47)

Author: thomas-vilte

internal/infrastructure/ai/gemini/commit_summarizer_service.go

@@ -54,7 +54,7 @@ func NewGeminiService(ctx context.Context, cfg *config.Config, trans *i18n.Trans
 		Backend: genai.BackendGeminiAPI,
 	})
 	if err != nil {
-		msg := trans.GetMessage("error_gemini_client", 0, map[string]interface{}{
+		msg := trans.GetMessage("ai_service.error_ai_client", 0, map[string]interface{}{
 			"Error": err,
 		})
 		return nil, fmt.Errorf("%s", msg)

internal/infrastructure/ai/gemini/pull_requests_summarizer_service.go

@@ -40,7 +40,7 @@ func NewGeminiPRSummarizer(ctx context.Context, cfg *config.Config, trans *i18n.
 		Backend: genai.BackendGeminiAPI,
 	})
 	if err != nil {
-		msg := trans.GetMessage("error_gemini_client", 0, map[string]interface{}{
+		msg := trans.GetMessage("ai_service.error_ai_client", 0, map[string]interface{}{
 			"Error": err,
 		})
 		return nil, fmt.Errorf("%s", msg)

internal/infrastructure/ai/gemini/release_generator.go

@@ -47,7 +47,7 @@ func NewReleaseNotesGenerator(ctx context.Context, cfg *config.Config, trans *i1
 		Backend: genai.BackendGeminiAPI,
 	})
 	if err != nil {
-		msg := trans.GetMessage("error_gemini_client", 0, map[string]interface{}{
+		msg := trans.GetMessage("ai_service.error_ai_client", 0, map[string]interface{}{
 			"Error": err,
 		})
 		return nil, fmt.Errorf("%s", msg)
@@ -75,14 +75,14 @@ func (g *ReleaseNotesGenerator) GenerateNotes(ctx context.Context, release *mode
 
 	resp, err := g.client.Models.GenerateContent(ctx, g.model, genai.Text(prompt), genConfig)
 	if err != nil {
-		msg := g.trans.GetMessage("error_generating_release_notes", 0, map[string]interface{}{
+		msg := g.trans.GetMessage("ai_service.error_generating_release_notes", 0, map[string]interface{}{
 			"Error": err,
 		})
 		return nil, fmt.Errorf("%s", msg)
 	}
 
 	if len(resp.Candidates) == 0 {
-		msg := g.trans.GetMessage("error_no_ai_response", 0, nil)
+		msg := g.trans.GetMessage("ai_service.error_no_ai_response", 0, nil)
 		return nil, fmt.Errorf("%s", msg)
 	}
 

internal/services/pull_request_service.go

@@ -33,7 +33,7 @@ func NewPRService(vcsClient ports.VCSClient, aiService ports.PRSummarizer, trans
 
 func (s *PRService) SummarizePR(ctx context.Context, prNumber int, progress func(string)) (models.PRSummary, error) {
 	if s.aiService == nil {
-		msg := s.trans.GetMessage("ai_missing_for_pr", 0, nil)
+		msg := s.trans.GetMessage("ai_missing.ai_missing_for_pr", 0, nil)
 		return models.PRSummary{}, fmt.Errorf("%s", msg)
 	}
 

internal/services/pull_request_service_test.go

@@ -162,17 +162,17 @@ func TestPRService_SummarizePR_UpdateError(t *testing.T) {
 }
 
 func TestPRService_SummarizePR_NilAIService(t *testing.T) {
-	ctx := context.Background()
 	trans, err := i18n.NewTranslations("es", "../i18n/locales")
 	require.NoError(t, err)
 	cfg := &config.Config{}
 
 	service := NewPRService(nil, nil, trans, cfg)
 
-	_, err = service.SummarizePR(ctx, 123, func(s string) {})
+	summary, err := service.SummarizePR(context.Background(), 1, func(msg string) {})
 
 	assert.Error(t, err)
-	assert.Contains(t, err.Error(), "ai_missing_for_pr")
+	assert.Contains(t, err.Error(), "La IA no está configurada")
+	assert.Empty(t, summary.Title)
 }
 
 func TestPRService_SummarizePR_WithRelatedIssues(t *testing.T) {