Fallback: if Strategist output lacks a Current Price row, call get_price_change directly

The previous fallback only worked when the live price happened to
leak into the Chief Strategist's final answer. For NVDA the Strategist
emitted a long narrative without echoing the price through from the
Market Scanner, so the fallback returned None and the user saw 'Failed
to parse'. This version calls the get_price_change tool directly as a
backstop whenever the transcript scan fails — yfinance is on the same
machine so there's no real cost.

crew/crew.py

@@ -219,45 +219,84 @@ class FinAgentCrew:
     ) -> Optional[TradingSignal]:
         """Build a TradingSignal directly from tool outputs when LLM parsing fails.
 
-        The agents have already called ``get_price_change``, returning a
-        formatted block that contains the live price and the recent percent
-        change. Those lines also leak into the crew's full raw_output, so we
-        scan there for the canonical ``Current Price: $XXX.XX`` and
-        ``Change: +/-$X.XX (+/-YY.YY%)`` rows emitted by ``tools.market_scanner``.
+        First we try to scan ``raw_output`` for the canonical
+        ``Current Price: $X.XX`` row emitted by ``tools.market_scanner``
+        — that row leaks into the transcript when the agents call
+        ``get_price_change``. If the Strategist's final answer doesn't
+        include it (it often doesn't — the Strategist is the last agent
+        and only sees the prior agents' *summaries*), we call
+        ``get_price_change`` directly as a backstop so we still have a
+        live price to ground the card on.
+
+        From the live price we derive:
 
-        From those we derive:
         * **entry** = live price
         * **action** = BUY if today's change is ≥ +1 %, SELL if ≤ −1 %,
           otherwise HOLD
-        * **stop / target** = ± 3 % / ± 5 % of entry (per the signal parser's
-          sanity fix)
+        * **stop / target** = ± 3 % / ± 5 % of entry
         * **confidence** = 50 baseline, + up to 25 scaled by |% change|
         * **reasoning** = preserve the LLM's narrative (first ~800 chars)
 
-        Returns ``None`` only if no live price was ever retrieved.
+        Returns ``None`` only if no live price can be retrieved at all.
         """
+        entry: Optional[float] = None
+        pct_change = 0.0
+
+        # Try the transcript first (cheap — no extra network call).
         price_match = re.search(
             r"Current Price:\s*\$\s*([\d,]+\.?\d*)", raw_output
         )
-        if not price_match:
-            return None
-
-        try:
-            entry = float(price_match.group(1).replace(",", ""))
-        except ValueError:
-            return None
-
-        # Percent-change row, e.g. "Change: +$5.90 (+2.05%)"
-        pct_match = re.search(
-            r"Change:[^()]*\(([+-]?)([\d.]+)%\)", raw_output
-        )
-        pct_change = 0.0
-        if pct_match:
-            sign = -1.0 if pct_match.group(1) == "-" else 1.0
+        if price_match:
             try:
-                pct_change = sign * float(pct_match.group(2))
+                entry = float(price_match.group(1).replace(",", ""))
             except ValueError:
-                pct_change = 0.0
+                entry = None
+
+            pct_match = re.search(
+                r"Change:[^()]*\(([+-]?)([\d.]+)%\)", raw_output
+            )
+            if pct_match:
+                sign = -1.0 if pct_match.group(1) == "-" else 1.0
+                try:
+                    pct_change = sign * float(pct_match.group(2))
+                except ValueError:
+                    pct_change = 0.0
+
+        # Backstop: call get_price_change directly if the Strategist's
+        # response did not carry the price row through from upstream.
+        if entry is None:
+            try:
+                market_tools = self._tools.get("market_scanner", [])
+                price_tool = next(
+                    (t for t in market_tools
+                     if getattr(t, "name", "") == "Get Price Change"),
+                    None,
+                )
+                if price_tool is None:
+                    return None
+                # crewai wraps tools; the underlying function is .func.
+                price_fn = getattr(price_tool, "func", price_tool)
+                result = price_fn(ticker)
+                p = re.search(
+                    r"Current Price:\s*\$\s*([\d,]+\.?\d*)", str(result)
+                )
+                if not p:
+                    return None
+                entry = float(p.group(1).replace(",", ""))
+                pct = re.search(
+                    r"Change:[^()]*\(([+-]?)([\d.]+)%\)", str(result)
+                )
+                if pct:
+                    sign = -1.0 if pct.group(1) == "-" else 1.0
+                    try:
+                        pct_change = sign * float(pct.group(2))
+                    except ValueError:
+                        pct_change = 0.0
+            except Exception:
+                return None
+
+        if entry is None or entry <= 0:
+            return None
 
         # Choose action from today's trend.
         if pct_change >= 1.0: