Skip to content

feat(notifications): top-N contributor summary section for the maintainer recap (public-safe) #2244

Description

@JSONbored

Digest content section (#1963 'top-N contributor summary, no scoring/reward internals'): a leaderboard of the window's most-merged contributor logins with merged-PR counts only — NO scoring, reward, or trust internals. Each login + line MUST pass isPublicSafeText (src/signals/redaction.ts:48) before emitting, matching the public-safe framing the notifications service already enforces (src/notifications/service.ts:38).

Deliverables

  • buildTopContributorsRecapSection(report, limit) → sorted [{login, merged}] capped at limit, ties broken by login
  • Reject any line failing isPublicSafeText (src/signals/redaction.ts:48); test that a reward/score-bearing string is dropped
  • Unit test: sorting, tie-break, the cap, and the public-safe gate (both pass and reject branches)

References

  • src/signals/redaction.ts:48
  • src/notifications/service.ts:38
  • src/review/alerts.ts:158

Part of #1963.


size: S · held for later release (gittensor:feature once activated)

Metadata

Metadata

Assignees

No one assigned

    Labels

    gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.

    Projects

    Status
    Done

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions