feat(makefile): enhance formatting and vetting for all Go modules

- Updated the `fmt`, `vet`, and `tidy` targets in the Makefile to format, vet, and tidy all Go modules individually, improving modularity and clarity in the build process.
- Enhanced output messages to reflect the processing of multiple modules, providing better feedback during execution.
- Cleaned up unnecessary blank lines and improved code readability in several example files across the project.

This update improves the maintainability and usability of the Makefile for developers working with multiple Go modules.

Author: juicycleff

Makefile

@@ -127,11 +127,27 @@ install:
 # ==============================================================================
 
 .PHONY: test
-## test: Run all tests with race detector
+## test: Run all tests with race detector in all modules
 test:
-	@echo "$(COLOR_GREEN)Running tests...$(COLOR_RESET)"
-	@$(GOTEST) $(TEST_FLAGS) $(TEST_DIRS)
-	@echo "$(COLOR_GREEN)✓ All tests passed$(COLOR_RESET)"
+	@echo "$(COLOR_GREEN)Running tests in all modules...$(COLOR_RESET)"
+	@COUNT=0; FAILED=0; \
+	ROOT_DIR=$$(pwd); \
+	for modfile in $$(find . -name "go.mod" -type f | grep -v "/vendor/" | sort); do \
+		dir=$$(dirname $$modfile); \
+		echo "  Testing $$dir..."; \
+		if cd "$$ROOT_DIR/$$dir" && $(GOTEST) $(TEST_FLAGS) ./...; then \
+			COUNT=$$((COUNT + 1)); \
+		else \
+			FAILED=$$((FAILED + 1)); \
+		fi; \
+	done; \
+	cd "$$ROOT_DIR"; \
+	if [ $$FAILED -eq 0 ]; then \
+		echo "$(COLOR_GREEN)✓ All tests passed in $$COUNT modules$(COLOR_RESET)"; \
+	else \
+		echo "$(COLOR_RED)✗ Tests failed in $$FAILED module(s), passed in $$COUNT module(s)$(COLOR_RESET)"; \
+		exit 1; \
+	fi
 
 .PHONY: test-short
 ## test-short: Run tests with -short flag
@@ -217,16 +233,31 @@ bench-compare:
 # ==============================================================================
 
 .PHONY: lint
-## lint: Run golangci-lint
+## lint: Run golangci-lint on all modules
 lint:
-	@echo "$(COLOR_GREEN)Running linter...$(COLOR_RESET)"
-	@if command -v $(GOLANGCI_LINT) >/dev/null 2>&1; then \
-		$(GOLANGCI_LINT) run $(LINT_DIRS) --timeout=5m; \
-		echo "$(COLOR_GREEN)✓ Linting passed$(COLOR_RESET)"; \
-	else \
+	@echo "$(COLOR_GREEN)Running linter on all modules...$(COLOR_RESET)"
+	@if ! command -v $(GOLANGCI_LINT) >/dev/null 2>&1; then \
 		echo "$(COLOR_RED)Error: golangci-lint not found. Run 'make install-tools' to install$(COLOR_RESET)"; \
 		exit 1; \
 	fi
+	@COUNT=0; FAILED=0; \
+	ROOT_DIR=$$(pwd); \
+	for modfile in $$(find . -name "go.mod" -type f | grep -v "/vendor/" | sort); do \
+		dir=$$(dirname $$modfile); \
+		echo "  Linting $$dir..."; \
+		if cd "$$ROOT_DIR/$$dir" && $(GOLANGCI_LINT) run ./... --timeout=5m; then \
+			COUNT=$$((COUNT + 1)); \
+		else \
+			FAILED=$$((FAILED + 1)); \
+		fi; \
+	done; \
+	cd "$$ROOT_DIR"; \
+	if [ $$FAILED -eq 0 ]; then \
+		echo "$(COLOR_GREEN)✓ Linting passed for $$COUNT modules$(COLOR_RESET)"; \
+	else \
+		echo "$(COLOR_RED)✗ Linting failed in $$FAILED module(s), passed in $$COUNT module(s)$(COLOR_RESET)"; \
+		exit 1; \
+	fi
 
 .PHONY: lint-fix
 ## lint-fix: Run golangci-lint with auto-fix
@@ -235,11 +266,19 @@ lint-fix:
 	@$(GOLANGCI_LINT) run $(LINT_DIRS) --fix --timeout=5m
 
 .PHONY: fmt
-## fmt: Format Go code
+## fmt: Format Go code in all modules
 fmt:
-	@echo "$(COLOR_GREEN)Formatting code...$(COLOR_RESET)"
-	@$(GOFMT) $(TEST_DIRS)
-	@echo "$(COLOR_GREEN)✓ Code formatted$(COLOR_RESET)"
+	@echo "$(COLOR_GREEN)Formatting code in all modules...$(COLOR_RESET)"
+	@COUNT=0; \
+	ROOT_DIR=$$(pwd); \
+	for modfile in $$(find . -name "go.mod" -type f | grep -v "/vendor/" | sort); do \
+		dir=$$(dirname $$modfile); \
+		echo "  Formatting $$dir..."; \
+		cd "$$ROOT_DIR/$$dir" && $(GOFMT) ./...; \
+		COUNT=$$((COUNT + 1)); \
+	done; \
+	cd "$$ROOT_DIR"; \
+	echo "$(COLOR_GREEN)✓ Formatted $$COUNT modules$(COLOR_RESET)"
 
 .PHONY: fmt-check
 ## fmt-check: Check if code is formatted
@@ -249,18 +288,34 @@ fmt-check:
 		(echo "$(COLOR_RED)Code is not formatted. Run 'make fmt'$(COLOR_RESET)" && exit 1)
 
 .PHONY: vet
-## vet: Run go vet
+## vet: Run go vet on all modules
 vet:
-	@echo "$(COLOR_GREEN)Running go vet...$(COLOR_RESET)"
-	@$(GOVET) $(TEST_DIRS)
-	@echo "$(COLOR_GREEN)✓ Vet passed$(COLOR_RESET)"
+	@echo "$(COLOR_GREEN)Running go vet on all modules...$(COLOR_RESET)"
+	@COUNT=0; \
+	ROOT_DIR=$$(pwd); \
+	for modfile in $$(find . -name "go.mod" -type f | grep -v "/vendor/" | sort); do \
+		dir=$$(dirname $$modfile); \
+		echo "  Vetting $$dir..."; \
+		cd "$$ROOT_DIR/$$dir" && $(GOVET) ./...; \
+		COUNT=$$((COUNT + 1)); \
+	done; \
+	cd "$$ROOT_DIR"; \
+	echo "$(COLOR_GREEN)✓ Vet passed for $$COUNT modules$(COLOR_RESET)"
 
 .PHONY: tidy
-## tidy: Tidy go modules
+## tidy: Tidy all go modules
 tidy:
-	@echo "$(COLOR_GREEN)Tidying modules...$(COLOR_RESET)"
-	@$(GOMOD) tidy
-	@echo "$(COLOR_GREEN)✓ Modules tidied$(COLOR_RESET)"
+	@echo "$(COLOR_GREEN)Tidying all modules...$(COLOR_RESET)"
+	@COUNT=0; \
+	ROOT_DIR=$$(pwd); \
+	for modfile in $$(find . -name "go.mod" -type f | grep -v "/vendor/" | sort); do \
+		dir=$$(dirname $$modfile); \
+		echo "  Tidying $$dir..."; \
+		cd "$$ROOT_DIR/$$dir" && $(GOMOD) tidy; \
+		COUNT=$$((COUNT + 1)); \
+	done; \
+	cd "$$ROOT_DIR"; \
+	echo "$(COLOR_GREEN)✓ Tidied $$COUNT modules$(COLOR_RESET)"
 
 .PHONY: tidy-check
 ## tidy-check: Check if go.mod is tidy

cmd/forge/plugins/dev_test.go

@@ -322,7 +322,7 @@ func main() {}`
 		config: &config.ForgeConfig{
 			RootDir: tmpDir,
 			Project: config.ProjectConfig{
-				Structure: config.StructureConfig{
+				Structure: &config.StructureConfig{
 					Cmd: "cmd",
 				},
 			},

cmd/forge/plugins/generate_test.go

@@ -113,7 +113,7 @@ func TestDiscoverTargets(t *testing.T) {
 			RootDir: tmpDir,
 			Project: config.ProjectConfig{
 				Layout: "single-module",
-				Structure: config.StructureConfig{
+				Structure: &config.StructureConfig{
 					Cmd: "cmd",
 				},
 			},
@@ -162,7 +162,7 @@ func TestDiscoverTargetsNoApps(t *testing.T) {
 			RootDir: tmpDir,
 			Project: config.ProjectConfig{
 				Layout: "single-module",
-				Structure: config.StructureConfig{
+				Structure: &config.StructureConfig{
 					Cmd: "cmd",
 				},
 			},

examples/cors_preflight/main.go

@@ -24,8 +24,8 @@ func main() {
 	// This will automatically handle OPTIONS preflight requests
 	corsConfig := middleware.CORSConfig{
 		AllowOrigins: []string{
-			"http://localhost:3000",  // React dev server
-			"http://localhost:5173",  // Vite dev server
+			"http://localhost:3000",   // React dev server
+			"http://localhost:5173",   // Vite dev server
 			"https://app.example.com", // Production frontend
 		},
 		AllowMethods:     []string{"GET", "POST", "PUT", "DELETE", "OPTIONS"},
@@ -119,4 +119,3 @@ func main() {
 		log.Fatal(err)
 	}
 }
-

examples/optional_tag/main.go

@@ -216,4 +216,3 @@ func main() {
 		log.Fatal(err)
 	}
 }
-

extensions/ai/examples/ai-agents-demo/main.go

@@ -31,7 +31,7 @@ func main() {
 	})
 
 	// Create AI extension with production config
-	var ext *ai.Extension
+	var ext forge.Extension
 	if usePostgres {
 		fmt.Println("\n✓ Using PostgreSQL for persistence")
 		ext = ai.NewExtension(
@@ -143,7 +143,7 @@ func main() {
 		Config:      map[string]interface{}{},
 	}
 
-	anomalyAgent, err := agentMgr.CreateAgent(ctx, anomalyDef)
+	_, err = agentMgr.CreateAgent(ctx, anomalyDef)
 	if err != nil {
 		log.Printf("  Error creating anomaly agent: %v", err)
 	} else {
@@ -163,7 +163,7 @@ func main() {
 		Config:      map[string]interface{}{},
 	}
 
-	schedulerAgent, err := agentMgr.CreateAgent(ctx, schedulerDef)
+	_, err = agentMgr.CreateAgent(ctx, schedulerDef)
 	if err != nil {
 		log.Printf("  Error creating scheduler agent: %v", err)
 	} else {

extensions/ai/inference/examples/batch_inference.go

@@ -1,3 +1,5 @@
+//go:build ignore
+
 package main
 
 import (

extensions/ai/inference/examples/pipeline_example.go

@@ -1,3 +1,5 @@
+//go:build ignore
+
 package main
 
 import (

extensions/ai/inference/examples/pytorch_serving.go

@@ -1,3 +1,5 @@
+//go:build ignore
+
 package main
 
 import (
@@ -52,26 +54,26 @@ func main() {
 	// 3. Add text preprocessing pipeline
 	engine.AddPreprocessor(func(ctx context.Context, req inference.InferenceRequest) (inference.InferenceRequest, error) {
 		fmt.Printf("  → Tokenizing input for request %s\n", req.ID)
-		
+
 		// Text preprocessing:
 		// - Tokenization
 		// - Padding/truncation
 		// - Convert to tensor format
 		req.Input = tokenizeText(req.Input)
-		
+
 		return req, nil
 	})
 
 	// 4. Add output formatting
 	engine.AddPostprocessor(func(ctx context.Context, resp inference.InferenceResponse) (inference.InferenceResponse, error) {
 		fmt.Printf("  ← Formatting output for response %s\n", resp.ID)
-		
+
 		// Output postprocessing:
 		// - Apply softmax
 		// - Get top-k predictions
 		// - Format as human-readable labels
 		resp.Output = formatClassificationOutput(resp.Output)
-		
+
 		return resp, nil
 	})
 
@@ -128,16 +130,16 @@ func main() {
 // loadPyTorchModel loads a PyTorch model (via ONNX Runtime)
 func loadPyTorchModel(path string) models.Model {
 	// Example stub - replace with actual ONNX loading:
-	// 
+	//
 	// import "github.com/yalue/onnxruntime_go"
-	// 
+	//
 	// session, err := onnxruntime_go.NewSession(path)
 	// if err != nil {
 	//     return nil
 	// }
-	// 
+	//
 	// return &ONNXModelWrapper{session: session}
-	
+
 	return &MockPyTorchModel{name: "text-classifier"}
 }
 
@@ -149,7 +151,7 @@ type MockPyTorchModel struct {
 func (m *MockPyTorchModel) Predict(ctx context.Context, input interface{}) (interface{}, error) {
 	// Simulate model inference with GPU latency
 	time.Sleep(20 * time.Millisecond)
-	
+
 	return map[string]float64{
 		"positive": 0.78,
 		"negative": 0.15,
@@ -181,7 +183,7 @@ func (s *ModelAwareBatchingStrategy) ShouldBatch(req inference.InferenceRequest,
 	if batch.ModelID != req.ModelID {
 		return false
 	}
-	
+
 	// Batch if we haven't reached optimal size
 	return batch.Size < s.optimalBatchSize
 }
@@ -216,9 +218,9 @@ func formatClassificationOutput(output interface{}) interface{} {
 func simulateLoad(ctx context.Context, engine *inference.InferenceEngine, reqPerSec int, duration time.Duration) {
 	ticker := time.NewTicker(time.Second / time.Duration(reqPerSec))
 	defer ticker.Stop()
-	
+
 	done := time.After(duration)
-	
+
 	for {
 		select {
 		case <-done:

extensions/ai/inference/examples/tensorflow_serving.go

@@ -1,3 +1,5 @@
+//go:build ignore
+
 package main
 
 import (
@@ -59,12 +61,12 @@ func main() {
 		// - Resize images to 224x224
 		// - Normalize pixel values to [0, 1]
 		// - Convert to model input format
-		
+
 		fmt.Printf("  → Preprocessing request %s\n", req.ID)
-		
+
 		// Your preprocessing logic here
 		req.Input = preprocessImage(req.Input)
-		
+
 		return req, nil
 	})
 
@@ -74,12 +76,12 @@ func main() {
 		// - Convert logits to probabilities
 		// - Apply confidence threshold
 		// - Format output labels
-		
+
 		fmt.Printf("  ← Postprocessing response %s\n", resp.ID)
-		
+
 		// Your postprocessing logic here
 		resp.Output = postprocessOutput(resp.Output)
-		
+
 		return resp, nil
 	})
 
@@ -113,7 +115,7 @@ func main() {
 	// Batch of requests (will be automatically batched)
 	fmt.Println("\nSending batch of 20 requests...")
 	start := time.Now()
-	
+
 	for i := 0; i < 20; i++ {
 		go func(idx int) {
 			result, err := engine.Infer(ctx, inference.InferenceRequest{
@@ -127,14 +129,14 @@ func main() {
 			}
 		}(i)
 	}
-	
+
 	time.Sleep(2 * time.Second) // Wait for batch processing
-	
+
 	fmt.Printf("\nBatch completed in %v\n", time.Since(start))
 
 	// 7. Display statistics
 	fmt.Println("\n--- Engine Statistics ---\n")
-	
+
 	stats := engine.Stats()
 	fmt.Printf("Requests Processed: %d\n", stats.RequestsProcessed)
 	fmt.Printf("Batches Processed:  %d\n", stats.BatchesProcessed)
@@ -156,16 +158,16 @@ func main() {
 // Replace with actual TensorFlow loading logic
 func loadTensorFlowModel(path string) models.Model {
 	// Example stub - replace with actual TensorFlow model loading:
-	// 
+	//
 	// import tf "github.com/tensorflow/tensorflow/tensorflow/go"
-	// 
+	//
 	// model, err := tf.LoadSavedModel(path, []string{"serve"}, nil)
 	// if err != nil {
 	//     return nil
 	// }
-	// 
+	//
 	// return &TensorFlowModelWrapper{model: model}
-	
+
 	return &MockModel{name: "image-classifier"}
 }