Repository navigation
stack-report: dedupe dependency graph nodes by name - #3
Merged
Merged
Conversation
buildGraphJson previously assigned one graph node per scope-spec instance, so a package listed under two per-product layers (e.g. MariaDB Connector/C under both MySQL and MariaDB) rendered as two separate boxes. Node identity is now the normalized name: instances sharing a name merge into one node, with criticality taking the highest-priority value seen and edges from every instance attaching to the single merged node. Regenerated databases.graph.json from its existing classification/edge data (221->218 nodes, 477->458 edges); sample and agentic-ai-cpu had no duplicate names and are unchanged.
luhenry
force-pushed
the
dependency-graph
branch
from
September 5, 2026 10:27
209f427 to
b62e409
Compare
|
luhenry
changed the title
Uses dagre-d3's experimental clusters feature (compound graph + setParent) to draw a labeled box around every group of nodes sharing a `column` (product), one cluster per product per redraw so it stays correct under focus/external-toggle. Nodes with no column (shared or external) are left unparented, outside any cluster.
Replaces the layered dagre-d3 renderer with a d3-force simulation (link + charge + collision, synchronously settled) so dense stacks spread out radially instead of cramming into a few wide horizontal ranks with heavily overlapping edges. Drops the dagre-d3 CDN script tag, no longer needed. Per-product clustering (dagre-d3 compound nodes) was tried and then removed in favor of plain force-directed distribution. Edges are now thin, light lines (read as texture rather than competing with labels) that switch to a black, fully-opaque line and arrowhead when their node is hovered. A node with zero edges gets a small centering pull so it doesn't drift arbitrarily far from charge repulsion alone. Also default "Show external dependencies" to unchecked, so a report opens showing only in-scope nodes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.