docs(polymarket-browse): update SKILL.md with troubleshooting, examples, changelog, credits #37

shoko · 2026-03-26T20:16:39+01:00

shoko commented

2026-03-26 20:16:39 +01:00

Summary

Update SKILL.md with missing documentation as requested in #11 and #12.

Changes

Issue #12 - --detail argument clarification

Clarify --detail only applies to MATCH markets (not non-match)
Document error behavior when out of range

Issue #11 - Missing documentation

Troubleshooting section: Common issues like partial fetch, no markets, match disappearance, Telegram problems
Examples section: Basic usage, team search, filtering, --detail usage, debugging, timezone
Changelog section: Version history (v0.0.1, v0.0.2)
Credits section: Author and repository info

Testing

70/70 tests passing

## Summary Update SKILL.md with missing documentation as requested in #11 and #12. ## Changes ### Issue #12 - --detail argument clarification - Clarify --detail only applies to MATCH markets (not non-match) - Document error behavior when out of range ### Issue #11 - Missing documentation - **Troubleshooting section**: Common issues like partial fetch, no markets, match disappearance, Telegram problems - **Examples section**: Basic usage, team search, filtering, --detail usage, debugging, timezone - **Changelog section**: Version history (v0.0.1, v0.0.2) - **Credits section**: Author and repository info ## Testing 70/70 tests passing

shoko added 1 commit 2026-03-26 20:16:40 +01:00

docs(polymarket-browse): update SKILL.md with troubleshooting, examples, changelog, credits 32ed72868b

- Clarify --detail argument (only for MATCH markets, error on out of range)
- Add Troubleshooting section with common issues
- Add Examples section with usage patterns
- Add Changelog section
- Add Credits section

shoko reviewed 2026-03-27 01:51:05 +01:00

shoko left a comment

Review requested

shoko added the need review label 2026-03-27 01:52:47 +01:00

shoko added 20 commits 2026-03-27 03:40:59 +01:00

feat(polymarket-browse): add --starts-before filter for match events cc197b0c7e

- Add --starts-before CLI argument accepting Unix timestamp
- Filter match events to only show those starting before timestamp
- LIVE events are always included regardless of timestamp
- Update SKILL.md documentation
- Add TestStartsBeforeFilter with 3 unit tests

fix(polymarket-browse): add --timezone CLI argument for display timezone 0a1aab7883

- Add parse_timezone() function supporting UTC+X format
- Add --timezone argument (default: UTC+7)
- Module-level _DISPLAY_TZ controls all time display formatting
- get_header_date() and _get_time_data() use _DISPLAY_TZ
- Add TestTimezoneParsing unit tests
- Update SKILL.md documentation

security(polymarket-browse): replace bare except: with specific exception handling 8bd76f3301

- Lines 386, 400: except: changed to except (ValueError, TypeError):
- ValueError: datetime.fromisoformat parse failure
- TypeError: input is not a string
- Prevents swallowing KeyboardInterrupt, SystemExit, MemoryError

security(polymarket-browse): validate --detail argument and show error if out of range 3928cdef7c

- Add sys import for stderr/exit
- Validate --detail index before accessing array
- Show error with available range instead of silent fallback to first event
- Exit with code 1 if --detail is out of range

security(polymarket-browse): use proper URL encoding for --search parameter bb7eebf502

- Import quote from urllib.parse
- Replace q.replace(' ', '%20') with quote(q, safe='')
- Properly encodes: &, =, %, +, #, ?, and other special chars
- Prevents URL injection attacks

security(polymarket-browse): add MAX_RESPONSE_SIZE limit to prevent memory exhaustion 36a7e8b3eb

- Add MAX_RESPONSE_SIZE = 10MB constant
- Check response size before json.loads() in fetch_page()
- Raises ValueError if response exceeds limit
- Prevents memory exhaustion from malicious/gigantic API responses

security(polymarket-browse): add token bucket rate limiter for API calls c0484ab340

- Add RateLimiter class (token bucket algorithm)
- Thread-safe for use with ThreadPoolExecutor
- RATE_LIMIT_CALLS = 10 per RATE_LIMIT_WINDOW = 1 second
- _rate_limiter.acquire() called before each API request
- Also add MAX_RESPONSE_SIZE check in fetch_page

docs(polymarket-browse): create SECURITY.md tracking audit findings 350fe17e87

- Document fixed security issues from 2026-03-25 audit
- Track all 7 security issues and their fixes
- Add reporting instructions

test(polymarket-browse): add URL encoding unit tests 3016d1287c

Add TestUrlEncoding class testing quote() encodes:
- Space -> %20
- & -> %26
- = -> %3D
- % -> %25
- + -> %2B
- ( -> %28
- ) -> %29
- # -> %23

security(polymarket-browse): improve response size limit with dynamic calculation 2c636048e7

- Replace fixed 10MB limit with dynamic calculation
- get_max_response_size() computes limit based on PAGE_SIZE * multiplier
- Uses 10x multiplier (e.g., PAGE_SIZE=50 -> ~500KB * 10 = ~5MB)
- Clamped between 10MB minimum and 100MB maximum
- Formula: max(PAGE_SIZE * multiplier, 10MB) capped at 100MB

Merge branch 'pr-30' into 0.0.3-draft b4148570f4

Merge branch 'pr-31' (timezone) into 0.0.3-draft aef5f79dad

Conflicts resolved:
- browse.py: keep both --starts-before and --timezone args
- test_browse.py: combine TestStartsBeforeFilter and TestTimezoneParsing
- SKILL.md: combine documentation for both args

Merge branch 'pr-32' into 0.0.3-draft 2b7a2bda90

Merge branch 'pr-33' into 0.0.3-draft 893243ba39

Merge branch 'pr-34' (URL encoding) into 0.0.3-draft ca13a2e194

Merge branch 'pr-35' into 0.0.3-draft 54679cac44

Merge branch 'pr-36' (rate limiting) into 0.0.3-draft a2aea41ae3

Combined MAX_RESPONSE_SIZE dynamic calculation with RateLimiter class.

Merge branch 'pr-38' into 0.0.3-draft efad771e85

Merge remote-tracking branch 'origin/0.0.3-draft' into docs/11-12-skill-updates b9ad8ac41b

docs(polymarket-browse): update SKILL.md for v0.0.3 release 3d7b136cca

- Bump version to 0.0.3
- Update changelog with all security fixes and features
- Add TokenBucket rate limiting to Rate Limiting section
- Add URL encoding note

shoko added 1 commit 2026-03-27 03:44:30 +01:00

docs(polymarket-browse): remove SECURITY.md - overhead for users/agents 0902bfafaa

Security findings tracked in GitHub issues instead.

shoko added 1 commit 2026-03-27 03:48:33 +01:00

docs(polymarket-browse): remove Changelog and Credits from SKILL.md 5bfec66a34

Changelog tracked via git tags instead.

shoko closed this pull request

2026-03-27 04:09:45 +01:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shoko/jujutsu-skills#37