Trim visible strategy plugin alert fields (#60)

Author: Pigbibi

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "quant-platform-kit"
-version = "0.7.27"
+version = "0.7.28"
 description = "Shared broker adapters, domain models, execution ports, and notification utilities for QuantStrategyLab strategies."
 readme = "README.md"
 requires-python = ">=3.9"

src/quant_platform_kit/common/strategy_plugins.py

@@ -483,10 +483,6 @@ def build_strategy_plugin_alert_messages(
         translated_route = translate_strategy_plugin_value("route", route, translator=translator)
         translated_action = translate_strategy_plugin_value("action", action, translator=translator)
         strategy = str(strategy_label or getattr(signal, "strategy", None) or "").strip() or "unknown"
-        would_trade = _translate_bool(
-            translator,
-            bool(getattr(signal, "would_trade_if_enabled", False)),
-        )
         subject = _translate(
             translator,
             "strategy_plugin_alert_subject",
@@ -548,24 +544,8 @@ def build_strategy_plugin_alert_messages(
                     fallback="Signal as-of: {as_of}",
                     as_of=getattr(signal, "as_of", None) or "unknown",
                 ),
-                _translate(
-                    translator,
-                    "strategy_plugin_alert_would_trade",
-                    fallback="Would trade if enabled: {value}",
-                    value=would_trade,
-                ),
             ]
         )
-        source = getattr(signal, "source_uri", None) or getattr(signal, "local_path", None)
-        if source:
-            body_lines.append(
-                _translate(
-                    translator,
-                    "strategy_plugin_alert_source",
-                    fallback="Source: {source}",
-                    source=source,
-                )
-            )
         metadata = {
             "strategy": getattr(signal, "strategy", None),
             "strategy_label": strategy,
@@ -686,14 +666,6 @@ def _translate(
     return translated if translated != key else fallback.format(**kwargs)
 
 
-def _translate_bool(translator: Callable[..., str] | None, value: bool) -> str:
-    return _translate(
-        translator,
-        "strategy_plugin_alert_yes" if value else "strategy_plugin_alert_no",
-        fallback="yes" if value else "no",
-    )
-
-
 def _required_string(value: Any, *, field_name: str) -> str:
     text = _optional_string(value)
     if text is None:

tests/test_strategy_plugins.py

@@ -279,9 +279,6 @@ def test_strategy_plugin_true_crisis_builds_generic_alert_message(self):
             "strategy_plugin_alert_action": "action={action}",
             "strategy_plugin_alert_mode": "mode={mode}",
             "strategy_plugin_alert_as_of": "as_of={as_of}",
-            "strategy_plugin_alert_would_trade": "would_trade={value}",
-            "strategy_plugin_alert_source": "source={source}",
-            "strategy_plugin_alert_yes": "yes",
             "strategy_plugin_name_crisis_response_shadow": "Crisis",
             "strategy_plugin_mode_shadow": "shadow",
             "strategy_plugin_route_true_crisis": "true crisis",
@@ -303,8 +300,9 @@ def test_strategy_plugin_true_crisis_builds_generic_alert_message(self):
         self.assertIn("status=true crisis", alerts[0].body)
         self.assertIn("action=defend", alerts[0].body)
         self.assertIn("mode=shadow", alerts[0].body)
-        self.assertIn("would_trade=yes", alerts[0].body)
-        self.assertIn("source=gs://bucket/latest_signal.json", alerts[0].body)
+        self.assertNotIn("would_trade=", alerts[0].body)
+        self.assertNotIn("source=", alerts[0].body)
+        self.assertTrue(alerts[0].metadata["would_trade_if_enabled"])
 
 
 if __name__ == "__main__":