Differential D15423

[lib] Add sync logging
ClosedPublic
Actions

Authored by tomek on Oct 6 2025, 4:16 PM.

Details

Reviewers

kamil

Commits

rCOMM4ee8263ae01a: [lib] Add sync logging

Summary

Add a lot of logs so that we can investigate why the sync might be failing

https://linear.app/comm/issue/ENG-11363/missing-older-messages-after-farcaster-re-sync

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Oct 6 2025, 4:16 PM

Herald added a subscriber: ashoat. · View Herald TranscriptOct 6 2025, 4:16 PM

kamil published this revision for review.Oct 6 2025, 4:24 PM

kamil accepted this revision.

This revision is now accepted and ready to land.Oct 6 2025, 4:24 PM

tomek added inline comments.Oct 6 2025, 4:26 PM

lib/shared/farcaster/farcaster-hooks.js
713–732	This is the only significant change - we never call this function with other limits, so simplifying the code

Harbormaster completed remote builds in B35876: Diff 50543.Oct 6 2025, 4:36 PM

Fix Flow

This revision was landed with ongoing or failed builds.Oct 6 2025, 4:48 PM

Closed by commit rCOMM4ee8263ae01a: [lib] Add sync logging. · Explain Why

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM4ee8263ae01a: [lib] Add sync logging.

Harbormaster completed remote builds in B35878: Diff 50546.Oct 6 2025, 5:02 PM

Revision Contents
Changeset List

Path

Size

lib/

shared/

farcaster/

farcaster-hooks.js

287 lines

Diff 50543

View Options

lib/shared/farcaster/farcaster-hooks.js

[lib] Add sync loggingClosedPublicActions