Exposed the useSidebar hook from Sidebar#527
Conversation
📝 WalkthroughSummaryThis pull request exposes the Changespackages/oxygen-ui/src/components/index.ts
Changeset
ImpactThis change enables developers to directly import and use the WalkthroughThis pull request introduces a new Changeset entry marking three 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
.changeset/early-points-add.md (1)
7-8: Minor: trailing blank line after content.Line 8 is an empty trailing line outside the changeset front matter/body. Not a functional issue, but consider trimming to keep the changeset tidy.
🤖 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 7 - 8, Remove the trailing blank line at the end of the changeset body in early-points-add.md so the file ends immediately after the content (no extra empty line); open the changeset file and delete the empty line following the front matter/body to keep the changeset tidy.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.changeset/early-points-add.md:
- Around line 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.
---
Nitpick comments:
In @.changeset/early-points-add.md:
- Around line 7-8: Remove the trailing blank line at the end of the changeset
body in early-points-add.md so the file ends immediately after the content (no
extra empty line); open the changeset file and delete the empty line following
the front matter/body to keep the changeset tidy.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 0f42b6f0-a285-46e4-bf96-a805f3e02987
📒 Files selected for processing (2)
.changeset/early-points-add.mdpackages/oxygen-ui/src/components/index.ts
| '@wso2/oxygen-ui': patch | ||
| '@wso2/oxygen-ui-charts-react': patch | ||
| '@wso2/oxygen-ui-icons-react': patch |
There was a problem hiding this comment.
🧩 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/nullRepository: 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.
This pull request makes a small but important update to the
Sidebarcomponent exports. TheuseSidebarhook is now exposed from theSidebarmodule, making it available for use in other parts of the application.useSidebarhook fromSidebarby updating the export statement inpackages/oxygen-ui/src/components/index.ts@wso2/oxygen-ui,@wso2/oxygen-ui-charts-react, and@wso2/oxygen-ui-icons-reactin.changeset/early-points-add.md