Conversation
- stringify(program, leafSerializer) accepts custom leaf serialization - parse(program, leafDeserializer) accepts custom leaf deserialization - leaf values wrapped as ['leaf', serializedString] nodes - supports Date, RegExp, BigInt, Set, Map, and undefined via superjson - preserves undefined in objects using internal sentinel mechanism
- simplify defaultLeafDeserializer to remove unreachable branch - add test for default deserializer when not provided - add test for fallback to default deserializer when null passed - add test to exercise defensive undefined parameter handling - achieves 100% branch coverage
- remove unused Symbol('replaced') that was vestigial from replacer pattern
- move awaitId tracking into state object passed to transformValue
- reset awaitId per stringify() call for proper isolation
- simplifies code path and removes global mutable state per-call
- JSON.parse reviver can only receive null (not undefined) - replace isNil(value) with direct null check - remove unused is-nil import
- document transformValue behavior for stringify - document resolveAwaitedValues behavior for parse - document restoreUndefined purpose - improve code readability through strategic comments
document the leaf (de)serializer API: - explain default JSON limitation - show how to extend with superjson - provide complete example with Date, RegExp, BigInt, Map, Set - clarify leaf wrapper format for safety
- add test to verify ['leaf', 1, 2, 3] is properly quoted and unquoted - fix: add 'leaf' to keywords list so it's quoted when appearing as array data - ensures leaf marker is not confused with actual leaf data
…e coverage - Added getTypeInfo() helper for better error messages - Improved assertion messages with type information - Added property test for type distinctness (different primitives have different serializations) - Added property test for serialization stability (idempotence) - Better error handling and reporting in all property tests - Now running 6 property tests with comprehensive coverage - All 60 tests passing
…and type coverage
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.