refactor(config): rename financial_quote_hints + provider_symbol; neutral logs

config/agent_config.json

@@ -576,7 +576,7 @@
   },
 
   "tool_routing": {
-    "description": "Executable tool policy and Yahoo Finance hints. domain_mapping.json = high-level domains/personas; this block = which tools run and how market queries map to symbols.",
+    "description": "Executable tool policy and market-quote hints. domain_mapping.json = high-level domains/personas; this block = which tools run and how questions map to provider symbols (backend may swap APIs).",
     "llm_router": {
       "min_cpu_exclusive": 4,
       "disable_env": "MEETARA_FAST",
@@ -586,24 +586,24 @@
     "market_quote_web_suppress": {
       "enabled": true,
       "unless_regex": "\\b(news|headlines?|articles?|stories|breaking|rumou?rs?|earnings|filing|why did|why is|fed |minutes |look up online|search for)\\b",
-      "note": "When financial + web_search both fired, skip DDG for pure quote/performance questions (Yahoo already has numbers)."
+      "note": "When financial + web_search both fired, skip DDG for pure quote/performance questions (market tool already returns levels)."
     },
-    "financial_yahoo_hints": [
-      { "label": "S&P 500", "yahoo_symbol": "^GSPC", "regex": "\\bs&p\\s*500\\b|\\bsp\\s*500\\b|\\bsp500\\b|\\bspx\\b" },
-      { "label": "S&P 500", "yahoo_symbol": "^GSPC", "regex": "\\bs\\s+and\\s+p\\s+500\\b" },
-      { "label": "S&P 500", "yahoo_symbol": "^GSPC", "regex": "\\bs\\s*&\\s*p\\b" },
-      { "label": "S&P 500", "yahoo_symbol": "^GSPC", "regex": "s&p" },
-      { "label": "Dow Jones", "yahoo_symbol": "^DJI", "regex": "\\bdow\\s+jones\\b|\\bdjia\\b|\\bthe\\s+dow\\b" },
-      { "label": "Dow Jones", "yahoo_symbol": "^DJI", "regex": "\\bdow\\b", "exclude_if_substring": ["download"] },
-      { "label": "NASDAQ", "yahoo_symbol": "^IXIC", "regex": "\\bnasdaq\\b" },
-      { "label": "Russell 2000", "yahoo_symbol": "^RUT", "regex": "\\brussell\\b" },
-      { "label": "VIX", "yahoo_symbol": "^VIX", "regex": "\\bvix\\b|fear\\s+index|volatility\\s+index" },
-      { "label": "BTC/USD", "yahoo_symbol": "BTC-USD", "regex": "\\bbitcoin\\b|\\bbtc\\b" },
-      { "label": "ETH/USD", "yahoo_symbol": "ETH-USD", "regex": "\\bethereum\\b|\\beth\\b" },
-      { "label": "Gold", "yahoo_symbol": "GC=F", "regex": "\\bgold\\b" },
-      { "label": "Oil (WTI)", "yahoo_symbol": "CL=F", "regex": "\\b(wti|crude\\s+oil)\\b" },
-      { "label": "Oil (WTI)", "yahoo_symbol": "CL=F", "regex": "\\boil\\b", "exclude_if_substring": ["spoil"] },
-      { "label": "EUR/USD", "yahoo_symbol": "EURUSD=X", "regex": "\\beur\\s*/\\s*usd\\b" }
+    "financial_quote_hints": [
+      { "label": "S&P 500", "provider_symbol": "^GSPC", "regex": "\\bs&p\\s*500\\b|\\bsp\\s*500\\b|\\bsp500\\b|\\bspx\\b" },
+      { "label": "S&P 500", "provider_symbol": "^GSPC", "regex": "\\bs\\s+and\\s+p\\s+500\\b" },
+      { "label": "S&P 500", "provider_symbol": "^GSPC", "regex": "\\bs\\s*&\\s*p\\b" },
+      { "label": "S&P 500", "provider_symbol": "^GSPC", "regex": "s&p" },
+      { "label": "Dow Jones", "provider_symbol": "^DJI", "regex": "\\bdow\\s+jones\\b|\\bdjia\\b|\\bthe\\s+dow\\b" },
+      { "label": "Dow Jones", "provider_symbol": "^DJI", "regex": "\\bdow\\b", "exclude_if_substring": ["download"] },
+      { "label": "NASDAQ", "provider_symbol": "^IXIC", "regex": "\\bnasdaq\\b" },
+      { "label": "Russell 2000", "provider_symbol": "^RUT", "regex": "\\brussell\\b" },
+      { "label": "VIX", "provider_symbol": "^VIX", "regex": "\\bvix\\b|fear\\s+index|volatility\\s+index" },
+      { "label": "BTC/USD", "provider_symbol": "BTC-USD", "regex": "\\bbitcoin\\b|\\bbtc\\b" },
+      { "label": "ETH/USD", "provider_symbol": "ETH-USD", "regex": "\\bethereum\\b|\\beth\\b" },
+      { "label": "Gold", "provider_symbol": "GC=F", "regex": "\\bgold\\b" },
+      { "label": "Oil (WTI)", "provider_symbol": "CL=F", "regex": "\\b(wti|crude\\s+oil)\\b" },
+      { "label": "Oil (WTI)", "provider_symbol": "CL=F", "regex": "\\boil\\b", "exclude_if_substring": ["spoil"] },
+      { "label": "EUR/USD", "provider_symbol": "EURUSD=X", "regex": "\\beur\\s*/\\s*usd\\b" }
     ]
   },
   

core/meetara_agent.py

@@ -44,13 +44,13 @@ except ImportError:
 
 try:
     from core.tool_routing import (
-        resolve_financial_yahoo_symbols,
+        resolve_financial_quote_symbols,
         should_run_llm_router,
         should_suppress_web_for_financial_quote,
     )
 except ImportError:
     from .tool_routing import (
-        resolve_financial_yahoo_symbols,
+        resolve_financial_quote_symbols,
         should_run_llm_router,
         should_suppress_web_for_financial_quote,
     )
@@ -1078,20 +1078,20 @@ def datetime_tool(query: str) -> str:
 
 
 # ── Financial Market Data ─────────────────────────────────────────────────
-# Yahoo symbol hints: config/agent_config.json → tool_routing.financial_yahoo_hints
+# Quote symbol hints: config/agent_config.json → tool_routing.financial_quote_hints
 
 
 def financial_data(query: str = "") -> str:
     """
-    Fetch real-time market prices from Yahoo Finance for major indices,
-    commodities, currencies, and crypto.
+    Fetch delayed market quotes (indices, FX, commodities, crypto).
+    Implementation uses the public Yahoo chart JSON endpoint today; symbols in
+    config are provider-neutral (`provider_symbol`) so the fetcher can be swapped.
 
     Args:
-        query: Optional symbol or market name to look up specifically.
-               Leave blank for the standard market overview.
+        query: Optional symbol or natural-language market question.
 
     Returns:
-        Formatted market snapshot string with prices and % changes.
+        Formatted snapshot with prices and % changes.
     """
     import re
     import urllib.request
@@ -1137,7 +1137,7 @@ def financial_data(query: str = "") -> str:
     if _q in _EXACT_ALIAS:
         symbols_to_fetch = [_EXACT_ALIAS[_q]]
     else:
-        nl = resolve_financial_yahoo_symbols(raw, _ac)
+        nl = resolve_financial_quote_symbols(raw, _ac)
         if nl:
             symbols_to_fetch = nl
         elif _q and len(raw) <= 16 and re.match(r"^[A-Z0-9=\-\.\^]+$", _q):
@@ -1194,9 +1194,9 @@ def financial_data(query: str = "") -> str:
             success += 1
 
     if success == 0:
-        return "Financial Data: Could not fetch market data (Yahoo Finance may be unavailable)."
+        return "Financial Data: Could not fetch market quotes (quote provider may be unavailable)."
 
-    lines.append(f"\n*Prices from Yahoo Finance. Delayed ~15 min.*")
+    lines.append(f"\n*Delayed quote (~15 min). Source: current deployment uses Yahoo chart data.*")
     result = "\n".join(lines)
     logger.info(f"[AGENT] 📈 Financial data: fetched {success}/{len(symbols_to_fetch)} symbols")
     return result
@@ -1297,7 +1297,7 @@ class MeeTARAAgent:
         # Financial Data Tool
         if 'financial_data' in globals():
             self.tools.append(financial_data)
-            logger.info("[AGENT] ✅ Financial Data tool loaded (Yahoo Finance)")
+            logger.info("[AGENT] ✅ Financial Data tool loaded (market quotes)")
 
         logger.info(f"[AGENT] ✅ meeTARA Agent loaded {len(self.tools)} tools (no smolagents needed)")
     
@@ -1789,7 +1789,7 @@ class MeeTARAAgent:
             needs["web_search"] = False
             needs["search_query"] = None
             logger.info(
-                "[AGENT] 📈 Skipping web search — quote-style market question (Yahoo only)"
+                "[AGENT] 📈 Skipping web search — quote-style question (market tool has levels)"
             )
 
         logger.debug(f"[AGENT] 🔍 Final detection result: {needs}")

core/tool_routing.py

@@ -26,11 +26,12 @@ def _routing_section(agent_config: Optional[Dict[str, Any]] = None) -> Dict[str,
         return {}
 
 
-def resolve_financial_yahoo_symbols(
+def resolve_financial_quote_symbols(
     query: str, agent_config: Optional[Dict[str, Any]] = None
 ) -> List[Tuple[str, str]]:
-    """Return [(label, yahoo_symbol), ...] from financial_yahoo_hints in agent_config."""
-    hints = _routing_section(agent_config).get("financial_yahoo_hints") or []
+    """Return [(label, provider_symbol), ...] from tool_routing.financial_quote_hints."""
+    sec = _routing_section(agent_config)
+    hints = sec.get("financial_quote_hints") or sec.get("financial_yahoo_hints") or []
     if not hints:
         return []
 
@@ -48,13 +49,19 @@ def resolve_financial_yahoo_symbols(
             if not re.search(rx, ql, re.IGNORECASE):
                 continue
         except re.error as ex:
-            logger.warning(f"[tool_routing] bad financial_yahoo_hints regex: {ex}")
+            logger.warning(f"[tool_routing] bad financial_quote_hints regex: {ex}")
             continue
         for bad in rule.get("exclude_if_substring") or []:
             if str(bad).lower() in ql:
                 break
         else:
-            sym = (rule.get("yahoo_symbol") or rule.get("yahoo") or "").strip()
+            sym = (
+                rule.get("provider_symbol")
+                or rule.get("yahoo_symbol")
+                or rule.get("symbol")
+                or rule.get("yahoo")
+                or ""
+            ).strip()
             label = (rule.get("label") or sym or "?").strip()
             if sym and sym not in seen:
                 seen.add(sym)
@@ -92,3 +99,7 @@ def should_run_llm_router(ncpu: int, agent_config: Optional[Dict[str, Any]] = No
         return True
     min_exc = int(lr.get("min_cpu_exclusive", 4))
     return ncpu > min_exc
+
+
+# Backward-compatible name (older docs / forks)
+resolve_financial_yahoo_symbols = resolve_financial_quote_symbols