Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat(server): default sync interval to 5m and document evented refresh#1320

Merged
jonnii merged 1 commit into
main from
jonnii/20260623030405/default-sync-interval-to-5m-and-document-evented
Jun 24, 2026
Merged

feat(server): default sync interval to 5m and document evented refresh #1320
jonnii merged 1 commit into
main from
jonnii/20260623030405/default-sync-interval-to-5m-and-document-evented

Conversation

@jonnii

@jonnii jonnii commented Jun 23, 2026
edited
Loading

Copy link
Copy Markdown
Collaborator

🔒 This PR has been locked (consolidating). To unlock it, run st unlock.

The sync loop was opt-in: STACKIT_SYNC_INTERVAL unset meant 0, which disabled it
entirely, so a server with no explicit interval never pulled remote changes. With
webhooks now able to drive immediate refreshes, the interval becomes the
backstop — so it should exist by default.

  • envSyncInterval defaults to 5m when STACKIT_SYNC_INTERVAL is unset; "0" still
    explicitly disables, and an unparseable value falls back to the default rather
    than silently disabling the loop.
  • docs/deploy.md: explain the three refresh triggers (interval, webhook, manual),
    document webhook setup (payload URL, secret, push event), the manual-sync
    endpoint, and how local servers stay current without webhooks (fsnotify
    watcher + manual sync). Calls out that GitHub sends no push event for
    refs/stackit/metadata/*, so the interval loop must stay on as a backstop.

Stack

💡 Notice: This PR is part of a stack merge into branch stack-merge-stack-1782266218.

Auto-generated by Stackit


Merged via consolidation into #1326 by jonnii

The sync loop was opt-in: STACKIT_SYNC_INTERVAL unset meant 0, which disabled it
entirely, so a server with no explicit interval never pulled remote changes. With
webhooks now able to drive immediate refreshes, the interval becomes the
backstop — so it should exist by default.
- envSyncInterval defaults to 5m when STACKIT_SYNC_INTERVAL is unset; "0" still
 explicitly disables, and an unparseable value falls back to the default rather
 than silently disabling the loop.
- docs/deploy.md: explain the three refresh triggers (interval, webhook, manual),
 document webhook setup (payload URL, secret, push event), the manual-sync
 endpoint, and how local servers stay current without webhooks (fsnotify
 watcher + manual sync). Calls out that GitHub sends no push event for
 refs/stackit/metadata/*, so the interval loop must stay on as a backstop.
This was referenced Jun 23, 2026
Base automatically changed from jonnii/20260623030002/add-manual-sync-endpoint-for-on-demand-refresh to main June 24, 2026 02:00
@jonnii jonnii merged commit 0a8ec1d into main Jun 24, 2026
7 of 12 checks passed
@jonnii jonnii deleted the jonnii/20260623030405/default-sync-interval-to-5m-and-document-evented branch June 24, 2026 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /