fix: relay sync create/update classification to prevent false create mode on updates #56

Merged
hodlbod merged 1 commits from userAdityaa/caravel:sync-classification into master 2026-05-01 14:21:37 +00:00
Contributor

The change replaces the incorrect use of synced == 0 for determining whether a relay is new with a proper check based on whether the relay has ever successfully synced. A new helper and classifier were introduced, and all infra sync entry points were updated to use this logic. As a result, only first-time syncs trigger create behavior, while updates and retries correctly use update semantics without regenerating secrets.

closes #55

The change replaces the incorrect use of `synced == 0` for determining whether a relay is new with a proper check based on whether the relay has ever successfully synced. A new helper and classifier were introduced, and all infra sync entry points were updated to use this logic. As a result, only first-time syncs trigger create behavior, while updates and retries correctly use update semantics without regenerating secrets. closes #55
userAdityaa added 1 commit 2026-05-01 10:10:24 +00:00
hodlbod merged commit 29f657635c into master 2026-05-01 14:21:37 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coracle/caravel#56