Merge branch 'master' into feature/issue-generator

Author: thomas-vilte

cmd/main.go

@@ -88,8 +88,13 @@ func initializeApp() (*cli.Command, error) {
 	var vcsClient = container.GetVCSRegistry()
 	vcsClientInstance, _ := vcsClient.CreateClientFromConfig(ctx, gitService, cfgApp, translations)
 	commitHandler := handler.NewSuggestionHandler(gitService, vcsClientInstance, translations)
+	aiSummarizer, err := container.GetPRSummarizer(ctx)
+	if err != nil {
+		log.Printf("Warning: no se pudo crear el servicio de IA para PRs: %v", err)
+		aiSummarizer = nil
+	}
 
-	prServiceFactory := factory.NewPrServiceFactory(cfgApp, translations, nil, gitService)
+	prServiceFactory := factory.NewPrServiceFactory(cfgApp, translations, aiSummarizer, gitService)
 	prCommand := pull_requests.NewSummarizeCommand(prServiceFactory)
 
 	registerCommand := registry.NewRegistry(cfgApp, translations)

internal/cli/command/handler/suggestions.go

@@ -47,10 +47,6 @@ func (h *SuggestionHandler) displaySuggestions(suggestions []models.CommitSugges
 		suggestionHeader := color.New(color.FgMagenta, color.Bold).Sprintf("📝 Sugerencia #%d", i+1)
 		fmt.Printf("%s\n\n", suggestionHeader)
 
-		if i == 0 && suggestion.Usage != nil {
-			ui.PrintTokenUsage(suggestion.Usage, h.t)
-		}
-
 		_, _ = sectionColor.Println(h.t.GetMessage("ui_labels.code_analysis", 0, nil))
 		printIndentedKeyValue(h.t.GetMessage("ui_labels.changes_overview", 0, nil), suggestion.CodeAnalysis.ChangesOverview)
 		printIndentedKeyValue(h.t.GetMessage("ui_labels.primary_purpose", 0, nil), suggestion.CodeAnalysis.PrimaryPurpose)
@@ -79,6 +75,11 @@ func (h *SuggestionHandler) displaySuggestions(suggestions []models.CommitSugges
 			h.displayTechnicalAnalysis(suggestion.RequirementsAnalysis)
 		}
 
+		if i == 0 && suggestion.Usage != nil {
+			fmt.Println()
+			ui.PrintTokenUsage(suggestion.Usage, h.t)
+		}
+
 		fmt.Printf("%s\n", separator)
 	}
 

internal/cli/command/pull_requests/summarize.go

@@ -64,6 +64,12 @@ func (c *SummarizeCommand) CreateCommand(t *i18n.Translations, _ *cfg.Config) *c
 				"Number": prNumber,
 				"Title":  summary.Title,
 			}))
+
+			if summary.Usage != nil {
+				fmt.Println()
+				ui.PrintTokenUsage(summary.Usage, t)
+			}
+
 			return nil
 		},
 	}

internal/cli/command/release/create.go

@@ -201,6 +201,11 @@ func createReleaseAction(releaseService ports.ReleaseService, trans *i18n.Transl
 			fmt.Println(trans.GetMessage("release.create_push_help", 0, nil))
 		}
 
+		if notes.Usage != nil {
+			fmt.Println()
+			ui.PrintTokenUsage(notes.Usage, trans)
+		}
+
 		fmt.Println()
 
 		return nil

internal/cli/command/release/generate.go

@@ -8,6 +8,7 @@ import (
 	"github.com/Tomas-vilte/MateCommit/internal/cli/completion_helper"
 	"github.com/Tomas-vilte/MateCommit/internal/domain/ports"
 	"github.com/Tomas-vilte/MateCommit/internal/i18n"
+	"github.com/Tomas-vilte/MateCommit/internal/ui"
 	"github.com/urfave/cli/v3"
 )
 
@@ -77,6 +78,12 @@ func generateReleaseAction(releaseService ports.ReleaseService, trans *i18n.Tran
 		fmt.Println(trans.GetMessage("release.version_label", 0, map[string]interface{}{
 			"Version": release.Version,
 		}))
+
+		if notes.Usage != nil {
+			fmt.Println()
+			ui.PrintTokenUsage(notes.Usage, trans)
+		}
+
 		fmt.Println()
 
 		return nil

internal/cli/command/release/preview.go

@@ -7,6 +7,7 @@ import (
 	"github.com/Tomas-vilte/MateCommit/internal/cli/completion_helper"
 	"github.com/Tomas-vilte/MateCommit/internal/domain/ports"
 	"github.com/Tomas-vilte/MateCommit/internal/i18n"
+	"github.com/Tomas-vilte/MateCommit/internal/ui"
 	"github.com/urfave/cli/v3"
 )
 
@@ -104,6 +105,11 @@ func previewReleaseAction(releaseService ports.ReleaseService, trans *i18n.Trans
 		fmt.Println(trans.GetMessage("release.next_steps_cmd", 0, nil))
 		fmt.Println()
 
+		if notes.Usage != nil {
+			ui.PrintTokenUsage(notes.Usage, trans)
+			fmt.Println()
+		}
+
 		return nil
 	}
 }

internal/cli/command/release/publish.go

@@ -7,6 +7,7 @@ import (
 	"github.com/Tomas-vilte/MateCommit/internal/cli/completion_helper"
 	"github.com/Tomas-vilte/MateCommit/internal/domain/ports"
 	"github.com/Tomas-vilte/MateCommit/internal/i18n"
+	"github.com/Tomas-vilte/MateCommit/internal/ui"
 	"github.com/urfave/cli/v3"
 )
 
@@ -94,6 +95,11 @@ func publishReleaseAction(releaseService ports.ReleaseService,
 			"Version": release.Version,
 		}))
 
+		if notes.Usage != nil {
+			fmt.Println()
+			ui.PrintTokenUsage(notes.Usage, trans)
+		}
+
 		return nil
 	}
 }

internal/infrastructure/di/container.go

@@ -118,6 +118,25 @@ func (c *Container) GetCommitService(ctx context.Context) (ports.CommitService,
 	return c.commitService, nil
 }
 
+// GetPRSummarizer retorna el servicio de IA para resumir PRs (lazy initialization)
+func (c *Container) GetPRSummarizer(ctx context.Context) (ports.PRSummarizer, error) {
+	if c.config.AIConfig.ActiveAI == "" {
+		return nil, fmt.Errorf("no hay IA activa configurada")
+	}
+
+	aiFactory, err := c.aiRegistry.Get(string(c.config.AIConfig.ActiveAI))
+	if err != nil {
+		return nil, fmt.Errorf("proveedor de IA '%s' no encontrado: %w", c.config.AIConfig.ActiveAI, err)
+	}
+
+	aiSummarizer, err := aiFactory.CreatePRSummarizer(ctx, c.config, c.translations)
+	if err != nil {
+		return nil, fmt.Errorf("error al crear el servicio de IA para PRs: %w", err)
+	}
+
+	return aiSummarizer, nil
+}
+
 // GetPRService retorna el servicio de PRs (lazy initialization)
 func (c *Container) GetPRService(ctx context.Context) (ports.PRService, error) {
 	if c.prService != nil {
@@ -129,15 +148,9 @@ func (c *Container) GetPRService(ctx context.Context) (ports.PRService, error) {
 	}
 
 	var aiSummarizer ports.PRSummarizer
-	if c.config.AIConfig.ActiveAI != "" {
-		aiFactory, err := c.aiRegistry.Get(string(c.config.AIConfig.ActiveAI))
-		if err == nil {
-			aiSummarizer, err = aiFactory.CreatePRSummarizer(ctx, c.config, c.translations)
-			if err != nil {
-				// Log warning pero continuar sin AI
-				aiSummarizer = nil
-			}
-		}
+	aiSummarizer, err := c.GetPRSummarizer(ctx)
+	if err != nil {
+		aiSummarizer = nil
 	}
 
 	vcsClient, err := c.vcsRegistry.CreateClientFromConfig(ctx, c.gitService, c.config, c.translations)

internal/infrastructure/di/container_test.go

@@ -290,7 +290,6 @@ func TestGetIssueGeneratorService(t *testing.T) {
 		aiFactory := &mockAIFactory{}
 		vcsFactory := &mockVCSFactory{}
 
-		// Set expectations - Return nil client/generator is fine for basic instantiation test
 		aiFactory.On("CreateIssueContentGenerator", mock.Anything, cfg, trans).Return(nil, nil)
 		vcsFactory.On("CreateClient", mock.Anything, "owner", "repo", "", trans).Return(nil, nil)
 		vcsFactory.On("ValidateConfig", mock.Anything).Return(nil)