fitzroy 1.1.1 → 1.3.0

package/dist/index.js

@@ -20,6 +20,12 @@ var UnsupportedSourceError = class extends Error {
   }
   name = "UnsupportedSourceError";
 };
+function aflwUnsupportedError(source) {
+  return new UnsupportedSourceError(
+    `AFLW data is not available from ${source}. Use --source afl-api for AFLW data.`,
+    source
+  );
+}
 var ValidationError = class extends Error {
   constructor(message, issues) {
     super(message);
@@ -119,6 +125,10 @@ function toAestString(date) {
   });
   return formatter.format(date);
 }
+function resolveDefaultSeason(competition = "AFLM") {
+  const year = (/* @__PURE__ */ new Date()).getFullYear();
+  return competition === "AFLW" ? year - 1 : year;
+}
 var MONTH_ABBREV_TO_INDEX = /* @__PURE__ */ new Map([
   ["jan", 0],
   ["feb", 1],
@@ -233,6 +243,26 @@ function normaliseTeamName(raw) {
   const trimmed = raw.trim();
   return ALIAS_MAP.get(trimmed.toLowerCase()) ?? trimmed;
 }
+var AFL_API_TEAM_IDS = /* @__PURE__ */ new Map([
+  ["CD_T10", "Adelaide Crows"],
+  ["CD_T20", "Brisbane Lions"],
+  ["CD_T30", "Carlton"],
+  ["CD_T40", "Collingwood"],
+  ["CD_T50", "Essendon"],
+  ["CD_T60", "Fremantle"],
+  ["CD_T70", "Geelong Cats"],
+  ["CD_T1000", "Gold Coast Suns"],
+  ["CD_T1010", "GWS Giants"],
+  ["CD_T80", "Hawthorn"],
+  ["CD_T90", "Melbourne"],
+  ["CD_T100", "North Melbourne"],
+  ["CD_T110", "Port Adelaide"],
+  ["CD_T120", "Richmond"],
+  ["CD_T130", "St Kilda"],
+  ["CD_T160", "Sydney Swans"],
+  ["CD_T150", "West Coast Eagles"],
+  ["CD_T140", "Western Bulldogs"]
+]);
 
 // src/transforms/footywire-player-stats.ts
 import * as cheerio from "cheerio";
@@ -487,6 +517,14 @@ var FINALS_PATTERN = /final|elimination|qualifying|preliminary|semi|grand/i;
 function inferRoundType(roundName) {
   return FINALS_PATTERN.test(roundName) ? "Finals" : "HomeAndAway";
 }
+function finalsRoundNumber(headerText, lastHARound) {
+  const lower = headerText.toLowerCase();
+  if (lower.includes("qualifying") || lower.includes("elimination")) return lastHARound + 1;
+  if (lower.includes("semi")) return lastHARound + 2;
+  if (lower.includes("preliminary")) return lastHARound + 3;
+  if (lower.includes("grand")) return lastHARound + 4;
+  return lastHARound + 1;
+}
 function toMatchStatus(raw) {
   switch (raw) {
     case "CONCLUDED":
@@ -783,6 +821,7 @@ function parseMatchList(html, year) {
   const $ = cheerio2.load(html);
   const results = [];
   let currentRound = 0;
+  let lastHARound = 0;
   let currentRoundType = "HomeAndAway";
   $("tr").each((_i, row) => {
     const roundHeader = $(row).find("td[colspan='7']");
@@ -792,6 +831,11 @@ function parseMatchList(html, year) {
       const roundMatch = /Round\s+(\d+)/i.exec(text);
       if (roundMatch?.[1]) {
         currentRound = Number.parseInt(roundMatch[1], 10);
+        if (currentRoundType === "HomeAndAway") {
+          lastHARound = currentRound;
+        }
+      } else if (currentRoundType === "Finals") {
+        currentRound = finalsRoundNumber(text, lastHARound);
       }
       return;
     }
@@ -862,6 +906,7 @@ function parseFixtureList(html, year) {
   const $ = cheerio2.load(html);
   const fixtures = [];
   let currentRound = 0;
+  let lastHARound = 0;
   let currentRoundType = "HomeAndAway";
   let gameNumber = 0;
   $("tr").each((_i, row) => {
@@ -872,6 +917,11 @@ function parseFixtureList(html, year) {
       const roundMatch = /Round\s+(\d+)/i.exec(text);
       if (roundMatch?.[1]) {
         currentRound = Number.parseInt(roundMatch[1], 10);
+        if (currentRoundType === "HomeAndAway") {
+          lastHARound = currentRound;
+        }
+      } else if (currentRoundType === "Finals") {
+        currentRound = finalsRoundNumber(text, lastHARound);
       }
       return;
     }
@@ -1407,6 +1457,22 @@ async function fetchCoachesVotes(query) {
   return ok(votes);
 }
 
+// src/lib/concurrency.ts
+async function batchedMap(items, fn, options) {
+  const batchSize = options?.batchSize ?? 5;
+  const delayMs = options?.delayMs ?? 0;
+  const results = [];
+  for (let i = 0; i < items.length; i += batchSize) {
+    const batch = items.slice(i, i + batchSize);
+    const batchResults = await Promise.all(batch.map(fn));
+    results.push(...batchResults);
+    if (delayMs > 0 && i + batchSize < items.length) {
+      await new Promise((resolve) => setTimeout(resolve, delayMs));
+    }
+  }
+  return results;
+}
+
 // src/lib/validation.ts
 import { z } from "zod/v4";
 var AflApiTokenSchema = z.object({
@@ -1512,7 +1578,14 @@ var CfsPlayerInnerSchema = z.object({
   captain: z.boolean().optional(),
   playerJumperNumber: z.number().optional()
 }).passthrough();
-var statNum = z.number().nullable().optional();
+var statNum = z.union([
+  z.number(),
+  z.string().transform((s) => {
+    if (s === "" || s === "-") return null;
+    const n = Number(s);
+    return Number.isNaN(n) ? null : n;
+  })
+]).nullable().optional();
 var PlayerGameStatsSchema = z.object({
   goals: statNum,
   behinds: statNum,
@@ -1590,8 +1663,8 @@ var PlayerStatsItemSchema = z.object({
   teamId: z.string(),
   playerStats: z.object({
     stats: PlayerGameStatsSchema,
-    timeOnGroundPercentage: z.number().nullable().optional()
-  }).passthrough()
+    timeOnGroundPercentage: statNum
+  }).passthrough().nullable().optional()
 }).passthrough();
 var PlayerStatsListSchema = z.object({
   homeTeamPlayerStats: z.array(PlayerStatsItemSchema),
@@ -1692,6 +1765,7 @@ var AflApiClient = class {
   fetchFn;
   tokenUrl;
   cachedToken = null;
+  pendingAuth = null;
   constructor(options) {
     this.fetchFn = options?.fetchFn ?? globalThis.fetch;
     this.tokenUrl = options?.tokenUrl ?? TOKEN_URL;
@@ -1699,9 +1773,21 @@ var AflApiClient = class {
   /**
    * Authenticate with the WMCTok token endpoint and cache the token.
    *
+   * Concurrent callers share the same in-flight request to avoid
+   * redundant token fetches (thundering herd prevention).
+   *
    * @returns The access token on success, or an error Result.
    */
   async authenticate() {
+    if (this.pendingAuth) {
+      return this.pendingAuth;
+    }
+    this.pendingAuth = this.doAuthenticate().finally(() => {
+      this.pendingAuth = null;
+    });
+    return this.pendingAuth;
+  }
+  async doAuthenticate() {
     try {
       const response = await this.fetchFn(this.tokenUrl, {
         method: "POST",
@@ -1956,7 +2042,7 @@ var AflApiClient = class {
       return roundsResult;
     }
     const providerIds = roundsResult.data.flatMap((r) => r.providerId ? [r.providerId] : []);
-    const results = await Promise.all(providerIds.map((id) => this.fetchRoundMatchItems(id)));
+    const results = await batchedMap(providerIds, (id) => this.fetchRoundMatchItems(id));
     const allItems = [];
     for (const result of results) {
       if (!result.success) {
@@ -2255,12 +2341,14 @@ function toFixture(item, season, fallbackRoundNumber, competition) {
 async function fetchFixture(query) {
   const competition = query.competition ?? "AFLM";
   if (query.source === "squiggle") {
+    if (competition === "AFLW") return err(aflwUnsupportedError("squiggle"));
     const client2 = new SquiggleClient();
     const result = await client2.fetchGames(query.season, query.round ?? void 0);
     if (!result.success) return result;
     return ok(transformSquiggleGamesToFixture(result.data.games, query.season));
   }
   if (query.source === "footywire") {
+    if (competition === "AFLW") return err(aflwUnsupportedError("footywire"));
     const fwClient = new FootyWireClient();
     const result = await fwClient.fetchSeasonFixture(query.season);
     if (!result.success) return result;
@@ -2290,8 +2378,9 @@ async function fetchFixture(query) {
   const roundProviderIds = roundsResult.data.flatMap(
     (r) => r.providerId ? [{ providerId: r.providerId, roundNumber: r.roundNumber }] : []
   );
-  const roundResults = await Promise.all(
-    roundProviderIds.map((r) => client.fetchRoundMatchItems(r.providerId))
+  const roundResults = await batchedMap(
+    roundProviderIds,
+    (r) => client.fetchRoundMatchItems(r.providerId)
   );
   const fixtures = [];
   for (let i = 0; i < roundResults.length; i++) {
@@ -2603,6 +2692,7 @@ function parseSeasonPage(html, year) {
   const results = [];
   let currentRound = 0;
   let currentRoundType = "HomeAndAway";
+  let lastHARound = 0;
   let matchCounter = 0;
   $("table").each((_i, table) => {
     const $table = $(table);
@@ -2612,10 +2702,14 @@ function parseSeasonPage(html, year) {
     if (roundMatch?.[1] && border !== "1") {
       currentRound = Number.parseInt(roundMatch[1], 10);
       currentRoundType = inferRoundType(text);
+      if (currentRoundType === "HomeAndAway") {
+        lastHARound = currentRound;
+      }
       return;
     }
     if (border !== "1" && inferRoundType(text) === "Finals") {
       currentRoundType = "Finals";
+      currentRound = finalsRoundNumber(text, lastHARound);
       return;
     }
     if (border !== "1") return;
@@ -2930,6 +3024,7 @@ function transformLadderEntries(entries) {
 async function fetchLadder(query) {
   const competition = query.competition ?? "AFLM";
   if (query.source === "squiggle") {
+    if (competition === "AFLW") return err(aflwUnsupportedError("squiggle"));
     const client2 = new SquiggleClient();
     const result = await client2.fetchStandings(query.season, query.round ?? void 0);
     if (!result.success) return result;
@@ -2941,6 +3036,7 @@ async function fetchLadder(query) {
     });
   }
   if (query.source === "afl-tables") {
+    if (competition === "AFLW") return err(aflwUnsupportedError("afl-tables"));
     const atClient = new AflTablesClient();
     const resultsResult = await atClient.fetchSeasonResults(query.season);
     if (!resultsResult.success) return resultsResult;
@@ -3042,8 +3138,9 @@ async function fetchLineup(query) {
   if (matchItems.data.length === 0) {
     return err(new AflApiError(`No matches found for round ${query.round}`));
   }
-  const rosterResults = await Promise.all(
-    matchItems.data.map((item) => client.fetchMatchRoster(item.match.matchId))
+  const rosterResults = await batchedMap(
+    matchItems.data,
+    (item) => client.fetchMatchRoster(item.match.matchId)
   );
   const lineups = [];
   for (const rosterResult of rosterResults) {
@@ -3074,6 +3171,7 @@ async function fetchMatchResults(query) {
       return ok(transformMatchItems(itemsResult.data, query.season, competition));
     }
     case "footywire": {
+      if (competition === "AFLW") return err(aflwUnsupportedError("footywire"));
       const client = new FootyWireClient();
       const result = await client.fetchSeasonResults(query.season);
       if (!result.success) return result;
@@ -3083,6 +3181,7 @@ async function fetchMatchResults(query) {
       return result;
     }
     case "afl-tables": {
+      if (competition === "AFLW") return err(aflwUnsupportedError("afl-tables"));
       const client = new AflTablesClient();
       const result = await client.fetchSeasonResults(query.season);
       if (!result.success) return result;
@@ -3092,6 +3191,7 @@ async function fetchMatchResults(query) {
       return result;
     }
     case "squiggle": {
+      if (competition === "AFLW") return err(aflwUnsupportedError("squiggle"));
       const client = new SquiggleClient();
       const result = await client.fetchGames(query.season, query.round ?? void 0, 100);
       if (!result.success) return result;
@@ -3117,7 +3217,7 @@ async function resolveTeamId(client, teamName, competition) {
 async function fetchFromAflApi(query) {
   const client = new AflApiClient();
   const competition = query.competition ?? "AFLM";
-  const season = query.season ?? (/* @__PURE__ */ new Date()).getFullYear();
+  const season = query.season ?? resolveDefaultSeason(competition);
   const [teamIdResult, seasonResult] = await Promise.all([
     resolveTeamId(client, query.team, competition),
     client.resolveCompSeason(competition, season)
@@ -3140,8 +3240,8 @@ async function fetchFromAflApi(query) {
     jumperNumber: p.jumperNumber ?? null,
     position: p.position ?? null,
     dateOfBirth: p.player.dateOfBirth ?? null,
-    heightCm: p.player.heightInCm ?? null,
-    weightKg: p.player.weightInKg ?? null,
+    heightCm: p.player.heightInCm || null,
+    weightKg: p.player.weightInKg || null,
     gamesPlayed: null,
     goals: null,
     draftYear: p.player.draftYear ? Number.parseInt(p.player.draftYear, 10) || null : null,
@@ -3155,8 +3255,9 @@ async function fetchFromAflApi(query) {
   return ok(players);
 }
 async function fetchFromFootyWire(query) {
-  const client = new FootyWireClient();
   const competition = query.competition ?? "AFLM";
+  if (competition === "AFLW") return err(aflwUnsupportedError("footywire"));
+  const client = new FootyWireClient();
   const teamName = normaliseTeamName(query.team);
   const result = await client.fetchPlayerList(teamName);
   if (!result.success) return result;
@@ -3168,8 +3269,9 @@ async function fetchFromFootyWire(query) {
   return ok(players);
 }
 async function fetchFromAflTables(query) {
-  const client = new AflTablesClient();
   const competition = query.competition ?? "AFLM";
+  if (competition === "AFLW") return err(aflwUnsupportedError("afl-tables"));
+  const client = new AflTablesClient();
   const teamName = normaliseTeamName(query.team);
   const result = await client.fetchPlayerList(teamName);
   if (!result.success) return result;
@@ -3204,80 +3306,82 @@ function toNullable(value) {
 }
 function transformOne(item, matchId, season, roundNumber, competition, source, teamIdMap) {
   const inner = item.player.player.player;
-  const stats = item.playerStats.stats;
-  const clearances = stats.clearances;
+  const stats = item.playerStats?.stats;
+  const clearances = stats?.clearances;
   return {
     matchId,
     season,
     roundNumber,
-    team: normaliseTeamName(teamIdMap?.get(item.teamId) ?? item.teamId),
+    team: normaliseTeamName(
+      teamIdMap?.get(item.teamId) ?? AFL_API_TEAM_IDS.get(item.teamId) ?? item.teamId
+    ),
     competition,
     playerId: inner.playerId,
     givenName: inner.playerName.givenName,
     surname: inner.playerName.surname,
     displayName: `${inner.playerName.givenName} ${inner.playerName.surname}`,
     jumperNumber: item.player.jumperNumber ?? null,
-    kicks: toNullable(stats.kicks),
-    handballs: toNullable(stats.handballs),
-    disposals: toNullable(stats.disposals),
-    marks: toNullable(stats.marks),
-    goals: toNullable(stats.goals),
-    behinds: toNullable(stats.behinds),
-    tackles: toNullable(stats.tackles),
-    hitouts: toNullable(stats.hitouts),
-    freesFor: toNullable(stats.freesFor),
-    freesAgainst: toNullable(stats.freesAgainst),
-    contestedPossessions: toNullable(stats.contestedPossessions),
-    uncontestedPossessions: toNullable(stats.uncontestedPossessions),
-    contestedMarks: toNullable(stats.contestedMarks),
-    intercepts: toNullable(stats.intercepts),
+    kicks: toNullable(stats?.kicks),
+    handballs: toNullable(stats?.handballs),
+    disposals: toNullable(stats?.disposals),
+    marks: toNullable(stats?.marks),
+    goals: toNullable(stats?.goals),
+    behinds: toNullable(stats?.behinds),
+    tackles: toNullable(stats?.tackles),
+    hitouts: toNullable(stats?.hitouts),
+    freesFor: toNullable(stats?.freesFor),
+    freesAgainst: toNullable(stats?.freesAgainst),
+    contestedPossessions: toNullable(stats?.contestedPossessions),
+    uncontestedPossessions: toNullable(stats?.uncontestedPossessions),
+    contestedMarks: toNullable(stats?.contestedMarks),
+    intercepts: toNullable(stats?.intercepts),
     centreClearances: toNullable(clearances?.centreClearances),
     stoppageClearances: toNullable(clearances?.stoppageClearances),
     totalClearances: toNullable(clearances?.totalClearances),
-    inside50s: toNullable(stats.inside50s),
-    rebound50s: toNullable(stats.rebound50s),
-    clangers: toNullable(stats.clangers),
-    turnovers: toNullable(stats.turnovers),
-    onePercenters: toNullable(stats.onePercenters),
-    bounces: toNullable(stats.bounces),
-    goalAssists: toNullable(stats.goalAssists),
-    disposalEfficiency: toNullable(stats.disposalEfficiency),
-    metresGained: toNullable(stats.metresGained),
-    goalAccuracy: toNullable(stats.goalAccuracy),
-    marksInside50: toNullable(stats.marksInside50),
-    tacklesInside50: toNullable(stats.tacklesInside50),
-    shotsAtGoal: toNullable(stats.shotsAtGoal),
-    scoreInvolvements: toNullable(stats.scoreInvolvements),
-    totalPossessions: toNullable(stats.totalPossessions),
-    timeOnGroundPercentage: toNullable(item.playerStats.timeOnGroundPercentage),
-    ratingPoints: toNullable(stats.ratingPoints),
-    dreamTeamPoints: toNullable(stats.dreamTeamPoints),
-    effectiveDisposals: toNullable(stats.extendedStats?.effectiveDisposals),
-    effectiveKicks: toNullable(stats.extendedStats?.effectiveKicks),
-    kickEfficiency: toNullable(stats.extendedStats?.kickEfficiency),
-    kickToHandballRatio: toNullable(stats.extendedStats?.kickToHandballRatio),
-    pressureActs: toNullable(stats.extendedStats?.pressureActs),
-    defHalfPressureActs: toNullable(stats.extendedStats?.defHalfPressureActs),
-    spoils: toNullable(stats.extendedStats?.spoils),
-    hitoutsToAdvantage: toNullable(stats.extendedStats?.hitoutsToAdvantage),
-    hitoutWinPercentage: toNullable(stats.extendedStats?.hitoutWinPercentage),
-    hitoutToAdvantageRate: toNullable(stats.extendedStats?.hitoutToAdvantageRate),
-    groundBallGets: toNullable(stats.extendedStats?.groundBallGets),
-    f50GroundBallGets: toNullable(stats.extendedStats?.f50GroundBallGets),
-    interceptMarks: toNullable(stats.extendedStats?.interceptMarks),
-    marksOnLead: toNullable(stats.extendedStats?.marksOnLead),
-    contestedPossessionRate: toNullable(stats.extendedStats?.contestedPossessionRate),
-    contestOffOneOnOnes: toNullable(stats.extendedStats?.contestOffOneOnOnes),
-    contestOffWins: toNullable(stats.extendedStats?.contestOffWins),
-    contestOffWinsPercentage: toNullable(stats.extendedStats?.contestOffWinsPercentage),
-    contestDefOneOnOnes: toNullable(stats.extendedStats?.contestDefOneOnOnes),
-    contestDefLosses: toNullable(stats.extendedStats?.contestDefLosses),
-    contestDefLossPercentage: toNullable(stats.extendedStats?.contestDefLossPercentage),
-    centreBounceAttendances: toNullable(stats.extendedStats?.centreBounceAttendances),
-    kickins: toNullable(stats.extendedStats?.kickins),
-    kickinsPlayon: toNullable(stats.extendedStats?.kickinsPlayon),
-    ruckContests: toNullable(stats.extendedStats?.ruckContests),
-    scoreLaunches: toNullable(stats.extendedStats?.scoreLaunches),
+    inside50s: toNullable(stats?.inside50s),
+    rebound50s: toNullable(stats?.rebound50s),
+    clangers: toNullable(stats?.clangers),
+    turnovers: toNullable(stats?.turnovers),
+    onePercenters: toNullable(stats?.onePercenters),
+    bounces: toNullable(stats?.bounces),
+    goalAssists: toNullable(stats?.goalAssists),
+    disposalEfficiency: toNullable(stats?.disposalEfficiency),
+    metresGained: toNullable(stats?.metresGained),
+    goalAccuracy: toNullable(stats?.goalAccuracy),
+    marksInside50: toNullable(stats?.marksInside50),
+    tacklesInside50: toNullable(stats?.tacklesInside50),
+    shotsAtGoal: toNullable(stats?.shotsAtGoal),
+    scoreInvolvements: toNullable(stats?.scoreInvolvements),
+    totalPossessions: toNullable(stats?.totalPossessions),
+    timeOnGroundPercentage: toNullable(item.playerStats?.timeOnGroundPercentage),
+    ratingPoints: toNullable(stats?.ratingPoints),
+    dreamTeamPoints: toNullable(stats?.dreamTeamPoints),
+    effectiveDisposals: toNullable(stats?.extendedStats?.effectiveDisposals),
+    effectiveKicks: toNullable(stats?.extendedStats?.effectiveKicks),
+    kickEfficiency: toNullable(stats?.extendedStats?.kickEfficiency),
+    kickToHandballRatio: toNullable(stats?.extendedStats?.kickToHandballRatio),
+    pressureActs: toNullable(stats?.extendedStats?.pressureActs),
+    defHalfPressureActs: toNullable(stats?.extendedStats?.defHalfPressureActs),
+    spoils: toNullable(stats?.extendedStats?.spoils),
+    hitoutsToAdvantage: toNullable(stats?.extendedStats?.hitoutsToAdvantage),
+    hitoutWinPercentage: toNullable(stats?.extendedStats?.hitoutWinPercentage),
+    hitoutToAdvantageRate: toNullable(stats?.extendedStats?.hitoutToAdvantageRate),
+    groundBallGets: toNullable(stats?.extendedStats?.groundBallGets),
+    f50GroundBallGets: toNullable(stats?.extendedStats?.f50GroundBallGets),
+    interceptMarks: toNullable(stats?.extendedStats?.interceptMarks),
+    marksOnLead: toNullable(stats?.extendedStats?.marksOnLead),
+    contestedPossessionRate: toNullable(stats?.extendedStats?.contestedPossessionRate),
+    contestOffOneOnOnes: toNullable(stats?.extendedStats?.contestOffOneOnOnes),
+    contestOffWins: toNullable(stats?.extendedStats?.contestOffWins),
+    contestOffWinsPercentage: toNullable(stats?.extendedStats?.contestOffWinsPercentage),
+    contestDefOneOnOnes: toNullable(stats?.extendedStats?.contestDefOneOnOnes),
+    contestDefLosses: toNullable(stats?.extendedStats?.contestDefLosses),
+    contestDefLossPercentage: toNullable(stats?.extendedStats?.contestDefLossPercentage),
+    centreBounceAttendances: toNullable(stats?.extendedStats?.centreBounceAttendances),
+    kickins: toNullable(stats?.extendedStats?.kickins),
+    kickinsPlayon: toNullable(stats?.extendedStats?.kickinsPlayon),
+    ruckContests: toNullable(stats?.extendedStats?.ruckContests),
+    scoreLaunches: toNullable(stats?.extendedStats?.scoreLaunches),
     source
   };
 }
@@ -3334,8 +3438,9 @@ async function fetchPlayerStats(query) {
         teamIdMap.set(item.match.homeTeamId, item.match.homeTeam.name);
         teamIdMap.set(item.match.awayTeamId, item.match.awayTeam.name);
       }
-      const statsResults = await Promise.all(
-        matchItemsResult.data.map((item) => client.fetchPlayerStats(item.match.matchId))
+      const statsResults = await batchedMap(
+        matchItemsResult.data,
+        (item) => client.fetchPlayerStats(item.match.matchId)
       );
       const allStats = [];
       for (let i = 0; i < statsResults.length; i++) {
@@ -3359,6 +3464,7 @@ async function fetchPlayerStats(query) {
       return ok(allStats);
     }
     case "footywire": {
+      if (competition === "AFLW") return err(aflwUnsupportedError("footywire"));
       const fwClient = new FootyWireClient();
       const idsResult = await fwClient.fetchSeasonMatchIds(query.season);
       if (!idsResult.success) return idsResult;
@@ -3388,6 +3494,7 @@ async function fetchPlayerStats(query) {
       return ok(allStats);
     }
     case "afl-tables": {
+      if (competition === "AFLW") return err(aflwUnsupportedError("afl-tables"));
       const atClient = new AflTablesClient();
       const atResult = await atClient.fetchSeasonPlayerStats(query.season);
       if (!atResult.success) return atResult;
@@ -3411,7 +3518,37 @@ async function fetchTeamStats(query) {
     }
     case "afl-tables": {
       const client = new AflTablesClient();
-      return client.fetchTeamStats(query.season);
+      const statsResult = await client.fetchTeamStats(query.season);
+      if (!statsResult.success) return statsResult;
+      const needsGp = statsResult.data.some((e) => e.gamesPlayed === 0);
+      const gpMap = /* @__PURE__ */ new Map();
+      if (needsGp) {
+        const resultsResult = await client.fetchSeasonResults(query.season);
+        if (resultsResult.success) {
+          for (const m of resultsResult.data) {
+            gpMap.set(m.homeTeam, (gpMap.get(m.homeTeam) ?? 0) + 1);
+            gpMap.set(m.awayTeam, (gpMap.get(m.awayTeam) ?? 0) + 1);
+          }
+        }
+      }
+      const enriched = statsResult.data.map((entry) => ({
+        ...entry,
+        gamesPlayed: gpMap.get(entry.team) ?? entry.gamesPlayed
+      }));
+      if (summaryType === "averages") {
+        return ok(
+          enriched.map((entry) => ({
+            ...entry,
+            stats: Object.fromEntries(
+              Object.entries(entry.stats).map(([k, v]) => [
+                k,
+                entry.gamesPlayed > 0 ? +(v / entry.gamesPlayed).toFixed(1) : 0
+              ])
+            )
+          }))
+        );
+      }
+      return ok(enriched);
     }
     case "afl-api":
     case "squiggle":
@@ -3430,19 +3567,30 @@ async function fetchTeamStats(query) {
 function teamTypeForComp(comp) {
   return comp === "AFLW" ? "WOMEN" : "MEN";
 }
-async function fetchTeams(query) {
-  const client = new AflApiClient();
-  const teamType = query?.teamType ?? teamTypeForComp(query?.competition ?? "AFLM");
-  const result = await client.fetchTeams(teamType);
-  if (!result.success) return result;
-  const competition = query?.competition ?? "AFLM";
-  const teams = result.data.map((t) => ({
+function toTeams(data, competition) {
+  return data.map((t) => ({
     teamId: String(t.id),
     name: normaliseTeamName(t.name),
     abbreviation: t.abbreviation ?? "",
     competition
   })).filter((t) => AFL_SENIOR_TEAMS.has(t.name));
-  return ok(teams);
+}
+async function fetchTeams(query) {
+  const client = new AflApiClient();
+  if (!query?.competition && !query?.teamType) {
+    const [menResult, womenResult] = await Promise.all([
+      client.fetchTeams("MEN"),
+      client.fetchTeams("WOMEN")
+    ]);
+    if (!menResult.success) return menResult;
+    if (!womenResult.success) return womenResult;
+    return ok([...toTeams(menResult.data, "AFLM"), ...toTeams(womenResult.data, "AFLW")]);
+  }
+  const competition = query?.competition ?? "AFLM";
+  const teamType = query?.teamType ?? teamTypeForComp(competition);
+  const result = await client.fetchTeams(teamType);
+  if (!result.success) return result;
+  return ok(toTeams(result.data, competition));
 }
 async function fetchSquad(query) {
   const client = new AflApiClient();
@@ -3463,8 +3611,8 @@ async function fetchSquad(query) {
     jumperNumber: p.jumperNumber ?? null,
     position: p.position ?? null,
     dateOfBirth: p.player.dateOfBirth ? new Date(p.player.dateOfBirth) : null,
-    heightCm: p.player.heightInCm ?? null,
-    weightKg: p.player.weightInKg ?? null,
+    heightCm: p.player.heightInCm || null,
+    weightKg: p.player.weightInKg || null,
     draftYear: p.player.draftYear ? Number.parseInt(p.player.draftYear, 10) || null : null,
     draftPosition: p.player.draftPosition ? Number.parseInt(p.player.draftPosition, 10) || null : null,
     draftType: p.player.draftType ?? null,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fitzroy",
-  "version": "1.1.1",
+  "version": "1.3.0",
   "description": "TypeScript library and CLI for AFL data — match results, player stats, fixtures, ladders, and more",
   "type": "module",
   "main": "./dist/index.js",