fix: guard Buffer.from island arguments before bytesNew - #91
Conversation
|
@yearth is attempting to deploy a commit to the Vercel Labs Team on Vercel. A member of the Team first needs to authorize it. |
a3dd28e to
c09bdb5
Compare
|
Refreshed this PR onto current Validation on the refreshed commit (
|
|
@ctate Hi! Just a gentle follow-up on this PR when you have a chance. I refreshed it onto the current Happy to adjust anything if you would prefer a different direction. Thanks for your time! |
Fixes #35.
Summary
bytesNewforBuffer.fromUint8Arrayandnumber[]valuesThis preserves the currently supported
Buffer.fromsurface; it only prevents an invalidjsvalsource from reachingbytesNew.Testing
pnpm -r buildpnpm lintpnpm exec vitest run tests/harness/diagnostics.test.ts(101 passed)pnpm exec vitest run packages/compiler/test/ts7/order-parity.test.ts(23 passed)SCRIPTC_TEST_WORKERS=4 pnpm test(3348 passed, 54 skipped)SCRIPTC_TEST_WORKERS=4 SCRIPTC_SAN=1 pnpm test(3345 passed, 54 skipped, 3 parallel timeouts; all three passed in a single-worker targeted rerun)