refactor(#121): Change log level to debug for class analysis messages (#128)

Author: volodya-lombrozo

internal/critic/agent.go

@@ -30,7 +30,7 @@ type promptData struct {
 // Review sends the provided Java class to the Critic for analysis and returns suggested improvements.
 func (c *agent) Review(job *domain.Job) (*domain.Artifacts, error) {
 	class := job.Classes[0]
-	c.log.Info("Received class %q for analysis", class.Name())
+	c.log.Debug("Received class %q for analysis", class.Name())
 	imperfections := tool.NewCombined(c.tools...).Imperfections()
 	var imp []string
 	if imperfections != "" {
@@ -81,15 +81,13 @@ func parseAnswer(answer string) []string {
 	return suggestions
 }
 
-func (c *agent) associated(suggestions []string, class string) []domain.Suggestion {
+func (c *agent) associated(critique []string, class string) []domain.Suggestion {
 	res := make([]domain.Suggestion, 0)
-	for i, suggestion := range suggestions {
-		if strings.EqualFold(suggestion, notFound) {
-			c.log.Info("No suggestions found for the class #%d: %s", i+1, class)
-		} else {
-			res = append(res, *domain.NewSuggestion(suggestion, class))
+	for _, c := range critique {
+		if !strings.EqualFold(c, notFound) {
+			res = append(res, *domain.NewSuggestion(c, class))
 		}
 	}
-	c.log.Info("Total suggestions associated with class %s: %d", class, len(res))
+	c.log.Info("Found %d suggestions for %s", len(res), class)
 	return res
 }

internal/critic/server.go

@@ -48,7 +48,7 @@ func (c *Critic) ListenAndServe() error {
 // Review sends the provided Java class to the Critic for analysis and returns suggested improvements.
 func (c *Critic) Review(job *domain.Job) (*domain.Artifacts, error) {
 	address := fmt.Sprintf("http://localhost:%d", c.port)
-	c.log.Info("Asking critic (%s) to lint the class...", address)
+	c.log.Debug("Asking critic (%s) to lint the class...", address)
 	critic := protocol.NewClient(address)
 	resp, err := critic.SendMessage(job.Marshal())
 	if err != nil {

internal/domain/agents.go

@@ -1,6 +1,9 @@
 package domain
 
-import "fmt"
+import (
+	"fmt"
+	"strconv"
+)
 
 // Facilitator represents an interface to refactor tasks into multiple classes.
 type Facilitator interface {
@@ -37,6 +40,15 @@ func (j *Job) Param(key string) (any, bool) {
 	return res, ok
 }
 
+func (j *Job) MaxSize() (int, error) {
+	size, ok := j.Param("max-size")
+	ssize := fmt.Sprintf("%v", size)
+	if !ok {
+		ssize = "200"
+	}
+	return strconv.Atoi(ssize)
+}
+
 type Artifacts struct {
 	Descr       *Description
 	Classes     []Class