fix: (UI)Globe/cloud button on Attack Paths page shows correct data - BED-6661

[nicole-m-martin]

Description

Updated the functionality for the "Globe/Cloud" button click in the graph view on Attack Paths page. This updates API endpoint to use /asset-group-tags/:agtId/members/counts and renders the Objects Accordion to show the Zone Membership panel when tier flag is enabled. Fixes the panel only showing 'Tier 0' data.

Motivation and Context

Resolves BED-6661 - UI - Clicking on globe/cloud on Attack Paths page shows only "Tier 0" data

How Has This Been Tested?

Manual Tested all Expected Behaviors:

Given I am on the attack paths page (with any zone or environment selected)
When I click on the globe or cloud
✅ Then I see a list of objects in both the selected zone and environment sorted by type

Given I clicked the globe
When I click Go to Privilege Zone Management
✅ Then I am taken to the summary view of Privilege Zone Management

Given I am viewing the Zone Membership panel on attack paths page
When I select a new zone or environment
✅ Then the zone membership panel remains displayed but with new environment/zone information.

Given I am view the zone membership panel on attack paths page,
When I navigate back through browser history
✅ Then I am shown the Findings panel.

Given I have link to BHE that was captured when the Zone Membership panel was open
When I open that link
✅ Then the Zone Membership panel is displayed with the expected zone and environment displayed.

Test file added: ZoneObjectsInformationPanel.test.tsx (In Bloodhound Enterprise)

Types of changes

• Bug fix (non-breaking change which fixes an issue)

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

• New Features
  • Exposed tag-object-counts hook as a public API with safer handling of optional environment parameters.
  • Objects list component now accepts tagId/ruleId/objectId and a customizable onObjectClick callback so external code can handle object clicks.
  • Objects list component is re-exported for easier consumption from the Privilege Zones module.

[coderabbitai]

📝 Walkthrough

Walkthrough

Exports the useTagObjectCounts hook publicly and defensively handles optional environments. ObjectsAccordion gains tagId, ruleId, objectId props and an onObjectClick callback; internal navigation is replaced by delegating clicks to the callback. Barrel files now re-export ObjectsAccordion.

Changes

Cohort / File(s)	Summary
Hook Export packages/javascript/bh-shared-ui/src/hooks/useAssetGroupTags/useObjectCounts.ts	Made useTagObjectCounts a public export; query key construction now spreads (environments ?? []) and retains reject-on-missing-tagId behavior. Minor formatting cleanup.
ObjectsAccordion & Items packages/javascript/bh-shared-ui/src/views/PrivilegeZones/Details/ObjectsAccordion.tsx, packages/javascript/bh-shared-ui/src/views/PrivilegeZones/Details/ObjectsAccordion.*.tsx	Added public props tagId, ruleId, objectId, and onObjectClick; removed internal path-based navigation in favor of invoking onObjectClick(item); active-row highlight now uses objectId; props propagated to item components.
Details Integration packages/javascript/bh-shared-ui/src/views/PrivilegeZones/Details/Details.tsx	Wired ObjectsAccordion with onObjectClick handler that sets selected details tab and navigates to object details; passes tagId, ruleId, and memberId-derived objectId.
Tests Updated packages/javascript/bh-shared-ui/src/views/PrivilegeZones/Details/ObjectsAccordion.rule.test.tsx, .../ObjectsAccordion.tag.test.tsx	Removed internal usePZPathParams mocks; tests now render ObjectsAccordion with explicit tagId, ruleId, and onObjectClick mocks and updated assertions to match new prop-driven behavior.
Barrel Exports packages/javascript/bh-shared-ui/src/views/PrivilegeZones/Details/index.tsx, packages/javascript/bh-shared-ui/src/views/PrivilegeZones/index.tsx	Re-exported ObjectsAccordion from Details and added it to the PrivilegeZones barrel exports.

Sequence Diagram(s)

mermaid
sequenceDiagram
participant User as User
participant ObjectsAccordion as ObjectsAccordion
participant Details as Details (parent)
participant Router as Router/Navigate
User->>ObjectsAccordion: click object row
ObjectsAccordion->>Details: onObjectClick(item)
Details->>Details: setSelectedDetailsTab(Object)
Details->>Router: navigate(objectDetailsLink)
Router-->>User: route to details

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat: Objects Accordion for Details redesign BED-6844 #2160 — Overlaps changes to useTagObjectCounts and ObjectsAccordion hook/component surface.
• feat(PZ): Added Selected Details Tabs BED-6836 #2189 — Introduces SelectedDetailsTabs context and related wiring used by Details’ object-click handler.
• chore: wire up new details, replace and remove old details BED-7081 #2224 — Modifies PrivilegeZones UI and exports that intersect with these changes.

Suggested reviewers

• urangel
• mistahj67
• specter-flq

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main fix: updating the Globe/cloud button to show correct zone and environment data, directly addressing the bug mentioned in BED-6661.
Description check	✅ Passed	The PR description fully completes the required template with clear Description, Motivation and Context (with ticket reference), comprehensive testing details, and proper checklist completion.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch BED-6661--globe

---

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.}