solidjs / solid / 33844098086

72%

main: 89%

push

github

fix(web): walk an Error carrier's non-enumerable own data slots (#3268)

The #3235 guard walked Error carriers with Object.keys while seroval
encodes an Error's own properties through getOwnPropertyNames — so a
failure channel parked on a non-enumerable data slot (cause, the
ordinary home of a wrapped driver error's context, is non-enumerable by
spec) was encoded without being walked: unsanitized on a committed 200,
its rejection unowned. The guard now descends the Error's own
string-keyed data slots, enumerable or not. Hidden accessors remain the
codec's read per 47995412's pinned ruling — the walk still does not
invoke what the author hid.

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

515 of 783 branches covered (65.77%)

Branch coverage included in aggregate %.

764 of 998 relevant lines covered (76.55%)

14.95 hits per line