Skip to content

fix: route web prediction prompts to market data - #21

Merged
VickyXAI merged 2 commits into
masterfrom
fix/web-prediction-market-routing
Jul 16, 2026
Merged

fix: route web prediction prompts to market data#21
VickyXAI merged 2 commits into
masterfrom
fix/web-prediction-market-routing

Conversation

@KillerQueen-Z

@KillerQueen-Z KillerQueen-Z commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • route explicit Polymarket and odds prompts to the prediction-markets capability
  • force the first Web tool call to Predexon instead of allowing a generic web-search detour
  • retain explicit tool-focus selection as the higher-priority override

Verification

franklin-run@1.0.0 typecheck
tsc --noEmit

franklin-run@1.0.0 lint
eslint

franklin-run@1.0.0 build
next build

▲ Next.js 16.2.1 (Turbopack)

Creating an optimized production build ...

  • verified Chinese Polymarket, English prediction-odds, and ordinary web-search routing cases

Review

  • /review pass (Claude structured + Claude adversarial + Codex cross-model): tightened the matcher in 1fb96cd — dropped bare odds/limitless/胜率/盘口 triggers (forced-paid-call false positives), fixed the prediction markets plural under-match, gated implied routing on a connected wallet, honest focus nudge for implied routing.

KillerQueen-Z and others added 2 commits July 16, 2026 11:03
…ll false positives

A regex match here forces a paid tool_choice call the model cannot
decline, so the matcher must be near-zero false positive. Review of
PR #21 found the opposite:

- bare "odds" fired on everyday English ("odds and ends", "odds
  ratio", "odds of rolling two sixes") and made "betting odds" dead
- bare "limitless" fired on the common adjective; now requires market
  context (limitless.exchange / limitless exchange|market(s))
- bare CJK 胜率/盘口/赔率 fired on game win rates, order-book slang and
  lottery questions; now require betting context (预测/押注/下注/博彩)
- the canonical plural "prediction markets" never matched because the
  trailing \b rejected the "s"; now prediction markets? and
  implied probabilit(y|ies) both match

Also from review:
- implied routing now only runs with a connected wallet, so wallet-less
  visitors asking a matching question get an answer instead of a
  wallet-connect wall
- the focus nudge no longer claims "the user selected" a capability the
  regex chose; on a residual false positive the model is told to ignore
  an irrelevant result and answer the actual question
@VickyXAI
VickyXAI merged commit 108fbb3 into master Jul 16, 2026
1 check passed
@VickyXAI
VickyXAI deleted the fix/web-prediction-market-routing branch July 16, 2026 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants