Skip to content

Exposed the useSidebar hook from Sidebar #527

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jeradrutnam merged 2 commits into from
Apr 27, 2026
Merged

Exposed the useSidebar hook from Sidebar #527

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fe99209
Expose `useSidebar` from the Sidebar component
jeradrutnam Apr 27, 2026
c26bdd3
🦋 Add changeset
jeradrutnam Apr 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions .changeset/early-points-add.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
---
'@wso2/oxygen-ui': patch
'@wso2/oxygen-ui-charts-react': patch
'@wso2/oxygen-ui-icons-react': patch
Comment on lines +2 to +4
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check whether the charts-react and icons-react packages reference useSidebar or depend on `@wso2/oxygen-ui` in a way that warrants a coordinated bump.
fd -t f 'package.json' packages/oxygen-ui-charts-react packages/oxygen-ui-icons-react --exec cat {}
rg -nP '\buseSidebar\b' -C2 packages/oxygen-ui-charts-react packages/oxygen-ui-icons-react 2>/dev/null

Repository: wso2/oxygen-ui

Length of output: 4227

Remove unrelated packages from changeset.

The changeset includes @wso2/oxygen-ui-charts-react and @wso2/oxygen-ui-icons-react, but neither package uses useSidebar. Additionally, @wso2/oxygen-ui-icons-react has no dependency on @wso2/oxygen-ui at all, making its inclusion unjustified. Only include packages that directly consume or re-export the new export.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In @.changeset/early-points-add.md around lines 2 - 4, The changeset erroneously
lists packages that don't consume or re-export the new useSidebar export; remove
the two unrelated entries '@wso2/oxygen-ui-charts-react' and
'@wso2/oxygen-ui-icons-react' from .changeset/early-points-add.md so only
'@wso2/oxygen-ui' remains, ensuring the changeset only targets the package that
actually uses the new export.

---

Expose `useSidebar` hook
2 changes: 1 addition & 1 deletion packages/oxygen-ui/src/components/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ export type {
HeaderContextValue,
} from './Header';

export { Sidebar } from './Sidebar';
export { Sidebar, useSidebar } from './Sidebar';
export type {
SidebarProps,
SidebarNavProps,
Expand Down
Loading