feat: Removed redraw of graph on entity panel click - BED-7543

[specter-flq]

Description

Removed a block of code that would set the necessary params in the url to re render the graph. I didn't remove the relationship query functionality from the url because we spoke the possibility of bring it back as an optional rendering in the near future.

Motivation and Context

Resolves: BED-7543

How Has This Been Tested?

Manually validated and existing tests pass

Screenshots (optional):

EntityPanel.mp4

Types of changes

• New feature (non-breaking change which adds functionality)

Checklist:

• I have met the contributing prerequisites
  • Assigned myself to this PR
  • Added the appropriate labels
  • Associated an issue: Issues and Pull Requests README #672
  • Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing
• I have ensured that related documentation is up-to-date
  • Open API docs
  • Code comments (GoDocs / JSDocs)
• I have followed proper test practices
  • Added/updated tests to cover my changes
  • All new and existing tests passed

Summary by CodeRabbit

• Bug Fixes
  • Enhanced expanded panel functionality to consistently load relationship data. Removed unnecessary query restrictions that could prevent proper execution of data relationships. Users now experience more reliable and predictable behavior when expanding panels to view related information across all use cases.

[coderabbitai]

📝 Walkthrough

Walkthrough

Removed render-limit-based conditional logic from the EntityInfoDataTableGraphed component. The component no longer checks if data is under a render limit before setting expanded panel section parameters, simplifying the query parameter construction flow.

Changes

Cohort / File(s)	Summary
Render Limit Logic Removal packages/javascript/bh-shared-ui/src/components/EntityInfoDataTableGraphed/EntityInfoDataTableGraphed.tsx	Removed NODE_GRAPH_RENDER_LIMIT import, isUnderRenderLimit calculation, and conditional inclusion of relationship query params (searchType, relationshipQueryType, relationshipQueryItemId) in expandedPanelSectionsParams. Expanded panel params now set expandedPanelSections unconditionally without relationship query fields injection.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing graph redraw logic on entity panel click, with the associated ticket reference.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	PR description covers the template requirements with clear sections for Description, Motivation and Context (with ticket reference), testing approach, and type of change classification.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch BED-7543--dis-exp-accordion-redraw

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}