feat(dashboard): employee data flow graph#3111
Conversation
There was a problem hiding this comment.
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.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: 121a260 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
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 |
There was a problem hiding this comment.
1 issue found across 43 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="server/clickhouse/schema.sql">
<violation number="1" location="server/clickhouse/schema.sql:98">
P2: Missing bloom filter indexes for `endpoint_os` and `endpoint_arch`. Every other materialized column in this table has a corresponding index. If these columns will be used in WHERE clauses, the missing indexes will hurt query performance on large datasets.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
There was a problem hiding this comment.
1 issue found across 56 files
Partial review: This PR has more than 50 files, so cubic reviewed the highest-priority files first. During the trial, paid plans get a higher file limit.
You can try an ultrareview to bypass the file limit, comment @cubic-dev-ai ultrareview. Learn more.
Fix all with cubic | Re-trigger cubic
d18bf64 to
050b3c9
Compare
🚀 Preview Environment (PR #3111)Preview URL: https://pr-3111.dev.getgram.ai
Gram Preview Bot |
simplesagar
left a comment
There was a problem hiding this comment.
Could you add a screenshot for the new UX ?
@simplesagar done. |
Summary
telemetry.getEmployeeDataFlowGraphplus generated SDK and React Query support for typed graph nodes and weighted edges.gram.hook.hostnameattribute and uses it as the graph origin.Test plan
go test ./server/internal/telemetry -run 'Test(BuildEmployeeDataFlowGraph_DoesNotCollideEdgesWhenLabelsContainDelimiter|GetEmployeeDataFlowGraph)' -count=1CleanShot.2026-05-29.at.1.49.40.mp4
Fixes AGE-2503.