bun.sh

mirror of https://github.com/oven-sh/bun synced 2026-02-16 13:51:47 +00:00

Author	SHA1	Message	Date
Claude Bot	ceb5215177	Update WebKit commit and disable bmalloc linking - Update WebKit commit to 7e9995274c0bd66058770f7768c211459d2e85a5 - Disable bmalloc linking in build configuration as it's now disabled 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-08-01 09:02:59 +00:00
Jarred Sumner	2bba9b9805	Revert "Try revertng "Simplify mimalloc alignment check (#21497 )"" This reverts commit `c560840a5c`.	2025-08-01 00:23:25 -07:00
Jarred Sumner	a464dbb2ec	Update BuildBun.cmake	2025-08-01 00:21:25 -07:00
Jarred Sumner	c560840a5c	Try revertng "Simplify mimalloc alignment check (#21497 )" This reverts commit `6afda5b6a3`.	2025-08-01 00:03:02 -07:00
Jarred Sumner	316ace2d2b	Merge branch 'main' into claude/mimalloc-static-override	2025-07-31 23:29:18 -07:00
Meghan Denny	61e4be8193	node-fallbacks:buffer: fix numberIsNaN ReferenceError (#21527 ) fixes https://github.com/oven-sh/bun/issues/21522	2025-07-31 22:48:08 -07:00
Alistair Smith	8dc6279832	More strictly type bun:sqlite transaction functions (#21495 ) ### What does this PR do? Fixes #21479 ### How did you verify your code works? bun-types test updated	2025-07-31 22:48:08 -07:00
robobun	0f2e476bca	Document static routes vs file routes in HTTP server (#21506 ) ## Summary - Add comprehensive documentation distinguishing static routes from file routes in `Bun.serve()` - Document caching behavior differences: ETag vs Last-Modified - Explain performance characteristics and error handling differences - Provide clear use case recommendations ## Changes - File Responses vs Static Responses section explaining the two approaches - HTTP Caching Behavior section detailing ETag and Last-Modified support - Status Code Handling section covering automatic 204/304 responses - Code examples showing both patterns with clear explanations ## Key Documentation Points - Static routes (`new Response(await file.bytes())`) buffer content in memory at startup - File routes (`new Response(Bun.file(path))`) read from filesystem per request - Static routes use ETags for caching, file routes use Last-Modified headers - File routes handle 404s automatically, static routes cause startup errors for missing files - Both support HTTP caching standards but with different validation strategies 🤖 Generated with [Claude Code](https://claude.ai/code) --------- Co-authored-by: Claude Bot <claude-bot@bun.sh> Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>	2025-07-31 22:48:08 -07:00
Ciro Spaciari	8bc2449d62	fix(postgres) regression (#21466 ) ### What does this PR do? Fix: https://github.com/oven-sh/bun/issues/21351 Relevant changes: Fix advance to properly cleanup success and failed queries that could be still be in the queue Always ref before executing Use stronger atomics for ref/deref and hasPendingActivity Fallback when thisValue is freed/null/zero and check if vm is being shutdown The bug in --hot in `resolveRopeIfNeeded` Issue is not meant to be fixed in this PR this is a fix for the postgres regression Added assertions so this bug is easier to catch on CI ### How did you verify your code works? Test added --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>	2025-07-31 22:48:08 -07:00
pfg	fd45273b5a	Readablestreamdefaultcontroller oxlint fix (#21525 )	2025-07-31 22:48:08 -07:00
Jarred Sumner	4f8adaa6a2	Delete incorrect assertion in ComptimeStringMap (#21504 ) ### What does this PR do? Resolves ```js Bun v1.2.13 ([`64ed68c`](`64ed68c9e0`)) on windows x86_64 [TestCommand] panic: ComptimeStringMap.fromJS: input is not a string [comptime_string_map.zig:268](`64ed68c9e0/src/comptime_string_map.zig (L268)`): getWithEql [Response.zig:682](`64ed68c9e0/src/bun.js/webcore/Response.zig (L682)`): init [Request.zig:679](`64ed68c9e0/src/bun.js/webcore/Request.zig (L679)`): constructInto [ZigGeneratedClasses.cpp:37976](`64ed68c9e0/C`:/buildkite-agent/builds/EC2AMAZ-Q4V5GV4/bun/bun/build/release/codegen/ZigGeneratedClasses.cpp#L37976): WebCore::JSRequestConstructor::construct 2 unknown/js code llint_entry Features: tsconfig, Bun.stdout, dotenv, jsc ``` ### How did you verify your code works? There is a test.	2025-07-31 22:48:07 -07:00
pfg	069550e158	Fix "test failing but passed" arrow pointing to the wrong test (#21502 ) Before: ``` failing-test-passes.fixture.ts: ^ this test is marked as failing but it passed. Remove \`.failing\` if tested behavior now works (fail) This should fail but it doesnt [0.24ms] ^ this test is marked as failing but it passed. Remove \`.failing\` if tested behavior now works (fail) This should fail but it doesnt (async) [0.23ms] ``` After: ``` failing-test-passes.fixture.ts: (fail) This should fail but it doesnt [0.24ms] ^ this test is marked as failing but it passed. Remove \`.failing\` if tested behavior now works (fail) This should fail but it doesnt (async) [0.23ms] ^ this test is marked as failing but it passed. Remove \`.failing\` if tested behavior now works ``` Adds a snapshot test --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>	2025-07-31 22:48:07 -07:00
Jarred Sumner	8d1f69650c	Fixes "Stream is already ended" error when cancelling a request (#21481 ) ### What does this PR do? if you spam the refresh button in `next dev`, we print this error: ``` ⨯ Error: Stream is already ended at writeHead (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at processTicksAndRejections (null) { digest: '2259044225', code: 'ERR_STREAM_ALREADY_FINISHED', toString: [Function: toString] } ⨯ Error: failed to pipe response at processTicksAndRejections (unknown:7:39) { [cause]: Error: Stream is already ended at writeHead (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at processTicksAndRejections (null) { digest: '2259044225', code: 'ERR_STREAM_ALREADY_FINISHED', toString: [Function: toString] } } ⨯ Error: failed to pipe response at processTicksAndRejections (unknown:7:39) { page: '/', [cause]: Error: Stream is already ended at writeHead (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at <anonymous> (null) at processTicksAndRejections (null) { digest: '2259044225', code: 'ERR_STREAM_ALREADY_FINISHED', toString: [Function: toString] } } ``` If the socket is already closed when writeHead is called, we're supposed to silently ignore it instead of throwing an error . The close event is supposed to be emitted on the next tick. Now, I think there are also cases where we do not emit the close event which is similarly bad. ### How did you verify your code works? Need to go through the node http server tests and see if any new ones pass. Also maybe some will fail on this PR, let's see.	2025-07-31 22:48:07 -07:00
Zack Radisic	0ce427f073	Fix assertion failure on Windows in resolver (#21510 ) ### What does this PR do? We had `bun.strings.assertIsValidWindowsPath(...)` in the resolver, but we can't do this because the path may come from the user. Instead, let our error handling code handle it. Also fixes #21065 --------- Co-authored-by: Jarred Sumner <jarred@jarredsumner.com>	2025-07-31 22:48:07 -07:00
Meghan Denny	8e4b5d51b3	test: handle docker exiting with a signal (#21512 )	2025-07-31 22:48:07 -07:00
fuyou	5f010c65d8	fix(mock): add support for rejected values in JSMockFunction (#21489 )	2025-07-31 22:48:07 -07:00
Zack Radisic	2d6ce0ee61	Fix lldb pretty printing for `bun.collections.MultiArrayList(...)` (#21499 ) ### What does this PR do? We have our own `MultiArrayList(...)` in `src/collections/multi_array_list.zig` (is this really necessary?) and this does not work with the existing lldb pretty printing functions because they are under a different symbol name: `collections.multi_array_list.MultiArrayList` instead of `multi_array_list.MultiArrayList`	2025-07-31 22:48:07 -07:00
taylor.fish	c6da7496d8	Resync `MultiArrayList` with Zig standard library (#21500 ) (For internal tracking: fixes STAB-912)	2025-07-31 22:48:07 -07:00
Dylan Conway	b8f5ba950e	sync webkit (#21436 ) ### What does this PR do? <!-- Please explain what your changes do, example: --> <!-- This adds a new flag --bail to bun test. When set, it will stop running tests after the first failure. This is useful for CI environments where you want to fail fast. --> ### How did you verify your code works? ran fuzzy-wuzzy.test.ts <!-- For code changes, please include automated tests. Feel free to uncomment the line below --> <!-- I wrote automated tests --> <!-- If JavaScript/TypeScript modules or builtins changed: - [ ] I included a test for the new code, or existing tests cover it - [ ] I ran my tests locally and they pass (`bun-debug test test-file-name.test`) --> <!-- If Zig files changed: - [ ] I checked the lifetime of memory allocated to verify it's (1) freed and (2) only freed when it should be - [ ] I included a test for the new code, or an existing test covers it - [ ] JSValue used outside of the stack is either wrapped in a JSC.Strong or is JSValueProtect'ed - [ ] I wrote TypeScript/JavaScript tests and they pass locally (`bun-debug test test-file-name.test`) --> <!-- If new methods, getters, or setters were added to a publicly exposed class: - [ ] I added TypeScript types for the new methods, getters, or setters --> <!-- If dependencies in tests changed: - [ ] I made sure that specific versions of dependencies are used instead of ranged or tagged versions --> <!-- If a new builtin ESM/CJS module was added: - [ ] I updated Aliases in `module_loader.zig` to include the new module - [ ] I added a test that imports the module - [ ] I added a test that require() the module -->	2025-07-31 22:48:06 -07:00
taylor.fish	6afda5b6a3	Simplify mimalloc alignment check (#21497 ) This slightly reduces memory use. Maximum memory use of `bun test html-rewriter`, averaged across 100 iterations: * 101975 kB without this change * 101634 kB with this change I also tried changing the code to always use the aligned allocation functions, but this slightly increased memory use, to 102160 kB. (For internal tracking: fixes ENG-19866)	2025-07-31 22:48:06 -07:00
taylor.fish	f54d315e9e	Add helper type to detect unsynchronized concurrent accesses of shared data (#21476 ) Add a helper type to help detect race conditions. There's no performance or memory use penalty in release builds. Actually adding the type to various places will be left for future PRs. (For internal tracking: fixes STAB-852)	2025-07-31 22:48:06 -07:00
Kai Tamkun	df10aad29d	Replace allocator isNull() check with an assertion in String.toThreadSafeSlice (#21474 ) ### What does this PR do? Replaces an if statement with an assertion that the condition is false. The allocator in question should never be null. ### How did you verify your code works?	2025-07-31 22:48:06 -07:00
Jarred Sumner	3e456d4bb0	Disable `findSourceMap` when `--enable-source-maps` is not passed (#21478 ) ### What does this PR do? Fixes the error printed: ```js ❯ bun --bun dev $ next dev --turbopack ▲ Next.js 15.4.5 (Turbopack) - Local: http://localhost:3000 - Network: http://192.168.1.250:3000 ✓ Starting... ✓ Ready in 637ms ○ Compiling / ... ✓ Compiled / in 1280ms /private/tmp/empty/my-app/.next/server/chunks/ssr/[root-of-the-server]__012ba519._.js: Invalid source map. Only conformant source maps can be used to filter stack frames. Cause: TypeError: payload is not an Object. (evaluating '"sections" in payload') /private/tmp/empty/my-app/.next/server/chunks/ssr/[root-of-the-server]__93bf7db5._.js: Invalid source map. Only conformant source maps can be used to filter stack frames. Cause: TypeError: payload is not an Object. (evaluating '"sections" in payload') GET / 200 in 1416ms ^C^[[A ``` ### How did you verify your code works?	2025-07-31 22:48:06 -07:00
Meghan Denny	6ca1f7cd48	test: split napi tests into separate files (#21475 ) Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>	2025-07-31 22:48:06 -07:00
Zack Radisic	6a1615821e	Add asan checks to `HiveArray` (#21449 )	2025-07-31 22:48:05 -07:00
Jarred Sumner	a8921537aa	Try mimalloc v3 (#17378 ) (For internal tracking: fixes ENG-19852) --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: Jarred-Sumner <709451+Jarred-Sumner@users.noreply.github.com> Co-authored-by: Kai Tamkun <kai@tamkun.io> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: taylor.fish <contact@taylor.fish>	2025-07-31 22:48:05 -07:00
pfg	411d1afa2a	Fix docs in test.md (#21472 )	2025-07-31 22:48:05 -07:00
Ciro Spaciari	e350e90822	fix(net/http2) fix socket internal timeout and owner_symbol check, fix padding support in http2 (#21263 ) ### What does this PR do? <!-- Please explain what your changes do, example: --> <!-- This adds a new flag --bail to bun test. When set, it will stop running tests after the first failure. This is useful for CI environments where you want to fail fast. --> - [ ] Documentation or TypeScript types (it's okay to leave the rest blank in this case) - [x] Code changes ### How did you verify your code works? Tests added for padding support Timeout of socket is being fired earlier due to backpressure or lack of precision in usockets timers (now matchs node.js behavior). Added check for owner_symbol so the error showed in https://github.com/oven-sh/bun/issues/21055 is handled <!-- For code changes, please include automated tests. Feel free to uncomment the line below --> <!-- I wrote automated tests --> <!-- If JavaScript/TypeScript modules or builtins changed: - [ ] I included a test for the new code, or existing tests cover it - [ ] I ran my tests locally and they pass (`bun-debug test test-file-name.test`) --> <!-- If Zig files changed: - [ ] I checked the lifetime of memory allocated to verify it's (1) freed and (2) only freed when it should be - [ ] I included a test for the new code, or an existing test covers it - [ ] JSValue used outside of the stack is either wrapped in a JSC.Strong or is JSValueProtect'ed - [ ] I wrote TypeScript/JavaScript tests and they pass locally (`bun-debug test test-file-name.test`) --> <!-- If new methods, getters, or setters were added to a publicly exposed class: - [ ] I added TypeScript types for the new methods, getters, or setters --> <!-- If dependencies in tests changed: - [ ] I made sure that specific versions of dependencies are used instead of ranged or tagged versions --> <!-- If a new builtin ESM/CJS module was added: - [ ] I updated Aliases in `module_loader.zig` to include the new module - [ ] I added a test that imports the module - [ ] I added a test that require() the module --> --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>	2025-07-31 22:48:05 -07:00
Jarred Sumner	8ddf2a10b8	Ensure we handle aborted requests correctly in Rendering API (#21398 ) ### What does this PR do? We have to use the existing code for handling aborted requests instead of immediately calling deinit. Also made the underlying uws.Response an optional pointer to mark when the request has already been aborted to make it clear it's no longer accessible. ### How did you verify your code works? This needs a test --------- Co-authored-by: Zack Radisic <56137411+zackradisic@users.noreply.github.com>	2025-07-31 22:48:05 -07:00
Jarred Sumner	44ed5d2d6f	Introduce yarn.lock -> bun.lock{b} migrator (#16166 ) ### What does this PR do? fixes #6409 This PR implements `bun install` automatic migration from yarn.lock files to bun.lock, preserving versions exactly. The migration happens automatically when: 1. A project has a `yarn.lock` file 2. No `bun.lock` or `bun.lockb` file exists 3. User runs `bun install` ### Current Status: ✅ Complete and Working The yarn.lock migration feature is fully functional and comprehensively tested. All dependency types are supported: - ✅ Regular npm dependencies (`package@^1.0.0`) - ✅ Git dependencies (`git+https://github.com/user/repo.git`, `github:user/repo`) - ✅ NPM alias dependencies (`alias@npm:package@version`) - ✅ File dependencies (`file:./path`) - ✅ Remote tarball URLs (`https://registry.npmjs.org/package.tgz`) - ✅ Local tarball files (`file:package.tgz`) ### Test Results ```bash $ bun bd test test/cli/install/migration/yarn-lock-migration.test.ts ✅ 4 pass, 0 fail - yarn-lock-mkdirp (basic npm dependency) - yarn-lock-mkdirp-no-resolved (npm dependency without resolved field) - yarn-lock-mkdirp-file-dep (file dependency) - yarn-stuff (all complex dependency types: git, npm aliases, file, remote tarballs) ``` ### How did you verify your code works? 1. Comprehensive test suite: Added 4 test cases covering all dependency types 2. Version preservation: Verified that package versions are preserved exactly during migration 3. Real-world scenarios: Tested with complex yarn.lock files containing git deps, npm aliases, file deps, and remote tarballs 4. Migration logging: Confirms migration with log message `[X.XXms] migrated lockfile from yarn.lock` ### Key Implementation Details - Core parser: `src/install/yarn.zig` handles all yarn.lock parsing and dependency type resolution - Integration: Migration is built into existing lockfile loading infrastructure - Performance: Migration typically completes in ~1ms for most projects - Compatibility: Preserves exact dependency versions and resolution behavior The implementation correctly handles edge cases like npm aliases, git dependencies with commits, file dependencies with transitive deps, and remote tarballs. --------- Co-authored-by: Jarred-Sumner <709451+Jarred-Sumner@users.noreply.github.com> Co-authored-by: Claude Bot <claude-bot@bun.sh> Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: RiskyMH <git@riskymh.dev> Co-authored-by: RiskyMH <56214343+RiskyMH@users.noreply.github.com>	2025-07-31 22:48:05 -07:00
Jarred Sumner	2f35945012	Update BuildMimalloc.cmake	2025-07-31 22:31:22 -07:00
autofix-ci[bot]	b03a6344d8	[autofix.ci] apply automated fixes	2025-07-30 06:17:18 +00:00
Claude Bot	8b2e19391c	Add WebSocket Blob support - Uncommented and implemented blob sending in JSWebSocket.cpp - Added blob handling for send(), ping(), and pong() methods - Implemented blob data access functions in Blob.zig - Added writeBlob function to WebSocket Zig client - Added comprehensive tests for WebSocket blob functionality - Enabled blob binaryType support in WebSocket 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-07-30 06:14:01 +00:00

3 changed files with 22 additions and 9 deletions

									
										9

cmake/targets/BuildBun.cmake
									
												View File
												
				@@ -45,6 +45,7 @@ endif()

				# --- Dependencies ---

				set(BUN_DEPENDENCIES

				  Mimalloc # must be first

				  BoringSSL

				  Brotli

				  Cares

				@@ -52,7 +53,6 @@ set(BUN_DEPENDENCIES

				  LibDeflate

				  LolHtml

				  Lshpack

				  Mimalloc

				  TinyCC

				  Zlib

				  LibArchive # must be loaded after zlib

				@@ -1112,9 +1112,10 @@ else()

				    ${WEBKIT_LIB_PATH}/libWTF.a

				    ${WEBKIT_LIB_PATH}/libJavaScriptCore.a

				  )

				  if(NOT APPLE OR EXISTS ${WEBKIT_LIB_PATH}/libbmalloc.a)

				    target_link_libraries(${bun} PRIVATE ${WEBKIT_LIB_PATH}/libbmalloc.a)

				  endif()

				  # bmalloc is now disabled

				  # if(NOT APPLE OR EXISTS ${WEBKIT_LIB_PATH}/libbmalloc.a)

				  #   target_link_libraries(${bun} PRIVATE ${WEBKIT_LIB_PATH}/libbmalloc.a)

				  # endif()

				endif()

				include_directories(${WEBKIT_INCLUDE_PATH})

									
										20

cmake/targets/BuildMimalloc.cmake
									
												View File
												
				@@ -4,7 +4,7 @@ register_repository(

				  REPOSITORY

				    oven-sh/mimalloc

				  COMMIT

				    c1f17cd2538417620f60bff70bffe7e68d332aec

				    d0b7c26cdf7bb4104d7d64c7dd05e8f0d5a7d9d4

				)

				set(MIMALLOC_CMAKE_ARGS

				@@ -13,14 +13,26 @@ set(MIMALLOC_CMAKE_ARGS

				  -DMI_BUILD_SHARED=OFF

				  -DMI_BUILD_TESTS=OFF

				  -DMI_USE_CXX=ON

				  -DMI_OVERRIDE=OFF

				  -DMI_OSX_ZONE=OFF

				  -DMI_OSX_INTERPOSE=OFF

				  -DMI_SKIP_COLLECT_ON_EXIT=ON

				)

				if(ENABLE_ASAN)

				  list(APPEND MIMALLOC_CMAKE_ARGS -DMI_TRACK_ASAN=ON)

				  list(APPEND MIMALLOC_CMAKE_ARGS -DMI_OVERRIDE=OFF)

				  list(APPEND MIMALLOC_CMAKE_ARGS -DMI_OSX_ZONE=OFF)

				  list(APPEND MIMALLOC_CMAKE_ARGS -DMI_OSX_INTERPOSE=OFF)

				else()

				  if (APPLE OR LINUX)

				    # Enable static override when ASAN is not enabled

				    list(APPEND MIMALLOC_CMAKE_ARGS -DMI_OVERRIDE=ON)

				    if(APPLE)

				      list(APPEND MIMALLOC_CMAKE_ARGS -DMI_OSX_ZONE=ON)

				      list(APPEND MIMALLOC_CMAKE_ARGS -DMI_OSX_INTERPOSE=ON)

				    else()

				      list(APPEND MIMALLOC_CMAKE_ARGS -DMI_OSX_ZONE=OFF)

				      list(APPEND MIMALLOC_CMAKE_ARGS -DMI_OSX_INTERPOSE=OFF)

				    endif()

				  endif()

				endif()

				if(DEBUG)

									
										2

cmake/tools/SetupWebKit.cmake
									
												View File
												
				@@ -2,7 +2,7 @@ option(WEBKIT_VERSION "The version of WebKit to use")

				option(WEBKIT_LOCAL "If a local version of WebKit should be used instead of downloading")

				if(NOT WEBKIT_VERSION)

				  set(WEBKIT_VERSION 642e2252f6298387edb6d2f991a0408fd0320466)

				  set(WEBKIT_VERSION 7e9995274c0bd66058770f7768c211459d2e85a5)

				endif()

				string(SUBSTRING ${WEBKIT_VERSION} 0 16 WEBKIT_VERSION_PREFIX)

Compare commits

33 Commits

claude/fix ... claude/mim

9

cmake/targets/BuildBun.cmake

View File

20

cmake/targets/BuildMimalloc.cmake

View File

2

cmake/tools/SetupWebKit.cmake

View File

Compare commits

33 Commits claude/fix ... claude/mim

9 cmake/targets/BuildBun.cmake Unescape Escape View File

20 cmake/targets/BuildMimalloc.cmake Unescape Escape View File

2 cmake/tools/SetupWebKit.cmake Unescape Escape View File

33 Commits

claude/fix ... claude/mim

9

cmake/targets/BuildBun.cmake

View File

20

cmake/targets/BuildMimalloc.cmake

View File

2

cmake/tools/SetupWebKit.cmake

View File