refactor(ui-react): improve useSidebarLayout hook

[luizhf42]

What

Improve the useSidebarLayout.ts hook and fix a mobile hamburger menu bug.

Why

That hook had a lot of unnecessary code, such as pathname tracking, useCallback and a buggy prevIsDesktop checking that broke the sidebar when resizing from mobile → desktop.

Testing

• Go to the main React UI and ensure the sidebar works flawlessly in desktop and mobile.
• On mobile, try to toggle (open/close) the sidebar clicking in the hamburger menu.
• Resize the page from mobile to desktop and check if the sidebar goes from the hidden, opened by hamburger menu state, to the collapsed, open-on-hover state.
• Make the same checks in the admin UI.

[github-actions]

Claude finished @luizhf42's task in 3m 30s —— View job

---

Code Review Complete

Reviewed 1 file across code quality, security, testing, TypeScript patterns, and architecture — no blocking issues found. 1 inline comment posted (medium severity, optional suggestion).

The removal of the render-phase prevIsDesktop check is a clear net improvement — it fixes the reported bug where the sidebar would become permanently hidden on mobile→desktop transition. One optional suggestion was left inline about a minor stale-state edge case on resize round-trips, with a useEffect-based alternative if you want to handle it.

---

If you've addressed the feedback and want a new review, tag @shellhub-io/admin and a team member can trigger it.

[gustavosbarreto]

/review

[github-actions]

Claude finished @gustavosbarreto's task in 3m 15s —— View job

---

Code Review Complete

Reviewed 3 files across code quality, security, testing, TypeScript patterns, and architecture — no new issues found. The code looks good as-is.

The stale expanded state edge case on resize round-trips was already discussed in the previous review thread and the author's decision to leave it is reasonable given the minimal UX impact and self-correcting behavior.

---

If you push additional changes and want a new review, tag @shellhub-io/admin and a team member can trigger it.

[luizhf42]

Closed because #6166 was merged and already included the commits from that branch.