Skip to content

feat(risk): add overview analytics#2912

Open
alx-xo wants to merge 19 commits into
mainfrom
alexm/age-2411-feat-risk-overview-can-show-interesting-data
Open

feat(risk): add overview analytics#2912
alx-xo wants to merge 19 commits into
mainfrom
alexm/age-2411-feat-risk-overview-can-show-interesting-data

Conversation

@alx-xo
Copy link
Copy Markdown
Contributor

@alx-xo alx-xo commented May 19, 2026

Summary

  • Add risk overview summary analytics for recent findings, top users, top categories, and time-series activity.
  • Add the management API endpoint, generated SDK wiring, and dashboard integration.
  • Keep policy category aggregation centralized in the time-series query and derive top categories from that result.

Visual

Gram _ Local · 7 20pm · 05-18

Test Plan

  • mise run gen:sqlc-server
  • mise run test:server ./internal/risk/...
  • mise run lint:server
  • git diff --check

Related PRs

@alx-xo alx-xo requested review from a team as code owners May 19, 2026 02:20
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 19, 2026

AGE-2411

@vercel
Copy link
Copy Markdown

vercel Bot commented May 19, 2026

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

Project Deployment Actions Updated (UTC)
gram-docs-redirect Ready Ready Preview, Comment May 19, 2026 11:59pm

Request Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 19, 2026

🦋 Changeset detected

Latest commit: 991c134

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
server Minor
dashboard Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Copy Markdown
Member

@simplesagar simplesagar left a comment

Choose a reason for hiding this comment

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

Suggestions

  • bring in 7,30,60 day look back options like insight or project. home

Related added in this tool for insights #2922

@alx-xo alx-xo force-pushed the alexm/age-2263-update-risk-overview-styles branch from b1e230c to ab0f160 Compare May 19, 2026 23:59
@alx-xo alx-xo force-pushed the alexm/age-2411-feat-risk-overview-can-show-interesting-data branch from e2d7991 to 991c134 Compare May 19, 2026 23:59
Base automatically changed from alexm/age-2263-update-risk-overview-styles to main May 20, 2026 01:19
Thegreatsura pushed a commit to Thegreatsura/gram that referenced this pull request May 20, 2026
### Summary
- Add an admin-only Risk Events page under Logs
- Move the Recent Findings risk data into a wider tabular workbench with
policy filtering, refresh, load more, chat drawer, and shadow MCP
exclude action support.
- Add a reusable `LogWorkbench` shell for future log/activity pages
without changing `LogsMCPContent` in this PR.
- Extract shared risk UI/util helpers for reuse across Risk Activity and
future security pages.
- Keep `/risk-overview` behavior/layout intact for now, aside from
shared helper imports.

### Visual

<img width="5402" height="3204" alt="Gram _ Local"
src="https://github.com/user-attachments/assets/f7860812-1a4e-45b7-be05-22c0de7d7a44"
/>
<img width="5402" height="3204" alt="image"
src="https://github.com/user-attachments/assets/b1d59fe3-85ce-4a38-b489-0631b7cd4d02"
/>

### Related PRs
- speakeasy-api#2912
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