hicommonwealth / commonwealth / 20929920950

38%

push

github

refactor: move market discovery APIs to unified backend proxy (#13288) (#13289)

* refactor: move market discovery APIs to state/api/markets pattern

Completed GitHub issue #13288
- Created state/api/markets/ directory with React Query hooks following existing patterns
- Added useDiscoverKalshiMarketsQuery and useDiscoverPolymarketMarketsQuery hooks
- Added Kalshi and Polymarket endpoints to ExternalEndpoints in config.ts
- Added proper TypeScript interfaces for API responses (KalshiEvent, PolymarketMarketResponse)
- Improved Polymarket error handling with array validation
- Updated useMarketData.ts to use new hooks from state/api/markets/
- Removed deprecated services/kalshiApi.ts and services/polymarketApi.ts
- Tests: Manual verification required (external API integrations)

🤖 Generated with Claude Code
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: proxy Polymarket API through backend to avoid CORS issues

Completed GitHub issue #13288 (follow-up fix)
- Polymarket API does not include CORS headers, blocking browser requests
- Added DiscoverExternalMarkets schema and ExternalMarket type to schemas
- Created DiscoverExternalMarkets.query.ts to proxy Polymarket API on backend
- Added discoverExternalMarkets endpoint to community tRPC router
- Updated useDiscoverPolymarketMarketsQuery to use tRPC proxy

🤖 Generated with Claude Code
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: unify market discovery through single backend API

- Extended DiscoverExternalMarkets to support all providers (kalshi, polymarket, all)
- Added filters (search, category) and limit parameter to API
- Server-side filtering applied after fetching from external APIs
- Created single useDiscoverExternalMarketsQuery hook
- Removed separate client-side Kalshi/Polymarket API files
- Removed kalshi/polymarket endpoints from client config.ts
- Simplified useMarketData.ts - backend handles all filtering

🤖 Generated with Claude Code
Co-Authored-By: ... (continued)

2051 of 5890 branches covered (34.82%)

Branch coverage included in aggregate %.

2 of 44 new or added lines in 1 file covered. (4.55%)

3604 of 9164 relevant lines covered (39.33%)

46.8 hits per line