Repository navigation
Print every native function's source on one line - #545
Merged
Jarred-Sumner merged 1 commit intoSep 1, 2026
Merged
Claude / Claude Code Review
completed
Sep 1, 2026 in 24m 4s
Code review found 1 important issue
Found 4 candidates, confirmed 2. See review comments for details.
Details
| Severity | Count |
|---|---|
| 🔴 Important | 1 |
| 🟡 Nit | 1 |
| 🟣 Pre-existing | 0 |
| Severity | File:Line | Issue |
|---|---|---|
| 🔴 Important | Source/JavaScriptCore/runtime/NativeExecutable.cpp:111 |
Switching host-function toString to one line breaks several enabled ChakraCore baseline tests that pass on the base bran |
| 🟡 Nit | Source/JavaScriptCore/inspector/InjectedScriptSource.js:809 |
The injected-script native sniff was updated to '"[native code] }"' but the frontend counterpart 'isFunctionStringNative |
Loading