Skip to content

Fix duplicate watcher weighting in repository rankings - #423

Draft
GaneshDeshmane wants to merge 1 commit into
AOSSIE-Org:mainfrom
GaneshDeshmane:fix/duplicate-watcher-score-clean
Draft

GaneshDeshmane wants to merge 1 commit into
AOSSIE-Org:mainfrom
GaneshDeshmane:fix/duplicate-watcher-score-clean

Conversation

@GaneshDeshmane

Copy link
Copy Markdown

Addressed Issues:

Fixes the incorrect repository ranking caused by double-counting stargazers through watchers_count.

Screenshots/Recordings:

Not applicable — this change affects repository ranking logic and does not introduce UI changes.

Additional Notes:

  • Removed watchers_count from the repository scoring formula because it duplicates the stargazer count in GitHub's repository API.
  • Added regression tests to verify that watchers_count does not affect repository scores.
  • Preserved scoring based on stars, forks, and repository activity.
  • Tests passed: 101 tests across 13 test files.
  • Production build succeeds. Vite reports an existing large-chunk warning.

Checklist

  • My code follows the project's code style and conventions
  • I have made corresponding changes to the documentation (not applicable; no documentation changes were needed)
  • My changes generate no new warnings or errors (the existing Vite and test warnings remain)
  • I have joined the [Discord server](https://discord.gg/hjUhu33uAn) and I will share a link to this PR with the project maintainers there
  • I have read the [Contributing Guidelines](./CONTRIBUTING.md)

⚠️ AI Notice - Important!

I have reviewed the changes and added regression tests to validate the updated scoring behavior. The full test suite passes successfully.

@coderabbitai

coderabbitai Bot commented Oct 27, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added no-issue-linked PR has no linked issue javascript JavaScript/TypeScript changes tests Test changes size/M 51-200 lines changed first-time-contributor First time contributor labels Oct 27, 2026
@gitcordapp

gitcordapp Bot commented Oct 28, 2026

Copy link
Copy Markdown

Link your account with Gitcord

Thanks for opening this PR, @GaneshDeshmane!

To receive Discord notifications and contributor tracking for this organization:

  1. Join Discord: https://discord.gg/hjUhu33uAn
  2. In Discord, run /link GaneshDeshmane
  3. Paste the verification code into your GitHub bio (or a public gist)
  4. Click Verify in Discord (or run /verify-link GaneshDeshmane)

Once linked, Gitcord can notify you about reviews, merges, and more.

— Posted by Gitcord

This branch has not been deployed

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

Labels

first-time-contributor First time contributor javascript JavaScript/TypeScript changes no-issue-linked PR has no linked issue size/M 51-200 lines changed tests Test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant