Skip to content

Add graph view filtering and label wrapping#73

Open
stalep wants to merge 1 commit intoHyperfoil:mainfrom
stalep:graph-view-improvements
Open

Add graph view filtering and label wrapping#73
stalep wants to merge 1 commit intoHyperfoil:mainfrom
stalep:graph-view-improvements

Conversation

@stalep
Copy link
Copy Markdown
Member

@stalep stalep commented Apr 29, 2026

Improve the 'list nodes as Graph' command for large DAGs:

  • Collapse coalesce nodes by default (hide _primary/_alt nodes)
  • Add --type filter for comma-separated node types
  • Add --depth limit for max levels from root
  • Add --show-type flag to prefix labels with [type]
  • Add --max-label-width to wrap long labels at word boundaries (default 30)

@stalep stalep requested a review from willr3 April 29, 2026 15:03
@willr3
Copy link
Copy Markdown
Collaborator

willr3 commented Apr 29, 2026

the --type filtering looks like it is implemented as graph truncation not filtering. The graph will stop walking at the point it encounters a type and will not include any nodes that are only linked to the graph by nodes of that type. What node types do you find need to be removed from the graph and would it be better for the user to encode that information in some other way?

Improve the 'list nodes as Graph' command for large DAGs:
- Collapse coalesce nodes by default (hide _primary/_alt nodes)
- Add --type filter for comma-separated node types
- Add --depth limit for max levels from root
- Add --show-type flag to prefix labels with [type]
- Add --max-label-width to wrap long labels at word boundaries (default 30)
@stalep stalep force-pushed the graph-view-improvements branch from 942d01d to 101b8b5 Compare April 29, 2026 19:42
@stalep
Copy link
Copy Markdown
Member Author

stalep commented Apr 29, 2026

the --type filtering looks like it is implemented as graph truncation not filtering. The graph will stop walking at the point it encounters a type and will not include any nodes that are only linked to the graph by nodes of that type. What node types do you find need to be removed from the graph and would it be better for the user to encode that information in some other way?

Yes, it was truncating, not filtering. now the --type filter now skips non-matching nodes but continues walking through them — children reconnect to the nearest visible ancestor instead of being truncated. Skipped nodes aren't added to the nodes map so they don't appear in the graph, but their sources are still walked so deeper matching nodes remain reachable.

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.

2 participants