fix(attack-path): expand the focused endpoint's finding clusters and reclaim hidden columns' width (#6647) - #6982
Conversation
…reclaim hidden columns' width (#6647)
|
/review docs
|
Codecov Report❌ Patch coverage is ❌ Your patch check has failed because the patch coverage (41.66%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #6982 +/- ##
============================================
- Coverage 44.96% 44.94% -0.02%
+ Complexity 9327 9325 -2
============================================
Files 2681 2681
Lines 80441 80477 +36
Branches 12066 12073 +7
============================================
+ Hits 36168 36171 +3
- Misses 41653 41681 +28
- Partials 2620 2625 +5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Description
Two UX fixes on the attack path view, reported from the chokepoint drill-down flow.
1. Focusing an endpoint now opens its findings straight away. Clicking a chokepoint (from the "Top chokepoints" popover, the table, or the search) rebuilt the graph as the focused endpoint path but left every finding-type cluster collapsed — the analyst had to click each one (portscan, files, credentials…) to see anything. The focused view now pre-expands all of the endpoint's finding-type clusters, fed by a single
fetchEndpointFindingscall split per type (no extra round-trips versus expanding them by hand).2. The findings drawer no longer truncates the Assets column.
FindingListcolumn widths are percentages of the full six-column table. In compact mode the drawer hides three of them, so Type/Value/Assets only used 57% of the row: the asset chips ellipsized while the right half sat empty. The hidden columns' share is now redistributed over the visible ones, keeping their relative proportions. Full-page usage (no hidden fields) is unchanged.Follow-up to #6964.
Checks
yarn eslint,yarn tsc --noEmitclean.yarn vitest runon the attack-path and findings suites: 27 tests pass.