Skip to content

feat: badges backend#3767

Merged
gaspergrom merged 1 commit into
mainfrom
feat/leaderboard-backend
Jan 20, 2026
Merged

feat: badges backend#3767
gaspergrom merged 1 commit into
mainfrom
feat/leaderboard-backend

Conversation

@gaspergrom
Copy link
Copy Markdown
Contributor

@gaspergrom gaspergrom commented Jan 20, 2026

Note

Introduces a total rows count for each leaderboard snapshot and surfaces it in query results.

  • Extends leaderboards_copy_ds schema with totalCount and updates leaderboards.pipe to select it
  • Updates all copy pipes (commits, stars, forks, package-downloads, codebase-size, avg-commits-per-author, small-teams-massive-output, fastest-mergers, fastest-responders, resolution-rate, active-contributors, active-organizations, contributors, organizations) to compute count() OVER () as totalCount and write it to the datasource

Written by Cursor Bugbot for commit 5355ef7. This will update automatically on new commits. Configure here.

Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
@gaspergrom gaspergrom requested a review from epipav January 20, 2026 13:05
@gaspergrom gaspergrom self-assigned this Jan 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@gaspergrom gaspergrom merged commit dda1c33 into main Jan 20, 2026
17 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@gaspergrom gaspergrom deleted the feat/leaderboard-backend branch January 20, 2026 13:06
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

p.githubHandle as githubHandle,
'' as status
'' as status,
count() OVER () as totalCount
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Misleading totalCount for limited leaderboards

Medium Severity

The count() OVER () window function computes before LIMIT 100 is applied, so totalCount reflects the full count (potentially thousands) while only 100 rows are actually stored. API consumers querying the contributors or organizations leaderboards will see a totalCount suggesting thousands of pageable items, but attempting to paginate beyond the first 100 rows will return empty results.

Additional Locations (1)

Fix in Cursor Fix in Web

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.

1 participant