make make pretty actually fix imports

Signed-off-by: Andres Taylor <[email protected]>

Makefile

@@ -42,8 +42,8 @@ clean:
 # Pretty: formats the code using gofumpt and goimports-reviser
 pretty: check-tools
 	@echo "Running formatting tools..."
-	@gofumpt -l -w . >/dev/null 2>&1 || true
-	@goimports-reviser -project-name $$(go list -m) -rm-unused -set-alias -format . >/dev/null 2>&1 || true
+	@gofumpt -w . >/dev/null 2>&1
+	@goimports-reviser -recursive -project-name $$(go list -m) -rm-unused -set-alias ./go >/dev/null 2>&1
 
 # Tools installation command
 install-tools:

git-hooks/pre-commit

@@ -18,12 +18,12 @@ for file in $GO_FILES; do
     }
 done
 
-# 2. Run goimports-reviser to verify formatting on changed files (without writing changes)
-echo "Checking goimports-reviser formatting..."
+# Check imports with goimports-reviser
+echo "Checking goimports-reviser imports..."
 for file in $GO_FILES; do
-    goimports-reviser -project-name $(go list -m) -rm-unused -set-alias -format "$file" -diff >/dev/null 2>&1
+    goimports-reviser -project-name $(go list -m) -rm-unused -set-alias "$file" >/dev/null 2>&1
     if [[ $? -ne 0 ]]; then
-        echo "Error: Imports not correctly formatted in $file. Please run 'make pretty'."
+        echo "Error: Imports not correctly organized in $file. Please run 'make pretty'."
         exit 1
     fi
 done

go/summarize/utils_test.go

@@ -20,7 +20,6 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/assert"
-
 	"github.com/stretchr/testify/require"
 )