Block a user
security(polymarket-browse): add MAX_RESPONSE_SIZE limit to prevent memory exhaustion
Updated the implementation based on your feedback:
Changes Made
Dynamic Response Size Limit instead of fixed 10MB:
MAX_RESPONSE_SIZE_MULTIPLIER = 10 # 10x multiplier
MAX_RESPON…
security(polymarket-browse): add MAX_RESPONSE_SIZE limit to prevent memory exhaustion
Great questions! Let me clarify:
Current Implementation (PR #35)
- Current limit: 10MB fixed (
MAX_RESPONSE_SIZE = 10 * 1024 * 1024) - Applies to: Only the
public-search(global-sea…
security(polymarket-browse): validate --detail argument and show error if out of range
You raise a valid design concern. The current --detail N design is indeed flawed:
- List position is not stable - Position 3 can become position 7 in the next query
- **No one wants…
[Design Discussion] Redesign --detail to use event ID instead of list index
security(polymarket-browse): use proper URL encoding for --search parameter
Good point! I added unit tests for URL encoding of special characters:
Test Coverage Added
security(polymarket-browse): validate --detail argument and show error if out of range
Great question! Here is the case example:
Before the Patch
User runs: polymarket-browse --category "Counter Strike" --detail 100
Scenario: API returns only 5 match events.
**What…
fix(polymarket-browse): add --timezone CLI argument for display timezone
Tradeoffs: UTC+7 vs Asia/Jakarta
docs(polymarket-browse): update SKILL.md with troubleshooting, examples, changelog, credits
Review requested
security(polymarket-browse): add MAX_RESPONSE_SIZE limit to prevent memory exhaustion
Review requested