Skip to content

feat: configurable snapshot limit, default ten pages - #48

Merged
nybarius merged 3 commits into
mainfrom
work/snapshot-limit-config
Oct 20, 2026
Merged

nybarius merged 3 commits into
mainfrom
work/snapshot-limit-config

Conversation

@nybarius

Copy link
Copy Markdown
Owner

A repository with more open pull requests than the fixed snapshot limit (300) read no round at all: every tick retried with SNAPSHOT_LIMIT_EXCEEDED, including the configuration change that would have raised it. [train] snapshot_limit (a positive whole count, default 1000 = ten 100-row pages) now bounds the listing, and app, cli replay and doctor all build the repository client with it. RED c16f014, GREEN 694f497; full suite 420 OK.

svrf-maintainer and others added 3 commits October 12, 2026 15:34
…reaches every repository client

A repository with more open pull requests than the fixed 300 read no round at all. [train] snapshot_limit
(default 1000, a positive whole count) now bounds the listing, and app, cli replay and doctor build the client
with it.
Preserve the 1000-PR ceiling and tick_seconds from current main; pass the original full suite.
@nybarius
nybarius merged commit c5a90f2 into main Oct 20, 2026
8 checks passed
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.

1 participant