Skip to content

Fixed sports scrapers for box score + recap link - #72

Open
claiireyu wants to merge 4 commits into
masterfrom
claire/sportScrapers
Open

Fixed sports scrapers for box score + recap link#72
claiireyu wants to merge 4 commits into
masterfrom
claire/sportScrapers

Conversation

@claiireyu

Copy link
Copy Markdown
Contributor

Overview

Refactored the games scraper to clearly distinguish between traditional Box Score sports and Recap Link sports. Added recap article image support for recap-only sports while disabling unsupported sports.

Changes Made

  • Added explicit BOX_SCORE and RECAP_LINK sport modes.
  • Box Score sports now only use Box Score parsing.
  • Recap Link sports now scrape:
    • Recap article title
    • Published date
    • Primary article image
  • Added recap_article_image to the Game model, MongoDB mapping, GraphQL GameType, and CreateGame mutation.
  • Extracted recap images from .sidearm-story-template-media img.
  • Commented out unsupported sports in SPORT_URLS.

Test Coverage

Graphql Playground

- Added optional fields for game recap links, article titles, and published dates in the GameType and Game models.
- Updated CreateGame mutation to accept new recap fields.
- Implemented a new scraping function to extract recap headlines and published times from Cornell Sidearm story pages.
@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 28f55aa1-14ff-45a3-90a6-788389616be5


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Restore softball scoring parsing, preserve inning periods, normalize schedule results, and match updates by concrete game time.
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.

1 participant