Skip to content

Add pollInterval & minRefreshInterval settings - #656

Open
hannah-computer wants to merge 1 commit into
brychanrobot:mainfrom
hannah-computer:hannah/pr/push-kkkzzkkkznpn
Open

hannah-computer wants to merge 1 commit into
brychanrobot:mainfrom
hannah-computer:hannah/pr/push-kkkzzkkkznpn

Conversation

@hannah-computer

Copy link
Copy Markdown
Contributor

The default 60s polling frequency costs around 3300 GraphQL points/hour, which is over half of the 5000 points/hour default quota and may be too much in combination with other tools (it happened to me! Made worse by a bug in gh that ended up caching the out-of-quota error so it stuck around longer than necessary, otherwise I'd not have noticed tbqh)

This makes the interval configurable and adds a minimum interval for event-triggered polls. Manually triggered polls aren't affected.

Comment thread src/test/code-forge-service.test.ts Outdated
Comment thread src/test/code-forge-service.test.ts Outdated
Comment thread src/test/log-view-controller.test.ts Outdated
Comment thread src/test/log-view-controller.test.ts Outdated
@hannah-computer

Copy link
Copy Markdown
Contributor Author

Thanks for the review! I've put my changes in a separate commit for now since that works better with the Github UI, or would you prefer them squashed directly?

@brychanrobot

Copy link
Copy Markdown
Owner

Please squash into a single commit

The default 60s polling frequency costs around 3300 GraphQL points/hour,
which may be too much in combination with other tools.
@hannah-computer
hannah-computer force-pushed the hannah/pr/push-kkkzzkkkznpn branch from 81561bb to c936785 Compare October 13, 2026 12:07
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