Skip to content

Add atomic path-based modification API and JSON patch support#69

Merged
IvanMurzak merged 3 commits into
feature/atomic-apifrom
feature/atomic-modify
Mar 3, 2026
Merged

Add atomic path-based modification API and JSON patch support#69
IvanMurzak merged 3 commits into
feature/atomic-apifrom
feature/atomic-modify

Conversation

@IvanMurzak
Copy link
Copy Markdown
Owner

  • Introduced TryModifyAt method for direct modification of fields, array elements, and dictionary entries without constructing full objects.
  • Implemented path parsing logic to navigate to specific members using a string-based path format.
  • Added TryPatch method to apply JSON Merge Patch documents, allowing multiple fields to be modified in a single call.
  • Created new partial files: Reflector.ModifyAt.cs for atomic modifications and Reflector.Patch.cs for JSON patching.
  • Enhanced error handling with detailed messages for navigation failures.
  • Added tests to verify the functionality of the new modification methods and JSON patching capabilities.

- Introduced `TryModifyAt` method for direct modification of fields, array elements, and dictionary entries without constructing full objects.
- Implemented path parsing logic to navigate to specific members using a string-based path format.
- Added `TryPatch` method to apply JSON Merge Patch documents, allowing multiple fields to be modified in a single call.
- Created new partial files: `Reflector.ModifyAt.cs` for atomic modifications and `Reflector.Patch.cs` for JSON patching.
- Enhanced error handling with detailed messages for navigation failures.
- Added tests to verify the functionality of the new modification methods and JSON patching capabilities.
@IvanMurzak IvanMurzak self-assigned this Mar 3, 2026
@IvanMurzak IvanMurzak added the enhancement New feature or request label Mar 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 3, 2026

Test Results

    2 files      2 suites   8m 46s ⏱️
1 378 tests 1 378 ✅ 0 💤 0 ❌
2 756 runs  2 756 ✅ 0 💤 0 ❌

Results for commit 25b2617.

♻️ This comment has been updated with latest results.

@IvanMurzak IvanMurzak changed the base branch from main to feature/atomic-api March 3, 2026 10:58
@IvanMurzak IvanMurzak merged commit 0cefb4a into feature/atomic-api Mar 3, 2026
2 checks passed
@IvanMurzak IvanMurzak deleted the feature/atomic-modify branch March 3, 2026 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant