Skip to content

Fix Bun.inspect null deref with Proxy prototypes; stop reporting from inside the sql lazy-property builders - #30245

Closed
robobun wants to merge 23 commits into
mainfrom
farm/8928bc04/fix-sql-lazy-init-null-deref
Closed

Fix Bun.inspect null deref with Proxy prototypes; stop reporting from inside the sql lazy-property builders#30245
robobun wants to merge 23 commits into
mainfrom
farm/8928bc04/fix-sql-lazy-init-null-deref

Merge origin/main into farm/8928bc04/fix-sql-lazy-init-null-deref

43f57e5
Select commit
Loading
Failed to load commit list.
Claude / Claude Code Review completed Jul 12, 2026 in 27m 17s

Code review found 1 potential issue

Found 1 candidates, confirmed 1. See review comments for details.

Details

Severity Count
🔴 Important 0
🟡 Nit 1
🟣 Pre-existing 0
Severity File:Line Issue
🟡 Nit test/regression/issue/19650.test.ts:4-8 Regression test missing standard issue-URL comment

Annotations

Check warning on line 8 in test/regression/issue/19650.test.ts

See this annotation in the file changed.

@claude claude / Claude Code Review

Regression test missing standard issue-URL comment

Consider prepending `// https://github.com/oven-sh/bun/issues/19650` at the top of this file — regression tests under `test/regression/issue/` conventionally start with the issue URL so future readers can find the original report. Not blocking.