Skip to content

Conversation

@philippemoro
Copy link
Contributor

@philippemoro philippemoro commented Oct 28, 2025

Summary

This pull request introduces support for explicitly passing the limit parameter, as described in the Rollbar API documentation.

Testing
It appears that some of the token limit issues reported in rollbar/rollbar-mcp-server#42
may have been caused by requests using a higher-than-intended default limit value.
According to the documentation, the intended default is 20. When explicitly setting limit:20, the response size (and number of tokens) was significantly reduced

@philippemoro philippemoro force-pushed the pm/add-limit-param-for-list-items branch from 38608e3 to dd37c64 Compare October 28, 2025 18:08
@brianr
Copy link
Member

brianr commented Nov 11, 2025

Thank you!

Just a note on the docs - the doc you linked is for a different endpoint, for fetching instances (called Occurrences in the UI). This tool is for getting items. For items, the default is 100 per page. In any case, exposing the limit param and defaulting it to 20 makes sense.

@brianr
Copy link
Member

brianr commented Nov 11, 2025

Merged these changes in #59. Thanks again!

@brianr brianr closed this Nov 11, 2025
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.

2 participants