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

refactor(api): extract Syncer.SyncRepo for per-repo sync triggers#1315

Merged
jonnii merged 1 commit into
main from
jonnii/20260623024716/extract-Syncer.SyncRepo-for-per-repo-sync-triggers
Jun 24, 2026
Merged

refactor(api): extract Syncer.SyncRepo for per-repo sync triggers #1315
jonnii merged 1 commit into
main from
jonnii/20260623024716/extract-Syncer.SyncRepo-for-per-repo-sync-triggers

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.

Pull the per-repo unit of work out of the interval loop so a future webhook
receiver can trigger a refresh for a single repo by its GitHub coordinates,
not just the ticker iterating over managed entries.

  • Registry.FindManaged(owner, name) resolves a managed mirror case-insensitively;
    it deliberately never returns the unmanaged -cwd working repo, which must not
    be mirror-fetched (mirror-fetch detaches HEAD).
  • Syncer.syncEntry now returns an error instead of logging-and-swallowing;
    syncOnce logs failures, and the new SyncRepo(owner, name) wrapper resolves the
    entry and delegates, returning ErrRepoNotManaged for un-onboarded repos.

Pure refactor: the interval loop behaves identically. Sets up the webhook-driven
refresh stack.


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

Pull the per-repo unit of work out of the interval loop so a future webhook
receiver can trigger a refresh for a single repo by its GitHub coordinates,
not just the ticker iterating over managed entries.
- Registry.FindManaged(owner, name) resolves a managed mirror case-insensitively;
 it deliberately never returns the unmanaged -cwd working repo, which must not
 be mirror-fetched (mirror-fetch detaches HEAD).
- Syncer.syncEntry now returns an error instead of logging-and-swallowing;
 syncOnce logs failures, and the new SyncRepo(owner, name) wrapper resolves the
 entry and delegates, returning ErrRepoNotManaged for un-onboarded repos.
Pure refactor: the interval loop behaves identically. Sets up the webhook-driven
refresh stack.
This was referenced Jun 23, 2026
Base automatically changed from jonnii/20260623024214/drive-deploy-posture-from-STACKIT_ENV-not-PORT to main June 24, 2026 02:00
@jonnii jonnii merged commit 4a1cacf into main Jun 24, 2026
7 of 12 checks passed
@jonnii jonnii deleted the jonnii/20260623024716/extract-Syncer.SyncRepo-for-per-repo-sync-triggers 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 によって変換されたページ (->オリジナル) /