Files
polymarket-bot/bot/data/news.py
T
chemavxandClaude Opus 4.8 54fc8fa11a fix(news): strip GNews operator dashes and stop phantom query-budget counting
Two minor faults found during the GNews capture/prioritisation diagnostic:

1. Hyphens/dashes reached the GNews query verbatim. '-' is GNews's exclusion
   operator, so a token like "El-Sayed" returned HTTP 400 and wasted a query.
   _PUNCT_RE now strips '-', en dash and em dash to spaces.

2. The per-cycle GNews budget counter incremented in evaluate() before
   get_sentiment() checked the API key, so with no key configured the
   [CYCLE SUMMARY] reported a phantom "gnews_queries_used: 5/5" with zero real
   requests. Added NewsClient.enabled and gated the GNews block on it; with no
   key the counter stays 0/5 and no spurious SKIP_GNEWS_PRIORITY is logged.
   No behaviour change when a key is present.

Prioritisation itself was confirmed correct and is left untouched: politics
markets are sorted by gnews_priority DESC and prior-extreme markets return
before the budget is consumed, so no query is ever spent on a market that
cannot trade.

Tests: tests/test_news_query.py (4 new); full suite 66 passed.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-06-26 08:07:05 +00:00

253 lines
9.5 KiB
Python
Raw Blame History

This file contains ambiguous Unicode characters
This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.
"""
News sentiment client for GNews API.
Free tier: 100 requests/day. Budget:
- Cache TTL: 6 hours — same query is never repeated within 6 h
- Max 5 queries per trading cycle (politics markets only)
- 2-second sleep between actual API calls to avoid burst 429s
With ≤9 politics markets and 6 h cache → ≤9 requests per 6 h = ≤36/day.
Sentiment engine: VADER (Valence Aware Dictionary and sEntiment Reasoner).
Designed for short social/news text — handles negations, intensifiers, and
punctuation natively. Returns compound ∈ [-1, +1] per article.
Score returned: -1.0 (very negative headlines) → +1.0 (very positive).
Returns 0.0 on any error or missing API key so the caller degrades gracefully.
"""
import asyncio
import logging
import os
import re
import time
import httpx
from vaderSentiment.vaderSentiment import SentimentIntensityAnalyzer
log = logging.getLogger(__name__)
GNEWS_API = "https://gnews.io/api/v4/search"
CACHE_TTL = 6 * 3600 # seconds — ≤9 politics markets × 4 cycles/day = ≤36 req/day
_INTER_REQUEST_SLEEP = 2 # seconds between consecutive real API calls
_NEUTRAL_THRESHOLD = 0.05 # |compound| below this → article excluded from average
# Shared VADER analyzer (stateless, thread-safe, cheap to create once)
_vader = SentimentIntensityAnalyzer()
# Words stripped when building the search query (too generic to help relevance)
_QUERY_STOPWORDS = {
"will", "the", "a", "an", "by", "in", "on", "at", "to", "of",
"and", "or", "is", "be", "are", "was", "were", "have", "has",
"had", "do", "does", "did", "for", "from", "with", "not", "no",
"this", "that", "it", "its", "their", "they", "he", "she", "we",
"most", "more", "least", "less", "any", "all", "both", "each",
"win", "lose", "get", "make", "take",
}
# Regex patterns for dates / noise in market questions
_DATE_RE = re.compile(
r"\b(january|february|march|april|may|june|july|august|"
r"september|october|november|december)\s+\d{1,2}\b"
r"|\b20\d{2}\b"
r"|\bQ[1-4]\b",
flags=re.IGNORECASE,
)
# Hyphens/dashes are GNews query operators (a leading '-' means "exclude the
# next term"), so a token like "El-Sayed" makes the API return HTTP 400. Strip
# them to spaces along with the rest of the punctuation so the query stays a
# plain keyword list. = en dash, — = em dash.
_PUNCT_RE = re.compile(r"[?!\"'.,;:()\[\]{}\-–—]")
class NewsClient:
"""
Async GNews client with in-memory result cache.
Usage::
client = NewsClient()
score = await client.get_sentiment("Will Trump visit China")
# score ∈ [-1.0, +1.0] — positive means bullish for the YES outcome
await client.close()
"""
def __init__(self) -> None:
self._api_key = os.getenv("GNEWS_API_KEY", "")
self._client = httpx.AsyncClient(
timeout=10,
headers={"User-Agent": "Mozilla/5.0 (compatible; polymarket-bot/1.0)"},
)
# {cache_key: (fetched_at_monotonic, score)}
self._cache: dict[str, tuple[float, float]] = {}
# ------------------------------------------------------------------
# Public API
# ------------------------------------------------------------------
@property
def enabled(self) -> bool:
"""True only when a GNews API key is configured.
When False, get_sentiment() is a no-op that returns 0.0 without any
network call, so callers must skip GNews entirely — including the
per-cycle query budget accounting — instead of "spending" a query that
never reaches the API (which inflated gnews_queries_used to a phantom
5/5 while the key was missing).
"""
return bool(self._api_key)
async def get_sentiment(self, question: str) -> float:
"""
Return a sentiment score ∈ [-1.0, +1.0] for the market question.
- Positive: most recent headlines suggest the YES outcome is more likely
- Negative: headlines suggest the YES outcome is less likely
- 0.0: neutral, no data, or API unavailable
"""
if not self._api_key:
log.debug("GNEWS_API_KEY not set — skipping news signal")
return 0.0
query = self._build_query(question)
if len(query) < 3:
return 0.0
cache_key = query.lower()
now = time.monotonic()
cached = self._cache.get(cache_key)
if cached is not None:
fetched_at, score = cached
if now - fetched_at < CACHE_TTL:
log.debug("News cache hit %r%.3f", query, score)
return score
# Build URL exactly as documented for free tier:
# https://gnews.io/api/v4/search?q=...&lang=en&max=10&token=...
# NOTE: "from"/"to" date filters are paid-tier only — omit them.
try:
resp = await self._client.get(
GNEWS_API,
params={
"q": query,
"lang": "en",
"max": 10,
"token": self._api_key,
},
)
except Exception as exc:
log.warning("GNews network error for %r: %s", query, exc)
return 0.0
finally:
# Always sleep after a real network attempt to avoid burst 429s
await asyncio.sleep(_INTER_REQUEST_SLEEP)
log.info("GNews HTTP %d for query %r", resp.status_code, query)
if resp.status_code != 200:
try:
body = resp.json()
except Exception:
body = resp.text[:200]
log.warning("GNews error body: %s", body)
# Cache neutral for 1 h on client errors to avoid hammering the endpoint
if resp.status_code in (400, 401, 403, 429):
self._cache[cache_key] = (now, 0.0)
return 0.0
try:
data = resp.json()
except Exception as exc:
log.warning("GNews JSON decode error for %r: %s", query, exc)
return 0.0
articles = data.get("articles", [])
score = self._score_headlines(articles)
self._cache[cache_key] = (now, score)
log.info(
"GNews %r%d articles, sentiment=%.3f",
query, len(articles), score,
)
return score
async def close(self) -> None:
await self._client.aclose()
def get_freshness(self, question: str) -> float:
"""
Return a freshness score [0.1, 1.0] for GNews priority calculation.
Score interpretation:
1.00 — never queried (maximum priority for GNews budget)
0.75 — last queried >6 h ago (cache expired, worth re-querying)
0.40 — queried 26 h ago (in-cache but moderately stale)
0.10 — queried <2 h ago (cache very fresh, low re-query value)
If the API key is absent, always returns 1.0 (key missing means the
query will be skipped anyway; don't penalise the priority score).
"""
if not self._api_key:
return 1.0
query = self._build_query(question)
cached = self._cache.get(query.lower())
if cached is None:
return 1.0
age_seconds = time.monotonic() - cached[0]
if age_seconds > 6 * 3600:
return 0.75
if age_seconds > 2 * 3600:
return 0.40
return 0.10
# ------------------------------------------------------------------
# Internal helpers
# ------------------------------------------------------------------
@staticmethod
def _build_query(question: str) -> str:
"""Extract meaningful search terms from a market question."""
q = _DATE_RE.sub(" ", question)
q = _PUNCT_RE.sub(" ", q)
tokens = [
w for w in q.split()
if w.lower() not in _QUERY_STOPWORDS and len(w) > 2
]
return " ".join(tokens[:8]) # GNews handles ~8 keyword queries well
@staticmethod
def _score_headlines(articles: list[dict]) -> float:
"""
Score articles with VADER, weight by recency, return ∈ [-1, +1].
Algorithm:
1. For each article: run VADER on title + description → compound ∈ [-1, +1]
2. Exclude articles with |compound| ≤ _NEUTRAL_THRESHOLD (no clear signal)
3. GNews returns articles newest-first, so assign linear recency weights:
article[0] → weight N, article[1] → weight N-1, ..., article[N-1] → weight 1
(only counted for articles that passed the threshold filter)
4. Return weighted mean, clamped to [-1, +1]
"""
if not articles:
return 0.0
scored: list[tuple[float, int]] = [] # (compound, original_index)
for idx, art in enumerate(articles):
text = f"{art.get('title', '')} {art.get('description', '')}"
compound = _vader.polarity_scores(text)["compound"]
if abs(compound) > _NEUTRAL_THRESHOLD:
scored.append((compound, idx))
if not scored:
return 0.0
n = len(scored)
# Weight by recency: earlier index (newer article) → higher weight.
# scored is already in original (newest-first) order since we enumerate
# articles sequentially and only append those that pass the threshold.
total_weight = 0.0
weighted_sum = 0.0
for rank, (compound, _idx) in enumerate(scored):
weight = n - rank # rank 0 → weight n, rank n-1 → weight 1
weighted_sum += compound * weight
total_weight += weight
return max(-1.0, min(1.0, weighted_sum / total_weight))