Skip to content

Commit 66ad287

Browse files
committed
fix: OrderedDict has no .add(), use key assignment for delivery dedup
Commit 75a6d36 changed processed_deliveries from set to OrderedDict for deterministic eviction, but mark_delivery_processed() still used .add() which is a set-only method. Changed to key assignment: processed_deliveries[delivery_id] = None Fixes 6 failing tests in CI (AttributeError on main).
1 parent c9d3167 commit 66ad287

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

runner/core/validation.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ def is_duplicate_delivery(delivery_id: str) -> bool:
6161
def mark_delivery_processed(delivery_id: str):
6262
"""Mark a delivery ID as processed for replay protection."""
6363
cleanup_delivery_cache()
64-
processed_deliveries.add(delivery_id)
64+
processed_deliveries[delivery_id] = None
6565

6666

6767
def reset_caches():

0 commit comments

Comments
 (0)