fix: deduplicate monthly Russell execution

Co-Authored-By: Codex <noreply@openai.com>

Author: Pigbibi

decision_mapper.py

@@ -388,7 +388,13 @@ def _build_weight_translation_annotations(
             else None
         ),
         signal_date=(
-            str(execution_annotations.get("signal_date") or diagnostics.get("signal_date") or "").strip() or None
+            str(
+                execution_annotations.get("signal_date")
+                or diagnostics.get("signal_date")
+                or diagnostics.get("snapshot_as_of")
+                or ""
+            ).strip()
+            or None
         ),
         effective_date=(
             str(execution_annotations.get("effective_date") or diagnostics.get("effective_date") or "").strip()

main.py

@@ -295,6 +295,15 @@ def _summarize_cycle_result_for_report(cycle_result, *, dry_run: bool) -> dict:
         summary["quote_snapshot"] = {
             "quotes": [dict(snapshot) for snapshot in quote_snapshots],
         }
+    execution = dict(getattr(cycle_result, "execution", {}) or {})
+    for field in (
+        "signal_date",
+        "effective_date",
+        "execution_timing_contract",
+        "execution_calendar_source",
+    ):
+        if execution.get(field) not in (None, ""):
+            summary[field] = execution[field]
     return summary
 
 

tests/test_decision_mapper.py

@@ -483,6 +483,7 @@ def test_translates_weight_decision_for_russell_strategy(self):
                 "signal_description": "risk on",
                 "status_description": "breadth=62.0% | regime=risk_on | benchmark=up",
                 "benchmark_symbol": "SPY",
+                "snapshot_as_of": "2026-07-31",
             },
         )
         snapshot = PortfolioSnapshot(
@@ -506,6 +507,7 @@ def test_translates_weight_decision_for_russell_strategy(self):
         self.assertEqual(plan["allocation"]["targets"]["AAPL"], 2550.0)
         self.assertEqual(plan["allocation"]["targets"]["BOXX"], 1700.0)
         self.assertEqual(plan["execution"]["signal_display"], "risk on")
+        self.assertEqual(plan["execution"]["signal_date"], "2026-07-31")
         self.assertEqual(
             plan["execution"]["status_display"],
             "breadth=62.0% | regime=risk_on | benchmark=up",

tests/test_request_handling.py

@@ -854,6 +854,10 @@ def test_cycle_result_summary_counts_dry_run_order_previews(self):
             skip_logs=("skip",),
             note_logs=("note",),
             action_done=True,
+            execution={
+                "signal_date": "2026-07-31",
+                "execution_timing_contract": "monthly_snapshot_window",
+            },
             dry_run_orders=(
                 {"symbol": "02800.HK", "side": "buy", "quantity": 100, "status": "dry_run"},
                 {"symbol": "03033.HK", "side": "buy", "quantity": 200, "status": "dry_run"},
@@ -872,6 +876,8 @@ def test_cycle_result_summary_counts_dry_run_order_previews(self):
         self.assertEqual(summary["orders_skipped_count"], 1)
         self.assertEqual(summary["notes_count"], 1)
         self.assertTrue(summary["dry_run_order_preview_available"])
+        self.assertEqual(summary["signal_date"], "2026-07-31")
+        self.assertEqual(summary["execution_timing_contract"], "monthly_snapshot_window")
         self.assertEqual(summary["orders_previewed"][0]["symbol"], "02800.HK")
         self.assertEqual(summary["quote_snapshot"]["quotes"][0]["symbol"], "02800.HK")