Skip to content

Conversation

@Ivansete-status
Copy link
Collaborator

@Ivansete-status Ivansete-status commented Sep 30, 2025

Original PR: #3318
Original author: @s-tikhomirov

The original PR got abruptly closed after a deep cleanup and refactor applied by this on 2025-09-30


Description

This is a feature branch for the service incentivization POC, which includes eligibility and reputation systems integrated into Lightpush (v3) behind a feature flag.

A Lightpush client uses the reputation manager to keep track of peers it used previously to publish messages. Peers that had failed at publishing a message get assigned a negative reputation and will not be chosen for future requests. If, however, a neutral- or positive-reputation peer isn't selected by the peer manager after a maximum number of attempts (10), a negative-reputation peer is still used.

This is a countinuaton of #3166, #3264, #3293, and #3309 with associated deliverable logos-messaging/pm#245.

Changes

  • merge the prior work (initially for Lightpush v2) onto Lightpush v3
  • move the reputation-based peer selection logic into PeerManager
  • add or adapt tests for reputation functionality

* chore: remove unnecessary comment

* move reputation manager logic to peer manager

* remove peer selection from lightpush client

* chore: lint fix
@s-tikhomirov
Copy link
Contributor

AFAIU, this branch is not intended to be merged. The PoC was presented at the Townhall of 2025-08-08, and continues in the form of the incentivization MVP discussions.

Can / should we close this issue? CC @jm-clius

@jm-clius
Copy link
Contributor

jm-clius commented Oct 6, 2025

Yes, I think we want to maintain the branch at least until we make progress with the MVP, but there's no need for an ongoing, open PR. If we want to keep the feature branches clean, a fork of the repo could also be considered.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants