diff --git a/pkg/strategy/xmaker/strategy.go b/pkg/strategy/xmaker/strategy.go
index 3c43bec9c..7abd6b42d 100644
--- a/pkg/strategy/xmaker/strategy.go
+++ b/pkg/strategy/xmaker/strategy.go
@@ -928,6 +928,7 @@ func (s *Strategy) CrossRun(
 		}
 	})
 
+	// TODO: remove this nil value behavior, check all OnProfit usage and remove the EmitProfit call with nil profit
 	s.tradeCollector.OnProfit(func(trade types.Trade, profit *types.Profit) {
 		if profit != nil {
 			if s.CircuitBreaker != nil {