Skip to content

doc feature: add bun as default tab in PackageManagers component#714

Merged
alex72508 merged 1 commit intomainfrom
tamal/update/add-bun-to-package-managers
Apr 15, 2026
Merged

doc feature: add bun as default tab in PackageManagers component#714
alex72508 merged 1 commit intomainfrom
tamal/update/add-bun-to-package-managers

Conversation

@tamalchowdhury
Copy link
Copy Markdown
Collaborator

@tamalchowdhury tamalchowdhury commented Apr 13, 2026 •
edited by coderabbitai bot
Loading

Adds bun as a default tab alongside npm, pnpm, and yarn in all <PackageManagers> blocks across the docs.

Done via a thin wrapper component (src/components/PackageManagers.astro) that sets bun in the default pkgManagers array and forwards all other props to the underlying starlight-package-managers component. The auto-import in astro.config.mjs is updated to use the local wrapper instead of the direct package import. No MDX files needed to change.

image

Summary by CodeRabbit

  • Updates
    • PackageManagers component now includes default options for npm, pnpm, yarn, and bun when no managers are specified.
    • Component configuration has been restructured for improved integration.

@tamalchowdhury tamalchowdhury requested a review from a team as a code owner April 13, 2026 11:56
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 13, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ba3b9d52-4947-43b5-9a33-771524d3c5ca

📥 Commits

Reviewing files that changed from the base of the PR and between d03feb0 and 24be2d4.

📒 Files selected for processing (2)
  • astro.config.mjs
  • src/components/PackageManagers.astro

Walkthrough

The changes refactor how the PackageManagers component is exposed through Astro's auto-import system. Instead of directly importing from the starlight-package-managers package, a new wrapper component is introduced in the project that defaults package managers and delegates to the original component.

Changes

Cohort / File(s) Summary
Configuration
astro.config.mjs		 Modified AutoImport integration's imports configuration to reference the new local wrapper component file path instead of the direct package-managers namespace mapping.
Component Wrapper
src/components/PackageManagers.astro		 New wrapper component that imports PackageManagers from starlight-package-managers, applies default package managers list (['npm', 'pnpm', 'yarn', 'bun']), and forwards all props to the underlying component.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A wrapper small, so neat and spry,
Now PackageManagers hop on by,
With defaults set for npm's crew,
Pnpm, yarn, and bun too! 📦✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding bun as a default tab in the PackageManagers component through the creation of a wrapper component and configuration update.
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
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch tamal/update/add-bun-to-package-managers

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

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

alex72508
alex72508 approved these changes Apr 15, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@alex72508 alex72508 left a comment

Choose a reason for hiding this comment

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

LGTM

@alex72508 alex72508 merged commit d55fa62 into main Apr 15, 2026
6 checks passed
@alex72508 alex72508 deleted the tamal/update/add-bun-to-package-managers branch April 15, 2026 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alex72508 alex72508 alex72508 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tamalchowdhury @alex72508