fix: correct gofmt formatting and nolint directive syntax

- Fix gofmt alignment in feeders/yaml.go, phase.go, tenant_service.go
- Change //nolint:G118 to //nolint:gosec (golangci-lint uses linter names)
- Add G118 suppression for durable_memory.go and kafka.go
- Fix contextcheck nolint in scheduler.go
- Suppress SA1019 on proxy.Director = nil (required when using Rewrite)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: intel352

feeders/yaml.go

@@ -56,7 +56,7 @@ func getFieldNameFromTag(fieldType *reflect.StructField) (string, bool) {
 type YamlFeeder struct {
 	Path         string
 	verboseDebug bool
-	debugFn func(string)
+	debugFn      func(string)
 	fieldTracker FieldTracker
 	priority     int
 }

modules/eventbus/durable_memory.go

@@ -216,7 +216,7 @@ func (d *DurableMemoryEventBus) Start(ctx context.Context) error {
 	if d.isStarted {
 		return nil
 	}
-	d.ctx, d.cancel = context.WithCancel(ctx)
+	d.ctx, d.cancel = context.WithCancel(ctx) //nolint:gosec // G118: cancel is stored in d.cancel and called in Stop()
 	d.isStarted = true
 	return nil
 }

modules/eventbus/kafka.go

@@ -235,7 +235,7 @@ func (k *KafkaEventBus) Start(ctx context.Context) error {
 		return nil
 	}
 
-	k.ctx, k.cancel = context.WithCancel(ctx)
+	k.ctx, k.cancel = context.WithCancel(ctx) //nolint:gosec // G118: cancel is stored in k.cancel and called in Stop()
 	k.isStarted = true
 	return nil
 }

modules/eventbus/memory.go

@@ -113,7 +113,7 @@ func (m *MemoryEventBus) Start(ctx context.Context) error {
 		return nil
 	}
 
-	m.ctx, m.cancel = context.WithCancel(ctx) //nolint:G118 // cancel is stored in m.cancel and called in Stop()
+	m.ctx, m.cancel = context.WithCancel(ctx) //nolint:gosec // G118: cancel is stored in m.cancel and called in Stop()
 
 	// Initialize worker pool for async event handling.
 	// Buffer size is MaxEventQueueSize so the task queue can absorb bursts

modules/eventbus/nats.go

@@ -177,7 +177,7 @@ func (n *NatsEventBus) Start(ctx context.Context) error {
 		return ErrNATSConnectionNotEstablished
 	}
 
-	n.ctx, n.cancel = context.WithCancel(ctx) //nolint:G118 // cancel is stored in n.cancel and called in Stop()
+	n.ctx, n.cancel = context.WithCancel(ctx) //nolint:gosec // G118: cancel is stored in n.cancel and called in Stop()
 	n.isStarted = true
 	return nil
 }

modules/eventbus/redis.go

@@ -140,7 +140,7 @@ func (r *RedisEventBus) Start(ctx context.Context) error {
 		return fmt.Errorf("failed to connect to Redis: %w", err)
 	}
 
-	r.ctx, r.cancel = context.WithCancel(ctx) //nolint:G118 // cancel is stored in r.cancel and called in Stop()
+	r.ctx, r.cancel = context.WithCancel(ctx) //nolint:gosec // G118: cancel is stored in r.cancel and called in Stop()
 	r.isStarted = true
 	return nil
 }

modules/reverseproxy/module.go

@@ -1698,7 +1698,7 @@ func (m *ReverseProxyModule) createReverseProxyForBackend(ctx context.Context, t
 			}
 		}
 	}
-	proxy.Director = nil
+	proxy.Director = nil //nolint:staticcheck // SA1019: explicitly nil Director when using Rewrite per Go docs
 
 	// Set up error handler to return proper HTTP status codes for connection failures
 	proxy.ErrorHandler = func(w http.ResponseWriter, r *http.Request, err error) {

modules/scheduler/scheduler.go

@@ -194,13 +194,13 @@ func (s *Scheduler) Start(ctx context.Context) error {
 		s.logger.Info("Starting scheduler", "workers", s.workerCount, "queueSize", s.queueSize)
 	}
 
-	s.ctx, s.cancel = context.WithCancel(ctx) //nolint:G118 // cancel is stored in s.cancel and called in Stop()
+	s.ctx, s.cancel = context.WithCancel(ctx) //nolint:gosec // G118: cancel is stored in s.cancel and called in Stop()
 	s.jobQueue = make(chan Job, s.queueSize)
 
 	// Start worker goroutines
 	for i := 0; i < s.workerCount; i++ {
 		s.wg.Add(1)
-		//nolint:contextcheck,G118 // Context is passed through s.ctx field; workers use s.ctx
+		//nolint:contextcheck,gosec // Context is passed through s.ctx field; workers use s.ctx
 		go s.worker(i)
 
 		// Emit worker started event

phase.go

@@ -4,7 +4,7 @@ package modular
 type AppPhase int32
 
 const (
-	PhaseCreated      AppPhase = iota
+	PhaseCreated AppPhase = iota
 	PhaseInitializing
 	PhaseInitialized
 	PhaseStarting

tenant_service.go

@@ -3,8 +3,8 @@
 package modular
 
 import (
-	"slices"
 	"fmt"
+	"slices"
 	"sync"
 )