This repository was archived by the owner on Oct 3, 2025. It is now read-only.
Migrate from npm to bun for improved build performance#439
Closed
pjcdawkins wants to merge 6 commits intomainfrom
Closed
Migrate from npm to bun for improved build performance#439pjcdawkins wants to merge 6 commits intomainfrom
pjcdawkins wants to merge 6 commits intomainfrom
Conversation
Updated Upsun configuration to use bun for installation and builds. Removed npm-specific dependencies and configuration, including npm-force-resolutions and preinstall script. Consolidated resolutions into overrides field. Replaced package-lock.json with bun.lock. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Hello, and thank you for contributing to the Upsun demo project! Setting up testsUntil all tests pass, a pull request will not be merged. Build tests for combinations of Python and Node.js will happen automatically, and must pass in order to merge. Failing testsIf your PR is failing, see the Running tests section of the README for troubleshooting details. Reviewing these changes locally:
|
Added bun setup step to GitHub Actions workflow. Updated root package.json scripts to use bun for frontend commands. Modified vulnerability scanning to use bun pm audit instead of npm audit. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Updated GitHub Actions matrix to test Node.js 22.x and 24.x only, matching current Upsun support and bun compatibility. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Replaced npm audit with Bun's Security Scanner API for checking high and critical vulnerabilities in frontend dependencies. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Created bunfig.toml with security scanner configuration and telemetry disabled. Removed dedicated vulnerability check steps from workflow as bun install will fail if critical issues are found when a scanner is configured. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The @acme/bun-security-scanner is just a documentation placeholder. Relying on GitHub Dependabot for vulnerability scanning instead. Fixed telemetry configuration format. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Collaborator
Author
|
Done together with |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code