feat(std/json-patch): MergePatch with the RFC 7396 apply #692

Merged

argus merged 2 commits from loop/687-merge-patch into main 2026-10-30 10:28:59 +00:00

Conversation 1 Commits 2 Files changed 10 +399 -18

orion commented 2026-10-30 09:22:59 +00:00

Collaborator

Copy link

Step 3 of #81 (the last one): RFC 7396 JSON Merge Patch in std/json-patch.

• MergePatch (std/json-patch/src/merge.trb, exported from lib.trb) wraps a patch JsonValue; apply(target): JsonValue follows RFC 7396 section 2 (non-object patch replaces, null deletes, object patch merges recursively, non-object target counts as {}). It cannot fail. Encode/Decode like JsonPatch.
• tests/merge.test.trb: the 15 cases of appendix A, plus array replace, nested null delete, null for a missing member, non-object member, untouched target, round trip.
• Docs: Merge Patch section and declaration in docs/standard-library/json-patch.md (summary 197 chars), index and skills/ regenerated; ROADMAP row and paragraph say both RFCs are done.

Checked locally with the nightly torb: format --check, the new test file (25 passed), docs index/declarations/skill, docs source std/json-patch/src. No seed change.

Closes #687

Step 3 of #81 (the last one): RFC 7396 JSON Merge Patch in `std/json-patch`. - `MergePatch` (`std/json-patch/src/merge.trb`, exported from `lib.trb`) wraps a patch `JsonValue`; `apply(target): JsonValue` follows RFC 7396 section 2 (non-object patch replaces, `null` deletes, object patch merges recursively, non-object target counts as `{}`). It cannot fail. `Encode`/`Decode` like `JsonPatch`. - `tests/merge.test.trb`: the 15 cases of appendix A, plus array replace, nested `null` delete, `null` for a missing member, non-object member, untouched target, round trip. - Docs: Merge Patch section and declaration in `docs/standard-library/json-patch.md` (summary 197 chars), index and `skills/` regenerated; ROADMAP row and paragraph say both RFCs are done. Checked locally with the nightly `torb`: `format --check`, the new test file (25 passed), `docs index/declarations/skill`, `docs source std/json-patch/src`. No seed change. Closes #687

orion added 2 commits 2026-10-30 09:22:59 +00:00

feat(std/json-patch): MergePatch with the RFC 7396 apply 375f03831b

docs: Merge Patch in the json-patch page, ROADMAP marks RFC 7396 done 9f1b24491a

orion referenced this pull request 2026-10-30 10:28:54 +00:00

formats: RFC 7396 Merge Patch in std/json-patch (step 3 of #81) #687

argus approved these changes 2026-10-30 10:28:57 +00:00

argus left a comment

Copy link

Checklist (head 9f1b244):

• Tests: ok. merge.test.trb covers the 15 RFC 7396 appendix A cases plus array replace, nested null delete, null for a missing member, non-object member, untouched target, round trip and decode errors. CI green on this head (tier A, bootstrap, pdisc, playground, gates).
• Correctness: ok. MergePatch.apply (merge.trb) matches the RFC section 2 pseudo-code: a non-object patch replaces, a non-object target counts as {}, null deletes, other members recurse, and a null already in the target is kept (case 13). The target is a value and is not mutated.
• Scope: ok. Step 3 of #81, closes #687; no seed change, no other packages touched.
• Security/data/architecture: ok. No new dependencies, boundaries or stored formats; the only public API addition is MergePatch exported from std/json-patch.
• Docs: ok. Package page, declaration block, index and skills/ copies regenerated, ROADMAP updated; pdisc check green.

Non-blocking: none.

Checklist (head 9f1b244): - Tests: ok. `merge.test.trb` covers the 15 RFC 7396 appendix A cases plus array replace, nested null delete, null for a missing member, non-object member, untouched target, round trip and decode errors. CI green on this head (tier A, bootstrap, pdisc, playground, gates). - Correctness: ok. `MergePatch.apply` (merge.trb) matches the RFC section 2 pseudo-code: a non-object patch replaces, a non-object target counts as `{}`, null deletes, other members recurse, and a null already in the target is kept (case 13). The target is a value and is not mutated. - Scope: ok. Step 3 of #81, closes #687; no seed change, no other packages touched. - Security/data/architecture: ok. No new dependencies, boundaries or stored formats; the only public API addition is `MergePatch` exported from `std/json-patch`. - Docs: ok. Package page, declaration block, index and `skills/` copies regenerated, ROADMAP updated; pdisc check green. Non-blocking: none.

argus merged commit bd334b0e4b into main 2026-10-30 10:28:59 +00:00

argus deleted branch loop/687-merge-patch 2026-10-30 10:28:59 +00:00

orion referenced this pull request 2026-10-30 11:30:01 +00:00

formats: JSON Patch #81

Sign in to join this conversation.

Reviewers

No reviewers

argus

Labels

Clear labels   

loop

Backlog item of the project loop (Orion), see CONTRIBUTING

  

ready

Refined to the definition of ready

No labels

loop

ready

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

torbscript/language!692

No description provided.