Abort & report on GitHub secondary rate-limit during push#916
Merged
SorraTheOrc merged 6 commits intomainfrom Mar 11, 2026
Merged
Abort & report on GitHub secondary rate-limit during push#916SorraTheOrc merged 6 commits intomainfrom
SorraTheOrc merged 6 commits intomainfrom
Conversation
…timestamp - Batch-aware progress: Push phase now shows 'Push: Batch x/y Item a/b' format using closure variables updated by the batch loop. Other phases (Hierarchy, Comments, etc.) retain the existing 'Label: current/total' format. - Extract writeProgressMessage helper: centralizes TTY output with padding, dedup, and newline-on-completion logic. - Per-batch error reporting: after each batch completes, any errors are immediately logged to the log file and printed to stderr, rather than being deferred to the final summary. - Per-batch timestamp update: writeLastPushTimestamp is now called after each successful batch so that interrupted or re-run pushes skip already-synced batches. The pushStartTimestamp captured before processing ensures items modified during the push window are still re-processed on the next run. A safety-net final write covers the zero-item case. --no-update-timestamp continues to suppress all writes. - Pre-filter module import is captured eagerly during the pre-filter path so writeLastPushTimestamp is available even when --all bypasses pre-filtering, and falls back to a standalone import when needed.
…rottle state-sync writes (WL-0MMLD7CV908H2HEK, WL-0MMLD7FII0N38U8R)
…ary rate limit and expose helper
…paths (sync+async), add SharedArrayBuffer sleep for sync backoff; throttle state-sync writes
…during push; add SecondaryRateLimitError; wire runners -> throw; top-level abort and report
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stop and report on GitHub secondary rate-limit (abuse) during 'wl gh push'.\n\nChanges in this branch: add SecondaryRateLimitError, backoff/retry for gh calls, and top-level abort/reporting with structured JSON. See work item WL-0MMLT5UJC02BRTDB for planned tests and follow-ups.