From 76f934f95d3b5cbb96765e7158976e4a4c879fa9 Mon Sep 17 00:00:00 2001
From: TheSiahxyz <164138827+TheSiahxyz@users.noreply.github.com>
Date: Wed, 1 Apr 2026 17:11:10 +0900
Subject: feat(strategy): add parameter validation to all strategies

---
 services/strategy-engine/strategies/bollinger_strategy.py | 7 +++++++
 1 file changed, 7 insertions(+)

(limited to 'services/strategy-engine/strategies/bollinger_strategy.py')

diff --git a/services/strategy-engine/strategies/bollinger_strategy.py b/services/strategy-engine/strategies/bollinger_strategy.py
index bee7ee4..4aceee4 100644
--- a/services/strategy-engine/strategies/bollinger_strategy.py
+++ b/services/strategy-engine/strategies/bollinger_strategy.py
@@ -29,6 +29,13 @@ class BollingerStrategy(BaseStrategy):
         self._min_bandwidth = float(params.get("min_bandwidth", 0.02))
         self._quantity = Decimal(str(params.get("quantity", "0.01")))
 
+        if self._period < 2:
+            raise ValueError(f"Bollinger period must be >= 2, got {self._period}")
+        if self._num_std <= 0:
+            raise ValueError(f"Bollinger num_std must be > 0, got {self._num_std}")
+        if self._quantity <= 0:
+            raise ValueError(f"Quantity must be positive, got {self._quantity}")
+
     def reset(self) -> None:
         self._closes.clear()
         self._was_below_lower = False
-- 
cgit v1.2.3