bbdata-cli 0.1.0 → 0.2.0

package/dist/src/index.d.ts

@@ -17,6 +17,7 @@ interface QueryOptions {
     format?: OutputFormat;
     source?: string;
     cache?: boolean;
+    stdin?: boolean;
 }
 interface QueryResult {
     data: Record<string, unknown>[];
@@ -44,6 +45,7 @@ interface ReportOptions {
     audience?: Audience;
     format?: 'markdown' | 'json';
     validate?: boolean;
+    stdin?: boolean;
 }
 interface ReportResult {
     content: string;
@@ -207,7 +209,7 @@ type BbdataConfig = z.infer<typeof ConfigSchema>;
 declare function getConfig(): BbdataConfig;
 declare function setConfig(updates: Partial<BbdataConfig>): BbdataConfig;
 
-type DataSource = 'savant' | 'fangraphs' | 'mlb-stats-api' | 'baseball-reference';
+type DataSource = 'savant' | 'fangraphs' | 'mlb-stats-api' | 'baseball-reference' | 'stdin';
 declare const PitchDataSchema: z.ZodObject<{
     pitcher_id: z.ZodString;
     pitcher_name: z.ZodString;

package/dist/src/index.js

@@ -162,8 +162,8 @@ var SavantAdapter = class {
     }
     const params = new URLSearchParams({
       all: "true",
-      type: "detail",
-      ...query2.stat_type === "pitching" ? { player_type: "pitcher", pitchers_lookup: playerId ?? "" } : { player_type: "batter", batters_lookup: playerId ?? "" },
+      type: "details",
+      ...query2.stat_type === "pitching" ? { player_type: "pitcher", "pitchers_lookup[]": playerId ?? "" } : { player_type: "batter", "batters_lookup[]": playerId ?? "" },
       ...query2.start_date ? { game_date_gt: query2.start_date } : { game_date_gt: `${query2.season}-01-01` },
       ...query2.end_date ? { game_date_lt: query2.end_date } : { game_date_lt: `${query2.season}-12-31` }
     });
@@ -189,6 +189,9 @@ var SavantAdapter = class {
       cast: true,
       cast_date: false
     });
+    if (rawRows.length === 0 && csvText.trim().length > 50) {
+      log.debug("Savant returned headers but no data rows");
+    }
     const filteredRows = rawRows.filter(
       (row) => row.pitch_type && String(row.pitch_type).trim() !== ""
     );
@@ -228,6 +231,7 @@ var SavantAdapter = class {
 
 // src/adapters/fangraphs.ts
 var FG_LEADERS_URL = "https://www.fangraphs.com/api/leaders/major-league/data";
+var stripHtml = (s) => s.replace(/<[^>]*>/g, "");
 var FanGraphsAdapter = class {
   source = "fangraphs";
   description = "FanGraphs \u2014 aggregated stats, leaderboards, WAR, wRC+, FIP";
@@ -256,7 +260,7 @@ var FanGraphsAdapter = class {
       const parsed = JSON.parse(data);
       const nameNorm = name.toLowerCase().trim();
       const players = parsed.data ?? [];
-      const getName = (p) => String(p.PlayerName ?? p.Name ?? "").toLowerCase().trim();
+      const getName = (p) => stripHtml(String(p.PlayerName ?? p.Name ?? "")).toLowerCase().trim();
       let match = players.find((p) => getName(p) === nameNorm);
       if (!match) {
         const tokens = nameNorm.split(/\s+/);
@@ -272,8 +276,8 @@ var FanGraphsAdapter = class {
       return {
         mlbam_id: String(match.xMLBAMID ?? ""),
         fangraphs_id: String(match.playerid ?? ""),
-        name: String(match.PlayerName ?? match.Name),
-        team: String(match.Team ?? "")
+        name: stripHtml(String(match.PlayerName ?? match.Name)),
+        team: stripHtml(String(match.TeamNameAbb ?? match.TeamName ?? match.Team ?? ""))
       };
     } catch (error) {
       log.warn(`FanGraphs player resolution failed: ${error}`);
@@ -305,7 +309,7 @@ var FanGraphsAdapter = class {
     let filtered = rows;
     if (query2.player_name) {
       const nameNorm = query2.player_name.toLowerCase().trim();
-      const getName = (r) => String(r.PlayerName ?? r.Name ?? "").toLowerCase().trim();
+      const getName = (r) => stripHtml(String(r.PlayerName ?? r.Name ?? "")).toLowerCase().trim();
       const exact = rows.filter((r) => getName(r) === nameNorm);
       if (exact.length > 0) {
         filtered = exact;
@@ -319,8 +323,8 @@ var FanGraphsAdapter = class {
     }
     const stats = filtered.map((row) => ({
       player_id: String(row.xMLBAMID ?? row.playerid ?? ""),
-      player_name: String(row.PlayerName ?? row.Name ?? ""),
-      team: String(row.Team ?? ""),
+      player_name: stripHtml(String(row.PlayerName ?? row.Name ?? "")),
+      team: stripHtml(String(row.TeamNameAbb ?? row.TeamName ?? row.Team ?? "")),
       season: query2.season,
       stat_type: query2.stat_type,
       stats: row
@@ -394,13 +398,88 @@ var BaseballReferenceAdapter = class {
   }
 };
 
+// src/adapters/stdin.ts
+var StdinAdapter = class {
+  source = "stdin";
+  description = "Local data from stdin (for sandboxed environments)";
+  data = [];
+  player = null;
+  loaded = false;
+  /**
+   * Load data from a pre-read stdin string.
+   * Called by the CLI before the adapter is used.
+   */
+  load(raw) {
+    try {
+      const parsed = JSON.parse(raw);
+      if (Array.isArray(parsed)) {
+        this.data = parsed;
+      } else if (parsed.data && Array.isArray(parsed.data)) {
+        this.data = parsed.data;
+        if (parsed.player) {
+          this.player = parsed.player;
+        }
+      } else {
+        throw new Error('Expected JSON array or { "data": [...] } object');
+      }
+      this.loaded = true;
+      log.info(`Stdin adapter loaded ${this.data.length} records`);
+    } catch (error) {
+      throw new Error(
+        `Failed to parse stdin data: ${error instanceof Error ? error.message : String(error)}`
+      );
+    }
+  }
+  supports(_query) {
+    return this.loaded && this.data.length > 0;
+  }
+  async fetch(query2) {
+    if (!this.loaded) {
+      throw new Error("Stdin adapter has no data \u2014 pipe JSON via stdin with --stdin flag");
+    }
+    return {
+      data: this.data,
+      source: "stdin",
+      cached: false,
+      fetchedAt: (/* @__PURE__ */ new Date()).toISOString(),
+      meta: {
+        rowCount: this.data.length,
+        season: query2.season,
+        query: query2
+      }
+    };
+  }
+  async resolvePlayer(name) {
+    if (this.player && this.player.name.toLowerCase() === name.toLowerCase()) {
+      return this.player;
+    }
+    const firstRecord = this.data[0];
+    if (firstRecord) {
+      const id = firstRecord.pitcher_id ?? firstRecord.player_id ?? "";
+      const recordName = firstRecord.pitcher_name ?? firstRecord.player_name ?? name;
+      if (id) {
+        return {
+          mlbam_id: id,
+          name: recordName
+        };
+      }
+    }
+    return null;
+  }
+};
+
 // src/adapters/index.ts
+var stdinAdapter = new StdinAdapter();
 var adapters = {
   "mlb-stats-api": new MlbStatsApiAdapter(),
   "savant": new SavantAdapter(),
   "fangraphs": new FanGraphsAdapter(),
-  "baseball-reference": new BaseballReferenceAdapter()
+  "baseball-reference": new BaseballReferenceAdapter(),
+  "stdin": stdinAdapter
 };
+function getStdinAdapter() {
+  return stdinAdapter;
+}
 function resolveAdapters(preferred) {
   return preferred.map((source) => adapters[source]).filter(Boolean);
 }
@@ -579,6 +658,20 @@ function getTemplatesDir() {
   return dir;
 }
 
+// src/utils/stdin.ts
+function readStdin() {
+  return new Promise((resolve, reject) => {
+    if (process.stdin.isTTY) {
+      reject(new Error(`--stdin flag requires piped input. Usage: echo '{"data":[...]}' | bbdata query ... --stdin`));
+      return;
+    }
+    const chunks = [];
+    process.stdin.on("data", (chunk) => chunks.push(chunk));
+    process.stdin.on("end", () => resolve(Buffer.concat(chunks).toString("utf-8")));
+    process.stdin.on("error", reject);
+  });
+}
+
 // src/templates/queries/registry.ts
 var templates = /* @__PURE__ */ new Map();
 function registerTemplate(template13) {
@@ -1179,11 +1272,12 @@ var template9 = {
     "bbdata query leaderboard-custom --stat ERA --min-ip 100 --top 10 --format table"
   ],
   buildQuery(params) {
+    const pitchingStats = ["era", "fip", "xfip", "siera", "whip", "k/9", "bb/9", "hr/9", "ip", "w", "sv", "hld"];
+    const isPitching = pitchingStats.includes((params.stat ?? "").toLowerCase());
     return {
       season: params.season ?? (/* @__PURE__ */ new Date()).getFullYear(),
       team: params.team,
-      stat_type: "batting",
-      // Will be overridden based on stat
+      stat_type: isPitching ? "pitching" : "batting",
       min_pa: params.minPa,
       min_ip: params.minIp
     };
@@ -1429,6 +1523,12 @@ registerTemplate(template12);
 
 // src/commands/query.ts
 async function query(options) {
+  if (options.stdin) {
+    const raw = await readStdin();
+    const adapter = getStdinAdapter();
+    adapter.load(raw);
+    options.source = "stdin";
+  }
   const config = getConfig();
   const outputFormat = options.format ?? config.defaultFormat;
   const template13 = getTemplate(options.template);
@@ -1469,6 +1569,10 @@ ${available}`);
       });
       const rows = template13.transform(adapterResult.data, params);
       const columns = template13.columns(params);
+      if (rows.length === 0) {
+        log.debug(`${adapter.source} returned 0 rows. Trying next source...`);
+        continue;
+      }
       result = {
         rows,
         columns,
@@ -1763,6 +1867,11 @@ ${sections}
 `;
 }
 async function report(options) {
+  if (options.stdin) {
+    const raw = await readStdin();
+    const adapter = getStdinAdapter();
+    adapter.load(raw);
+  }
   const config = getConfig();
   const audience = options.audience ?? config.defaultAudience;
   const template13 = getReportTemplate(options.template);
@@ -1782,7 +1891,8 @@ ${available}`);
         player: options.player,
         team: options.team,
         season,
-        format: "json"
+        format: "json",
+        ...options.stdin ? { source: "stdin" } : {}
       });
       dataResults[req.queryTemplate] = result.data;
       if (!dataSources.includes(result.meta.source)) {