feat: BED-6556 - Remove API version from User Profile page

[StephenHinck]

Description

Removes the listed API version from the User Profile page. This data is now available on the Nav Bar and is no longer relevant/necessary.

Motivation and Context

Resolves BED-6556

Why is this change required? What problem does it solve?

How Has This Been Tested?

Validated that, after change, all updated tests passed and that the API version no longer appears on the profile page.

Screenshots (optional):

Types of changes

• Chore (a change that does not modify the application 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

• Refactor

  • Simplified the User Profile by removing the API version fetch and display.

• Tests

  • Updated profile tests to align with UI changes and cleaned up redundant imports and mocks.

• Style

  • Standardized license headers and reorganized imports for consistency.

• Chores

  • Minor formatting adjustments with no impact on functionality.

• Bug Fixes

  • No user-facing bug fixes included in this release.

[coderabbitai]

Walkthrough

Edits include a comment formatting tweak in ha.go, import reordering in a UI component, addition of a license header and a type import in freeIconsList.ts, a duplicate import in a test, removal of a version endpoint mock in a test, and removal of API version usage and display from the UserProfile component.

Changes

Cohort / File(s)	Summary of Changes
HA daemon header tweak cmd/api/src/daemons/ha/ha.go	License header URL line formatting changed (space to tab). No logic changes.
UserProfile: remove API version usage packages/javascript/bh-shared-ui/src/views/UserProfile/UserProfile.tsx, .../UserProfile.test.tsx	Component: removed useApiVersion import, variable, and UI display of API version. Test: removed MSW handler for GET /api/version.
PrivilegeZones test import duplication packages/javascript/bh-shared-ui/src/views/PrivilegeZones/Details/TagList.test.tsx	Added a duplicate import of zoneHandlers; no functional test changes.
FileIngestTable import reorder packages/javascript/bh-shared-ui/src/components/FileIngestTable/FileIngestTable.tsx	Reordered an import statement; no runtime changes.
Shared UI utils header and type import packages/javascript/bh-shared-ui/src/utils/freeIconsList.ts	Added license header and imported IconName from @fortawesome/free-solid-svg-icons; no exported API changes.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• BED-6380: Summary Page Included in BHCE #1932 — Also modifies TagList.test.tsx around MSW zoneHandlers; likely related to consolidating or correcting test handlers.

Suggested labels

user interface

Suggested reviewers

• urangel
• specter-flq
• kpowderly

Poem

I hop through diffs with whiskers bright,
Snipping version text from view tonight.
Imports shuffled, headers neat,
Tests cleaned up with tidy feet.
No logic burrows dug too deep—
Just gentle hops to keep things sleek. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description Check	⚠️ Warning	While the description follows the template structure and includes most required headings, the Motivation and Context section still contains the placeholder prompt “Why is this change required? What problem does it solve?” instead of an explanation, so it remains incomplete.	Replace the placeholder text in the Motivation and Context section with a clear rationale for removing the API version from the profile page and, if possible, provide additional detail on your testing environment or steps.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.
Title Check	✅ Passed	The title clearly summarizes the primary change by indicating the removal of the API version from the User Profile page, aligns with the pull request objectives, and is concise and specific without extraneous details.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

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

---

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