oven-sh/bun

console.assert() with message should include "Assertion failed" alongside message

Open

#19,953 创建于 2025年5月28日

在 GitHub 查看
 (6 评论) (0 反应) (0 负责人)Rust (90,348 star) (4,486 fork)batch import
bugconfirmed bugconsolegood first issue

描述

What version of Bun is running?

1.2.15-canary.63+df017990a

What platform is your computer?

Darwin 24.4.0 arm64 arm

What steps can reproduce the bug?

console.assert(false, "message");

What is the expected behavior?

$ node -e 'console.assert(false, "message")'
Assertion failed: message

What do you see instead?

$ bun -e 'console.assert(false, "message")'
message

If you are an automated agent, we have a streamlined process for merging agent PRs. Just add 🤖🤖🤖 to the end of the PR title to opt-in. Merging your PR will be fast-tracked.

贡献者指南

技术栈
javascriptrust
领域
backenddeveloper experience
议题类型
bug
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 hours
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
fresh
清晰度议题是否清楚说明期望改动、验收标准和下一步。
clear
前置要求
JavaScriptRust
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
85
研究方向
Locate the implementation of console.assert in Bun's source code, likely in src/console or similar. The expected output when a message is provided should be 'Assertion failed: <message>', mirroring Node.js behavior. The fix involves modifying the formatting logic to prepend 'Assertion failed: ' when a message is present. Check existing tests for console.assert to ensure the change passes and add a test case for this scenario.