Skip to content

Yahoo relevance filter, card identity cleanup, 208 tests#58

Merged
Pyronewbic merged 3 commits into
mainfrom
dev
May 12, 2026
Merged

Yahoo relevance filter, card identity cleanup, 208 tests#58
Pyronewbic merged 3 commits into
mainfrom
dev

Conversation

@Pyronewbic
Copy link
Copy Markdown
Owner

Summary

  • Yahoo Auctions: relevance filtering in all paths (search, sold, arbitrage, alerts)
  • Card identity: strip pack names and condition text from parsed names
  • track-prices: tracks cards from active alerts, not just 3 defaults
  • Condition filter: checks detectedCondition in demo data
  • Tests: 92 unit + 76 API + 40 smoke = 208 total
  • Dead code: removed Redis import, updateCardField
  • Yahoo cheerio migration (Playwright removed)

Breaking changes

None

Demo data

  • Sample data still works
  • No new fields missing

Manual verification

  • Verified locally: all 4 sources, arbitrage, share pages, alerts, condition filter

Yahoo cheerio scraper returns irrelevant results (1-yen box auctions,
unrelated cards). filterRelevantResults now applied in search, sold,
arbitrage, and check-alerts paths — matching eBay's filtering.
…integrity

Unit tests (92): resolveCardIdToQuery, findDemoByNumber, multi-source
demo validation, sold date spans, condition filter with detectedCondition.

API tests (76): share endpoint for all 3 cards, demo price-history with
dates, arbitrage/price alert creation, condition filter mint/nm,
detectedCondition + _priceOutlier fields. Updated Umbreon to 8 active + 7 sold.
@Pyronewbic Pyronewbic merged commit 77f6b37 into main May 12, 2026
2 checks passed
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