cito-mcp 0.3.12 → 0.3.13

package/dist/tools/insight.js

@@ -47,10 +47,14 @@ async function loadSide(ctx, game, side, recentLimit, includeRosters) {
             recentForm = extractRows(unwrapPayload(fights.data) ?? fights.data)
                 .slice(0, recentLimit)
                 .map((row) => {
-                // history rows nest bout
+                // History rows nest the bout and carry the fighter's own side as
+                // fighterSlug/opponent/outcome. Merge so normalizeMatch sees both,
+                // and do NOT force 'completed': the fighter's next scheduled bout is
+                // in this list too, and stamping it completed put an unfought fight
+                // at the top of the form with no result.
                 const r = asRecord(row) ?? {};
-                const bout = asRecord(r.bout) ?? r;
-                return normalizeMatch('ufc', { ...bout, ...(r.fighterName ? {} : {}) }, 'completed');
+                const bout = asRecord(r.bout) ?? {};
+                return normalizeMatch('ufc', { ...bout, ...r });
             });
         }
         const stats = await fetchJson(ctx, `/ufc/fighters/${encodeURIComponent(side)}/stats`);
@@ -201,7 +205,7 @@ async function loadSide(ctx, game, side, recentLimit, includeRosters) {
     }
     return { entity, roster, recentForm, keyPlayers, calls, partial, rateLimit };
 }
-function composeH2H(game, sideA, sideB, rows, limit) {
+export function composeH2H(game, sideA, sideB, rows, limit) {
     const al = sideA.toLowerCase();
     const bl = sideB.toLowerCase();
     const meetings = rows
@@ -406,8 +410,10 @@ Example: { "game": "lol", "teamA": "t1", "teamB": "gen-g", "includeH2H": true, "
                     rateLimit = { ...rateLimit, ...hist.headers };
                     if (hist.ok) {
                         rows = extractRows(unwrapPayload(hist.data) ?? hist.data).map((row) => {
+                            // Keep fighterSlug/opponent/outcome alongside the bout; they are
+                            // the only place the two corners exist on a history row.
                             const r = asRecord(row) ?? {};
-                            return asRecord(r.bout) ?? row;
+                            return { ...(asRecord(r.bout) ?? {}), ...r };
                         });
                     }
                     else {
@@ -757,6 +763,11 @@ Example: { "game": "ufc", "eventIdOrSlug": "ufc-300", "includeBouts": true, "inc
                     }));
                 }
             }
+            // Bout rows carry no start time of their own; inherit the card's. Without
+            // this the same bout said startTime:null here and the event time on
+            // upcoming_schedule, and a caller had to fetch both to learn when it is.
+            const cardStart = typeof event.date === 'string' ? event.date : null;
+            bouts = bouts.map((b) => ({ ...b, startTime: b.startTime ?? cardStart }));
             if (includeStandings) {
                 const ranks = await fetchJson(ctx, '/ufc/rankings');
                 upstreamCalls += 1;

package/dist/tools/live.js

@@ -77,10 +77,16 @@ export function extractLiveRows(data, game) {
 export function inferUfcEmptyReason(events, health, liveCount) {
     if (liveCount > 0)
         return undefined;
+    const pulses = events.map((ev) => asRecord(ev) ?? {});
+    const hotNow = pulses.some((p) => ['live', 'warming', 'between_bouts', 'degraded'].includes(String(pickString(p.status, p.eventStatus) ?? '').toLowerCase()));
+    // The API's own health block is trusted first, EXCEPT when it blames the
+    // worker on a day nothing is live. It computes worker_stale from heartbeat
+    // lag alone, so an idle Thursday reads the same as a dead worker on fight
+    // night. With no hot card there is nothing to be stale about.
     const fromHealth = pickString(health?.emptyReason);
-    if (fromHealth)
+    if (fromHealth && !(['worker_stale', 'worker_down_inferred'].includes(fromHealth) && !hotNow)) {
         return fromHealth;
-    const pulses = events.map((ev) => asRecord(ev) ?? {});
+    }
     if (!pulses.length) {
         const alive = health?.workerAlive ?? health?.workerStarted;
         if (alive === false)
@@ -94,9 +100,13 @@ export function inferUfcEmptyReason(events, health, liveCount) {
     const freshest = lags.length ? Math.min(...lags) : null;
     const workerAlive = health?.workerAlive ?? health?.workerStarted;
     const hot = statuses.some((s) => ['live', 'warming', 'between_bouts', 'degraded'].includes(s));
-    if (workerAlive === false && (hot || (freshest != null && freshest > 120))) {
+    // Only a HOT card can be stale. On an idle day every event is "scheduled"
+    // and the heartbeat lags because there is nothing to beat about; reporting
+    // that as worker_stale made a quiet Thursday indistinguishable from a dead
+    // worker on fight night. freshest stays computed for the health block.
+    void freshest;
+    if (workerAlive === false && hot)
         return 'worker_stale';
-    }
     if (statuses.some((s) => s === 'degraded'))
         return 'event_degraded';
     if (statuses.some((s) => s === 'warming'))
@@ -126,8 +136,10 @@ export function humanUfcEmptyNote(emptyReason, health) {
         supervisor_warming: 'Card open / pre-gate; no bout live yet',
         between_bouts: 'Between bouts; next may be armed',
         armed_only: 'Next bout armed; not started',
-        worker_stale: 'Worker heartbeat stale — check pm2 ufc-live on VPS 169',
-        worker_down_inferred: 'Worker appears down — check pm2 ufc-live on VPS 169',
+        // Customer-facing. The pm2/VPS operator hint used to be in here and went
+        // out in every empty board; health{} already carries the numbers for us.
+        worker_stale: 'Live feed is behind; scores may lag',
+        worker_down_inferred: 'Live feed unavailable right now',
         event_degraded: 'Sources empty clock / event degraded; not inventing stats',
     };
     const msg = human[reason] || `No live|watching bouts (${reason})`;
@@ -284,9 +296,14 @@ Example: { "game": "all", "limitPerGame": 10 }`,
             if (game === 'ufc') {
                 const health = asRecord(meta?.health) ?? asRecord(payload?.health);
                 const events = Array.isArray(payload?.events) ? payload.events : [];
+                // One decision point. This used to short-circuit on the API's own
+                // emptyReason and only fall back to inferUfcEmptyReason when the API
+                // said nothing, so the idle-day rule inside it never ran: the API
+                // blames the worker on heartbeat lag alone and an empty Thursday came
+                // back as worker_stale. Hand the hint in and let the function weigh it.
+                const upstreamReason = pickString(meta?.emptyReason, health?.emptyReason);
                 const emptyReason = normalized.length === 0
-                    ? pickString(meta?.emptyReason, health?.emptyReason) ??
-                        inferUfcEmptyReason(events, health, normalized.length)
+                    ? inferUfcEmptyReason(events, { ...(health ?? {}), ...(upstreamReason ? { emptyReason: upstreamReason } : {}) }, normalized.length)
                     : undefined;
                 let note;
                 if (normalized.length === 0) {

package/dist/tools/normalize.js

@@ -119,9 +119,17 @@ function sideExtras(o, game) {
         ...(rank ? { rank } : {}),
         // "none" carries no signal for a page; only surface an actual belt state.
         ...(championStatus && championStatus.toLowerCase() !== 'none' ? { championStatus } : {}),
-        ...(division ? { division } : {}),
-        ...(country ? { country } : {}),
-        ...(flag ? { flag } : {}),
+        // UFC corners always carry these keys. A debutant with no ranked division
+        // was shipping with the key ABSENT while every other corner had it, and a
+        // fighter from a country with no flag mapping did the same, which breaks
+        // strict parsers. Null says "not known"; absence says nothing.
+        ...(game === 'ufc'
+            ? { division: division ?? null, country: country ?? null, flag: flag ?? null }
+            : {
+                ...(division ? { division } : {}),
+                ...(country ? { country } : {}),
+                ...(flag ? { flag } : {}),
+            }),
         ...(outcome ? { outcome } : {}),
     };
 }
@@ -151,6 +159,40 @@ function scoreNum(v) {
         return Number(v);
     return null;
 }
+/**
+ * Collapse the six spellings one outcome arrives in ("SUB" / "Submission",
+ * "U-DEC" / "Decision - Unanimous", "KO/TKO" / "TKO") onto one enum. The raw
+ * string is kept alongside as methodRaw; this is a stable key for consumers,
+ * not a rewrite of the source.
+ */
+export function normalizeUfcMethod(raw) {
+    if (!raw)
+        return undefined;
+    const s = raw.trim().toLowerCase();
+    if (!s)
+        return undefined;
+    if (/overturn/.test(s))
+        return 'Overturned';
+    if (/no[\s-]?contest|^nc$/.test(s))
+        return 'NC';
+    if (/^dq$|disqualif/.test(s))
+        return 'DQ';
+    if (/draw/.test(s))
+        return 'Draw';
+    if (/sub/.test(s))
+        return 'SUB';
+    if (/ko|tko|knock|stoppage|strikes|punch|kick|elbow|knee/.test(s))
+        return 'KO/TKO';
+    if (/unanim|^u-?dec$/.test(s))
+        return 'U-DEC';
+    if (/split|^s-?dec$/.test(s))
+        return 'S-DEC';
+    if (/major|^m-?dec$/.test(s))
+        return 'M-DEC';
+    if (/dec/.test(s))
+        return 'DEC';
+    return raw.trim();
+}
 export function normalizeMatch(game, row, forcedStatus) {
     // Always peel { success, data } so UFC bout fighters[] / status are visible.
     const r = asRecord(unwrapPayload(row)) ?? asRecord(row) ?? {};
@@ -216,7 +258,11 @@ export function normalizeMatch(game, row, forcedStatus) {
             const profile = asRecord(o.profile);
             const fighter = asRecord(o.fighter);
             const name = pickString(o.name, o.fighterName, o.displayName, o.nickname, profile?.name, profile?.nickname, fighter?.name, fighter?.nickname);
-            const id = pickString(o.id, o.fighterId, profile?.id, fighter?.id);
+            // o.id on a serialized corner is the ufc_bout_fighters ROW id, unique per
+            // bout, so the same human got a different "id" on every card. Prefer the
+            // fighter's own id wherever the API supplies it; fall back to the row id
+            // only when nothing better exists.
+            const id = pickString(o.fighterId, profile?.id, fighter?.id, o.id);
             const slug = pickString(o.slug, o.fighterSlug, profile?.slug, fighter?.slug);
             const score = scoreNum(o.score ?? o.points);
             return sideFrom(name, id, slug, score, sideExtras(o, game));
@@ -247,6 +293,31 @@ export function normalizeMatch(game, row, forcedStatus) {
             team2 = team2 ?? fromCornerObj(blueF);
         }
     }
+    // UFC fighter-history rows (/ufc/fighters/{slug}/fights). These carry the
+    // fighter as fighterSlug/fighterName, the other corner under opponent, and
+    // the result as outcome, with NO fighters[] and no team1/team2. Every reader
+    // was left with two null sides: head_to_head filtered on those nulls and
+    // reported Jones vs Cormier as never having met, the profile showed
+    // team1:null with the division as the label, and the preview form could not
+    // say who won. One shape, three symptoms.
+    if (game === 'ufc' && !team1 && !team2 && typeof r.fighterSlug === 'string') {
+        const opp = asRecord(r.opponent);
+        const selfCorner = pickString(r.corner)?.toLowerCase();
+        const self = sideFrom(pickString(r.fighterName) ?? String(r.fighterSlug), undefined, String(r.fighterSlug), undefined, sideExtras(r, game));
+        const other = opp
+            ? sideFrom(pickString(opp.name) ?? pickString(opp.slug), undefined, pickString(opp.slug), undefined, sideExtras(opp, game))
+            : null;
+        // Keep red on team1 when we know the corner, so a card and a history row
+        // agree on which side is which.
+        if (selfCorner === 'blue' && other) {
+            team1 = other;
+            team2 = self;
+        }
+        else {
+            team1 = self;
+            team2 = other;
+        }
+    }
     // COD sometimes uses teams[]
     if ((!team1 || !team2) && Array.isArray(r.teams)) {
         team1 = team1 ?? nestedSide(r.teams[0], game);
@@ -296,9 +367,20 @@ export function normalizeMatch(game, row, forcedStatus) {
         else if (hasResult)
             status = 'completed';
     }
+    // The winner, resolved once. Explicit winner fields first; otherwise a
+    // history row states the result from the fighter's own side (outcome: win /
+    // loss), which translates back into a slug. Both the score below and the
+    // result block use this, so a fight the profile shows as a win also counts
+    // as a win in head_to_head's tally, which reads the scores.
+    const historyOutcome = pickString(r.outcome)?.toLowerCase();
+    const historyOpponent = pickString(asRecord(r.opponent)?.slug);
+    const resolvedWinnerSlug = pickString(r.winnerFighterSlug, r.winnerSlug, r.winner) ??
+        (historyOutcome === 'win' && typeof r.fighterSlug === 'string' ? r.fighterSlug
+            : historyOutcome === 'loss' && historyOpponent ? historyOpponent
+                : undefined);
     // Winner → score 1-0 for UFC card display when numeric scores absent
     if (game === 'ufc' && team1 && team2 && team1.score == null && team2.score == null) {
-        const winner = pickString(r.winnerFighterSlug, r.winnerSlug, r.winner)?.toLowerCase();
+        const winner = resolvedWinnerSlug?.toLowerCase();
         if (winner) {
             const t1hit = [team1.slug, team1.id, team1.name].some((x) => x && (String(x).toLowerCase() === winner || String(x).toLowerCase().includes(winner) || winner.includes(String(x).toLowerCase())));
             const t2hit = [team2.slug, team2.id, team2.name].some((x) => x && (String(x).toLowerCase() === winner || String(x).toLowerCase().includes(winner) || winner.includes(String(x).toLowerCase())));
@@ -324,7 +406,14 @@ export function normalizeMatch(game, row, forcedStatus) {
     // When both sides lack names, prefer event / weight / bout id over opaque "? vs ?"
     // so agent boards (live_matches, upcoming_schedule, event_card) stay legible.
     const vsLabel = `${team1?.name ?? '?'} vs ${team2?.name ?? '?'}`;
-    const explicit = pickString(r.label, r.title, r.name);
+    const explicitRaw = pickString(r.label, r.title, r.name);
+    // UFC history rows put the division where a title would go ("Heavyweight"),
+    // which then displaced "Jon Jones vs Stipe Miocic" as the label. A label that
+    // merely repeats the weight class is not a matchup; use the sides instead.
+    const explicit = game === 'ufc' && explicitRaw && (team1?.name || team2?.name) &&
+        explicitRaw.toLowerCase() === (pickString(r.weightClass, r.division) ?? '').toLowerCase()
+        ? undefined
+        : explicitRaw;
     const explicitIsPlaceholder = explicit != null && /^\?\s*vs\s*\?$/i.test(explicit.trim());
     const weightOrClass = pickString(r.weightClass, r.division, r.weight_class, r.boutClass);
     let label;
@@ -348,12 +437,13 @@ export function normalizeMatch(game, row, forcedStatus) {
     const cardSectionOrder = numOrNull(r.cardSectionOrder);
     const boutOrder = numOrNull(r.boutOrder);
     const hasCard = cardSection != null || cardPosition != null || cardSectionOrder != null || boutOrder != null;
-    const method = pickString(r.method);
+    const methodRaw = pickString(r.method);
+    const method = normalizeUfcMethod(methodRaw);
     const methodDetails = pickString(r.methodDetails);
     const resultTime = pickString(r.resultTime);
     // UFC sends referee as { id, name, firstName, lastName } — not a string.
     const referee = pickString(r.referee, asRecord(r.referee)?.name);
-    const winnerSlug = pickString(r.winnerFighterSlug, r.winnerSlug, r.winner);
+    const winnerSlug = resolvedWinnerSlug;
     const resultRound = numOrNull(r.resultRound);
     const hasResultDetail = method != null || resultRound != null || winnerSlug != null || resultTime != null;
     const isCancelled = r.isCancelled === true;
@@ -383,6 +473,7 @@ export function normalizeMatch(game, row, forcedStatus) {
             ? {
                 result: {
                     method: method ?? null,
+                    methodRaw: methodRaw ?? null,
                     methodDetails: methodDetails ?? null,
                     round: resultRound,
                     time: resultTime ?? null,

package/dist/tools/player.js

@@ -479,7 +479,11 @@ Example: { "game": "cs2", "playerId": "cs2-player-1", "recentLimit": 10, "includ
                         .slice(0, recentLimit)
                         .map((row) => {
                         const r = asRecord(row) ?? {};
-                        const bout = asRecord(r.bout) ?? row;
+                        // Merge the row's own corner fields (fighterSlug / opponent /
+                        // outcome) over the nested bout: they are the only place the two
+                        // sides exist on a history row, and without them every entry
+                        // came back team1:null, team2:null with the division as label.
+                        const bout = { ...(asRecord(r.bout) ?? {}), ...r };
                         // Never force 'completed'. A fighter's history endpoint also
                         // returns bouts that are booked but not yet fought, and forcing
                         // the status told agents that a future main event had already
@@ -488,11 +492,16 @@ Example: { "game": "cs2", "playerId": "cs2-player-1", "recentLimit": 10, "includ
                         // reported as completed with result null). Let normalizeMatch
                         // derive it from the result and the start time instead.
                         const m = normalizeMatch('ufc', bout);
+                        // Keep the normalized result OBJECT (method, round, time,
+                        // winnerSlug), the same shape match_preview and match_summary
+                        // return. This used to overwrite it with the string "win",
+                        // giving the profile a fourth result schema of its own. The
+                        // fighter's-eye view lives on outcome instead.
                         return {
                             ...m,
-                            result: pickString(r.result, r.outcome, asRecord(bout)?.result) ?? null,
+                            outcome: pickString(r.outcome)?.toLowerCase() ?? null,
                             opponent: pickString(r.opponentName, asRecord(r.opponent)?.name) ?? null,
-                            method: pickString(asRecord(bout)?.method, r.method) ?? null,
+                            method: m.result?.method ?? null,
                         };
                     });
                 }
@@ -508,7 +517,7 @@ Example: { "game": "cs2", "playerId": "cs2-player-1", "recentLimit": 10, "includ
                             .slice(0, recentLimit)
                             .map((row) => {
                             const r = asRecord(row) ?? {};
-                            const bout = asRecord(r.bout) ?? row;
+                            const bout = { ...(asRecord(r.bout) ?? {}), ...r };
                             // Same as above: derive, never assert. See the note there.
                             return normalizeMatch('ufc', bout);
                         });

package/dist/tools/standings.js

@@ -21,7 +21,10 @@ export function normalizeStandingRow(row, index) {
         isInterim;
     let rank;
     if (isChampion) {
-        rank = isInterim || rankText === 'IC' ? 'IC' : 'C';
+        // rank is numeric or null; the belt lives in rankText ("C"/"IC") and
+        // championStatus. Mixing a string into a numeric column forced every
+        // consumer to special-case it.
+        rank = null;
     }
     else if (typeof r.rank === 'number') {
         rank = r.rank;
@@ -314,7 +317,11 @@ Example: { "game": "cod", "season": "2026", "limit": 50 }`,
                 rateLimit: res.headers,
             });
         }
-        let rows = extractRows(res.data).map((row, i) => normalizeStandingRow(row, i)).slice(0, limit);
+        // Keep the unsliced count. UFC world scope is every division back to back,
+        // so limit=50 stops a third of the way through flyweight; a consumer needs
+        // to know that happened rather than receive total:null and guess.
+        let allRows = extractRows(res.data).map((row, i) => normalizeStandingRow(row, i));
+        let rows = allRows.slice(0, limit);
         // UFC rankings may be nested by division
         if (game === 'ufc' && rows.length === 0) {
             const obj = asRecord(res.data) ?? {};
@@ -326,7 +333,8 @@ Example: { "game": "cod", "season": "2026", "limit": 50 }`,
                     nested.push(...asRecord(value).rankings);
                 }
             }
-            rows = nested.map((row, i) => normalizeStandingRow(row, i)).slice(0, limit);
+            allRows = nested.map((row, i) => normalizeStandingRow(row, i));
+            rows = allRows.slice(0, limit);
         }
         const obj = asRecord(res.data);
         const upstreamMeta = asRecord(obj?.meta) ?? {};
@@ -369,6 +377,8 @@ Example: { "game": "cod", "season": "2026", "limit": 50 }`,
                 season: season ?? null,
                 stage: stage ?? null,
                 rows,
+                total: allRows.length,
+                hasMore: allRows.length > rows.length,
                 updatedAt: pickString(obj?.updatedAt, obj?.lastUpdated, upstreamMeta.syncedAt, upstreamMeta.fetchedAt) ??
                     null,
                 ...(dataFreshness ? { dataFreshness } : {}),

package/dist/tools/team.js

@@ -829,8 +829,11 @@ Example: { "game": "cs2", "sideA": "faze", "sideB": "navi", "limit": 20 }`,
                 });
             }
             const toBouts = (payload) => extractRows(payload).map((row) => {
+                // History rows carry the two corners as fighterSlug/opponent, not on
+                // the nested bout. Dropping them here is why Jones vs Cormier reported
+                // zero meetings while the profile listed both fights.
                 const r = asRecord(row) ?? {};
-                return asRecord(r.bout) ?? row;
+                return { ...(asRecord(r.bout) ?? {}), ...r };
             });
             const fromA = histA.ok ? toBouts(histA.data) : [];
             const fromB = histB.ok ? toBouts(histB.data) : [];

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "cito-mcp",
-  "version": "0.3.12",
-  "description": "Standalone MCP server for the Cito esports API — 15 curated outcome tools for agents (live, schedule, profiles, standings, previews, event cards).",
+  "version": "0.3.13",
+  "description": "Standalone MCP server for the Cito esports API \u2014 15 curated outcome tools for agents (live, schedule, profiles, standings, previews, event cards).",
   "type": "module",
   "bin": {
     "cito-mcp": "dist/index.js"