Skip to content

fix(me): align Bun migration paths - #526

Merged
baleen37 merged 1 commit into
mainfrom
feat/migrate-to-bun
Feb 24, 2026
Merged

fix(me): align Bun migration paths#526
baleen37 merged 1 commit into
mainfrom
feat/migrate-to-bun

Conversation

@baleen37

@baleen37 baleen37 commented Feb 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • migrate remaining LSP auto-install hooks from npm install -g to bun add -g
  • align release example workflow with Bun (setup-bun, bun install, bunx semantic-release, jq version read)
  • restore missing scripts/sync-marketplace-version.sh and add regression checks for workflow/script and Bun-only patterns

Test plan

  • bats tests/me/me-specific.bats
  • bash tests/run-unit-tests.sh
  • bash tests/run-all-tests.sh
  • pre-commit run --all-files

Summary by CodeRabbit

  • New Features

    • Introduced automated marketplace version synchronization.
    • Added constraint documentation for Databricks repair workflow operations (mutually exclusive parameters).
  • Documentation

    • Updated Databricks notebook documentation formatting and spacing.
  • Tests

    • Added test coverage for marketplace sync functionality and build tool configuration.

@coderabbitai

coderabbitai Bot commented Feb 24, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 136ed27 and 200e557.

📒 Files selected for processing (9)
  • docs/release-with-github-app.yml
  • plugins/me/hooks/lsp-bash-check-install.sh
  • plugins/me/hooks/lsp-python-check-install.sh
  • plugins/me/hooks/lsp-typescript-check-install.sh
  • plugins/me/skills/databricks-execute/SKILL.md
  • plugins/me/skills/databricks-notebook/SKILL.md
  • scripts/sync-marketplace-version.sh
  • tests/github_workflows.bats
  • tests/me/me-specific.bats

📝 Walkthrough

Walkthrough

This PR migrates package management from Node.js/npm to Bun across the release workflow and LSP hook installation scripts, introduces a marketplace version synchronization script for automated plugin version updates, and adds corresponding tests and documentation updates.

Changes

Cohort / File(s) Summary
Package Manager Migration to Bun
docs/release-with-github-app.yml, plugins/me/hooks/lsp-bash-check-install.sh, plugins/me/hooks/lsp-python-check-install.sh, plugins/me/hooks/lsp-typescript-check-install.sh
Replaced npm with Bun and Node.js setup with oven-sh/setup-bun throughout release workflow and LSP installation hooks. Changed version extraction from Node.js to jq in the workflow.
Marketplace Sync Automation
scripts/sync-marketplace-version.sh
New bash script that aggregates plugin versions from individual plugin.json files and synchronizes marketplace.json. Uses jq for conditional updates and implements temporary file handling with cleanup traps.
Test Coverage
tests/github_workflows.bats, tests/me/me-specific.bats
Added tests validating Bun adoption in LSP hooks and release workflow, plus test confirming marketplace sync workflow triggers the correct script.
Documentation Updates
plugins/me/skills/databricks-execute/SKILL.md, plugins/me/skills/databricks-notebook/SKILL.md
Added mutually-exclusive constraint documentation for repair task parameters and corrected Markdown formatting in notebook skill documentation.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Possibly related PRs

Poem

🐰 Bun hops in to speed the race,
npm exits, takes its place,
Versions sync without a care,
Marketplace updated fair!
LSP hooks now bun-ified,
Tests confirm our tools allied! 🐇

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/migrate-to-bun

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.

@baleen37
baleen37 enabled auto-merge (squash) February 24, 2026 05:39
@baleen37
baleen37 merged commit a52c46b into main Feb 24, 2026
4 of 5 checks passed
@baleen37
baleen37 deleted the feat/migrate-to-bun branch February 24, 2026 05:40
baleen-release-app Bot pushed a commit that referenced this pull request Feb 24, 2026
## [5.34.1](v5.34.0...v5.34.1) (2026-02-24)

### Bug Fixes

* **me:** align Bun paths and restore marketplace sync script ([#526](#526)) ([a52c46b](a52c46b))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant