Merge pull request 'Fix: Event happening exactly now shows LIVE instead of In 0m' (#28 ) from fix/live-time-display into master

Add test for event happening exactly now shows LIVE
Added test_get_time_data_live_exactly_now to prevent regression.
2026-03-26 18:48:23 +01:00 · 2026-03-26 17:46:53 +00:00 · 2026-03-26 17:43:28 +00:00 · 2026-03-26 17:37:41 +00:00 · 2026-03-26 18:23:20 +01:00 · 2026-03-26 17:19:03 +00:00
3 changed files with 1384 additions and 404 deletions
--- a/skills/polymarket-browse/SKILL.md
+++ b/skills/polymarket-browse/SKILL.md
@@ -34,7 +34,7 @@ hermes mcp add polymarket https://docs.polymarket.com/mcp
 ## Usage

 ```
-polymarket-browse [--category "Counter Strike"] [--limit 5] [--matches N] [--non-matches N] [--search "TeamName"] [--matches-only] [--non-matches-only] [--detail N] [--raw] [--telegram]
+polymarket-browse [--category "Counter Strike"] [--limit 5] [--matches N] [--non-matches N] [--search "TeamName"] [--matches-only] [--non-matches-only] [--detail N] [--raw] [--telegram] [--no-cache] [--max-total N]
 ```

 ## Arguments
@@ -49,6 +49,8 @@ polymarket-browse [--category "Counter Strike"] [--limit 5] [--matches N] [--non
 - `--detail` : Index of match event (1-indexed) to show detailed markets. Default: 1. Set to 0 to disable.
 - `--list-categories` : List available game categories and exit
 - `--raw` : Show all events without tradeable filter (for debugging). Includes fetch stats.
+- `--no-cache` : Disable caching and fetch fresh data from the API.
+- `--max-total` : Maximum total events to fetch before early exit. Default: no limit. Useful for quick snapshots.
 - `--telegram` : Send results to Telegram. Requires `BOT_TOKEN` and `CHAT_ID` in environment variables.

 ## Output Format
@@ -118,13 +120,32 @@ Use `--raw` to disable the tradeable filter and see all match markets regardless

 ## Pagination

-The script fetches **ALL pages** until the API runs out of results (up to 100 pages as a safety cap).
+The script fetches **ALL pages** until the API runs out of results.
+
+### Parallel Fetching
+
+Pages are fetched in **parallel batches of 5** using ThreadPoolExecutor. This significantly reduces fetch time:
+
+| Scenario | Without Parallelization | With Parallelization |
+|----------|------------------------|---------------------|
+| 10 pages (50 events) | ~20s (2s per page × 10) | ~4s (2s per batch × 2 batches) |
+| 20 pages (100 events) | ~40s | ~8s |
+
+The script first fetches page 1 to determine total pages, then fetches remaining pages in parallel batches of 5.

 ## Rate Limiting

 - Exponential backoff: 2s → 4s → 8s → 16s → 32s
 - Max 5 retries before aborting

+## Caching
+
+Results are cached in `~/.cache/polymarket-browse/` with a **5-minute TTL** to reduce redundant API calls.
+
+- Use `--no-cache` to bypass the cache and fetch fresh data
+- Cached data is automatically used when available and not expired
+- Useful when running the script repeatedly (e.g., for monitoring)
+
 ## Odds Format

 All odds are shown in **cents** format:
--- a/skills/polymarket-browse/scripts/browse.py
+++ b/skills/polymarket-browse/scripts/browse.py
--- a/skills/polymarket-browse/tests/test_browse.py
+++ b/skills/polymarket-browse/tests/test_browse.py
Author	SHA1	Message	Date
shoko	c0f008ab8b	Merge pull request 'Fix: Event happening exactly now shows LIVE instead of In 0m' (#28 ) from fix/live-time-display into master	2026-03-26 18:48:23 +01:00
shoko	53c268511a	Add test for event happening exactly now shows LIVE Added test_get_time_data_live_exactly_now to prevent regression.	2026-03-26 17:46:53 +00:00
shoko	7f3b885521	Fix: Event happening exactly now shows 'LIVE' instead of 'In 0m' Changed condition from total_sec < 0 to total_sec <= 0 to catch the edge case where an event is happening right now.	2026-03-26 17:43:28 +00:00
shoko	e0ae6bbd14	Remove incorrect '100 pages safety cap' claim from SKILL.md This limit doesn't exist in the code.	2026-03-26 17:37:41 +00:00
shoko	f468a12ea4	Merge pull request 'Add parallel fetching, caching, and max_total parameter' (#26 ) from feat/parallel-fetch-cache into master	2026-03-26 18:23:20 +01:00
shoko	c75d123dfd	Update SKILL.md with new caching and parallel fetching documentation	2026-03-26 17:19:03 +00:00
shoko	9d1e328f53	Make page size calculation dynamic based on first API response - Uses actual event count from page 1 to calculate total_pages - Removes hardcoded '5' for events per page - API changes to page size will be handled automatically - Updated tests to match real API behavior (5 events per page)	2026-03-26 17:15:28 +00:00
shoko	09f3cb9066	Add comment explaining total_pages ceiling division calculation	2026-03-26 17:06:25 +00:00
shoko	1ae60f5661	Fix total_pages calculation bug and add tests - Fixed total_pages calculation: API returns 5 events/page, not PAGE_SIZE - This was causing partial=false positives when max_total was used - Updated tests to use correct pagination values	2026-03-26 16:54:41 +00:00
shoko	bab373ab8f	Add unit tests for parallelization, cache, and max_total - TestParallelFetchConcurrency: verify batch size of 5 and concurrency limit - TestCacheFunctions: test cache read/write error handling - TestMaxTotalParameter: test max_total event limiting	2026-03-26 16:43:13 +00:00
shoko	eafbdba4a5	Add parallel fetching, caching, and max_total parameter - Parallel page fetching with ThreadPoolExecutor (concurrency=5) - File-based cache with 5 min TTL in ~/.cache/polymarket-browse/ - New --no-cache flag to bypass cache - New --max-total parameter for early exit - Updated tests to work with new implementation	2026-03-26 16:29:25 +00:00
shoko	0a7911653b	Merge pull request 'Fix line lengths in browse.py' (#24 ) from fix/line-lengths into master	2026-03-26 16:42:53 +01:00
shoko	bae69043f8	Merge pull request 'Add type hints to browse.py' (#23 ) from feat/add-type-hints into master	2026-03-26 16:42:08 +01:00
shoko	d6c0342c0f	Fix line lengths in browse.py Split 2 lines that exceeded 120 chars in print_detail function. All 62 tests pass.	2026-03-26 15:40:21 +00:00
shoko	ce526b1aa3	Add type hints to browse.py - Added TypedDict classes for typed event/market structures - Added type annotations to all functions - Used Python 3.10+ union syntax (str \| None, dict[str, Any]) - All 62 tests pass	2026-03-26 15:35:18 +00:00
shoko	ae50fd14f0	Merge pull request 'Fix #14 : Refactor print_browse/send_to_telegram into single pipeline' (#22 ) from fix/issue-14-refactor-browse into master	2026-03-25 20:11:07 +01:00