Skip to content

Add author reputation signal to trending algorithm#582

Merged
realproject7 merged 1 commit intomainfrom
task/579-author-reputation-trending
Mar 27, 2026
Merged

Add author reputation signal to trending algorithm#582
realproject7 merged 1 commit intomainfrom
task/579-author-reputation-trending

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

Fixes #288

  • Add computeAuthorReputation() combining five sub-signals: follower_count (Farcaster, log-scaled), x_followers_count (Twitter, log-scaled), x_verified (boolean boost), neynar_score (0-1), and quotient_score (0-1000 normalized)
  • Integrate as 6th trend signal at 20% weight; rebalance existing signals (rating 20%, price 15%, TVL 15%, continuation 15%, recency 15%)
  • Batch-fetch writer user rows via primary_address + verified_addresses (overlaps query) to avoid N+1 — two parallel queries regardless of storyline count
  • Gracefully returns 0 reputation for writers with no user record

Test plan

  • Verify trending page loads without errors
  • Confirm storylines by writers with higher reputation scores rank higher (all else equal)
  • Check that storylines by unknown writers (no user row) still appear with 0 reputation penalty
  • Validate no N+1 queries in server logs (only 2 user queries total)

🤖 Generated with Claude Code

Implement computeAuthorReputation() combining follower_count,
x_followers_count, x_verified, neynar_score, and quotient_score
as the sixth trend signal at 20% weight. Batch-fetches writer
user rows via primary_address and verified_addresses to avoid
N+1 queries. Rebalances existing signal weights accordingly.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Mar 27, 2026 6:07am

Request Review

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

The PR matches issue #581: it adds a composite author reputation signal, integrates it as the sixth trending input at 20% weight, and keeps the user lookup batched instead of introducing N+1 queries.

Findings

  • No blocking issues found.

Decision

Approve. The implementation is scoped correctly, follows the existing Supabase/address-normalization patterns, and the ranking changes are coherent with the acceptance criteria.

@realproject7 realproject7 merged commit 16f5d57 into main Mar 27, 2026
5 checks passed
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.

2 participants