address linter and tests

internal/pool/conn_state.go

@@ -218,17 +218,20 @@ func (sm *ConnStateMachine) AwaitAndTransition(
 	targetState ConnState,
 ) (ConnState, error) {
 	// Fast path: try immediate transition with CAS to prevent race conditions
-	for _, fromState := range validFromStates {
-		// Check if we're already in target state
-		if fromState == targetState && sm.GetState() == targetState {
-			return targetState, nil
-		}
+	// BUT: only if there are no waiters in the queue (to maintain FIFO ordering)
+	if sm.waiterCount.Load() == 0 {
+		for _, fromState := range validFromStates {
+			// Check if we're already in target state
+			if fromState == targetState && sm.GetState() == targetState {
+				return targetState, nil
+			}
 
-		// Try to atomically swap from fromState to targetState
-		if sm.state.CompareAndSwap(uint32(fromState), uint32(targetState)) {
-			// Success! We transitioned atomically
-			sm.notifyWaiters()
-			return targetState, nil
+			// Try to atomically swap from fromState to targetState
+			if sm.state.CompareAndSwap(uint32(fromState), uint32(targetState)) {
+				// Success! We transitioned atomically
+				sm.notifyWaiters()
+				return targetState, nil
+			}
 		}
 	}
 

internal/semaphore.go

@@ -14,12 +14,6 @@ var semTimers = sync.Pool{
 	},
 }
 
-// waiter represents a goroutine waiting for a token.
-type waiter struct {
-	ready chan struct{}
-	next  *waiter
-}
-
 // FastSemaphore is a channel-based semaphore optimized for performance.
 // It uses a fast path that avoids timer allocation when tokens are available.
 // The channel is pre-filled with tokens: Acquire = receive, Release = send.