Skip to content

fix: add missing health score v2 category columns to project_repo_insights pipe - #4511

Merged
gaspergrom merged 1 commit into
mainfrom
fix/CM-IN-1239-project-repo-insights-columns
Aug 26, 2026
Merged

fix: add missing health score v2 category columns to project_repo_insights pipe#4511
gaspergrom merged 1 commit into
mainfrom
fix/CM-IN-1239-project-repo-insights-columns

Conversation

@gaspergrom

Copy link
Copy Markdown
Contributor

Summary

  • project_repo_insights.pipe (backs Insights' collection detail page project/repo list) was missing maintainerHealthScoreV2, securitySupplyChainScoreV2, and developmentActivityScoreV2 in its SELECT list, even though the sibling project_insights.pipe already exposed them.
  • This caused the collection page's Health Score tooltip category breakdown to show placeholder dashes instead of real values (see insights#2114).

JIRA: IN-1239

Deploy status

Already validated and deployed to staging and production via tb push.

Test plan

  • Verified via curl against the collection project-repos endpoint that the 3 columns now return real values (e.g. PixiJS: 37/12/22)
  • Verified live in browser — collection detail page tooltip shows real numbers

…ights pipe (CM-IN-1239)

project_repo_insights.pipe (backs the Insights collection detail page's
project/repo list) was missing maintainerHealthScoreV2,
securitySupplyChainScoreV2, and developmentActivityScoreV2 in its SELECT
list, even though its sibling project_insights.pipe already exposed them.
This left the collection page's Health Score tooltip category breakdown
showing placeholder values instead of real numbers.

Already validated and deployed to staging and production.

Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Copilot AI balanced review requested due to automatic review settings August 26, 2026 11:34
@cursor

cursor Bot commented Aug 26, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Read-only API surface change: three existing datasource columns are added to a Tinybird pipe SELECT with no auth or data-mutation impact.

Overview
The project_repo_insights Tinybird pipe now returns maintainerHealthScoreV2, securitySupplyChainScoreV2, and developmentActivityScoreV2 in its endpoint SELECT, matching project_insights.pipe. The pipe description documents these as the Health Score v2 category rollups (0–40 / 0–35 / 0–25) that sum to healthScoreV2.

This fixes the Insights collection detail project/repo list, where the Health Score tooltip category breakdown was showing dashes because the API response omitted those fields even though they exist on project_insights_copy_ds.

Reviewed by Cursor Bugbot for commit d06f747. Bugbot is set up for automated code reviews on this repo. Configure here.

@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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Exposes missing Health Score v2 category values through the combined project/repository Insights endpoint.

Changes:

  • Adds three category score columns to the endpoint.
  • Documents their ranges and nullability.
  • One documentation accuracy issue was identified.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

- `page`: Optional integer for pagination offset calculation, defaults to 0
- Response: Project and repository records with insights metrics
- `healthScoreV2`, `healthLabel`, `lifecycleLabel`, `impactScore`, `impactLabel` are the Akrites-methodology fields (see `project_insights_copy_ds` for band definitions); may be null when the record has no linked package data. `healthScoreV2` is distinct from the legacy `healthScore` field.
- `maintainerHealthScoreV2` (0-40), `securitySupplyChainScoreV2` (0-35), `developmentActivityScoreV2` (0-25) are the rollup of the Health Score v2 category breakdown (IN-1212) — the three categories that sum to `healthScoreV2`. Null when the record has no value for that category.
@gaspergrom
gaspergrom requested a review from joanagmaia August 26, 2026 11:40
@gaspergrom
gaspergrom merged commit 9a943a4 into main Aug 26, 2026
17 checks passed
@gaspergrom
gaspergrom deleted the fix/CM-IN-1239-project-repo-insights-columns branch August 26, 2026 11:41
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.

3 participants