Skip to content

feat: standalone widget 07 category maximum table - #2172

Merged
gaspergrom merged 1 commit into
release/IN-1276-health-score-coveragefrom
feat/IN-1292
Sep 9, 2026
Merged

feat: standalone widget 07 category maximum table#2172
gaspergrom merged 1 commit into
release/IN-1276-health-score-coveragefrom
feat/IN-1292

Conversation

@gaspergrom

@gaspergrom gaspergrom commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Standalone widget code only for IN-1292, widget 07 "Repositories scoring full marks" (Health Score Coverage report, epic IN-1276) — NOT yet wired into any shared file (report view, API service, TanstackKey enum, shared types, shared data). Wiring happens once this widget's turn comes up in the sequential release-branch merge order (matches the pattern of the prior seven widget PRs on this epic, e.g. #2168, #2171).

Adds:

  • types/report/health-score-coverage-category-maximum.types.ts
  • server/data/tinybird/report/health-score-coverage-category-maximum.ts (mapper + fetcher) and its Vitest test
  • server/api/report/health-score-coverage/category-maximum.get.ts
  • app/components/modules/report/health-score-coverage/services/category-maximum.query.ts
  • app/components/modules/report/health-score-coverage/components/category-maximum-table.vue (table, per the ticket's own file name — matches the sibling table widget's github-security-funnel.vue convention of a layout-shape suffix)

Table columns: Category, Maximum, Repositories at maximum. Rows: Maintainer health (40), Development activity (25), Security & supply chain (35). Copy is verbatim from the design (eyebrow, title, text, footnote, caption).

crowd.dev pipe dependency

Consumes health_score_report_category_maximum from crowd.dev PR linuxfoundation/crowd.dev#4585. That pipe is not yet deployed to Tinybird — per the ticket-supervisor state file, the deploy was deliberately withheld by the owner pending their own review pass. This insights PR cannot be QA'd against live data until the pipe is deployed.

Test plan

  • pnpm test --run — new mapper/fetcher test passes (4 tests)
  • pnpm tsc-check — clean
  • pnpm lint — clean (auto-fixed formatting)
  • Manual QA against live Tinybird data — blocked until pipe #4585 is deployed
  • release/IN-1276-health-score-coverage - ⚠️ No PR associated with branch
    • feat: standalone widget 07 category maximum table 👈

Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Copilot AI balanced review requested due to automatic review settings September 9, 2026 07:38

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

Adds the standalone “Repositories scoring full marks” widget for the Health Score Coverage report.

Changes:

  • Adds Tinybird mapping, API, query, and shared types.
  • Adds a three-row category maximum table.
  • Adds four mapper/fetcher tests.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
frontend/types/report/health-score-coverage-category-maximum.types.ts Defines widget data types.
frontend/server/data/tinybird/report/health-score-coverage-category-maximum.ts Fetches and maps Tinybird data.
frontend/server/data/tinybird/report/health-score-coverage-category-maximum.test.ts Tests mapping and fetching.
frontend/server/api/report/health-score-coverage/category-maximum.get.ts Exposes the report endpoint.
frontend/app/components/modules/report/health-score-coverage/services/category-maximum.query.ts Adds the TanStack query.
frontend/app/components/modules/report/health-score-coverage/components/category-maximum-table.vue Renders loading, empty, and table states.

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

@gaspergrom
gaspergrom requested a review from epipav September 9, 2026 09:52
@gaspergrom
gaspergrom merged commit 606eab7 into release/IN-1276-health-score-coverage Sep 9, 2026
6 checks passed
@gaspergrom
gaspergrom deleted the feat/IN-1292 branch September 9, 2026 13:39
gaspergrom added a commit that referenced this pull request Sep 10, 2026
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
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