feat: add weight validation (must be >= 1) to HttpRequestRegexMatcher

Reject non-positive weight values in the constructor to prevent
weight=0 (free requests) or negative weights (budget restoration)
from bypassing rate limits. Adds two tests for validation.

Co-Authored-By: Daryna Ishchenko <darina.ishchenko17@gmail.com>

Author: devin-ai-integration[bot]

airbyte_cdk/sources/streams/call_rate.py

@@ -179,6 +179,8 @@ def __init__(
             where different endpoints consume different amounts from a shared budget.
             If not set, each request counts as 1.
         """
+        if weight is not None and weight < 1:
+            raise ValueError(f"weight must be >= 1, got {weight}")
         self._weight = weight
         self._method = method.upper() if method else None
 

unit_tests/sources/streams/test_call_rate.py

@@ -376,6 +376,16 @@ def test_matcher_weight_is_stored(self):
         matcher = HttpRequestRegexMatcher(url_path_pattern=r"/api/test", weight=60)
         assert matcher.weight == 60
 
+    def test_matcher_rejects_zero_weight(self):
+        """HttpRequestRegexMatcher raises ValueError for weight=0."""
+        with pytest.raises(ValueError, match="weight must be >= 1"):
+            HttpRequestRegexMatcher(url_path_pattern=r"/api/test", weight=0)
+
+    def test_matcher_rejects_negative_weight(self):
+        """HttpRequestRegexMatcher raises ValueError for negative weight."""
+        with pytest.raises(ValueError, match="weight must be >= 1"):
+            HttpRequestRegexMatcher(url_path_pattern=r"/api/test", weight=-5)
+
     def test_policy_get_weight_returns_matcher_weight(self):
         """BaseCallRatePolicy.get_weight returns weight from the matching matcher."""
         policy = MovingWindowCallRatePolicy(