Skip to content

feat(reputation): nightly snapshot sweep — they were never being created - #680

Closed
yggdrasilworld wants to merge 2 commits into
MergeFi:mainfrom
yggdrasilworld:fix/reputation-snapshot-cron-589
Closed

yggdrasilworld wants to merge 2 commits into
MergeFi:mainfrom
yggdrasilworld:fix/reputation-snapshot-cron-589

Conversation

@yggdrasilworld

@yggdrasilworld yggdrasilworld commented Oct 14, 2026 •

Copy link
Copy Markdown

What

computeAndSave only ran from the owner-only POST /reputation/:userId/recompute — GET /reputation/:userId returned nothing for anyone who never manually recomputed.

Fix

A nightly @Cron recomputes for every contributor with a claimed bounty.

Resolves #589

Payment: Base USDC 0xD40Df5c27cEa7dd78A3c345401995c319589334e

computeAndSave only ran from the owner-only POST recompute, so
GET /reputation/:userId returned nothing for anyone who never manually
recomputed. A nightly @Cron recomputes for every contributor with a
claimed bounty.

Resolves MergeFi#589
@vercel

vercel Bot commented Oct 14, 2026

Copy link
Copy Markdown

@yggdrasilworld is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

The shared base carried half-written merge artifacts: an orphaned
invokeOnLockedEscrow block in escrow.service.ts, duplicated returns plus
a dead parsePagination in sponsors.controller.ts, and missing
describe/it closers in github-webhooks.service.spec.ts. Aligned each
with the intended form so the branch typechecks and its specs load.
@yggdrasilworld
yggdrasilworld deleted the fix/reputation-snapshot-cron-589 branch October 19, 2026 02:34
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.

Nothing ever creates reputation snapshots — only a manual POST /recompute does, so every profile reads empty

1 participant