Gate deploy on CI success and relabel dashboard buying power for cash-only mode. (#209)

Deploy waits for a green CI run on main; notification renderers swap buying power labels based on cash_only_execution.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: Pigbibi

.github/workflows/sync-cloud-run-env.yml

@@ -1,8 +1,10 @@
 name: Deploy Cloud Run
 
 on:
-  push:
-    branches: [ main ]
+  workflow_run:
+    workflows: [CI]
+    types: [completed]
+    branches: [main]
   workflow_dispatch:
     inputs:
       target:
@@ -63,6 +65,13 @@ concurrency:
 jobs:
   sync:
     name: Deploy / Sync ${{ matrix.target.label }} Cloud Run
+    if: >
+      github.event_name == 'workflow_dispatch' ||
+      (
+        github.event_name == 'workflow_run' &&
+        github.event.workflow_run.conclusion == 'success' &&
+        github.event.workflow_run.event == 'push'
+      )
     runs-on: ubuntu-latest
     timeout-minutes: 20
     strategy:
@@ -79,6 +88,7 @@ jobs:
             environment: longbridge-sg
             default_account_region: SG
     permissions:
+      actions: read
       contents: read
       id-token: write
     environment: ${{ matrix.target.environment }}
@@ -257,6 +267,8 @@ jobs:
       - name: Checkout repository
         if: steps.config.outputs.enabled == 'true'
         uses: actions/checkout@v6
+        with:
+          ref: ${{ github.event_name == 'workflow_run' && github.event.workflow_run.head_sha || github.sha }}
 
       - name: Apply HK verify-only dispatch defaults
         if: steps.config.outputs.enabled == 'true' && github.event_name == 'workflow_dispatch' && inputs.target == 'hk-verify'

notifications/renderers.py

@@ -331,7 +331,22 @@ def _build_risk_control_lines(execution, *, translator):
     return []
 
 
-def _format_dashboard_text(text, *, translator=None) -> str:
+def _relabel_dashboard_buying_power(text: str, *, cash_only_execution: bool, translator) -> str:
+    value = str(text or "")
+    if cash_only_execution:
+        target = translator("buying_power")
+        for source in ("Buying power", "购买力"):
+            if source != target:
+                value = value.replace(source, target)
+        return value
+    target = translator("buying_power_margin")
+    for source in ("Available cash", "可用现金"):
+        if source != target:
+            value = value.replace(source, target)
+    return value
+
+
+def _format_dashboard_text(text, *, translator=None, cash_only_execution: bool = True) -> str:
     lines = []
     for raw_line in str(text or "").splitlines():
         line = raw_line.rstrip()
@@ -340,11 +355,23 @@ def _format_dashboard_text(text, *, translator=None) -> str:
         if translator is not None and _translator_uses_zh(translator):
             line = _localize_notification_text(line, translator=translator)
         lines.append(line)
-    return "\n".join(lines)
+    result = "\n".join(lines)
+    if translator is not None:
+        result = _relabel_dashboard_buying_power(
+            result,
+            cash_only_execution=cash_only_execution,
+            translator=translator,
+        )
+    return result
 
 
 def _append_dashboard_block(lines, *, execution, separator, translator) -> None:
-    dashboard_text = _format_dashboard_text(execution.get("dashboard_text"), translator=translator)
+    cash_only_execution = bool(execution.get("cash_only_execution", True))
+    dashboard_text = _format_dashboard_text(
+        execution.get("dashboard_text"),
+        translator=translator,
+        cash_only_execution=cash_only_execution,
+    )
     if dashboard_text:
         lines.append(separator)
         lines.extend(dashboard_text.splitlines())

tests/test_notifications.py

@@ -14,7 +14,7 @@
     build_strategy_display_name,
     build_translator,
 )
-from notifications.renderers import render_heartbeat_notification
+from notifications.renderers import render_heartbeat_notification, render_rebalance_notification
 from strategy_registry import SUPPORTED_STRATEGY_PROFILES
 
 
@@ -388,6 +388,44 @@ def test_heartbeat_localizes_strategy_diagnostics_and_source_input_status(self):
         self.assertNotIn("target_stock=", rendered.compact_text)
         self.assertNotIn("partial_history_refresh", rendered.compact_text)
 
+    def test_dashboard_relabels_buying_power_for_cash_only_execution(self):
+        rendered = render_rebalance_notification(
+            execution={
+                "dashboard_text": "  - 购买力: $100.00 | 可投资现金: $50.00",
+                "cash_only_execution": True,
+                "signal_display": "hold",
+                "status_display": "hold",
+            },
+            logs=(),
+            skip_logs=(),
+            note_logs=(),
+            translator=build_translator("zh"),
+            separator="━━━━━━━━━━━━━━━━━━",
+            strategy_display_name="demo",
+            dry_run_only=False,
+        )
+        self.assertIn("可用现金: $100.00", rendered.compact_text)
+        self.assertNotIn("购买力: $100.00", rendered.compact_text)
+
+    def test_dashboard_keeps_margin_buying_power_label_when_cash_only_disabled(self):
+        rendered = render_rebalance_notification(
+            execution={
+                "dashboard_text": "  - Available cash: $100.00 | Investable cash: $50.00",
+                "cash_only_execution": False,
+                "signal_display": "hold",
+                "status_display": "hold",
+            },
+            logs=(),
+            skip_logs=(),
+            note_logs=(),
+            translator=build_translator("en"),
+            separator="━━━━━━━━━━━━━━━━━━",
+            strategy_display_name="demo",
+            dry_run_only=False,
+        )
+        self.assertIn("Buying power: $100.00", rendered.compact_text)
+        self.assertNotIn("Available cash: $100.00", rendered.compact_text)
+
     def test_build_prefixer_prefers_account_prefix_only(self):
         with_prefix = build_prefixer("HK", "longbridge-quant-semiconductor-rotation-income-hk")
         self.assertEqual(with_prefix("hello"), "[HK] hello")

tests/test_rebalance_service.py

@@ -71,7 +71,7 @@ def _build_plan(
         dashboard_lines = [
             "📌 策略账户概览",
             f"  - 总资产（策略标的+现金）: ${float(total_strategy_equity):,.2f}",
-            f"  - 购买力: ${float(available_cash):.2f} | 可投资现金: ${float(investable_cash):.2f}",
+            f"  - 可用现金: ${float(available_cash):.2f} | 可投资现金: ${float(investable_cash):.2f}",
         ]
         nonzero_cash = {
             currency: amount
@@ -1899,7 +1899,7 @@ def test_zero_investable_cash_reports_buying_power_without_trade_note(self):
 
         self.assertEqual(len(sent_messages), 1)
         self.assertNotIn("账户现金", sent_messages[0])
-        self.assertIn("购买力: $3065.61 | 可投资现金: $0.00", sent_messages[0])
+        self.assertIn("可用现金: $3065.61 | 可投资现金: $0.00", sent_messages[0])
         self.assertIn("BOXX: $24,880.00 / 214股", sent_messages[0])
         self.assertIn("✅ 无需调仓", sent_messages[0])
         self.assertNotIn("本轮没有可执行订单", sent_messages[0])
@@ -2585,7 +2585,7 @@ def test_heartbeat_accepts_normalized_portfolio_and_execution_sections(self):
         self.assertNotIn("💵 资金\n  - 账户现金:", sent_messages[0])
         self.assertIn("📌 策略账户概览", sent_messages[0])
         self.assertIn("总资产（策略标的+现金）: $60,000.00", sent_messages[0])
-        self.assertIn("购买力: $101.95 | 可投资现金: $101.95", sent_messages[0])
+        self.assertIn("可用现金: $101.95 | 可投资现金: $101.95", sent_messages[0])
         self.assertIn("SOXX: $0.00 / 0股", sent_messages[0])
 
     def test_hybrid_heartbeat_hides_empty_semiconductor_fields_and_shows_benchmark_line(self):