Skip to content

refactor: internalize save query retry cadence - #7

Merged
ReallocAll merged 10 commits into
mainfrom
fix/internalize-query-retry
Oct 6, 2026
Merged

ReallocAll merged 10 commits into
mainfrom
fix/internalize-query-retry

Conversation

@ReallocAll

Copy link
Copy Markdown
Owner

Summary

Follow-up to v0.1.3: the save-query retry count is an internal capture-state-machine detail and should not be exposed in config.toml.

This change:

  • removes capture.query_retries from config parsing, defaults, docs, and plugin wiring;
  • retries transient save query misses every 10 server ticks;
  • keeps the existing 15-second hard query deadline;
  • preserves the v0.1.3 fix that accepts a valid manifest even when Endstone's command-success boolean is false;
  • keeps fail-safe save resume recovery unchanged;
  • adds regression coverage for the exact 10-tick retry cadence.

Existing configs that still contain query_retries remain harmless because unknown capture keys are ignored.

@ReallocAll
ReallocAll merged commit b28450c into main Oct 6, 2026
1 check passed
@ReallocAll
ReallocAll deleted the fix/internalize-query-retry branch October 9, 2026 04:45
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