fix: Bun Feature Failure on Apple Silicon ARM64 + Feature Hardening by trevordcampbell · Pull Request #149 · devcontainers-extra/features

trevordcampbell · 2025-09-22T00:50:52Z

Problem

The Bun feature failed to install on Apple Silicon Mac hosts when creating arm64 Linux containers.
Users encountered this error: no matches found for asset regex: ^bun-linux-aarch64-baseline\.zip$

Root Cause

The Bun feature was attempting to download bun-linux-aarch64-baseline.zip, but Bun does not publish
"baseline" variants for the ARM64 architecture. For linux / arm64, Bun only publishes:

bun-linux-aarch64.zip (glibc)
bun-linux-aarch64-musl.zip (musl)

This worked on linux / amd64-based DevContainers because baseline variants exist there, but failed on arm64-based DevContainers.

Solution

1. `arm64` Asset Pattern Fix

arm64 now uses standard builds instead of non-existent baseline variants
Removed 5x larger profile builds from fallbacks
Asset patterns now match actual available Bun releases

2. Comprehensive Feature Enhancements

Robust libc Detection

Enhanced from single /sbin/apk check to 4 detection methods:
- Alpine package manager detection
- ldd --version output analysis
- musl library file presence
- /proc/mounts analysis
- Detection logs are emitted to stderr; function returns clean stdout
- Fixed /proc/mounts musl check to avoid broken grep pipeline

Enhanced Architecture Support

Proper detection of aarch64 and arm64 aliases
Clear error messages for unsupported architectures

Better Error Handling & Diagnostics

Detailed troubleshooting information in error messages
Architecture, libc, version, and asset pattern details
Actionable links to Bun releases for manual troubleshooting

Improved Logging & User Experience

Structured logging with timestamps: [2024-01-15 10:30:15] INFO: ...
Consistent log levels: INFO, SUCCESS, WARNING, ERROR
Progress indicators throughout installation process
Optional debug logs via BUN_FEATURE_DEBUG=1

Performance Optimizations

Efficient nanolayer location handling
Early exit conditions in detection functions
Optimized array handling for asset patterns
Simplified install loop with early return on first successful asset

Version Handling Enhancements

Robust version canonicalization function
Multiple input format support: 1.2.3, v1.2.3, bun-v1.2.3
Validation with graceful fallback to "latest" for invalid formats

Benefits

For Apple Silicon Users

Fixes the original installation failure
Proper arm64 asset selection
Works across Debian, Ubuntu and Alpine containers

For All Users

Better error messages and troubleshooting
More robust across different Linux distributions
Structured logging for better debugging
Performance optimizations
Future-proof version handling

Files Changed

src/bun/install.sh

Rewrite with enhanced architecture detection
Robust libc detection across multiple distros
Structured logging and error handling
Performance optimizations and version enhancements

src/bun/NOTES.md

Updated documentation to reflect arm64 support
Clarified architecture limitations
Added information about asset selection logic

Testing

✅ All Tests Passed

References

trevordcampbell · 2025-11-17T20:15:10Z

Would appreciate a review + merge on this to clear the related issue

koralowiec · 2025-11-19T20:09:23Z

Hi Trevor! Thanks for your PR, and sorry for the delay with the review, I'm being busy lately. I'm going to review it by the end of this week

koralowiec

To be honest, I think that there's too much logging, and maybe too much logic in some places.

Generally, the solution which you provided looks nice, but I would like to get rid of the logging part. So please remove the log and debug_log functions, and their execution. If you feel something has to be output, use echo for now.

Like I mentioned in one of the comments, I'm going to think about adding the logging functions to the shared script (#165)

koralowiec · 2025-11-22T15:35:08Z

src/bun/install.sh

+# Helper function for consistent logging (robust for container environments)
+log() {
+    local level="$1"
+    shift
+    local message="$*"
+    local timestamp
+
+    # Try to get timestamp, fallback if date command fails
+    if timestamp=$(date '+%Y-%m-%d %H:%M:%S' 2>/dev/null); then
+        timestamp="[$timestamp]"
+    else
+        timestamp="[$(date 2>/dev/null || echo 'unknown')]"
+    fi
+
+    case "$level" in
+        INFO)    echo "$timestamp INFO: $message" ;;
+        SUCCESS) echo "$timestamp SUCCESS: $message" ;;
+        WARNING) echo "$timestamp WARNING: $message" ;;
+        ERROR)   echo "$timestamp ERROR: $message" >&2 ;;
+        *)       echo "$timestamp $level: $message" ;;
+    esac
+}
+
+# Debug logging helper (enabled when BUN_FEATURE_DEBUG is set)
+debug_log() {
+    if [ -n "$BUN_FEATURE_DEBUG" ]; then
+        log INFO "$@"
+    fi
+}
+
+log INFO "Starting Bun installation"
+


Having a more robust logging functions is not a bad idea (along with having a debug option). But I think that this shouldn't be introduced on a single feature level. It probably belongs to library_scripts.sh which is a "shared" functions library. For now I'm not sure if I want to introduce the functions you provided in this exact shape into that library_scripts.sh, I think I need to go over it (#165)

So for now, I would like you to not define/use that functions

koralowiec · 2025-11-22T15:37:08Z

src/bun/install.sh

+detect_libc() {
+    # Method 1: Check for apk (Alpine)
+    if [ -x "/sbin/apk" ]; then
+        log INFO "Detected musl libc (Alpine package manager found)" >&2
+        echo "-musl"
+        return 0
+    fi
+
+    # Method 2: Check ldd output for musl
+    if command -v ldd >/dev/null 2>&1; then
+        if ldd --version 2>&1 | grep -qi musl; then
+            log INFO "Detected musl libc (ldd output)" >&2
+            echo "-musl"
+            return 0
+        fi
+    fi
+
+    # Method 3: Check for musl library files
+    if [ -f "/lib/ld-musl-x86_64.so.1" ] || [ -f "/lib/ld-musl-aarch64.so.1" ]; then
+        log INFO "Detected musl libc (library files)" >&2
+        echo "-musl"
+        return 0
+    fi
+
+    # Method 4: Check /proc/mounts for musl
+    if grep -qi musl /proc/mounts 2>/dev/null; then
+        log INFO "Detected musl libc (proc mounts)" >&2
+        echo "-musl"
+        return 0
+    fi
+
+    # Default to glibc
+    log INFO "Using glibc (default)" >&2
+    echo ""
+}


I think it also makes sense to keep it in the library script, but let's keep it here for now, as it's crucial for the installation

koralowiec · 2025-11-22T15:39:17Z

src/bun/install.sh

+            # Check if bun binary was actually installed
+            if [ -f "/usr/local/bin/bun" ]; then
+                log INFO "Bun binary found at /usr/local/bin/bun"
+                log INFO "Bun binary permissions: $(ls -la /usr/local/bin/bun)"
+            else
+                log ERROR "Bun binary NOT found at /usr/local/bin/bun after installation"
+                log ERROR "Checking /usr/local/bin contents: $(ls -la /usr/local/bin/ 2>/dev/null || echo 'Directory not accessible')"
+
+                # Try to find where bun might have been installed
+                log INFO "Searching for bun binary in common locations..."
+                find /usr/local -name "*bun*" -type f 2>/dev/null || log INFO "No bun binaries found in /usr/local"
+            fi


I think it's redundant check - We actually verify the installation in the tests. I think we can omit it here

koralowiec · 2025-11-22T15:39:56Z

src/bun/install.sh

+# Verify the installation worked
+log INFO "Verifying Bun installation..."
+if [ -x "/usr/local/bin/bun" ]; then
+    log INFO "Bun binary is executable at /usr/local/bin/bun"
+    if bun_binary_version=$(/usr/local/bin/bun --version 2>&1); then
+        log INFO "Bun version check successful: $bun_binary_version"
+    else
+        log INFO "Bun binary exists but version check failed - this may be environment-specific"
+        log INFO "Binary info: $(file /usr/local/bin/bun 2>/dev/null || echo 'file command failed')"
+        log INFO "Library dependencies: $(ldd /usr/local/bin/bun 2>/dev/null || echo 'ldd command failed')"
+
+        # Try to run with explicit library path for debugging
+        if [ -d "/lib" ] && [ -d "/usr/lib" ]; then
+            log INFO "Attempting to run bun with library path for debugging..."
+            if bun_binary_version=$(LD_LIBRARY_PATH=/lib:/usr/lib:/usr/local/lib /usr/local/bin/bun --version 2>&1); then
+                log INFO "Bun version check successful with LD_LIBRARY_PATH: $bun_binary_version"
+            else
+                log INFO "Bun version check still failed with LD_LIBRARY_PATH"
+            fi
+        fi
+    fi
+else
+    log ERROR "Bun binary not found or not executable at /usr/local/bin/bun"
+    log ERROR "Checking if bun exists: $([ -f "/usr/local/bin/bun" ] && echo 'YES' || echo 'NO')"
+    if [ -f "/usr/local/bin/bun" ]; then
+        log ERROR "Bun file exists but is not executable. Permissions: $(ls -la /usr/local/bin/bun)"
+        log ERROR "Attempting to make executable..."
+        chmod +x /usr/local/bin/bun
+        if [ -x "/usr/local/bin/bun" ]; then
+            log INFO "Fixed: Bun binary is now executable"
+        else
+            log ERROR "Failed to make Bun binary executable"
+        fi
+    else
+        log ERROR "Bun binary file does not exist at /usr/local/bin/bun"
+    fi
+fi


Again, let's remove it

myktra · 2026-02-04T20:48:12Z

@trevordcampbell possible to get these changes as advised in so it can be merged up and released? Thankful for your contribution - eager to drop workarounds!

…re Enhancements

Co-authored-by: Cursor <cursoragent@cursor.com>

trevordcampbell · 2026-02-05T03:04:11Z

I just pushed the requested changes to the Bun feature. Hopefully should be ready to ship now. Apologies for the delay

koralowiec self-requested a review November 19, 2025 20:09

koralowiec requested changes Nov 22, 2025

View reviewed changes

trevordcampbell and others added 2 commits February 5, 2026 02:36

fix: Bun Feature Failure on Apple Silicon ARM64 + Comprehensive Featu…

15db101

…re Enhancements

fix: address reviewer feedback and fix Alpine musl detection bug

058d31c

Co-authored-by: Cursor <cursoragent@cursor.com>

trevordcampbell force-pushed the fix-bun-feature branch from 2fd562b to 058d31c Compare February 5, 2026 02:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: Bun Feature Failure on Apple Silicon ARM64 + Feature Hardening#149

fix: Bun Feature Failure on Apple Silicon ARM64 + Feature Hardening#149
trevordcampbell wants to merge 2 commits intodevcontainers-extra:mainfrom
trevordcampbell:fix-bun-feature

trevordcampbell commented Sep 22, 2025

Uh oh!

trevordcampbell commented Nov 17, 2025

Uh oh!

koralowiec commented Nov 19, 2025

Uh oh!

koralowiec left a comment

Uh oh!

koralowiec Nov 22, 2025

Uh oh!

koralowiec Nov 22, 2025

Uh oh!

koralowiec Nov 22, 2025

Uh oh!

koralowiec Nov 22, 2025

Uh oh!

myktra commented Feb 4, 2026

Uh oh!

trevordcampbell commented Feb 5, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

trevordcampbell commented Sep 22, 2025

Problem

Root Cause

Solution

1. arm64 Asset Pattern Fix

2. Comprehensive Feature Enhancements

Robust libc Detection

Enhanced Architecture Support

Better Error Handling & Diagnostics

Improved Logging & User Experience

Performance Optimizations

Version Handling Enhancements

Benefits

For Apple Silicon Users

For All Users

Files Changed

Testing

References

Uh oh!

trevordcampbell commented Nov 17, 2025

Uh oh!

koralowiec commented Nov 19, 2025

Uh oh!

koralowiec left a comment

Choose a reason for hiding this comment

Uh oh!

koralowiec Nov 22, 2025

Choose a reason for hiding this comment

Uh oh!

koralowiec Nov 22, 2025

Choose a reason for hiding this comment

Uh oh!

koralowiec Nov 22, 2025

Choose a reason for hiding this comment

Uh oh!

koralowiec Nov 22, 2025

Choose a reason for hiding this comment

Uh oh!

myktra commented Feb 4, 2026

Uh oh!

trevordcampbell commented Feb 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

1. `arm64` Asset Pattern Fix

trevordcampbell commented Feb 5, 2026 •

edited

Loading