Fix lint errors: remove unused import and duplicate assert binding

- Removed unused 'strings' import from jsonl_logger_test.go
- Fixed duplicate assert.New(t) call in TestLogRPCMessageJSONLDirectionTypes
  The parent function already creates the assert binding, so the subtest
  was attempting to call .New() on *assert.Assertions instead of the package
- Applied gofmt formatting fixes

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

Author: Copilot

internal/logger/jsonl_logger_test.go

@@ -6,7 +6,6 @@ import (
 	"fmt"
 	"os"
 	"path/filepath"
-	"strings"
 	"testing"
 
 	"github.com/githubnext/gh-aw-mcpg/internal/logger/sanitize"
@@ -150,7 +149,7 @@ func TestSanitizePayload(t *testing.T) {
 		t.Run(tt.name, func(t *testing.T) {
 			require := require.New(t)
 			assert := assert.New(t)
-			
+
 			result := sanitize.SanitizeJSON([]byte(tt.input))
 			require.NotNil(result, "sanitize.SanitizeJSON returned nil")
 
@@ -342,7 +341,7 @@ func TestMultipleMessagesInJSONL(t *testing.T) {
 func TestSanitizePayloadCompactsJSON(t *testing.T) {
 	require := require.New(t)
 	assert := assert.New(t)
-	
+
 	// Test that multi-line JSON is compacted to a single line
 	multilineJSON := `{
 		"jsonrpc": "2.0",
@@ -372,7 +371,7 @@ func TestSanitizePayloadCompactsJSON(t *testing.T) {
 
 func TestInitJSONLLoggerWithInvalidPath(t *testing.T) {
 	assert := assert.New(t)
-	
+
 	// Test initialization with an invalid directory path (permission denied scenario)
 	// Using /proc/self as it's read-only and will fail to create subdirectories
 	err := InitJSONLLogger("/proc/self/invalid", "test.jsonl")
@@ -423,30 +422,29 @@ func TestLogRPCMessageJSONLDirectionTypes(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			assert := assert.New(t)
 			testPayload := []byte(`{"jsonrpc":"2.0","id":1}`)
-			
+
 			// Clear previous log file
 			logPath := filepath.Join(logDir, "test.jsonl")
 			os.Remove(logPath)
-			
+
 			LogRPCMessageJSONL(tt.direction, tt.msgType, "test-server", "test-method", testPayload, nil)
 			CloseJSONLLogger()
-			
+
 			// Re-init for next iteration
 			if t.Name() != tests[len(tests)-1].name {
 				InitJSONLLogger(logDir, "test.jsonl")
 			}
-			
+
 			// Read and verify
 			content, err := os.ReadFile(logPath)
 			if err != nil {
 				return // File might not exist yet
 			}
-			
+
 			var entry JSONLRPCMessage
 			json.Unmarshal(content, &entry)
-			
+
 			assert.Equal(tt.expected["direction"], entry.Direction, "Direction should match")
 			assert.Equal(tt.expected["type"], entry.Type, "Type should match")
 		})

internal/server/auth.go

@@ -20,7 +20,7 @@ func authMiddleware(apiKey string, next http.HandlerFunc) http.HandlerFunc {
 	logAuth.Printf("Initialized auth middleware")
 	return func(w http.ResponseWriter, r *http.Request) {
 		logAuth.Printf("Authenticating request: method=%s, path=%s, remote=%s", r.Method, r.URL.Path, r.RemoteAddr)
-		
+
 		// Extract Authorization header
 		authHeader := r.Header.Get("Authorization")
 
@@ -52,7 +52,7 @@ func authMiddleware(apiKey string, next http.HandlerFunc) http.HandlerFunc {
 // logRuntimeError logs runtime errors to stdout per spec section 9.2
 func logRuntimeError(errorType, detail string, r *http.Request, serverName *string) {
 	logAuth.Printf("Logging runtime error: type=%s, detail=%s", errorType, detail)
-	
+
 	timestamp := time.Now().UTC().Format(time.RFC3339)
 	requestID := r.Header.Get("X-Request-ID")
 	if requestID == "" {

internal/server/transport.go

@@ -98,9 +98,9 @@ func CreateHTTPServerForMCP(addr string, unifiedServer *UnifiedServer, apiKey st
 
 		return unifiedServer.server
 	}, &sdk.StreamableHTTPOptions{
-		Stateless:      false,                                      // Support stateful sessions
+		Stateless:      false,                                         // Support stateful sessions
 		Logger:         logger.NewSlogLoggerWithHandler(logTransport), // Integrate SDK logging with project logger
-		SessionTimeout: 30 * time.Minute,                            // Prevent resource leaks from idle connections
+		SessionTimeout: 30 * time.Minute,                              // Prevent resource leaks from idle connections
 	})
 
 	// Wrap SDK handler with detailed logging for JSON-RPC translation debugging